omcustomcodex-sauron-watch

name	omcustomcodex:sauron-watch
description	Full R017 verification with cost-aware 5+3 rounds before commit
scope	harness
disable-model-invocation	false
user-invocable	true

Sauron Watch Skill

Execute full R017 verification process with 5 rounds of manager agent verification and 3 rounds of deep review.

Purpose

Ensure complete synchronization of agents, skills, documentation, and project structure before committing changes.

Workflow

Phase 1: Manager Agent Verification (5 rounds)

Round 1-2: Basic Checks

□ mgr-supplier:audit - Check all agent dependencies and skill refs
□ mgr-updater:docs - Verify documentation sync
□ Fix any issues found

Round 3-4: Conditional Re-verify + Update

Round 1 and Round 2 both report exactly 0 issues before these rounds may be skipped. Record both as SKIPPED (clean). Any warning, issue, execution error, or indeterminate result requires Round 3 and Round 4 to run.

□ Exact clean: Round 3 — SKIPPED (clean); Round 4 — SKIPPED (clean)
□ Otherwise: mgr-supplier:audit - Re-verify after fixes
□ Otherwise: mgr-updater:docs - Re-run and apply any detected changes
□ Otherwise: fix any remaining issues

Round 5: Deterministic Evidence + Semantic Verification

□ [script] bash .github/scripts/verify-template-sync.sh
□ [script] bash .github/scripts/verify-wiki-sync.sh
□ [script] bash .github/scripts/verify-version-sync.sh
□ [script] bash .github/scripts/verify-fork-list.sh
□ [script] bun run .github/scripts/validate-docs.ts --programmatic-only
□ Consume successful script output for deterministic count/template/wiki/version/docs checks
□ All frontmatter valid
□ All skill refs exist
□ All memory scopes valid (project|user|local)
□ Routing patterns updated

Phase 2: Deep Review (3 rounds)

Deep Review rounds are never skipped, including after an exact-clean manager path.

Deep Round 1: Workflow Alignment

□ Agent workflows match purpose
□ Command definitions match implementations
□ Routing skill patterns are valid
□ All routing skills have complete agent mappings

Structural Lint Rules

In addition to workflow alignment checks, verify these structural invariants:

Lint 1: Routing Coverage

For each agent in .codex/agents/*.md:
  Check if agent name appears in at least one routing skill:
    - secretary-routing/SKILL.md
    - dev-lead-routing/SKILL.md
    - de-lead-routing/SKILL.md
    - qa-lead-routing/SKILL.md

  If not found in any routing skill:
    WARN: "{agent} is unreachable — not referenced in any routing skill"

Lint 2: Orphan Skill Detection

For each skill in .codex/skills/*/SKILL.md:
  Check if skill name appears in:
    - Any agent's skills: [] frontmatter field
    - Any routing skill's content
    - AGENTS.md command table (for user-invocable skills)

  If not referenced anywhere:
    WARN: "{skill} is orphaned — not referenced by any agent or routing skill"

Lint 3: Circular Dependency Check

Build dependency graph:
  For each agent → extract skills: [] references
  For each skill → extract agent references in body

  Detect cycles: agent-A → skill-X → agent-A
  If cycle found:
    ERROR: "Circular dependency: {cycle path}"

Lint 4: Context Fork Cap Verification

Count skills with context: fork in frontmatter:
  grep "context: fork" .codex/skills/*/SKILL.md

  If count > 12:
    ERROR: "Context fork cap exceeded: {count}/12"
  If count >= 10:
    WARN: "Context fork usage high: {count}/12 — only {12-count} slots remaining"

Lint 5: R006 Fork List Cross-Validation

Run: bash .github/scripts/verify-fork-list.sh

Compare:
  - R006 Context Fork Criteria current count/list
  - Actual .codex/skills/*/SKILL.md frontmatter with context: fork

If count or list differs:
  ERROR: "R006 fork list drift detected"

All structural lints are advisory (WARN level) except circular dependencies and fork cap exceeded (ERROR level — should block commit).

