mirror of
https://github.com/simstudioai/sim.git
synced 2026-09-24 15:45:35 +08:00
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>
This commit is contained in:
co-authored by
Waleed
Theodore Li
Siddharth Ganesan
Vikhyath Mondreti
Theodore Li
parent
5e891da819
commit
faccc6113d
@@ -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
|
||||
|
||||
@@ -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
|
||||
|
||||
@@ -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
|
||||
|
||||
@@ -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
|
||||
|
||||
@@ -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
|
||||
|
||||
@@ -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
|
||||
|
||||
@@ -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
|
||||
|
||||
@@ -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
|
||||
|
||||
@@ -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
|
||||
|
||||
Reference in New Issue
Block a user