@travisennis/acai 0.0.9 → 0.0.11

package/dist/commands/review/review-panel.js

@@ -0,0 +1,186 @@
+import { join } from "node:path";
+import { initExecutionEnvironment } from "../../execution/index.js";
+import { getTerminalSize } from "../../terminal/control.js";
+import style from "../../terminal/style.js";
+import { Markdown } from "../../tui/components/markdown.js";
+import { Spacer } from "../../tui/components/spacer.js";
+import { Container, SelectList, Text, } from "../../tui/index.js";
+import { formatFileDiffForDisplay, getUntrackedFiles, openFileInEditor, parseGitDiffFiles, } from "./utils.js";
+export async function showReviewPanel(tui, container, inputContainer, editor) {
+    try {
+        const execEnv = await initExecutionEnvironment();
+        const stagedResult = await execEnv.executeCommand("git diff --cached", {
+            cwd: process.cwd(),
+            timeout: 5000,
+            preserveOutputOnError: true,
+            captureStderr: true,
+            throwOnError: false,
+        });
+        const unstagedResult = await execEnv.executeCommand("git diff", {
+            cwd: process.cwd(),
+            timeout: 5000,
+            preserveOutputOnError: true,
+            captureStderr: true,
+            throwOnError: false,
+        });
+        const untrackedResult = await execEnv.executeCommand("git ls-files --others --exclude-standard", {
+            cwd: process.cwd(),
+            timeout: 5000,
+            preserveOutputOnError: true,
+            captureStderr: true,
+            throwOnError: false,
+        });
+        const stagedOutput = stagedResult.exitCode === 0 ? stagedResult.output : "";
+        const unstagedOutput = unstagedResult.exitCode === 0 ? unstagedResult.output : "";
+        const untrackedOutput = untrackedResult.exitCode === 0 ? untrackedResult.output : "";
+        const combinedOutput = stagedOutput +
+            (stagedOutput && unstagedOutput ? "\n" : "") +
+            unstagedOutput;
+        if (!combinedOutput.trim()) {
+            container.addChild(new Spacer(1));
+            container.addChild(new Markdown("No changes detected in the current directory.", {
+                customBgRgb: {
+                    r: 52,
+                    g: 53,
+                    b: 65,
+                },
+                paddingX: 1,
+                paddingY: 1,
+            }));
+            tui.requestRender();
+            return;
+        }
+        const fileChanges = parseGitDiffFiles(combinedOutput);
+        const untrackedFiles = await getUntrackedFiles(untrackedOutput, process.cwd());
+        fileChanges.push(...untrackedFiles);
+        if (fileChanges.length === 0) {
+            container.addChild(new Spacer(1));
+            container.addChild(new Markdown("No file changes could be parsed.", {
+                customBgRgb: {
+                    r: 52,
+                    g: 53,
+                    b: 65,
+                },
+                paddingX: 1,
+                paddingY: 1,
+            }));
+            tui.requestRender();
+            return;
+        }
+        const selectItems = fileChanges.map((file) => ({
+            value: file.fileName,
+            label: file.fileName,
+            description: file.stats,
+        }));
+        const selectList = new SelectList(selectItems, 10);
+        const selectContainer = new Container();
+        const { columns } = getTerminalSize();
+        selectContainer.addChild(new Text(style.blue("─".repeat(columns)), 0, 0));
+        selectContainer.addChild(new Spacer(1));
+        selectContainer.addChild(selectList);
+        selectContainer.addChild(new Spacer(1));
+        selectContainer.addChild(new Text(style.blue("─".repeat(columns)), 0, 0));
+        const originalEditor = editor;
+        inputContainer.clear();
+        inputContainer.addChild(selectContainer);
+        tui.setFocus(selectList);
+        selectList.onSelect = (selectedItem) => {
+            const selectedFile = fileChanges.find((file) => file.fileName === selectedItem.value);
+            if (!selectedFile) {
+                container.addChild(new Spacer(1));
+                container.addChild(new Markdown(style.red("Error: Selected file not found."), {
+                    customBgRgb: {
+                        r: 52,
+                        g: 53,
+                        b: 65,
+                    },
+                    paddingX: 1,
+                    paddingY: 1,
+                }));
+                tui.requestRender();
+                return;
+            }
+            const actionItems = [
+                { value: "diff", label: "View diff", description: "Show changes" },
+                {
+                    value: "edit",
+                    label: "Open in editor",
+                    description: `Open in ${process.env["EDITOR"] || process.env["VISUAL"] || "vi"}`,
+                },
+            ];
+            const actionSelectList = new SelectList(actionItems, 5);
+            const actionContainer = new Container();
+            actionContainer.addChild(new Spacer(1));
+            actionContainer.addChild(new Text(style.blue(`Selected: ${selectedFile.fileName}`), 0, 0));
+            actionContainer.addChild(new Spacer(1));
+            actionContainer.addChild(actionSelectList);
+            actionContainer.addChild(new Spacer(1));
+            actionContainer.addChild(new Text(style.blue("─".repeat(columns)), 0, 0));
+            inputContainer.clear();
+            inputContainer.addChild(actionContainer);
+            tui.setFocus(actionSelectList);
+            tui.requestRender();
+            actionSelectList.onSelect = (actionItem) => {
+                if (actionItem.value === "diff") {
+                    container.addChild(new Spacer(1));
+                    container.addChild(new Markdown(formatFileDiffForDisplay(selectedFile.fileName, selectedFile.diff), {
+                        customBgRgb: {
+                            r: 52,
+                            g: 53,
+                            b: 65,
+                        },
+                        paddingX: 1,
+                        paddingY: 1,
+                    }));
+                }
+                else if (actionItem.value === "edit") {
+                    const filePath = join(process.cwd(), selectedFile.fileName);
+                    const result = openFileInEditor(filePath, tui.getTerminal());
+                    if (!result.success) {
+                        container.addChild(new Spacer(1));
+                        container.addChild(new Markdown(`Failed to open file in editor: ${result.error || "Unknown error"}`, {
+                            customBgRgb: {
+                                r: 52,
+                                g: 53,
+                                b: 65,
+                            },
+                            paddingX: 1,
+                            paddingY: 1,
+                        }));
+                    }
+                }
+                inputContainer.clear();
+                inputContainer.addChild(originalEditor);
+                tui.setFocus(originalEditor);
+                tui.requestRender();
+            };
+            actionSelectList.onCancel = () => {
+                inputContainer.clear();
+                inputContainer.addChild(selectContainer);
+                tui.setFocus(selectList);
+                tui.requestRender();
+            };
+        };
+        selectList.onCancel = () => {
+            inputContainer.clear();
+            inputContainer.addChild(originalEditor);
+            tui.setFocus(originalEditor);
+            tui.requestRender();
+        };
+        tui.requestRender();
+    }
+    catch (error) {
+        console.error("Error executing git diff:", error);
+        container.addChild(new Spacer(1));
+        container.addChild(new Markdown("Failed to retrieve git changes. Ensure git is installed and initialized.", {
+            customBgRgb: {
+                r: 52,
+                g: 53,
+                b: 65,
+            },
+            paddingX: 1,
+            paddingY: 1,
+        }));
+        tui.requestRender();
+    }
+}

