@vertaaux/cli 0.4.0 → 0.5.0

package/dist/commands/patch-review.js

@@ -9,15 +9,15 @@
  *   git diff HEAD~1 | vertaa patch-review --job abc123
  *   cat fix.patch | vertaa patch-review --findings findings.json
  */
-import chalk from "chalk";
+import { bold, dim, colorize, boldColor, brand, severity as severityPalette, renderError, createRenderer, runSteps } from "@vertaaux/tui";
 import { ExitCode } from "../utils/exit-codes.js";
-import { resolveApiBase, getApiKey, apiRequest } from "../utils/client.js";
+import { resolveApiBase, getApiKey, apiRequest, createClient } from "../utils/client.js";
 import { resolveConfig } from "../config/loader.js";
 import { writeJsonOutput, writeOutput } from "../output/envelope.js";
 import { resolveCommandFormat } from "../output/formats.js";
-import { createSpinner, succeedSpinner } from "../ui/spinner.js";
 import { readTextInput, readJsonInput } from "../utils/stdin.js";
-import { handleAiCommandError, AI_TIMEOUT_MS } from "../utils/ai-error.js";
+import { AI_TIMEOUT_MS } from "../utils/ai-error.js";
+import { strings } from "../ui/strings.js";
 // ---------------------------------------------------------------------------
 // Helpers
 // ---------------------------------------------------------------------------
