aiwcli 0.12.3 → 0.12.7

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

@@ -1,70 +1,70 @@
-/**
- * Plan orchestrator — analyzes complexity and selects reviewer agents.
- * Uses OrchestratorClaudeAgent (BaseCliAgent framework) for subprocess execution.
- * See cc-native-plan-review-spec.md §4.8
- */
-
-import { OrchestratorClaudeAgent } from "./reviewers/providers/orchestrator-claude-agent.js";
-import type { AgentConfig, OrchestratorConfig, OrchestratorResult } from "./types.js";
-import { logInfo, logWarn } from "../../_shared/lib-ts/base/logger.js";
-
-// Re-export for backward compatibility (moved to reviewers/schemas.ts)
-export { buildOrchestratorSchema } from "./reviewers/schemas.js";
-
-// ---------------------------------------------------------------------------
-// Orchestrator
-// ---------------------------------------------------------------------------
-
-/**
- * Run the orchestrator agent to analyze plan complexity and select reviewers.
- * Never throws — returns fallback OrchestratorResult on failure.
- */
-export async function runOrchestrator(
-  plan: string,
-  agentLibrary: AgentConfig[],
-  config: OrchestratorConfig,
-  settings: Record<string, unknown>,
-  mandatoryNames?: Set<string>,
-): Promise<OrchestratorResult> {
-  logInfo("orchestrator", "Starting plan analysis...");
-
-  const mandatory = mandatoryNames ?? new Set<string>();
-
-  // Create a synthetic AgentConfig for the orchestrator
-  const orchestratorAgent: AgentConfig = {
-    name: "orchestrator",
-    model: config.model,
-    provider: "claude",
-    focus: "plan analysis and agent selection",
-    categories: [],
-    description: "Plan orchestrator",
-    system_prompt: "",
-  };
-
-  try {
-    const agent = new OrchestratorClaudeAgent(
-      orchestratorAgent,
-      agentLibrary,
-      mandatory,
-      settings,
-      config.timeout,
-    );
-
-    const result = await agent.review(plan);
-
-    logInfo("orchestrator", `Result: complexity=${result.complexity}, category=${result.category}, agents=${JSON.stringify(result.selected_agents)}`);
-
-    return result;
-  } catch (error) {
-    logWarn("orchestrator", `Unexpected error: ${error}`);
-    const nonMandatory = agentLibrary.filter((a) => !mandatory.has(a.name));
-    const fallbackCount = ((settings.agentSelection as Record<string, unknown>)?.fallbackCount as number) ?? 2;
-    return {
-      complexity: "medium",
-      category: "code",
-      selected_agents: nonMandatory.slice(0, fallbackCount).map((a) => a.name),
-      reasoning: `Orchestrator failed: ${error}`,
-      error: String(error),
-    };
-  }
-}
+/**
+ * Plan orchestrator — analyzes complexity and selects reviewer agents.
+ * Uses OrchestratorClaudeAgent (BaseCliAgent framework) for subprocess execution.
+ * See cc-native-plan-review-spec.md §4.8
+ */
+
+import { OrchestratorClaudeAgent } from "./reviewers/providers/orchestrator-claude-agent.js";
+import type { AgentConfig, OrchestratorConfig, OrchestratorResult } from "./types.js";
+import { logInfo, logWarn } from "../../_shared/lib-ts/base/logger.js";
+
+// Re-export for backward compatibility (moved to reviewers/schemas.ts)
+export { buildOrchestratorSchema } from "./reviewers/schemas.js";
+
+// ---------------------------------------------------------------------------
+// Orchestrator
+// ---------------------------------------------------------------------------
+
+/**
+ * Run the orchestrator agent to analyze plan complexity and select reviewers.
+ * Never throws — returns fallback OrchestratorResult on failure.
+ */
+export async function runOrchestrator(
+  plan: string,
+  agentLibrary: AgentConfig[],
+  config: OrchestratorConfig,
+  settings: Record<string, unknown>,
+  mandatoryNames?: Set<string>,
+): Promise<OrchestratorResult> {
+  logInfo("orchestrator", "Starting plan analysis...");
+
+  const mandatory = mandatoryNames ?? new Set<string>();
+
+  // Create a synthetic AgentConfig for the orchestrator
+  const orchestratorAgent: AgentConfig = {
+    name: "orchestrator",
+    model: config.model,
+    provider: "claude",
+    focus: "plan analysis and agent selection",
+    categories: [],
+    description: "Plan orchestrator",
+    system_prompt: "",
+  };
+
+  try {
+    const agent = new OrchestratorClaudeAgent(
+      orchestratorAgent,
+      agentLibrary,
+      mandatory,
+      settings,
+      config.timeout,
+    );
+
+    const result = await agent.review(plan);
+
+    logInfo("orchestrator", `Result: complexity=${result.complexity}, category=${result.category}, agents=${JSON.stringify(result.selected_agents)}`);
+
+    return result;
+  } catch (error) {
+    logWarn("orchestrator", `Unexpected error: ${error}`);
+    const nonMandatory = agentLibrary.filter((a) => !mandatory.has(a.name));
+    const fallbackCount = ((settings.agentSelection as Record<string, unknown>)?.fallbackCount as number) ?? 2;
+    return {
+      complexity: "medium",
+      category: "code",
+      selected_agents: nonMandatory.slice(0, fallbackCount).map((a) => a.name),
+      reasoning: `Orchestrator failed: ${error}`,
+      error: String(error),
+    };
+  }
+}

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

