aiwcli 0.12.6 → 0.12.8

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

@@ -1,144 +1,144 @@
-/**
- * Unified Claude CLI output parser.
- * Deduplicates identical logic from orchestrator.ts and reviewers/agent.ts.
- * See cc-native-plan-review-spec.md §4.6
- */
-
-import { parseJsonMaybe } from "./json-parser.js";
-import { logDebug, logError, logWarn } from "../../_shared/lib-ts/base/logger.js";
-
-/**
- * Parse Claude CLI JSON output, handling various formats.
- *
- * Claude CLI can output in several formats:
- * - Direct structured_output dict
- * - Assistant message with StructuredOutput tool use
- * - List of events with assistant messages
- * - Raw text with embedded JSON (heuristic fallback)
- *
- * @param raw - Raw stdout from Claude CLI
- * @param requireFields - Optional fields to validate in heuristic fallback
- * @returns Parsed JSON dict or null if parsing failed
- */
-export function parseCliOutput(
-  raw: string,
-  requireFields?: string[],
-): null | Record<string, unknown> {
-  try {
-    const result: unknown = JSON.parse(raw);
-
-    if (result !== null && typeof result === "object" && !Array.isArray(result)) {
-      const dict = result as Record<string, unknown>;
-
-      // Strategy 1: Direct structured_output key
-      if ("structured_output" in dict) {
-        logDebug("cli_parser", "Found structured_output in root dict");
-        return dict.structured_output as Record<string, unknown>;
-      }
-
-      // Strategy 1.5: Session result envelope (type: "result")
-      // When the model fails to call StructuredOutput, the CLI returns a
-      // session metadata object with type/subtype/duration_ms/usage/etc.
-      // but no structured_output key. Check for the `result` text field
-      // (model may have written review as text) and for error states.
-      if (dict.type === "result" || ("duration_ms" in dict && "session_id" in dict)) {
-        if (dict.is_error === true || (Array.isArray(dict.errors) && (dict.errors as unknown[]).length > 0)) {
-          logWarn("cli_parser", `CLI returned error result: ${JSON.stringify(dict.errors ?? "is_error=true")}`);
-          return null;
-        }
-        if (typeof dict.result === "string" && dict.result) {
-          logDebug("cli_parser", "Found result text in session envelope, attempting JSON extraction");
-          const extracted = parseJsonMaybe(dict.result as string, requireFields);
-          if (extracted) return extracted;
-          logWarn("cli_parser", "Session envelope result text contained no extractable JSON");
-        }
-        logDebug("cli_parser", "Session result envelope with no structured_output or extractable result");
-        return null;
-      }
-
-      // Strategy 2: Assistant message with StructuredOutput tool use
-      if (dict.type === "assistant") {
-        const message = dict.message as Record<string, unknown> | undefined;
-        const content = message?.content;
-        if (Array.isArray(content)) {
-          for (const item of content) {
-            if (
-              item !== null &&
-              typeof item === "object" &&
-              (item as Record<string, unknown>).name === "StructuredOutput"
-            ) {
-              logDebug(
-                "cli_parser",
-                "Found StructuredOutput in assistant message content",
-              );
-              return (item as Record<string, unknown>).input as Record<
-                string,
-                unknown
-              >;
-            }
-          }
-        }
-
-        logDebug(
-          "cli_parser",
-          "Assistant message found but no StructuredOutput tool use in content",
-        );
-      }
-    } else if (Array.isArray(result)) {
-      // Strategy 3: List of events with assistant messages
-      logDebug(
-        "cli_parser",
-        `Received list of ${(result as unknown[]).length} events, searching for assistant message`,
-      );
-      for (let i = 0; i < (result as unknown[]).length; i++) {
-        const event = (result as unknown[])[i];
-        if (event === null || typeof event !== "object") continue;
-
-        const dict = event as Record<string, unknown>;
-        if (dict.type === "assistant") {
-          const message = dict.message as Record<string, unknown> | undefined;
-          const content = message?.content;
-          if (Array.isArray(content)) {
-            for (const item of content) {
-              if (
-                item !== null &&
-                typeof item === "object" &&
-                (item as Record<string, unknown>).name === "StructuredOutput"
-              ) {
-                logDebug(
-                  "cli_parser",
-                  `Found StructuredOutput in event[${i}] assistant message`,
-                );
-                return (item as Record<string, unknown>).input as Record<
-                  string,
-                  unknown
-                >;
-              }
-            }
-          }
-        }
-      }
-
-      logDebug(
-        "cli_parser",
-        "No StructuredOutput found in any assistant message in event list",
-      );
-    }
-  } catch (error: unknown) {
-    if (error instanceof SyntaxError) {
-      logWarn("cli_parser", `JSON decode error: ${error.message}`);
-    } else {
-      logError(
-        "cli_parser",
-        `Unexpected error during structured parsing: ${error}`,
-      );
-    }
-  }
-
-  // Strategy 4: Heuristic {…} extraction fallback
-  logDebug(
-    "cli_parser",
-    "No structured output found, falling back to heuristic JSON extraction",
-  );
-  return parseJsonMaybe(raw, requireFields);
-}
+/**
+ * Unified Claude CLI output parser.
+ * Deduplicates identical logic from orchestrator.ts and reviewers/agent.ts.
+ * See cc-native-plan-review-spec.md §4.6
+ */
+
+import { parseJsonMaybe } from "./json-parser.js";
+import { logDebug, logError, logWarn } from "../../_shared/lib-ts/base/logger.js";
+
+/**
+ * Parse Claude CLI JSON output, handling various formats.
+ *
+ * Claude CLI can output in several formats:
+ * - Direct structured_output dict
+ * - Assistant message with StructuredOutput tool use
+ * - List of events with assistant messages
+ * - Raw text with embedded JSON (heuristic fallback)
+ *
+ * @param raw - Raw stdout from Claude CLI
+ * @param requireFields - Optional fields to validate in heuristic fallback
+ * @returns Parsed JSON dict or null if parsing failed
+ */
+export function parseCliOutput(
+  raw: string,
+  requireFields?: string[],
+): null | Record<string, unknown> {
+  try {
+    const result: unknown = JSON.parse(raw);
+
+    if (result !== null && typeof result === "object" && !Array.isArray(result)) {
+      const dict = result as Record<string, unknown>;
+
+      // Strategy 1: Direct structured_output key
+      if ("structured_output" in dict) {
+        logDebug("cli_parser", "Found structured_output in root dict");
+        return dict.structured_output as Record<string, unknown>;
+      }
+
+      // Strategy 1.5: Session result envelope (type: "result")
+      // When the model fails to call StructuredOutput, the CLI returns a
+      // session metadata object with type/subtype/duration_ms/usage/etc.
+      // but no structured_output key. Check for the `result` text field
+      // (model may have written review as text) and for error states.
+      if (dict.type === "result" || ("duration_ms" in dict && "session_id" in dict)) {
+        if (dict.is_error === true || (Array.isArray(dict.errors) && (dict.errors as unknown[]).length > 0)) {
+          logWarn("cli_parser", `CLI returned error result: ${JSON.stringify(dict.errors ?? "is_error=true")}`);
+          return null;
+        }
+        if (typeof dict.result === "string" && dict.result) {
+          logDebug("cli_parser", "Found result text in session envelope, attempting JSON extraction");
+          const extracted = parseJsonMaybe(dict.result as string, requireFields);
+          if (extracted) return extracted;
+          logWarn("cli_parser", "Session envelope result text contained no extractable JSON");
+        }
+        logDebug("cli_parser", "Session result envelope with no structured_output or extractable result");
+        return null;
+      }
+
+      // Strategy 2: Assistant message with StructuredOutput tool use
+      if (dict.type === "assistant") {
+        const message = dict.message as Record<string, unknown> | undefined;
+        const content = message?.content;
+        if (Array.isArray(content)) {
+          for (const item of content) {
+            if (
+              item !== null &&
+              typeof item === "object" &&
+              (item as Record<string, unknown>).name === "StructuredOutput"
+            ) {
+              logDebug(
+                "cli_parser",
+                "Found StructuredOutput in assistant message content",
+              );
+              return (item as Record<string, unknown>).input as Record<
+                string,
+                unknown
+              >;
+            }
+          }
+        }
+
+        logDebug(
+          "cli_parser",
+          "Assistant message found but no StructuredOutput tool use in content",
+        );
+      }
+    } else if (Array.isArray(result)) {
+      // Strategy 3: List of events with assistant messages
+      logDebug(
+        "cli_parser",
+        `Received list of ${(result as unknown[]).length} events, searching for assistant message`,
+      );
+      for (let i = 0; i < (result as unknown[]).length; i++) {
+        const event = (result as unknown[])[i];
+        if (event === null || typeof event !== "object") continue;
+
+        const dict = event as Record<string, unknown>;
+        if (dict.type === "assistant") {
+          const message = dict.message as Record<string, unknown> | undefined;
+          const content = message?.content;
+          if (Array.isArray(content)) {
+            for (const item of content) {
+              if (
+                item !== null &&
+                typeof item === "object" &&
+                (item as Record<string, unknown>).name === "StructuredOutput"
+              ) {
+                logDebug(
+                  "cli_parser",
+                  `Found StructuredOutput in event[${i}] assistant message`,
+                );
+                return (item as Record<string, unknown>).input as Record<
+                  string,
+                  unknown
+                >;
+              }
+            }
+          }
+        }
+      }
+
+      logDebug(
+        "cli_parser",
+        "No StructuredOutput found in any assistant message in event list",
+      );
+    }
+  } catch (error: unknown) {
+    if (error instanceof SyntaxError) {
+      logWarn("cli_parser", `JSON decode error: ${error.message}`);
+    } else {
+      logError(
+        "cli_parser",
+        `Unexpected error during structured parsing: ${error}`,
+      );
+    }
+  }
+
+  // Strategy 4: Heuristic {…} extraction fallback
+  logDebug(
+    "cli_parser",
+    "No structured output found, falling back to heuristic JSON extraction",
+  );
+  return parseJsonMaybe(raw, requireFields);
+}

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

