Skip to content

Quote argument-hint in phaser SKILL.md frontmatter#31

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

Quote argument-hint in phaser SKILL.md frontmatter#31
thejesh23 wants to merge 1 commit into
PlayableIntelligence:mainfrom
thejesh23:fix/copilot-cli-1.0.65-argument-hint-string

Conversation

@thejesh23

@thejesh23 thejesh23 commented Jul 3, 2026

Copy link
Copy Markdown

Summary

skills/phaser/SKILL.md has argument-hint: [topic or question] (unquoted). Because [...] in YAML is the flow-sequence syntax, this parses as a two-element list (['topic or question'] after the pipe collapses, or a sequence node depending on parser), not a string.

Copilot CLI 1.0.65 requires argument-hint to be a string; when it encounters a sequence the skill fails to load. Every other skill in this repo already quotes the value (argument-hint: "[topic]"), so this one-line change brings phaser in line with the rest of the tree.

YAML rule

  • argument-hint: [X] -> YAML sequence (breaks Copilot CLI 1.0.65)
  • argument-hint: "[X]" -> YAML string (correct)

Verified locally: after the fix, yaml.safe_load returns str with value '[topic or question]'.

Related

Test plan

  • grep -rn 'argument-hint:' skills/ shows all 30 entries are now quoted strings
  • python3 -c "import yaml; ..." confirms the frontmatter parses with argument-hint as str
  • Load the phaser skill in Copilot CLI 1.0.65 and confirm it no longer errors on the argument-hint field

Summary by CodeRabbit

  • Documentation
    • Updated a configuration hint to use quoted text, improving formatting consistency and compatibility.

Unquoted `[topic or question]` parses as a YAML sequence, but Copilot
CLI 1.0.65 requires `argument-hint` to be a string. All other skills in
this repo already quote it; this brings phaser in line and prevents the
skill from failing to load.
@coderabbitai

coderabbitai Bot commented Jul 3, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The YAML frontmatter field argument-hint in skills/phaser/SKILL.md was changed to wrap its value in quotes, converting it from an unquoted to a quoted string.

Changes

SKILL.md Frontmatter Update

Layer / File(s) Summary
Quote argument-hint value
skills/phaser/SKILL.md
The argument-hint frontmatter value is changed from unquoted [topic or question] to quoted "[topic or question]".

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

Related issues: None found in the provided information.

Suggested labels: documentation

Suggested reviewers: None specified.

Poem

A rabbit hops through YAML lines,
Adds two quotes where none confines,
A tiny tweak, a frontmatter cheer,
The hint now wrapped both far and near. 🐰

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and accurately describes the main change: quoting the argument-hint frontmatter in the Phaser skill file.
✨ 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.

@thejesh23

Copy link
Copy Markdown
Author

Tracking issue: #32 — 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