feat(package list): add --synthesize-debug-packages to package list#108
Merged
reubeno merged 5 commits intomicrosoft:mainfrom Apr 21, 2026
Merged
feat(package list): add --synthesize-debug-packages to package list#108reubeno merged 5 commits intomicrosoft:mainfrom
reubeno merged 5 commits intomicrosoft:mainfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds an opt-in azldev package list mode to emit additional “synthetic” debug-related packages alongside the normally-resolved package list, primarily to support short-term downstream routing needs until component-level publish-channel inheritance lands (ref #107).
Changes:
- Add
--synthesize-debug-packagesflag and option plumbing forazldev package list. - Implement synthesis of
*-debuginfo(per listed package) and*-debugsource(per component) entries. - Add unit tests and regenerate CLI reference docs for the new flag.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| internal/app/azldev/cmds/pkg/list.go | Adds flag/option and implements synthesizeDebugPackages to augment list results. |
| internal/app/azldev/cmds/pkg/list_test.go | Adds tests covering basic synthesis behavior and collision-skipping behavior. |
| docs/user/reference/cli/azldev_package_list.md | Updates CLI docs to include the new --synthesize-debug-packages option. |
Augments 'azldev package list' with synthetic '-debuginfo' packages (one per reported package, sharing the original publish channel) and '-debugsource' packages (one per component, resolved with no per-package override so they fall back to the configured default publishing channel). This is a short-term approach until we're able to properly merge changes that shift us over to an inheritance-driven declaration for where source packages, debuginfo packages, etc. should go for a given component. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
reubeno
force-pushed
the
synthesize-debug-packages
branch
from
c53827a to
89e2a20
Compare
- Skip synthesis for source names already ending in '-debuginfo' or '-debugsource' to avoid recursive doubled suffixes (e.g. 'foo-debuginfo- debuginfo') when those packages are already in the listed set. - Restore '-debugsource' channel resolution via ResolvePackageConfig so the result carries an honest channel value (component default → project default) rather than an implicit empty 'consumer applies default' contract. - Emit an slog.Warn that the flag is transitional and may change. - Document that '-debugsource' is emitted for every component regardless of the requested package set, since most listed packages come from groups with no component association. - Tests: add assertions that doubled-suffix names are NOT synthesized; add a '-p' + flag test covering the headline CLI path. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Synthesized debug packages now publish to a parallel channel: any resolved, non-empty, non-'none' channel is suffixed with '-debuginfo' (channels that already end in '-debuginfo' are left as-is). Empty and 'none' channels pass through unchanged so 'do not publish' and 'use default' semantics are preserved. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
binujp
reviewed
Apr 21, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Augments
azldev package listwith synthetic-debuginfopackages (one per reported package, with-debuginfomodifier of the publish channel) and-debugsourcepackages.This is a short-term approach until we're able to properly merge changes that shift us over to an inheritance-driven declaration for where source packages,
debuginfopackages, etc. should go for a given component (i.e. #107).