@@ -50,46 +50,172 @@ function normalizeIssues(issues) {
 // Formatters
 // ---------------------------------------------------------------------------
 const VERDICT_DISPLAY = {
-    SAFE: chalk.green.bold("SAFE"),
-    UNSAFE: chalk.red.bold("UNSAFE"),
-    NEEDS_REVIEW: chalk.yellow.bold("NEEDS REVIEW"),
+    SAFE: boldColor("SAFE", brand.lime),
+    UNSAFE: boldColor("UNSAFE", severityPalette.error),
+    NEEDS_REVIEW: boldColor("NEEDS REVIEW", severityPalette.warning),
 };
 const CONCERN_SEVERITY = {
-    critical: chalk.red("critical"),
-    warning: chalk.yellow("warning"),
-    info: chalk.cyan("info"),
+    critical: colorize("critical", severityPalette.critical),
+    warning: colorize("warning", severityPalette.warning),
+    info: colorize("info", brand.cyan),
 };
 function formatPatchReviewHuman(data) {
     const lines = [];
-    const verdictDisplay = VERDICT_DISPLAY[data.verdict] || chalk.bold(data.verdict);
+    const verdictDisplay = VERDICT_DISPLAY[data.verdict] || bold(data.verdict);
     lines.push(`Verdict: ${verdictDisplay}  (confidence: ${data.confidence}%)`);
     lines.push("");
     lines.push(data.summary);
     if (data.concerns.length > 0) {
         lines.push("");
-        lines.push(chalk.bold("Concerns:"));
+        lines.push(bold("Concerns:"));
         for (const concern of data.concerns) {
-            const sev = CONCERN_SEVERITY[concern.severity] || chalk.dim(concern.severity);
-            const loc = concern.location ? chalk.dim(` @ ${concern.location}`) : "";
+            const sev = CONCERN_SEVERITY[concern.severity] || dim(concern.severity);
+            const loc = concern.location ? dim(` @ ${concern.location}`) : "";
             lines.push(`  ${sev}${loc}: ${concern.description}`);
         }
     }
     if (data.findings_addressed.length > 0) {
         lines.push("");
-        lines.push(chalk.green(`Findings addressed (${data.findings_addressed.length}):`));
+        lines.push(colorize(`Findings addressed (${data.findings_addressed.length}):`, brand.lime));
         for (const id of data.findings_addressed) {
-            lines.push(`  ${chalk.green("+")} ${id}`);
+            lines.push(`  ${colorize("+", brand.lime)} ${id}`);
         }
     }
     if (data.findings_remaining.length > 0) {
         lines.push("");
-        lines.push(chalk.yellow(`Findings remaining (${data.findings_remaining.length}):`));
+        lines.push(colorize(`Findings remaining (${data.findings_remaining.length}):`, severityPalette.warning));
         for (const id of data.findings_remaining) {
-            lines.push(`  ${chalk.yellow("-")} ${id}`);
+            lines.push(`  ${colorize("-", severityPalette.warning)} ${id}`);
         }
     }
     return lines.join("\n");
 }
+export async function handlePatchReview(opts) {
+    const config = { apiKey: opts.apiKey };
+    const format = resolveCommandFormat("patch-review", opts.format, opts.machine || false);
+    const renderer = createRenderer("auto");
+    const baseState = {
+        phase: "patch-review",
+        phaseIndex: 1,
+        phaseTotal: 2,
+        url: "",
+        mode: "patch-review",
+        progress: {},
+        totals: {},
+        issueCount: 0,
+        scorePreview: null,
+        verbose: false,
+        elapsed: 0,
+    };
+    const startTime = Date.now();
+    let responseData = null;
+    const steps = [
+        {
+            id: "load",
+            actionText: "Loading patch...",
+            summaryText: "Patch loaded",
+            run: async () => {
+                // Read diff content
+                let diffContent;
+                if (opts.diffFile) {
+                    const fs = await import("fs");
+                    const path = await import("path");
+                    const resolved = path.resolve(process.cwd(), opts.diffFile);
+                    if (!fs.existsSync(resolved)) {
+                        throw new Error(`Diff file not found: ${opts.diffFile}`);
+                    }
+                    diffContent = fs.readFileSync(resolved, "utf-8");
+                }
+                else {
+                    const text = await readTextInput();
+                    if (!text) {
+                        throw new Error("No diff provided.\nPipe a diff via stdin or use --diff-file:\n  gh pr diff 123 | vertaa patch-review --job abc123\n  vertaa patch-review --diff-file fix.patch --findings audit.json");
+                    }
+                    diffContent = text;
+                }
+                // Resolve findings
+                let findingsList = null;
+                let jobId = null;
+                if (opts.job) {
+                    const sdkClient = createClient({ base: opts.base, apiKey: config.apiKey });
+                    const result = await sdkClient.audits.retrieve(opts.job);
+                    findingsList = normalizeIssues(result.issues);
+                    jobId = result.job_id || opts.job;
+                }
+                else if (opts.findings) {
+                    const input = await readJsonInput(opts.findings);
+                    if (input) {
+                        const data = input;
+                        const innerData = (data.data && typeof data.data === "object" ? data.data : data);
+                        findingsList = normalizeIssues(innerData.issues);
+                        jobId = innerData.job_id || null;
+                    }
+                }
+                if (opts.dryRun) {
+                    const lines = [];
+                    lines.push(`${colorize("[dry-run]", severityPalette.warning)} ${strings.patchReview.dryRun}`);
+                    lines.push(`  Diff size: ${diffContent.length} characters`);
+                    lines.push(`  Findings: ${findingsList ? findingsList.length : "none"}`);
+                    lines.push(`  Job ID: ${jobId || "none"}`);
+                    writeOutput(lines.join("\n"));
+                    return;
+                }
+                const base = resolveApiBase(opts.base);
+                const apiKey = getApiKey(config.apiKey);
+                const response = await Promise.race([
+                    apiRequest(base, "/cli/ai/patch-review", {
+                        method: "POST",
+                        body: {
+                            diff: diffContent,
+                            findings: findingsList,
+                            job_id: jobId,
+                        },
+                    }, apiKey),
+                    new Promise((_, reject) => setTimeout(() => reject(new Error("LLM request timed out")), AI_TIMEOUT_MS)),
+                ]);
+                responseData = response.data;
+            },
+        },
+        {
+            id: "review",
+            actionText: strings.patchReview.analyze.action,
+            summaryText: responseData ? strings.patchReview.analyze.done(responseData.verdict) : "Review complete",
+            run: async () => {
+                // Review happened in load step via API call
+            },
+        },
+    ];
+    const { success, states } = await runSteps(steps, {
+        onStateChange: (stepStates) => {
+            renderer.update({ ...baseState, stepStates, elapsed: Date.now() - startTime });
+        },
+    });
+    renderer.finish({
+        url: "",
+        mode: "patch-review",
+        overallScore: success ? 100 : 0,
+        scores: {},
+        issueCount: 0,
+        passed: success,
+        elapsed: Date.now() - startTime,
+    });
+    if (!success) {
+        const failed = states.find(s => s.status === "failed");
+        process.stderr.write(renderError({
+            message: failed?.failReason || "Command failed",
+            suggestion: "vertaa doctor",
+        }) + "\n");
+        process.exitCode = ExitCode.ERROR;
+    }
+    if (success && responseData && !opts.dryRun) {
+        if (format === "json") {
+            writeJsonOutput(responseData, "patch-review");
+        }
+        else {
+            writeOutput(formatPatchReviewHuman(responseData));
+        }
+    }
+}
 // ---------------------------------------------------------------------------
 // Command Registration
 // ---------------------------------------------------------------------------
@@ -113,87 +239,23 @@ Examples:
             const config = await resolveConfig(globalOpts.config);
             const machineMode = globalOpts.machine || false;
             const dryRun = globalOpts.dryRun || false;
-            const format = resolveCommandFormat("patch-review", options.format, machineMode);
-            // Read diff content
-            let diffContent;
-            if (options.diffFile) {
-                const fs = await import("fs");
-                const path = await import("path");
-                const resolved = path.resolve(process.cwd(), options.diffFile);
-                if (!fs.existsSync(resolved)) {
-                    console.error(`Error: Diff file not found: ${options.diffFile}`);
-                    process.exit(ExitCode.ERROR);
-                }
-                diffContent = fs.readFileSync(resolved, "utf-8");
-            }
-            else {
-                const text = await readTextInput();
-                if (!text) {
-                    console.error("Error: No diff provided.");
-                    console.error("Pipe a diff via stdin or use --diff-file:");
-                    console.error("  gh pr diff 123 | vertaa patch-review --job abc123");
-                    console.error("  vertaa patch-review --diff-file fix.patch --findings audit.json");
-                    process.exit(ExitCode.ERROR);
-                }
-                diffContent = text;
-            }
-            // Resolve findings
-            let findingsList = null;
-            let jobId = null;
-            if (options.job) {
-                const base = resolveApiBase(globalOpts.base);
-                const apiKey = getApiKey(config.apiKey);
-                const result = await apiRequest(base, `/audit/${options.job}`, { method: "GET" }, apiKey);
-                findingsList = normalizeIssues(result.issues);
-                jobId = result.job_id || options.job;
-            }
-            else if (options.findings) {
-                const input = await readJsonInput(options.findings);
-                if (input) {
-                    const data = input;
-                    const innerData = (data.data && typeof data.data === "object" ? data.data : data);
-                    findingsList = normalizeIssues(innerData.issues);
-                    jobId = innerData.job_id || null;
-                }
-            }
-            if (dryRun) {
-                console.log(chalk.yellow("[dry-run]") + " Would analyze diff against findings");
-                console.log(`  Diff size: ${diffContent.length} characters`);
-                console.log(`  Findings: ${findingsList ? findingsList.length : "none"}`);
-                console.log(`  Job ID: ${jobId || "none"}`);
-                return;
-            }
-            // Auth check
-            const base = resolveApiBase(globalOpts.base);
-            const apiKey = getApiKey(config.apiKey);
-            // Call LLM patch-review API
-            const spinner = createSpinner("Reviewing patch...");
-            try {
-                const response = await Promise.race([
-                    apiRequest(base, "/cli/ai/patch-review", {
-                        method: "POST",
-                        body: {
-                            diff: diffContent,
-                            findings: findingsList,
-                            job_id: jobId,
-                        },
-                    }, apiKey),
-                    new Promise((_, reject) => setTimeout(() => reject(new Error("LLM request timed out")), AI_TIMEOUT_MS)),
-                ]);
-                succeedSpinner(spinner, "Review complete");
-                if (format === "json") {
-                    writeJsonOutput(response.data, "patch-review");
-                }
-                else {
-                    writeOutput(formatPatchReviewHuman(response.data));
-                }
-            }
-            catch (error) {
-                handleAiCommandError(error, "patch-review", spinner);
-            }
+            await handlePatchReview({
+                job: options.job,
+                findings: options.findings,
+                diffFile: options.diffFile,
+                format: options.format,
+                base: globalOpts.base,
+                machine: machineMode,
+                dryRun,
+                apiKey: config.apiKey,
+            });
         }
         catch (error) {
-            console.error("Error:", error instanceof Error ? error.message : String(error));
+            process.stderr.write(renderError({
+                message: error instanceof Error ? error.message : String(error),
+                suggestion: "vertaa doctor",
+                exitCode: ExitCode.ERROR,
+            }) + "\n");
             process.exit(ExitCode.ERROR);
         }
     });

