nitm-ai-dev-toolkit 5.0.1 → 5.1.1

package/src/repo/.cursor/commands/factory.md

@@ -11,14 +11,14 @@ argument-hint: "[--in-place] <feature request> | start [--in-place] <request | K
 
 | Agent | When to use |
 |-------|-------------|
-| `factory-spec-writer` | Spec stage (`spec` mode) writes `spec.md`; plan stage (`plan` mode) writes `plan.md` and `tasks.md` |
-| `factory-builder` | Build, repair, and rework: writes `build.md` in the job folder; its code goes in the build location |
-| `factory-reviewer-code` | Review round, `code` lens |
-| `factory-reviewer-security` | Review round, `security` lens |
-| `factory-reviewer-ux` | Review round, `ux` lens (opt-in) |
-| `factory-reviewer-ui` | Review round, `ui` lens (opt-in) |
-| `factory-approver` | Approval gate: writes `decision.md` |
+| `planner` | Spec writer. Spec stage (`spec` mode) writes `spec.md`; plan stage (`plan` mode) writes `plan.md` and `tasks.md`. Brief: `.agents/skills/factory/roles/spec.md` |
+| `implementer` | Builder. Build, repair, and rework: writes `build.md` in the job folder; its code goes in the build location. Brief: `.agents/skills/factory/roles/build.md` |
+| `reviewer` | Code reviewer. Review round, `code` lens. Brief: `.agents/skills/factory/roles/review-code.md` |
+| `reviewer` | Security reviewer, fresh context. Review round, `security` lens. Brief: `.agents/skills/factory/roles/review-security.md` |
+| `reviewer` | UX reviewer. Review round, `ux` lens (opt-in). Brief: `.agents/skills/factory/roles/review-ux.md` |
+| `reviewer` | UI reviewer. Review round, `ui` lens (opt-in). Brief: `.agents/skills/factory/roles/review-ui.md` |
+| `reviewer` | Approver, fresh context. Approval gate: returns the decision, which the orchestrator saves as `decision.md`. Brief: `.agents/skills/factory/roles/approve.md` |
 
-Dispatch each agent by its kebab-case name, as in the first column. Copilot may show a display name; pick the agent with that name.
+Dispatch each native agent by the name in the first column, and tell it to read the brief in the last column first. Reviewers and the approver return their report, and the orchestrator saves it. Copilot may show a display name; pick the agent with that name.
 
 **CRITICAL: READ:** [.claude/prompt-snippets/factory.md](../../.claude/prompt-snippets/factory.md)

package/src/repo/.github/prompts/factory.prompt.md

@@ -12,14 +12,14 @@ argument-hint: "[--in-place] <feature request> | start [--in-place] <request | K
 
 | Agent | When to use |
 |-------|-------------|
-| `factory-spec-writer` | Spec stage (`spec` mode) writes `spec.md`; plan stage (`plan` mode) writes `plan.md` and `tasks.md` |
-| `factory-builder` | Build, repair, and rework: writes `build.md` in the job folder; its code goes in the build location |
-| `factory-reviewer-code` | Review round, `code` lens |
-| `factory-reviewer-security` | Review round, `security` lens |
-| `factory-reviewer-ux` | Review round, `ux` lens (opt-in) |
-| `factory-reviewer-ui` | Review round, `ui` lens (opt-in) |
-| `factory-approver` | Approval gate: writes `decision.md` |
+| `planner` | Spec writer. Spec stage (`spec` mode) writes `spec.md`; plan stage (`plan` mode) writes `plan.md` and `tasks.md`. Brief: `.agents/skills/factory/roles/spec.md` |
+| `implementer` | Builder. Build, repair, and rework: writes `build.md` in the job folder; its code goes in the build location. Brief: `.agents/skills/factory/roles/build.md` |
+| `reviewer` | Code reviewer. Review round, `code` lens. Brief: `.agents/skills/factory/roles/review-code.md` |
+| `reviewer` | Security reviewer, fresh context. Review round, `security` lens. Brief: `.agents/skills/factory/roles/review-security.md` |
+| `reviewer` | UX reviewer. Review round, `ux` lens (opt-in). Brief: `.agents/skills/factory/roles/review-ux.md` |
+| `reviewer` | UI reviewer. Review round, `ui` lens (opt-in). Brief: `.agents/skills/factory/roles/review-ui.md` |
+| `reviewer` | Approver, fresh context. Approval gate: returns the decision, which the orchestrator saves as `decision.md`. Brief: `.agents/skills/factory/roles/approve.md` |
 
-Dispatch each agent by its kebab-case name, as in the first column. Copilot may show a display name; pick the agent with that name.
+Dispatch each native agent by the name in the first column, and tell it to read the brief in the last column first. Reviewers and the approver return their report, and the orchestrator saves it. Copilot may show a display name; pick the agent with that name.
 
 **CRITICAL: READ:** [.claude/prompt-snippets/factory.md](../../.claude/prompt-snippets/factory.md).

