@vertaaux/cli 0.4.0 → 0.5.0

package/dist/commands/login.js

@@ -6,12 +6,14 @@
  * - vertaa logout: Clear stored credentials
  * - vertaa whoami: Show current authentication status
  */
-import chalk from "chalk";
+import { colorize, bold, brand, severity as severityPalette, renderError, renderWarning, runSteps, createRenderer } from "@vertaaux/tui";
 import { saveToken, loadToken, clearToken, isTokenExpired, } from "../auth/token-store.js";
 import { startDeviceFlow } from "../auth/device-flow.js";
 import { validateCIToken, getCIToken, getTokenInfo } from "../auth/ci-token.js";
 import { isInteractive } from "../interactive/prompts.js";
 import { ExitCode } from "../utils/exit-codes.js";
+import { writeOutput } from "../output/envelope.js";
+import { strings } from "../ui/strings.js";
 /**
  * OAuth client ID for the CLI.
  * This is a public client ID, not a secret.
@@ -40,147 +42,285 @@ function getApiBase() {
 /**
  * Handle the login command.
  */
-async function handleLogin(options) {
+export async function handleLogin(options) {
     // Check for SSO placeholder
     if (options.sso) {
-        console.error("SSO login is not yet implemented.");
-        console.error("Use --token for CI/API token authentication or omit flags for device code flow.");
+        process.stderr.write(renderWarning({
+            message: strings.login.ssoNotImplemented,
+            suggestion: "vertaa login",
+        }) + "\n");
         process.exit(ExitCode.ERROR);
     }
+    const renderer = createRenderer("auto");
+    const baseState = {
+        phase: "login",
+        phaseIndex: 1,
+        phaseTotal: 1,
+        url: "",
+        mode: "login",
+        progress: {},
+        totals: {},
+        issueCount: 0,
+        scorePreview: null,
+        verbose: false,
+        elapsed: 0,
+    };
+    const startTime = Date.now();
     // Option 1: Direct token authentication (CI mode)
     if (options.token) {
-        console.error("Validating token...");
-        const apiBase = options.base || getApiBase();
-        const isValid = await validateCIToken(options.token, apiBase);
-        if (!isValid) {
-            console.error(chalk.red("Error: Invalid token. Please check your token and try again."));
-            process.exit(ExitCode.ERROR);
-        }
-        // Save the CI token
-        const tokenData = {
-            accessToken: options.token,
-            type: "ci",
-            savedAt: new Date().toISOString(),
-        };
-        await saveToken(tokenData);
-        console.error(chalk.green("Authentication successful!"));
-        console.error("Token saved to ~/.vertaaux/credentials.json");
+        const steps = [
+            {
+                id: "validate",
+                actionText: strings.login.validatingToken,
+                summaryText: "Token validated",
+                run: async () => {
+                    const apiBase = options.base || getApiBase();
+                    const isValid = await validateCIToken(options.token, apiBase);
+                    if (!isValid) {
+                        throw new Error(strings.login.errors.invalidToken);
+                    }
+                },
+            },
+            {
+                id: "save",
+                actionText: "Saving credentials...",
+                summaryText: "Credentials saved",
+                run: async () => {
+                    const tokenData = {
+                        accessToken: options.token,
+                        type: "ci",
+                        savedAt: new Date().toISOString(),
+                    };
+                    await saveToken(tokenData);
+                },
+            },
+        ];
+        await runSteps(steps, {
+            failFast: true,
+            onStateChange: (stepStates) => {
+                renderer.update({ ...baseState, stepStates, elapsed: Date.now() - startTime });
+            },
+        });
+        renderer.finish({ url: "", mode: "login", overallScore: 0, scores: {}, issueCount: 0, passed: true, elapsed: Date.now() - startTime });
+        writeOutput(colorize(strings.login.success, brand.lime) + "\n" + strings.login.tokenSaved);
         return;
     }
     // Option 2: Check for CI token in environment
     const envToken = getCIToken();
     if (envToken && !isInteractive()) {
-        console.error("Found token in environment, validating...");
-        const apiBase = options.base || getApiBase();
-        const isValid = await validateCIToken(envToken, apiBase);
-        if (!isValid) {
-            console.error(chalk.red("Error: Environment token is invalid."));
-            process.exit(ExitCode.ERROR);
-        }
-        const tokenData = {
-            accessToken: envToken,
-            type: "ci",
-            savedAt: new Date().toISOString(),
-        };
-        await saveToken(tokenData);
-        console.error(chalk.green("Authentication successful!"));
-        console.error("Token from environment saved to ~/.vertaaux/credentials.json");
+        const steps = [
+            {
+                id: "validate",
+                actionText: strings.login.envTokenFound,
+                summaryText: "Environment token validated",
+                run: async () => {
+                    const apiBase = options.base || getApiBase();
+                    const isValid = await validateCIToken(envToken, apiBase);
+                    if (!isValid) {
+                        throw new Error(strings.login.errors.envTokenInvalid);
+                    }
+                },
+            },
+            {
+                id: "save",
+                actionText: "Saving credentials...",
+                summaryText: "Credentials saved",
+                run: async () => {
+                    const tokenData = {
+                        accessToken: envToken,
+                        type: "ci",
+                        savedAt: new Date().toISOString(),
+                    };
+                    await saveToken(tokenData);
+                },
+            },
+        ];
+        await runSteps(steps, {
+            failFast: true,
+            onStateChange: (stepStates) => {
+                renderer.update({ ...baseState, stepStates, elapsed: Date.now() - startTime });
+            },
+        });
+        renderer.finish({ url: "", mode: "login", overallScore: 0, scores: {}, issueCount: 0, passed: true, elapsed: Date.now() - startTime });
+        writeOutput(colorize(strings.login.success, brand.lime) + "\n" + strings.login.envTokenSaved);
         return;
     }
     // Option 3: Interactive device code flow (requires TTY)
     if (!isInteractive()) {
-        console.error(chalk.red("Error: Interactive login requires a terminal."));
-        console.error("Use --token <token> for non-interactive authentication,");
-        console.error("or set VERTAAUX_TOKEN environment variable.");
+        process.stderr.write(renderError({
+            message: strings.login.errors.requiresTerminal,
+            suggestion: "vertaa login --token <api-key>",
+            exitCode: ExitCode.ERROR,
+        }) + "\n");
         process.exit(ExitCode.ERROR);
     }
     // Start device code flow
-    console.error("Starting authentication...");
-    try {
-        const authBase = options.base ? options.base.replace("/v1", "") : getAuthBase();
-        const result = await startDeviceFlow(CLI_CLIENT_ID, authBase);
-        // Calculate expiration time
-        const expiresAt = new Date(Date.now() + result.expiresIn * 1000).toISOString();
-        // Save token
-        const tokenData = {
-            accessToken: result.accessToken,
-            refreshToken: result.refreshToken,
-            expiresAt,
-            type: "device",
-            savedAt: new Date().toISOString(),
-        };
-        await saveToken(tokenData);
-        console.error(chalk.green("\nAuthentication successful!"));
-        console.error("Token saved to ~/.vertaaux/credentials.json");
-    }
-    catch (error) {
-        console.error(chalk.red("\nError:"), error instanceof Error ? error.message : String(error));
-        process.exit(ExitCode.ERROR);
-    }
+    const steps = [
+        {
+            id: "auth",
+            actionText: strings.login.startingAuth,
+            summaryText: "Authenticated",
+            run: async () => {
+                const authBase = options.base ? options.base.replace("/v1", "") : getAuthBase();
+                const result = await startDeviceFlow(CLI_CLIENT_ID, authBase);
+                // Calculate expiration time
+                const expiresAt = new Date(Date.now() + result.expiresIn * 1000).toISOString();
+                // Save token
+                const tokenData = {
+                    accessToken: result.accessToken,
+                    refreshToken: result.refreshToken,
+                    expiresAt,
+                    type: "device",
+                    savedAt: new Date().toISOString(),
+                };
+                await saveToken(tokenData);
+            },
+        },
+    ];
+    await runSteps(steps, {
+        failFast: true,
+        onStateChange: (stepStates) => {
+            renderer.update({ ...baseState, stepStates, elapsed: Date.now() - startTime });
+        },
+    });
+    renderer.finish({ url: "", mode: "login", overallScore: 0, scores: {}, issueCount: 0, passed: true, elapsed: Date.now() - startTime });
+    writeOutput(colorize(strings.login.success, brand.lime) + "\n" + strings.login.tokenSaved);
 }
 /**
  * Handle the logout command.
  */
-async function handleLogout() {
+export async function handleLogout() {
+    const renderer = createRenderer("auto");
+    const baseState = {
+        phase: "logout",
+        phaseIndex: 1,
+        phaseTotal: 1,
+        url: "",
+        mode: "logout",
+        progress: {},
+        totals: {},
+        issueCount: 0,
+        scorePreview: null,
+        verbose: false,
+        elapsed: 0,
+    };
+    const startTime = Date.now();
+    const steps = [
+        {
+            id: "clear",
+            actionText: "Clearing credentials...",
+            summaryText: "Credentials cleared",
+            run: async () => {
+                const existingToken = await loadToken();
+                if (!existingToken) {
+                    // Not an error — just nothing to clear
+                    return;
+                }
+                await clearToken();
+            },
+        },
+    ];
+    // Check if already logged out before step-list
     const existingToken = await loadToken();
     if (!existingToken) {
-        console.error("No stored credentials found.");
+        writeOutput(strings.login.noStoredCredentials);
         return;
     }
-    await clearToken();
-    console.error(chalk.green("Logged out successfully."));
-    console.error("Credentials removed from ~/.vertaaux/credentials.json");
+    await runSteps(steps, {
+        failFast: true,
+        onStateChange: (stepStates) => {
+            renderer.update({ ...baseState, stepStates, elapsed: Date.now() - startTime });
+        },
+    });
+    renderer.finish({ url: "", mode: "logout", overallScore: 0, scores: {}, issueCount: 0, passed: true, elapsed: Date.now() - startTime });
+    writeOutput(colorize(strings.login.logoutSuccess, brand.lime) + "\n" + strings.login.credentialsRemoved);
 }
 /**
  * Handle the whoami command.
  */
-async function handleWhoami(options) {
+export async function handleWhoami(options) {
+    const renderer = createRenderer("auto");
+    const baseState = {
+        phase: "whoami",
+        phaseIndex: 1,
+        phaseTotal: 1,
+        url: "",
+        mode: "whoami",
+        progress: {},
+        totals: {},
+        issueCount: 0,
+        scorePreview: null,
+        verbose: false,
+        elapsed: 0,
+    };
+    const startTime = Date.now();
     // Check for token in credentials file
     const storedToken = await loadToken();
     // Check for token in environment
     const envToken = getCIToken();
     if (!storedToken && !envToken) {
-        console.error(chalk.yellow("Not authenticated."));
-        console.error("\nRun `vertaa login` to authenticate.");
+        writeOutput(colorize(strings.login.notAuthenticated, severityPalette.warning) +
+            "\n" + strings.login.runLoginHint);
         return;
     }
     // Use stored token or env token
     const token = storedToken?.accessToken || envToken;
-    const tokenSource = storedToken ? "stored" : "environment";
     if (!token) {
-        console.error(chalk.yellow("Not authenticated."));
-        console.error("\nRun `vertaa login` to authenticate.");
+        writeOutput(colorize(strings.login.notAuthenticated, severityPalette.warning) +
+            "\n" + strings.login.runLoginHint);
         return;
     }
     // Check expiration for stored device tokens
     if (storedToken && storedToken.type === "device" && isTokenExpired(storedToken)) {
-        console.error(chalk.yellow("Token expired."));
-        console.error("\nRun `vertaa login` to re-authenticate.");
+        writeOutput(colorize(strings.login.tokenExpired, severityPalette.warning) +
+            "\n" + strings.login.runLoginRefreshHint);
         return;
     }
-    // Get token info from API
-    const apiBase = options.base || getApiBase();
-    const info = await getTokenInfo(token, apiBase);
-    if (!info || !info.valid) {
-        console.error(chalk.yellow("Token invalid or expired."));
-        console.error("\nRun `vertaa login` to re-authenticate.");
+    let whoamiResult = null;
+    const steps = [
+        {
+            id: "check",
+            actionText: "Checking identity...",
+            summaryText: "Identity verified",
+            run: async () => {
+                const apiBase = options.base || getApiBase();
+                whoamiResult = await getTokenInfo(token, apiBase);
+                if (!whoamiResult || !whoamiResult.valid) {
+                    throw new Error(strings.login.tokenInvalidOrExpired);
+                }
+            },
+        },
+    ];
+    await runSteps(steps, {
+        failFast: true,
+        onStateChange: (stepStates) => {
+            renderer.update({ ...baseState, stepStates, elapsed: Date.now() - startTime });
+        },
+    });
+    renderer.finish({ url: "", mode: "whoami", overallScore: 0, scores: {}, issueCount: 0, passed: true, elapsed: Date.now() - startTime });
+    if (!whoamiResult || !whoamiResult.valid) {
+        writeOutput(colorize(strings.login.tokenInvalidOrExpired, severityPalette.warning) +
+            "\n" + strings.login.runLoginRefreshHint);
         return;
     }
-    // Display auth status
+    const info = whoamiResult;
+    // Display auth status via writeOutput
     const displayName = info.name || info.email || info.user_id || "Unknown";
-    console.error(chalk.green(`Authenticated as ${chalk.bold(displayName)}`));
-    console.error("");
+    const lines = [
+        colorize(strings.login.successWithName(bold(displayName)), brand.lime),
+        "",
+    ];
     if (info.email) {
-        console.error(`  Email: ${info.email}`);
+        lines.push(`  Email: ${info.email}`);
     }
     if (info.tier) {
-        console.error(`  Plan: ${info.tier.charAt(0).toUpperCase() + info.tier.slice(1)}`);
+        lines.push(`  Plan: ${info.tier.charAt(0).toUpperCase() + info.tier.slice(1)}`);
     }
     if (info.organization) {
-        console.error(`  Organization: ${info.organization}`);
+        lines.push(`  Organization: ${info.organization}`);
     }
     if (info.scopes && info.scopes.length > 0) {
-        console.error(`  Scopes: ${info.scopes.join(", ")}`);
+        lines.push(`  Scopes: ${info.scopes.join(", ")}`);
     }
     if (storedToken?.expiresAt) {
         const expiresAt = new Date(storedToken.expiresAt);
@@ -188,9 +328,10 @@ async function handleWhoami(options) {
         const diffMs = expiresAt.getTime() - now.getTime();
         const diffHours = Math.round(diffMs / (1000 * 60 * 60));
         if (diffHours > 0) {
-            console.error(`  Expires: in ${diffHours} hour${diffHours === 1 ? "" : "s"}`);
+            lines.push(`  Expires: in ${diffHours} hour${diffHours === 1 ? "" : "s"}`);
         }
     }
+    writeOutput(lines.join("\n"));
 }
 /**
  * Register authentication commands with the Commander program.
@@ -199,19 +340,39 @@ export function registerLoginCommand(program) {
     // Login command
     program
         .command("login")
-        .description("Authenticate with VertaaUX")
-        .option("--token <token>", "Use API token directly (for CI/non-interactive use)")
+        .description("Authenticate with VertaaUX (stores SDK credentials in ~/.vertaaux/credentials.json)")
+        .option("--token <token>", "Use API token directly (stored as SDK default credentials)")
         .option("--sso", "Use SSO login (not yet implemented)")
         .option("-b, --base <url>", "API base URL")
         .action(async (options) => {
-        await handleLogin(options);
+        try {
+            await handleLogin(options);
+        }
+        catch (error) {
+            process.stderr.write(renderError({
+                message: error instanceof Error ? error.message : String(error),
+                suggestion: "vertaa login",
+                exitCode: ExitCode.ERROR,
+            }) + "\n");
+            process.exit(ExitCode.ERROR);
+        }
     });
     // Logout command
     program
         .command("logout")
         .description("Clear stored credentials")
         .action(async () => {
-        await handleLogout();
+        try {
+            await handleLogout();
+        }
+        catch (error) {
+            process.stderr.write(renderError({
+                message: error instanceof Error ? error.message : String(error),
+                suggestion: "vertaa login",
+                exitCode: ExitCode.ERROR,
+            }) + "\n");
+            process.exit(ExitCode.ERROR);
+        }
     });
     // Whoami command
     program
@@ -219,6 +380,16 @@ export function registerLoginCommand(program) {
         .description("Show current authentication status")
         .option("-b, --base <url>", "API base URL")
         .action(async (options) => {
-        await handleWhoami(options);
+        try {
+            await handleWhoami(options);
+        }
+        catch (error) {
+            process.stderr.write(renderError({
+                message: error instanceof Error ? error.message : String(error),
+                suggestion: "vertaa whoami",
+                exitCode: ExitCode.ERROR,
+            }) + "\n");
+            process.exit(ExitCode.ERROR);
+        }
     });
 }

package/dist/commands/patch-review.d.ts

@@ -10,5 +10,16 @@
  *   cat fix.patch | vertaa patch-review --findings findings.json
  */
 import { Command } from "commander";
+export interface PatchReviewCommandOptions {
+    job?: string;
+    findings?: string;
+    diffFile?: string;
+    format?: string;
+    base?: string;
+    machine?: boolean;
+    dryRun?: boolean;
+    apiKey?: string;
+}
+export declare function handlePatchReview(opts: PatchReviewCommandOptions): Promise<void>;
 export declare function registerPatchReviewCommand(program: Command): void;
 //# sourceMappingURL=patch-review.d.ts.map

package/dist/commands/patch-review.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"patch-review.d.ts","sourceRoot":"","sources":["../../src/commands/patch-review.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAmIpC,wBAAgB,0BAA0B,CAAC,OAAO,EAAE,OAAO,GAAG,IAAI,CA2HjE"}
+{"version":3,"file":"patch-review.d.ts","sourceRoot":"","sources":["../../src/commands/patch-review.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAmIpC,MAAM,WAAW,yBAAyB;IACxC,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,wBAAsB,iBAAiB,CAAC,IAAI,EAAE,yBAAyB,GAAG,OAAO,CAAC,IAAI,CAAC,CAkJtF;AAMD,wBAAgB,0BAA0B,CAAC,OAAO,EAAE,OAAO,GAAG,IAAI,CA6CjE"}