@event4u/agent-config 14.17.0 → 14.19.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.
Files changed (184) hide show
  1. package/.augment-plugin/marketplace.json +3 -3
  2. package/.augment-plugin/plugin.json +1 -1
  3. package/.claude-plugin/marketplace.json +1 -1
  4. package/CHANGELOG.md +207 -77
  5. package/README.md +72 -150
  6. package/dist/agent-src/README.md +1 -1
  7. package/dist/agent-src/commands/agent-handoff.md +2 -2
  8. package/dist/agent-src/commands/pr/merge.md +35 -44
  9. package/dist/agent-src/commands/review/changes.md +19 -19
  10. package/dist/agent-src/commands/review.md +2 -2
  11. package/dist/agent-src/contexts/execution/host-capability-manifest.md +30 -3
  12. package/dist/agent-src/rules/autonomous-execution.md +1 -1
  13. package/dist/agent-src/rules/git-history-discipline.md +2 -2
  14. package/dist/agent-src/rules/guidelines.md +1 -1
  15. package/dist/agent-src/rules/session-canary.md +2 -2
  16. package/dist/agent-src/rules/token-efficiency.md +1 -1
  17. package/dist/agent-src/skills/accessibility-auditor/SKILL.md +2 -0
  18. package/dist/agent-src/skills/adversarial-review/SKILL.md +0 -8
  19. package/dist/agent-src/skills/ai-code-blindspots/SKILL.md +27 -1
  20. package/dist/agent-src/skills/canvas-design/SKILL.md +1 -1
  21. package/dist/agent-src/skills/code-intelligence/SKILL.md +60 -12
  22. package/dist/agent-src/skills/complexity-first-planning/SKILL.md +27 -0
  23. package/dist/agent-src/skills/design-intelligence/SKILL.md +1 -1
  24. package/dist/agent-src/skills/design-intelligence/data/app-interface.csv +1 -1
  25. package/dist/agent-src/skills/design-intelligence/data/motion.csv +17 -17
  26. package/dist/agent-src/skills/design-intelligence/data/ux-guidelines.csv +2 -2
  27. package/dist/agent-src/skills/design-intelligence/references/context-and-registers.md +17 -0
  28. package/dist/agent-src/skills/design-intelligence/references/design-rules-checklist.md +4 -4
  29. package/dist/agent-src/skills/design-review/SKILL.md +3 -1
  30. package/dist/agent-src/skills/design-variations/SKILL.md +1 -1
  31. package/dist/agent-src/skills/existing-ui-audit/SKILL.md +1 -1
  32. package/dist/agent-src/skills/fe-design/SKILL.md +32 -5
  33. package/dist/agent-src/skills/fe-design/references/craft-floor.md +7 -0
  34. package/dist/agent-src/skills/fe-design/references/design-patterns.md +47 -2
  35. package/dist/agent-src/skills/feature-planning/SKILL.md +34 -0
  36. package/dist/agent-src/skills/git-workflow/SKILL.md +37 -12
  37. package/dist/agent-src/skills/git-workflow/references/push-closes-its-loop.md +176 -0
  38. package/dist/agent-src/skills/laravel-mail/SKILL.md +44 -0
  39. package/dist/agent-src/skills/laravel-scheduling/SKILL.md +3 -1
  40. package/dist/agent-src/skills/project-analyzer/SKILL.md +0 -6
  41. package/dist/agent-src/skills/react-native-setup/SKILL.md +3 -3
  42. package/dist/agent-src/skills/react-shadcn-ui/SKILL.md +1 -1
  43. package/dist/agent-src/skills/sequential-thinking/SKILL.md +18 -8
  44. package/dist/agent-src/skills/tailwind-engineer/SKILL.md +1 -1
  45. package/dist/agent-src/skills/testing-anti-patterns/SKILL.md +40 -0
  46. package/dist/agent-src/skills/ui-apply-generic/SKILL.md +1 -1
  47. package/dist/agent-src/skills/ui-component-architect/SKILL.md +1 -1
  48. package/dist/agent-src/skills/using-git-worktrees/SKILL.md +29 -2
  49. package/dist/agent-src/skills/verify-repair-loop/SKILL.md +53 -8
  50. package/dist/agent-src/skills/worktree-lifecycle/SKILL.md +3 -1
  51. package/dist/agent-src/templates/agents/agent-project-settings.example.yml +1 -1
  52. package/dist/cli/commands/recordTriggerEval.js +2 -2
  53. package/dist/cli/main.js +1 -1
  54. package/dist/cli-delegate/{chunk-AKJGUQNO.js → chunk-GZTMRYCN.js} +405 -341
  55. package/dist/cli-delegate/{chunk-7RXIKF62.js → chunk-IDHUQIXP.js} +283 -132
  56. package/dist/cli-delegate/cmd_adr_effective.js +318 -81
  57. package/dist/cli-delegate/cmd_conformance.js +749 -669
  58. package/dist/cli-delegate/cmd_doctor.js +1 -1
  59. package/dist/cli-delegate/cmd_export.js +1 -1
  60. package/dist/cli-delegate/cmd_handoff.js +1 -1
  61. package/dist/cli-delegate/cmd_prune.js +1 -1
  62. package/dist/cli-delegate/cmd_route_audit.js +1 -1
  63. package/dist/cli-delegate/cmd_settings_migrate.js +1 -1
  64. package/dist/cli-delegate/cmd_sync.js +1 -1
  65. package/dist/cli-delegate/cmd_uninstall.js +1 -1
  66. package/dist/cli-delegate/cmd_validate.js +1 -1
  67. package/dist/discovery/deprecation-report.md +1 -1
  68. package/dist/discovery/discovery-manifest.json +39 -38
  69. package/dist/discovery/discovery-manifest.json.sha256 +1 -1
  70. package/dist/discovery/discovery-manifest.summary.md +1 -1
  71. package/dist/discovery/dormancy-report.md +1 -1
  72. package/dist/discovery/orphan-report.md +1 -1
  73. package/dist/discovery/packs.json +2 -2
  74. package/dist/discovery/trust-report.md +1 -1
  75. package/dist/discovery/workspaces.json +2 -2
  76. package/dist/hooks/dispatch.js +2764 -2439
  77. package/dist/install/install.mjs +588 -527
  78. package/dist/mcp/registry-manifest.json +2 -2
  79. package/dist/mcp/server.json +2 -2
  80. package/dist/mcp/server.mjs +1381 -1138
  81. package/dist/scripts/_lib/model_tier.js +1 -1
  82. package/dist/shared/settingsConsent.js +1 -1
  83. package/docs/guidelines/agent-infra/evaluator-independence-mechanics.md +1 -1
  84. package/docs/guidelines/agent-infra/roadmap-progress-mechanics.md +1 -1
  85. package/docs/guidelines/design-antipatterns.md +87 -64
  86. package/package.json +1 -1
  87. package/src/agent-src/README.md +1 -1
  88. package/src/config/ci-local-parity.yml +6 -0
  89. package/src/config/conformance-claim-baseline.json +9 -0
  90. package/src/config/gate-coverage.yml +170 -0
  91. package/src/config/gate-violation-baselines.json +4 -4
  92. package/src/config/release-gate-locality.yml +141 -0
  93. package/src/scripts/_cli/cmd_doctor.ts +2 -2
  94. package/src/scripts/_cli/cmd_handoff.ts +1 -1
  95. package/src/scripts/_cli/cmd_update.ts +1 -1
  96. package/src/scripts/_cli/cmd_versions.ts +1 -1
  97. package/src/scripts/_lib/adr_frontmatter.ts +44 -0
  98. package/src/scripts/_lib/consequence_objects.ts +152 -0
  99. package/src/scripts/_lib/exec_evidence.ts +7 -0
  100. package/src/scripts/_lib/global_deploy_inventory.ts +2 -2
  101. package/src/scripts/_lib/host_permission_checks.ts +251 -0
  102. package/src/scripts/_lib/model_tier.ts +1 -1
  103. package/src/scripts/_lib/obligation_frequency.ts +2 -2
  104. package/src/scripts/_lib/release_highlights.ts +265 -1
  105. package/src/scripts/_lib/release_material.ts +129 -0
  106. package/src/scripts/_lib/repeated_run.ts +77 -0
  107. package/src/scripts/_lib/runtime_journal.ts +26 -3
  108. package/src/scripts/_lib/runtime_wiring_checks.ts +12 -1
  109. package/src/scripts/_lib/session_register.ts +11 -6
  110. package/src/scripts/_lib/sqlite_schema.ts +22 -0
  111. package/src/scripts/_lib/surface.ts +86 -0
  112. package/src/scripts/_lib/tamper_vocabulary.ts +242 -0
  113. package/src/scripts/_lib/ui_authority.ts +116 -0
  114. package/src/scripts/_lib/unified_diff.ts +121 -0
  115. package/src/scripts/adr/regenerate_index.ts +54 -1
  116. package/src/scripts/adr_cite_check.ts +71 -15
  117. package/src/scripts/ai_council/clients.ts +1 -1
  118. package/src/scripts/autonomy_friction_corpus.ts +229 -0
  119. package/src/scripts/build_proof.ts +15 -2
  120. package/src/scripts/check_beta_review_markers.ts +141 -17
  121. package/src/scripts/check_enforcement_coverage.ts +1 -1
  122. package/src/scripts/check_enforcement_denominator.ts +121 -11
  123. package/src/scripts/check_finding_dispositions.ts +239 -6
  124. package/src/scripts/check_gate_coverage.ts +96 -0
  125. package/src/scripts/check_installed_hooks_fresh.ts +381 -0
  126. package/src/scripts/check_release_highlights.ts +99 -14
  127. package/src/scripts/check_skill_activation_claim.ts +179 -0
  128. package/src/scripts/check_suppression_hygiene.ts +11 -0
  129. package/src/scripts/check_trigger_evals.ts +1 -1
  130. package/src/scripts/code_graph/build.ts +230 -33
  131. package/src/scripts/code_graph/extract.ts +168 -33
  132. package/src/scripts/code_graph/types.ts +37 -2
  133. package/src/scripts/code_graph/validate.ts +12 -1
  134. package/src/scripts/conformance_scan.ts +45 -4
  135. package/src/scripts/detect_silent_catch.ts +403 -0
  136. package/src/scripts/detect_verification_tampering.ts +538 -0
  137. package/src/scripts/explain_run.ts +1 -1
  138. package/src/scripts/generate_capability_matrix.ts +1 -1
  139. package/src/scripts/git_authorization_hook.ts +228 -18
  140. package/src/scripts/hook_manifest.json +1 -1
  141. package/src/scripts/hook_manifest.yaml +47 -30
  142. package/src/scripts/hooks/augment-dispatcher.sh +1 -1
  143. package/src/scripts/hooks/block_no_verify.ts +3 -3
  144. package/src/scripts/hooks/category_a.ts +291 -0
  145. package/src/scripts/hooks/cline-dispatcher.sh +1 -1
  146. package/src/scripts/hooks/concern_registry.ts +2 -2
  147. package/src/scripts/hooks/cursor-dispatcher.sh +11 -5
  148. package/src/scripts/hooks/dispatch_hook.ts +17 -22
  149. package/src/scripts/hooks/evidence_independence.ts +21 -2
  150. package/src/scripts/hooks/{block_unauthorized_git.ts → git_command_classifier.ts} +175 -364
  151. package/src/scripts/hooks/host_lowering.ts +215 -0
  152. package/src/scripts/hooks/host_lowering.yaml +185 -0
  153. package/src/scripts/hooks/host_semantics.ts +105 -37
  154. package/src/scripts/hooks/journal_record_hook.ts +10 -0
  155. package/src/scripts/hooks/permission_gate.ts +117 -0
  156. package/src/scripts/hooks/push_settle_hook.ts +250 -0
  157. package/src/scripts/hooks/ship_diff_volume_hook.ts +1 -1
  158. package/src/scripts/hooks_status.ts +12 -10
  159. package/src/scripts/install-hooks.sh +134 -1
  160. package/src/scripts/install.ts +53 -82
  161. package/src/scripts/install_anthropic_key.sh +2 -2
  162. package/src/scripts/language_mirror_hook.ts +27 -5
  163. package/src/scripts/lint_design_antipattern_parity.ts +80 -0
  164. package/src/scripts/lint_evidence_artifacts.ts +45 -1
  165. package/src/scripts/lint_hook_manifest.ts +87 -3
  166. package/src/scripts/lint_motion_authority_drift.ts +731 -0
  167. package/src/scripts/lint_readme_size.ts +158 -12
  168. package/src/scripts/lint_reasoning_fixtures.ts +216 -0
  169. package/src/scripts/lint_skill_trigger_contradiction.ts +161 -0
  170. package/src/scripts/measure_sibling_naming.ts +150 -0
  171. package/src/scripts/redact_hook_capture.ts +1 -1
  172. package/src/scripts/release.ts +31 -143
  173. package/src/scripts/release_drill.ts +71 -1
  174. package/src/scripts/release_publication.ts +243 -3
  175. package/src/scripts/release_verify.ts +220 -0
  176. package/src/scripts/report_skill_activation.ts +319 -7
  177. package/src/scripts/schemas/review-findings.schema.json +6 -1
  178. package/src/scripts/schemas/skill.schema.json +12 -2
  179. package/src/scripts/self_review_gate.ts +10 -1
  180. package/src/scripts/session_canary_hook.ts +1 -1
  181. package/src/scripts/session_register_hook.ts +4 -3
  182. package/src/scripts/trigger_eval_grandfather.json +5 -1
  183. package/src/scripts/validate_agent_settings.ts +1 -1
  184. package/src/shared/settingsConsent.ts +1 -1
