oh-my-customcode 0.9.2 → 0.9.4

package/templates/.codex/rules/MUST-sync-verification.md

@@ -1,215 +1,38 @@
 # [MUST] Sync Verification Rules
 
-> **Priority**: MUST - ENFORCED after any structural changes
-> **ID**: R017
-> **Violation**: Committing or pushing without verification = Rule violation
+> **Priority**: MUST - ENFORCED | **ID**: R017
 
-## CRITICAL
+## Core Rule
 
-**After modifying agents, skills, or guides, you MUST run the full verification process before committing AND pushing.**
+After modifying agents, skills, or guides: run full verification before committing AND pushing. Never ask to commit/push before `mgr-sauron:watch` passes.
 
-```
-╔══════════════════════════════════════════════════════════════════╗
-║  ⚠️  ABSOLUTE PROHIBITION                                        ║
-║                                                                   ║
-║  DO NOT even ASK "커밋하시겠습니까?" or "푸시하시겠습니까?"      ║
-║  after structural changes until mgr-sauron:watch has been        ║
-║  executed and passed.                                            ║
-║                                                                   ║
-║  WRONG:                                                          ║
-║    [Make changes] → "커밋하시겠습니까?"                          ║
-║    [Commit done] → "푸시하시겠습니까?"                           ║
-║                                                                   ║
-║  CORRECT:                                                        ║
-║    [Make changes] → mgr-sauron:watch → [All pass] → commit → push ║
-║                                                                   ║
-║  Asking to commit/push before verification = Rule violation      ║
-╚══════════════════════════════════════════════════════════════════╝
-```
+Every `git push` requires: `mgr-sauron:watch` → all pass → `git push`
 
-```
-╔══════════════════════════════════════════════════════════════════╗
-║  ⚠️  PUSH REQUIRES SAURON VERIFICATION                           ║
-║                                                                   ║
-║  EVERY git push in this project MUST be preceded by:             ║
-║    mgr-sauron:watch → [All pass] → git push                      ║
-║                                                                   ║
-║  This applies to ALL pushes, not just structural changes.        ║
-║  Sauron verification is the quality gate for this repository.    ║
-║                                                                   ║
-║  NO EXCEPTIONS. NO SHORTCUTS.                                    ║
-╚══════════════════════════════════════════════════════════════════╝
-```
+## Verification Phases
 
-```
-╔══════════════════════════════════════════════════════════════════╗
-║  MANDATORY VERIFICATION PROCESS:                                 ║
-║                                                                   ║
-║  Phase 1: Manager Agent Verification (5 rounds)                  ║
-║    - mgr-supplier:audit                                          ║
-║    - mgr-sync-checker:check                                      ║
-║    - mgr-updater:docs                                            ║
-║    - mgr-claude-code-bible:verify                                ║
-║                                                                   ║
-║  Phase 2: Deep Review (3 rounds)                                 ║
-║    - Workflow alignment                                          ║
-║    - Reference verification                                      ║
-║    - Philosophy compliance                                       ║
-║                                                                   ║
-║  Phase 3: Fix all issues                                         ║
-║                                                                   ║
-║  Phase 4: Commit via mgr-gitnerd                                 ║
-║                                                                   ║
-║  Phase 5: Push via mgr-gitnerd                                   ║
-║    - ONLY after sauron verification passes                       ║
-║                                                                   ║
-║  Skipping ANY phase = Rule violation                             ║
-╚══════════════════════════════════════════════════════════════════╝
-```
+### Phase 1: Manager Verification (5 rounds)
 
-## Phase 1: Manager Agent Verification (5 Rounds)
+| Round | Actions |
+|-------|---------|
+| 1-2 | mgr-supplier:audit, mgr-sync-checker:check, fix issues |
+| 3-4 | Re-verify + mgr-updater:docs, fix remaining |
+| 5 | Final: all counts match, frontmatter valid, skill refs exist, memory scopes valid, routing patterns updated |
 