package/dist/commands/policy.d.ts

@@ -6,8 +6,39 @@
  * Implements CICD-17: Policy-as-code support.
  */
 import { Command } from "commander";
+import { type PolicyTemplate } from "../policy/index.js";
+/**
+ * Policy init command options.
+ */
+interface PolicyInitOptions {
+    template?: PolicyTemplate;
+    output?: string;
+    force?: boolean;
+}
+/**
+ * Policy validate command options.
+ */
+interface PolicyValidateOptions {
+    quiet?: boolean;
+}
+/**
+ * Policy show command options.
+ */
+interface PolicyShowOptions {
+    branch?: string;
+    format?: "yaml" | "json";
+    policy?: string;
+}
+/**
+ * Unified handlePolicy entry point for CommandRunnerView.
+ * Routes to the appropriate sub-command handler.
+ */
+export declare function handlePolicy(subCommand: "init" | "validate" | "show" | "schema", options: PolicyInitOptions & PolicyValidateOptions & PolicyShowOptions & {
+    file?: string;
+}): Promise<void>;
 /**
  * Register the policy command with the Commander program.
  */
 export declare function registerPolicyCommand(program: Command): void;
+export {};
 //# sourceMappingURL=policy.d.ts.map

package/dist/commands/policy.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"policy.d.ts","sourceRoot":"","sources":["../../src/commands/policy.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAuWpC;;GAEG;AACH,wBAAgB,qBAAqB,CAAC,OAAO,EAAE,OAAO,GAAG,IAAI,CA8E5D"}
+{"version":3,"file":"policy.d.ts","sourceRoot":"","sources":["../../src/commands/policy.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAKpC,OAAO,EASL,KAAK,cAAc,EAIpB,MAAM,oBAAoB,CAAC;AAM5B;;GAEG;AACH,UAAU,iBAAiB;IACzB,QAAQ,CAAC,EAAE,cAAc,CAAC;IAC1B,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,KAAK,CAAC,EAAE,OAAO,CAAC;CACjB;AAED;;GAEG;AACH,UAAU,qBAAqB;IAC7B,KAAK,CAAC,EAAE,OAAO,CAAC;CACjB;AAED;;GAEG;AACH,UAAU,iBAAiB;IACzB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,MAAM,CAAC,EAAE,MAAM,GAAG,MAAM,CAAC;IACzB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAmZD;;;GAGG;AACH,wBAAsB,YAAY,CAChC,UAAU,EAAE,MAAM,GAAG,UAAU,GAAG,MAAM,GAAG,QAAQ,EACnD,OAAO,EAAE,iBAAiB,GAAG,qBAAqB,GAAG,iBAAiB,GAAG;IAAE,IAAI,CAAC,EAAE,MAAM,CAAA;CAAE,GACzF,OAAO,CAAC,IAAI,CAAC,CAef;AAED;;GAEG;AACH,wBAAgB,qBAAqB,CAAC,OAAO,EAAE,OAAO,GAAG,IAAI,CAqI5D"}