package/README.md CHANGED
@@ -18,11 +18,30 @@
18
18
 
19
19
  **A deep library of skills, commands and governed rules** — plus a capability router that loads the right skill on intent and multi-agent orchestration with consensus review. The whole layer is compiled into **20 host agents** — of 23 detected, 3 being export-only<!-- claim:host-agent-count --> (Claude Code, Cursor, Augment, Cline, Windsurf, Copilot, Gemini CLI, Codex, Continue, Zed, JetBrains, Aider and more). **Resident processes are permitted only under the supervision contract ADR-249 establishes** — a policy this repository adopted on 2026-08-27, not a description of anything running today. Six role-shaped entry paths sit on top, so any host becomes a reliable team member — without locking you to a single model or vendor.
20
20
 
21
+ ## Try it in 30 seconds
22
+
23
+ **Try one thing in 30 seconds** — before the full suite, drop in a single
24
+ self-contained subagent and see the discipline on your own repo:
25
+
26
+ ```bash
27
+ mkdir -p .claude/agents
28
+ curl -fsSL https://raw.githubusercontent.com/event4u-app/agent-config/main/docs/wedge/production-validator/production-validator.md \
29
+ -o .claude/agents/production-validator.md
30
+ # then in Claude Code: @production-validator check this branch is actually done
31
+ ```
32
+
33
+ `production-validator` is read-only and installs nothing else — it gates "done"
34
+ by hunting mocks/stubs on the shipped path and demanding real-system evidence
35
+ ([what it does](docs/wedge/production-validator/README.md)). Like it? The full
36
+ suite installs in one command — [Quickstart ↓](#quickstart).
37
+
38
+ ---
39
+
21
40
  ### What's different
22
41
 
23
- It is both deep **and** disciplined — and honest about what it deliberately is not:
42
+ Deep **and** disciplined, and honest about what it deliberately is not:
24
43
 
25
- - **Depth that routes itself** — a deep skill and command library, with a capability router that loads the right one on intent, not a 500-artefact context dump.
44
+ - **Depth that routes itself** — a capability router loads the right skill on intent, not a 500-artefact context dump.
26
45
  - **Governance on every host** — rules compiled into each tool's native format at projection time; deterministic runtime hooks added on hook-capable hosts. This config-space, host-agnostic governance is the moat ([the governance advantage](docs/governance-advantage.md) · [enforcement by host](docs/enforcement-by-host.md)).
27
46
  - **Surgical uninstall** — removes only its own keys from a shared host config (matched by JSON-pointer + SHA-256), never a neighbour tool's entries.<!-- claim:surgical-uninstall -->
28
47
  - **Pack-scoped install** — writes the active pack only, not a 500-artefact dump.
@@ -39,41 +58,31 @@ See exactly [what works on which host](docs/capability-matrix.md) or jump to [th
39
58
 
40
59
  ### Pick your profile — six entry paths
41
60
 
42
- `agent-config setup` writes `profile.id` to `.agent-settings.yml`; each
43
- anchor below is the first-screen the wizard sends you to. One README,
44
- six entries, no role-detection guesswork.
45
-
46
- | Profile (`profile.id`) | Audience | First commands | First skills |
47
- |---|---|---|---|
48
- | 👩‍💻 [`developer`](docs/profiles.md#profile-developer) | IC engineer | `/implement-ticket` · `/work` · `/review-changes` · `/fix` · `/commit` | `developer-like-execution` · `verify-completion-evidence` · `minimal-safe-diff` · `systematic-debugging` · `test-driven-development` |
49
- | ✍️ [`content_creator`](docs/profiles.md#profile-content_creator) | Writers, ghostwriters, marketers | `/work` · `/post-as` · `/ghostwriter` · `/optimize-prompt` · `/video:from-script` · `/video:storyboard` | `voice-and-tone-design` · `messaging-architecture` · `editorial-calendar` · `release-comms` · `character-consistency` |
50
- | 🚀 [`founder`](docs/profiles.md#profile-founder) | Solo / early-stage founder | `/work` · `/feature` · `/challenge-me` · `/council` | `refine-prompt` · `rice-prioritization` · `vision-articulation` · `fundraising-narrative` · `runway-cognition` |
51
- | 🏛 [`agency`](docs/profiles.md#profile-agency) | Multi-client delivery shop | `/work` · `/implement-ticket` · `/refine-ticket` · `/feature` · `/roadmap` | `doc-coauthoring` · `decision-record` · `refine-ticket` · `estimate-ticket` · `perf-feedback-craft` |
52
- | 💼 [`finance`](docs/profiles.md#profile-finance) | CFO / fractional finance / FP&A | `/work` · `/council` · `/challenge-me` | `dcf-modeling` · `forecasting` · `scenario-modeling` · `unit-economics-modeling` · `runway-cognition` |
53
- | 🛡 [`ops`](docs/profiles.md#profile-ops) | RevOps, support, SRE-adjacent | `/work` · `/threat-model` · `/review-changes` · `/fix` | `incident-commander` · `dashboard-design` · `logging-monitoring` · `threat-modeling` · `launch-readiness` |
54
-
55
- **Not sure which one?** Run `npx @event4u/agent-config init` then
56
- `agent-config setup` — the browser wizard asks a single 8-option role
57
- question and maps to the closest profile. Source-of-truth:
58
- [`src/agent-src/profiles/`](src/agent-src/profiles/) ·
59
- schema: [`docs/contracts/profile-system.md`](docs/contracts/profile-system.md).
60
- Beyond software: [`user-types/`](src/agent-src/user-types/)
61
- (galabau · metalworking · truck — see [Beyond software](#beyond-software).
62
-
63
- **Per-profile experience pages** (who it's for · first tasks · packs + flows ·
64
- what is *not* loaded · examples):
65
- [developer](docs/experiences/developer.md) ·
66
- [content_creator](docs/experiences/content_creator.md) ·
67
- [founder](docs/experiences/founder.md) ·
68
- [agency](docs/experiences/agency.md) ·
69
- [finance](docs/experiences/finance.md) ·
70
- [ops](docs/experiences/ops.md).
61
+ `agent-config setup` writes `profile.id` to `.agent-settings.yml` and sends you
62
+ to that profile's first screen. Each experience page carries the full detail —
63
+ who it's for, first commands and skills, packs and flows, and what is
64
+ deliberately *not* loaded.
65
+
66
+ | Profile | Audience | Experience page |
67
+ |---|---|---|
68
+ | 👩‍💻 [`developer`](docs/profiles.md#profile-developer) | IC engineer | [page](docs/experiences/developer.md) |
69
+ | ✍️ [`content_creator`](docs/profiles.md#profile-content_creator) | Writers, ghostwriters, marketers | [page](docs/experiences/content_creator.md) |
70
+ | 🚀 [`founder`](docs/profiles.md#profile-founder) | Solo / early-stage founder | [page](docs/experiences/founder.md) |
71
+ | 🏛 [`agency`](docs/profiles.md#profile-agency) | Multi-client delivery shop | [page](docs/experiences/agency.md) |
72
+ | 💼 [`finance`](docs/profiles.md#profile-finance) | CFO / fractional finance / FP&A | [page](docs/experiences/finance.md) |
73
+ | 🛡 [`ops`](docs/profiles.md#profile-ops) | RevOps, support, SRE-adjacent | [page](docs/experiences/ops.md) |
74
+
75
+ **Not sure which one?** The wizard asks a single 8-option role question and maps
76
+ to the closest profile. Source-of-truth
77
+ [`src/agent-src/profiles/`](src/agent-src/profiles/) · schema
78
+ [`docs/contracts/profile-system.md`](docs/contracts/profile-system.md) · beyond
79
+ software [`user-types/`](src/agent-src/user-types/) (galabau · metalworking ·
80
+ truck — see [Beyond software](#beyond-software)).
71
81
 
72
82
  ### Workflows, not raw commands
73
83
 
74
- You don't memorize every command — you run a **work journey**. Four flows span the
75
- developer story end-to-end; each names the command you TYPE to start and the
76
- skills it composes:
84
+ You don't memorize commands — you run a **work journey**. Four flows span the
85
+ developer story end-to-end:
77
86
 
78
87
  | Flow | Start with | The journey |
79
88
  |---|---|---|
@@ -82,9 +91,7 @@ skills it composes:
82
91
  | 🔎 **Review** | `/review-changes` · `/judge` | self-review → judge → quality-fix → threat-model |
83
92
  | 🚢 **Delivery** | `/commit` · `/pr:create` | commit in chunks → open PR → answer review |
84
93
 
85
- Full detail — entry commands, canonical path, composed skills per flow:
86
- [`docs/flows.md`](docs/flows.md). (`agent-admin` — memory / analytics / config —
87
- is platform operation, not a user-work flow.)
94
+ Composed skills and the canonical path per flow: [`docs/flows.md`](docs/flows.md).
88
95
 
89
96
  <p align="center">
90
97
  <a href="CHANGELOG.md">CHANGELOG</a> ·
@@ -122,23 +129,19 @@ lists the end-to-end workflows (`/implement-ticket`, `/work`,
122
129
  Don't take the claims on trust — **verify them.** [`docs/proof.md`](docs/proof.md)
123
130
  is generated from source: a claim→evidence table (every public claim binds to a
124
131
  resolvable pointer or CI fails), honest-null benchmarks *including the runs where
125
- the package changed nothing* and a "verify it yourself" block you run on a fresh
126
- checkout. The proof page fails CI if it drifts from its sources — reproducibility
127
- is the proof. Browse it on the deployed docs site — the proof page is the primary entry:
128
- [event4u-app.github.io/agent-config/proof/](https://event4u-app.github.io/agent-config/proof/).
129
- The honest comparison frame lives at
130
- [`docs/us-vs-the-category.md`](docs/us-vs-the-category.md).
132
+ the package changed nothing*, and a "verify it yourself" block you run on a fresh
133
+ checkout. It fails CI if it drifts from its sources — reproducibility is the
134
+ proof. Read it at
135
+ [event4u-app.github.io/agent-config/proof/](https://event4u-app.github.io/agent-config/proof/),
136
+ with the comparison frame at [`docs/us-vs-the-category.md`](docs/us-vs-the-category.md).
131
137
  Freshest measured row: in one post-fix session, advisory context injection cut language-mirror violations 555 → 19 while the two blocking guards went 8 → 0 and 1 → 0 — advisory reduced massively, only blocking eliminated. One session and a post-hoc reading, so a recorded prior rather than a law.<!-- claim:conformance-advisory-vs-blocking -->
132
138
 
133
139
  Maintaining a skills catalog yourself? The **anti-reskin gate** that blocks
134
- find-replace re-skin PRs here runs on your catalog too —
135
- [`docs/anti-reskin-gate.md`](docs/anti-reskin-gate.md).
140
+ find-replace re-skin PRs here runs on yours too — [`docs/anti-reskin-gate.md`](docs/anti-reskin-gate.md).
136
141
 
137
- Audit-disciplined by construction — every memory consult, decision
138
- key and hook concern lands in `agents/runtime/state/` so you can replay it.
139
- [Core principles](#core-principles) names the four invariants;
140
- [What `agent-config` is — and what it isn't](#what-agent-config-is--and-what-it-isnt)
141
- draws the scope boundary.
142
+ Audit-disciplined by construction — every memory consult, decision key and hook
143
+ concern lands in `agents/runtime/state/` so you can replay it.
144
+ [Core principles](#core-principles) names the four invariants.
142
145
 
143
146
  ## Contribute
144
147
 
@@ -154,30 +157,13 @@ opt-in measurement loop. Source-of-truth tree is
154
157
 
155
158
  ## Quickstart
156
159
 
157
- **Try one thing in 30 seconds** — before the full suite, drop in a single
158
- self-contained subagent and see the discipline on your own repo:
159
-
160
- ```bash
161
- mkdir -p .claude/agents
162
- curl -fsSL https://raw.githubusercontent.com/event4u-app/agent-config/main/docs/wedge/production-validator/production-validator.md \
163
- -o .claude/agents/production-validator.md
164
- # then in Claude Code: @production-validator check this branch is actually done
165
- ```
166
-
167
- `production-validator` is read-only and installs nothing else — it gates "done"
168
- by hunting mocks/stubs on the shipped path and demanding real-system evidence
169
- ([what it does](docs/wedge/production-validator/README.md)). Like it? Install the
170
- full suite:
171
-
172
160
  **One command. Detection-driven — your installed AI tools are found and
173
161
  pre-selected. Nothing is written until you click Finish. No YAML by hand.**
174
162
 
175
- Those four are structural: they hold on every run, because they are properties
176
- of the code path rather than of your machine. How *long* it takes is not one of
177
- them — that is dominated by network and registry latency, which we do not
178
- control. The install → `doctor` wall-clock is measured by CI on every umbrella
179
- run and published with its conditions, as evidence; it is never a promised
180
- number.
163
+ Those four are structural — properties of the code path, not of your machine.
164
+ How *long* it takes is not: that is dominated by network and registry latency.
165
+ CI measures the install → `doctor` wall-clock on every umbrella run and
166
+ publishes it with its conditions, as evidence rather than as a promise.
181
167
 
182
168
  ```bash
183
169
  # 1. Install — on a terminal with a display, the browser wizard launches
@@ -191,18 +177,17 @@ npx -y @event4u/agent-config init
191
177
  /work "your first real task"
192
178
  ```
193
179
 
194
- **Headless / CI:** `init` skips the GUI automatically on CI, on a non-TTY, on a headless host, and whenever any CLI-mode flag is present — it then runs the non-interactive installer directly. The full opt-out set is listed once, against the code, in [`gui-wizard` § When the GUI is skipped](docs/contracts/gui-wizard.md#when-the-gui-is-skipped). Pass flags (`--profile=balanced --tools=claude-code,cursor`); add `--dry-run` to preview writes. The GUI and the CLI share one installer (`src/scripts/install.ts`), so both produce identical results. Reference: [`docs/wizard.md`](docs/wizard.md).
180
+ **Headless / CI:** `init` skips the GUI on CI, a non-TTY, a headless host, or any CLI-mode flag, and runs the non-interactive installer instead. The GUI and the CLI share one installer (`src/scripts/install.ts`), so both produce identical results. Flags, the full opt-out set and `--dry-run`: [`docs/wizard.md`](docs/wizard.md) · [`gui-wizard` § When the GUI is skipped](docs/contracts/gui-wizard.md#when-the-gui-is-skipped).
195
181
 
196
- **Pick specific AIs:** `--tools=claude-code,cursor,augment,windsurf,cline,gemini-cli,copilot,roocode,aider,codex,claude-desktop,continue` (any subset). Visual picker: add `--gui` (loopback-bound, CSRF-gated; contract [`gui-wizard`](docs/contracts/gui-wizard.md)). `--gui` is an opt-in that forces the wizard past the TTY and headless checks — it does **not** override `CI`, `AGENT_CONFIG_NO_UI`, or a CLI-mode flag; combining it with one of those exits non-zero rather than quietly running the CLI install. On a headless host add `--allow-headless` and connect a browser to the printed URL.
182
+ **Pick specific AIs:** `--tools=claude-code,cursor,augment,…` (any subset). `--gui` forces the loopback-bound, CSRF-gated picker past the TTY and headless checks; it does **not** override `CI`, `AGENT_CONFIG_NO_UI` or a CLI-mode flag, and combining it with one of those exits non-zero rather than quietly running the CLI install.
197
183
 
198
- **Verify hook coverage:** `npx @event4u/agent-config hooks:status` prints the per-platform matrix (`--strict` for CI, `--format json` for tooling).
184
+ **Verify hook coverage:** `npx @event4u/agent-config hooks:status` (`--strict` for CI, `--format json` for tooling).
199
185
 
200
- > **Scope (v2.5+):** `init` writes **global** only — `~/.event4u/agent-config/`, `~/.claude/`, `~/.cursor/`, …. The project tree gets `agents/overrides/` only (the bridge marker was retired — ADR-020 amendment 2026-07-13; the global root resolves from `~/.event4u/agent-config`). `--project` is maintainer-only behind `AGENT_CONFIG_DEV_MODE=1` ([ADR-020](docs/decisions/ADR-020-global-only-consumer-scope.md), [dev-mode](docs/maintainers/dev-mode.md)).
186
+ > **Scope (v2.5+):** `init` writes **global** only — `~/.event4u/agent-config/`, `~/.claude/`, `~/.cursor/`, …. The project tree gets `agents/overrides/` only. `--project` is maintainer-only behind `AGENT_CONFIG_DEV_MODE=1` ([ADR-020](docs/decisions/ADR-020-global-only-consumer-scope.md), [dev-mode](docs/maintainers/dev-mode.md)).
201
187
 
202
- Migrating from a v1.x install? `npx @event4u/agent-config migrate` — full notes in [`docs/migration/v1-to-v2.md`](docs/migration/v1-to-v2.md).
188
+ Migrating from v1.x? `npx @event4u/agent-config migrate` — [`docs/migration/v1-to-v2.md`](docs/migration/v1-to-v2.md).
203
189
 
204
190
  ---
205
-
206
191
  ## What `agent-config` is — and what it isn't
207
192
 
208
193
  A **content layer** — skills, rules, commands, guidelines, personas — distributed via npm and projected into every supported AI tool's native config format. It follows the [Agent Skills open standard](https://agentskills.io).
@@ -316,7 +301,7 @@ Create the user file interactively: `/agents user init` ([schema](docs/contracts
316
301
 
317
302
  ### Self-hosted MCP on Cloudflare — zero local install
318
303
 
319
- Skills, commands, rules and guidelines can be served as an MCP endpoint from your own Cloudflare Worker — any MCP client (Claude Desktop, Claude Code, Cursor, Zed, Continue, hosted agents) talks to it over HTTP. Two auth modes: `public` (default, OSS read-only deploys) and `bearer-auth` (operator opt-in, `MCP-Token` Wrangler secret).
304
+ Skills, commands, rules and guidelines can be served as an MCP endpoint from your own Cloudflare Worker, reachable over HTTP by any MCP client. Two auth modes: `public` (default) and `bearer-auth` (operator opt-in, `MCP-Token` Wrangler secret).
320
305
 
321
306
  ```bash
322
307
  task mcp:cloud:login # one-time, opens browser
@@ -328,7 +313,7 @@ task mcp:cloud:secret-put # opt in to bearer-auth (recommended for private de
328
313
 
329
314
  > **Scope — Lite, not Full.** The Worker serves read-only governance (skills · commands · rules · guidelines · contexts) as MCP prompts and resources, plus small read-only tools (`memory_lookup`, `chat_history_read`, `list_*`). It does **not** execute the repository's local scripts (linters, audits, `task ci`, work-engine hooks) — those require local install per [Quickstart](#quickstart).
330
315
 
331
- > The built-in **local stdio** server is listed for discovery in the [Glama MCP Registry](https://glama.ai/mcp/servers/event4u-app/agent-config) (agent developers / contributors; requires a local checkout, not a turnkey install — see [ADR-067](docs/decisions/ADR-067-glama-registry-listing.md)).
316
+ > The built-in **local stdio** server is listed in the [Glama MCP Registry](https://glama.ai/mcp/servers/event4u-app/agent-config) — it requires a local checkout, not a turnkey install ([ADR-067](docs/decisions/ADR-067-glama-registry-listing.md)).
332
317
 
333
318
  ### Deployment posture
334
319
 
@@ -338,15 +323,13 @@ task mcp:cloud:secret-put # opt in to bearer-auth (recommended for private de
338
323
  | **Small team (3–10 people)** | ✅ today | Shared `agents/overrides/` Git repo + shared NAS for knowledge — no code change, no new server. Recipe: [`docs/deploy/small-team-recipe.md`](docs/deploy/small-team-recipe.md) |
339
324
  | **Organization mode** (SSO · central policy · team context · internal connectors) | ⏸ not started | Each shape gated on a recruited customer + funded audit + maintainer ADR. Posture rationale: [`docs/deploy/team-deployment-posture.md`](docs/deploy/team-deployment-posture.md) |
340
325
 
341
- The Hard Floor on organization-mode features (SSO, central policy, OAuth connectors, team-context) is preserved by design — they stay cancelled until a real first customer + funded security audit lifts them. The small-team recipe is the supported path in the meantime.
342
-
343
- > *The 9.3/10 feedback round (2026-05-25) re-asked for OAuth knowledge connectors, IAM / org governance and organization-shared memory. Each is a stable cancellation row in [`team-deployment-posture`](docs/deploy/team-deployment-posture.md) under the same three release gates — recruited team customer · funded audit · maintainer ADR.*
326
+ Organization-mode features (SSO, central policy, OAuth connectors, team-context) stay cancelled by design until a recruited customer and a funded security audit lift them; the small-team recipe is the supported path meanwhile. Each is a stable cancellation row in [`team-deployment-posture`](docs/deploy/team-deployment-posture.md).
344
327
 
345
328
  ---
346
329
 
347
330
  ## Harness expectations
348
331
 
349
- Three classes of install/runtime behaviour look like package bugs but are host-harness behaviour the package cannot control — sibling-plugin namespaces (`codex:*`, `cc-gemini-plugin:*`), deferred tools surfaced via `ToolSearch` and cross-scope skill drift (real bug, fixed in the distribution-channels track). Diagnostics + the package's response: [`docs/contracts/harness-expectations.md`](docs/contracts/harness-expectations.md). First step when a skill appears twice: `task probe:skills`.
332
+ Three classes of install/runtime behaviour look like package bugs and are host-harness behaviour the package cannot control: sibling-plugin namespaces, deferred tools surfaced via `ToolSearch`, and cross-scope skill drift. Diagnostics and the package's response: [`docs/contracts/harness-expectations.md`](docs/contracts/harness-expectations.md). When a skill appears twice, start with `task probe:skills`.
350
333
 
351
334
  ## Supported tools
352
335
 
@@ -431,7 +414,7 @@ accepts a host-supplied config root so its own settings stay profile-scoped.
431
414
 
432
415
  ## Who this is for
433
416
 
434
- Stack-agnostic governance core (orchestration · role modes · command clusters · quality gates · audit-discipline) plus parallel stack-specific skill sets:
417
+ A stack-agnostic governance core (orchestration · role modes · command clusters · quality gates · audit discipline), plus stack-specific skill sets:
435
418
 
436
419
  | Stack | Coverage |
437
420
  |---|---|
@@ -455,7 +438,7 @@ Three domain-safety rules ([`domain-safety-pii`](src/rules/domain-safety-pii.md)
455
438
 
456
439
  ### Code provenance & license governance
457
440
 
458
- Every diff is checked against a **license policy derived from the target repo's own detected license** (`LICENSE`/`package.json`/`composer.json`, precedence-ordered; sources disagree → escalate, never guess) and a **strict linter over our own borrow ledger** ([`provenance/borrows.jsonl`](provenance/) → [`docs/THIRD-PARTY-NOTICES.md`](docs/THIRD-PARTY-NOTICES.md)) that fails a deny-class license, an unknown license, a missing transformation note or a rename-only-phrased one — wired into `ci`/`ci-strict` from day one. A third piece, `license-compliance-audit`, runs an offline/online similarity scan on demand — a human invokes it deliberately, never a pipeline. This is **provenance-governed**, **license-policy-enforced** borrow discipline backed by an **audited borrow trail** — not a copy detector.
441
+ Every diff is checked against a **license policy derived from the target repo's own detected license** (precedence-ordered; sources disagree → escalate, never guess) and a **strict linter over our borrow ledger** ([`provenance/borrows.jsonl`](provenance/) → [`docs/THIRD-PARTY-NOTICES.md`](docs/THIRD-PARTY-NOTICES.md)) that fails a deny-class or unknown license, a missing transformation note, or a rename-only-phrased one — wired into `ci`/`ci-strict`. `license-compliance-audit` runs a similarity scan on demand, invoked by a human and never by a pipeline. This is provenance-governed borrow discipline with an audited trail — not a copy detector.
459
442
 
460
443
  <!-- provenance-scope-box -->
461
444
  #### Scope & limits
@@ -483,7 +466,7 @@ When a prompt matches a command's purpose ("setze ticket ABC-123 um" → `/imple
483
466
  - **Verify with real execution** — no "should work"
484
467
  - **Challenge to improve** — agents are thought partners, not yes-machines
485
468
  - **Strict by design** — quality over flexibility
486
- - **Zero overhead by default** — nothing runs until you ask for it
469
+ - **Governed runtime** — resident processes require supervision, scoped writes, and a stop control
487
470
 
488
471
  ---
489
472
 
@@ -517,71 +500,10 @@ layer`), the platform's concern chain, host hook registration, and router +
517
500
  projection freshness. Deeper hook internals (fail-open/closed posture, last
518
501
  dispatcher feedback per concern): `agent-config hooks:doctor`.
519
502
 
520
- ### A new command / skill is missing in Claude Code after an upgrade
521
-
522
- Under the single-surface model, `agent-config upgrade` refreshes the
523
- `~/.claude/` file projection — that IS the content surface, so a fresh
524
- session picks the new commands up directly. If commands are still missing,
525
- the usual cause is a leftover **marketplace plugin**: it is a git-SHA
526
- snapshot that never moves with the npm upgrade and it shadows nothing —
527
- it just lists everything twice while lagging behind. Remove it:
528
-
529
- ```bash
530
- claude plugin uninstall agent-config@event4u-agent-config
531
- ```
532
-
533
- Then start a **new** Claude Code session. `agent-config doctor` reports a
534
- leftover plugin as `claude-plugin: duplicate surface`; hooks are unaffected
535
- (they live in a managed `~/.claude/settings.json` block — verify with the
536
- `hook-wiring` check).
537
-
538
- ### Skills / commands appear twice in Claude Code
539
-
540
- Same cause as above: the deprecated marketplace plugin is installed next to
541
- the `~/.claude/` file projection, so every skill lists plain **and**
542
- `agent-config:`-prefixed. Uninstall the plugin (command above) and start a
543
- new session.
544
-
545
- ### `agent-config upgrade` fails with `Unknown argument: --no-ui`
546
-
547
- Known bug in 8.2.0: `upgrade` passed a `--no-ui` flag that the install
548
- orchestrator did not accept yet, so the run aborted early. Fixed on `main`;
549
- until the next release, work around it with:
550
-
551
- ```bash
552
- AGENT_CONFIG_NO_UI=1 agent-config global # refresh the global install, no wizard
553
- ```
554
-
555
- ### Upgrade was interrupted (Ctrl-C, wizard closed, step failed)
556
-
557
- Only the initial `npm install -g` hard-aborts an upgrade. Every later step
558
- (global re-deploy with hook registration, settings sync, wrapper + git-hook
559
- refresh) runs independently — a single failed step is reported in the
560
- end-of-run summary instead of silently skipping the rest. Re-run
561
- `agent-config upgrade` to converge and use `agent-config doctor` to name
562
- anything left in a mixed state.
563
-
564
- ### `agent-config: command not found` / hooks stopped firing
565
-
566
- Runtime hooks resolve the **global** binary on `PATH` — a project-local
567
- install alone is not enough for them. Reinstall the binary:
568
-
569
- ```bash
570
- npm install -g @event4u/agent-config
571
- agent-config doctor # verifies PATH + plugin wiring
572
- ```
573
-
574
- ### Project files look stale after a package update
575
-
576
- Project-local projections are only rewritten on an explicit refresh:
577
-
578
- ```bash
579
- agent-config refresh # re-apply the installed version to this project
580
- agent-config refresh --global # same-version re-install of the global root
581
- ```
582
503
 
583
- More per-version steps: [Migration](docs/MIGRATION.md) ·
584
- [getting-started § Keeping current](docs/getting-started.md#keeping-current).
504
+ Upgrade and staleness symptoms — a command or skill missing after an upgrade,
505
+ skills appearing twice, an interrupted upgrade, `command not found`, stale
506
+ project files: [`docs/troubleshooting.md` § Upgrade and staleness](docs/troubleshooting.md#upgrade-and-staleness).
585
507
 
586
508
  ---
587
509
 
@@ -17,7 +17,7 @@ it as `.augment/` (local projection).
17
17
  ├── contexts/ # Background context documents for the agent
18
18
  ├── guidelines/ # Coding guidelines by language (reference material)
19
19
  ├── rules/ # Always-active behavior rules (auto-loaded every conversation)
20
- ├── skills/ # On-demand expertise (matched by topic)
20
+ ├── skills/ # On-demand expertise (named or trigger-matched)
21
21
  └── templates/ # File templates for roadmaps, features, skills, etc.
22
22
  ```
23
23
 
@@ -65,9 +65,9 @@ exactly once, then deletes it). In a terminal-capable context, offer
65
65
  to spawn the fresh session directly:
66
66
 
67
67
  - `claude` — seeds via the session_start hook (clean first message).
68
- - `codex` — no hook surface there; the adapter passes the handoff as the
68
+ - `codex` — this package binds no hook there; the adapter passes the handoff as the
69
69
  initial prompt instead. Same UX, different transport.
70
- - Bundle hosts without hooks (Antigravity, Copilot) → use `--print` and
70
+ - Bundle hosts this package binds no hook on (Antigravity, Copilot) → use `--print` and
71
71
  copy-paste.
72
72
 
73
73
  ## Fallback mode — summarize the LIVE conversation (`--print`)
@@ -34,12 +34,13 @@ ACTIVATION NEEDS THE OWNER DECISION IN THE `merge-authority` BLOCKER OF
34
34
  `road-to-drain-commands` — NEVER A COMMAND EDIT, AND NEVER A GUARD EDIT.
35
35
  ```
36
36
 
37
- **The invocation is the authorization, and nothing else is.** `pr-merge` is a
38
- `BLOCK_OPS` member in `block_unauthorized_git.ts` because it is irreversible.
39
- This command would consume the authorization the user's own prompt text already
40
- wrote to the per-session ledger on `UserPromptSubmit`; it introduces no second
41
- authorization store, and it never writes one. When that window closes the run
42
- stops and reports (§ 7) — the window never grows.
37
+ **The invocation is the authorization, and nothing else is.** `pr-merge` is
38
+ irreversible, and since 2026-09-04 (ADR-254) NOTHING MECHANICAL CHECKS IT —
39
+ the gate that did was removed after it refused authorizations the owner had in
40
+ fact given. The per-session ledger still records what the user's own prompt
41
+ authorized, and this command still reads it as an audit trail; it introduces no
42
+ second authorization store, and it never writes one. When the work left is
43
+ outside what the owner authorized, the run stops and reports (§ 7).
43
44
 
44
45
  Why it ships inert rather than not at all: everything before § 9 — the target
45
46
  manifest, the four conflict classes, the superseded check, the bounded CI
@@ -266,54 +267,44 @@ exceeds the remaining window, stop entering CI-fix loops and merge everything
266
267
  already green first. Pre-greening several PRs ahead of their merge is forbidden
267
268
  under window pressure.
268
269
 
269
- **The signal, named — otherwise neither this rule nor § 7 is actionable.** The
270
- window is the per-session authorization ledger's own freshness bound. Read it,
271
- never write it:
270
+ **No mechanical window remains, and that is the change.** Until 2026-09-04 the
271
+ run read a freshness bound off the authorization ledger and paused when it
272
+ closed. The owner removed the enforcing gate (ADR-254) after it was measured
273
+ refusing authorizations that had in fact been given, so there is no constant to
274
+ read and no expiry to schedule against. Authorization is carried by the
275
+ conversation and by `non-destructive-by-default`, not by a clock.
276
+
277
+ The per-session ledger is still written, and is still read-only to this run:
272
278
 
273
279
  ```bash
274
280
  cat "agents/state/git-authorization/$(<session-slug>).json" # detected_at
275
281
  ```
276
282
 
277
- Remaining window = `detected_at + LEDGER_MAX_AGE_MS − now`. Take the constant
278
- from the guard source **only after** `check_hook_bundle_content` says the
279
- source and the executing bundle agree — reading the source alone is the
280
- 2026-08-21 failure re-expressed as an instruction, since the bundle is what
281
- enforces the value and a source edit without a rebuild is silently inert. Under
282
- pressure means
283
- the remaining window is shorter than one CI cycle on this repository. The read
284
- is the whole interaction: the run never edits that file, the constant, or the
285
- built bundle.
283
+ It records what a turn authorized. It no longer gates anything, so treat it as
284
+ an audit trail, never as permission. The run never edits that file.
286
285
 
287
- ## 7. Expiry is a reported state, never a stall
286
+ ## 7. Authorization is the owner's sentence, not a timer
288
287
 
289
288
  ```
290
- WHEN THE AUTHORIZATION WINDOW CLOSES WITH WORK LEFT, THE RUN PAUSES, REPORTS,
291
- AND ASKS FOR RE-AUTHORIZATION. IT NEVER RETRIES THE GUARD, AND IT NEVER EDITS
292
- THE GUARD, ITS SOURCE, OR ITS BUNDLES — READ-ONLY VERIFICATION ONLY.
289
+ THE RUN MERGES WHAT THE OWNER AUTHORIZED AND NOTHING ELSE. WHEN THE WORK LEFT
290
+ IS OUTSIDE WHAT WAS AUTHORIZED, THE RUN PAUSES, REPORTS, AND ASKS. IT NEVER
291
+ INFERS A WIDER MANDATE FROM A LONG RUN OR FROM MOMENTUM.
293
292
  ```
294
293
 
295
- **Unreachable while the merge step is gated**, and stated rather than left for
296
- a reader to discover: with nothing merging, the run performs no `BLOCK_OPS`
297
- operation (`push` and `commit` are `WARN_OPS`), so the window governs nothing
298
- it does. This section is the contract for when the gate opens.
299
-
300
- Write the summary as-is with a `window-expired` disposition per unprocessed PR
301
- and name the exact re-authorization needed. Then STOP and wait — do not end the
302
- run, and do not proceed to the next PR. ADR-251 (2026-08-30) decided this
303
- shape: expiry is a pause, not a termination.
304
-
305
- **The operator's reply is what resumes it, and it renews the window through the
306
- path that already exists.** `git_authorization_hook.ts` rewrites this session's
307
- ledger with a fresh `detected_at` on every human-typed prompt, so a reply
308
- carrying the re-authorization you named resets the age the guard compares
309
- against — no new mechanism, and the signal still comes from text the user typed.
310
- Re-verify the manifest entry (number and head SHA) before resuming, exactly as
311
- § 8 requires: the pause is a window in which the world can move.
312
-
313
- Widening `LEDGER_MAX_AGE_MS` is still forbidden practice, and ADR-251 kept the
314
- 30-minute value deliberately rather than by omission: on 2026-08-21 it was
315
- patched to six hours for a drain run and the widening reached the trunk. The run proposes no value for it, edits neither it nor its bundles,
316
- and treats a refusal as the answer.
294
+ With the gate removed nothing refuses an unauthorized merge, which makes this
295
+ section the whole control rather than a second layer behind one. Two rules
296
+ follow, and they are stricter than the ones they replace:
297
+
298
+ - **A cardinality word does not authorize a PR the owner never named.** "All the
299
+ open PRs" is an instruction to work the queue as it stood when the owner said
300
+ it. A PR opened after that sentence is new work and needs its own go-ahead.
301
+ - **A one-shot authorization is spent on the operation it named.** Per
302
+ `commit-policy`, "merge #1499" does not authorize the next merge, and a
303
+ conflict cycle or a CI fix in between does not renew it.
304
+
305
+ Write the summary as-is with an `unauthorized` disposition per unprocessed PR
306
+ and name the exact authorization needed. Then STOP and wait — do not end the
307
+ run, and do not proceed to the next PR.
317
308
 
318
309
  ## 8. Kill switches, and what happens after a merge
319
310
 
@@ -2,14 +2,14 @@
2
2
  model_tier: high
3
3
  name: review-changes
4
4
  pack: engineering-base
5
- intent: "Multi-judge review of the current diff — bugs, security, tests, quality, architecture"
6
- routes_to: [code-review, judge-bug-hunter, judge-security-auditor, judge-test-coverage, judge-code-quality, architecture-review-lens, overbuild-review-lens]
5
+ intent: "Multi-judge review of the current diff — bugs, security, tests, quality, architecture, spec, overbuild"
6
+ routes_to: [code-review, judge-bug-hunter, judge-security-auditor, judge-test-coverage, judge-code-quality, architecture-review-lens, judge-spec-compliance, overbuild-review-lens]
7
7
  replaces: []
8
8
  visibility: internal
9
9
  sub: changes
10
10
  cluster: review
11
- skills: [code-review, subagent-orchestration, judge-bug-hunter, judge-security-auditor, judge-test-coverage, judge-code-quality, architecture-review-lens, overbuild-review-lens, judge-synthesis, git-workflow]
12
- description: Self-review local changes before creating a PR — dispatches to six specialized judges (bug, security, tests, quality, architecture, spec) and consolidates verdicts
11
+ skills: [code-review, subagent-orchestration, judge-bug-hunter, judge-security-auditor, judge-test-coverage, judge-code-quality, architecture-review-lens, judge-spec-compliance, overbuild-review-lens, judge-synthesis, git-workflow]
12
+ description: Self-review local changes before creating a PR — dispatches to seven specialized judges (bug, security, tests, quality, architecture, spec, overbuild) and consolidates verdicts
13
13
  suggestion:
14
14
  eligible: false
15
15
  rationale: "Cluster sub-command — reached via its cluster head's routing or its explicit /cluster:sub name; not independently suggested (surface-consolidation)."
@@ -24,7 +24,7 @@ packs:
24
24
  ## Instructions
25
25
 
26
26
  Review all uncommitted and committed-but-not-pushed changes against
27
- the default branch (`main`) by dispatching to six specialized judge
27
+ the default branch (`main`) by dispatching to seven specialized judge
28
28
  sub-skills and consolidating their verdicts.
29
29
 
30
30
  ### 1. Update the current branch
@@ -109,7 +109,7 @@ folded together, a handover the reviewer could not parse would report as "there
109
109
  was nothing to check", which is the one outcome that hides a real failure
110
110
  behind a benign one.
111
111
 
112
- ### 4. Dispatch to the six judges
112
+ ### 4. Dispatch to the seven judges
113
113
 
114
114
  Each judge receives **the same diff plus the task context** (ticket,
115
115
  PR body, commit messages) and runs independently. The judges are:
@@ -137,8 +137,8 @@ diff cannot answer about itself. It is dispatched on demand like the others, so
137
137
  it adds no standing token cost — verified: `check_preamble_payload_budget`'s
138
138
  measured total is unchanged by this routing.
139
139
 
140
- **Why the sixth exists, and why it is not a severity.** The other five all ask a
141
- craft-or-correctness question, so a change that is correct, clean, well-tested
140
+ **Why the sixth exists, and why it is not a severity.** The five that preceded it
141
+ all ask a craft-or-correctness question, so a change that is correct, clean, well-tested
142
142
  and architecturally sound — and **does not do what was asked** — used to pass
143
143
  this path with five green verdicts. Its finding does **not** enter the shared
144
144
  severity axis: `judge-synthesis` carries it on its own dimension, because "this
@@ -180,14 +180,14 @@ Read `verbosity.offer_council_in_delivery` from `.agent-settings.yml`
180
180
  - `true`: when `ai_council.enabled: true` **and** at least one member
181
181
  enabled, ask (in the user's language):
182
182
 
183
- > 1. Add an external council review alongside the six internal judges? (billable)
183
+ > 1. Add an external council review alongside the seven internal judges? (billable)
184
184
  > 2. Skip — internal judges only
185
185
 
186
186
  Also suppress when `personal.autonomy: on` (council is billable).
187
187
 
188
188
  If picked **1**:
189
189
 
190
- - Run `/council diff:<base>..<head>` in parallel with the six
190
+ - Run `/council diff:<base>..<head>` in parallel with the seven
191
191
  internal judges (or sequentially after them — whichever the
192
192
  dispatch mode picked in step 4 supports).
193
193
  - Treat each council member as one extra "judge" in the consolidated
@@ -202,12 +202,12 @@ If picked **2** → continue with internal judges only.
202
202
 
203
203
  ### 4b-fresh. The fresh reviewer — the one input with no implementation context
204
204
 
205
- The six judges above all read **the same diff plus the task context**, and on the common
205
+ The seven judges above all read **the same diff plus the task context**, and on the common
206
206
  path they run in-session: the party that wrote the diff also reads it, with the whole
207
207
  implementation context in scope. That is a self-review, and calling it a review is the
208
208
  defect `road-to-review-independence` exists to close.
209
209
 
210
- So a **seventh input** is added, and its distinguishing property is a negative one: it
210
+ So **one further input** is added, and its distinguishing property is a negative one: it
211
211
  has **no implementation context**. Not a new mechanism —
212
212
  [`dispatch_r2_reviewer.ts`](../../../../scripts/dispatch_r2_reviewer.ts) already derives
213
213
  the scope and persists the prompt it sent:
@@ -216,7 +216,7 @@ the scope and persists the prompt it sent:
216
216
  ./scripts-run src/scripts/dispatch_r2_reviewer --slug <slug> --base origin/main
217
217
  ```
218
218
 
219
- The six in-session judges stay exactly as they are. This is an addition, not a
219
+ The seven in-session judges stay exactly as they are. This is an addition, not a
220
220
  replacement — a fresh reviewer is worse than a spec judge at spec compliance, and the
221
221
  point is not to be better at their job but to be uncontaminated by the author's framing.
222
222
 
@@ -262,7 +262,7 @@ Read `subagents.adversarial_council` from `.agent-settings.yml` (default
262
262
  which supplies the four slots, the one-question budget, and the storage line
263
263
  (not persisted: `subagents.adversarial_council` is class C). What only this
264
264
  command knows is *why now*: the diff is high-risk, and the offer is a paid
265
- distinct-model red-team for finding coverage against the six internal judges.
265
+ distinct-model red-team for finding coverage against the seven internal judges.
266
266
  - `on`: run it automatically on a high-risk diff.
267
267
 
268
268
  When run, dispatch the
@@ -315,7 +315,7 @@ window back with `./scripts-run src/scripts/review_axis_report`; it reports
315
315
  before proceeding
316
316
  - If **any** judge returned `revise` → fix 🔴 findings automatically,
317
317
  ask before fixing 🟡 findings, report 🟢 as suggestions
318
- - If all six returned `apply` → the diff is ready; report and stop
318
+ - If all seven returned `apply` → the diff is ready; report and stop
319
319
 
320
320
  **Opt-in (never auto-on):** when the `revise` findings are *test-driven*
321
321
  (failing checks, not subjective craft) and you want bounded auto-repair
@@ -360,7 +360,7 @@ Per `verbosity.routine_confirmations` (default `false`):
360
360
  ## Use this command when
361
361
 
362
362
  - Preparing a self-review before opening a PR
363
- - Stress-testing a local branch with the same six lenses a reviewer
363
+ - Stress-testing a local branch with the same seven lenses a reviewer
364
364
  would apply
365
365
  - Sanity-checking a diff before handing it to `/create-pr`
366
366
 
@@ -389,6 +389,6 @@ Per `verbosity.routine_confirmations` (default `false`):
389
389
 
390
390
  - **LLM-as-a-Judge** — [arxiv.org/abs/2306.05685](https://arxiv.org/abs/2306.05685)
391
391
  MT-Bench and Chatbot Arena — judging LLM outputs with LLM judges.
392
- This command adapts the pattern by dispatching to six specialized
393
- judges (bug, security, tests, quality) instead of a single generic
394
- judge, and consolidating their verdicts.
392
+ This command adapts the pattern by dispatching to seven specialized
393
+ judges (bug, security, tests, quality, architecture, spec, overbuild)
394
+ instead of a single generic judge, and consolidating their verdicts.
@@ -8,7 +8,7 @@ intent: "Review dispatcher — multi-judge self-review of the current diff, or r
8
8
  routes_to: [review-changes, review-routing]
9
9
  replaces: []
10
10
  visibility: advanced
11
- description: Review orchestrator — routes to changes (five-judge self-review of the local diff) and routing (compute reviewer roles + historical bug patterns)
11
+ description: Review orchestrator — routes to changes (seven-judge self-review of the local diff) and routing (compute reviewer roles + historical bug patterns)
12
12
  cluster: review
13
13
  type: orchestrator
14
14
  suggestion:
@@ -49,7 +49,7 @@ Sub-command names match the locked contract in
49
49
  4. If the sub-command is unknown or missing, print the menu and ask — do not
50
50
  guess:
51
51
 
52
- > 1. changes — five-judge self-review of the local diff (pre-PR)
52
+ > 1. changes — seven-judge self-review of the local diff (pre-PR)
53
53
  > 2. routing — suggest reviewers + surface matched historical bug patterns
54
54
 
55
55
  ## Rules