prizmkit 1.1.127 → 1.1.128

package/bundled/VERSION.json

@@ -1,5 +1,5 @@
 {
-  "frameworkVersion": "1.1.127",
-  "bundledAt": "2026-07-12T15:13:24.439Z",
-  "bundledFrom": "d634625"
+  "frameworkVersion": "1.1.128",
+  "bundledAt": "2026-07-12T17:44:30.283Z",
+  "bundledFrom": "341ff48"
 }

package/bundled/adapters/claude/command-adapter.js

@@ -7,6 +7,8 @@
  *   - Only `description` in frontmatter
  *   - `${SKILL_DIR}` references rewritten to relative paths
  *   - Skills with assets/scripts use directory structure
+ *
+ * Code-review-specific named agents are retired; review runs in the Main Agent.
  */
 
 import { parseFrontmatter, buildMarkdown } from '../shared/frontmatter.js';
@@ -20,67 +22,6 @@ function toClaudePrizmkitCommand(sub) {
   return `prizmkit-${normalized}`;
 }
 
-const REVIEWER_ROLE_SKILL = 'prizmkit-code-review';
-const REVIEWER_ROLE_FILE = 'prizmkit-code-reviewer.md';
-const REVIEWER_REQUIRED_UNAVAILABLE = new Set([
-  'workspace-mutation',
-  'shell-execution',
-  'downstream-execution-creation',
-  'delegation-equivalent-behavior',
-  'orchestration-reentry',
-  'skill-invocation',
-]);
-const CLAUDE_OBSERVATIONAL_TOOLS = new Map([
-  ['read-files', 'Read'],
-  ['search-content', 'Grep'],
-  ['search-paths', 'Glob'],
-]);
-
-function buildReviewerRoleDefinition(roleManifest) {
-  if (!roleManifest || roleManifest.role_name !== 'prizmkit-code-reviewer') {
-    throw new Error('prizmkit-code-review requires assets/reviewer-role.json');
-  }
-  const unavailable = new Set(roleManifest.unavailable_capabilities || []);
-  for (const capability of REVIEWER_REQUIRED_UNAVAILABLE) {
-    if (!unavailable.has(capability)) {
-      throw new Error(`Reviewer role must make ${capability} unavailable`);
-    }
-  }
-  if (roleManifest.execution_mode !== 'foreground-independent-review') {
-    throw new Error('Reviewer role must use foreground independent review');
-  }
-
-  const tools = (roleManifest.observational_capabilities || []).map(capability => {
-    const tool = CLAUDE_OBSERVATIONAL_TOOLS.get(capability);
-    if (!tool) throw new Error(`Unsupported Reviewer observational capability: ${capability}`);
-    return tool;
-  });
-  if (tools.length === 0) throw new Error('Reviewer role requires observational capabilities');
-
-  return `---
-name: ${roleManifest.role_name}
-description: ${roleManifest.description}
-tools: ${tools.join(', ')}
-permissionMode: dontAsk
-maxTurns: 80
----
-
-You are a capability-restricted read-only Reviewer execution role.
-
-Read and follow the complete prompt supplied by the Main Agent. Perform the complete review yourself. Your tool allowlist intentionally excludes workspace mutation, shell execution, skill invocation, downstream execution creation, delegation-equivalent behavior, and orchestration re-entry.
-
-If the supplied review cannot be completed with the available read-only capabilities and current context, return \`BLOCKED\` with the missing capability or context. Do not delegate, request a helper, mutate the workspace, or claim \`PASS\` from incomplete work.
-`;
-}
-
-export function getClaudeSkillAgentDefinitions(skillName, roleManifest = null) {
-  if (skillName !== REVIEWER_ROLE_SKILL) return [];
-  return [{
-    fileName: REVIEWER_ROLE_FILE,
-    content: buildReviewerRoleDefinition(roleManifest),
-  }];
-}
-
 /**
  * Convert a core SKILL.md to Claude Code command.md format.
  * @param {string} skillContent - Content of the core SKILL.md
@@ -158,20 +99,6 @@ export async function installCommand(corePath, targetRoot) {
     for (const subdir of subdirs) {
       cpSync(path.join(corePath, subdir), path.join(targetDir, subdir), { recursive: true });
     }
-
-    // Install platform-native capability roles generated from canonical skill semantics.
-    const roleManifestPath = path.join(corePath, 'assets', 'reviewer-role.json');
-    const roleManifest = existsSync(roleManifestPath)
-      ? JSON.parse(await readFile(roleManifestPath, 'utf8'))
-      : null;
-    const agentDefinitions = getClaudeSkillAgentDefinitions(skillName, roleManifest);
-    if (agentDefinitions.length > 0) {
-      const agentsDir = path.join(targetRoot, '.claude', 'agents');
-      mkdirSync(agentsDir, { recursive: true });
-      for (const definition of agentDefinitions) {
-        await writeFile(path.join(agentsDir, definition.fileName), definition.content);
-      }
-    }
   } else {
     // Single file command
     const targetDir = path.join(targetRoot, COMMANDS_DIR);

package/bundled/dev-pipeline/assets/skill-subagent-integration.md

@@ -2,18 +2,19 @@
 
 ## Overview
 
-`dev-pipeline` drives feature, bugfix, and refactor sessions through the unified Python outer loop. Each iteration spawns a new AI CLI session with a bootstrap prompt that makes the main session orchestrator responsible for planning, implementation, review coordination, testing, retrospective, and commit.
+`dev-pipeline` drives feature, bugfix, and refactor sessions through the unified Python outer loop. Each iteration starts an AI CLI session with a bootstrap prompt that makes the main session orchestrator responsible for planning, implementation, review, testing, retrospective, and commit.
+
+Skill behavior is owned by canonical skills:
 
-Implementation and review behavior is owned by skills:
 - `/prizmkit-implement` executes plan tasks directly and may use its local implementation-subagent reference for narrow active-checkout delegation.
-- `/prizmkit-code-review` owns its internal inline reviewer subagent review loop through its local reviewer prompt reference.
+- `/prizmkit-code-review` runs its complete bounded review and repair loop directly in the Main Agent.
 
 ## Architecture
 
 ```text
 dev-pipeline (outer loop)
     |
-    +-- cli.py feature/bugfix/refactor       Python runtime CLI — picks next item, spawns CLI
+    +-- cli.py feature/bugfix/refactor       Python runtime CLI — picks next item, starts CLI
     +-- scripts/                             Python state and prompt management scripts
     +-- templates/                           Session prompt templates and sections
     |
@@ -22,7 +23,7 @@ dev-pipeline (outer loop)
             +-- Context snapshot + specify + plan (orchestrator)
             +-- Local plan/spec review loop inside prizmkit-plan
             +-- Implement through /prizmkit-implement
-            +-- Code review through /prizmkit-code-review inline reviewer loop
+            +-- Main-Agent code review through /prizmkit-code-review
             +-- Scoped test gate and browser attempt
             +-- Retrospective & commit (orchestrator)
 ```
@@ -30,24 +31,26 @@ dev-pipeline (outer loop)
 ## Skill-Owned References
 
 | Skill | Reference | Purpose |
-|-------|-----------|---------|
+|---|---|---|
 | `/prizmkit-implement` | skill-local `references/implementation-subagent-procedure.md` | Optional narrow implementation delegation contract |
-| `/prizmkit-code-review` | skill-local `references/reviewer-agent-prompt.md` | Required independent reviewer-loop prompt |
+| `/prizmkit-code-review` | skill-local `references/review-report-template.md` | Main-Agent review progress and terminal-result contract |
 
-The orchestrator role is handled by the main agent directly — no separate platform agent definition or team topology is installed.
+The orchestrator role is handled by the Main Agent directly. Code review does not install or invoke a separate platform agent definition.
 
 ## Active Checkout Contract
 
-Any skill-level implementation or review subagent must run in the expected active git top-level. The prompt contract forbids git worktrees, tool-created worktree isolation, copied repository checkouts, remote isolated checkouts, branch switching, and operating outside the expected active checkout.
+Any optional implementation subagent must run in the expected active git top-level. Its prompt contract forbids git worktrees, tool-created worktree isolation, copied repository checkouts, remote isolated checkouts, branch switching, and operating outside the expected active checkout.
+
+Code review always runs in the Main Agent's current workspace so staged, unstaged, and untracked content remains authoritative.
 
-Pipeline runner-owned linked worktrees are an outer runtime option. They do not authorize skill-level subagents to create or enter their own worktrees.
+Pipeline runner-owned linked worktrees are an outer runtime option. They do not authorize a skill-level execution unit to create or enter another worktree.
 
 ## Pipeline Scripts
 
 Located at `.prizmkit/dev-pipeline/scripts/`:
 
 | Script | Purpose |
-|--------|---------|
+|---|---|
 | `init-pipeline.py` | Initialize feature pipeline state |
 | `init-bugfix-pipeline.py` | Initialize bugfix pipeline state |
 | `init-refactor-pipeline.py` | Initialize refactor pipeline state |

package/bundled/dev-pipeline/prizmkit_runtime/runtime_helper.py

@@ -258,7 +258,6 @@ def _text_final_verdict(args: argparse.Namespace) -> HelperResult:
     markers = {
         "PASS": "REVIEW_PASS",
         "NEEDS_FIXES": "REVIEW_NEEDS_FIXES",
-        "BLOCKED": "REVIEW_BLOCKED",
     }
     marker = markers.get(verdict)
     if not marker:

package/bundled/dev-pipeline/scripts/generate-recovery-prompt.py

@@ -97,9 +97,9 @@ Implement the minimal fix (red → green):
         """\
 Verify fix quality:
 1. Run `/prizmkit-code-review` with the current bugfix artifact directory.
-2. The Main Agent self-reviews for up to ten rounds; semantic risk limits independent Reviewers to low=0, medium=1, high=2.
-3. Reviewer 2 requires an accepted high finding from Reviewer 1 plus repair, tests, and Main-Agent reconvergence; Reviewer 3 is forbidden.
-4. Continue only after the report's last Final Result is valid; preserve append-only progress for recovery.""",
+2. The Main Agent reviews the complete current change for up to ten rounds, directly repairs accepted findings, and verifies repairs.
+3. Converge only when accepted and unresolved findings are both zero; missing required evidence or round-ten non-convergence produces NEEDS_FIXES.
+4. Do not spawn a code-review-specific subagent. Continue only after the report's last Final Result is valid; preserve append-only progress for recovery.""",
     ),
     6: (
         "User Verification",

package/bundled/dev-pipeline/templates/bootstrap-prompt.md

@@ -30,7 +30,7 @@ Infer what needs to be done from the feature context above and follow the standa
 
 3. **Implement**: Run `/prizmkit-implement` with `artifact_dir=.prizmkit/specs/{{FEATURE_SLUG}}/` to execute the plan using TDD (write tests first, then implement).
 
-4. **Review**: Run `/prizmkit-code-review` with `artifact_dir=.prizmkit/specs/{{FEATURE_SLUG}}/`. The Main Agent self-reviews for up to ten rounds; semantic risk then limits independent Reviewers to `low=0`, `medium=1`, `high=2`, with Reviewer 2 only after an accepted high finding is repaired and reconverged. Reviewer 3 is forbidden. Require the last `## Final Result` verdict before continuing.
+4. **Review**: Run `/prizmkit-code-review` with `artifact_dir=.prizmkit/specs/{{FEATURE_SLUG}}/`. The Main Agent reviews the complete current change for up to ten rounds, directly repairs accepted findings, verifies repairs, and converges when accepted and unresolved findings are both zero. Do not spawn a code-review-specific subagent. Require the last `## Final Result` verdict before continuing.
 
 5. **Test**: Run `/prizmkit-test scope=this-change artifact_dir=.prizmkit/specs/{{FEATURE_SLUG}}/` to generate and verify tests only for this feature's changed scope after review. Do not use bugfix/refactor artifact directories for this gate.
 

package/bundled/dev-pipeline/templates/bootstrap-tier3.md

@@ -240,13 +240,13 @@ For each Verification Gate from the Task Contract, write one evidence line to Im
 
 Review the completed implementation before the scoped feature test gate runs. Use `artifact_dir=.prizmkit/specs/{{FEATURE_SLUG}}/` so the code-review skill can read spec, plan, context snapshot, implementation log, and current diff artifacts. Do not require or read a prior `/prizmkit-test` report here; test generation and test-fix looping happen in the next phase.
 
-Run `/prizmkit-code-review` with `artifact_dir=.prizmkit/specs/{{FEATURE_SLUG}}/`. The skill first runs a Main-Agent self-review loop for up to ten completed rounds, then semantic risk permits independent Reviewers at limits `low=0`, `medium=1`, `high=2`. High risk starts Reviewer 2 only after an accepted high finding was repaired, tested, and reconverged; Reviewer 3 is forbidden. The skill owns launch parameters, current-checkout preference, forced-worktree changed-content fallback, and append-only report progress.
+Run `/prizmkit-code-review` with `artifact_dir=.prizmkit/specs/{{FEATURE_SLUG}}/`. The skill runs a Main-Agent review loop for up to ten completed rounds, directly repairs accepted findings, verifies repairs, and converges when accepted and unresolved findings are both zero. Do not spawn a code-review-specific subagent. Missing required evidence, unsafe repair, failed verification, or round-ten non-convergence produces `NEEDS_FIXES`. The skill owns append-only report progress.
 
 **Gate Check — Final Review Result**:
 ```bash
 {{RUNTIME_HELPER_CMD}} text final-verdict .prizmkit/specs/{{FEATURE_SLUG}}/review-report.md
 ```
-Handle `REVIEW_PASS`, `REVIEW_NEEDS_FIXES`, and `REVIEW_BLOCKED` according to task policy. `REVIEW_INCOMPLETE` or `REVIEW_INVALID` means review did not complete: inspect progress once, write `failure-log.md`, and stop. Do not accept old or incidental verdict text or synthesize a passing result through Main-Agent self-review.
+Handle `REVIEW_PASS` and `REVIEW_NEEDS_FIXES` according to task policy. `REVIEW_INCOMPLETE` or `REVIEW_INVALID` means review did not complete: inspect progress once, write `failure-log.md`, and stop. Do not accept old or incidental verdict text or synthesize a passing result through Main-Agent self-review.
 
 **CP-3**: Review has a valid Final Result and the report preserves this execution's progress.
 
@@ -266,7 +266,7 @@ Gate requirements:
 - Record unrelated historical gaps under `Out of Scope Gaps`; do not generate tests for them and do not fail this feature because of them.
 - Accept only a current-run `.prizmkit/test/*/test-report.md` whose `Scope` mode is `this-change`, whose `Artifact Dir` matches `.prizmkit/specs/{{FEATURE_SLUG}}/`, whose `Verdict` is `PASS`, whose `Boundary Completion Gate` reports `Completion gate passed: yes` with zero `Boundary-missing` and zero `Happy-path-only`, and whose `Boundary Validation` result is `passed`.
 - Only after a valid `PASS`, write the report path to `.prizmkit/specs/{{FEATURE_SLUG}}/test-report-path.txt` and append a 3-5 bullet `## PrizmKit Test Gate` summary to `context-snapshot.md`.