@@ -1,57 +1,57 @@
-/**
- * Configuration loading for cc-native plan review.
- * See cc-native-plan-review-spec.md §4.4
- */
-
-import * as fs from "node:fs";
-import * as path from "node:path";
-
-import type { DisplaySettings, PlanReviewConfig } from "./types.js";
-import { DEFAULT_DISPLAY } from "./types.js";
-import { logWarn } from "../../_shared/lib-ts/base/logger.js";
-
-/**
- * Load full CC-Native config from _cc-native/cc-native.config.json.
- */
-export function loadConfig(projectDir: string): PlanReviewConfig {
-  const settingsPath = path.join(
-    projectDir,
-    "_cc-native",
-    "cc-native.config.json",
-  );
-
-  if (!fs.existsSync(settingsPath)) {
-    return {};
-  }
-
-  try {
-    const raw = fs.readFileSync(settingsPath, "utf8");
-    return JSON.parse(raw) as PlanReviewConfig;
-  } catch (error: unknown) {
-    logWarn("cc-native", `Failed to load config: ${error}`);
-    return {};
-  }
-}
-
-/**
- * Get display settings, checking section-specific first, then root.
- */
-export function getDisplaySettings(
-  config: PlanReviewConfig,
-  section: string,
-): DisplaySettings {
-  const sectionConfig = config[section];
-  const sectionDisplay =
-    sectionConfig && typeof sectionConfig === "object"
-      ? ((sectionConfig as Record<string, unknown>).display as
-          | Partial<DisplaySettings>
-          | undefined) ?? {}
-      : {};
-  const rootDisplay = config.display ?? {};
-
-  return {
-    ...DEFAULT_DISPLAY,
-    ...rootDisplay,
-    ...sectionDisplay,
-  };
-}
+/**
+ * Configuration loading for cc-native plan review.
+ * See cc-native-plan-review-spec.md §4.4
+ */
+
+import * as fs from "node:fs";
+import * as path from "node:path";
+
+import type { DisplaySettings, PlanReviewConfig } from "./types.js";
+import { DEFAULT_DISPLAY } from "./types.js";
+import { logWarn } from "../../_shared/lib-ts/base/logger.js";
+
+/**
+ * Load full CC-Native config from _cc-native/cc-native.config.json.
+ */
+export function loadConfig(projectDir: string): PlanReviewConfig {
+  const settingsPath = path.join(
+    projectDir,
+    "_cc-native",
+    "cc-native.config.json",
+  );
+
+  if (!fs.existsSync(settingsPath)) {
+    return {};
+  }
+
+  try {
+    const raw = fs.readFileSync(settingsPath, "utf8");
+    return JSON.parse(raw) as PlanReviewConfig;
+  } catch (error: unknown) {
+    logWarn("cc-native", `Failed to load config: ${error}`);
+    return {};
+  }
+}
+
+/**
+ * Get display settings, checking section-specific first, then root.
+ */
+export function getDisplaySettings(
+  config: PlanReviewConfig,
+  section: string,
+): DisplaySettings {
+  const sectionConfig = config[section];
+  const sectionDisplay =
+    sectionConfig && typeof sectionConfig === "object"
+      ? ((sectionConfig as Record<string, unknown>).display as
+          | Partial<DisplaySettings>
+          | undefined) ?? {}
+      : {};
+  const rootDisplay = config.display ?? {};
+
+  return {
+    ...DEFAULT_DISPLAY,
+    ...rootDisplay,
+    ...sectionDisplay,
+  };
+}

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

