@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.
- package/.augment-plugin/marketplace.json +3 -3
- package/.augment-plugin/plugin.json +1 -1
- package/.claude-plugin/marketplace.json +1 -1
- package/CHANGELOG.md +207 -77
- package/README.md +72 -150
- package/dist/agent-src/README.md +1 -1
- package/dist/agent-src/commands/agent-handoff.md +2 -2
- package/dist/agent-src/commands/pr/merge.md +35 -44
- package/dist/agent-src/commands/review/changes.md +19 -19
- package/dist/agent-src/commands/review.md +2 -2
- package/dist/agent-src/contexts/execution/host-capability-manifest.md +30 -3
- package/dist/agent-src/rules/autonomous-execution.md +1 -1
- package/dist/agent-src/rules/git-history-discipline.md +2 -2
- package/dist/agent-src/rules/guidelines.md +1 -1
- package/dist/agent-src/rules/session-canary.md +2 -2
- package/dist/agent-src/rules/token-efficiency.md +1 -1
- package/dist/agent-src/skills/accessibility-auditor/SKILL.md +2 -0
- package/dist/agent-src/skills/adversarial-review/SKILL.md +0 -8
- package/dist/agent-src/skills/ai-code-blindspots/SKILL.md +27 -1
- package/dist/agent-src/skills/canvas-design/SKILL.md +1 -1
- package/dist/agent-src/skills/code-intelligence/SKILL.md +60 -12
- package/dist/agent-src/skills/complexity-first-planning/SKILL.md +27 -0
- package/dist/agent-src/skills/design-intelligence/SKILL.md +1 -1
- package/dist/agent-src/skills/design-intelligence/data/app-interface.csv +1 -1
- package/dist/agent-src/skills/design-intelligence/data/motion.csv +17 -17
- package/dist/agent-src/skills/design-intelligence/data/ux-guidelines.csv +2 -2
- package/dist/agent-src/skills/design-intelligence/references/context-and-registers.md +17 -0
- package/dist/agent-src/skills/design-intelligence/references/design-rules-checklist.md +4 -4
- package/dist/agent-src/skills/design-review/SKILL.md +3 -1
- package/dist/agent-src/skills/design-variations/SKILL.md +1 -1
- package/dist/agent-src/skills/existing-ui-audit/SKILL.md +1 -1
- package/dist/agent-src/skills/fe-design/SKILL.md +32 -5
- package/dist/agent-src/skills/fe-design/references/craft-floor.md +7 -0
- package/dist/agent-src/skills/fe-design/references/design-patterns.md +47 -2
- package/dist/agent-src/skills/feature-planning/SKILL.md +34 -0
- package/dist/agent-src/skills/git-workflow/SKILL.md +37 -12
- package/dist/agent-src/skills/git-workflow/references/push-closes-its-loop.md +176 -0
- package/dist/agent-src/skills/laravel-mail/SKILL.md +44 -0
- package/dist/agent-src/skills/laravel-scheduling/SKILL.md +3 -1
- package/dist/agent-src/skills/project-analyzer/SKILL.md +0 -6
- package/dist/agent-src/skills/react-native-setup/SKILL.md +3 -3
- package/dist/agent-src/skills/react-shadcn-ui/SKILL.md +1 -1
- package/dist/agent-src/skills/sequential-thinking/SKILL.md +18 -8
- package/dist/agent-src/skills/tailwind-engineer/SKILL.md +1 -1
- package/dist/agent-src/skills/testing-anti-patterns/SKILL.md +40 -0
- package/dist/agent-src/skills/ui-apply-generic/SKILL.md +1 -1
- package/dist/agent-src/skills/ui-component-architect/SKILL.md +1 -1
- package/dist/agent-src/skills/using-git-worktrees/SKILL.md +29 -2
- package/dist/agent-src/skills/verify-repair-loop/SKILL.md +53 -8
- package/dist/agent-src/skills/worktree-lifecycle/SKILL.md +3 -1
- package/dist/agent-src/templates/agents/agent-project-settings.example.yml +1 -1
- package/dist/cli/commands/recordTriggerEval.js +2 -2
- package/dist/cli/main.js +1 -1
- package/dist/cli-delegate/{chunk-AKJGUQNO.js → chunk-GZTMRYCN.js} +405 -341
- package/dist/cli-delegate/{chunk-7RXIKF62.js → chunk-IDHUQIXP.js} +283 -132
- package/dist/cli-delegate/cmd_adr_effective.js +318 -81
- package/dist/cli-delegate/cmd_conformance.js +749 -669
- package/dist/cli-delegate/cmd_doctor.js +1 -1
- package/dist/cli-delegate/cmd_export.js +1 -1
- package/dist/cli-delegate/cmd_handoff.js +1 -1
- package/dist/cli-delegate/cmd_prune.js +1 -1
- package/dist/cli-delegate/cmd_route_audit.js +1 -1
- package/dist/cli-delegate/cmd_settings_migrate.js +1 -1
- package/dist/cli-delegate/cmd_sync.js +1 -1
- package/dist/cli-delegate/cmd_uninstall.js +1 -1
- package/dist/cli-delegate/cmd_validate.js +1 -1
- package/dist/discovery/deprecation-report.md +1 -1
- package/dist/discovery/discovery-manifest.json +39 -38
- package/dist/discovery/discovery-manifest.json.sha256 +1 -1
- package/dist/discovery/discovery-manifest.summary.md +1 -1
- package/dist/discovery/dormancy-report.md +1 -1
- package/dist/discovery/orphan-report.md +1 -1
- package/dist/discovery/packs.json +2 -2
- package/dist/discovery/trust-report.md +1 -1
- package/dist/discovery/workspaces.json +2 -2
- package/dist/hooks/dispatch.js +2764 -2439
- package/dist/install/install.mjs +588 -527
- package/dist/mcp/registry-manifest.json +2 -2
- package/dist/mcp/server.json +2 -2
- package/dist/mcp/server.mjs +1381 -1138
- package/dist/scripts/_lib/model_tier.js +1 -1
- package/dist/shared/settingsConsent.js +1 -1
- package/docs/guidelines/agent-infra/evaluator-independence-mechanics.md +1 -1
- package/docs/guidelines/agent-infra/roadmap-progress-mechanics.md +1 -1
- package/docs/guidelines/design-antipatterns.md +87 -64
- package/package.json +1 -1
- package/src/agent-src/README.md +1 -1
- package/src/config/ci-local-parity.yml +6 -0
- package/src/config/conformance-claim-baseline.json +9 -0
- package/src/config/gate-coverage.yml +170 -0
- package/src/config/gate-violation-baselines.json +4 -4
- package/src/config/release-gate-locality.yml +141 -0
- package/src/scripts/_cli/cmd_doctor.ts +2 -2
- package/src/scripts/_cli/cmd_handoff.ts +1 -1
- package/src/scripts/_cli/cmd_update.ts +1 -1
- package/src/scripts/_cli/cmd_versions.ts +1 -1
- package/src/scripts/_lib/adr_frontmatter.ts +44 -0
- package/src/scripts/_lib/consequence_objects.ts +152 -0
- package/src/scripts/_lib/exec_evidence.ts +7 -0
- package/src/scripts/_lib/global_deploy_inventory.ts +2 -2
- package/src/scripts/_lib/host_permission_checks.ts +251 -0
- package/src/scripts/_lib/model_tier.ts +1 -1
- package/src/scripts/_lib/obligation_frequency.ts +2 -2
- package/src/scripts/_lib/release_highlights.ts +265 -1
- package/src/scripts/_lib/release_material.ts +129 -0
- package/src/scripts/_lib/repeated_run.ts +77 -0
- package/src/scripts/_lib/runtime_journal.ts +26 -3
- package/src/scripts/_lib/runtime_wiring_checks.ts +12 -1
- package/src/scripts/_lib/session_register.ts +11 -6
- package/src/scripts/_lib/sqlite_schema.ts +22 -0
- package/src/scripts/_lib/surface.ts +86 -0
- package/src/scripts/_lib/tamper_vocabulary.ts +242 -0
- package/src/scripts/_lib/ui_authority.ts +116 -0
- package/src/scripts/_lib/unified_diff.ts +121 -0
- package/src/scripts/adr/regenerate_index.ts +54 -1
- package/src/scripts/adr_cite_check.ts +71 -15
- package/src/scripts/ai_council/clients.ts +1 -1
- package/src/scripts/autonomy_friction_corpus.ts +229 -0
- package/src/scripts/build_proof.ts +15 -2
- package/src/scripts/check_beta_review_markers.ts +141 -17
- package/src/scripts/check_enforcement_coverage.ts +1 -1
- package/src/scripts/check_enforcement_denominator.ts +121 -11
- package/src/scripts/check_finding_dispositions.ts +239 -6
- package/src/scripts/check_gate_coverage.ts +96 -0
- package/src/scripts/check_installed_hooks_fresh.ts +381 -0
- package/src/scripts/check_release_highlights.ts +99 -14
- package/src/scripts/check_skill_activation_claim.ts +179 -0
- package/src/scripts/check_suppression_hygiene.ts +11 -0
- package/src/scripts/check_trigger_evals.ts +1 -1
- package/src/scripts/code_graph/build.ts +230 -33
- package/src/scripts/code_graph/extract.ts +168 -33
- package/src/scripts/code_graph/types.ts +37 -2
- package/src/scripts/code_graph/validate.ts +12 -1
- package/src/scripts/conformance_scan.ts +45 -4
- package/src/scripts/detect_silent_catch.ts +403 -0
- package/src/scripts/detect_verification_tampering.ts +538 -0
- package/src/scripts/explain_run.ts +1 -1
- package/src/scripts/generate_capability_matrix.ts +1 -1
- package/src/scripts/git_authorization_hook.ts +228 -18
- package/src/scripts/hook_manifest.json +1 -1
- package/src/scripts/hook_manifest.yaml +47 -30
- package/src/scripts/hooks/augment-dispatcher.sh +1 -1
- package/src/scripts/hooks/block_no_verify.ts +3 -3
- package/src/scripts/hooks/category_a.ts +291 -0
- package/src/scripts/hooks/cline-dispatcher.sh +1 -1
- package/src/scripts/hooks/concern_registry.ts +2 -2
- package/src/scripts/hooks/cursor-dispatcher.sh +11 -5
- package/src/scripts/hooks/dispatch_hook.ts +17 -22
- package/src/scripts/hooks/evidence_independence.ts +21 -2
- package/src/scripts/hooks/{block_unauthorized_git.ts → git_command_classifier.ts} +175 -364
- package/src/scripts/hooks/host_lowering.ts +215 -0
- package/src/scripts/hooks/host_lowering.yaml +185 -0
- package/src/scripts/hooks/host_semantics.ts +105 -37
- package/src/scripts/hooks/journal_record_hook.ts +10 -0
- package/src/scripts/hooks/permission_gate.ts +117 -0
- package/src/scripts/hooks/push_settle_hook.ts +250 -0
- package/src/scripts/hooks/ship_diff_volume_hook.ts +1 -1
- package/src/scripts/hooks_status.ts +12 -10
- package/src/scripts/install-hooks.sh +134 -1
- package/src/scripts/install.ts +53 -82
- package/src/scripts/install_anthropic_key.sh +2 -2
- package/src/scripts/language_mirror_hook.ts +27 -5
- package/src/scripts/lint_design_antipattern_parity.ts +80 -0
- package/src/scripts/lint_evidence_artifacts.ts +45 -1
- package/src/scripts/lint_hook_manifest.ts +87 -3
- package/src/scripts/lint_motion_authority_drift.ts +731 -0
- package/src/scripts/lint_readme_size.ts +158 -12
- package/src/scripts/lint_reasoning_fixtures.ts +216 -0
- package/src/scripts/lint_skill_trigger_contradiction.ts +161 -0
- package/src/scripts/measure_sibling_naming.ts +150 -0
- package/src/scripts/redact_hook_capture.ts +1 -1
- package/src/scripts/release.ts +31 -143
- package/src/scripts/release_drill.ts +71 -1
- package/src/scripts/release_publication.ts +243 -3
- package/src/scripts/release_verify.ts +220 -0
- package/src/scripts/report_skill_activation.ts +319 -7
- package/src/scripts/schemas/review-findings.schema.json +6 -1
- package/src/scripts/schemas/skill.schema.json +12 -2
- package/src/scripts/self_review_gate.ts +10 -1
- package/src/scripts/session_canary_hook.ts +1 -1
- package/src/scripts/session_register_hook.ts +4 -3
- package/src/scripts/trigger_eval_grandfather.json +5 -1
- package/src/scripts/validate_agent_settings.ts +1 -1
- 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
|
-
|
|
42
|
+
Deep **and** disciplined, and honest about what it deliberately is not:
|
|
24
43
|
|
|
25
|
-
- **Depth that routes itself** — a
|
|
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
|
|
43
|
-
|
|
44
|
-
|
|
45
|
-
|
|
46
|
-
|
|
47
|
-
|
|
48
|
-
|
|
49
|
-
|
|
|
50
|
-
|
|
|
51
|
-
|
|
|
52
|
-
|
|
|
53
|
-
|
|
|
54
|
-
|
|
55
|
-
|
|
56
|
-
|
|
57
|
-
|
|
58
|
-
[`src/agent-src/profiles/`](src/agent-src/profiles/) ·
|
|
59
|
-
|
|
60
|
-
|
|
61
|
-
|
|
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
|
|
75
|
-
developer story end-to-end
|
|
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
|
-
|
|
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
|
|
126
|
-
checkout.
|
|
127
|
-
|
|
128
|
-
[event4u-app.github.io/agent-config/proof/](https://event4u-app.github.io/agent-config/proof/)
|
|
129
|
-
|
|
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
|
|
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
|
-
|
|
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
|
|
176
|
-
|
|
177
|
-
|
|
178
|
-
|
|
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
|
|
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
|
|
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`
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
-
|
|
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
|
|
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
|
-
|
|
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** (
|
|
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
|
-
- **
|
|
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
|
-
|
|
584
|
-
|
|
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
|
|
package/dist/agent-src/README.md
CHANGED
|
@@ -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 (
|
|
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
|
|
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
|
|
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
|
|
38
|
-
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
|
|
42
|
-
|
|
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
|
-
**
|
|
270
|
-
|
|
271
|
-
|
|
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
|
-
|
|
278
|
-
|
|
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.
|
|
286
|
+
## 7. Authorization is the owner's sentence, not a timer
|
|
288
287
|
|
|
289
288
|
```
|
|
290
|
-
|
|
291
|
-
|
|
292
|
-
|
|
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
|
-
|
|
296
|
-
|
|
297
|
-
|
|
298
|
-
|
|
299
|
-
|
|
300
|
-
|
|
301
|
-
|
|
302
|
-
|
|
303
|
-
|
|
304
|
-
|
|
305
|
-
|
|
306
|
-
|
|
307
|
-
|
|
308
|
-
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
393
|
-
judges (bug, security, tests, quality
|
|
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 (
|
|
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 —
|
|
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
|