-- If the report verdict is `NEEDS_FIXES`, fix in-scope implementation/test issues and rerun `/prizmkit-test`. If the report is `BLOCKED`, missing, stale, or for the wrong scope/artifact dir, write `failure-log.md` and stop for recovery.
+- If the report verdict is `NEEDS_FIXES`, fix in-scope implementation/test issues and rerun `/prizmkit-test`. If the report is missing, stale, invalid, or for the wrong scope/artifact dir, write `failure-log.md` and stop for recovery.
 
 {{IF_BROWSER_INTERACTION}}
 ### Phase 5.5: Browser Verification — MANDATORY

package/bundled/dev-pipeline/templates/bugfix-bootstrap-prompt.md

@@ -180,15 +180,15 @@ After implement completes, verify:
 If `FAST_PATH=true` (≤ 2 tasks, obvious root cause), skip this phase entirely and mark checkpoint step `prizmkit-code-review` as `skipped` before continuing.
 
 Run `/prizmkit-code-review` with `artifact_dir=.prizmkit/bugfix/{{BUG_ID}}/`:
-- The skill first runs up to ten Main-Agent self-review rounds, then semantic risk limits independent Reviewers to `low=0`, `medium=1`, `high=2`
-- High risk starts Reviewer 2 only after an accepted high finding was repaired, tested, and reconverged; Reviewer 3 is forbidden
-- The skill preserves its independent Reviewer prompt and appends progress to `review-report.md`
+- The skill runs up to ten Main-Agent review rounds, directly repairs accepted findings, and verifies each repair
+- Missing required evidence, unsafe repair, failed verification, or round-ten non-convergence produces `NEEDS_FIXES`
+- The skill appends Main-Agent review and repair progress to `review-report.md`
 
 **Gate Check — Final Review Result**:
 ```bash
 {{RUNTIME_HELPER_CMD}} text final-verdict .prizmkit/bugfix/{{BUG_ID}}/review-report.md
 ```