package/src/repo/.opencode/agents/admin-portal.md

@@ -1,9 +1,6 @@
 ---
 description: "Build administrator portals with RBAC, system dashboards, reporting, analytics, and operational tooling. Specializes in admin frameworks and monitoring ecosystem tools."
 mode: subagent
-request:
-  body:
-    temperature: 0.1
 permissions:
   - action: edit
     resource: "*"

package/src/repo/.opencode/agents/api-specialist.md

@@ -1,9 +1,6 @@
 ---
 description: "Design and implement API architecture, documentation, and developer experience. Use for REST design, GraphQL, OpenAPI specs, SDK generation, API versioning, and integration patterns."
 mode: subagent
-request:
-  body:
-    temperature: 0.1
 permissions:
   - action: edit
     resource: "*"

package/src/repo/.opencode/agents/backend-architect.md

@@ -1,9 +1,6 @@
 ---
 description: "Design APIs, databases, and server-side architecture. Use for scalable backend systems, data modeling, caching strategies, system design, and infrastructure decisions."
 mode: subagent
-request:
-  body:
-    temperature: 0.1
 permissions:
   - action: edit
     resource: "*"

package/src/repo/.opencode/agents/documenter.md

@@ -1,9 +1,6 @@
 ---
 description: "Analyze codebases and create comprehensive documentation. Use for AGENTS.md, README files, API docs, architecture documentation, and onboarding guides."
 mode: subagent
-request:
-  body:
-    temperature: 0.1
 permissions:
   - action: edit
     resource: "*"

package/src/repo/.opencode/agents/feature-builder.md

@@ -1,9 +1,6 @@
 ---
 description: "Coordinate end-to-end feature development using subagents for planning, implementation, and review. Orchestrates the full development lifecycle."
 mode: subagent
-request:
-  body:
-    temperature: 0.3
 permissions:
   - action: edit
     resource: "*"

package/src/repo/.opencode/agents/frontend-developer.md

@@ -1,9 +1,6 @@
 ---
 description: "Build user interfaces, implement components, handle state management, and optimize frontend performance. Use for responsive, accessible, and performant web applications."
 mode: subagent
-request:
-  body:
-    temperature: 0.1
 permissions:
   - action: edit
     resource: "*"

package/src/repo/.opencode/agents/green.md

@@ -1,9 +1,6 @@
 ---
 description: "Write minimal code to make failing tests pass. Part of the TDD red-green-refactor cycle."
 mode: subagent
-request:
-  body:
-    temperature: 0.1
 permissions:
   - action: edit
     resource: "*"

package/src/repo/.opencode/agents/implementer.md

@@ -1,9 +1,6 @@
 ---
 description: "Implement code changes for a specific task. Follow TDD, write tests alongside code, and self-review before reporting completion."
 mode: subagent
-request:
-  body:
-    temperature: 0.1
 permissions:
   - action: edit
     resource: "*"

package/src/repo/.opencode/agents/planner.md

@@ -1,9 +1,6 @@
 ---
 description: "Break down feature requests into implementation tasks. Read specifications, analyze codebase patterns, and produce structured plans with dependencies."
 mode: subagent
-request:
-  body:
-    temperature: 0.1
 permissions:
   - action: edit
     resource: "*"

package/src/repo/.opencode/agents/red.md

@@ -1,9 +1,6 @@
 ---
 description: "Write failing tests that define expected behavior. Part of the TDD red-green-refactor cycle."
 mode: subagent
-request:
-  body:
-    temperature: 0.1
 permissions:
   - action: edit
     resource: "*"

package/src/repo/.opencode/agents/refactor.md

@@ -1,9 +1,6 @@
 ---
 description: "Improve code quality and structure while keeping all tests passing. Part of the TDD red-green-refactor cycle."
 mode: subagent
-request:
-  body:
-    temperature: 0.1
 permissions:
   - action: edit
     resource: "*"

package/src/repo/.opencode/agents/researcher.md

@@ -1,9 +1,6 @@
 ---
 description: "Research codebase patterns, dependencies, and technical context. Analyze code structure, find relevant examples, and report findings without making changes."
 mode: subagent
-request:
-  body:
-    temperature: 0.1
 permissions:
   - action: edit
     resource: "*"

package/src/repo/.opencode/agents/reviewer.md

@@ -1,9 +1,6 @@
 ---
 description: "Review code changes for correctness, code quality, security, and adherence to project patterns. Provide actionable feedback with specific file and line references. Use immediately after writing or modifying code, or as a quality gate in orchestrated workflows."
 mode: subagent
-request:
-  body:
-    temperature: 0.1
 permissions:
   - action: edit
     resource: "*"

package/src/repo/.opencode/agents/tdd.md

@@ -1,9 +1,6 @@
 ---
 description: "Implement a feature using test-driven development with red-green-refactor cycle. Coordinates specialized subagents for writing failing tests, implementing code, and refactoring."
 mode: subagent