package/dist/commands/review/utils.d.ts

@@ -1,4 +1,18 @@
-import type { FileChange } from "./types.ts";
+import type { Terminal } from "../../tui/terminal.ts";
+interface FileChange {
+    fileName: string;
+    diff: string;
+    stats: string;
+}
 export declare function parseGitDiffFiles(diffOutput: string): FileChange[];
 export declare function formatFileDiffForDisplay(fileName: string, diff: string): string;
+export declare function getUntrackedFiles(untrackedOutput: string, cwd: string): Promise<FileChange[]>;
+/**
+ * Open a file in the user's preferred editor ($EDITOR or $VISUAL)
+ */
+export declare function openFileInEditor(filePath: string, terminal: Terminal): {
+    success: boolean;
+    error?: string;
+};
+export {};
 //# sourceMappingURL=utils.d.ts.map

package/dist/commands/review/utils.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../../../source/commands/review/utils.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,YAAY,CAAC;AAE7C,wBAAgB,iBAAiB,CAAC,UAAU,EAAE,MAAM,GAAG,UAAU,EAAE,CAuDlE;AAED,wBAAgB,wBAAwB,CACtC,QAAQ,EAAE,MAAM,EAChB,IAAI,EAAE,MAAM,GACX,MAAM,CAgCR"}
+{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../../../source/commands/review/utils.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,uBAAuB,CAAC;AAEtD,UAAU,UAAU;IAClB,QAAQ,EAAE,MAAM,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,MAAM,CAAC;CACf;AA6CD,wBAAgB,iBAAiB,CAAC,UAAU,EAAE,MAAM,GAAG,UAAU,EAAE,CAwBlE;AAgBD,wBAAgB,wBAAwB,CACtC,QAAQ,EAAE,MAAM,EAChB,IAAI,EAAE,MAAM,GACX,MAAM,CAaR;AAED,wBAAsB,iBAAiB,CACrC,eAAe,EAAE,MAAM,EACvB,GAAG,EAAE,MAAM,GACV,OAAO,CAAC,UAAU,EAAE,CAAC,CAsCvB;AAED;;GAEG;AACH,wBAAgB,gBAAgB,CAC9B,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,QAAQ,GACjB;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,CAyBtC"}

package/dist/commands/review/utils.js

@@ -1,87 +1,146 @@
+import { spawnSync } from "node:child_process";
+import fs from "node:fs";
+import { join } from "node:path";
+function isDiffContentLine(line) {
+    return ((line.startsWith("+") && !line.startsWith("+++")) ||
+        (line.startsWith("-") && !line.startsWith("---")) ||
+        line.startsWith(" ") ||
+        line.startsWith("index") ||
+        line.startsWith("old mode") ||
+        line.startsWith("new mode") ||
+        line.startsWith("deleted file") ||
+        line.startsWith("new file"));
+}
+function parseHunkStats(line, isNewFile) {
+    const statsMatch = line.match(/@@ -(\d+),(\d+) \+(\d+),(\d+) @@/);
+    if (!statsMatch) {
+        return "Additions: 1, Deletions: 0";
+    }
+    const deletions = isNewFile ? 0 : Number.parseInt(statsMatch[2], 10);
+    const additions = Number.parseInt(statsMatch[4], 10);
+    return `Additions: ${additions}, Deletions: ${deletions}`;
+}
+function startNewFile(state, line) {
+    if (state.currentFile) {
+        state.fileChanges.push(state.currentFile);
+    }
+    const fileMatch = line.match(/diff --git a\/(.*) b\/(.*)/);
+    if (!fileMatch)
+        return;
+    state.currentFile = { fileName: fileMatch[1], diff: "", stats: "" };
+    state.inDiff = true;
+    state.isNewFile =
+        fileMatch[1] === "/dev/null" || fileMatch[2] === "/dev/null";
+}
 export function parseGitDiffFiles(diffOutput) {
     const lines = diffOutput.split("\n");
-    const fileChanges = [];
-    let currentFile = null;
-    let inDiff = false;
-    let isNewFile = false;
+    const state = {
+        fileChanges: [],
+        currentFile: null,
+        inDiff: false,
+        isNewFile: false,
+    };
     for (const line of lines) {
         if (line.startsWith("diff --git")) {
-            if (currentFile) {
-                fileChanges.push(currentFile);
-            }
-            const fileMatch = line.match(/diff --git a\/(.*) b\/(.*)/);
-            if (fileMatch) {
-                currentFile = {
-                    fileName: fileMatch[1],
-                    diff: "",
-                    stats: "",
-                };
-                inDiff = true;
-                isNewFile =
-                    fileMatch[1] === "/dev/null" || fileMatch[2] === "/dev/null";
-            }
+            startNewFile(state, line);
         }
-        else if (line.startsWith("@@")) {
-            const statsMatch = line.match(/@@ -(\d+),(\d+) \+(\d+),(\d+) @@/);
-            if (statsMatch && currentFile) {
-                const deletions = Number.parseInt(statsMatch[2], 10);
-                const additions = Number.parseInt(statsMatch[4], 10);
-                const actualDeletions = isNewFile ? 0 : deletions;
-                currentFile.stats = `Additions: ${additions}, Deletions: ${actualDeletions}`;
-            }
-            else if (currentFile) {
-                currentFile.stats = "Additions: 1, Deletions: 0";
-            }
+        else if (line.startsWith("@@") && state.currentFile) {
+            state.currentFile.stats = parseHunkStats(line, state.isNewFile);
         }
-        else if (inDiff && currentFile) {
-            if ((line.startsWith("+") && !line.startsWith("+++")) ||
-                (line.startsWith("-") && !line.startsWith("---")) ||
-                line.startsWith(" ") ||
-                line.startsWith("index") ||
-                line.startsWith("old mode") ||
-                line.startsWith("new mode") ||
-                line.startsWith("deleted file") ||
-                line.startsWith("new file")) {
-                currentFile.diff += `${line}\n`;
-            }
+        else if (state.inDiff && state.currentFile && isDiffContentLine(line)) {
+            state.currentFile.diff += `${line}\n`;
         }
     }
-    if (currentFile) {
-        fileChanges.push(currentFile);
+    if (state.currentFile) {
+        state.fileChanges.push(state.currentFile);
     }
-    return fileChanges;
+    return state.fileChanges;
+}
+function formatDiffLine(line) {
+    if (line.trim() === "")
+        return null;
+    if (line.startsWith("+") && !line.startsWith("+++")) {
+        return `+${line.substring(1)}`;
+    }
+    if (line.startsWith("-") && !line.startsWith("---")) {
+        return `-${line.substring(1)}`;
+    }
+    if (line.startsWith(" ")) {
+        return ` ${line.substring(1)}`;
+    }
+    return line;
 }
 export function formatFileDiffForDisplay(fileName, diff) {
-    // Note: Cannot use style here as it's a UI concern - this is a display utility
     const lines = diff.split("\n");
-    const formattedLines = [];
-    formattedLines.push(`### ${fileName}`);
-    formattedLines.push("");
+    const formattedLines = [`### ${fileName}`, "", "```diff"];
     for (const line of lines) {
-        if (line.startsWith("+") && !line.startsWith("+++")) {
-            formattedLines.push(`+${line.substring(1)}`);
-        }
-        else if (line.startsWith("-") && !line.startsWith("---")) {
-            formattedLines.push(`-${line.substring(1)}`);
+        const formatted = formatDiffLine(line);
+        if (formatted !== null) {
+            formattedLines.push(formatted);
         }
-        else if (line.startsWith(" ")) {
-            formattedLines.push(` ${line.substring(1)}`);
-        }
-        else if (line.startsWith("@@")) {
-            formattedLines.push(line);
-        }
-        else if (line.startsWith("index") ||
-            line.startsWith("old mode") ||
-            line.startsWith("new mode") ||
-            line.startsWith("deleted file") ||
-            line.startsWith("new file")) {
-            formattedLines.push(line);
+    }
+    formattedLines.push("```");
+    return formattedLines.join("\n");
+}
+export async function getUntrackedFiles(untrackedOutput, cwd) {
+    const fileChanges = [];
+    const untrackedFiles = untrackedOutput
+        .trim()
+        .split("\n")
+        .filter((file) => file.length > 0);
+    for (const fileName of untrackedFiles) {
+        try {
+            const filePath = join(cwd, fileName);
+            const fileContent = await fs.promises.readFile(filePath, "utf-8");
+            const lines = fileContent.split("\n").length;
+            // Create a diff-like representation for the untracked file
+            const diff = `diff --git a/${fileName} b/${fileName}
+new file mode 100644
+index 0000000..0000000
+--- /dev/null
++++ b/${fileName}
+@@ -0,0 +1,${lines} @@
+${fileContent
+                .split("\n")
+                .map((line) => `+${line}`)
+                .join("\n")}
+`;
+            fileChanges.push({
+                fileName,
+                diff,
+                stats: `Additions: ${lines}, Deletions: 0`,
+            });
         }
-        else if (line.trim() === "") {
+        catch (error) {
+            // Skip files that can't be read (e.g., directories, binary files)
+            console.error(`Failed to read untracked file: ${fileName}`, error);
         }
-        else {
-            formattedLines.push(line);
+    }
+    return fileChanges;
+}
+/**
+ * Open a file in the user's preferred editor ($EDITOR or $VISUAL)
+ */
+export function openFileInEditor(filePath, terminal) {
+    const editor = process.env["EDITOR"] || process.env["VISUAL"] || "vi";
+    terminal.enterExternalMode();
+    try {
+        const result = spawnSync(editor, [filePath], {
+            stdio: "inherit",
+        });
+        if (result.error) {
+            return { success: false, error: result.error.message };
         }
+        // Note: Editors often exit with non-zero codes (e.g., :cq in vim, or when
+        // quitting without saving). We only treat spawn errors as failures, not
+        // exit codes, since the user may intentionally exit with an error code.
+        return { success: true };
+    }
+    catch (error) {
+        const errorMessage = error instanceof Error ? error.message : "Unknown error";
+        return { success: false, error: errorMessage };
+    }
+    finally {
+        terminal.exitExternalMode();
     }
-    return formattedLines.join("\n");
 }

package/dist/commands/session/index.d.ts

@@ -1,3 +1,3 @@
 import type { CommandOptions, ReplCommand } from "../types.ts";
-export declare function sessionCommand({ config, tokenCounter, modelManager, sessionManager: messageHistory, workspace, tokenTracker, }: CommandOptions): ReplCommand;
+export declare function sessionCommand({ config, tokenCounter, modelManager, sessionManager, workspace, tokenTracker, }: CommandOptions): ReplCommand;
 //# sourceMappingURL=index.d.ts.map

package/dist/commands/session/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../source/commands/session/index.ts"],"names":[],"mappings":"AAuBA,OAAO,KAAK,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAI/D,wBAAgB,cAAc,CAAC,EAC7B,MAAM,EACN,YAAY,EACZ,YAAY,EACZ,cAAc,EAAE,cAAc,EAC9B,SAAS,EACT,YAAY,GACb,EAAE,cAAc,GAAG,WAAW,CAkR9B"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../source/commands/session/index.ts"],"names":[],"mappings":"AAoBA,OAAO,KAAK,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAoH/D,wBAAgB,cAAc,CAAC,EAC7B,MAAM,EACN,YAAY,EACZ,YAAY,EACZ,cAAc,EACd,SAAS,EACT,YAAY,GACb,EAAE,cAAc,GAAG,WAAW,CAgK9B"}