Deep Round 2: Reference Verification

□ All skill references exist
□ All agent frontmatter valid
□ memory field values valid (user | project | local)
□ No orphaned agents
□ No circular references

Deep Round 3: Philosophy Compliance

□ R006: Agent design rules (including memory field spec)
□ R007: Agent identification rules
□ R008: Tool identification rules
□ R009: Parallel execution rules
□ R010: Orchestrator coordination rules
□ R011: Memory integration (native-first architecture)
□ All MUST rules enforced, SHOULD rules recommended

Spec Density Check (Advisory)

Check each agent's body-to-skill ratio to detect agents with too much inline implementation detail:

For each agent in .codex/agents/*.md:
  agent_body_LOC = count non-frontmatter, non-blank lines in agent body
  referenced_skills = extract skills from frontmatter skills: field
  total_skill_LOC = sum of LOC in each referenced SKILL.md

  if total_skill_LOC > 0:
    spec_density = agent_body_LOC / total_skill_LOC
    if spec_density > 0.5:
      WARN: "{agent} has spec density {spec_density:.2f} — consider extracting inline details to skills"
  else:
    if agent_body_LOC > 50:
      INFO: "{agent} has {agent_body_LOC} LOC with no skill references — consider creating skills"

Thresholds:

Density	Status	Meaning
≤ 0.3	Good	Agent properly delegates to skills
0.3-0.5	OK	Acceptable inline detail
> 0.5	WARNING	Too much implementation in agent body

This check is advisory only — it does not block commits.

Phase 2.5: Documentation Accuracy

□ Every agent name in AGENTS.md matches actual filename
□ All counts cross-verified against filesystem
□ Every slash command has corresponding skill
□ Every agent reachable through routing skills

Phase 3: Fix Issues

□ Auto-fix: count mismatches, missing fields, outdated refs
□ Report: missing files, invalid scopes, philosophy violations
□ Re-run verification if major fixes made

Phase 4: Commit Ready

□ All verification passed
□ Ready to delegate to mgr-gitnerd for commit

Output Format

[mgr-sauron:watch]

Starting full R017 verification...

═══════════════════════════════════════════════════════════
 PHASE 1: Manager Agent Verification (5 rounds)
═══════════════════════════════════════════════════════════

[Round 1/5] mgr-supplier:audit
  ✓ 41 agents checked, 0 issues

[Round 2/5] mgr-updater:docs
  ✓ Documentation sync: OK

[Round 3/5] Re-verify: mgr-supplier:audit
  — SKIPPED (clean): Round 1 and Round 2 both reported exactly 0 issues

[Round 4/5] Re-verify: mgr-updater:docs
  — SKIPPED (clean): no warning, issue, execution error, or indeterminate result

[Round 5/5] Deterministic evidence + semantic verification
  ✓ [script] verify-template-sync.sh
  ✓ [script] verify-wiki-sync.sh
  ✓ [script] verify-version-sync.sh
  ✓ [script] verify-fork-list.sh
  ✓ [script] validate-docs.ts --programmatic-only
  ✓ All frontmatter valid
  ✓ All skill refs valid
  ✓ All memory scopes valid

═══════════════════════════════════════════════════════════
 PHASE 2: Deep Review (3 rounds)
═══════════════════════════════════════════════════════════

[Round 1/3] Workflow alignment
  ✓ All routing skills have complete agent mappings
  ✓ Command definitions match implementations

[Round 2/3] Reference verification
  ✓ All skill references valid
  ✓ No orphaned agents

[Round 3/3] Philosophy compliance
  ✓ R006 separation enforced
  ✓ R009 parallel execution enabled
  ✓ R010 orchestrator coordination documented
  ✓ R007/R008 identification rules present

═══════════════════════════════════════════════════════════
 VERIFICATION COMPLETE
═══════════════════════════════════════════════════════════

Status: ✓ ALL CHECKS PASSED

Ready to commit. 커밋할까요?

Related

• R017: Sync Verification Rules
• mgr-gitnerd: Git operations agent