-Handle `REVIEW_PASS`, `REVIEW_NEEDS_FIXES`, and `REVIEW_BLOCKED` according to task policy. `REVIEW_INCOMPLETE` or `REVIEW_INVALID` means review did not complete: inspect progress once, write `failure-log.md`, and stop. Do not accept stale verdict text or replace required independent review with Main-Agent self-review.
+Handle `REVIEW_PASS` and `REVIEW_NEEDS_FIXES` according to task policy. `REVIEW_INCOMPLETE` or `REVIEW_INVALID` means review did not complete: inspect progress once, write `failure-log.md`, and stop. Do not accept stale verdict text or replace required independent review with Main-Agent self-review.
 
 {{IF_BROWSER_INTERACTION}}
 

package/bundled/dev-pipeline/templates/refactor-bootstrap-prompt.md

@@ -139,7 +139,7 @@ Apply the browser behavior preservation protocol where UI behavior can be affect
 
 ### Phase 4: Review — Code Review & Behavior Verification
 
-Run `/prizmkit-code-review` with `artifact_dir=.prizmkit/refactor/{{REFACTOR_ID}}/` directly from the main orchestrator. The skill first runs up to ten Main-Agent self-review rounds, then semantic risk limits independent Reviewers to `low=0`, `medium=1`, `high=2`. High risk starts Reviewer 2 only after an accepted high finding was repaired, tested, and reconverged; Reviewer 3 is forbidden.
+Run `/prizmkit-code-review` with `artifact_dir=.prizmkit/refactor/{{REFACTOR_ID}}/` directly from the main orchestrator. The skill runs up to ten Main-Agent review rounds, directly repairs accepted findings, and verifies each repair. Missing required evidence, unsafe repair, failed verification, or round-ten non-convergence produces `NEEDS_FIXES`.
 
 Review requirements:
 - verify refactor quality against spec.md and plan.md;
