-
Notifications
You must be signed in to change notification settings - Fork 284
Skill Reference Suggested Fixes #158
Description
Archived Skills Still Listed as Active
These 4 skills are in archive/ but appear in the active skills list. Their archived SKILL.md files are stubs with no body content — they do nothing if invoked.
| Skill | Active Successor | Status |
|---|---|---|
async-repl-protocol |
None (retired) | [ ] |
router-first-architecture |
None (retired, listed twice) | [ ] |
search-hierarchy |
search-router |
[ ] |
search-tools |
search-router |
[ ] |
Fix: Remove from the active skills list.
Naming Convention Violations
Skills should use kebab-case in both the directory name and the frontmatter name: field. These violate that:
| Directory | name field |
Severity | Status |
|---|---|---|---|
implement_task/ |
implement_task |
Both wrong | [ ] |
implement_plan/ |
implement_plan |
Both wrong | [ ] |
continuity_ledger/ |
continuity-ledger |
Directory wrong only | [ ] |
create_handoff/ |
create-handoff |
Directory wrong only | [ ] |
resume_handoff/ |
resume-handoff |
Directory wrong only | [ ] |
describe_pr/ |
describe-pr |
Directory wrong only | [ ] |
implement_task and implement_plan are the worst offenders — the name: frontmatter field also uses underscores, which may break skill discovery.
Fix: Rename directories to kebab-case; update name: field in implement_task and implement_plan.
Rule Documents Masquerading as Skills
These 8 skills have no workflow, no tool calls, and no invocation steps. They are pure behavioral guidelines that belong in .claude/rules/ rather than .claude/skills/.
| Skill | What it actually is | Status |
|---|---|---|
graceful-degradation |
DO/DON'T coding guideline | [ ] |
modular-code |
File size and split heuristics | [ ] |
explicit-identity |
Pattern about passing IDs across agent boundaries | [ ] |
no-polling-agents |
Anti-polling behavioral rule | [ ] |
background-agent-pings |
Anti-polling duplicate — overlaps no-polling-agents |
[ ] |
idempotent-redundancy |
Short pattern DO/DON'T with session citations | [ ] |
llm-tuning-patterns |
LLM parameter reference guide | [ ] |
agent-context-isolation |
RIGHT/WRONG code examples for background agents | [ ] |
Outdated Model Pins
Two skills are pinned to claude-opus-4-5-20251101 in their frontmatter model: field. The current model is claude-opus-4-6, so these silently use an older model on every invocation.
| Skill | File | Line | Status |
|---|---|---|---|
discovery-interview |
SKILL.md |
5 | [ ] |
research |
SKILL.md |
4 | [ ] |
Additionally, two skills have stale model IDs in documentation examples (not functional bugs, but misleading):
| Skill | Notes | Status |
|---|---|---|
sub-agents |
Line 68 lists claude-opus-4-5-20251101 as a valid option |
[ ] |
help |
Line 256 shows claude-opus-4-5-20251101 in an example |
[ ] |
Fix: Remove the version suffix or update to claude-opus-4-6. Omitting the model: field entirely lets the skill inherit from the parent.