-request:
-  body:
-    temperature: 0.3
 permissions:
   - action: edit
     resource: "*"

package/src/repo/.opencode/commands/factory.md

@@ -10,15 +10,15 @@ description: "Run an opt-in factory job: intake, spec and plan gates, build in a
 
 | Agent | When to use |
 |-------|-------------|
-| `factory-spec-writer` | Spec stage (`spec` mode) writes `spec.md`; plan stage (`plan` mode) writes `plan.md` and `tasks.md` |
-| `factory-builder` | Build, repair, and rework: writes `build.md` in the job folder; edits code at the build location |
-| `factory-reviewer-code` | Review round, `code` lens |
-| `factory-reviewer-security` | Review round, `security` lens |
-| `factory-reviewer-ux` | Review round, `ux` lens (opt-in) |
-| `factory-reviewer-ui` | Review round, `ui` lens (opt-in) |
-| `factory-approver` | Approval gate: writes `decision.md` |
-
-Dispatch each agent by its kebab-case name, as in the first column. Copilot may show a display name; pick the agent with that name.
+| `planner` | Spec writer. Spec stage (`spec` mode) writes `spec.md`; plan stage (`plan` mode) writes `plan.md` and `tasks.md`. Brief: `.agents/skills/factory/roles/spec.md` |
+| `implementer` | Builder. Build, repair, and rework: writes `build.md` in the job folder; edits code at the build location. Brief: `.agents/skills/factory/roles/build.md` |
+| `reviewer` | Code reviewer. Review round, `code` lens. Brief: `.agents/skills/factory/roles/review-code.md` |
+| `reviewer` | Security reviewer, fresh context. Review round, `security` lens. Brief: `.agents/skills/factory/roles/review-security.md` |
+| `reviewer` | UX reviewer. Review round, `ux` lens (opt-in). Brief: `.agents/skills/factory/roles/review-ux.md` |
+| `reviewer` | UI reviewer. Review round, `ui` lens (opt-in). Brief: `.agents/skills/factory/roles/review-ui.md` |
+| `reviewer` | Approver, fresh context. Approval gate: returns the decision, which the orchestrator saves as `decision.md`. Brief: `.agents/skills/factory/roles/approve.md` |
+
+Dispatch each native agent by the name in the first column, and tell it to read the brief in the last column first. Reviewers and the approver return their report, and the orchestrator saves it. Copilot may show a display name; pick the agent with that name.
 
 ## Arguments
 

package/src/repo/AGENTS.md

