@vertaaux/cli 0.4.0 → 0.5.1

package/dist/commands/comment.js

@@ -7,13 +7,16 @@
  */
 import fs from "fs";
 import path from "path";
+import { renderError, runSteps, createRenderer } from "@vertaaux/tui";
 import { loadBaseline } from "../baseline/manager.js";
 import { ExitCode } from "../utils/exit-codes.js";
 import { formatMarkdownComment, categorizeIssuesForComment, } from "../output/markdown.js";
-import { createEnvelope, writeJsonOutput, writeOutput } from "../output/envelope.js";
+import { createEnvelope, writeDataOutput, writeJsonOutput, writeOutput } from "../output/envelope.js";
 import { resolveCommandFormat } from "../output/formats.js";
 import { postOrUpdateGitHubComment, parseRepository, extractPRNumber, } from "../ci/github-api.js";
 import { postOrUpdateGitLabNote, getGitLabConfig, extractMRIid, } from "../ci/gitlab-api.js";
+import { strings } from "../ui/strings.js";
+import { unwrapJsonEnvelope } from "../utils/stdin.js";
 /**
  * Default path for latest audit results.
  */
@@ -126,18 +129,37 @@ async function postComment(context, markdown) {
 /**
  * Handle the comment command.
  */
-async function handleComment(options) {
+export async function handleComment(options) {
+    const renderer = createRenderer("auto");
+    const baseState = {
+        phase: "comment",
+        phaseIndex: 1,
+        phaseTotal: 1,
+        url: "",
+        mode: "comment",
+        progress: {},
+        totals: {},
+        issueCount: 0,
+        scorePreview: null,
+        verbose: false,
+        elapsed: 0,
+    };
+    const startTime = Date.now();
     let auditResult;
-    // Load audit results
+    // Load audit results (pre-step, may need async iteration for stdin)
     if (options.input) {
         // From file
         const inputPath = path.resolve(process.cwd(), options.input);
         if (!fs.existsSync(inputPath)) {
-            console.error(`Error: Input file not found: ${options.input}`);
+            process.stderr.write(renderError({
+                message: `Input file not found: ${options.input}`,
+                suggestion: "vertaa audit <url>",
+                exitCode: ExitCode.ERROR,
+            }) + "\n");
             process.exit(ExitCode.ERROR);
         }
         const content = fs.readFileSync(inputPath, "utf-8");
-        auditResult = JSON.parse(content);
+        auditResult = unwrapJsonEnvelope(JSON.parse(content));
     }
     else if (!process.stdin.isTTY) {
         // From stdin
@@ -146,17 +168,21 @@ async function handleComment(options) {
             chunks.push(chunk);
         }
         const content = Buffer.concat(chunks).toString("utf-8");
-        auditResult = JSON.parse(content);
+        auditResult = unwrapJsonEnvelope(JSON.parse(content));
     }
     else {
         // Try latest audit file
         const latestPath = path.resolve(process.cwd(), LATEST_AUDIT_PATH);
         if (fs.existsSync(latestPath)) {
             const content = fs.readFileSync(latestPath, "utf-8");
-            auditResult = JSON.parse(content);
+            auditResult = unwrapJsonEnvelope(JSON.parse(content));
         }
         else {
-            console.error("Error: No audit results found. Provide --input or pipe results.");
+            process.stderr.write(renderError({
+                message: strings.comment.errors.noInput,
+                suggestion: "vertaa audit <url>",
+                exitCode: ExitCode.ERROR,
+            }) + "\n");
             process.exit(ExitCode.ERROR);
         }
     }
@@ -169,58 +195,85 @@ async function handleComment(options) {
         // Try default baseline
         baseline = await loadBaseline();
     }
-    // Categorize issues
-    const issues = normalizeIssues(auditResult.issues);
-    const { newIssues, fixedIssues, existingIssues } = categorizeIssuesForComment(issues, baseline);
-    // Build comment data
-    const commentData = {
-        auditId: auditResult.job_id,
-        url: auditResult.url,
-        newIssues,
-        fixedIssues,
-        existingIssues,
-        scores: auditResult.scores,
-    };
-    // Generate output
+    // State to capture within steps
+    let outputContent = "";
+    let jsonData = {};
     const isJson = options.format === "json";
-    const jsonData = {
-        auditId: auditResult.job_id,
-        url: auditResult.url,
-        newIssues: newIssues.length,
-        existingIssues: existingIssues.length,
-        fixedIssues: fixedIssues.length,
-        issues: {
-            new: newIssues,
-            existing: existingIssues,
-            fixed: fixedIssues,
+    const steps = [
+        {
+            id: "generate",
+            actionText: "Generating review comment...",
+            summaryText: "Comment generated",
+            run: async () => {
+                // Categorize issues
+                const issues = normalizeIssues(auditResult.issues);
+                const { newIssues, fixedIssues, existingIssues } = categorizeIssuesForComment(issues, baseline);
+                // Build comment data
+                const commentData = {
+                    auditId: auditResult.job_id,
+                    url: auditResult.url,
+                    newIssues,
+                    fixedIssues,
+                    existingIssues,
+                    scores: auditResult.scores,
+                };
+                jsonData = {
+                    auditId: auditResult.job_id,
+                    url: auditResult.url,
+                    newIssues: newIssues.length,
+                    existingIssues: existingIssues.length,
+                    fixedIssues: fixedIssues.length,
+                    issues: {
+                        new: newIssues,
+                        existing: existingIssues,
+                        fixed: fixedIssues,
+                    },
+                };
+                if (isJson) {
+                    outputContent = JSON.stringify(jsonData, null, 2);
+                }
+                else {
+                    outputContent = formatMarkdownComment(commentData, {
+                        groupBy: options.groupBy || "severity",
+                        collapse: options.collapse ?? newIssues.length > 3,
+                        collapseThreshold: 3,
+                        includeEvidence: true,
+                        includeFixes: true,
+                        baseUrl: "https://vertaaux.ai",
+                    });
+                }
+            },
         },
-    };
-    let output;
-    if (isJson) {
-        // JSON format - will be wrapped in envelope at output time
-        output = JSON.stringify(jsonData, null, 2);
+    ];
+    // Check if posting and we should skip
+    if (options.post && options.noPostIfClean) {
+        // We need to generate first to check, so we still run the step
     }
-    else {
-        // Markdown format (default)
-        output = formatMarkdownComment(commentData, {
-            groupBy: options.groupBy || "severity",
-            collapse: options.collapse ?? newIssues.length > 3,
-            collapseThreshold: 3,
-            includeEvidence: true,
-            includeFixes: true,
-            baseUrl: "https://vertaaux.ai",
-        });
+    const { success, states } = await runSteps(steps, {
+        failFast: true,
+        onStateChange: (stepStates) => {
+            renderer.update({ ...baseState, stepStates, elapsed: Date.now() - startTime });
+        },
+    });
+    renderer.finish({ url: "", mode: "comment", overallScore: 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;
     }
     // Handle posting
     if (options.post) {
+        const newIssuesCount = jsonData.newIssues ?? 0;
         // Check if we should skip posting
-        if (options.noPostIfClean && newIssues.length === 0) {
-            console.error("No new issues found. Skipping comment post (--no-post-if-clean).");
+        if (options.noPostIfClean && newIssuesCount === 0) {
             // Still write output if requested
             if (options.output) {
                 const outputPath = path.resolve(process.cwd(), options.output);
-                fs.writeFileSync(outputPath, output, "utf-8");
-                console.error(`Comment written to: ${outputPath}`);
+                fs.writeFileSync(outputPath, outputContent, "utf-8");
+                writeOutput(`Comment written to: ${outputPath}`);
             }
             return;
         }
@@ -264,30 +317,38 @@ async function handleComment(options) {
             context = detectCIContext();
         }
         if (!context) {
-            console.error("Error: Cannot post comment - not in a recognized CI environment.");
-            console.error("Provide --github-token with --repo and --pr, or run in GitHub Actions/GitLab CI.");
+            process.stderr.write(renderError({
+                message: "Cannot post comment — not in a recognized CI environment.",
+                context: "Provide --github-token with --repo and --pr, or run in GitHub Actions/GitLab CI.",
+                suggestion: "vertaa comment --github-token <token> --repo owner/repo --pr <number>",
+                exitCode: ExitCode.ERROR,
+            }) + "\n");
             // Output for debugging via stdout
             if (isJson) {
                 writeJsonOutput(jsonData, "comment");
             }
             else {
-                writeOutput(output);
+                writeDataOutput(outputContent);
             }
             process.exit(ExitCode.ERROR);
         }
         try {
-            const result = await postComment(context, output);
-            console.error(`Comment posted: ${result.url}`);
+            const result = await postComment(context, outputContent);
+            writeOutput(`Comment posted: ${result.url}`);
         }
         catch (error) {
-            console.error(`Error posting comment: ${error instanceof Error ? error.message : String(error)}`);
+            process.stderr.write(renderError({
+                message: `Error posting comment: ${error instanceof Error ? error.message : String(error)}`,
+                context: "Generated output written for debugging.",
+                suggestion: "vertaa comment --github-token <token> --repo owner/repo --pr <number>",
+                exitCode: ExitCode.ERROR,
+            }) + "\n");
             // Output for debugging even on failure
-            console.error("\nGenerated output (for debugging):");
             if (isJson) {
                 writeJsonOutput(jsonData, "comment");
             }
             else {
-                writeOutput(output);
+                writeDataOutput(outputContent);
             }
             process.exit(ExitCode.ERROR);
         }
@@ -297,9 +358,9 @@ async function handleComment(options) {
         const outputPath = path.resolve(process.cwd(), options.output);
         const fileContent = isJson
             ? JSON.stringify(createEnvelope(jsonData, "comment"), null, 2)
-            : output;
+            : outputContent;
         fs.writeFileSync(outputPath, fileContent, "utf-8");
-        console.error(`Comment written to: ${outputPath}`);
+        writeOutput(`Comment written to: ${outputPath}`);
     }
     else if (!options.post) {
         // Only print to stdout if not posting (posting already logs URL)
@@ -307,7 +368,7 @@ async function handleComment(options) {
             writeJsonOutput(jsonData, "comment");
         }
         else {
-            writeOutput(output);
+            writeDataOutput(outputContent);
         }
     }
 }
@@ -354,7 +415,11 @@ export function registerCommentCommand(program) {
             });
         }
         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 audit <url>",
+                exitCode: ExitCode.ERROR,
+            }) + "\n");
             process.exit(ExitCode.ERROR);
         }
     });