@@ -151,9 +151,9 @@ Review requirements:
 ```bash
 {{RUNTIME_HELPER_CMD}} text final-verdict .prizmkit/refactor/{{REFACTOR_ID}}/review-report.md
 ```
-Handle `REVIEW_PASS`, `REVIEW_NEEDS_FIXES`, and `REVIEW_BLOCKED` according to behavior-preservation policy. `REVIEW_INCOMPLETE` or `REVIEW_INVALID` means review did not complete: inspect progress once, write `failure-log.md`, and stop. Do not accept stale verdict text or replace required independent review with Main-Agent self-review.
+Handle `REVIEW_PASS` and `REVIEW_NEEDS_FIXES` according to behavior-preservation policy. `REVIEW_INCOMPLETE` or `REVIEW_INVALID` means review did not complete: inspect progress once, write `failure-log.md`, and stop. Do not accept stale verdict text or replace required independent review with Main-Agent self-review.
 
-**Checkpoint update**: set step `prizmkit-code-review` to `completed` only for an accepted valid Final Result. Do not complete it for blocked, incomplete, or invalid reports.
+**Checkpoint update**: set step `prizmkit-code-review` to `completed` only for an accepted valid Final Result. Do not complete it for incomplete or invalid reports.
 
 ---
 

package/bundled/dev-pipeline/templates/sections/bugfix-phase-review.md

