Skip to content

Commit faccc61

Browse files
thejesh23waleedlatif1TheodoreSpeaksSg312icecrasher321
authored
fix(skills): quote argument-hint YAML values so Copilot CLI ≥1.0.65 loads all skills (#5627)
* v0.6.29: login improvements, posthog telemetry (#4026) * feat(posthog): Add tracking on mothership abort (#4023) Co-authored-by: Theodore Li <theo@sim.ai> * 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 <walif6@gmail.com> Co-authored-by: Theodore Li <theodoreqili@gmail.com> Co-authored-by: Siddharth Ganesan <33737564+Sg312@users.noreply.github.com> Co-authored-by: Vikhyath Mondreti <vikhyathvikku@gmail.com> Co-authored-by: Theodore Li <theo@sim.ai>
1 parent 5e891da commit faccc61

9 files changed

Lines changed: 9 additions & 9 deletions

.claude/commands/cleanup.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
description: Run all code quality skills in sequence — effects, memo, callbacks, state, React Query, emcn design review, and url-state
3-
argument-hint: [scope] [fix=true|false]
3+
argument-hint: "[scope] [fix=true|false]"
44
---
55

66
# Cleanup

.claude/commands/emcn-design-review.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
description: Review UI code for alignment with the emcn design system — components, tokens, patterns, and conventions
3-
argument-hint: [scope] [fix=true|false]
3+
argument-hint: "[scope] [fix=true|false]"
44
---
55

66
# EMCN Design Review

.claude/commands/react-query-best-practices.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
description: Audit React Query usage for best practices — key factories, staleTime, mutations, and server state ownership
3-
argument-hint: [scope] [fix=true|false]
3+
argument-hint: "[scope] [fix=true|false]"
44
---
55

66
# React Query Best Practices

.claude/commands/ship.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
description: Commit, push, and open a PR to staging in one shot
3-
argument-hint: [optional context or scope notes]
3+
argument-hint: "[optional context or scope notes]"
44
---
55

66
# Ship Command

.claude/commands/you-might-not-need-a-callback.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
description: Analyze and fix useCallback anti-patterns in your code
3-
argument-hint: [scope] [fix=true|false]
3+
argument-hint: "[scope] [fix=true|false]"
44
---
55

66
# You Might Not Need a Callback

.claude/commands/you-might-not-need-a-memo.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
description: Analyze and fix useMemo/React.memo anti-patterns in your code
3-
argument-hint: [scope] [fix=true|false]
3+
argument-hint: "[scope] [fix=true|false]"
44
---
55

66
# You Might Not Need a Memo

.claude/commands/you-might-not-need-an-effect.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
description: Analyze and fix useEffect anti-patterns in your code
3-
argument-hint: [scope] [fix=true|false]
3+
argument-hint: "[scope] [fix=true|false]"
44
---
55

66
# You Might Not Need an Effect

.claude/commands/you-might-not-need-state.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
description: Analyze and fix unnecessary useState, derived state, and server-state-in-local-state anti-patterns
3-
argument-hint: [scope] [fix=true|false]
3+
argument-hint: "[scope] [fix=true|false]"
44
---
55

66
# You Might Not Need State

.claude/commands/you-might-not-need-url-state.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
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
3-
argument-hint: [scope] [fix=true|false]
3+
argument-hint: "[scope] [fix=true|false]"
44
---
55

66
# You Might Not Need URL State

0 commit comments

Comments
 (0)