@@ -1,130 +1,130 @@
-/**
- * Output construction: issue truncation, verdict override, message building.
- * Extracted from cc-native-plan-review.ts.
- */
-
-import { logInfo } from "../../_shared/lib-ts/base/logger.js";
-
-import type {
-  ReviewerResult,
-  CombinedReviewResult,
-  CorroborationResult,
-  IterationState,
-} from "./types.js";
-import {
-  buildInlineReviewSummary,
-  extractTopIssuesText,
-} from "./artifacts/format.js";
-import { extractTopIssuesForTracker } from "./graduation.js";
-
-const HOOK = "output-builder";
-
-const REVIEWER_CAVEAT = "Reviewers have limited context compared to your full session — use your judgment to adopt valid points and dismiss genuine false positives. However, treat false positives as a clarity signal: if a reviewer misunderstood your plan, an agent executing it will likely hit the same confusion. Revise those sections to be unambiguous so no future reader — human or AI — makes the same mistake.";
-const RESUBMIT_INSTRUCTION = "IMPORTANT: After revising the plan file, you MUST call ExitPlanMode again to trigger re-review. Do not end your turn or ask the user without calling ExitPlanMode.";
-
-// ---------------------------------------------------------------------------
-// Issue Truncation
-// ---------------------------------------------------------------------------
-
-/**
- * Truncate per-agent issues to top N by severity.
- * @mutates agentResults[name].data.issues in place.
- */
-export function truncateAgentIssues(
-  agentResults: Record<string, ReviewerResult>,
-  maxPerAgent: number,
-): void {
-  for (const r of Object.values(agentResults)) {
-    if (!Array.isArray(r.data?.issues)) continue;
-    const issues = r.data.issues as Array<{ severity?: string }>;
-    if (issues.length <= maxPerAgent) continue;
-    const severityOrder: Record<string, number> = { high: 0, medium: 1, low: 2 };
-    issues.sort((a, b) => (severityOrder[a.severity ?? "low"] ?? 2) - (severityOrder[b.severity ?? "low"] ?? 2));
-    const originalCount = issues.length;
-    r.data.issues = issues.slice(0, maxPerAgent);
-    logInfo(HOOK, `${r.name}: truncated issues ${originalCount} → ${maxPerAgent}`);
-  }
-}
-
-// ---------------------------------------------------------------------------
-// Verdict Override
-// ---------------------------------------------------------------------------
-
-/**
- * Override verdict to "fail" for agents exceeding high-issue threshold.
- * @mutates agentResults[name].verdict in place.
- */
-export function overrideVerdictsByThreshold(
-  agentResults: Record<string, ReviewerResult>,
-  threshold: number,
-): void {
-  for (const r of Object.values(agentResults)) {
-    if (!r.verdict || r.verdict === "skip" || r.verdict === "error") continue;
-    const issues = Array.isArray(r.data?.issues) ? r.data.issues as Array<{ severity?: string }> : [];
-    const agentHigh = issues.filter(i => i.severity === "high").length;
-    if (agentHigh >= threshold) {
-      logInfo(HOOK, `${r.name}: verdict overridden to 'fail' (${agentHigh} high issues >= ${threshold})`);
-      r.verdict = "fail";
-    }
-  }
-}
-
-// ---------------------------------------------------------------------------
-// Output Building
-// ---------------------------------------------------------------------------
-
-export interface ReviewOutputParams {
-  combinedResult: CombinedReviewResult;
-  corroborationResult: CorroborationResult;
-  iterationState: IterationState;
-  reviewFile: string;
-  highIssuesPath: string;
-}
-
-export interface ReviewOutput {
-  contextText: string;
-  blockReason: string;
-  shouldDeny: boolean;
-}
-
-/**
- * Build the final review output: context text and block reason.
- */
-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 overall = corroborationResult.verdict;
-
-  // Build inline summary
-  const inlineSummary = buildInlineReviewSummary(combinedResult, 5, 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("");
-  const iterInfo = ` (iteration ${iterationState.current}/${iterationState.max}, score=${reviewScore.toFixed(2)})`;
-
-  let blockReason: string;
-  if (shouldDeny) {
-    const topIssuesText = extractTopIssuesText(combinedResult, 3, "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. ` +
-      `${REVIEWER_CAVEAT} ` +
-      `Revise the plan to address these issues, then call ExitPlanMode again. ` +
-      RESUBMIT_INSTRUCTION;
-  } else {
-    blockReason = `Plan review ${overall.toUpperCase()}${iterInfo}. Review complete. ${REVIEWER_CAVEAT}`;
-  }
-
-  return { contextText, blockReason, shouldDeny };
-}
-
+/**
+ * Output construction: issue truncation, verdict override, message building.
+ * Extracted from cc-native-plan-review.ts.
+ */
+
+import { logInfo } from "../../_shared/lib-ts/base/logger.js";
+
+import type {
+  ReviewerResult,
+  CombinedReviewResult,
+  CorroborationResult,
+  IterationState,
+} from "./types.js";
+import {
+  buildInlineReviewSummary,
+  extractTopIssuesText,
+} from "./artifacts/format.js";
+import { extractTopIssuesForTracker } from "./graduation.js";
+
+const HOOK = "output-builder";
+
+const REVIEWER_CAVEAT = "Reviewers have limited context compared to your full session — use your judgment to adopt valid points and dismiss genuine false positives. However, treat false positives as a clarity signal: if a reviewer misunderstood your plan, an agent executing it will likely hit the same confusion. Revise those sections to be unambiguous so no future reader — human or AI — makes the same mistake.";
+const RESUBMIT_INSTRUCTION = "IMPORTANT: After revising the plan file, you MUST call ExitPlanMode again to trigger re-review. Do not end your turn or ask the user without calling ExitPlanMode.";
+
+// ---------------------------------------------------------------------------
+// Issue Truncation
+// ---------------------------------------------------------------------------
+
+/**
+ * Truncate per-agent issues to top N by severity.
+ * @mutates agentResults[name].data.issues in place.
+ */
+export function truncateAgentIssues(
+  agentResults: Record<string, ReviewerResult>,
+  maxPerAgent: number,
+): void {
+  for (const r of Object.values(agentResults)) {
+    if (!Array.isArray(r.data?.issues)) continue;
+    const issues = r.data.issues as Array<{ severity?: string }>;
+    if (issues.length <= maxPerAgent) continue;
+    const severityOrder: Record<string, number> = { high: 0, medium: 1, low: 2 };
+    issues.sort((a, b) => (severityOrder[a.severity ?? "low"] ?? 2) - (severityOrder[b.severity ?? "low"] ?? 2));
+    const originalCount = issues.length;
+    r.data.issues = issues.slice(0, maxPerAgent);
+    logInfo(HOOK, `${r.name}: truncated issues ${originalCount} → ${maxPerAgent}`);
+  }
+}
+
+// ---------------------------------------------------------------------------
+// Verdict Override
+// ---------------------------------------------------------------------------
+
+/**
+ * Override verdict to "fail" for agents exceeding high-issue threshold.
+ * @mutates agentResults[name].verdict in place.
+ */
+export function overrideVerdictsByThreshold(
+  agentResults: Record<string, ReviewerResult>,
+  threshold: number,
+): void {
+  for (const r of Object.values(agentResults)) {
+    if (!r.verdict || r.verdict === "skip" || r.verdict === "error") continue;
+    const issues = Array.isArray(r.data?.issues) ? r.data.issues as Array<{ severity?: string }> : [];
+    const agentHigh = issues.filter(i => i.severity === "high").length;
+    if (agentHigh >= threshold) {
+      logInfo(HOOK, `${r.name}: verdict overridden to 'fail' (${agentHigh} high issues >= ${threshold})`);
+      r.verdict = "fail";
+    }
+  }
+}
+
+// ---------------------------------------------------------------------------
+// Output Building
+// ---------------------------------------------------------------------------
+
+export interface ReviewOutputParams {
+  combinedResult: CombinedReviewResult;
+  corroborationResult: CorroborationResult;
+  iterationState: IterationState;
+  reviewFile: string;
+  highIssuesPath: string;
+}
+
+export interface ReviewOutput {
+  contextText: string;
+  blockReason: string;
+  shouldDeny: boolean;
+}
+
+/**
+ * Build the final review output: context text and block reason.
+ */
+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 overall = corroborationResult.verdict;
+
+  // Build inline summary
+  const inlineSummary = buildInlineReviewSummary(combinedResult, 5, 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("");
+  const iterInfo = ` (iteration ${iterationState.current}/${iterationState.max}, score=${reviewScore.toFixed(2)})`;
+
+  let blockReason: string;
+  if (shouldDeny) {
+    const topIssuesText = extractTopIssuesText(combinedResult, 3, "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. ` +
+      `${REVIEWER_CAVEAT} ` +
+      `Revise the plan to address these issues, then call ExitPlanMode again. ` +
+      RESUBMIT_INSTRUCTION;
+  } else {
+    blockReason = `Plan review ${overall.toUpperCase()}${iterInfo}. Review complete. ${REVIEWER_CAVEAT}`;
+  }
+
+  return { contextText, blockReason, shouldDeny };
+}
+

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

@@ -1,80 +1,80 @@
-/**
- * Plan file discovery: find, read, and hash the plan file.
- * Extracted from cc-native-plan-review.ts.
- */
-
-import * as fs from "node:fs";
-import * as path from "node:path";
-import * as os from "node:os";
-import * as crypto from "node:crypto";
-
-import { logInfo, logDebug } from "../../_shared/lib-ts/base/logger.js";
-import { findPlanPathInTranscript } from "../../_shared/lib-ts/context/plan-manager.js";
-
-import type { DiscoveredPlan } from "./types.js";
-
-const HOOK = "plan-discovery";
-
-/**
- * Find the most recently modified plan file in ~/.claude/plans/.
- */
-export function findPlanFile(): string | null {
-  const plansDir = path.join(os.homedir(), ".claude", "plans");
-  if (!fs.existsSync(plansDir)) return null;
-  const files = fs.readdirSync(plansDir)
-    .filter(f => f.endsWith(".md"))
-    .map(f => {
-      const p = path.join(plansDir, f);
-      return { path: p, mtime: fs.statSync(p).mtimeMs };
-    })
-    .sort((a, b) => b.mtime - a.mtime);
-  return files.length > 0 ? files[0]!.path : null;
-}
-
-/**
- * Compute a short SHA-256 hash of plan content.
- */
-export function computePlanHash(content: string): string {
-  return crypto.createHash("sha256").update(content, "utf-8").digest("hex").slice(0, 16);
-}
-
-/**
- * Discover the plan file, read its content, and compute its hash.
- * Prefers transcript-based discovery (session-accurate), falls back to mtime scan.
- */
-export function discoverPlan(transcriptPath?: string): DiscoveredPlan | null {
-  let planPath: string | null = null;
-
-  if (transcriptPath) {
-    planPath = findPlanPathInTranscript(transcriptPath);
-    if (planPath) {
-      logInfo(HOOK, `Found plan via transcript: ${planPath}`);
-    } else {
-      logDebug(HOOK, "No plan Write found in transcript, falling back to mtime scan");
-    }
-  }
-
-  if (!planPath) {
-    planPath = findPlanFile();
-  }
-
-  if (!planPath) return null;
-
-  let content: string;
-  try {
-    content = fs.readFileSync(planPath, "utf-8").trim();
-  } catch {
-    return null;
-  }
-
-  if (!content) return null;
-
-  logInfo(HOOK, `Found plan at: ${planPath}`);
-  logDebug(HOOK, `Plan length: ${content.length} chars`);
-
-  return {
-    path: planPath,
-    content,
-    hash: computePlanHash(content),
-  };
-}
+/**
+ * Plan file discovery: find, read, and hash the plan file.
+ * Extracted from cc-native-plan-review.ts.
+ */
+
+import * as fs from "node:fs";
+import * as path from "node:path";
+import * as os from "node:os";
+import * as crypto from "node:crypto";
+
+import { logInfo, logDebug } from "../../_shared/lib-ts/base/logger.js";
+import { findPlanPathInTranscript } from "../../_shared/lib-ts/context/plan-manager.js";
+
+import type { DiscoveredPlan } from "./types.js";
+
+const HOOK = "plan-discovery";
+
+/**
+ * Find the most recently modified plan file in ~/.claude/plans/.
+ */
+export function findPlanFile(): string | null {
+  const plansDir = path.join(os.homedir(), ".claude", "plans");
+  if (!fs.existsSync(plansDir)) return null;
+  const files = fs.readdirSync(plansDir)
+    .filter(f => f.endsWith(".md"))
+    .map(f => {
+      const p = path.join(plansDir, f);
+      return { path: p, mtime: fs.statSync(p).mtimeMs };
+    })
+    .sort((a, b) => b.mtime - a.mtime);
+  return files.length > 0 ? files[0]!.path : null;
+}
+
+/**
+ * Compute a short SHA-256 hash of plan content.
+ */
+export function computePlanHash(content: string): string {
+  return crypto.createHash("sha256").update(content, "utf-8").digest("hex").slice(0, 16);
+}
+
+/**
+ * Discover the plan file, read its content, and compute its hash.
+ * Prefers transcript-based discovery (session-accurate), falls back to mtime scan.
+ */
+export function discoverPlan(transcriptPath?: string): DiscoveredPlan | null {
+  let planPath: string | null = null;
+
+  if (transcriptPath) {
+    planPath = findPlanPathInTranscript(transcriptPath);
+    if (planPath) {
+      logInfo(HOOK, `Found plan via transcript: ${planPath}`);
+    } else {
+      logDebug(HOOK, "No plan Write found in transcript, falling back to mtime scan");
+    }
+  }
+
+  if (!planPath) {
+    planPath = findPlanFile();
+  }
+
+  if (!planPath) return null;
+
+  let content: string;
+  try {
+    content = fs.readFileSync(planPath, "utf-8").trim();
+  } catch {
+    return null;
+  }
+
+  if (!content) return null;
+
+  logInfo(HOOK, `Found plan at: ${planPath}`);
+  logDebug(HOOK, `Plan length: ${content.length} chars`);
+
+  return {
+    path: planPath,
+    content,
+    hash: computePlanHash(content),
+  };
+}

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

@@ -1,41 +1,41 @@
-/**
- * Plan quality guidance for context emission.
- *
- * Provides prompt text that guides the main agent to review plans before
- * presenting them to the user. Emitted via emitContext() — NOT appended to plan files.
- *
- * Used by both SubagentStop hook (Plan agents) and PostToolUse:Write hook (direct writes).
- */
-
-/**
- * Returns the plan quality review prompt to emit as context after a plan is written.
- * This guides the main agent to review the plan before calling ExitPlanMode.
- *
- * Design principles:
- * - No hardcoded skill names — agent discovers relevant skills from system-reminders
- * - Documentation focuses on WHY (preserve decisions) not WHERE (file paths)
- * - Concise — every token in emitted context costs attention budget
- * - Trusts the agent's judgment — guidance, not mandate
- */
-export function getPlanQualityReviewContext(): string {
-  return `## Plan Quality Review
-
-Before presenting this plan, review it from the perspective of an agent with zero conversation history.
-
-### Self-Check
-- File paths are absolute and verified (not "the auth file" or "as discussed")
-- Function and class names are exact references (not "the handler" or "it")
-- Each step is specific enough to execute without this conversation's context
-- Verification steps are binary-testable (pass/fail in one check)
-
-### Skills Integration
-Review the skills listed in your system-reminder messages. Where a step would benefit from a specific skill, reference it inline (e.g., "Use \`SkillName\` skill for [specific purpose]"). Only reference skills relevant to this plan's domain.
-
-### Documentation Reasoning
-Evaluate whether the plan captures decisions that would be lost when this session ends. The implementation agent should understand:
-- What was decided and why alternatives were rejected
-- What constraints exist that aren't obvious from the code
-- What would break if assumptions change
-
-If the plan has gaps, address them before presenting to the user.`;
-}
+/**
+ * Plan quality guidance for context emission.
+ *
+ * Provides prompt text that guides the main agent to review plans before
+ * presenting them to the user. Emitted via emitContext() — NOT appended to plan files.
+ *
+ * Used by both SubagentStop hook (Plan agents) and PostToolUse:Write hook (direct writes).
+ */
+
+/**
+ * Returns the plan quality review prompt to emit as context after a plan is written.
+ * This guides the main agent to review the plan before calling ExitPlanMode.
+ *
+ * Design principles:
+ * - No hardcoded skill names — agent discovers relevant skills from system-reminders
+ * - Documentation focuses on WHY (preserve decisions) not WHERE (file paths)
+ * - Concise — every token in emitted context costs attention budget
+ * - Trusts the agent's judgment — guidance, not mandate
+ */
+export function getPlanQualityReviewContext(): string {
+  return `## Plan Quality Review
+
+Before presenting this plan, review it from the perspective of an agent with zero conversation history.
+
+### Self-Check
+- File paths are absolute and verified (not "the auth file" or "as discussed")
+- Function and class names are exact references (not "the handler" or "it")
+- Each step is specific enough to execute without this conversation's context
+- Verification steps are binary-testable (pass/fail in one check)
+
+### Skills Integration
+Review the skills listed in your system-reminder messages. Where a step would benefit from a specific skill, reference it inline (e.g., "Use \`SkillName\` skill for [specific purpose]"). Only reference skills relevant to this plan's domain.
+
+### Documentation Reasoning
+Evaluate whether the plan captures decisions that would be lost when this session ends. The implementation agent should understand:
+- What was decided and why alternatives were rejected
+- What constraints exist that aren't obvious from the code
+- What would break if assumptions change
+
+If the plan has gaps, address them before presenting to the user.`;
+}