@@ -94,9 +94,9 @@ When stopping to ask a question, do not just post an open-ended block of text. F
 
 OpenCode use with toolkit 4.0.0 and later requires native V2; Slim >=3.0.0 requires OpenCode >=2.0.7. Read [native setup](README.md#native-opencode-v2) before migration. All bootstrap flows run `opencode --version` before changes for confirmed OpenCode use: V1 must stop and upgrade OpenCode or remain on toolkit 3.x; missing/unparseable versions block OpenCode steps and marking the installed toolkit version (4.0.0 or later). Directory presence is not harness proof; other-harness-only setup is unaffected.
 
-Use `agents`/`system`, `request.body.temperature`, ordered `permissions` (`action`/`resource`/`effect`, `shell`/`subagent`), `commands`, `plugins`, directory-based `skills`, and `mcp.servers`. Preserve user models/providers/plugins and Slim's separate strict schema; do not host-migrate its `prompt`/`permission`/`variant` fields or silently upgrade the separate external starter. The OpenCode Ponytail plugin (`@dietrichgebert/ponytail`) is installed globally by the operator; the toolkit does not register it. Bootstrap may check for it and tell the operator to install it globally, but must never add it to project config. `opentmux` stays commented as V1-only. Remove only known obsolete toolkit Ponytail `.js`/`.mjs`/`.ts` copies after backup and approval, and preserve unrelated local plugins. No local API port or bridge is used; plugin/runtime integration is untested. The Claude Code hooks in `hooks/*.json` use `${CLAUDE_PLUGIN_ROOT}` and are plugin-only: they fire only when installed through the Ponytail plugin, not from project settings.
+Use `agents`/`system`, ordered `permissions` (`action`/`resource`/`effect`, `shell`/`subagent`), `commands`, `plugins`, directory-based `skills`, and `mcp.servers`. Preserve user models/providers/plugins and Slim's separate strict schema; do not host-migrate its `prompt`/`permission`/`variant` fields or silently upgrade the separate external starter. The OpenCode Ponytail plugin (`@dietrichgebert/ponytail`) is installed globally by the operator; the toolkit does not register it. Bootstrap may check for it and tell the operator to install it globally, but must never add it to project config. `opentmux` stays commented as V1-only. Remove only known obsolete toolkit Ponytail `.js`/`.mjs`/`.ts` copies after backup and approval, and preserve unrelated local plugins. No local API port or bridge is used; plugin/runtime integration is untested. The Claude Code hooks in `hooks/*.json` use `${CLAUDE_PLUGIN_ROOT}` and are plugin-only: they fire only when installed through the Ponytail plugin, not from project settings.
 
-Ambient `AGENTS.md` works. Config `instructions` entries load only when the runtime injects them (OpenChamber does; other hosts may not), so do not rely on them: write explicit "Read and follow" wrappers regardless. `@refs` are ordinary text: explicitly read linked snippets. `request.body.temperature` remains in config, but the current runner does not send it. Preserve `lsp: true`; use project lint/typecheck/compiler commands, not unavailable V2 LSP diagnostics. Back up V1 before native conversion; converted files are incompatible with V1. Activate on the next run/restart.
+Ambient `AGENTS.md` works. Config `instructions` entries load only when the runtime injects them (OpenChamber does; other hosts may not), so do not rely on them: write explicit "Read and follow" wrappers regardless. `@refs` are ordinary text: explicitly read linked snippets. Do not set a temperature; the current runner does not send one. Preserve `lsp: true`; use project lint/typecheck/compiler commands, not unavailable V2 LSP diagnostics. Back up V1 before native conversion; converted files are incompatible with V1. Activate on the next run/restart.
 
 ## Detailed Instructions
 
@@ -381,7 +381,7 @@ Subagents are typically **agent-initiated** — the coordinator decides when to
 
 ### Factory (opt-in)
 
-An optional file-driven pipeline for feature-sized work, alongside Feature Builder and TDD. Start it with `/factory <request>` in Claude Code, Copilot, Cursor, or OpenCode. The orchestrator reads the project and asks only the glaring questions before it creates anything, then asks you to continue or change the answers. Resume a feature with `/factory start <KEY | slug | job-id>`. Codex runs the orchestrator skill directly. Each job belongs to a feature folder in `docs/features/`, stops at a spec gate and a plan gate for your confirmation before any ticket or build, builds in its own git worktree unless you ask for `--in-place`, and moves its tickets when the project tracks them. The orchestrator is [.agents/skills/factory/SKILL.md](.agents/skills/factory/SKILL.md). Its seven agents are `factory-spec-writer`, `factory-builder`, `factory-reviewer-code`, `factory-reviewer-security`, `factory-reviewer-ux`, `factory-reviewer-ui`, and `factory-approver`, with wrappers in each harness's agents folder that point to the shared snippets in `.claude/agents-snippets/`. Setup, placeholders, and approval modes are in [factory/README.md](factory/README.md).
+An optional file-driven pipeline for feature-sized work, alongside Feature Builder and TDD. Start it with `/factory <request>` in Claude Code, Copilot, Cursor, or OpenCode. The orchestrator reads the project and asks only the glaring questions before it creates anything, then asks you to continue or change the answers. Resume a feature with `/factory start <KEY | slug | job-id>`. Codex runs the orchestrator skill directly. Each job belongs to a feature folder in `docs/features/`, stops at a spec gate and a plan gate for your confirmation before any ticket or build, builds in its own git worktree unless you ask for `--in-place`, and moves its tickets when the project tracks them. The orchestrator is [.agents/skills/factory/SKILL.md](.agents/skills/factory/SKILL.md). It runs on the native `planner` (spec and plan), `implementer` (build, repair, and rework), and `reviewer` (the code, security, ux, and ui lenses, and approval) agents. Each role's Factory rules are in its brief in `.agents/skills/factory/roles/`. Setup, placeholders, and approval modes are in [factory/README.md](factory/README.md).
 
 ## Documentation Requirements
 

package/src/repo/README.md

@@ -124,9 +124,9 @@ The template's local `plugins` array registers `opencode-mem` and `@tarquinen/op
 
 With approval, merge `"plugins": ["oh-my-opencode-slim@3.0.0"]` locally without replacing other registrations. Slim's separate strict schema retains `prompt`, `permission`, and `variant`; do not host-migrate it. The external starter is a separate package, not automatically upgraded. Plugin/runtime integration has not been tested.
 
-Use native `agents`/`system`, `request.body.temperature`, ordered `permissions` (`action`/`resource`/`effect`, `shell`/`subagent`), `commands`/`subagent`, `plugins` with `{ "package": "…", "options": {} }`, skill-directory arrays, and `mcp.servers` with inverse `disabled` and `timeout.catalog`/`timeout.execution`. Follow the [V2 migration guide](https://opencode.ai/v2/docs/migrate-v1) and [plugin guide](https://opencode.ai/v2/docs/build/plugins); the published schema may describe V1 and is not V2 validation authority.
+Use native `agents`/`system`, ordered `permissions` (`action`/`resource`/`effect`, `shell`/`subagent`), `commands`/`subagent`, `plugins` with `{ "package": "…", "options": {} }`, skill-directory arrays, and `mcp.servers` with inverse `disabled` and `timeout.catalog`/`timeout.execution`. Follow the [V2 migration guide](https://opencode.ai/v2/docs/migrate-v1) and [plugin guide](https://opencode.ai/v2/docs/build/plugins); the published schema may describe V1 and is not V2 validation authority.
 
-Ambient `AGENTS.md` works. Config `instructions` entries load only when the runtime injects them (OpenChamber does; other hosts may not), so do not rely on them: write explicit "Read and follow" wrappers regardless. `@refs` are ordinary text: read linked snippets explicitly. `request.body.temperature` remains in config, but the current runner does not send it. Preserve `lsp: true`; use project lint/typecheck/compiler commands (no V2 LSP tools/diagnostics currently).
+Ambient `AGENTS.md` works. Config `instructions` entries load only when the runtime injects them (OpenChamber does; other hosts may not), so do not rely on them: write explicit "Read and follow" wrappers regardless. `@refs` are ordinary text: read linked snippets explicitly. Do not set a temperature; the current runner does not send one. Preserve `lsp: true`; use project lint/typecheck/compiler commands (no V2 LSP tools/diagnostics currently).
 
 `--env opencode` supplies minimal shared references without activating other harnesses. Back up V1, merge rather than clobber models/providers/plugins, and activate on the next run/restart. Native-converted files are incompatible with V1. Verify with installed V2; `doctor` is not runtime validation.
 

package/src/repo/factory/README.md

@@ -4,14 +4,14 @@ A file-driven pipeline for feature-sized work. One orchestrator session runs a f
 
 The factory is **opt-in**. It sits alongside Feature Builder and TDD and changes nothing they do. Nothing runs until you invoke `/factory`. Quick fixes stay direct.
 
-Orchestrator: [.agents/skills/factory/SKILL.md](../.agents/skills/factory/SKILL.md). Its agents have wrappers in `.claude/agents/`, `.github/agents/`, `.cursor/agents/`, and `.opencode/agents/`. See [Harnesses](#harnesses).
+Orchestrator: [.agents/skills/factory/SKILL.md](../.agents/skills/factory/SKILL.md). It runs on the native `planner`, `implementer`, and `reviewer` agents, and its role briefs live in `.agents/skills/factory/roles/`. See [Harnesses](#harnesses).
 
 ## Install
 
 1. **Install it.** The toolkit installer copies this folder into your project root as `factory/`.
 2. **Document your commands in `AGENTS.md`.** Agents read setup, dev, test, lint, and build commands from there, and the dev command must document how to pass a port (a flag or an env var). Without them the factory stops and asks you. Edit `factory/worktree-bootstrap.md` for the setup each new worktree needs (see Build location).
-3. **Replace the placeholders** below in `.agents/skills/factory/SKILL.md` and `.claude/agents-snippets/factory-*.md`. Bootstrap does this when it runs.
-4. **Commit** `factory/README.md`, `factory/.gitignore`, and the factory agents and skill. Runtime state is ignored (see Layout). Feature docs in the feature folders are committed with the code.
+3. **Replace the placeholders** below in `.agents/skills/factory/SKILL.md` and `.agents/skills/factory/roles/*.md`. Bootstrap does this when it runs.
+4. **Commit** `factory/README.md`, `factory/.gitignore`, and the factory skill with its role briefs. Runtime state is ignored (see Layout). Feature docs in the feature folders are committed with the code.
 5. **Run it** from your harness: `/factory <feature request>` to start from a request, or `/factory start <KEY | slug | job-id>` to pick up an existing feature. See [Harnesses](#harnesses) for each entry point.
 
 ## Placeholders
@@ -47,15 +47,17 @@ There is no queue. Start each feature with a request, a ticket key, or a slug.
 
 ## Harnesses
 
-The orchestrator and agents are the same in every harness. Each harness has its own entry point and agent wrappers. Every wrapper points to the shared snippets in `.claude/agents-snippets/`.
+The orchestrator and agents are the same in every harness. Each harness has its own entry point. Every role runs on a native agent: `planner` for the spec writer, `implementer` for the builder, and `reviewer` for each review lens and the approver, with the lens or approval mode named in the dispatch. The role briefs in `.agents/skills/factory/roles/` hold the Factory rules.
 
-| Harness | Entry point | Agent wrappers |
+| Harness | Entry point | Native agents |
 | --- | --- | --- |
 | Claude Code | `/factory` (`.claude/commands/factory.md`) | `.claude/agents/` |
 | GitHub Copilot | `/factory` prompt (`.github/prompts/factory.prompt.md`) | `.github/agents/` |
 | Cursor | `/factory` (`.cursor/commands/factory.md`) | `.cursor/agents/` |
 | OpenCode | `/factory` (`.opencode/commands/factory.md`) | `.opencode/agents/` |
-| Codex | The factory skill (`.agents/skills/factory/SKILL.md`) | None. Runs the snippets directly, one role at a time in one session (see Harness notes in SKILL.md) |
+| Codex | The factory skill (`.agents/skills/factory/SKILL.md`) | None. Runs each role brief directly, one role at a time in one session (see Harness notes in SKILL.md) |
+
+In OpenCode with the Slim plugin, `oracle` and `designer` are optional alternatives to `reviewer`, if you configure them as edit-denied. The default is the native agents.
 
 ## Feature folders and sizing
 
@@ -89,7 +91,7 @@ Any value other than exactly `auto` is treated as `human`.
 
 ## Build location
 
-By default each job builds in its own git worktree on branch `factory/<id>`, so the main checkout stays free. A job's `job.json` records this in `worktree` (`true`) and the worktree path in `worktree_path`.
+By default each job builds in its own git worktree on branch `factory/<id>`, so the main checkout stays free. A job's `job.json` records this in `worktree` (`true`) and the worktree path in `worktree_path`. Worktrees live in `.worktrees/<id>/` at the repo root; the factory adds `.worktrees/` to `.gitignore` if it is missing.
 
 ### Worktree bootstrap
 
@@ -136,15 +138,15 @@ The parent and every sub-task move together. The orchestrator uses the project's
 ## How a job runs
 
 1. **Intake.** For `/factory <request>`, the orchestrator reads the project and asks only the glaring questions. You answer them, then choose to continue or change the answers. Nothing is created before you continue.
-2. **Spec.** `factory-spec-writer` inspects the code and writes `spec.md` with numbered acceptance criteria.
+2. **Spec.** `planner` (spec mode) inspects the code and writes `spec.md` with numbered acceptance criteria.
 3. **Spec gate.** You confirm the spec, or request changes.
-4. **Plan.** `factory-spec-writer` in plan mode writes `plan.md` (with the size) and `tasks.md` for large work.
+4. **Plan.** `planner` in plan mode writes `plan.md` (with the size) and `tasks.md` for large work.
 5. **Plan gate.** You confirm the plan, or request changes. Nothing is ticketed or built before you confirm.
 6. **Tickets and build location.** If tracked, the parent and sub-tasks are created and their keys are written into `tasks.md`. The folder is renamed to carry the key. Then a builder gets its own git worktree on branch `factory/<id>` (bootstrapped first, see Build location), or the in-place branch. The job's feature docs are committed on that branch.
 7. **Build.** The builder follows TDD, ticks `tasks.md` as it completes tasks, writes `build.md`, and writes `qa.md`.
-8. **Review.** Every configured lens reviews the branch, writing `round-N/review-<lens>.md` with `VERDICT: PASS` or `VERDICT: CHANGES`.
+8. **Review.** Every configured lens reviews the branch, and each reviewer's report is saved as `round-N/review-<lens>.md` with `VERDICT: PASS` or `VERDICT: CHANGES`.
 9. **Repair.** Any `CHANGES` sends the builder back to fix it. Two repairs at most, so three review rounds. Then the job escalates to you.
-10. **Approve.** `factory-approver` writes `decision.md` with `APPROVE` or `ESCALATE`.
+10. **Approve.** `reviewer` in approval mode (fresh context) returns `APPROVE` or `ESCALATE`, and the orchestrator writes it to `decision.md`.
 11. **Merge.** `local` mode: local only, never pushed. `push-mr` mode: push and open the request, then finish on the second approve (see Merge modes). `--no-ff` merge, the project's test command, then the worktree (if any) and branch are removed. Feature docs in the main checkout are set aside first (copies in `main-copies/`), so the merge brings in the branch's committed copies.
 
 Jobs that escalate (`needs-human`) keep their branch and worktree so you can inspect them. Use `/factory rework` or `/factory approve` to continue.

package/src/monorepo/.claude/agents/factory-approver.agent.md

@@ -1,12 +0,0 @@
----
-name: "factory-approver"
-description: "Decide whether a factory job is approved for merge or escalated to a human, checking history, final diff, a spot-checked criterion, a dry-run merge, and escalation triggers. Dispatched by the factory orchestrator."
-tools: ["Read", "Grep", "Glob", "Write", "Bash(*)"]
-user-invocable: true
----
-
-# Factory Approver Agent
-
-Decide whether a factory job is approved for merge or escalated to a human.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-approver.md](../agents-snippets/factory-approver.md)

package/src/monorepo/.claude/agents/factory-builder.agent.md

@@ -1,12 +0,0 @@
----
-name: "factory-builder"
-description: "Build a factory job's feature inside its git worktree, following TDD, and log results in build.md. Runs in build, repair, or rework mode. Dispatched by the factory orchestrator."
-tools: ["Read", "Grep", "Glob", "Edit", "Write", "Bash(*)"]
-user-invocable: true
----
-
-# Factory Builder Agent
-
-Build a factory job's feature inside its git worktree, following TDD, and log results in build.md.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-builder.md](../agents-snippets/factory-builder.md)

package/src/monorepo/.claude/agents/factory-reviewer-code.agent.md

@@ -1,12 +0,0 @@
----
-name: "factory-reviewer-code"
-description: "Review a factory job's branch for correctness, simplicity, reuse, conventions, and docs, and run the project's checks. Dispatched by the factory orchestrator."
-tools: ["Read", "Grep", "Glob", "Write", "Bash(*)"]
-user-invocable: true
----
-
-# Factory Reviewer (Code) Agent
-
-Review a factory job's branch for correctness, simplicity, reuse, conventions, and docs, and run the project's checks.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-reviewer-code.md](../agents-snippets/factory-reviewer-code.md)

package/src/monorepo/.claude/agents/factory-reviewer-security.agent.md

@@ -1,12 +0,0 @@
----
-name: "factory-reviewer-security"
-description: "Audit a factory job's branch diff for security problems in a fresh context. Dispatched by the factory orchestrator."
-tools: ["Read", "Grep", "Glob", "Write", "Bash(*)"]
-user-invocable: true
----
-
-# Factory Reviewer (Security) Agent
-
-Audit a factory job's branch diff for security problems in a fresh context.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-reviewer-security.md](../agents-snippets/factory-reviewer-security.md)

package/src/monorepo/.claude/agents/factory-reviewer-ui.agent.md

@@ -1,12 +0,0 @@
----
-name: "factory-reviewer-ui"
-description: "Review a factory job's feature at its build location for design-system consistency, spacing, typography, contrast, and responsive layout, without signing in or writing to real services. Dispatched by the factory orchestrator."
-tools: ["Read", "Grep", "Glob", "Write", "Bash(*)"]
-user-invocable: true
----
-
-# Factory Reviewer (UI) Agent
-
-Review a factory job's feature at its build location for design-system consistency, spacing, typography, contrast, and responsive layout, without signing in or writing to real services.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-reviewer-ui.md](../agents-snippets/factory-reviewer-ui.md)

package/src/monorepo/.claude/agents/factory-reviewer-ux.agent.md

@@ -1,12 +0,0 @@
----
-name: "factory-reviewer-ux"
-description: "Review a factory job's feature at its build location for flows, states, copy, accessibility, and mobile, without signing in or writing to real services. Dispatched by the factory orchestrator."
-tools: ["Read", "Grep", "Glob", "Write", "Bash(*)"]
-user-invocable: true
----
-
-# Factory Reviewer (UX) Agent
-
-Review a factory job's feature at its build location for flows, states, copy, accessibility, and mobile, without signing in or writing to real services.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-reviewer-ux.md](../agents-snippets/factory-reviewer-ux.md)

package/src/monorepo/.claude/agents/factory-spec-writer.agent.md

@@ -1,12 +0,0 @@
----
-name: "factory-spec-writer"
-description: "Write a testable spec.md for a factory job from a feature request, after inspecting the real code. Dispatched by the factory orchestrator."
-tools: ["Read", "Grep", "Glob", "Write", "Bash(*)"]
-user-invocable: true
----
-
-# Factory Spec Writer Agent
-
-Write a testable spec.md for a factory job from a feature request, after inspecting the real code.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-spec-writer.md](../agents-snippets/factory-spec-writer.md)

package/src/monorepo/.cursor/agents/factory-approver.agent.md

@@ -1,11 +0,0 @@
----
-description: "Decide whether a factory job is approved for merge or escalated to a human, checking history, final diff, a spot-checked criterion, a dry-run merge, and escalation triggers."
-globs:
-alwaysApply: agent
----
-
-# Factory Approver Agent
-
-Decide whether a factory job is approved for merge or escalated to a human.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-approver.md](../../.claude/agents-snippets/factory-approver.md)

package/src/monorepo/.cursor/agents/factory-builder.agent.md

@@ -1,11 +0,0 @@
----
-description: "Build a factory job's feature inside its git worktree, following TDD, and log results in build.md."
-globs:
-alwaysApply: agent
----
-
-# Factory Builder Agent
-
-Build a factory job's feature inside its git worktree, following TDD, and log results in build.md.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-builder.md](../../.claude/agents-snippets/factory-builder.md)

package/src/monorepo/.cursor/agents/factory-reviewer-code.agent.md

@@ -1,11 +0,0 @@
----
-description: "Review a factory job's branch for correctness, simplicity, reuse, conventions, and docs, and run the project's checks."
-globs:
-alwaysApply: agent
----
-
-# Factory Reviewer (Code) Agent
-
-Review a factory job's branch for correctness, simplicity, reuse, conventions, and docs, and run the project's checks.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-reviewer-code.md](../../.claude/agents-snippets/factory-reviewer-code.md)

package/src/monorepo/.cursor/agents/factory-reviewer-security.agent.md

@@ -1,11 +0,0 @@
----
-description: "Audit a factory job's branch diff for security problems in a fresh context."
-globs:
-alwaysApply: agent
----
-
-# Factory Reviewer (Security) Agent
-
-Audit a factory job's branch diff for security problems in a fresh context.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-reviewer-security.md](../../.claude/agents-snippets/factory-reviewer-security.md)

package/src/monorepo/.cursor/agents/factory-reviewer-ui.agent.md

@@ -1,11 +0,0 @@
----
-description: "Review a factory job's feature at its build location for design-system consistency, spacing, typography, contrast, and responsive layout, without signing in or writing to real services."
-globs:
-alwaysApply: agent
----
-
-# Factory Reviewer (UI) Agent
-
-Review a factory job's feature at its build location for design-system consistency, spacing, typography, contrast, and responsive layout, without signing in or writing to real services.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-reviewer-ui.md](../../.claude/agents-snippets/factory-reviewer-ui.md)

package/src/monorepo/.cursor/agents/factory-reviewer-ux.agent.md

@@ -1,11 +0,0 @@
----
-description: "Review a factory job's feature at its build location for flows, states, copy, accessibility, and mobile, without signing in or writing to real services."
-globs:
-alwaysApply: agent
----
-
-# Factory Reviewer (UX) Agent
-
-Review a factory job's feature at its build location for flows, states, copy, accessibility, and mobile, without signing in or writing to real services.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-reviewer-ux.md](../../.claude/agents-snippets/factory-reviewer-ux.md)

package/src/monorepo/.cursor/agents/factory-spec-writer.agent.md

@@ -1,11 +0,0 @@
----
-description: "Write a testable spec.md for a factory job from a feature request, after inspecting the real code."
-globs:
-alwaysApply: agent
----
-
-# Factory Spec Writer Agent
-
-Write a testable spec.md for a factory job from a feature request, after inspecting the real code.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-spec-writer.md](../../.claude/agents-snippets/factory-spec-writer.md)