package/dist/commands/compare.d.ts

@@ -16,5 +16,22 @@
  *   vertaa compare --before baseline.json --after current.json --verbose
  */
 import { Command } from "commander";
+export interface CompareCommandOptions {
+    urlA?: string;
+    urlB?: string;
+    before?: string;
+    after?: string;
+    mode?: string;
+    wait?: boolean;
+    timeout?: string;
+    interval?: string;
+    failOnScore?: string;
+    format?: string;
+    base?: string;
+    verbose?: boolean;
+    machine?: boolean;
+    apiKey?: string;
+}
+export declare function handleCompare(opts: CompareCommandOptions): Promise<void>;
 export declare function registerCompareCommand(program: Command): void;
 //# sourceMappingURL=compare.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"compare.d.ts","sourceRoot":"","sources":["../../src/commands/compare.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAkNpC,wBAAgB,sBAAsB,CAAC,OAAO,EAAE,OAAO,GAAG,IAAI,CAmE7D"}
+{"version":3,"file":"compare.d.ts","sourceRoot":"","sources":["../../src/commands/compare.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AA6MpC,MAAM,WAAW,qBAAqB;IACpC,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,IAAI,CAAC,EAAE,OAAO,CAAC;IACf,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,wBAAsB,aAAa,CAAC,IAAI,EAAE,qBAAqB,GAAG,OAAO,CAAC,IAAI,CAAC,CAmR9E;AAMD,wBAAgB,sBAAsB,CAAC,OAAO,EAAE,OAAO,GAAG,IAAI,CAuE7D"}