-Run manager agents 5 times to catch all issues:
+Also run: mgr-claude-code-bible:verify (official spec compliance)
 
-### Round 1-2: Basic Sync
-```
-□ mgr-supplier:audit - Check all agent dependencies
-□ mgr-sync-checker:check - Verify registry synchronization
-□ Fix any issues found
-```
+### Phase 2: Deep Review (3 rounds)
 
-### Round 3-4: Deep Sync
-```
-□ mgr-supplier:audit - Re-verify after fixes
-□ mgr-sync-checker:check - Re-verify registries
-□ mgr-updater:docs - Check documentation sync
-□ Fix any remaining issues
-```
+| Round | Focus |
+|-------|-------|
+| 1 | Workflow alignment: routing skills have complete agent mappings |
+| 2 | References: no orphans, no circular refs, valid skill/memory refs |
+| 3 | Philosophy: R006 separation, R009 parallel, R010 delegation, R007/R008 identification |
 
-### Round 5: Final Verification
-```
-□ All agent counts match (AGENTS.md, actual .md files in .codex/agents/)
-□ All agent frontmatter valid (required fields present)
-□ All skill references in agents exist in .codex/skills/
-□ All memory field values are valid (user | project | local)
-□ Memory distribution matches AGENTS.md (24 project + 3 user + 7 none)
-□ All routing skill patterns updated
-```
-
-## Phase 2: Deep Review (3 Rounds)
-
-### Deep Round 1: Workflow Alignment
-```
-□ Agent creation workflow documented and functional
-□ Development workflow uses proper routing skills
-□ Deployment workflow defined (if applicable)
-□ All routing skills have complete agent pattern mappings
-```
-
-### Deep Round 2: Reference Verification
-```
-□ All routing skills properly reference their managed agents
-□ All rules are properly referenced
-□ No orphaned agents (not referenced by any routing skill)
-□ No circular references
-□ All skill references in agent frontmatter are valid
-□ All agent memory fields use valid scope values
-```
-
-### Deep Round 3: Philosophy Compliance
-```
-□ R006: Separation of concerns (agent file = role only, no details)
-□ R009: Parallel execution enabled for main conversation
-□ R010: Multi-agent tasks use routing skills and Task tool
-□ R007/R008: Agent/tool identification documented
-□ All MUST rules enforced, SHOULD rules recommended
-```
-
-## Verification Checklist Summary
-
-```
-Phase 1: Manager Verification (5x)
-  □ Round 1: mgr-supplier:audit
-  □ Round 2: mgr-sync-checker:check
-  □ Round 3: mgr-supplier:audit (re-verify)
-  □ Round 4: mgr-sync-checker:check + mgr-updater:docs
-  □ Round 5: Final count verification
-  □ mgr-claude-code-bible:verify (official spec compliance)
-
-Phase 2: Deep Review (3x)
-  □ Round 1: Workflow alignment
-  □ Round 2: Reference verification
-  □ Round 3: Philosophy compliance
-
-Phase 3: Fix Issues
-  □ All issues from Phase 1 fixed
-  □ All issues from Phase 2 fixed
-  □ Re-run verification if major fixes made
-
-Phase 4: Commit
-  □ Delegate to mgr-gitnerd
-  □ Meaningful commit message
-
-Phase 5: Push
-  □ mgr-sauron:watch passed (MANDATORY)
-  □ Delegate push to mgr-gitnerd
-  □ Verify push succeeded
-```
+### Phase 3: Fix all discovered issues
 
-## When to Run Full Verification
+### Phase 4: Commit via mgr-gitnerd
 
