claudecode-omc 4.3.5 → 4.4.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +89 -208
- package/dist/hooks/bridge.js +5 -2
- package/dist/hooks/bridge.js.map +1 -1
- package/docs/CLAUDE.md +29 -7
- package/docs/plans/2026-02-26-skill-optimization-design.md +158 -0
- package/docs/plans/2026-02-26-skill-optimization-impl.md +606 -0
- package/package.json +1 -1
- package/skills/AGENTS.md +1 -0
- package/skills/ai-commenting/SKILL.md +108 -0
- package/skills/skill-development/SKILL.md +218 -0
- package/skills/skill-development/references/description-patterns.md +160 -0
- package/skills/skill-development/references/format-guide.md +203 -0
- package/skills/skill-quality-analyzer/SKILL.md +39 -41
|
@@ -0,0 +1,606 @@
|
|
|
1
|
+
# Skill Ecosystem Optimization — Implementation Plan
|
|
2
|
+
|
|
3
|
+
> **For Claude:** REQUIRED SUB-SKILL: Use superpowers:executing-plans to implement this plan task-by-task.
|
|
4
|
+
|
|
5
|
+
**Goal:** Optimize all ~40 oh-my-claudecode skills: precise trigger descriptions, progressive disclosure for heavy skills, new `writing-skills` standard, and de-Codex ported skills.
|
|
6
|
+
|
|
7
|
+
**Architecture:** Three-layer overhaul — (1) YAML description standardization across all skills, (2) progressive disclosure refactor of `skill/` (838→~150 lines), (3) new `writing-skills/` quality pipeline skill + de-Codex adaptation of `skill-quality-analyzer` and `skill-tester`.
|
|
8
|
+
|
|
9
|
+
**Tech Stack:** Markdown, YAML frontmatter, Bash (for verification); no code compilation needed.
|
|
10
|
+
|
|
11
|
+
**Design doc:** `docs/plans/2026-02-26-skill-optimization-design.md`
|
|
12
|
+
|
|
13
|
+
**Skill directory:** `/Users/WangQiao/Desktop/github/ios-dev/ZeroNet-Space/openSource/oh-my-cc/oh-my-claudecode/skills/`
|
|
14
|
+
|
|
15
|
+
---
|
|
16
|
+
|
|
17
|
+
## Task 1: Create `writing-skills/` Skill (New)
|
|
18
|
+
|
|
19
|
+
**Files:**
|
|
20
|
+
- Create: `skills/writing-skills/SKILL.md`
|
|
21
|
+
- Create: `skills/writing-skills/references/format-guide.md`
|
|
22
|
+
- Create: `skills/writing-skills/references/description-patterns.md`
|
|
23
|
+
|
|
24
|
+
**Step 1: Create directory structure**
|
|
25
|
+
```bash
|
|
26
|
+
mkdir -p skills/writing-skills/references
|
|
27
|
+
```
|
|
28
|
+
|
|
29
|
+
**Step 2: Create `skills/writing-skills/SKILL.md`**
|
|
30
|
+
|
|
31
|
+
```markdown
|
|
32
|
+
---
|
|
33
|
+
name: writing-skills
|
|
34
|
+
description: "Use when creating or improving a Claude Code skill ('write a skill', 'create a skill', 'new skill', 'improve this skill'). OMC standard for skill authorship: description format, XML structure, progressive disclosure, quality pipeline."
|
|
35
|
+
---
|
|
36
|
+
|
|
37
|
+
<Purpose>
|
|
38
|
+
Writing-skills is the OMC authorship guide for creating and improving Claude Code skills. It defines the standard format (YAML frontmatter + XML body), description trigger-phrase conventions, size targets, progressive disclosure patterns, and the quality pipeline that every skill should pass before shipping.
|
|
39
|
+
</Purpose>
|
|
40
|
+
|
|
41
|
+
<Use_When>
|
|
42
|
+
- User wants to create a new skill from scratch
|
|
43
|
+
- User wants to improve an existing skill's description, structure, or content
|
|
44
|
+
- User says "write a skill", "new skill", "create a skill", "skill template"
|
|
45
|
+
- A skill is not triggering and the description needs to be redesigned
|
|
46
|
+
- A skill is too long and needs progressive disclosure refactoring
|
|
47
|
+
</Use_When>
|
|
48
|
+
|
|
49
|
+
<Do_Not_Use_When>
|
|
50
|
+
- User wants to debug why a skill isn't triggering -- use `skill-debugger` instead
|
|
51
|
+
- User wants to score skill quality -- use `skill-quality-analyzer` instead
|
|
52
|
+
- User wants to test if a skill triggers correctly -- use `skill-tester` instead
|
|
53
|
+
- User wants to manage (list/add/remove) installed skills -- use `skill` instead
|
|
54
|
+
</Do_Not_Use_When>
|
|
55
|
+
|
|
56
|
+
<Why_This_Exists>
|
|
57
|
+
Without a writing standard, OMC skills accumulate inconsistently: vague descriptions that don't trigger, monolithic SKILL.md files that bloat context, Codex-style conventions that don't fit Claude Code. This skill establishes the canonical pattern so every new skill is immediately high quality.
|
|
58
|
+
</Why_This_Exists>
|
|
59
|
+
|
|
60
|
+
<Execution_Policy>
|
|
61
|
+
- Follow the Hybrid Workflow: Hard Steps (file creation, tool-verifiable checks) first, Soft Steps (description quality judgment) second
|
|
62
|
+
- Always verify description contains at least 2 specific trigger phrases in quotes
|
|
63
|
+
- Target SKILL.md body ≤200 lines; move detail to references/ when exceeded
|
|
64
|
+
- Run `skill-quality-analyzer` before declaring a skill done
|
|
65
|
+
- Consult `references/format-guide.md` for XML tag structure details
|
|
66
|
+
- Consult `references/description-patterns.md` for description examples by category
|
|
67
|
+
</Execution_Policy>
|
|
68
|
+
|
|
69
|
+
<Steps>
|
|
70
|
+
|
|
71
|
+
### Hybrid Workflow
|
|
72
|
+
|
|
73
|
+
**Hard Step 1 (tool): Check if skill directory exists**
|
|
74
|
+
```bash
|
|
75
|
+
ls skills/<skill-name>/ 2>/dev/null || echo "NEW"
|
|
76
|
+
```
|
|
77
|
+
|
|
78
|
+
**Soft Step 1 (judgment): Gather concrete trigger examples**
|
|
79
|
+
Ask: "What are 3 exact phrases a user would say to trigger this skill?"
|
|
80
|
+
These become the quoted trigger phrases in the description.
|
|
81
|
+
|
|
82
|
+
**Hard Step 2 (tool): Create directory structure**
|
|
83
|
+
```bash
|
|
84
|
+
mkdir -p skills/<skill-name>/references
|
|
85
|
+
```
|
|
86
|
+
|
|
87
|
+
**Soft Step 2 (judgment): Choose body format**
|
|
88
|
+
- Workflow/process skills (autopilot, ralph, plan) → XML tag format
|
|
89
|
+
- Utility/tool skills (note, trace, hud) → markdown header format
|
|
90
|
+
|
|
91
|
+
**Hard Step 3 (tool): Write SKILL.md**
|
|
92
|
+
Frontmatter (required):
|
|
93
|
+
```yaml
|
|
94
|
+
---
|
|
95
|
+
name: skill-name
|
|
96
|
+
description: "Use when user [specific scenario] ('[trigger 1]', '[trigger 2]', '[trigger 3]'). [One-sentence capability summary]."
|
|
97
|
+
---
|
|
98
|
+
```
|
|
99
|
+
Body: follow XML or markdown format per Step 2 above. Target ≤200 lines.
|
|
100
|
+
|
|
101
|
+
**Hard Step 4 (tool): Verify description format**
|
|
102
|
+
```bash
|
|
103
|
+
grep "^description:" skills/<skill-name>/SKILL.md | grep -q '"' && echo "OK: has quotes" || echo "FAIL: missing trigger phrases"
|
|
104
|
+
```
|
|
105
|
+
|
|
106
|
+
**Soft Step 3 (judgment): Check progressive disclosure**
|
|
107
|
+
If SKILL.md body > 200 lines: identify what to move to `references/`.
|
|
108
|
+
- Detailed reference tables → `references/`
|
|
109
|
+
- Long examples → `references/`
|
|
110
|
+
- Utility scripts → `scripts/` (if applicable)
|
|
111
|
+
|
|
112
|
+
**Hard Step 5 (tool): Run quality check**
|
|
113
|
+
```
|
|
114
|
+
Invoke Skill("oh-my-claudecode:skill-quality-analyzer") on the new skill path.
|
|
115
|
+
Target score ≥80.
|
|
116
|
+
```
|
|
117
|
+
|
|
118
|
+
**Hard Step 6 (tool): Commit**
|
|
119
|
+
```bash
|
|
120
|
+
git add skills/<skill-name>/
|
|
121
|
+
git commit -m "feat(skills): add <skill-name> skill"
|
|
122
|
+
```
|
|
123
|
+
|
|
124
|
+
### Full Process Checklist
|
|
125
|
+
|
|
126
|
+
1. Understand use cases (get 3 concrete trigger examples)
|
|
127
|
+
2. Choose format (XML for workflow, markdown for utility)
|
|
128
|
+
3. Create directory: `mkdir -p skills/<name>/references`
|
|
129
|
+
4. Write SKILL.md: frontmatter + body ≤200 lines
|
|
130
|
+
5. Write references/ files if needed
|
|
131
|
+
6. Verify: description has quoted triggers, line count within target
|
|
132
|
+
7. Run skill-quality-analyzer (score ≥80)
|
|
133
|
+
8. Commit
|
|
134
|
+
|
|
135
|
+
</Steps>
|
|
136
|
+
|
|
137
|
+
<Tool_Usage>
|
|
138
|
+
- Use `Glob("**/SKILL.md")` to check existing skills for reference and conflict detection
|
|
139
|
+
- Use `Read` to examine existing skills as templates
|
|
140
|
+
- Use `Grep` to check description format patterns across all skills
|
|
141
|
+
- Use `Bash` for line count checks: `wc -l skills/<name>/SKILL.md`
|
|
142
|
+
- Consult `references/format-guide.md` for complete XML tag reference
|
|
143
|
+
- Consult `references/description-patterns.md` for Good/Bad description catalog
|
|
144
|
+
</Tool_Usage>
|
|
145
|
+
|
|
146
|
+
<Examples>
|
|
147
|
+
<Good>
|
|
148
|
+
Creating a new skill with correct description:
|
|
149
|
+
```yaml
|
|
150
|
+
---
|
|
151
|
+
name: api-validator
|
|
152
|
+
description: "Use when validating API contracts or checking endpoint compatibility ('validate api', 'check api contract', 'api breaking change'). Runs schema validation and backward compatibility checks."
|
|
153
|
+
---
|
|
154
|
+
```
|
|
155
|
+
Why good: Third-party perspective, quoted triggers, one-sentence summary.
|
|
156
|
+
</Good>
|
|
157
|
+
|
|
158
|
+
<Good>
|
|
159
|
+
Correctly using progressive disclosure:
|
|
160
|
+
```
|
|
161
|
+
skills/heavy-skill/
|
|
162
|
+
├── SKILL.md (150 lines — core workflow only)
|
|
163
|
+
└── references/
|
|
164
|
+
├── patterns.md (detailed patterns reference)
|
|
165
|
+
└── examples.md (worked examples)
|
|
166
|
+
```
|
|
167
|
+
Why good: SKILL.md stays lean; detail loaded only when needed.
|
|
168
|
+
</Good>
|
|
169
|
+
|
|
170
|
+
<Bad>
|
|
171
|
+
Vague description with no trigger phrases:
|
|
172
|
+
```yaml
|
|
173
|
+
description: Provides guidance for API validation tasks
|
|
174
|
+
```
|
|
175
|
+
Why bad: No quoted trigger phrases; "provides guidance" is jargon; won't trigger reliably.
|
|
176
|
+
</Bad>
|
|
177
|
+
|
|
178
|
+
<Bad>
|
|
179
|
+
Monolithic SKILL.md:
|
|
180
|
+
```
|
|
181
|
+
skills/heavy-skill/
|
|
182
|
+
└── SKILL.md (850 lines — everything in one file)
|
|
183
|
+
```
|
|
184
|
+
Why bad: Full 850 lines loaded into context every invocation. Move detail to references/.
|
|
185
|
+
</Bad>
|
|
186
|
+
</Examples>
|
|
187
|
+
|
|
188
|
+
<Escalation_And_Stop_Conditions>
|
|
189
|
+
- If skill-quality-analyzer scores below 60, fix Critical and High issues before shipping
|
|
190
|
+
- If description trigger phrases don't feel natural, ask the user "would you actually say this?"
|
|
191
|
+
- If references/ files grow beyond 400 lines each, split further
|
|
192
|
+
</Escalation_And_Stop_Conditions>
|
|
193
|
+
|
|
194
|
+
<Final_Checklist>
|
|
195
|
+
- [ ] SKILL.md frontmatter has name and description
|
|
196
|
+
- [ ] Description contains at least 2 quoted trigger phrases
|
|
197
|
+
- [ ] Description written from "Use when user..." perspective
|
|
198
|
+
- [ ] SKILL.md body ≤200 lines
|
|
199
|
+
- [ ] Referenced files in references/ actually exist
|
|
200
|
+
- [ ] No "Codex", "Codex CLI", or other platform-specific branding in body
|
|
201
|
+
- [ ] skill-quality-analyzer score ≥80
|
|
202
|
+
- [ ] Committed with `feat(skills):` commit message
|
|
203
|
+
</Final_Checklist>
|
|
204
|
+
|
|
205
|
+
<Advanced>
|
|
206
|
+
## XML Tag Reference
|
|
207
|
+
|
|
208
|
+
See `references/format-guide.md` for complete tag-by-tag guide.
|
|
209
|
+
|
|
210
|
+
Quick reference:
|
|
211
|
+
- `<Purpose>` — what it does and produces
|
|
212
|
+
- `<Use_When>` — trigger conditions (specific)
|
|
213
|
+
- `<Do_Not_Use_When>` — negative triggers (redirect to correct skill)
|
|
214
|
+
- `<Why_This_Exists>` — rationale (prevents misuse)
|
|
215
|
+
- `<Execution_Policy>` — hard rules for execution
|
|
216
|
+
- `<Steps>` — ordered procedure
|
|
217
|
+
- `<Tool_Usage>` — which tools and when
|
|
218
|
+
- `<Examples>` — Good/Bad with explanations
|
|
219
|
+
- `<Escalation_And_Stop_Conditions>` — when to stop or escalate
|
|
220
|
+
- `<Final_Checklist>` — verification gate
|
|
221
|
+
- `<Advanced>` — detail deferred until needed (progressive disclosure)
|
|
222
|
+
|
|
223
|
+
## Description Format Reference
|
|
224
|
+
|
|
225
|
+
See `references/description-patterns.md` for catalog.
|
|
226
|
+
|
|
227
|
+
Format: `"Use when user [scenario] ('[trigger 1]', '[trigger 2]'). [Summary]."`
|
|
228
|
+
|
|
229
|
+
Three categories:
|
|
230
|
+
1. **Workflow triggers**: user wants to do something big (autopilot, ralph, plan)
|
|
231
|
+
2. **Command triggers**: user invokes by name (cancel, trace, hud)
|
|
232
|
+
3. **Domain triggers**: user has a domain problem (security-review, tdd, code-review)
|
|
233
|
+
</Advanced>
|
|
234
|
+
```
|
|
235
|
+
|
|
236
|
+
**Step 3: Create `skills/writing-skills/references/format-guide.md`**
|
|
237
|
+
|
|
238
|
+
Content — complete XML tag reference for OMC skills (see implementation notes below). ~180 lines.
|
|
239
|
+
|
|
240
|
+
Key sections:
|
|
241
|
+
- When to use XML vs markdown headers
|
|
242
|
+
- Tag-by-tag reference with purpose, required/optional, anti-patterns
|
|
243
|
+
- Size targets (SKILL.md ≤200 lines, references/ ≤400 lines each)
|
|
244
|
+
- Progressive disclosure decision guide
|
|
245
|
+
|
|
246
|
+
**Step 4: Create `skills/writing-skills/references/description-patterns.md`**
|
|
247
|
+
|
|
248
|
+
Content — Good/Bad description catalog for all ~40 OMC skill categories. ~200 lines.
|
|
249
|
+
|
|
250
|
+
Key sections:
|
|
251
|
+
- Format standard with annotated example
|
|
252
|
+
- Workflow skills patterns (autopilot, ralph, plan, ultrawork, team)
|
|
253
|
+
- Utility skills patterns (note, trace, hud, cancel)
|
|
254
|
+
- Domain skills patterns (tdd, security-review, code-review, ai-commenting)
|
|
255
|
+
- Anti-patterns (vague taglines, platform branding, missing trigger quotes)
|
|
256
|
+
|
|
257
|
+
**Step 5: Verify**
|
|
258
|
+
```bash
|
|
259
|
+
wc -l skills/writing-skills/SKILL.md
|
|
260
|
+
# Target: ≤200 lines
|
|
261
|
+
|
|
262
|
+
grep -c "description" skills/writing-skills/SKILL.md
|
|
263
|
+
# Should find description in frontmatter
|
|
264
|
+
```
|
|
265
|
+
|
|
266
|
+
**Step 6: Commit**
|
|
267
|
+
```bash
|
|
268
|
+
git add skills/writing-skills/
|
|
269
|
+
git commit -m "feat(skills): add writing-skills authorship guide"
|
|
270
|
+
```
|
|
271
|
+
|
|
272
|
+
---
|
|
273
|
+
|
|
274
|
+
## Task 2: Update All Skill Descriptions (Layer 1)
|
|
275
|
+
|
|
276
|
+
**Files:** All `skills/*/SKILL.md` — frontmatter `description` field only
|
|
277
|
+
|
|
278
|
+
**Target format:** `"Use when user [scenario] ('[trigger]', '[trigger]'). [Summary]."`
|
|
279
|
+
|
|
280
|
+
**Step 1: Apply all description rewrites**
|
|
281
|
+
|
|
282
|
+
Edit the `description:` line in each skill's frontmatter. Complete rewrite table:
|
|
283
|
+
|
|
284
|
+
| Skill | New Description |
|
|
285
|
+
|-------|----------------|
|
|
286
|
+
| `ai-commenting` | `"Use when annotating high-risk code with AI-readable metadata ('ai comment', 'annotate code', 'add ai tags'). Machine-parseable @ai: protocol encoding risk, intent, deps, auth, invariants."` |
|
|
287
|
+
| `analyze` | `"Use when debugging or investigating failures ('analyze', 'debug', 'investigate', 'why is this failing'). Delegates to debugger agent for root-cause analysis and regression isolation."` |
|
|
288
|
+
| `autopilot` | `"Use when user wants full-auto development ('build me', 'autopilot', 'I want a', 'create me'). End-to-end pipeline: spec→plan→code→QA→validation with no manual intervention."` |
|
|
289
|
+
| `build-fix` | `"Use when build or TypeScript compilation fails ('fix build', 'type errors', 'compilation error', 'build broken'). Minimal-diff fixes via build-fixer agent; no architectural changes."` |
|
|
290
|
+
| `cancel` | `"Use to cancel any active OMC execution mode ('cancel', 'stop', 'abort'). Cleanly exits autopilot, ralph, ultrawork, ultraqa, team, and pipeline state."` |
|
|
291
|
+
| `ccg` | `"Use when task benefits from tri-model parallel AI ('ccg', 'use all models', 'fan out to codex and gemini'). Backend→Codex, UI→Gemini, Claude synthesizes results."` |
|
|
292
|
+
| `code-review` | `"Use when reviewing code or PRs ('code review', 'review this PR', 'review code', 'check my code'). Multi-layer review: style, quality, security, API compatibility, and performance."` |
|
|
293
|
+
| `conductor` | `"Use when wanting structured Context→Spec→Implement workflow ('conductor', 'structured workflow', 'gather context then plan'). Ensures spec and plan are solid before execution."` |
|
|
294
|
+
| `configure-notifications` | `"Use when setting up agent completion notifications ('configure notifications', 'notify me on Telegram', 'Discord webhook', 'Slack alerts'). Natural language configuration of notification integrations."` |
|
|
295
|
+
| `deepinit` | `"Use when initializing a new codebase for agent work ('deepinit', 'initialize codebase', 'generate AGENTS.md', 'document codebase'). Creates hierarchical agent documentation across all modules."` |
|
|
296
|
+
| `external-context` | `"Use when needing external web or documentation lookup ('research', 'look up docs', 'search for', 'find documentation on'). Prefers Gemini MCP; falls back to parallel document-specialist agents."` |
|
|
297
|
+
| `hud` | `"Use when configuring the status bar display ('hud', 'configure display', 'hud layout', 'status bar presets'). Sets layout, visible elements, and display presets."` |
|
|
298
|
+
| `learn-about-omc` | `"Use when wanting OMC usage tips or personalized recommendations ('learn about omc', 'omc tips', 'how do I use omc', 'what omc features exist'). Reviews session history and suggests optimizations."` |
|
|
299
|
+
| `learner` | `"Use after solving a tricky bug or discovering a non-obvious pattern ('save this as a skill', 'extract skill', 'learn this', 'remember this pattern'). Packages conversation insight into a reusable skill."` |
|
|
300
|
+
| `mcp-setup` | `"Use when setting up MCP servers for Claude Code ('mcp setup', 'configure mcp', 'add mcp server', 'install context7'). Interactive wizard for Context7, filesystem, GitHub, and other MCP integrations."` |
|
|
301
|
+
| `note` | `"Use when saving quick notes that must survive context compaction ('note', 'remember this', 'save note', 'notepad'). Persists to .omc/notepad.md — survives /compact and session restarts."` |
|
|
302
|
+
| `omc-doctor` | `"Use when OMC is broken or behaving unexpectedly ('omc doctor', 'diagnose omc', 'fix omc', 'hooks not working'). Checks installation, hooks, config, and attempts automatic repair."` |
|
|
303
|
+
| `omc-help` | `"Use when asking how OMC works or what commands are available ('help', 'omc help', 'what can omc do', 'how do I'). Contextual guide with examples and command reference."` |
|
|
304
|
+
| `omc-setup` | `"Use when installing or reconfiguring oh-my-claudecode ('omc setup', 'setup omc', 'install omc', 'configure omc'). One-command wizard for hooks, MCP servers, HUD, and settings."` |
|
|
305
|
+
| `pipeline` | `"Use when chaining agents sequentially with data passing ('pipeline', 'chain agents', 'sequential workflow', 'agent pipeline'). Supports branching and error handling between stages."` |
|
|
306
|
+
| `plan` | `"Use when planning before coding ('plan this', 'plan the', 'let's plan', 'make a plan'). Interview, direct, consensus (--consensus), and review (--review) modes."` |
|
|
307
|
+
| `project-session-manager` | `"Use when managing isolated dev environments ('psm', 'project session', 'worktree session', 'open in tmux'). Creates git worktree + tmux session combos for parallel feature development."` |
|
|
308
|
+
| `ralph-init` | `"Use when starting a large task that needs structured requirements ('ralph-init', '--prd', 'create PRD', 'user stories first'). Generates PRD with user stories and acceptance criteria before ralph loop."` |
|
|
309
|
+
| `ralph` | `"Use when task must complete regardless of difficulty ('ralph', 'don't stop', 'must complete', 'finish this', 'keep going'). Persistence loop with ultrawork parallelism and architect sign-off."` |
|
|
310
|
+
| `ralplan` | `"Use when wanting multi-perspective plan validation ('ralplan', 'consensus plan', 'plan with multiple agents reviewing'). Alias for /plan --consensus: Planner→Architect→Critic iteration loop."` |
|
|
311
|
+
| `release` | `"Use when cutting a new release of oh-my-claudecode ('release', 'cut release', 'publish version', 'new release'). Automates changelog, version bump, tag, and npm publish."` |
|
|
312
|
+
| `review` | `"Use when reviewing or critiquing an existing plan ('review plan', 'critique plan', 'review this plan', 'is this plan good'). Alias for /plan --review: Critic evaluation with APPROVED/REVISE/REJECT verdict."` |
|
|
313
|
+
| `sciomc` | `"Use when doing data analysis or research with multiple agents ('sciomc', 'parallel research', 'analyze data with multiple agents', 'scientist swarm'). AUTO mode self-decomposes task; spawns parallel scientist agents."` |
|
|
314
|
+
| `security-review` | `"Use when checking code for security vulnerabilities ('security review', 'check for vulnerabilities', 'OWASP audit', 'security scan'). Comprehensive review: OWASP Top 10, authn/authz, trust boundaries."` |
|
|
315
|
+
| `skill-debugger` | `"Use when a skill isn't triggering or behaving unexpectedly ('debug skill', 'skill not triggering', 'why isn't skill working', 'skill discovery issue'). Diagnoses description quality, naming, conflicts, and file structure."` |
|
|
316
|
+
| `skill-quality-analyzer` | `"Use when checking a Claude Code skill's quality ('analyze skill quality', 'score skill', 'skill quality report', 'audit skill'). 6-dimension scoring: clarity, structure, examples, triggers, practices, maintainability."` |
|
|
317
|
+
| `skill-tester` | `"Use when verifying a Claude Code skill works as expected ('test skill', 'verify skill', 'does this skill trigger', 'skill test'). Trigger tests, functional tests, and edge case validation."` |
|
|
318
|
+
| `skill` | `"Use when managing local Claude Code skills ('skill list', 'skill add', 'skill remove', 'skill search', 'skill sync'). CRUD + setup wizard for ~/.claude/skills/ and .omc/skills/."` |
|
|
319
|
+
| `tdd` | `"Use when implementing features with test-first discipline ('tdd', 'test first', 'red green refactor', 'write tests first', 'TDD'). Enforces Red→Green→Refactor cycle with no implementation before failing test."` |
|
|
320
|
+
| `team` | `"Use when coordinating multiple agents on a project ('team', 'coordinated team', 'multi-agent', 'swarm'). TeamCreate→tasks→parallel agents→verify→TeamDelete staged pipeline."` |
|
|
321
|
+
| `trace` | `"Use when reviewing what agents ran in a session ('trace', 'show agent trace', 'what agents ran', 'session timeline'). Renders timeline of agent calls, durations, and results."` |
|
|
322
|
+
| `ultrapilot` | `"Use when running autopilot with maximum file-partitioned parallelism ('ultrapilot', 'parallel autopilot', 'parallel build'). File-ownership partitioned parallel execution; facade over Team staged pipeline."` |
|
|
323
|
+
| `ultraqa` | `"Use when cycling QA until all tests pass ('ultraqa', 'qa loop', 'fix until tests pass', 'keep fixing until green'). Build→test→lint→fix cycle, repeats up to configured limit."` |
|
|
324
|
+
| `ultrawork` | `"Use when running multiple independent tasks in parallel ('ulw', 'ultrawork', 'run in parallel', 'parallel execution'). Routes tasks to tiered agents (haiku/sonnet/opus) simultaneously."` |
|
|
325
|
+
| `writer-memory` | `"Use when writing fiction and needing persistent story tracking ('writer-memory', 'track characters', 'remember scenes', 'writing continuity'). Maintains character, relationship, scene, and theme state across sessions."` |
|
|
326
|
+
|
|
327
|
+
**Step 2: Verify descriptions have trigger quotes**
|
|
328
|
+
```bash
|
|
329
|
+
# Check all descriptions contain quoted trigger phrases
|
|
330
|
+
for f in skills/*/SKILL.md; do
|
|
331
|
+
skill=$(basename $(dirname $f))
|
|
332
|
+
desc=$(grep -m1 "^description:" "$f")
|
|
333
|
+
if echo "$desc" | grep -q "'"; then
|
|
334
|
+
echo "✅ $skill"
|
|
335
|
+
else
|
|
336
|
+
echo "❌ $skill — missing quoted triggers"
|
|
337
|
+
fi
|
|
338
|
+
done
|
|
339
|
+
```
|
|
340
|
+
Expected: all ✅
|
|
341
|
+
|
|
342
|
+
**Step 3: Commit**
|
|
343
|
+
```bash
|
|
344
|
+
git add skills/
|
|
345
|
+
git commit -m "feat(skills): standardize all skill descriptions with trigger phrases"
|
|
346
|
+
```
|
|
347
|
+
|
|
348
|
+
---
|
|
349
|
+
|
|
350
|
+
## Task 3: Refactor `skill/SKILL.md` — Progressive Disclosure
|
|
351
|
+
|
|
352
|
+
**Files:**
|
|
353
|
+
- Modify: `skills/skill/SKILL.md` (838→~150 lines)
|
|
354
|
+
- Create: `skills/skill/references/templates.md`
|
|
355
|
+
- Create: `skills/skill/references/setup-guide.md`
|
|
356
|
+
|
|
357
|
+
**Step 1: Create references directory**
|
|
358
|
+
```bash
|
|
359
|
+
mkdir -p skills/skill/references
|
|
360
|
+
```
|
|
361
|
+
|
|
362
|
+
**Step 2: Move content to `references/templates.md`**
|
|
363
|
+
|
|
364
|
+
Extract these four complete templates from `skills/skill/SKILL.md` into `references/templates.md`:
|
|
365
|
+
- Error Solution Template
|
|
366
|
+
- Workflow Skill Template
|
|
367
|
+
- Code Pattern Template
|
|
368
|
+
- Integration Skill Template
|
|
369
|
+
|
|
370
|
+
Add header:
|
|
371
|
+
```markdown
|
|
372
|
+
# Skill Templates Reference
|
|
373
|
+
|
|
374
|
+
Templates for creating new skills via `/skill add`. Copy and customize.
|
|
375
|
+
```
|
|
376
|
+
|
|
377
|
+
**Step 3: Move content to `references/setup-guide.md`**
|
|
378
|
+
|
|
379
|
+
Extract from `skills/skill/SKILL.md` into `references/setup-guide.md`:
|
|
380
|
+
- Step 1 directory check + bash script (the full `if [ -d "$USER_SKILLS_DIR" ]` block)
|
|
381
|
+
- Step 2 skill scan bash script (the `find "$HOME/.claude/skills/omc-learned"` block)
|
|
382
|
+
- Example session (the `> /oh-my-claudecode:skill list` full output example)
|
|
383
|
+
- Implementation Notes section
|
|
384
|
+
- Future Enhancements section
|
|
385
|
+
|
|
386
|
+
Add header:
|
|
387
|
+
```markdown
|
|
388
|
+
# Setup Guide & Examples Reference
|
|
389
|
+
|
|
390
|
+
Detailed setup wizard behavior and complete usage examples.
|
|
391
|
+
```
|
|
392
|
+
|
|
393
|
+
**Step 4: Rewrite `skills/skill/SKILL.md`** to lean version
|
|
394
|
+
|
|
395
|
+
Lean SKILL.md keeps:
|
|
396
|
+
- Frontmatter (updated description from Task 2)
|
|
397
|
+
- Subcommands overview table (name | behavior | example)
|
|
398
|
+
- Core behavior for each command — condensed to 3-5 bullets max
|
|
399
|
+
- Error handling format (4 lines)
|
|
400
|
+
- Skill quality guidelines (4 bullets)
|
|
401
|
+
- Reference pointers to `references/templates.md` and `references/setup-guide.md`
|
|
402
|
+
- Related skills
|
|
403
|
+
- Usage examples (quick reference only, no full session output)
|
|
404
|
+
|
|
405
|
+
**Step 5: Verify line count**
|
|
406
|
+
```bash
|
|
407
|
+
wc -l skills/skill/SKILL.md
|
|
408
|
+
# Target: ≤200 lines
|
|
409
|
+
|
|
410
|
+
wc -l skills/skill/references/templates.md skills/skill/references/setup-guide.md
|
|
411
|
+
# Each should be reasonable (100-300 lines)
|
|
412
|
+
```
|
|
413
|
+
|
|
414
|
+
**Step 6: Verify references are linked**
|
|
415
|
+
```bash
|
|
416
|
+
grep "references/" skills/skill/SKILL.md
|
|
417
|
+
# Should show both references/templates.md and references/setup-guide.md mentioned
|
|
418
|
+
```
|
|
419
|
+
|
|
420
|
+
**Step 7: Commit**
|
|
421
|
+
```bash
|
|
422
|
+
git add skills/skill/
|
|
423
|
+
git commit -m "refactor(skills): progressive disclosure for skill management skill (838→~150 lines)"
|
|
424
|
+
```
|
|
425
|
+
|
|
426
|
+
---
|
|
427
|
+
|
|
428
|
+
## Task 4: De-Codex `skill-quality-analyzer/SKILL.md`
|
|
429
|
+
|
|
430
|
+
**Files:** Modify: `skills/skill-quality-analyzer/SKILL.md`
|
|
431
|
+
|
|
432
|
+
**Step 1: Replace all "Codex" references**
|
|
433
|
+
|
|
434
|
+
Find and replace:
|
|
435
|
+
- "Analyzes Codex skill quality" → "Analyzes Claude Code skill quality"
|
|
436
|
+
- "Codex CLI" → "Claude Code"
|
|
437
|
+
- "Codex skill" → "Claude Code skill"
|
|
438
|
+
- "Codex standards" → "Claude Code standards"
|
|
439
|
+
- "Codex naming conventions" → "Claude Code naming conventions"
|
|
440
|
+
- In path examples: `~/.codex/skills/` → `~/.claude/skills/`; `.agents/skills/` → `.claude/skills/`
|
|
441
|
+
|
|
442
|
+
**Step 2: Update Agent Workflow section**
|
|
443
|
+
|
|
444
|
+
Replace the Codex-specific Hybrid Workflow with Claude Code tool-based workflow:
|
|
445
|
+
|
|
446
|
+
```markdown
|
|
447
|
+
## Agent Workflow
|
|
448
|
+
|
|
449
|
+
To analyze effectively, follow this **Hybrid Workflow**:
|
|
450
|
+
|
|
451
|
+
1. **Hard Metrics (tool-based)**: Use `Glob` and `Read` tools to gather objective data:
|
|
452
|
+
- Check if `SKILL.md` exists: `Glob("skills/<name>/SKILL.md")`
|
|
453
|
+
- Count lines: note `wc -l` equivalent via Read
|
|
454
|
+
- Verify YAML frontmatter has `name` and `description`
|
|
455
|
+
- Check description contains quoted trigger phrases
|
|
456
|
+
- Verify any referenced `references/` files exist
|
|
457
|
+
- If `analyzer.py` exists in the skill directory, optionally run: `python3 skills/<name>/analyzer.py --skill-path skills/<name>/`
|
|
458
|
+
|
|
459
|
+
2. **Soft Metrics (agent judgment)**: Read the target skill's content and assess:
|
|
460
|
+
- Is the description specific enough to trigger reliably?
|
|
461
|
+
- Does the body use consistent format (XML or markdown headers)?
|
|
462
|
+
- Are examples present and concrete?
|
|
463
|
+
- Is content lean (≤200 lines for SKILL.md)?
|
|
464
|
+
|
|
465
|
+
3. **Synthesize Report**: Combine hard metrics + soft judgment into severity-ranked findings.
|
|
466
|
+
```
|
|
467
|
+
|
|
468
|
+
**Step 3: Verify no "Codex" remains**
|
|
469
|
+
```bash
|
|
470
|
+
grep -i "codex" skills/skill-quality-analyzer/SKILL.md
|
|
471
|
+
# Expected: no output
|
|
472
|
+
```
|
|
473
|
+
|
|
474
|
+
**Step 4: Commit**
|
|
475
|
+
```bash
|
|
476
|
+
git add skills/skill-quality-analyzer/SKILL.md
|
|
477
|
+
git commit -m "fix(skills): adapt skill-quality-analyzer for Claude Code (remove Codex branding)"
|
|
478
|
+
```
|
|
479
|
+
|
|
480
|
+
---
|
|
481
|
+
|
|
482
|
+
## Task 5: De-Codex and Simplify `skill-tester/SKILL.md`
|
|
483
|
+
|
|
484
|
+
**Files:** Modify: `skills/skill-tester/SKILL.md`
|
|
485
|
+
|
|
486
|
+
**Step 1: Replace frontmatter description**
|
|
487
|
+
|
|
488
|
+
Use the new description from Task 2:
|
|
489
|
+
```yaml
|
|
490
|
+
description: "Use when verifying a Claude Code skill works as expected ('test skill', 'verify skill', 'does this skill trigger', 'skill test'). Trigger tests, functional tests, and edge case validation."
|
|
491
|
+
```
|
|
492
|
+
|
|
493
|
+
**Step 2: Replace all "Codex" references**
|
|
494
|
+
|
|
495
|
+
- "Tests Codex skill functionality" → "Tests Claude Code skill behavior"
|
|
496
|
+
- "Codex CLI" → "Claude Code"
|
|
497
|
+
- "Codex skill" → "Claude Code skill"
|
|
498
|
+
- All path examples: `~/.codex/skills/` → `~/.claude/skills/`
|
|
499
|
+
|
|
500
|
+
**Step 3: Simplify test case format**
|
|
501
|
+
|
|
502
|
+
Replace the JSON-heavy test case structure (designed for Codex function calls) with scenario-based format for prompt skills:
|
|
503
|
+
|
|
504
|
+
```markdown
|
|
505
|
+
## Test Case Format for Claude Code Skills
|
|
506
|
+
|
|
507
|
+
Since Claude Code skills are prompt-based (not function calls), test cases describe scenarios:
|
|
508
|
+
|
|
509
|
+
**Trigger Test:**
|
|
510
|
+
```
|
|
511
|
+
Scenario: User says "review my code"
|
|
512
|
+
Expected: code-review skill activates
|
|
513
|
+
Verify: Skill tool invoked with "code-review"
|
|
514
|
+
```
|
|
515
|
+
|
|
516
|
+
**Behavioral Test:**
|
|
517
|
+
```
|
|
518
|
+
Scenario: User says "build me a todo app"
|
|
519
|
+
Expected: autopilot skill activates and begins Phase 0 expansion
|
|
520
|
+
Verify: analyst and architect agents spawned within 2 turns
|
|
521
|
+
```
|
|
522
|
+
|
|
523
|
+
**Negative Test:**
|
|
524
|
+
```
|
|
525
|
+
Scenario: User says "what is React?"
|
|
526
|
+
Expected: NO skill invokes (just answer the question)
|
|
527
|
+
Verify: No Skill tool call in response
|
|
528
|
+
```
|
|
529
|
+
```
|
|
530
|
+
|
|
531
|
+
**Step 4: Remove performance testing section** (not applicable to prompt-based skills)
|
|
532
|
+
|
|
533
|
+
Remove:
|
|
534
|
+
- "Performance Tests" subsection
|
|
535
|
+
- "Performance Benchmarks" section
|
|
536
|
+
- Time-limit scoring table
|
|
537
|
+
|
|
538
|
+
**Step 5: Verify no "Codex" remains**
|
|
539
|
+
```bash
|
|
540
|
+
grep -i "codex" skills/skill-tester/SKILL.md
|
|
541
|
+
# Expected: no output
|
|
542
|
+
```
|
|
543
|
+
|
|
544
|
+
**Step 6: Verify line count reduced** (target: from 346 lines to ~180 lines)
|
|
545
|
+
```bash
|
|
546
|
+
wc -l skills/skill-tester/SKILL.md
|
|
547
|
+
```
|
|
548
|
+
|
|
549
|
+
**Step 7: Commit**
|
|
550
|
+
```bash
|
|
551
|
+
git add skills/skill-tester/SKILL.md
|
|
552
|
+
git commit -m "fix(skills): adapt skill-tester for Claude Code prompt-based skill testing"
|
|
553
|
+
```
|
|
554
|
+
|
|
555
|
+
---
|
|
556
|
+
|
|
557
|
+
## Task 6: Final Verification
|
|
558
|
+
|
|
559
|
+
**Step 1: Check all descriptions have trigger quotes**
|
|
560
|
+
```bash
|
|
561
|
+
echo "=== Skills WITHOUT quoted triggers ==="
|
|
562
|
+
for f in skills/*/SKILL.md; do
|
|
563
|
+
skill=$(basename $(dirname $f))
|
|
564
|
+
desc=$(grep -m1 "^description:" "$f")
|
|
565
|
+
if ! echo "$desc" | grep -q "'"; then
|
|
566
|
+
echo "❌ $skill"
|
|
567
|
+
fi
|
|
568
|
+
done
|
|
569
|
+
echo "=== Done ==="
|
|
570
|
+
```
|
|
571
|
+
Expected: no output after "==="
|
|
572
|
+
|
|
573
|
+
**Step 2: Check no Codex branding remains in user-facing skills**
|
|
574
|
+
```bash
|
|
575
|
+
grep -rl "Codex" skills/*/SKILL.md | grep -v "ccg"
|
|
576
|
+
# ccg intentionally references Codex (it IS a Codex skill)
|
|
577
|
+
# All other results are problems
|
|
578
|
+
```
|
|
579
|
+
|
|
580
|
+
**Step 3: Check skill/ line count**
|
|
581
|
+
```bash
|
|
582
|
+
wc -l skills/skill/SKILL.md
|
|
583
|
+
# Target: ≤200
|
|
584
|
+
```
|
|
585
|
+
|
|
586
|
+
**Step 4: Verify writing-skills/ files exist**
|
|
587
|
+
```bash
|
|
588
|
+
ls skills/writing-skills/
|
|
589
|
+
ls skills/writing-skills/references/
|
|
590
|
+
```
|
|
591
|
+
Expected: SKILL.md, references/format-guide.md, references/description-patterns.md
|
|
592
|
+
|
|
593
|
+
**Step 5: Commit verification pass**
|
|
594
|
+
```bash
|
|
595
|
+
git log --oneline -6
|
|
596
|
+
# Should show all 5 task commits
|
|
597
|
+
```
|
|
598
|
+
|
|
599
|
+
---
|
|
600
|
+
|
|
601
|
+
## Execution Notes
|
|
602
|
+
|
|
603
|
+
- All changes are Markdown file edits — no compilation needed
|
|
604
|
+
- The `ccg` skill intentionally keeps "Codex" in description (it IS about Codex integration)
|
|
605
|
+
- `analyzer.py` in `skill-quality-analyzer/` is untouched — it's functional and platform-agnostic Python
|
|
606
|
+
- `project-session-manager/` bash scripts are untouched — not skill content
|
package/package.json
CHANGED
package/skills/AGENTS.md
CHANGED
|
@@ -44,6 +44,7 @@ Skills are reusable workflow templates that can be invoked via `/oh-my-claudecod
|
|
|
44
44
|
| `security-review/SKILL.md` | security-review | Security vulnerability detection |
|
|
45
45
|
| `tdd/SKILL.md` | tdd | Test-driven development workflow |
|
|
46
46
|
| `build-fix/SKILL.md` | build-fix | Fix build and TypeScript errors |
|
|
47
|
+
| `ai-commenting/SKILL.md` | ai-commenting | AI-native code annotation protocol for intent/risk/deps/test context |
|
|
47
48
|
|
|
48
49
|
### Exploration Skills
|
|
49
50
|
|