aiwcli 0.15.7 → 0.17.0

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

@@ -1,132 +1,132 @@
-/**
- * Graduation logic: pass eligibility, pass streaks, graduation threshold, iteration advancement.
- * Extracted from cc-native-plan-review.ts.
- */
-
-import type {
-  IterationState,
-  ReviewerResult,
-  CombinedReviewResult,
-  IterationAdvancement,
-} from "../../lib-ts/types.js";
-
-// ---------------------------------------------------------------------------
-// Pass Eligibility
-// ---------------------------------------------------------------------------
-
-/**
- * Determine which agents are pass-eligible this iteration.
- * Criteria: verdict === "pass" OR zero high-severity issues.
- * Agents with "skip"/"error" are NOT eligible (no signal).
- */
-export function computePassEligible(agentResults: Record<string, ReviewerResult>): string[] {
-  const eligible: string[] = [];
-  for (const [name, result] of Object.entries(agentResults)) {
-    if (result.verdict === "skip" || result.verdict === "error") continue;
-    if (result.verdict === "pass") { eligible.push(name); continue; }
-    const issues = Array.isArray(result.data?.issues)
-      ? (result.data.issues as Array<{ severity?: string }>) : [];
-    if (issues.filter(i => i.severity === "high").length === 0) {
-      eligible.push(name);
-    }
-  }
-  return eligible;
-}
-
-// ---------------------------------------------------------------------------
-// Tracker Issue Extraction
-// ---------------------------------------------------------------------------
-
-/**
- * Extract top high-severity issues for the review tracker.
- */
-export function extractTopIssuesForTracker(
-  combined: CombinedReviewResult,
-  maxCount = 5,
-): string[] {
-  const allReviewers = Object.values(combined.agents);
-  const issues: string[] = [];
-  for (const r of allReviewers) {
-    if (!r.data) continue;
-    const issueList = r.data.issues as Array<Record<string, unknown>> | undefined;
-    if (!issueList) continue;
-    for (const issue of issueList) {
-      if (issue.severity === "high") {
-        const text = String(issue.issue ?? "").trim();
-        if (text) {
-          issues.push(`[${r.name}] ${text}`);
-        }
-      }
-    }
-    if (issues.length >= maxCount) break;
-  }
-  return issues.slice(0, maxCount);
-}
-
-// ---------------------------------------------------------------------------
-// Iteration Advancement
-// ---------------------------------------------------------------------------
-
-const GRADUATION_THRESHOLD = 2;
-
-/**
- * Advance iteration state after a review cycle. Returns a new state copy
- * (does not mutate input).
- *
- * - On pass/warrant: sets current past max (stop iterating)
- * - On deny: increments current toward max (safety valve)
- * - Updates pass streaks and graduates agents that reached threshold
- */
-export function advanceIterationState(
-  state: IterationState,
-  planHash: string,
-  planPath: string,
-  verdict: string,
-  shouldDeny: boolean,
-  passEligible: string[],
-  agentResults: Record<string, ReviewerResult>,
-  graduationThreshold = GRADUATION_THRESHOLD,
-): IterationAdvancement {
-  const updated: IterationState = {
-    ...state,
-    history: [...state.history, { hash: planHash, verdict, timestamp: new Date().toISOString() }],
-    lastPlanHash: planHash,
-    lastPlanPath: planPath,
-    graduated: [...state.graduated],
-    passStreaks: { ...state.passStreaks },
-  };
-
-  if (!shouldDeny) {
-    // Pass/warrant: stop iterating
-    updated.current = updated.max + 1;
-  } else {
-    // Deny: advance toward max
-    updated.current = state.current + 1;
-  }
-
-  // Update pass streaks — only for agents that actually ran this iteration
-  const passEligibleSet = new Set(passEligible);
-  const graduatedSet = new Set(updated.graduated);
-
-  for (const name of Object.keys(agentResults)) {
-    if (graduatedSet.has(name)) continue;
-    if (passEligibleSet.has(name)) {
-      updated.passStreaks[name] = (updated.passStreaks[name] ?? 0) + 1;
-    } else {
-      updated.passStreaks[name] = 0;
-    }
-  }
-
-  // Graduate agents that reached threshold
-  const newGraduates: string[] = [];
-  for (const [name, streak] of Object.entries(updated.passStreaks)) {
-    if (streak >= graduationThreshold && !graduatedSet.has(name)) {
-      newGraduates.push(name);
-    }
-  }
-  if (newGraduates.length > 0) {
-    updated.graduated = [...updated.graduated, ...newGraduates];
-  }
-
-  return { updatedState: updated, newGraduates };
-}
+/**
+ * Graduation logic: pass eligibility, pass streaks, graduation threshold, iteration advancement.
+ * Extracted from cc-native-plan-review.ts.
+ */
+
+import type {
+  IterationState,
+  ReviewerResult,
+  CombinedReviewResult,
+  IterationAdvancement,
+} from "./types.js";
+
+// ---------------------------------------------------------------------------
+// Pass Eligibility
+// ---------------------------------------------------------------------------
+
+/**
+ * Determine which agents are pass-eligible this iteration.
+ * Criteria: verdict === "pass" OR zero high-severity issues.
+ * Agents with "skip"/"error" are NOT eligible (no signal).
+ */
+export function computePassEligible(agentResults: Record<string, ReviewerResult>): string[] {
+  const eligible: string[] = [];
+  for (const [name, result] of Object.entries(agentResults)) {
+    if (result.verdict === "skip" || result.verdict === "error") continue;
+    if (result.verdict === "pass") { eligible.push(name); continue; }
+    const issues = Array.isArray(result.data?.issues)
+      ? (result.data.issues as Array<{ severity?: string }>) : [];
+    if (issues.filter(i => i.severity === "high").length === 0) {
+      eligible.push(name);
+    }
+  }
+  return eligible;
+}
+
+// ---------------------------------------------------------------------------
+// Tracker Issue Extraction
+// ---------------------------------------------------------------------------
+
+/**
+ * Extract top high-severity issues for the review tracker.
+ */
+export function extractTopIssuesForTracker(
+  combined: CombinedReviewResult,
+  maxCount = 5,
+): string[] {
+  const allReviewers = Object.values(combined.agents);
+  const issues: string[] = [];
+  for (const r of allReviewers) {
+    if (!r.data) continue;
+    const issueList = r.data.issues as Array<Record<string, unknown>> | undefined;
+    if (!issueList) continue;
+    for (const issue of issueList) {
+      if (issue.severity === "high") {
+        const text = String(issue.issue ?? "").trim();
+        if (text) {
+          issues.push(`[${r.name}] ${text}`);
+        }
+      }
+    }
+    if (issues.length >= maxCount) break;
+  }
+  return issues.slice(0, maxCount);
+}
+
+// ---------------------------------------------------------------------------
+// Iteration Advancement
+// ---------------------------------------------------------------------------
+
+const GRADUATION_THRESHOLD = 2;
+
+/**
+ * Advance iteration state after a review cycle. Returns a new state copy
+ * (does not mutate input).
+ *
+ * - On pass/warrant: sets current past max (stop iterating)
+ * - On deny: increments current toward max (safety valve)
+ * - Updates pass streaks and graduates agents that reached threshold
+ */
+export function advanceIterationState(
+  state: IterationState,
+  planHash: string,
+  planPath: string,
+  verdict: string,
+  shouldDeny: boolean,
+  passEligible: string[],
+  agentResults: Record<string, ReviewerResult>,
+  graduationThreshold = GRADUATION_THRESHOLD,
+): IterationAdvancement {
+  const updated: IterationState = {
+    ...state,
+    history: [...state.history, { hash: planHash, verdict, timestamp: new Date().toISOString() }],
+    lastPlanHash: planHash,
+    lastPlanPath: planPath,
+    graduated: [...state.graduated],
+    passStreaks: { ...(state.passStreaks ?? {}) },
+  };
+
+  if (!shouldDeny) {
+    // Pass/warrant: stop iterating
+    updated.current = updated.max + 1;
+  } else {
+    // Deny: advance toward max
+    updated.current = state.current + 1;
+  }
+
+  // Update pass streaks — only for agents that actually ran this iteration
+  const passEligibleSet = new Set(passEligible);
+  const graduatedSet = new Set(updated.graduated);
+
+  for (const name of Object.keys(agentResults)) {
+    if (graduatedSet.has(name)) continue;
+    if (passEligibleSet.has(name)) {
+      updated.passStreaks[name] = (updated.passStreaks[name] ?? 0) + 1;
+    } else {
+      updated.passStreaks[name] = 0;
+    }
+  }
+
+  // Graduate agents that reached threshold
+  const newGraduates: string[] = [];
+  for (const [name, streak] of Object.entries(updated.passStreaks)) {
+    if (streak >= graduationThreshold && !graduatedSet.has(name)) {
+      newGraduates.push(name);
+    }
+  }
+  if (newGraduates.length > 0) {
+    updated.graduated = [...updated.graduated, ...newGraduates];
+  }
+
+  return { updatedState: updated, newGraduates };
+}

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

