From faccc6113d8720b41a1f590f79f57aef5e6b82a5 Mon Sep 17 00:00:00 2001 From: thejesh23 <35212698+thejesh23@users.noreply.github.com> Date: Sun, 12 Jul 2026 12:49:29 -0700 Subject: [PATCH] =?UTF-8?q?fix(skills):=20quote=20argument-hint=20YAML=20v?= =?UTF-8?q?alues=20so=20Copilot=20CLI=20=E2=89=A51.0.65=20loads=20all=20sk?= =?UTF-8?q?ills=20(#5627)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * v0.6.29: login improvements, posthog telemetry (#4026) * feat(posthog): Add tracking on mothership abort (#4023) Co-authored-by: Theodore Li * fix(login): fix captcha headers for manual login (#4025) * fix(signup): fix turnstile key loading * fix(login): fix captcha header passing * Catch user already exists, remove login form captcha * fix: quote argument-hint YAML values so Copilot CLI ≥1.0.65 loads all 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. --------- Co-authored-by: Waleed Co-authored-by: Theodore Li Co-authored-by: Siddharth Ganesan <33737564+Sg312@users.noreply.github.com> Co-authored-by: Vikhyath Mondreti Co-authored-by: Theodore Li --- .claude/commands/cleanup.md | 2 +- .claude/commands/emcn-design-review.md | 2 +- .claude/commands/react-query-best-practices.md | 2 +- .claude/commands/ship.md | 2 +- .claude/commands/you-might-not-need-a-callback.md | 2 +- .claude/commands/you-might-not-need-a-memo.md | 2 +- .claude/commands/you-might-not-need-an-effect.md | 2 +- .claude/commands/you-might-not-need-state.md | 2 +- .claude/commands/you-might-not-need-url-state.md | 2 +- 9 files changed, 9 insertions(+), 9 deletions(-) diff --git a/.claude/commands/cleanup.md b/.claude/commands/cleanup.md index 34bce559f4..b0cbbb42fa 100644 --- a/.claude/commands/cleanup.md +++ b/.claude/commands/cleanup.md @@ -1,6 +1,6 @@ --- description: Run all code quality skills in sequence — effects, memo, callbacks, state, React Query, emcn design review, and url-state -argument-hint: [scope] [fix=true|false] +argument-hint: "[scope] [fix=true|false]" --- # Cleanup diff --git a/.claude/commands/emcn-design-review.md b/.claude/commands/emcn-design-review.md index 0c96b07479..741c02c64b 100644 --- a/.claude/commands/emcn-design-review.md +++ b/.claude/commands/emcn-design-review.md @@ -1,6 +1,6 @@ --- description: Review UI code for alignment with the emcn design system — components, tokens, patterns, and conventions -argument-hint: [scope] [fix=true|false] +argument-hint: "[scope] [fix=true|false]" --- # EMCN Design Review diff --git a/.claude/commands/react-query-best-practices.md b/.claude/commands/react-query-best-practices.md index 0fc450958b..a1222866f9 100644 --- a/.claude/commands/react-query-best-practices.md +++ b/.claude/commands/react-query-best-practices.md @@ -1,6 +1,6 @@ --- description: Audit React Query usage for best practices — key factories, staleTime, mutations, and server state ownership -argument-hint: [scope] [fix=true|false] +argument-hint: "[scope] [fix=true|false]" --- # React Query Best Practices diff --git a/.claude/commands/ship.md b/.claude/commands/ship.md index 6141fa01c1..df54f36b95 100644 --- a/.claude/commands/ship.md +++ b/.claude/commands/ship.md @@ -1,6 +1,6 @@ --- description: Commit, push, and open a PR to staging in one shot -argument-hint: [optional context or scope notes] +argument-hint: "[optional context or scope notes]" --- # Ship Command diff --git a/.claude/commands/you-might-not-need-a-callback.md b/.claude/commands/you-might-not-need-a-callback.md index c25d4c32ea..a67f1ecb7d 100644 --- a/.claude/commands/you-might-not-need-a-callback.md +++ b/.claude/commands/you-might-not-need-a-callback.md @@ -1,6 +1,6 @@ --- description: Analyze and fix useCallback anti-patterns in your code -argument-hint: [scope] [fix=true|false] +argument-hint: "[scope] [fix=true|false]" --- # You Might Not Need a Callback diff --git a/.claude/commands/you-might-not-need-a-memo.md b/.claude/commands/you-might-not-need-a-memo.md index f0c5b3124a..5b0f5b7983 100644 --- a/.claude/commands/you-might-not-need-a-memo.md +++ b/.claude/commands/you-might-not-need-a-memo.md @@ -1,6 +1,6 @@ --- description: Analyze and fix useMemo/React.memo anti-patterns in your code -argument-hint: [scope] [fix=true|false] +argument-hint: "[scope] [fix=true|false]" --- # You Might Not Need a Memo diff --git a/.claude/commands/you-might-not-need-an-effect.md b/.claude/commands/you-might-not-need-an-effect.md index 1e798c804e..cd438e6f01 100644 --- a/.claude/commands/you-might-not-need-an-effect.md +++ b/.claude/commands/you-might-not-need-an-effect.md @@ -1,6 +1,6 @@ --- description: Analyze and fix useEffect anti-patterns in your code -argument-hint: [scope] [fix=true|false] +argument-hint: "[scope] [fix=true|false]" --- # You Might Not Need an Effect diff --git a/.claude/commands/you-might-not-need-state.md b/.claude/commands/you-might-not-need-state.md index 2e539146f6..eab91ca7b3 100644 --- a/.claude/commands/you-might-not-need-state.md +++ b/.claude/commands/you-might-not-need-state.md @@ -1,6 +1,6 @@ --- description: Analyze and fix unnecessary useState, derived state, and server-state-in-local-state anti-patterns -argument-hint: [scope] [fix=true|false] +argument-hint: "[scope] [fix=true|false]" --- # You Might Not Need State diff --git a/.claude/commands/you-might-not-need-url-state.md b/.claude/commands/you-might-not-need-url-state.md index 77ba1ebecc..bcab8ac2db 100644 --- a/.claude/commands/you-might-not-need-url-state.md +++ b/.claude/commands/you-might-not-need-url-state.md @@ -1,6 +1,6 @@ --- description: Analyze and fix URL/query-param state anti-patterns — manual useSearchParams reads, hand-built query mutations, view-state trapped in useState, and objects in the URL -argument-hint: [scope] [fix=true|false] +argument-hint: "[scope] [fix=true|false]" --- # You Might Not Need URL State