@@ -12,9 +12,9 @@ If `FAST_PATH=true`, skip this phase intentionally:
 
 Then continue to the next checkpoint step.
 
-Otherwise, run `/prizmkit-code-review` with `artifact_dir=.prizmkit/bugfix/{{BUG_ID}}/`. The skill first runs a Main-Agent self-review loop for up to ten completed rounds, then semantic risk permits independent Reviewers at limits `low=0`, `medium=1`, `high=2`. High risk starts Reviewer 2 only after Reviewer 1 produced an accepted high finding that was repaired, tested, and reconverged. Reviewer 3 is forbidden.
+Otherwise, run `/prizmkit-code-review` with `artifact_dir=.prizmkit/bugfix/{{BUG_ID}}/`. The skill runs a Main-Agent review loop for up to ten completed rounds. The Main Agent reviews the complete current change, rejects unsupported candidate findings, directly repairs accepted findings, verifies repairs, and repeats until `accepted = 0` with no unresolved findings.
 
-The skill owns its unchanged independent Reviewer prompt, current-checkout preference, forced-worktree changed-content fallback, one bounded infrastructure retry per required Reviewer position, and append-only progress in `review-report.md`.
+The skill owns the full loop and append-only progress in `review-report.md`. Do not spawn a top-level or code-review-specific subagent. Missing required evidence, an unsafe repair, a failed verification, or failure to converge by round ten produces `NEEDS_FIXES`.
 
 **Gate Check — Final Review Result**:
 
@@ -24,7 +24,6 @@ The skill owns its unchanged independent Reviewer prompt, current-checkout prefe
 
 - `REVIEW_PASS` → proceed.
 - `REVIEW_NEEDS_FIXES` → log unresolved findings and proceed only when explicit task policy allows them.
-- `REVIEW_BLOCKED` → log infrastructure evidence and stop so recovery can retry.
-- `REVIEW_INCOMPLETE` or `REVIEW_INVALID` → inspect progress once, write `failure-log.md`, and stop. Do not accept old or incidental verdict text and do not replace required independent review with Main-Agent self-review.
+- `REVIEW_INCOMPLETE` or `REVIEW_INVALID` → inspect progress once, write `failure-log.md`, and stop. Do not accept old or incidental verdict text and do not synthesize a passing result outside the skill.
 
 **Checkpoint update**: Set step `prizmkit-code-review` to `completed` only for an accepted valid Final Result, or `skipped` for FAST_PATH.

package/bundled/dev-pipeline/templates/sections/phase-review-agent.md

@@ -2,9 +2,9 @@
 
 Review the completed implementation before the scoped feature test gate runs. Use `artifact_dir=.prizmkit/specs/{{FEATURE_SLUG}}/` so the code-review skill can read spec, plan, context snapshot, implementation log, and current diff artifacts. Do not require a prior `/prizmkit-test` report here; the next phase owns the formal test gate.
 
-Run `/prizmkit-code-review` with `artifact_dir=.prizmkit/specs/{{FEATURE_SLUG}}/`. The skill first runs a Main-Agent self-review loop for up to ten completed rounds, then semantic risk permits independent Reviewers at limits `low=0`, `medium=1`, `high=2`. High risk starts Reviewer 2 only after Reviewer 1 produced an accepted high finding that was repaired, tested, and reconverged. Reviewer 3 is forbidden.
+Run `/prizmkit-code-review` with `artifact_dir=.prizmkit/specs/{{FEATURE_SLUG}}/`. The skill runs a Main-Agent review loop for up to ten completed rounds. The Main Agent reviews the complete current change, rejects unsupported candidate findings, directly repairs accepted findings, verifies repairs, and repeats until `accepted = 0` with no unresolved findings.
 