@@ -3,114 +3,116 @@
  * Re-exports the public API from all modules.
  */
 
-// Artifacts
-export {
-  buildCombinedJson,
-  buildHighIssuesDocument,
-  buildInlineReviewSummary,
-  extractTopIssuesText,
-  formatCombinedMarkdown,
-  formatReviewMarkdown,
-  generateReviewIndex,
-  writeCombinedArtifacts,
-} from "../artifacts/lib/index.js";
-
-// Orchestrator
-export { buildOrchestratorSchema, runOrchestrator } from "../plan-review/lib/orchestrator.js";
-
-// Reviewers
-export {
-  AgentReviewer,
-  runAgentReview,
-} from "../plan-review/lib/reviewers/index.js";
-
-// Verdict aggregation
-export { computeReviewDecision, worstVerdict } from "../plan-review/lib/verdict.js";
-
-// Agent aggregation
-export {
-  aggregateAgents,
-  extractBody,
-  extractFrontmatter,
-} from "./aggregate-agents.js";
+// Types & schemas
+export type {
+  Verdict,
+  ReviewDecision,
+  ComplexityCategory,
+  ReviewIssue,
+  ReviewData,
+  ReviewerResult,
+  OrchestratorResult,
+  CombinedReviewResult,
+  ReviewDecisionResult,
+  AgentConfig,
+  OrchestratorConfig,
+  IterationState,
+  IterationEntry,
+  CcNativeState,
+  PlanReviewState,
+  QuestionsAskedState,
+  StuckDetectionState,
+  PlanReviewConfig,
+  DisplaySettings,
+  ReviewOptions,
+  Reviewer,
+} from "./types.js";
 
