Skip to content

feat(debug): accept positional flavors on debug config - #4174

Merged
dgageot merged 1 commit into
docker:mainfrom
dgageot:worktree-board-825ca87b94432694
Sep 5, 2026
Merged

feat(debug): accept positional flavors on debug config#4174
dgageot merged 1 commit into
docker:mainfrom
dgageot:worktree-board-825ca87b94432694

Conversation

@dgageot

@dgageot dgageot commented Sep 4, 2026

Copy link
Copy Markdown
Member

Inspecting a resolved config — after one or more flavors have been applied — currently requires writing a small wrapper YAML or manually editing the file. This is tedious when you just want to see what debug config would look like with a particular flavor active.

debug config now accepts flavor names as positional arguments after the agent file or registry ref: docker agent debug config examples/flavors.yaml cheap with-shell. Any positional flavors are appended after values supplied via --flavor, so both forms can be combined freely. When at least one flavor is provided (by either means), the command prints the fully resolved config — patches applied and the flavors section omitted, since re-applying patches to an already-resolved document would produce incorrect results. Without flavors the output is unchanged.

No breaking changes; the existing --flavor flag and the no-flavor path both behave exactly as before.

@dgageot
dgageot requested a review from a team as a code owner September 4, 2026 15:20
@aheritier aheritier added area/docs Documentation changes kind/feat PR adds a new feature (maps to feat:). Use on PRs only. status/needs-rebase PR has merge conflicts or is out of date with main labels Sep 4, 2026
@aheritier

Copy link
Copy Markdown
Collaborator

👋 This PR has merge conflicts with the base branch. Please rebase or merge the latest base branch and resolve them. I've moved it to draft and added status/needs-rebase; it'll be picked back up automatically once the conflicts are cleared.

Allows `docker agent debug config agent.yaml cheap with-shell` as an
alternative (or complement) to `--flavor`. Resolved configs have their
`flavors` section stripped so re-running the output is idempotent.

Assisted-By: claude-sonnet-4-5
@dgageot
dgageot force-pushed the worktree-board-825ca87b94432694 branch from 8f4e303 to 67486bd Compare September 4, 2026 16:02
@aheritier aheritier removed the status/needs-rebase PR has merge conflicts or is out of date with main label Sep 4, 2026
@dgageot
dgageot merged commit 5683379 into docker:main Sep 5, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/docs Documentation changes kind/feat PR adds a new feature (maps to feat:). Use on PRs only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants