Prompting users to reload vscode window for proper manifest updates#811
Prompting users to reload vscode window for proper manifest updates#811
Conversation
|
Coverage Impact This PR will not change total coverage. Modified Files with Diff Coverage (1)
🛟 Help
|
There was a problem hiding this comment.
Pull request overview
This PR adds additional logging around Component Viewer Tree View registration and aligns its initialization pattern with Live Watch by instantiating and injecting the tree data provider from the extension activation flow.
Changes:
- Inject
ComponentViewerTreeDataProviderintoComponentViewervia constructor (instead of instantiating internally). - Add log statements around Component Viewer activation and tree/command registration.
- Update unit tests to construct
ComponentViewerwith a provider helper.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| src/views/component-viewer/test/unit/component-viewer-main.test.ts | Updates tests to use a shared createController(...) helper that injects a tree provider. |
| src/views/component-viewer/component-viewer-main.ts | Changes ComponentViewer to require an injected tree provider and adds logs during activation/registration. |
| src/desktop/extension.ts | Creates and wires a ComponentViewerTreeDataProvider during activation; clears provider on deactivate; adds activation logs. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…r view is not registered

Fixes
No view is registered with id: cmsis-debugger.componentViewer#812Changes
Screenshots
Checklist