-// CC-native state
 export {
-  getCcNativeState,
-  isPlanAlreadyReviewed,
-  markPlanReviewed,
-  markQuestionsAsked,
-  saveCcNativeState,
-  wasPlanPreviouslyDenied,
-  wasQuestionsAsked,
-} from "./cc-native-state.js";
-
-// CLI output parsing
-export { parseCliOutput } from "./cli-output-parser.js";
-
-// Configuration
-export { getDisplaySettings, loadConfig } from "./config.js";
+  REVIEW_SCHEMA,
+  ORCHESTRATOR_SCHEMA,
+  REVIEW_PROMPT_PREFIX,
+  AGENT_REVIEW_PROMPT_PREFIX,
+  DEFAULT_DISPLAY,
+  DEFAULT_SANITIZATION,
+} from "./types.js";
 
 // Constants & security
 export {
-  ENABLE_PLAN_NOTIFICATIONS,
   ENABLE_ROBUST_PLAN_WRITES,
-  MAX_ERROR_FILE_SIZE,
+  ENABLE_PLAN_NOTIFICATIONS,
+  PLANS_DIR,
   MAX_PLAN_PATH_LENGTH,
+  MAX_ERROR_FILE_SIZE,
   MAX_RETRY_ATTEMPTS,
-  MAX_TOTAL_RETRY_TIME_MS,
-  PLANS_DIR,
   RETRY_BACKOFF_MS,
+  MAX_TOTAL_RETRY_TIME_MS,
   validatePlanPath,
 } from "./constants.js";
 
-// Debug logging
-export { cleanupDebugFolder, debugLog, debugRaw, getDebugDir } from "./debug.js";
+// Verdict aggregation
+export { worstVerdict, computeReviewDecision } from "./verdict.js";
 
 // JSON parsing
-export { coerceToReview, parseJsonMaybe } from "./json-parser.js";
+export { parseJsonMaybe, coerceToReview } from "./json-parser.js";
+
+// CLI output parsing
+export { parseCliOutput } from "./cli-output-parser.js";
+
+// Configuration
+export { loadConfig, getDisplaySettings } from "./config.js";
+
+// Debug logging
+export { debugLog, debugRaw, getDebugDir, cleanupDebugFolder } from "./debug.js";
+
+// CC-native state
+export {
+  getCcNativeState,
+  saveCcNativeState,
+  isPlanAlreadyReviewed,
+  wasPlanPreviouslyDenied,
+  markPlanReviewed,
+  wasQuestionsAsked,
+  markQuestionsAsked,
+  getStuckDetectionState,
+  updateStuckDetectionState,
+} from "./cc-native-state.js";
 
 // Iteration state
 export {
-  DEFAULT_REVIEW_ITERATIONS,
-  deleteState,
-  getIterationState,
   getStateFilePath,
   loadState,
   saveStateToPlan,
-  shouldContinueIterating,
+  deleteState,
+  getIterationState,
   updateIterationState,
+  shouldContinueIterating,
 } from "./state.js";
 