-The skill owns Reviewer launch parameters, current-checkout preference, forced-worktree changed-content fallback, one bounded infrastructure retry per required Reviewer position, and append-only progress in `review-report.md`.
+The skill owns the full loop and append-only progress in `review-report.md`. Do not spawn a top-level or code-review-specific subagent. Missing required evidence, an unsafe repair, a failed verification, or failure to converge by round ten produces `NEEDS_FIXES`.
 
 **Gate Check — Final Review Result**:
 
@@ -16,8 +16,7 @@ Handle the marker:
 
 - `REVIEW_PASS` → proceed to next phase.
 - `REVIEW_NEEDS_FIXES` → log unresolved findings and follow the task failure policy; do not externally restart the review loop.
-- `REVIEW_BLOCKED` → log infrastructure evidence and stop so recovery can retry.
-- `REVIEW_INCOMPLETE` or `REVIEW_INVALID` → inspect the append-only progress once, write `failure-log.md`, and stop. Do not treat an old or incidental verdict as completion and do not synthesize a passing result through Main-Agent self-review.
+- `REVIEW_INCOMPLETE` or `REVIEW_INVALID` → inspect the append-only progress once, write `failure-log.md`, and stop. Do not treat an old or incidental verdict as completion and do not synthesize a passing result outside the skill.
 
 **CP-3**: Review has a valid Final Result and `review-report.md` preserves this execution's progress.
 

package/bundled/dev-pipeline/templates/sections/phase-review-full.md

@@ -2,9 +2,9 @@
 
 Review the completed implementation before the scoped feature test gate runs. Use `artifact_dir=.prizmkit/specs/{{FEATURE_SLUG}}/` so the code-review skill can read spec, plan, context snapshot, implementation log, and current diff artifacts. Do not require a prior `/prizmkit-test` report here; the next phase owns the formal test gate.
 
-Run `/prizmkit-code-review` with `artifact_dir=.prizmkit/specs/{{FEATURE_SLUG}}/`. The skill first runs a Main-Agent self-review loop for up to ten completed rounds, then semantic risk permits independent Reviewers at limits `low=0`, `medium=1`, `high=2`. High risk starts Reviewer 2 only after Reviewer 1 produced an accepted high finding that was repaired, tested, and reconverged. Reviewer 3 is forbidden.
+Run `/prizmkit-code-review` with `artifact_dir=.prizmkit/specs/{{FEATURE_SLUG}}/`. The skill runs a Main-Agent review loop for up to ten completed rounds. The Main Agent reviews the complete current change, rejects unsupported candidate findings, directly repairs accepted findings, verifies repairs, and repeats until `accepted = 0` with no unresolved findings.
 
-The skill owns Reviewer launch parameters, current-checkout preference, forced-worktree changed-content fallback, one bounded infrastructure retry per required Reviewer position, and append-only progress in `review-report.md`.
+The skill owns the full loop and append-only progress in `review-report.md`. Do not spawn a top-level or code-review-specific subagent. Missing required evidence, an unsafe repair, a failed verification, or failure to converge by round ten produces `NEEDS_FIXES`.
 
 **Gate Check — Final Review Result**:
 
@@ -16,8 +16,7 @@ Handle the marker:
 
 - `REVIEW_PASS` → proceed to next phase.
 - `REVIEW_NEEDS_FIXES` → log unresolved findings and follow the task failure policy; do not externally restart the review loop.
-- `REVIEW_BLOCKED` → log infrastructure evidence and stop so recovery can retry.
-- `REVIEW_INCOMPLETE` or `REVIEW_INVALID` → inspect the append-only progress once, write `failure-log.md`, and stop. Do not treat an old or incidental verdict as completion and do not synthesize a passing result through Main-Agent self-review.
+- `REVIEW_INCOMPLETE` or `REVIEW_INVALID` → inspect the append-only progress once, write `failure-log.md`, and stop. Do not treat an old or incidental verdict as completion and do not synthesize a passing result outside the skill.
 
 **CP-3**: Review has a valid Final Result and `review-report.md` preserves this execution's progress.
 

package/bundled/dev-pipeline/templates/sections/refactor-phase-review.md

@@ -1,8 +1,8 @@
 ### Phase 3: Review — Code Review & Behavior Verification
 
-Run `/prizmkit-code-review` directly with `artifact_dir=.prizmkit/refactor/{{REFACTOR_ID}}/`. The skill first runs a Main-Agent self-review loop for up to ten completed rounds, then semantic risk permits independent Reviewers at limits `low=0`, `medium=1`, `high=2`. High risk starts Reviewer 2 only after Reviewer 1 produced an accepted high finding that was repaired, tested, and reconverged. Reviewer 3 is forbidden.
+Run `/prizmkit-code-review` directly with `artifact_dir=.prizmkit/refactor/{{REFACTOR_ID}}/`. The skill runs a Main-Agent review loop for up to ten completed rounds. The Main Agent reviews the complete current change, rejects unsupported candidate findings, directly repairs accepted findings, verifies repairs, and repeats until `accepted = 0` with no unresolved findings.
 
-Do not spawn a top-level Reviewer subagent. The code-review skill owns its unchanged independent Reviewer prompt, current-checkout preference, forced-worktree changed-content fallback, one bounded infrastructure retry per required Reviewer position, and append-only report progress.
+Do not spawn a top-level or code-review-specific subagent. The code-review skill owns the complete loop and append-only report progress. Missing required evidence, an unsafe repair, a failed verification, or failure to converge by round ten produces `NEEDS_FIXES`.
 
 **Gate Check — Final Review Result**:
 
@@ -12,9 +12,8 @@ Do not spawn a top-level Reviewer subagent. The code-review skill owns its uncha
 
 - `REVIEW_PASS` → proceed.
 - `REVIEW_NEEDS_FIXES` → log unresolved findings and proceed only when they do not block behavior preservation under explicit task policy.
-- `REVIEW_BLOCKED` → log infrastructure evidence and stop so recovery can retry.
-- `REVIEW_INCOMPLETE` or `REVIEW_INVALID` → inspect progress once, write `failure-log.md`, and stop. Do not accept stale verdict text or replace required independent review with Main-Agent self-review.
+- `REVIEW_INCOMPLETE` or `REVIEW_INVALID` → inspect progress once, write `failure-log.md`, and stop. Do not accept stale verdict text or synthesize a passing result outside the skill.
 
 Verify the review explicitly checks behavior preservation, public API compatibility, imports, tests, and rollback assumptions.
 
-**Checkpoint update**: Set step `prizmkit-code-review` to `completed` only for an accepted valid Final Result. Do not complete the checkpoint for blocked, incomplete, or invalid reports.
+**Checkpoint update**: Set step `prizmkit-code-review` to `completed` only for an accepted valid Final Result. Do not complete the checkpoint for incomplete or invalid reports.

package/bundled/dev-pipeline/tests/test_generate_bootstrap_prompt.py

@@ -882,9 +882,11 @@ class TestDirectOrchestratorImplementationPrompts:
             assert FORBIDDEN_TEST_CMD_PLACEHOLDER not in rendered
             assert FORBIDDEN_THREE_STRIKE not in rendered
             assert FORBIDDEN_TWENTY_STEP not in rendered
-            assert "Main-Agent self-review loop" in rendered
-            assert "low=0" in rendered and "medium=1" in rendered and "high=2" in rendered
-            assert "Reviewer 3 is forbidden" in rendered
+            assert "Main-Agent review loop" in rendered
+            assert "up to ten completed rounds" in rendered
+            assert "code-review-specific subagent" in rendered
+            assert "low=0" not in rendered
+            assert "Reviewer 3" not in rendered
 
     def test_retained_tier3_fallback_template_uses_direct_orchestrator_implementation(self):
         rendered = Path("dev-pipeline/templates/bootstrap-tier3.md").read_text(encoding="utf-8")
@@ -1443,8 +1445,9 @@ class TestHeadlessPromptCleanupF033:
         refactor_prompt = _render_refactor_prompt(tmp_path / "refactor-mixed", platform="claude")
 
         assert "{{" + "REVIEWER_" + "SUBAGENT_PATH" + "}}" not in replacements
-        assert "Main-Agent self-review" in refactor_prompt
-        assert "Reviewer 3 is forbidden" in refactor_prompt
+        assert "Main-Agent review loop" in refactor_prompt
+        assert "code-review-specific subagent" in refactor_prompt
+        assert "Reviewer 3" not in refactor_prompt
 
     def test_bugfix_refactor_explicit_codex_override(self, tmp_path, monkeypatch):
         from generate_bugfix_prompt import build_replacements as bugfix_build_replacements
@@ -1465,8 +1468,9 @@ class TestHeadlessPromptCleanupF033:
         refactor_prompt = _render_refactor_prompt(tmp_path / "refactor-codex", platform="claude", explicit_platform="codex")
 
         assert "{{" + "REVIEWER_" + "SUBAGENT_PATH" + "}}" not in replacements
-        assert "Main-Agent self-review" in refactor_prompt
-        assert "Reviewer 3 is forbidden" in refactor_prompt
+        assert "Main-Agent review loop" in refactor_prompt
+        assert "code-review-specific subagent" in refactor_prompt
+        assert "Reviewer 3" not in refactor_prompt
 
 
 class TestFeatureBootstrapShellExtraction:

package/bundled/dev-pipeline/tests/test_runtime_helper.py