-| Action | Full Verification Required |
-|--------|---------------------------|
-| Create new agent | **YES** |
-| Rename agent | **YES** |
-| Delete agent | **YES** |
-| Modify agent frontmatter | **YES** |
-| Add/modify skill | **YES** |
-| Add/modify guide | **YES** |
-| Change routing skill patterns | **YES** |
-| Update rules | **YES** |
-
-## Workflow Diagram
-
-```
-Content Change Made
-        │
-        ▼
-┌───────────────────────────────┐
-│  Phase 1: Manager Verification │
-│  (5 rounds)                    │
-│  mgr-supplier → mgr-sync-checker → │
-│  mgr-updater                   │
-└───────────────┬───────────────┘
-                │
-                ▼
-┌───────────────────────────────┐
-│  Phase 2: Deep Review          │
-│  (3 rounds)                    │
-│  Workflow → References →       │
-│  Philosophy                    │
-└───────────────┬───────────────┘
-                │
-        ┌───────┴───────┐
-        ▼               ▼
-  [Issues Found]   [All Clean]
-        │               │
-        ▼               │
-   Fix Issues           │
-        │               │
-        └───────┬───────┘
-                │
-                ▼
-┌───────────────────────────────┐
-│  Phase 4: Commit via mgr-gitnerd │
-└───────────────┬───────────────┘
-                │
-                ▼
-┌───────────────────────────────┐
-│  Phase 5: Push via mgr-gitnerd  │
-│  (ONLY after sauron passes)    │
-└───────────────────────────────┘
-```
+### Phase 5: Push via mgr-gitnerd (only after sauron passes)
 
 ## Self-Check Before Commit and Push
 
@@ -239,43 +62,64 @@ Content Change Made
 ╚══════════════════════════════════════════════════════════════════╝
 ```
 
+## When Required
+
+Any change to: agents, agent frontmatter, skills, guides, routing patterns, rules.
+
 ## Quick Verification Commands
 
 ```bash
 # Agent count check
