agentcohort 0.1.1 → 0.3.0

package/dist/templates/commands/auto-flow.md

@@ -1,41 +1,77 @@
 ---
-description: Classify the task and route it to the correct Agentcohort workflow.
+description: Smart router — dispatcher classifies the task into a tier, prints a plan, and executes only after explicit user approval.
 argument-hint: <describe the task, paste the bug, or point at the diff>
 ---
 
-# /auto-flow — Task Router
+# /auto-flow — Dispatcher-driven router
 
-You are the **orchestrator**. Do not start working yet. First classify the
-task in `$ARGUMENTS`, announce the chosen flow and why, then execute it.
+You are the **orchestrator**. Do **not** start the downstream pipeline
+yet. First classify, then surface the plan, then wait for the user.
 
-## Classification rules (first match wins)
+## Step 1 — Classify (cheap, mandatory)
 
-1. **User has explicitly APPROVED a specific fix** ("approved", "go ahead and
-   fix", "implement the agreed fix")  → **BUG FIX APPROVED FLOW** → run
-   `/bug-fix-approved`.
-2. **Bug / crash / regression / failing test / incorrect data / security /
-   stability / "it's broken" / "wrong output"** (and not yet approved) →
-   **BUG AUDIT FLOW** → run `/bug-audit`. *No fixing.*
-3. **Slow / latency / bottleneck / high memory / profiling / "make it
-   faster"** → **PERFORMANCE FLOW** → run `/perf-hunt`.
-4. **Review a diff / PR / "is this safe to merge"** → **REVIEW FLOW** →
-   run `/review-diff`.
-5. **Feature / new behavior / refactor / "add" / "implement" / "change how X
-   works"** → **DEV FLOW** → run `/dev-flow`.
+Invoke the `dispatcher` subagent on `$ARGUMENTS`. The dispatcher
+returns a structured plan with: tier, pipeline, agents involved,
+agents skipped, escalation triggers, cost band, and the exact next
+command to run.
 
-If ambiguous, ask ONE clarifying question, then classify. If it is both a bug
-and a feature, prefer BUG AUDIT for the defect part and say so.
+If the dispatcher escalates the tier above the user's intuitive
+expectation, **trust the dispatcher** — escalation keywords (auth,
+schema, migration, payment, security, concurrency, cache, …) are
+non-negotiable.
+
+## Step 2 — Surface the plan
+
+Print the dispatcher's plan to the user verbatim, plus a single
+question line:
+
+```
+Proceed with this plan?  [y / escalate / abort / question]
+```
+
+- `y` → run the next step exactly as planned.
+- `escalate` → move up one tier (e.g. Tier 2b → Tier 3 `/dev-flow`,
+  Tier 3 → Tier 4 with forced architect + expert-council) and re-print
+  the new plan.
+- `abort` → stop. Do nothing else.
+- `question` → answer the user's question; do not execute the plan
+  until you re-confirm.
+
+## Step 3 — Execute the chosen next step
+
+Only after the user replies `y`:
+
+| Tier | Action |
+|---|---|
+| **0** | Answer inline using Read / Glob / Grep. No subagent. |
+| **1** | Invoke `repo-scout` on `$ARGUMENTS`. Return the briefing. Stop. |
+| **2a — quick-fix** | Run `/quick-fix` on `$ARGUMENTS`. |
+| **2b — quick-feature** | Run `/quick-feature` on `$ARGUMENTS`. |
+| **3 — dev** | Run `/dev-flow` on `$ARGUMENTS`. |
+| **3 — bug audit** | Run `/bug-audit` on `$ARGUMENTS`. (No fixing — invariant.) |
+| **3 — perf** | Run `/perf-hunt` on `$ARGUMENTS`. |
+| **3 — review** | Run `/review-diff`. |
+| **3 — bug fix approved** | Run `/bug-fix-approved` on `$ARGUMENTS`. |
+| **4 — escalated** | Run `/dev-flow` (or `/bug-audit`/`/perf-hunt`) and **force** the architect stage + expert-council stage on. |
 
 ## Hard rules
 
-- **Never fix a bug in the audit flow.** Audit produces evidence, root cause,
-  options and a recommendation — then stops for human approval.
-- Respect the model strategy: Haiku for scouting, Sonnet for
-  implement/test/hunt, Opus for architecture/root-cause/council/review.
-- Enforce scope discipline: no unrelated refactors; no API/schema/auth/
-  security/persistence semantic changes without explicit approval.
+- **Never run downstream agents before the user replies `y`.** Silent
+  routing destroys the value of having a plan.
+- **Bug audit never fixes.** Invariant from `/bug-audit`. The
+  dispatcher cannot route a Tier 4 bug into a fix path.
+- **Reviewer is never skipped** for any code change, regardless of
+  tier. The mini-commands `/quick-fix` and `/quick-feature` already
+  enforce this.
+- **Regression-guard is never skipped** for any bug fix.
+- Respect the model strategy: cheap for scouting/dispatcher, mid for
+  implement/test/hunt, premium for architecture/root-cause/council/review.
+- Enforce scope discipline: no unrelated refactors; no API / schema /
+  auth / security / persistence semantic changes without explicit
+  approval (Tier 4 + architect verdict).
 
-## Output
+## Notes for users skipping the plan
 
-1. **Classification:** `<FLOW>` — one-line reason.
-2. Then immediately execute the corresponding command on `$ARGUMENTS`.
+If your project's CLAUDE.md says "skip dispatcher for trivial questions"
+or similar, honour it — but only for Tier 0 questions. Anything that
+might touch code goes through the dispatcher first.

package/dist/templates/commands/quick-feature.md

@@ -0,0 +1,55 @@
+---
+description: Tier 2b — small, local feature in 1–3 files with no API / schema / auth surface. Skips architect and planner; keeps review.
+argument-hint: <the small, local feature to add>
+---
+
+# /quick-feature — Scout → Implement → Test → Review (no architect, no planner)
+
+Use this when the change is small and **clearly local**: a single
+component, a single utility, a small UI addition, a copy change with
+logic, etc. It is the shorter sibling of `/dev-flow` for tasks that do
+not deserve a full architect + planner stage.
+
+If the change is **anywhere near** API contract, data model, auth,
+schema, migrations, payment, security, concurrency, caching, or any
+other cross-cutting concern, do **not** use this — use `/dev-flow`.
+
+## Pre-flight
+
+If `$ARGUMENTS` is vague about scope, **stop and ask** for:
+- the user-visible behavior change,
+- the file(s) you expect to touch (or the surface area you expect).
+
+If the surface obviously exceeds 3 files, abort and route to `/dev-flow`.
+
+## Pipeline
+
+1. **repo-scout** — locate the exact files, confirm scope is local,
+   produce a compact briefing. If scope turns out to be larger than
+   expected, **stop** and recommend `/dev-flow`.
+2. **feature-implementer** — implement the change with the smallest
+   correct edit; add focused tests; targeted verification; no scope
+   creep.
+3. **test-verifier** — run tests, typecheck, lint; fix only breakages
+   caused by this change; report real output.
+4. **final-reviewer** — review the actual diff: correctness, regression,
+   scope creep, security, data consistency, missing tests. Verdict
+   required.
+
+## Rules
+
+- **No scope creep.** Anything found outside the stated change is
+  reported, not done.
+- **No architect, no expert-council** at this tier — that is precisely
+  what `/dev-flow` is for.
+- **Reviewer is mandatory.** This is the only catch for what the
+  skipped stages would have caught.
+- **No API / schema / auth / security / persistence semantic change.**
+  If any of those would be touched, abort and switch to `/dev-flow`.
+- If `final-reviewer` returns BLOCK, recommend `/fix-blockers` — do not
+  auto-loop.
+
+## Output
+
+Stage-by-stage summary + reviewer's APPROVE/BLOCK verdict + the
+concrete next step.

package/dist/templates/commands/quick-fix.md

@@ -0,0 +1,53 @@
+---
+description: Tier 2a — small bug fix where the root cause is already known. Skips audit; keeps regression test + review.
+argument-hint: <the bug + the known root cause + the fix to apply>
+---
+
+# /quick-fix — Fix → Regression → Verify → Review (no audit)
+
+Use this **only** when the root cause is already established and the
+fix is small (typically 1–2 files, ≤ a few lines). It is a shorter
+sibling of `/bug-fix-approved` for changes where a full audit and an
+explicit human approval gate are overkill.
+
+If you are *investigating* a bug, do **not** use this — use `/bug-audit`.
+
+## Pre-flight
+
+`$ARGUMENTS` must contain:
+1. the bug (symptom),
+2. the stated root cause,
+3. the proposed fix.
+
+If any of the three is missing, **stop and ask**. If the dispatcher
+escalated this task (any keyword from auth, schema, migration, payment,
+security, concurrency, cache, …), route to `/bug-fix-approved` instead.
+
+## Pipeline
+
+1. **bug-fixer** — apply the stated fix at the **root cause**; minimal
+   and reversible; strictly within the stated scope.
+2. **regression-guard** — add a regression test that fails on the old
+   behavior and passes on the fixed behavior. **Mandatory.**
+3. **test-verifier** — run tests, typecheck, lint; fix only breakages
+   caused by this change; report real output.
+4. **final-reviewer** — review the actual diff: correctness, regression,
+   scope creep, security, missing tests. Verdict required.
+
+## Rules
+
+- **No skipping regression-guard or final-reviewer.** Both are cheap
+  insurance against re-emergence; both are non-negotiable even at this
+  tier.
+- **Only the stated bug.** Other findings are reported, not fixed.
+- **No symptom-only patch.** If the stated "root cause" turns out to be
+  a symptom, **stop** and route to `/bug-audit`.
+- **No unapproved API/schema/auth/security/persistence change.** If the
+  fix needs one, abort and escalate to `/bug-audit`.
+- If `final-reviewer` returns BLOCK, recommend `/fix-blockers` — do not
+  auto-loop.
+
+## Output
+
+Stage summary + failing→passing regression evidence + reviewer verdict
++ a one-line confirmation that no out-of-scope changes were made.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentcohort",
-  "version": "0.1.1",
+  "version": "0.3.0",
   "description": "Install a principal/staff-level Claude Code AI software engineering organization (agents + workflows + routing rules) into any project with one command.",
   "keywords": [
     "claude",
@@ -43,7 +43,8 @@
   },
   "scripts": {
     "clean": "node -e \"require('fs').rmSync('dist',{recursive:true,force:true})\"",
-    "build": "npm run clean && tsc -p tsconfig.json && node scripts/copy-templates.mjs",
+    "sync-boot": "node scripts/sync-boot-directive.mjs",
+    "build": "npm run clean && npm run sync-boot && tsc -p tsconfig.json && node scripts/copy-templates.mjs",
     "test": "vitest run",
     "test:watch": "vitest",
     "prepublishOnly": "npm run build && npm test"
@@ -52,5 +53,8 @@
     "@types/node": "^20.14.0",
     "typescript": "^5.5.0",
     "vitest": "^2.0.0"
+  },
+  "dependencies": {
+    "@inquirer/prompts": "^7.10.1"
   }
-}
+}