Populate CMSIS view for image-only solutions#86
Conversation
Add board and device nodes
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
There was a problem hiding this comment.
Pull request overview
This PR fixes CMSIS-View population for “image-only” solutions by ensuring hardware (board/device) nodes are created even when no projects are present, and adds test fixtures + a unit test to validate the behavior.
Changes:
- Build the hardware subtree even when a
cbuildhas no associatedcproject. - Add ImageOnly solution fixtures (single-core + multicore) and a reference tree dump.
- Minor readability refactors via variable renames in outline item builders.
Reviewed changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| test-data/solutions/ImageOnly/image-only.csolution.yml | Adds a single-core image-only csolution fixture. |
| test-data/solutions/ImageOnly/image-only.cbuild-pack.yml | Adds resolved pack data for the single-core fixture. |
| test-data/solutions/ImageOnly/image-only.cbuild-idx.yml | Adds build index metadata for the single-core fixture. |
| test-data/solutions/ImageOnly/image-only-multicore.csolution.yml | Adds a multicore image-only csolution fixture used by the new unit test. |
| test-data/solutions/ImageOnly/image-only-multicore.cbuild-pack.yml | Adds resolved pack data for the multicore fixture. |
| test-data/solutions/ImageOnly/image-only-multicore.cbuild-idx.yml | Adds build index metadata for the multicore fixture. |
| test-data/solutions/ImageOnly/CmsisViewTreeMulticoreRef.txt | Reference dump to assert expected CMSIS-View tree output for multicore image-only solutions. |
| test-data/solutions/ImageOnly/.cmsis/image-only-multicore+CM0.dbgconf.base@0.0.2 | Debug config base file used by the tree output. |
| test-data/solutions/ImageOnly/.cmsis/image-only-multicore+CM0.dbgconf | Debug config file used by the tree output. |
| test-data/solutions/ImageOnly/.cmsis/image-only+CM0.dbgconf.base@0.0.2 | Debug config base file for the single-core fixture. |
| test-data/solutions/ImageOnly/.cmsis/image-only+CM0.dbgconf | Debug config file for the single-core fixture. |
| src/views/solution-outline/tree-structure/solution-outline-tree.ts | Ensures hardware nodes are created regardless of project presence per build. |
| src/views/solution-outline/tree-structure/solution-outline-tree.test.ts | Adds a unit test validating outline tree content for an image-only solution. |
| src/views/solution-outline/tree-structure/solution-outline-project-items.ts | Variable renames to improve readability (no behavior change). |
| src/views/solution-outline/tree-structure/solution-outline-hardware-item.ts | Variable renames to improve readability (no behavior change). |
You can also share your feedback on Copilot code review. Take the survey.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
by default 'out' directory is ignored
|
Coverage Impact ⬆️ Merging this pull request will increase total coverage on Modified Files with Diff Coverage (3)
🤖 Increase coverage with AI coding...🚦 See full report on Qlty Cloud » 🛟 Help
|

Fixes
Changes
Screenshots
Checklist