package/src/monorepo/.github/agents/factory-approver.agent.md

@@ -1,11 +0,0 @@
----
-name: Factory Approver
-description: "Decide whether a factory job is approved for merge or escalated to a human, checking history, final diff, a spot-checked criterion, a dry-run merge, and escalation triggers. Dispatched by the factory orchestrator."
-tools: ["read", "search", "edit", "runInTerminal", "terminalLastCommand"]
----
-
-# Factory Approver Agent
-
-Decide whether a factory job is approved for merge or escalated to a human.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-approver.md](../../.claude/agents-snippets/factory-approver.md)

package/src/monorepo/.github/agents/factory-builder.agent.md

@@ -1,11 +0,0 @@
----
-name: Factory Builder
-description: "Build a factory job's feature inside its git worktree, following TDD, and log results in build.md. Runs in build, repair, or rework mode. Dispatched by the factory orchestrator."
-tools: ["read", "search", "edit", "runInTerminal", "terminalLastCommand"]
----
-
-# Factory Builder Agent
-
-Build a factory job's feature inside its git worktree, following TDD, and log results in build.md.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-builder.md](../../.claude/agents-snippets/factory-builder.md)

package/src/monorepo/.github/agents/factory-reviewer-code.agent.md

@@ -1,11 +0,0 @@
----
-name: Factory Reviewer Code
-description: "Review a factory job's branch for correctness, simplicity, reuse, conventions, and docs, and run the project's checks. Dispatched by the factory orchestrator."
-tools: ["read", "search", "edit", "runInTerminal", "terminalLastCommand"]
----
-
-# Factory Reviewer (Code) Agent
-
-Review a factory job's branch for correctness, simplicity, reuse, conventions, and docs, and run the project's checks.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-reviewer-code.md](../../.claude/agents-snippets/factory-reviewer-code.md)

