Skip to content

fix: quote argument-hint YAML values so Copilot CLI ≥1.0.65 loads all skills (closes #1) - #2

Open
thejesh23 wants to merge 1 commit into
julianromli:mainfrom
thejesh23:fix/quote-argument-hint-yaml
Open

fix: quote argument-hint YAML values so Copilot CLI ≥1.0.65 loads all skills (closes #1)#2
thejesh23 wants to merge 1 commit into
julianromli:mainfrom
thejesh23:fix/quote-argument-hint-yaml

Conversation

@thejesh23

@thejesh23 thejesh23 commented Jul 13, 2026

Copy link
Copy Markdown

Closes #1.

Summary

Purely mechanical single-character transform to 3 file(s): wrap the value after argument-hint: in double quotes so YAML parses it as a string instead of a flow-sequence array. Fixes GitHub Copilot CLI ≥ 1.0.65 silently dropping the skill.

- argument-hint: [foo]
+ argument-hint: "[foo]"

Bare [...] is YAML flow-sequence syntax → the loader receives ["foo"] (a list), fails its str validation, and rejects the SKILL without an error. Claude Code, VS Code Agent Skills, and every other loader document argument-hint as a string; the quoted-string form is what all reference docs show.

Files (3)

  • command/debug.md
  • command/review.md
  • command/subagent-driven-development.md

Verification

  • YAML round-trip via yaml.safe_load on every changed frontmatter reports argument-hint as str
  • No vulnerable value contained " or \, so bare double-quote wrapping is safe

Sources


Summary by cubic

Quote the argument-hint frontmatter values so YAML parses them as strings, preventing GitHub Copilot CLI ≥ 1.0.65 from silently dropping these skills.

Written for commit f2286a8. Summary will update on new commits.

Review in cubic

Summary by CodeRabbit

  • Documentation
    • Updated command argument hints to use consistent quoted placeholder formatting.
    • Applied the formatting update to debug, review, and plan execution commands.

… skills

`argument-hint: [foo]` YAML-parses as a flow sequence (array), not a
string. Downstream slash-command loaders that validate `argument-hint`
as a string — notably GitHub Copilot CLI ≥ 1.0.65 — silently reject
the skill on load, and the command disappears from the CLI menu.

Wrap the value in double quotes so it parses as a string. No behaviour
change on Claude Code.
@coderabbitai

coderabbitai Bot commented Jul 13, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: cdaeea40-19e1-44d8-9e33-1d9fb517c18a

📥 Commits

Reviewing files that changed from the base of the PR and between 37ceab2 and f2286a8.

📒 Files selected for processing (3)
  • command/debug.md
  • command/review.md
  • command/subagent-driven-development.md

📝 Walkthrough

Walkthrough

Three command markdown files update YAML front-matter argument-hint values from bare bracket notation to quoted strings, preserving the existing placeholder text.

Changes

Command argument-hint normalization

Layer / File(s) Summary
Quote command placeholders
command/debug.md, command/review.md, command/subagent-driven-development.md
The argument-hint values are changed to quoted strings for the error, files, and plan-file placeholders.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Poem

I’m a rabbit with a careful quill,
Quoting hints with tidy skill.
Brackets stay, but strings now shine,
Three command paths align in line.
Hop, hop—metadata’s fine!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main fix: quoting argument-hint YAML values so Copilot CLI loads the skills.
Linked Issues check ✅ Passed The PR matches issue #1 by quoting the three affected argument-hint values in the specified command files.
Out of Scope Changes check ✅ Passed The changes stay within scope and only update the three quoted argument-hint values described in the issue.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 3 files

Re-trigger cubic

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.

Skills fail to load in GitHub Copilot CLI ≥1.0.65 due to array-shaped argument-hint

1 participant