Resolve file links in cbuild*.yml files#109
Conversation
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Snapshot WarningsEnsure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice. Scanned FilesNone |
There was a problem hiding this comment.
Pull request overview
Extends the VS Code document-linking support so users can Ctrl+Click/open referenced files from additional CMSIS YAML artifacts (notably *.cbuild*.yml and *.cgen.yml), including resolving ${CMSIS_PACK_ROOT}-based paths.
Changes:
- Register document link providers for
*.cgen.{yml,yaml}and*.cbuild{,-idx,-run}.ymlfiles. - Enhance
ReferenceLinkProviderto handle more reference tags and expand${CMSIS_PACK_ROOT}(with optional skipping of RPC variable expansion for generated build files). - Add/update unit tests for the new selectors and cbuild-link behavior.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| src/views/solution-outline/tree-structure/solution-outline-file-item.ts | Adjusts ${CMSIS_PACK_ROOT} detection in solution outline file nodes. |
| src/solutions/language-features/solution-language-features-provider.ts | Adds new document selectors and registers an additional link provider for build YAMLs. |
| src/solutions/language-features/solution-language-features-provider.test.ts | Updates activation test to expect both link providers to be registered. |
| src/solutions/language-features/reference-link-provider.ts | Adds cbuild-aware behavior, expands ${CMSIS_PACK_ROOT}, and broadens supported reference tags. |
| src/solutions/language-features/reference-link-provider.test.ts | Adds test coverage for cbuild link behavior and ${CMSIS_PACK_ROOT} expansion. |
|
Coverage Impact This PR will not change total coverage. Modified Files with Diff Coverage (3)
🤖 Increase coverage with AI coding...🚦 See full report on Qlty Cloud » 🛟 Help
|
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Fixes
Changes
Improvements after failed review :
Screenshots
Checklist