Skip to content

fix: quote argument-hint YAML for Copilot CLI 1.0.65 compatibility#9

Open
thejesh23 wants to merge 1 commit into
coleam00:mainfrom
thejesh23:fix/copilot-cli-1.0.65-argument-hint-string
Open

fix: quote argument-hint YAML for Copilot CLI 1.0.65 compatibility#9
thejesh23 wants to merge 1 commit into
coleam00:mainfrom
thejesh23:fix/copilot-cli-1.0.65-argument-hint-string

Conversation

@thejesh23

@thejesh23 thejesh23 commented Jul 3, 2026

Copy link
Copy Markdown

Summary

Several .claude/ files have argument-hint: frontmatter values whose YAML parses as a flow-sequence or flow-mapping (or fails entirely), instead of the string that Copilot CLI 1.0.65 now strictly requires. This PR quotes the affected values.

The YAML rule

In YAML, an unquoted scalar that begins with [ or { starts a flow-collection, not a string:

  • argument-hint: [files...] -> list(['files...'])
  • argument-hint: [X] [Y] -> invalid YAML (two flow-sequences at one key)
  • argument-hint: [X] (extra) -> invalid YAML
  • argument-hint: "[X] [Y]" -> "[X] [Y]" (string, correct)

Values that begin with < or a letter (plain scalars) are already strings, so they were left unchanged.

Verification

Before the fix, several frontmatter blocks failed yaml.safe_load outright (e.g. prd.md, and the same class as SKILL.md's two-sequences case). After the fix, all four affected frontmatter values parse as strings.

Files changed

File Before After
.claude/skills/build-with-agent-team/SKILL.md [plan-path] [num-agents] "[plan-path] [num-agents]"
.claude/commands/commit.md [files...] "[files...]"
.claude/commands/create-pr.md [base-branch] "[base-branch]"
.claude/commands/prd.md [feature/product idea] (blank = start with questions) quoted
.claude/commands/create-command.md template examples with {If needed} / {Input description} quoted (so newly generated commands are compliant)

Untouched (already valid): implement.md, plan.md, rca.md, review-pr.md, and the <input> example in create-command.md.

References

Test plan

  • yaml.safe_load on each fixed frontmatter returns a str
  • Confirmed prd.md pre-fix was unparseable YAML; post-fix parses cleanly
  • Load a fixed command / skill under Copilot CLI 1.0.65 and confirm no argument-hint type error
  • Existing Claude Code behavior unchanged (argument-hint continues to render as literal string in the UI)

Copilot CLI 1.0.65 requires `argument-hint` to be a YAML string, but
unquoted values starting with `[` or `{` are parsed as flow-sequences or
flow-mappings. Several frontmatter blocks and one template example were
either invalid YAML or produced non-string types.

- `.claude/skills/build-with-agent-team/SKILL.md`: `[plan-path] [num-agents]` -> quoted string
- `.claude/commands/commit.md`: `[files...]` -> quoted string
- `.claude/commands/create-pr.md`: `[base-branch]` -> quoted string
- `.claude/commands/prd.md`: `[feature/product idea] ...` -> quoted string (was unparseable YAML)
- `.claude/commands/create-command.md`: template examples using `{...}` placeholders now show the quoted form so newly generated commands are compliant

Values that already begin with `<` (plain scalar) or are already quoted
were left unchanged.
@thejesh23

Copy link
Copy Markdown
Author

Tracking issue: #10 — captures the bug diagnosis and reproducer separately for anyone searching the repo who lands there before this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant