aiwcli 0.12.8 → 0.13.1

package/dist/templates/cc-native/_cc-native/lib-ts/types.ts

@@ -273,7 +273,7 @@ export interface Reviewer {
 // JSON Schemas (moved to reviewers/schemas.ts)
 // ---------------------------------------------------------------------------
 // Re-export for backwards compatibility
-export { REVIEW_SCHEMA, ORCHESTRATOR_SCHEMA, REVIEW_PROMPT_PREFIX, AGENT_REVIEW_PROMPT_PREFIX } from "../plan-review/lib/reviewers/schemas.js";
+export { AGENT_REVIEW_PROMPT_PREFIX, ORCHESTRATOR_SCHEMA, REVIEW_PROMPT_PREFIX, REVIEW_SCHEMA } from "../plan-review/lib/reviewers/schemas.js";
 
 // ---------------------------------------------------------------------------
 // Display Defaults

package/dist/templates/cc-native/_cc-native/plan-review/CLAUDE.md

@@ -13,6 +13,7 @@ When a Claude Code agent exits plan mode (`ExitPlanMode`), the plan review hook
 ```
 plan-review/
 ├── CLAUDE.md            ← This file
+├── CODING-STANDARDS-CHECKLIST.md ← Standards injected during plan mode via plan_questions_early.ts
 ├── agents/
 │   ├── CLAUDE.md        ← Agent file format, frontmatter fields, selection rules
 │   ├── PLAN-ORCHESTRATOR.md   ← Orchestrator agent (complexity analysis)

package/dist/templates/cc-native/_cc-native/plan-review/CODING-STANDARDS-CHECKLIST.md

@@ -0,0 +1,75 @@
+# Coding Standards Checklist
+
+Standards that address the most common plan review failure modes. Reference this
+when planning code changes in established codebases.
+
+---
+
+## 1. Test-First Design Thinking
+
+Tests are an architectural constraint, not an afterthought. Design from the test
+perspective first.
+
+- **Interface-first:** Before describing implementation, ask: "Can I write the test
+  for this before the implementation exists?" If the answer is unclear, the interface
+  needs more thought.
+- **Structure tests before code:** Plans that describe "implement then test" consistently
+  fail review. Restructure: define what the tests assert, then describe the implementation
+  that satisfies them.
+- **Testability as architecture:** Design for dependency injection, interface seams, and
+  fakes. If a component can't be tested in isolation, the coupling is too tight.
+- **Test categories:** Consider which test types apply — unit (isolated logic), integration
+  (module boundaries), contract (API surfaces), and characterization (existing behavior
+  preservation during refactoring).
+- **Verification clarity:** Each planned change should have a corresponding verification
+  step that is binary-testable (pass/fail in one check, no subjective judgment).
+
+---
+
+## 2. File Structure & Codebase Convention Fit
+
+Don't pick a "plausible" location — pick the location that matches the project's
+established patterns.
+
+- **Discover before proposing:** Before suggesting new files or directories, verify where
+  similar things already live in this project. Use Glob/Grep to find existing patterns.
+- **Naming conventions:** Match existing module and file naming patterns. If the project
+  uses `kebab-case.ts`, don't introduce `camelCase.ts`. If hooks live in `hooks/`, don't
+  create a `hook-handlers/` directory.
+- **Co-location patterns:** Check if the project follows co-location (tests next to source,
+  types with implementation) or separation (dedicated `__tests__/`, `types/` directories).
+  Follow what exists.
+- **Import depth:** Verify that new files fit the existing import hierarchy. Adding a file
+  that requires imports to cross architectural boundaries (e.g., shared lib importing from
+  feature code) signals a structural problem.
+- **Existing system boundaries:** Check if the project has documented system boundaries
+  (CLAUDE.md, architecture docs). New files should respect these boundaries rather than
+  create cross-cutting dependencies.
+
+---
+
+## 3. Extensibility & Future-Proofing Analysis
+
+Balance: don't over-engineer (YAGNI), but don't create designs that actively resist
+extension.
+
+- **Adjacent features:** What features are most commonly built after this one? Does the
+  design accommodate those extensions without major restructuring?
+- **Extension points:** Where would future developers need to hook in? Are those seams
+  accessible, or does the design require forking/copying to extend?
+- **Configuration vs. code changes:** Will common customizations require code changes, or
+  can they be handled through configuration? Prefer the latter when the variation space
+  is predictable.
+- **Data model flexibility:** Are data structures designed to accommodate likely additions
+  (new fields, new types) without breaking existing consumers?
+- **Inversion of control:** Does the design allow callers to inject behavior, or does it
+  hardcode decisions that callers will need to override? Prefer interfaces and callbacks
+  over concrete implementations when variation is expected.
+
+---
+
+## Applicability
+
+These standards apply to production code in established codebases with existing conventions.
+For prototypes, scripts, spike explorations, or greenfield projects without established
+patterns, use judgment on which standards apply — not all will be relevant.

package/dist/templates/cc-native/_cc-native/plan-review/lib/agent-selection.ts

@@ -5,7 +5,6 @@
 
 import { logDebug, logInfo, logWarn } from "../../../_shared/lib-ts/base/logger.js";
 import { findExecutable } from "../../../_shared/lib-ts/base/subprocess-utils.js";
-
 import type {
   AgentConfig,
   ModelsConfig,
@@ -31,7 +30,7 @@ export function resolveMandatoryAgents(
     return new Set(configValue as string[]);
   }
   if (!configValue || typeof configValue !== "object") {
-    return new Set(["handoff-readiness", "clarity-auditor", "skeptic"]);
+    return new Set(["clarity-auditor", "handoff-readiness", "skeptic"]);
   }
   const cfg = configValue as Record<string, string[]>;
   const names = new Set(cfg.always ?? []);
@@ -64,7 +63,7 @@ export function assignModelsToAgents(
       if (!found) {
         logWarn(HOOK, `Provider '${name}' enabled but CLI '${cliName}' not found on PATH — skipping`);
       }
-      return !!found;
+      return Boolean(found);
     });
 
   if (enabledProviders.length === 0) {

package/dist/templates/cc-native/_cc-native/plan-review/lib/corroboration.ts

@@ -1,23 +1,35 @@
 /**
  * Corroboration-based verdict computation for plan review.
  *
- * Replaces the old per-verdict aggregation with proportional thresholding:
- * high-severity issues in a dimension only block when the total count
- * exceeds 2× the number of distinct agents contributing to that dimension.
+ * Uses agent-agreement thresholding: a dimension blocks only when a sufficient
+ * number of *distinct agents* independently flag it. This measures true
+ * corroboration (multiple independent reviewers converge) rather than issue
+ * density (one verbose agent floods a dimension).
  *
- * **Why proportional thresholding:**
- * The agent pool has dimensional imbalance (e.g., 10 completeness agents vs
- * 1 maintainability agent). A fixed "2+ agents agree = block" would mean
- * any 2 completeness agents always block. Proportional scaling (issues > 2×agents)
- * sets a fair bar regardless of how many agents focus on each dimension.
+ * **Algorithm:**
+ * For each dimension, compute: `effective_threshold = max(minAgreement, ceil(minRatio × totalAgents))`
+ * Block when `distinct_agents_in_dimension >= effective_threshold`.
+ *
+ * **Default config:** `minAgreement=2, minRatio=0.40`
+ * - At 6 agents: threshold=3 (50% must agree)
+ * - At 10 agents: threshold=4 (40% must agree)
+ * - At 20 agents: threshold=8 (40% must agree)
+ *
+ * **Why agent-agreement over issue-density:**
+ * The previous system (issues >= 2×agents_in_dimension) allowed a single agent
+ * to self-corroborate by raising 2+ issues, and made blocking harder as more
+ * agents covered a dimension (inverted scaling). Agent-agreement fixes both:
+ * a single agent can never self-corroborate, and more agents agreeing is a
+ * stronger signal, not a weaker one.
  *
  * **Convergence problem this solves:**
  * Agents with opposing philosophies (simplicity-guardian vs completeness-gaps)
  * produce contradictory high-severity issues. Because the old system treated
  * every agent's finding as independently authoritative, plans oscillated —
- * addressing one agent's feedback triggered the opposing agent.
+ * addressing one agent's feedback triggered the opposing agent. The minAgreement
+ * floor prevents any single agent's philosophy from blocking alone.
  *
- * **Revert path:** Change one line in cc-native-plan-review.ts back to
+ * **Revert path:** Change one line in review-pipeline.ts back to
  * `computeReviewDecision(allVerdicts)`. Old function kept in verdict.ts.
  */
 
@@ -30,22 +42,55 @@ import type {
   SoloFinding,
 } from "../../lib-ts/types.js";
 
+/** Configuration for corroboration thresholds */
+export interface CorroborationConfig {
+  /** Minimum distinct agents that must agree to trigger blocking (default: 2) */
+  minAgreement?: number;
+  /** Minimum fraction of total agent pool that must agree (default: 0.40) */
+  minRatio?: number;
+}
+
+const DEFAULT_MIN_AGREEMENT = 2;
+const DEFAULT_MIN_RATIO = 0.40;
+
+/**
+ * Compute the effective blocking threshold for a given agent pool size.
+ *
+ * Returns `max(minAgreement, ceil(minRatio × totalAgents))`.
+ * This ensures a fixed floor (no single-agent self-corroboration) while
+ * scaling proportionally at larger pool sizes.
+ */
+export function getEffectiveThreshold(
+  totalAgents: number,
+  config: CorroborationConfig = {},
+): number {
+  const minAgreement = config.minAgreement ?? DEFAULT_MIN_AGREEMENT;
+  const minRatio = config.minRatio ?? DEFAULT_MIN_RATIO;
+  return Math.max(minAgreement, Math.ceil(totalAgents * minRatio));
+}
+
 /**
  * Compute a corroboration-based review decision from all reviewer results.
  *
  * Algorithm:
  * 1. Collect all high-severity issues with a `dimension` field
  * 2. Group by dimension, tracking distinct agent names per group
- * 3. For each dimension: block if `issues.length > 2 × agentCount`
- * 4. Issues without `dimension` are unclassified (never block)
- * 5. Non-high issues are ignored (informational only)
+ * 3. Compute effective threshold: `max(minAgreement, ceil(minRatio × totalAgents))`
+ * 4. For each dimension: block if `distinct_agents >= effective_threshold`
+ * 5. Issues without `dimension` are unclassified (logged as warning, never block)
+ * 6. Non-high issues are ignored (informational only)
  *
  * @param allResults - Map of reviewer name → ReviewerResult (CLI + agent)
+ * @param config - Optional threshold configuration
  * @returns CorroborationResult with blocking groups, solo findings, and verdict
  */
 export function computeCorroboratedDecision(
   allResults: Record<string, ReviewerResult>,
+  config: CorroborationConfig = {},
 ): CorroborationResult {
+  const totalAgents = Object.keys(allResults).length;
+  const threshold = getEffectiveThreshold(totalAgents, config);
+
   // Accumulator: dimension → { issues, agentNames }
   const dimMap = new Map<
     IssueDimension,
@@ -66,7 +111,7 @@ export function computeCorroboratedDecision(
       // Only high-severity issues participate in corroboration
       if (issue.severity !== "high") continue;
 
-      // Issues without dimension are unclassified — cannot block
+      // Issues without dimension are unclassified — logged but cannot block
       if (!issue.dimension) {
         unclassified.push({ agent: agentName, issue });
         continue;
@@ -82,14 +127,22 @@ export function computeCorroboratedDecision(
     }
   }
 
+  // Warn about unclassified issues so they don't silently disappear
+  if (unclassified.length > 0) {
+    const agents = [...new Set(unclassified.map(u => u.agent))];
+    process.stderr.write(
+      `[corroboration] WARNING: ${unclassified.length} high-severity issue(s) from [${agents.join(", ")}] lack dimension classification and cannot participate in corroboration\n`,
+    );
+  }
+
   const blocking: CorroboratedGroup[] = [];
   const solo: SoloFinding[] = [];
 
   for (const [dimension, group] of dimMap) {
     const agentCount = group.agentNames.size;
-    const threshold = 2 * agentCount;
 
-    if (group.issues.length >= threshold) {
+    // Block when enough distinct agents independently flag this dimension
+    if (agentCount >= threshold) {
       blocking.push({
         dimension,
         issues: group.issues,

package/dist/templates/cc-native/_cc-native/plan-review/lib/graduation.ts

@@ -93,7 +93,7 @@ export function advanceIterationState(
     lastPlanHash: planHash,
     lastPlanPath: planPath,
     graduated: [...state.graduated],
-    passStreaks: { ...(state.passStreaks ?? {}) },
+    passStreaks: { ...state.passStreaks },
   };
 
   if (!shouldDeny) {

package/dist/templates/cc-native/_cc-native/plan-review/lib/orchestrator.ts

@@ -5,8 +5,8 @@
  */
 
 import { OrchestratorClaudeAgent } from "./reviewers/providers/orchestrator-claude-agent.js";
-import type { AgentConfig, OrchestratorConfig, OrchestratorResult } from "../../lib-ts/types.js";
 import { logInfo, logWarn } from "../../../_shared/lib-ts/base/logger.js";
+import type { AgentConfig, OrchestratorConfig, OrchestratorResult } from "../../lib-ts/types.js";
 
 // Re-export for backward compatibility (moved to reviewers/schemas.ts)
 export { buildOrchestratorSchema } from "./reviewers/schemas.js";

package/dist/templates/cc-native/_cc-native/plan-review/lib/output-builder.ts

@@ -4,18 +4,16 @@
  */
 
 import { logInfo } from "../../../_shared/lib-ts/base/logger.js";
-
+import {
+  buildInlineReviewSummary,
+  extractTopIssuesText,
+} from "../../artifacts/lib/format.js";
 import type {
   ReviewerResult,
   CombinedReviewResult,
   CorroborationResult,
   IterationState,
 } from "../../lib-ts/types.js";
-import {
-  buildInlineReviewSummary,
-  extractTopIssuesText,
-} from "../../artifacts/lib/format.js";
-import { extractTopIssuesForTracker } from "./graduation.js";
 
 const HOOK = "output-builder";
 
@@ -28,7 +26,7 @@ const RESUBMIT_INSTRUCTION = "IMPORTANT: After revising the plan file, you MUST
 
 /**
  * Truncate per-agent issues to top N by severity.
- * @mutates agentResults[name].data.issues in place.
+ * Mutates agentResults[name].data.issues in place.
  */
 export function truncateAgentIssues(
   agentResults: Record<string, ReviewerResult>,
@@ -52,7 +50,7 @@ export function truncateAgentIssues(
 
 /**
  * Override verdict to "fail" for agents exceeding high-issue threshold.
- * @mutates agentResults[name].verdict in place.
+ * Mutates agentResults[name].verdict in place.
  */
 export function overrideVerdictsByThreshold(
   agentResults: Record<string, ReviewerResult>,
@@ -94,18 +92,12 @@ export function buildReviewOutput(params: ReviewOutputParams): ReviewOutput {
   const { combinedResult, corroborationResult, iterationState } = params;
 
   const shouldDeny = corroborationResult.blocking.length > 0;
-  const reviewScore = shouldDeny ? 1.0 : 0.0;
+  const reviewScore = shouldDeny ? 1 : 0;
   const overall = corroborationResult.verdict;
 
-  // Build inline summary
-  const inlineSummary = buildInlineReviewSummary(combinedResult, 5, 800, corroborationResult);
+  // Build inline summary (no individual issues — those go in blockReason)
+  const inlineSummary = buildInlineReviewSummary(combinedResult, 0, 800, corroborationResult);
   const contextParts = [inlineSummary];
-
-  // Top issues
-  const topIssuesList = extractTopIssuesForTracker(combinedResult, 5);
-  if (topIssuesList.length > 0) {
-    contextParts.push(`\nTop high-severity issues:\n${topIssuesList.map(i => `- ${i}`).join("\n")}`);
-  }
   contextParts.push(`\nFull review: \`${params.reviewFile}\`\n`);
 
   const contextText = contextParts.join("");
@@ -113,11 +105,10 @@ export function buildReviewOutput(params: ReviewOutputParams): ReviewOutput {
 
   let blockReason: string;
   if (shouldDeny) {
-    const topIssuesText = extractTopIssuesText(combinedResult, 3, "high");
+    const topIssuesText = extractTopIssuesText(combinedResult, 15, "high");
     blockReason = `Plan review FAILED${iterInfo}. ` +
-      `Critical issues: ${topIssuesText}. ` +
-      `IMPORTANT: Read \`${params.highIssuesPath}\` for ALL high-severity issues — ` +
-      `this file contains only the most critical findings, no noise. ` +
+      `High-severity issues:\n${topIssuesText}\n\n` +
+      `Full details: \`${params.highIssuesPath}\`\n` +
       `${REVIEWER_CAVEAT} ` +
       `Revise the plan to address these issues, then call ExitPlanMode again. ` +
       RESUBMIT_INSTRUCTION;

package/dist/templates/cc-native/_cc-native/plan-review/lib/plan-questions.ts

@@ -6,11 +6,12 @@
 
 import * as path from "node:path";
 
-import { aggregateAgents } from "../../lib-ts/aggregate-agents.js";
 import { runAgentReview } from "./reviewers/index.js";
 import { QUESTIONS_SCHEMA } from "./reviewers/schemas.js";
-import type { AgentConfig, ModelsConfig } from "../../lib-ts/types.js";
 import { logInfo, logWarn, logError } from "../../../_shared/lib-ts/base/logger.js";
+import { findExecutable } from "../../../_shared/lib-ts/base/subprocess-utils.js";
+import { aggregateAgents } from "../../lib-ts/aggregate-agents.js";
+import type { AgentConfig } from "../../lib-ts/types.js";
 
 // ---------------------------------------------------------------------------
 // Types
@@ -26,8 +27,6 @@ export interface PlanQuestionsResult {
 // Provider assignment (local copy — avoids circular import from hook)
 // ---------------------------------------------------------------------------
 
-import { findExecutable } from "../../../_shared/lib-ts/base/subprocess-utils.js";
-
 function assignProvider(agent: AgentConfig): AgentConfig {
   // Default to claude provider with the agent's configured model
   const found = findExecutable("claude");

package/dist/templates/cc-native/_cc-native/plan-review/lib/review-pipeline.ts

@@ -7,37 +7,26 @@
 import * as fs from "node:fs";
 import * as path from "node:path";
 
+import { resolveMandatoryAgents, assignModelsToAgents, selectAgents } from "./agent-selection.js";
+import { computeCorroboratedDecision } from "./corroboration.js";
+import { computePassEligible, extractTopIssuesForTracker, advanceIterationState } from "./graduation.js";
+import { runOrchestrator } from "./orchestrator.js";
+import { truncateAgentIssues, overrideVerdictsByThreshold, buildReviewOutput } from "./output-builder.js";
+import { runPlanQuestions } from "./plan-questions.js";
+import { runAgentReview } from "./reviewers/index.js";
+import { getContextReviewsDir, getContextDir, getReviewFolderPath } from "../../../_shared/lib-ts/base/constants.js";
+import { logDiagnostic } from "../../../_shared/lib-ts/base/hook-utils.js";
 import {
   logDebug,
   logInfo,
   logWarn,
   logError,
 } from "../../../_shared/lib-ts/base/logger.js";
-import { logDiagnostic } from "../../../_shared/lib-ts/base/hook-utils.js";
 import { eprint } from "../../../_shared/lib-ts/base/utils.js";
-import { getContextReviewsDir, getContextDir, getReviewFolderPath } from "../../../_shared/lib-ts/base/constants.js";
 import { getContextBySessionId, getAllContexts } from "../../../_shared/lib-ts/context/context-store.js";
-
-import type {
-  AgentConfig,
-  OrchestratorConfig,
-  ReviewerResult,
-  CombinedReviewResult,
-  OrchestratorResult,
-  IterationState,
-  PipelineInput,
-  PipelineResult,
-} from "../../lib-ts/types.js";
-import { REVIEW_SCHEMA } from "../../lib-ts/types.js";
 import type { ContextState } from "../../../_shared/lib-ts/types.js";
-
-import { discoverPlan } from "../../lib-ts/plan-discovery.js";
-import { loadSettings, loadModelsConfig, loadAgentLibrary, DEFAULT_ORCHESTRATOR } from "../../lib-ts/settings.js";
-import { resolveMandatoryAgents, assignModelsToAgents, selectAgents } from "./agent-selection.js";
-import { computePassEligible, extractTopIssuesForTracker, advanceIterationState } from "./graduation.js";
-import { truncateAgentIssues, overrideVerdictsByThreshold, buildReviewOutput } from "./output-builder.js";
-import { DEFAULT_REVIEW_ITERATIONS, loadIterationState, saveIterationState } from "../../lib-ts/state.js";
-
+import { writeCombinedArtifacts, buildCorroborationReport, buildHighIssuesDocument, writeReviewTracker } from "../../artifacts/lib/index.js";
+import type { ReviewTrackerEntry } from "../../artifacts/lib/index.js";
 import {
   isPlanAlreadyReviewed,
   wasPlanPreviouslyDenied,
@@ -47,14 +36,21 @@ import {
   markQuestionsAsked,
   resetPlanQuestionsAsked,
 } from "../../lib-ts/cc-native-state.js";
-
-import { computeCorroboratedDecision } from "./corroboration.js";
-import { runOrchestrator } from "./orchestrator.js";
 import { debugLog } from "../../lib-ts/debug.js";
-import { writeCombinedArtifacts, buildCorroborationReport, buildHighIssuesDocument, writeReviewTracker } from "../../artifacts/lib/index.js";
-import type { ReviewTrackerEntry } from "../../artifacts/lib/index.js";
-import { runAgentReview } from "./reviewers/index.js";
-import { runPlanQuestions } from "./plan-questions.js";
+import { discoverPlan } from "../../lib-ts/plan-discovery.js";
+import { loadSettings, loadModelsConfig, loadAgentLibrary, DEFAULT_ORCHESTRATOR } from "../../lib-ts/settings.js";
+import { DEFAULT_REVIEW_ITERATIONS, loadIterationState, saveIterationState } from "../../lib-ts/state.js";
+import type {
+  AgentConfig,
+  OrchestratorConfig,
+  ReviewerResult,
+  CombinedReviewResult,
+  OrchestratorResult,
+  IterationState,
+  PipelineInput,
+  PipelineResult,
+} from "../../lib-ts/types.js";
+import { REVIEW_SCHEMA } from "../../lib-ts/types.js";
 
 const HOOK = "review-pipeline";
 
@@ -143,7 +139,7 @@ export async function runReviewPipeline(input: PipelineInput): Promise<PipelineR
       current: 1, max: 1, complexity: "medium",
       history: [], graduated: [], passStreaks: {},
       lastPlanHash: "", lastPlanPath: "",
-      sessionId: sessionId,
+      sessionId,
     };
     saveIterationState(reviewsDir, iterationState);
   }
@@ -156,7 +152,7 @@ export async function runReviewPipeline(input: PipelineInput): Promise<PipelineR
       current: 1, max: 1, complexity: "medium",
       history: [], graduated: [], passStreaks: {},
       lastPlanHash: "", lastPlanPath: "",
-      sessionId: sessionId,
+      sessionId,
     };
     saveIterationState(reviewsDir, iterationState);
     resetPlanQuestionsAsked(sessionId, base);
@@ -213,10 +209,10 @@ export async function runReviewPipeline(input: PipelineInput): Promise<PipelineR
         contextText: "[Plan Review] Plan content unchanged since last review which found issues.",
         blockReason: "Plan unchanged since denial. Modify the plan to address review findings, then attempt ExitPlanMode again.",
       };
-    } else {
+    } 
       const verdict = lastReview?.iteration?.latest_verdict || "pass";
       return { action: "skip", reason: `Plan already reviewed (verdict: ${verdict}). Skipping re-review.` };
-    }
+    
   }
 
   // 7. Iteration bounds check
@@ -316,7 +312,7 @@ export async function runReviewPipeline(input: PipelineInput): Promise<PipelineR
     // Update iteration state with complexity/max
     const reviewIterations: Record<string, number> = {
       ...DEFAULT_REVIEW_ITERATIONS,
-      ...(agentSettings.reviewIterations ?? {}),
+      ...agentSettings.reviewIterations,
     };
     iterationState.complexity = detectedComplexity;
     iterationState.max = reviewIterations[detectedComplexity] ?? iterationState.max;
@@ -401,8 +397,8 @@ export async function runReviewPipeline(input: PipelineInput): Promise<PipelineR
   };
 
   const displaySettings = {
-    ...(planSettings.display ?? {}),
-    ...(agentSettings.display ?? {}),
+    ...planSettings.display,
+    ...agentSettings.display,
   };
   const combinedSettings = { display: displaySettings };
 
@@ -427,8 +423,8 @@ export async function runReviewPipeline(input: PipelineInput): Promise<PipelineR
   try {
     fs.writeFileSync(path.join(reviewFolder, "plan.md"), plan, "utf-8");
     logDebug(HOOK, `Saved plan snapshot: ${path.join(reviewFolder, "plan.md")}`);
-  } catch (e) {
-    logWarn(HOOK, `Failed to save plan snapshot: ${e}`);
+  } catch (error) {
+    logWarn(HOOK, `Failed to save plan snapshot: ${error}`);
   }
 
   // Build high-issues document
@@ -438,7 +434,7 @@ export async function runReviewPipeline(input: PipelineInput): Promise<PipelineR
 
   // 15. Build output
   const shouldDeny = corroborationResult.blocking.length > 0;
-  const reviewScore = shouldDeny ? 1.0 : 0.0;
+  const reviewScore = shouldDeny ? 1 : 0;
   const denyReason = shouldDeny ? "corroborated_issues" : "no_corroboration";
 
   logInfo(HOOK, `REVIEW_DECISION: verdict=${combinedResult.overall_verdict}, deny=${shouldDeny}, reason=${denyReason}, score=${reviewScore.toFixed(2)}`);

package/dist/templates/cc-native/_cc-native/plan-review/lib/reviewers/agent.ts

@@ -4,13 +4,13 @@
  * See cc-native-plan-review-spec.md §4.10
  */
 
-import { logWarn } from "../../../../_shared/lib-ts/base/logger.js";
-import type { AgentConfig, ReviewerResult, ReviewOptions } from "../../../lib-ts/types.js";
 import { ClaudeAgent } from "./providers/claude-agent.js";
 import { CodexAgent } from "./providers/codex-agent.js";
 import { GeminiAgent } from "./providers/gemini-agent.js";
 import type { Reviewer } from "./types.js";
 import { makeResult } from "./types.js";
