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.
@@ -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
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "claudecode-omc",
3
- "version": "4.3.5",
3
+ "version": "4.4.0",
4
4
  "description": "Multi-agent orchestration system for Claude Code - Inspired by oh-my-opencode",
5
5
  "type": "module",
6
6
  "main": "dist/index.js",
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