Skip to content

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

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

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

Conversation

@thejesh23

Copy link
Copy Markdown

Closes #3613.

Summary

Purely mechanical single-character transform to 1 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 (1)

  • .atuin/skills/release/SKILL.md

Verification

  • YAML round-trip via yaml.safe_load on every changed frontmatter reports argument-hint as str
  • Regression sweep: no unquoted-bracket argument-hint: lines remain in these files
  • No vulnerable value contained " or \, so bare double-quote wrapping is safe

Sources

… 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.
@github-actions

Copy link
Copy Markdown
Contributor

Fossier: Manual Review Requested

@thejesh23 is a new contributor. A maintainer should review this PR before merging.

Score Breakdown

Total Score: 49.0/100 | Confidence: 100% | Outcome: REVIEW

Signal Value Score Weight
account_age 3107 1.00 0.09
public_repos 59 1.00 0.05
contribution_history 59 0.29 0.05
follower_ratio 0.26 0.13 0.05
bot_signals False 0.50 0.04
open_prs_elsewhere 37 0.00 0.09
closed_prs_elsewhere 0 1.00 0.10
merged_prs_elsewhere 14 1.00 0.08
prior_interaction 0 0.00 0.08
activity_velocity 2 0.67 0.08
pr_content ... 0.10 0.08
commit_email no_email 0.50 0.04
pr_description ... 0.65 0.05
repo_stars 30527 0.30 0.04
org_membership 0 0.20 0.03
commit_verification ... 0.30 0.04
contributor_stars 0 0.00 0.04

@greptile-apps

greptile-apps Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR updates the release skill frontmatter for CLI loader compatibility.

  • Quotes argument-hint in .atuin/skills/release/SKILL.md.
  • Keeps the displayed hint as [version] while making the YAML value a string.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
.atuin/skills/release/SKILL.md Quotes the release skill argument-hint so YAML parses it as a string.

Reviews (1): Last reviewed commit: "fix: quote argument-hint YAML values so ..." | Re-trigger Greptile

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

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