-ls .codex/agents/*.md | wc -l
+ls .claude/agents/*.md | wc -l
 
 # Skill count check
-find .codex/skills -name "SKILL.md" | wc -l
+find .claude/skills -name "SKILL.md" | wc -l
 
 # Frontmatter validation (check for missing YAML headers)
-for f in .codex/agents/*.md; do head -1 "$f" | grep -q "^---" || echo "MISSING FRONTMATTER: $f"; done
+for f in .claude/agents/*.md; do head -1 "$f" | grep -q "^---" || echo "MISSING FRONTMATTER: $f"; done
 
 # Check for agents with invalid skill references
-for f in .codex/agents/*.md; do
+for f in .claude/agents/*.md; do
   grep "^skills:" -A 10 "$f" | grep "  - " | sed 's/.*- //' | while read skill; do
-    [ -f ".codex/skills/$skill/SKILL.md" ] || echo "INVALID SKILL REF in $f: $skill"
+    [ -f ".claude/skills/$skill/SKILL.md" ] || echo "INVALID SKILL REF in $f: $skill"
   done
 done
 
 # Routing skill pattern coverage
-grep -c "agent_patterns:" .codex/skills/secretary-routing/SKILL.md
-grep -c "agent_patterns:" .codex/skills/dev-lead-routing/SKILL.md
-grep -c "agent_patterns:" .codex/skills/qa-lead-routing/SKILL.md
+grep -c "agent_patterns:" .claude/skills/secretary-routing/SKILL.md
+grep -c "agent_patterns:" .claude/skills/dev-lead-routing/SKILL.md
+grep -c "agent_patterns:" .claude/skills/qa-lead-routing/SKILL.md
 
 # Memory field validation
-for f in .codex/agents/*.md; do
+for f in .claude/agents/*.md; do
   mem=$(grep "^memory:" "$f" | awk '{print $2}')
   if [ -n "$mem" ] && [ "$mem" != "project" ] && [ "$mem" != "user" ] && [ "$mem" != "local" ]; then
     echo "INVALID MEMORY SCOPE in $f: $mem"
   fi
 done
-```
 
-## Integration with Other Rules
+# Hook count check
+ls .claude/hooks/*.json 2>/dev/null | wc -l
 
-| Rule | Integration |
-|------|-------------|
-| R009 (Parallel) | Verification checks can run in parallel |
-| R010 (Orchestrator) | Delegate commit to mgr-gitnerd |
-| R016 (Improvement) | Update this rule if new issues found |
+# Context count check
+ls .claude/contexts/*.md 2>/dev/null | wc -l
+
+# Guide count check
+find guides -mindepth 1 -maxdepth 1 -type d | wc -l
+
+# Agent name accuracy (compare CLAUDE.md table with actual files)
+# Extract agent names from files
+ls .claude/agents/*.md | xargs -I{} basename {} .md | sort > /tmp/actual-agents.txt
+
+# Slash command skill existence
+for cmd in $(grep "^| \`/" CLAUDE.md | sed 's/.*`\///' | sed 's/`.*//' | sed 's/ .*//')
+do
+  [ -d ".claude/skills/$cmd" ] || echo "MISSING SKILL: $cmd"
+done
+
+# Routing skill completeness check
+ls -d .claude/skills/*-routing 2>/dev/null | xargs -I{} basename {} | sort
+
+# Verify routing skill names in CLAUDE.md
+grep -oP '(secretary|dev-lead|de-lead|qa-lead)-routing' CLAUDE.md | sort -u
+```

package/templates/.codex/rules/MUST-tool-identification.md

@@ -1,46 +1,17 @@
 # [MUST] Tool Usage Identification Rules
 
-> **Priority**: MUST - ENFORCED, NO EXCEPTIONS
-> **ID**: R008
-> **Violation**: Immediate correction required
+> **Priority**: MUST - ENFORCED | **ID**: R008
 
-## CRITICAL
+## Core Rule
 
-**EVERY tool call MUST be prefixed with agent and model identification. This is NON-NEGOTIABLE.**
+Every tool call MUST be prefixed with agent and model identification:
 
 ```
-Before EVERY tool call, output:
 [agent-name][model] → Tool: <tool-name>
 [agent-name][model] → Target: <file/path/url>
 ```
 
-Failure to identify tool usage = Rule violation = Must be corrected immediately.
-
-### Self-Check Before EVERY Tool Call
-
-```
-╔══════════════════════════════════════════════════════════════════╗
-║  BEFORE INVOKING ANY TOOL, ASK YOURSELF:                         ║
-║                                                                   ║
-║  1. Did I output the tool identification line with MODEL?        ║
-║     [agent-name][model] → Tool: <tool-name>                      ║
-║     [agent-name][model] → Target: <target>                       ║
-║                                                                   ║
-║  2. Am I about to call multiple tools in parallel?               ║
-║     → EACH tool needs its own identification line                ║
-║     → List all identifications BEFORE the tool calls             ║
-║                                                                   ║
-║  If NO to #1 → STOP. Output identification FIRST.                ║
-╚══════════════════════════════════════════════════════════════════╝
-```
-
-### Model Values
-
-| Model | When to Use |
-|-------|-------------|
-| `opus` | Complex reasoning, architecture design |
-| `sonnet` | General tasks, code generation (default) |
-| `haiku` | Fast simple tasks, file search |
+For parallel calls: list ALL identifications BEFORE the tool calls.
 
 ### Common Violations to Avoid
 
@@ -76,120 +47,36 @@ Failure to identify tool usage = Rule violation = Must be corrected immediately.
    <tool_call>Bash(cmd)</tool_call>
 ```
 
-## Purpose
-
-Display which agent is using which tool for transparency and debugging. This extends R007 (Agent Identification) to cover tool operations.
-
-## Tool Usage Format
+## Models
 
-When invoking a tool, prefix with agent and model identification:
-
-```
-[agent-name][model] → Tool: <tool-name>
-```
-
-### Examples
-
-```
-[lang-golang-expert][sonnet] → Tool: Read
-[lang-golang-expert][sonnet] → Reading: src/main.go
-
-[mgr-supplier][haiku] → Tool: Glob
-[mgr-supplier][haiku] → Searching: .codex/agents/*.md
-
-[mgr-creator][sonnet] → Tool: Write
-[mgr-creator][sonnet] → Writing: .codex/agents/new-agent.md
-```
-
-## Extended Format (Verbose)
-
-For detailed tracking:
-
-```
-┌─ Agent: lang-golang-expert (sw-engineer)
-├─ Model: sonnet
-├─ Skill: go-best-practices
-├─ Tool: Read
-└─ Target: src/main.go
-```
+| Model | Use |
+|-------|-----|
+| `opus` | Complex reasoning, architecture |
+| `sonnet` | General tasks, code generation (default) |
+| `haiku` | Fast simple tasks, file search |
 
 ## Tool Categories
 
-| Category | Tools | Format |
-|----------|-------|--------|
-| File Read | Read, Glob, Grep | `→ Reading:` / `→ Searching:` |
-| File Write | Write, Edit | `→ Writing:` / `→ Editing:` |
-| Network | WebFetch | `→ Fetching:` |
-| Execution | Bash, Task | `→ Running:` / `→ Spawning:` |
-
-## When to Display
-
-| Situation | Display |
-|-----------|---------|
-| Reading files | Agent + file path |
-| Writing files | Agent + file path |
-| Searching | Agent + pattern |
-| Fetching URLs | Agent + URL |
-| Running commands | Agent + command |
-
-## Simplified Format
-
-For inline operations:
-
-```
-[mgr-supplier][haiku] → Glob: .codex/agents/*.md
-[mgr-creator][sonnet] → Write: .codex/agents/new-agent.md
-[mgr-updater][sonnet] → WebFetch: https://github.com/...
-```
-
-## Integration with R007
-
-R008 extends R007 for tool operations:
-
-```
-┌─ Agent: mgr-creator (manager)
-├─ Model: sonnet
-└─ Task: Creating new agent
-
-[mgr-creator][sonnet] → Write: .codex/agents/new-agent.md
-
-[Done] Agent created successfully
-```
-
-## Task Tool Display Format
+| Category | Tools | Verb |
+|----------|-------|------|
+| File Read | Read, Glob, Grep | Reading / Searching |
+| File Write | Write, Edit | Writing / Editing |
+| Network | WebFetch | Fetching |
+| Execution | Bash, Task | Running / Spawning |
 
-When spawning subagents via the Task tool, the display MUST use the actual `subagent_type` parameter:
+## Task Tool Format
 
 ```
 Task(subagent_type):model → description
 ```
 
-### Examples
+`subagent_type` MUST match actual Task tool parameter. Custom names not allowed.
+
+## Example
 
 ```
-[secretary][opus] → Spawning parallel agents:
+[mgr-creator][sonnet] → Write: .claude/agents/new-agent.md
+[secretary][opus] → Spawning:
   [1] Task(lang-golang-expert):sonnet → Go code review
   [2] Task(lang-python-expert):sonnet → Python code review
-  [3] Task(general-purpose):haiku → File validation
 ```
-
-### Rules
-
-- `subagent_type` MUST match the actual Task tool parameter value
-- Custom/invented names are NOT allowed (e.g., `code-review:sonnet` is WRONG)
-- Model specification shows cost/performance intent
-
-## Benefits
-
-1. **Debugging**: Know which agent performed which operation
-2. **Transparency**: User sees all agent activities
-3. **Audit Trail**: Track file changes by agent
-4. **Error Attribution**: Identify which agent caused issues
-
-## Implementation
-
-Agents should:
-1. Identify themselves before tool operations
-2. Include tool name and target
-3. Use consistent format across all tools
-4. Group related operations together