From 245ad6b39f0d669bbd2563db99926997b2a9546c Mon Sep 17 00:00:00 2001 From: Giancarmine Salucci Date: Thu, 10 Sep 2026 02:47:43 +0200 Subject: [PATCH] refactor(agents): hide architect and reviewer from /agent picker Enforce single-entry-point: only conductor is user-invocable now. architect and reviewer remain fully functional but are only reachable as subagents that conductor delegates to internally, matching the same internal-only treatment already given to scout/splitter/builder/ verifier/integrator/scribe. --- .github/agents/architect.agent.md | 2 +- .github/agents/reviewer.agent.md | 2 +- AGENTS.md | 4 ++-- README.md | 4 ++-- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/.github/agents/architect.agent.md b/.github/agents/architect.agent.md index 9d796ef..257ba78 100644 --- a/.github/agents/architect.agent.md +++ b/.github/agents/architect.agent.md @@ -4,7 +4,7 @@ description: Turns decisions into a checkable spec, acceptance criteria, archite model: claude-opus-5 tools: [read, search, edit, execute, agent, context7] agents: [Scout] -user-invocable: true +user-invocable: false --- # Architect diff --git a/.github/agents/reviewer.agent.md b/.github/agents/reviewer.agent.md index 8b72bac..4aa5710 100644 --- a/.github/agents/reviewer.agent.md +++ b/.github/agents/reviewer.agent.md @@ -3,7 +3,7 @@ name: Reviewer description: Reviews completed lanes against every acceptance criterion and scope rule, then returns explicit approval or rejection. model: claude-opus-5 tools: [read, search, execute] -user-invocable: true +user-invocable: false --- # Reviewer diff --git a/AGENTS.md b/AGENTS.md index dd3283c..4425689 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -33,11 +33,11 @@ planned around. | --- | --- | --- | --- | | `conductor` | strong | yes | Single entry point. Asks design questions itself, then owns the pipeline and the dynamic DAG. | | `scout` | cheap | no | Read-only recon, fanned out N-wide. | -| `architect` | strong | yes | Spec, acceptance criteria, architecture doc, ADRs. | +| `architect` | strong | – | Spec, acceptance criteria, architecture doc, ADRs. | | `splitter` | mid | no | Decomposes spec into lanes with file-scope globs. | | `builder` | mid | no | Implements one lane inside its worktree. | | `verifier` | cheap | no | Runs verify commands + structural gate. | -| `reviewer` | strong | yes | Acceptance-criteria and scope gate. | +| `reviewer` | strong | – | Acceptance-criteria and scope gate. | | `integrator` | strong | no | Sequential merge, conflict resolution, full verify. | | `scribe` | cheap | no | Journal, ADRs, living spec, conventions. | diff --git a/README.md b/README.md index c4da1e2..87d1da6 100644 --- a/README.md +++ b/README.md @@ -255,11 +255,11 @@ escalates to you. | --- | --- | --- | --- | | `conductor` | strong | yes | Single entry point. Asks design questions itself, then owns the pipeline and the dynamic DAG. | | `scout` | cheap | – | Read-only recon, fanned out N-wide. | -| `architect` | strong | yes | Spec, acceptance criteria, architecture doc, ADRs. | +| `architect` | strong | – | Spec, acceptance criteria, architecture doc, ADRs. | | `splitter` | mid | – | Decomposes the spec into lanes with file-scope globs. | | `builder` | mid | – | Implements one lane inside its worktree. | | `verifier` | cheap | – | Runs verify commands + the structural gate. | -| `reviewer` | strong | yes | Acceptance-criteria and scope gate. | +| `reviewer` | strong | – | Acceptance-criteria and scope gate. | | `integrator` | strong | – | Sequential merge, conflict escalation, full verify. | | `scribe` | cheap | – | Journal, ADRs, living spec, conventions. |