@@ -100,11 +100,17 @@ class TestArtifactAndTextHelpers:
         report = tmp_path / "review-report.md"
         report.write_text(
             "## Final Result\n- Verdict: PASS\n\n"
-            "## Final Result\n- Verdict: BLOCKED\n",
+            "## Final Result\n- Verdict: NEEDS_FIXES\n",
             encoding="utf-8",
         )
-        blocked = run_helper("text", "final-verdict", str(report))
-        assert first_line(blocked) == "REVIEW_BLOCKED"
+        needs_fixes = run_helper("text", "final-verdict", str(report))
+        assert needs_fixes.returncode == 0
+        assert first_line(needs_fixes) == "REVIEW_NEEDS_FIXES"
+
+        report.write_text("## Final Result\n- Verdict: BLOCKED\n", encoding="utf-8")
+        retired = run_helper("text", "final-verdict", str(report))
+        assert retired.returncode != 0
+        assert first_line(retired) == "REVIEW_INVALID"
 
         report.write_text("## Final Result\n- Verdict: UNKNOWN\n", encoding="utf-8")
         invalid = run_helper("text", "final-verdict", str(report))

package/bundled/dev-pipeline/tests/test_unified_cli.py

@@ -811,7 +811,7 @@ def test_session_preamble_writes_redacted_start_command_for_all_launch_profiles(
             assert str(prompt) not in "\n".join(lines[:6])
 
 
-def test_headless_review_guidance_uses_risk_routed_dual_layer_loop():
+def test_headless_review_guidance_uses_main_agent_review_loop():
     headless_guidance_files = [
         REPO_ROOT / "dev-pipeline" / "templates" / "sections" / "phase-review-full.md",
         REPO_ROOT / "dev-pipeline" / "templates" / "bootstrap-tier3.md",
@@ -827,32 +827,27 @@ def test_headless_review_guidance_uses_risk_routed_dual_layer_loop():
 
     for path in headless_guidance_files:
         text = path.read_text(encoding="utf-8")
-        assert "Main-Agent self-review" in text, path
-        assert "low=0" in text and "medium=1" in text and "high=2" in text, path
-        assert "Reviewer 3 is forbidden" in text, path
+        assert "Main-Agent review loop" in text, path
+        assert "up to ten completed rounds" in text, path
         assert "text final-verdict" in text, path
-        assert "workspace-equivalent" not in text, path
-        assert "workspace protocol" not in text, path
-        assert "current-workspace / active-checkout / no-worktree" not in text, path
-        assert ".claude/worktrees/..." not in text, path
-        assert ".prizmkit/state/worktrees/..." not in text, path
-        assert "USE_WORKTREE" not in text, path
+        assert "code-review-specific subagent" in text, path
+        assert "low=0" not in text and "Reviewer 3" not in text, path
+        assert "REVIEW_BLOCKED" not in text, path
+        assert "forced-worktree changed-content fallback" not in text, path
         assert "isolation:" not in text, path
 
     skill_text = skill_path.read_text(encoding="utf-8")
     assert "staged and unstaged tracked changes" in skill_text
     assert "untracked files" in skill_text
     assert "accepted = 0" in skill_text
-    assert "low: 0" in skill_text and "medium: 1" in skill_text and "high: 2" in skill_text
-    assert "accepted high-severity finding" in skill_text
-    assert "Do not launch Reviewer 3" in skill_text
+    assert "ten completed review rounds" in skill_text
     assert "## Final Result" in skill_text
-    assert "BLOCKED" in skill_text
-    assert "Do not substitute Main-Agent self-review" in skill_text
+    assert "PASS | NEEDS_FIXES" in skill_text
+    assert "BLOCKED" not in skill_text
+    assert "Reviewer 2" not in skill_text
     assert "reviewer-workspace-protocol.md" not in skill_text
     assert "reviewer-execution-protocol.md" not in skill_text
     assert "WORKSPACE_MISMATCH" not in skill_text
-    assert "REVIEW_INFRASTRUCTURE_BLOCKED" not in skill_text
 
 
 def test_progress_parser_adds_required_metadata(tmp_path):

package/bundled/skills/_metadata.json

@@ -1,5 +1,5 @@
 {
-  "version": "1.1.127",
+  "version": "1.1.128",
   "skills": {
     "prizmkit": {
       "description": "Full-lifecycle dev toolkit. Covers spec-driven development, Prizm context docs, code quality, debugging, deployment, and knowledge management.",
@@ -30,10 +30,10 @@
       "hasScripts": false
     },
     "prizmkit-code-review": {
-      "description": "Independent read-only review with Main-Agent finding adjudication, direct repairs, bounded convergence, and a minimal final report.",
+      "description": "Bounded Main-Agent code review with evidence-based finding adjudication, direct repairs, verification, and append-only progress reporting.",
       "tier": "1",
       "category": "prizmkit-skill",
-      "hasAssets": true,
+      "hasAssets": false,
       "hasScripts": true
     },
     "prizmkit-committer": {