package/src/monorepo/.github/agents/factory-reviewer-security.agent.md

@@ -1,11 +0,0 @@
----
-name: Factory Reviewer Security
-description: "Audit a factory job's branch diff for security problems in a fresh context. Dispatched by the factory orchestrator."
-tools: ["read", "search", "edit", "runInTerminal", "terminalLastCommand"]
----
-
-# Factory Reviewer (Security) Agent
-
-Audit a factory job's branch diff for security problems in a fresh context.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-reviewer-security.md](../../.claude/agents-snippets/factory-reviewer-security.md)

package/src/monorepo/.github/agents/factory-reviewer-ui.agent.md

@@ -1,11 +0,0 @@
----
-name: Factory Reviewer UI
-description: "Review a factory job's feature at its build location for design-system consistency, spacing, typography, contrast, and responsive layout, without signing in or writing to real services. Dispatched by the factory orchestrator."
-tools: ["read", "search", "edit", "runInTerminal", "terminalLastCommand"]
----
-
-# Factory Reviewer (UI) Agent
-
-Review a factory job's feature at its build location for design-system consistency, spacing, typography, contrast, and responsive layout, without signing in or writing to real services.
-
-**CRITICAL: READ:** [.claude/agents-snippets/factory-reviewer-ui.md](../../.claude/agents-snippets/factory-reviewer-ui.md)