+import { logWarn } from "../../../../_shared/lib-ts/base/logger.js";
+import type { AgentConfig, ReviewerResult, ReviewOptions } from "../../../lib-ts/types.js";
 
 /**
  * Agent reviewer — runs a CLI instance with a custom persona.
@@ -59,7 +59,6 @@ export async function runAgentReview(
         reviewer = new GeminiAgent(agent, schema, timeout, contextPath, sessionName);
         break;
       }
-      case "claude":
       default: {
         reviewer = new ClaudeAgent(agent, schema, timeout, contextPath, sessionName);
         break;

package/dist/templates/cc-native/_cc-native/plan-review/lib/reviewers/providers/codex-agent.ts

@@ -17,7 +17,7 @@ import { AGENT_REVIEW_PROMPT_PREFIX } from "../schemas.js";
 import { makeResult } from "../types.js";
 
 /** Temp directory for Codex schema/output files */
-const tmpDir: string | null = null;
+const _tmpDir: string | null = null;
 
 /**
  * Codex CLI-based agent reviewer.

package/oclif.manifest.json

@@ -416,5 +416,5 @@
       ]
     }
   },
-  "version": "0.12.8"
+  "version": "0.13.1"
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "aiwcli",
   "description": "AI Workflow CLI - Command-line interface for AI-powered workflows",
-  "version": "0.12.8",
+  "version": "0.13.1",
   "author": "jofu-tofu",
   "bin": {
     "aiw": "bin/run.js"
@@ -10,11 +10,13 @@
     "url": "https://github.com/jofu-tofu/AI-Workflow-CLI/issues"
   },
   "dependencies": {
-    "@inquirer/prompts": "^8.2.0",
+    "@inquirer/checkbox": "^5.0.0",
+    "@inquirer/confirm": "^6.0.0",
+    "@inquirer/input": "^5.0.0",
+    "@inquirer/select": "^5.0.0",
     "@oclif/core": "^4",
     "@oclif/plugin-autocomplete": "^3.2.39",
     "@oclif/plugin-help": "^6",
-    "@oclif/plugin-plugins": "^5",
     "chalk": "^5.6.2",
     "clipboardy": "^5.0.2",
     "ora": "^9.0.0"
@@ -69,8 +71,7 @@
     "commands": "./dist/commands",
     "plugins": [
       "@oclif/plugin-autocomplete",
-      "@oclif/plugin-help",
-      "@oclif/plugin-plugins"
+      "@oclif/plugin-help"
     ],
     "repositoryPrefix": "https://github.com/jofu-tofu/AI-Workflow-CLI/blob/main/<%- commandPath %>",
     "topicSeparator": " "