-// Types & schemas
-export type {
-  AgentConfig,
-  CcNativeState,
-  CombinedReviewResult,
-  ComplexityCategory,
-  DisplaySettings,
-  IterationEntry,
-  IterationState,
-  OrchestratorConfig,
-  OrchestratorResult,
-  PlanReviewConfig,
-  PlanReviewState,
-  QuestionsAskedState,
-  ReviewData,
-  ReviewDecision,
-  ReviewDecisionResult,
-  Reviewer,
-  ReviewerResult,
-  ReviewIssue,
-  ReviewOptions,
-  Verdict,
-} from "./types.js";
+// Orchestrator
+export { runOrchestrator, buildOrchestratorSchema } from "./orchestrator.js";
 
+// Agent aggregation
 export {
-  AGENT_REVIEW_PROMPT_PREFIX,
-  DEFAULT_DISPLAY,
-  DEFAULT_SANITIZATION,
-  ORCHESTRATOR_SCHEMA,
-  REVIEW_PROMPT_PREFIX,
-  REVIEW_SCHEMA,
-} from "./types.js";
+  aggregateAgents,
+  extractFrontmatter,
+  extractBody,
+} from "./aggregate-agents.js";
+
+// Artifacts
+export {
+  formatReviewMarkdown,
+  formatCombinedMarkdown,
+  buildInlineReviewSummary,
+  extractTopIssuesText,
+  buildHighIssuesDocument,
+  generateReviewIndex,
+  buildCombinedJson,
+  writeCombinedArtifacts,
+} from "./artifacts.js";
+
+// Reviewers
+export {
+  AgentReviewer,
+  runAgentReview,
+} from "./reviewers/index.js";

package/dist/templates/cc-native/_cc-native/lib-ts/json-parser.ts

@@ -3,8 +3,8 @@
  * See cc-native-plan-review-spec.md §4.10-4.11
  */
 
-import type { ReviewData, Verdict } from "./types.js";
 import { logDebug, logWarn } from "../../_core/lib-ts/runtime/logger.js";
+import type { ReviewData, Verdict } from "./types.js";
 
 /**
  * Try strict JSON parse. If that fails, attempt to extract the first {...} block.
@@ -16,12 +16,12 @@ import { logDebug, logWarn } from "../../_core/lib-ts/runtime/logger.js";
 export function parseJsonMaybe(
   text: string,
   requireFields?: string[],
-): null | Record<string, unknown> {
+): Record<string, unknown> | null {
   const trimmed = text.trim();
   if (!trimmed) return null;
 
-  let obj: null | Record<string, unknown> = null;
-  let parseMethod: null | string = null;
+  let obj: Record<string, unknown> | null = null;
+  let parseMethod: string | null = null;
 
   // Strict parse
   try {
@@ -93,7 +93,7 @@ export function parseJsonMaybe(
  * @returns Tuple of [ok, verdict, normalizedData]
  */
 export function coerceToReview(
-  obj: null | Record<string, unknown>,
+  obj: Record<string, unknown> | null,
   defaultFixMsg = "Retry or check configuration.",
 ): [boolean, Verdict, ReviewData] {
   if (!obj) {
@@ -144,7 +144,6 @@ export function coerceToReview(
       `verdict=${obj.verdict}, issues_count=${Array.isArray(obj.issues) ? (obj.issues as unknown[]).length : 0}`,
     );
   }
-
   if (!obj.issues) {
     logDebug("coerce", "issues array empty or missing");
   }

package/dist/templates/cc-native/_cc-native/{plan-review/lib → 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 { logInfo, logWarn } from "../../../_core/lib-ts/runtime/logger.js";
-import type { AgentConfig, AgentReviewSettings, OrchestratorConfig, OrchestratorResult } from "../../lib-ts/types.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: AgentReviewSettings,
-  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: config.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?.fallbackCount ?? 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 { logInfo, logWarn } from "../../_core/lib-ts/runtime/logger.js";
+import type { AgentConfig, OrchestratorConfig, OrchestratorResult } from "./types.js";
+import { OrchestratorClaudeAgent } from "./reviewers/providers/orchestrator-claude-agent.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 (e) {
+    logWarn("orchestrator", `Unexpected error: ${e}`);
+    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: ${e}`,
+      error: String(e),
+    };
+  }
+}