Reduce console noise

[allancascante]

Description

objectExplorerService methods that rely on nodes to be loaded were being called before the service state was initialized, this led to the errors, added logic to initialize the class state from main controller to avoid calling objectExplorerService methods on invalid state. Now it will load the root connections and nodes even if the Object Explorer from the UI hasn't been loaded.

Code Changes Checklist

• New or updated unit tests added
• All existing tests pass (npm run test)
• Code follows contributing guidelines
• Telemetry/logging updated if relevant
• No regressions or UX breakage

Reviewers: Please read our reviewer guidelines

[github-actions]

PR Changes

Category	Target Branch	PR Branch	Difference
Code Coverage	56.07%	56.07%	⚪ 0.00%
VSIX Size	4816 KB	4814 KB	⚪ -2 KB ( 0% )
Webview Bundle Size	3948 KB	3940 KB	⚪ -8 KB ( 0% )

[Benjin]

Should this be getting called in the constructor? Take a look at how connectionConfig.ts handles this:

public initialized: Deferred<void> = new Deferred<void>();


constructor() {
...
void initialize();
}

private async initialize(): Promise<void> {
//... perform async init tasks, then:
this.initialized.resolve();
}

[allancascante]

Should this be getting called in the constructor? Take a look at how connectionConfig.ts handles this:

public initialized: Deferred<void> = new Deferred<void>();


constructor() {
...
void initialize();
}

private async initialize(): Promise<void> {
//... perform async init tasks, then:
this.initialized.resolve();
}

that is a possibility, but wouldn't that affect plugin load time if the user has too many 'root nodes'?