@@ -1,83 +1,83 @@
-/**
- * Security and configuration constants for cc-native plan review.
- * See cc-native-plan-review-spec.md §4.15
- */
-
-import * as os from "node:os";
-import * as path from "node:path";
-
-// ---------------------------------------------------------------------------
-// Feature Flags
-// ---------------------------------------------------------------------------
-
-/** Enable atomic writes for plan state files (env: CC_NATIVE_ROBUST_WRITES) */
-export const ENABLE_ROBUST_PLAN_WRITES =
-  (process.env.CC_NATIVE_ROBUST_WRITES ?? "true").toLowerCase() === "true";
-
-/** Enable plan review notifications (env: CC_NATIVE_NOTIFICATIONS) */
-export const ENABLE_PLAN_NOTIFICATIONS =
-  (process.env.CC_NATIVE_NOTIFICATIONS ?? "false").toLowerCase() === "true";
-
-// ---------------------------------------------------------------------------
-// Security Constants
-// ---------------------------------------------------------------------------
-
-/** Default plans directory */
-export const PLANS_DIR = path.join(os.homedir(), ".claude", "plans");
-
-/** Maximum plan path length */
-export const MAX_PLAN_PATH_LENGTH = 4096;
-
-/** Maximum error file size (10KB) */
-export const MAX_ERROR_FILE_SIZE = 10 * 1024;
-
-// ---------------------------------------------------------------------------
-// Performance Constants
-// ---------------------------------------------------------------------------
-
-/** Fast-fail: 2 attempts max */
-export const MAX_RETRY_ATTEMPTS = 2;
-
-/** Backoff schedule: 0.5s, 1s */
-export const RETRY_BACKOFF_MS = [500, 1000];
-
-/** 3 seconds total, well under 5s hook timeout */
-export const MAX_TOTAL_RETRY_TIME_MS = 3000;
-
-// ---------------------------------------------------------------------------
-// Path Validation
-// ---------------------------------------------------------------------------
-
-/**
- * Validate and sanitize plan path to prevent traversal attacks.
- * Throws ValueError if path is invalid, too long, or outside allowed directory.
- */
-export function validatePlanPath(planPath: string): string {
-  // Input validation
-  if (!planPath || planPath.length > MAX_PLAN_PATH_LENGTH) {
-    throw new Error(
-      `Invalid plan path length: ${planPath ? planPath.length : 0}`,
-    );
-  }
-
-  if (planPath.includes("\u0000")) {
-    throw new Error("Null bytes not allowed in path");
-  }
-
-  // Normalize and resolve to absolute canonical path
-  let resolved: string;
-  try {
-    resolved = path.resolve(planPath);
-  } catch (error: unknown) {
-    throw new Error(`Path resolution failed: ${error}`);
-  }
-
-  // Verify path is within allowed directory (case-insensitive on Windows)
-  const resolvedLower = resolved.toLowerCase().replaceAll('\\', "/");
-  const plansDirLower = PLANS_DIR.toLowerCase().replaceAll('\\', "/");
-  if (!resolvedLower.startsWith(plansDirLower)) {
-    throw new Error(`Path outside allowed directory: ${PLANS_DIR}`);
-  }
-
-  return resolved;
-}
+/**
+ * Security and configuration constants for cc-native plan review.
+ * See cc-native-plan-review-spec.md §4.15
+ */
+
+import * as os from "node:os";
+import * as path from "node:path";
+
+// ---------------------------------------------------------------------------
+// Feature Flags
+// ---------------------------------------------------------------------------
+
+/** Enable atomic writes for plan state files (env: CC_NATIVE_ROBUST_WRITES) */
+export const ENABLE_ROBUST_PLAN_WRITES =
+  (process.env.CC_NATIVE_ROBUST_WRITES ?? "true").toLowerCase() === "true";
+
+/** Enable plan review notifications (env: CC_NATIVE_NOTIFICATIONS) */
+export const ENABLE_PLAN_NOTIFICATIONS =
+  (process.env.CC_NATIVE_NOTIFICATIONS ?? "false").toLowerCase() === "true";
+
+// ---------------------------------------------------------------------------
+// Security Constants
+// ---------------------------------------------------------------------------
+
+/** Default plans directory */
+export const PLANS_DIR = path.join(os.homedir(), ".claude", "plans");
+
+/** Maximum plan path length */
+export const MAX_PLAN_PATH_LENGTH = 4096;
+
+/** Maximum error file size (10KB) */
+export const MAX_ERROR_FILE_SIZE = 10 * 1024;
+
+// ---------------------------------------------------------------------------
+// Performance Constants
+// ---------------------------------------------------------------------------
+
+/** Fast-fail: 2 attempts max */
+export const MAX_RETRY_ATTEMPTS = 2;
+
+/** Backoff schedule: 0.5s, 1s */
+export const RETRY_BACKOFF_MS = [500, 1000];
+
+/** 3 seconds total, well under 5s hook timeout */
+export const MAX_TOTAL_RETRY_TIME_MS = 3000;
+
+// ---------------------------------------------------------------------------
+// Path Validation
+// ---------------------------------------------------------------------------
+
+/**
+ * Validate and sanitize plan path to prevent traversal attacks.
+ * Throws ValueError if path is invalid, too long, or outside allowed directory.
+ */
+export function validatePlanPath(planPath: string): string {
+  // Input validation
+  if (!planPath || planPath.length > MAX_PLAN_PATH_LENGTH) {
+    throw new Error(
+      `Invalid plan path length: ${planPath ? planPath.length : 0}`,
+    );
+  }
+
+  if (planPath.includes("\u0000")) {
+    throw new Error("Null bytes not allowed in path");
+  }
+
+  // Normalize and resolve to absolute canonical path
+  let resolved: string;
+  try {
+    resolved = path.resolve(planPath);
+  } catch (error: unknown) {
+    throw new Error(`Path resolution failed: ${error}`);
+  }
+
+  // Verify path is within allowed directory (case-insensitive on Windows)
+  const resolvedLower = resolved.toLowerCase().replaceAll('\\', "/");
+  const plansDirLower = PLANS_DIR.toLowerCase().replaceAll('\\', "/");
+  if (!resolvedLower.startsWith(plansDirLower)) {
+    throw new Error(`Path outside allowed directory: ${PLANS_DIR}`);
+  }
+
+  return resolved;
+}