git-coco 0.30.0 → 0.31.0

package/dist/index.d.ts

@@ -177,6 +177,18 @@ type BaseConfig = {
      * @default true
      */
     includeBranchName?: boolean;
+    /**
+     * The AI CLI tool to use for auto-fixing review issues.
+     * Must match a registered adapter key (e.g. "codex", "claude", "gemini").
+     * When unset, the auto-fix action is disabled.
+     */
+    autoFixTool?: string;
+    /**
+     * Additional key-value flags passed to the auto-fix CLI tool.
+     * Keys are flag names (without leading dashes); values are flag values.
+     * @example { "model": "o4-mini", "approval-mode": "auto-edit" }
+     */
+    autoFixToolOptions?: Record<string, string>;
 };
 type ConfigWithServiceObject = BaseConfig & Partial<BaseCommandOptions> & {
     service: LLMService;

package/dist/index.esm.mjs

@@ -37,7 +37,7 @@ import '@langchain/core/utils/env';
 import '@langchain/core/utils/async_caller';
 import { minimatch } from 'minimatch';
 import { encoding_for_model } from 'tiktoken';
-import { exec } from 'child_process';
+import { exec, spawn } from 'child_process';
 import * as readline from 'readline';
 import { pathToFileURL } from 'url';
 
@@ -46,7 +46,7 @@ import { pathToFileURL } from 'url';
 /**
  * Current build version from package.json
  */
-const BUILD_VERSION = "0.30.0";
+const BUILD_VERSION = "0.31.0";
 
 const isInteractive = (config) => {
     return config?.mode === 'interactive' || !!config?.interactive;
@@ -928,6 +928,23 @@ const schema$1 = {
                     "type": "boolean",
                     "description": "Whether to include the current branch name in the commit prompt for context. When enabled, the current git branch name will be included in the prompt.",
                     "default": true
+                },
+                "autoFixTool": {
+                    "type": "string",
+                    "description": "The AI CLI tool to use for auto-fixing review issues. Must match a registered adapter key (e.g. \"codex\", \"claude\", \"gemini\"). When unset, the auto-fix action is disabled."
+                },
+                "autoFixToolOptions": {
+                    "type": "object",
+                    "additionalProperties": {
+                        "type": "string"
+                    },
+                    "description": "Additional key-value flags passed to the auto-fix CLI tool. Keys are flag names (without leading dashes); values are flag values.",
+                    "examples": [
+                        {
+                            "model": "o4-mini",
+                            "approval-mode": "auto-edit"
+                        }
+                    ]
                 }
             },
             "required": [
@@ -5353,10 +5370,10 @@ class $ZodRegistry {
     }
 }
 // registries
-function registry() {
+function registry$1() {
     return new $ZodRegistry();
 }
-(_a = globalThis).__zod_globalRegistry ?? (_a.__zod_globalRegistry = registry());
+(_a = globalThis).__zod_globalRegistry ?? (_a.__zod_globalRegistry = registry$1());
 const globalRegistry = globalThis.__zod_globalRegistry;
 
 // @__NO_SIDE_EFFECTS__
@@ -7179,6 +7196,11 @@ function refine(fn, _params = {}) {
 function superRefine(fn) {
     return _superRefine(fn);
 }
+// preprocess
+// /** @deprecated Use `z.pipe()` and `z.transform()` instead. */
+function preprocess(fn, schema) {
+    return pipe(transform(fn), schema);
+}
 
 const ChangelogResponseSchema = object({
     title: string(),
@@ -8210,7 +8232,20 @@ function getStatus(file, location = 'index') {
                 return 'unknown';
         }
     }
+    else if ('binary' in file && file.binary === true) {
+        // DiffResultBinaryFile: has before/after, no changes/insertions/deletions
+        if (file.file.includes('=>'))
+            return 'renamed';
+        if (file.before === 0 && file.after > 0)
+            return 'added';
+        if (file.after === 0 && file.before > 0)
+            return 'deleted';
+        if (file.before > 0 && file.after > 0)
+            return 'modified';
+        return 'untracked';
+    }
     else if ('changes' in file && 'binary' in file) {
+        // DiffResultTextFile: has changes/insertions/deletions
         if (file.changes === 0)
             return 'untracked';
         if (file.file.includes('=>'))
@@ -12125,15 +12160,47 @@ async function fileChangeParser({ changes, commit, options: { tokenizer, git, ll
     return summary;
 }
 
+/**
+ * Detects whether a GitError was caused by a pre-commit hook that modified files.
+ * These hooks (e.g. black, prettier) reformat files and exit non-zero on the first run,
+ * but the commit can succeed once the modified files are re-staged.
+ */
+function isPreCommitHookModifiedFilesError(error) {
+    if (!(error instanceof Error))
+        return false;
+    const msg = error.message;
+    // pre-commit framework outputs "files were modified by this hook"
+    // git itself outputs "modified files exist in working tree" in some hook setups
+    return (msg.includes('files were modified by this hook') ||
+        msg.includes('modified by this hook') ||
+        msg.includes('hook id:'));
+}
 /**
  * Creates a commit with the specified commit message.
+ * Handles the case where pre-commit hooks modify files (e.g. black, prettier):
+ * when detected, stages the hook-modified files and retries the commit once.
  *
  * @param message The commit message.
  * @param git The SimpleGit instance.
+ * @param onHookModifiedFiles Optional callback invoked before the auto-retry so callers can notify the user.
  * @returns A Promise that resolves to the CommitResult.
  */
-async function createCommit(message, git) {
-    return await git.commit(message);
+async function createCommit(message, git, onHookModifiedFiles) {
+    try {
+        return await git.commit(message);
+    }
+    catch (error) {
+        if (isPreCommitHookModifiedFilesError(error)) {
+            // Notify caller (e.g. to show a spinner message or log)
+            if (onHookModifiedFiles) {
+                await onHookModifiedFiles();
+            }
+            // Stage all hook-modified files and retry the commit once
+            await git.add('.');
+            return await git.commit(message);
+        }
+        throw error;
+    }
 }
 
 /**
@@ -12712,7 +12779,9 @@ IMPORTANT RULES:
     handleResult({
         result: commitMsg,
         interactiveModeCallback: async (result) => {
-            await createCommit(result, git);
+            await createCommit(result, git, () => {
+                logger.log('⚠️  Pre-commit hook modified files. Staging changes and retrying commit...', { color: 'yellow' });
+            });
             logSuccess();
         },
         mode: MODE,
@@ -13563,15 +13632,103 @@ const builder = (yargs) => {
     return yargs.options(options).usage(getCommandUsageHeader(command));
 };
 
+async function buildPrompt(item) {
+    const ext = path.extname(item.filePath).slice(1);
+    let fileSection;
+    try {
+        const contents = await fs.promises.readFile(item.filePath, 'utf-8');
+        fileSection = `\`\`\`${ext}\n${contents}\n\`\`\``;
+    }
+    catch {
+        fileSection = `[WARNING: File "${item.filePath}" was not found on disk. Fix based on the issue description alone.]`;
+    }
+    return `You are an expert software engineer. Fix the following code review issue.
+
+## Issue
+Title:    ${item.title}
+Category: ${item.category}
+Severity: ${item.severity}/10
+File:     ${item.filePath}
+
+## Problem
+${item.summary}
+
+## File Contents
+${fileSection}
+
+Fix the issue described above. Make only the changes necessary to resolve this specific problem.`;
+}
+
+class CodexAdapter {
+    async run(prompt, options, apiKey) {
+        const args = ['exec'];
+        if (options) {
+            for (const [key, value] of Object.entries(options)) {
+                if (key === 'model' || key === 'm') {
+                    args.push('--model', value);
+                }
+                else if (key === 'sandbox' || key === 's') {
+                    args.push('--sandbox', value);
+                }
+                else {
+                    args.push('-c', `${key}=${value}`);
+                }
+            }
+        }
+        args.push('--full-auto', prompt);
+        // Preserve the caller's environment by default and only override the API key
+        // when an explicit non-empty key is provided through auto-fix config.
+        const env = { ...process.env };
+        if (apiKey)
+            env['OPENAI_API_KEY'] = apiKey;
+        return new Promise((resolve, reject) => {
+            const child = spawn('codex', args, { stdio: 'inherit', env });
+            child.on('error', (err) => {
+                if (err.code === 'ENOENT') {
+                    reject(new Error('codex binary not found. Please install it: npm i -g @openai/codex'));
+                }
+                else {
+                    reject(err);
+                }
+            });
+            child.on('close', (code) => {
+                if (code === 0) {
+                    resolve();
+                }
+                else {
+                    reject(new Error(`codex exited with code ${code}`));
+                }
+            });
+        });
+    }
+}
+
+const registry = {
+    codex: new CodexAdapter(),
+};
+async function runAutoFix(item, config) {
+    if (!config.autoFixTool) {
+        return;
+    }
+    const adapter = registry[config.autoFixTool];
+    if (!adapter) {
+        throw new Error(`Unknown autoFixTool: "${config.autoFixTool}"`);
+    }
+    const prompt = await buildPrompt(item);
+    await adapter.run(prompt, config.autoFixToolOptions, config.apiKey);
+}
+
 class TaskList {
-    constructor(items) {
+    constructor(items, config) {
         this.currentIndex = 0;
         this.items = items.map((item) => ({ ...item, status: 'pending' }));
+        this.config = config;
         this.rl = readline.createInterface({
             input: process.stdin,
             output: process.stdout,
         });
         process.stdin.setRawMode(true);
+        readline.emitKeypressEvents(process.stdin);
     }
     async displayCurrentItem() {
         const item = this.items[this.currentIndex];
@@ -13588,18 +13745,12 @@ class TaskList {
         return [
             { name: `✅ Mark as complete ${hotKey('d')}`, value: 'complete' },
             { name: `📂 Open file ${hotKey('o')}`, value: 'open' },
+            { name: `🤖 Auto-fix ${hotKey('a')}`, value: 'autofix' },
             { name: `⏩ Skip ${hotKey('s')}`, value: 'skip' },
             { name: `🙈 Omit ${hotKey('x')}`, value: 'omit' },
             { name: `${exitText} ${hotKey('q')}`, value: 'exit' },
         ];
     }
-    async promptAction() {
-        const action = await select({
-            message: 'Choose an action:',
-            choices: this.getChoices(),
-        });
-        return action;
-    }
     async openFile() {
         const item = this.items[this.currentIndex];
         await execPromise(`${process.env.EDITOR || 'code'} ${item.filePath}`);
@@ -13608,6 +13759,35 @@ class TaskList {
         this.items[this.currentIndex].status = 'completed';
         this.navigate(1);
     }
+    async autoFix() {
+        if (!this.config?.autoFixTool) {
+            console.log(chalk.yellow('No autoFixTool configured. Set "autoFixTool" in .coco.config.json'));
+            return;
+        }
+        const item = this.items[this.currentIndex];
+        console.clear();
+        console.log(chalk.bold.cyan(`🤖 Running auto-fix: ${item.title}`));
+        console.log(chalk.dim(`File: ${item.filePath}\n`));
+        // Fully release terminal control before handing off to child process
+        process.stdin.setRawMode(false);
+        process.stdin.pause();
+        await new Promise((r) => setTimeout(r, 50));
+        try {
+            await runAutoFix(item, this.config);
+            this.markAsComplete();
+            console.log(chalk.green('\n✅ Auto-fix completed successfully.'));
+            await new Promise((r) => setTimeout(r, 1200));
+        }
+        catch (err) {
+            console.log(chalk.red(`\n❌ Auto-fix failed: ${err.message}`));
+            await new Promise((r) => setTimeout(r, 1800));
+        }
+        finally {
+            // Restore terminal for our keypress handler
+            process.stdin.resume();
+            process.stdin.setRawMode(true);
+        }
+    }
     skip() {
         this.items[this.currentIndex].status = 'skipped';
         this.navigate(1);
@@ -13617,7 +13797,14 @@ class TaskList {
         this.navigate(1);
     }
     navigate(direction) {
-        this.currentIndex = (this.currentIndex + direction + this.items.length) % this.items.length;
+        const allDone = this.items.every((item) => item.status !== 'pending');
+        if (allDone)
+            return;
+        let next = (this.currentIndex + direction + this.items.length) % this.items.length;
+        while (this.items[next].status !== 'pending') {
+            next = (next + direction + this.items.length) % this.items.length;
+        }
+        this.currentIndex = next;
     }
     async start() {
         while (true) {
@@ -13630,6 +13817,9 @@ class TaskList {
                 case 'complete':
                     this.markAsComplete();
                     break;
+                case 'autofix':
+                    await this.autoFix();
+                    break;
                 case 'skip':
                     this.skip();
                     break;
@@ -13647,43 +13837,39 @@ class TaskList {
                     await this.displaySummary();
                     return;
             }
+            if (this.items.every((item) => item.status !== 'pending')) {
+                this.rl.close();
+                await this.displaySummary();
+                return;
+            }
         }
     }
+    renderMenu() {
+        const choices = this.getChoices();
+        console.log(chalk.dim('Choose an action:'));
+        choices.forEach((c) => console.log(`  ${c.name}`));
+    }
     getActionWithKeyboardShortcut() {
+        this.renderMenu();
         return new Promise((resolve) => {
-            const keyHandler = (_, key) => {
-                if (key) {
-                    switch (key.name) {
-                        case 'o':
-                            resolve('open');
-                            break;
-                        case 'd':
-                            resolve('complete');
-                            break;
-                        case 's':
-                            resolve('skip');
-                            break;
-                        case 'x':
-                            resolve('omit');
-                            break;
-                        case 'right':
-                            resolve('next');
-                            break;
-                        case 'left':
-                            resolve('prev');
-                            break;
-                        case 'q':
-                            resolve('exit');
-                            break;
-                    }
-                }
+            const actionMap = {
+                o: 'open',
+                a: 'autofix',
+                d: 'complete',
+                s: 'skip',
+                x: 'omit',
+                right: 'next',
+                left: 'prev',
+                q: 'exit',
             };
-            readline.emitKeypressEvents(process.stdin);
-            process.stdin.on('keypress', keyHandler);
-            this.promptAction().then((action) => {
-                process.stdin.removeListener('keypress', keyHandler);
+            const handleKeypress = (_, key) => {
+                const action = key ? actionMap[key.name] : undefined;
+                if (!action)
+                    return;
+                process.stdin.removeListener('keypress', handleKeypress);
                 resolve(action);
-            });
+            };
+            process.stdin.on('keypress', handleKeypress);
         });
     }
     async displaySummary() {
@@ -13744,6 +13930,9 @@ const REVIEW_PROMPT = new PromptTemplate({
     inputVariables,
 });
 
+// Some review prompts still produce a single feedback object. Normalize that shape
+// so the parser always returns an array for the rest of the review flow.
+const ReviewFeedbackResponseSchema = preprocess((value) => (Array.isArray(value) ? value : [value]), ReviewFeedbackItemArraySchema);
 const handler = async (argv, logger) => {
     const git = getRepo();
     const config = loadConfig(argv);
@@ -13844,7 +14033,7 @@ const handler = async (argv, logger) => {
         factory,
         parser,
         agent: async (context, options) => {
-            const parser = createSchemaParser(ReviewFeedbackItemArraySchema, llm);
+            const parser = createSchemaParser(ReviewFeedbackResponseSchema, llm);
             const formatInstructions = "Respond with a valid JSON object, containing four fields:'title' a string, 'summary' a short summary of the problem (include line number if big file), 'severity' a numeric enum up to ten, 'category' an enum string, and 'filePath' a relative filepath to file as string.";
             const prompt = getPrompt({
                 template: options.prompt,
@@ -13876,7 +14065,7 @@ const handler = async (argv, logger) => {
             process.exit(0);
         },
     });
-    const reviewer = new TaskList(recap);
+    const reviewer = new TaskList(recap, { ...config, apiKey: key ?? undefined });
     await reviewer.start();
 };
 

package/dist/index.js

@@ -68,7 +68,7 @@ var readline__namespace = /*#__PURE__*/_interopNamespaceDefault(readline);
 /**
  * Current build version from package.json
  */
-const BUILD_VERSION = "0.30.0";
+const BUILD_VERSION = "0.31.0";
 
 const isInteractive = (config) => {
     return config?.mode === 'interactive' || !!config?.interactive;
@@ -950,6 +950,23 @@ const schema$1 = {
                     "type": "boolean",
                     "description": "Whether to include the current branch name in the commit prompt for context. When enabled, the current git branch name will be included in the prompt.",
                     "default": true
+                },
+                "autoFixTool": {
+                    "type": "string",
+                    "description": "The AI CLI tool to use for auto-fixing review issues. Must match a registered adapter key (e.g. \"codex\", \"claude\", \"gemini\"). When unset, the auto-fix action is disabled."
+                },
+                "autoFixToolOptions": {
+                    "type": "object",
+                    "additionalProperties": {
+                        "type": "string"
+                    },
+                    "description": "Additional key-value flags passed to the auto-fix CLI tool. Keys are flag names (without leading dashes); values are flag values.",
+                    "examples": [
+                        {
+                            "model": "o4-mini",
+                            "approval-mode": "auto-edit"
+                        }
+                    ]
                 }
             },
             "required": [
@@ -5375,10 +5392,10 @@ class $ZodRegistry {
     }
 }
 // registries
-function registry() {
+function registry$1() {
     return new $ZodRegistry();
 }
-(_a = globalThis).__zod_globalRegistry ?? (_a.__zod_globalRegistry = registry());
+(_a = globalThis).__zod_globalRegistry ?? (_a.__zod_globalRegistry = registry$1());
 const globalRegistry = globalThis.__zod_globalRegistry;
 
 // @__NO_SIDE_EFFECTS__
@@ -7201,6 +7218,11 @@ function refine(fn, _params = {}) {
 function superRefine(fn) {
     return _superRefine(fn);
 }
+// preprocess
+// /** @deprecated Use `z.pipe()` and `z.transform()` instead. */
+function preprocess(fn, schema) {
+    return pipe(transform(fn), schema);
+}
 
 const ChangelogResponseSchema = object({
     title: string(),
@@ -8232,7 +8254,20 @@ function getStatus(file, location = 'index') {
                 return 'unknown';
         }
     }
+    else if ('binary' in file && file.binary === true) {
+        // DiffResultBinaryFile: has before/after, no changes/insertions/deletions
+        if (file.file.includes('=>'))
+            return 'renamed';
+        if (file.before === 0 && file.after > 0)
+            return 'added';
+        if (file.after === 0 && file.before > 0)
+            return 'deleted';
+        if (file.before > 0 && file.after > 0)
+            return 'modified';
+        return 'untracked';
+    }
     else if ('changes' in file && 'binary' in file) {
+        // DiffResultTextFile: has changes/insertions/deletions
         if (file.changes === 0)
             return 'untracked';
         if (file.file.includes('=>'))
@@ -12147,15 +12182,47 @@ async function fileChangeParser({ changes, commit, options: { tokenizer, git, ll
     return summary;
 }
 
+/**
+ * Detects whether a GitError was caused by a pre-commit hook that modified files.
+ * These hooks (e.g. black, prettier) reformat files and exit non-zero on the first run,
+ * but the commit can succeed once the modified files are re-staged.
+ */
+function isPreCommitHookModifiedFilesError(error) {
+    if (!(error instanceof Error))
+        return false;
+    const msg = error.message;
+    // pre-commit framework outputs "files were modified by this hook"
+    // git itself outputs "modified files exist in working tree" in some hook setups
+    return (msg.includes('files were modified by this hook') ||
+        msg.includes('modified by this hook') ||
+        msg.includes('hook id:'));
+}
 /**
  * Creates a commit with the specified commit message.
+ * Handles the case where pre-commit hooks modify files (e.g. black, prettier):
+ * when detected, stages the hook-modified files and retries the commit once.
  *
  * @param message The commit message.
  * @param git The SimpleGit instance.
+ * @param onHookModifiedFiles Optional callback invoked before the auto-retry so callers can notify the user.
  * @returns A Promise that resolves to the CommitResult.
  */
-async function createCommit(message, git) {
-    return await git.commit(message);
+async function createCommit(message, git, onHookModifiedFiles) {
+    try {
+        return await git.commit(message);
+    }
+    catch (error) {
+        if (isPreCommitHookModifiedFilesError(error)) {
+            // Notify caller (e.g. to show a spinner message or log)
+            if (onHookModifiedFiles) {
+                await onHookModifiedFiles();
+            }
+            // Stage all hook-modified files and retry the commit once
+            await git.add('.');
+            return await git.commit(message);
+        }
+        throw error;
+    }
 }
 
 /**
@@ -12734,7 +12801,9 @@ IMPORTANT RULES:
     handleResult({
         result: commitMsg,
         interactiveModeCallback: async (result) => {
-            await createCommit(result, git);
+            await createCommit(result, git, () => {
+                logger.log('⚠️  Pre-commit hook modified files. Staging changes and retrying commit...', { color: 'yellow' });
+            });
             logSuccess();
         },
         mode: MODE,
@@ -13585,15 +13654,103 @@ const builder = (yargs) => {
     return yargs.options(options).usage(getCommandUsageHeader(command));
 };
 
+async function buildPrompt(item) {
+    const ext = path__namespace.extname(item.filePath).slice(1);
+    let fileSection;
+    try {
+        const contents = await fs__namespace.promises.readFile(item.filePath, 'utf-8');
+        fileSection = `\`\`\`${ext}\n${contents}\n\`\`\``;
+    }
+    catch {
+        fileSection = `[WARNING: File "${item.filePath}" was not found on disk. Fix based on the issue description alone.]`;
+    }
+    return `You are an expert software engineer. Fix the following code review issue.
+
+## Issue
+Title:    ${item.title}
+Category: ${item.category}
+Severity: ${item.severity}/10
+File:     ${item.filePath}
+
+## Problem
+${item.summary}
+
+## File Contents
+${fileSection}
+
+Fix the issue described above. Make only the changes necessary to resolve this specific problem.`;
+}
+
+class CodexAdapter {
+    async run(prompt, options, apiKey) {
+        const args = ['exec'];
+        if (options) {
+            for (const [key, value] of Object.entries(options)) {
+                if (key === 'model' || key === 'm') {
+                    args.push('--model', value);
+                }
+                else if (key === 'sandbox' || key === 's') {
+                    args.push('--sandbox', value);
+                }
+                else {
+                    args.push('-c', `${key}=${value}`);
+                }
+            }
+        }
+        args.push('--full-auto', prompt);
+        // Preserve the caller's environment by default and only override the API key
+        // when an explicit non-empty key is provided through auto-fix config.
+        const env = { ...process.env };
+        if (apiKey)
+            env['OPENAI_API_KEY'] = apiKey;
+        return new Promise((resolve, reject) => {
+            const child = child_process.spawn('codex', args, { stdio: 'inherit', env });
+            child.on('error', (err) => {
+                if (err.code === 'ENOENT') {
+                    reject(new Error('codex binary not found. Please install it: npm i -g @openai/codex'));
+                }
+                else {
+                    reject(err);
+                }
+            });
+            child.on('close', (code) => {
+                if (code === 0) {
+                    resolve();
+                }
+                else {
+                    reject(new Error(`codex exited with code ${code}`));
+                }
+            });
+        });
+    }
+}
+
+const registry = {
+    codex: new CodexAdapter(),
+};
+async function runAutoFix(item, config) {
+    if (!config.autoFixTool) {
+        return;
+    }
+    const adapter = registry[config.autoFixTool];
+    if (!adapter) {
+        throw new Error(`Unknown autoFixTool: "${config.autoFixTool}"`);
+    }
+    const prompt = await buildPrompt(item);
+    await adapter.run(prompt, config.autoFixToolOptions, config.apiKey);
+}
+
 class TaskList {
-    constructor(items) {
+    constructor(items, config) {
         this.currentIndex = 0;
         this.items = items.map((item) => ({ ...item, status: 'pending' }));
+        this.config = config;
         this.rl = readline__namespace.createInterface({
             input: process.stdin,
             output: process.stdout,
         });
         process.stdin.setRawMode(true);
+        readline__namespace.emitKeypressEvents(process.stdin);
     }
     async displayCurrentItem() {
         const item = this.items[this.currentIndex];
@@ -13610,18 +13767,12 @@ class TaskList {
         return [
             { name: `✅ Mark as complete ${hotKey('d')}`, value: 'complete' },
             { name: `📂 Open file ${hotKey('o')}`, value: 'open' },
+            { name: `🤖 Auto-fix ${hotKey('a')}`, value: 'autofix' },
             { name: `⏩ Skip ${hotKey('s')}`, value: 'skip' },
             { name: `🙈 Omit ${hotKey('x')}`, value: 'omit' },
             { name: `${exitText} ${hotKey('q')}`, value: 'exit' },
         ];
     }
-    async promptAction() {
-        const action = await prompts.select({
-            message: 'Choose an action:',
-            choices: this.getChoices(),
-        });
-        return action;
-    }
     async openFile() {
         const item = this.items[this.currentIndex];
         await execPromise(`${process.env.EDITOR || 'code'} ${item.filePath}`);
@@ -13630,6 +13781,35 @@ class TaskList {
         this.items[this.currentIndex].status = 'completed';
         this.navigate(1);
     }
+    async autoFix() {
+        if (!this.config?.autoFixTool) {
+            console.log(chalk.yellow('No autoFixTool configured. Set "autoFixTool" in .coco.config.json'));
+            return;
+        }
+        const item = this.items[this.currentIndex];
+        console.clear();
+        console.log(chalk.bold.cyan(`🤖 Running auto-fix: ${item.title}`));
+        console.log(chalk.dim(`File: ${item.filePath}\n`));
+        // Fully release terminal control before handing off to child process
+        process.stdin.setRawMode(false);
+        process.stdin.pause();
+        await new Promise((r) => setTimeout(r, 50));
+        try {
+            await runAutoFix(item, this.config);
+            this.markAsComplete();
+            console.log(chalk.green('\n✅ Auto-fix completed successfully.'));
+            await new Promise((r) => setTimeout(r, 1200));
+        }
+        catch (err) {
+            console.log(chalk.red(`\n❌ Auto-fix failed: ${err.message}`));
+            await new Promise((r) => setTimeout(r, 1800));
+        }
+        finally {
+            // Restore terminal for our keypress handler
+            process.stdin.resume();
+            process.stdin.setRawMode(true);
+        }
+    }
     skip() {
         this.items[this.currentIndex].status = 'skipped';
         this.navigate(1);
@@ -13639,7 +13819,14 @@ class TaskList {
         this.navigate(1);
     }
     navigate(direction) {
-        this.currentIndex = (this.currentIndex + direction + this.items.length) % this.items.length;
+        const allDone = this.items.every((item) => item.status !== 'pending');
+        if (allDone)
+            return;
+        let next = (this.currentIndex + direction + this.items.length) % this.items.length;
+        while (this.items[next].status !== 'pending') {
+            next = (next + direction + this.items.length) % this.items.length;
+        }
+        this.currentIndex = next;
     }
     async start() {
         while (true) {
@@ -13652,6 +13839,9 @@ class TaskList {
                 case 'complete':
                     this.markAsComplete();
                     break;
+                case 'autofix':
+                    await this.autoFix();
+                    break;
                 case 'skip':
                     this.skip();
                     break;
@@ -13669,43 +13859,39 @@ class TaskList {
                     await this.displaySummary();
                     return;
             }
+            if (this.items.every((item) => item.status !== 'pending')) {
+                this.rl.close();
+                await this.displaySummary();
+                return;
+            }
         }
     }
+    renderMenu() {
+        const choices = this.getChoices();
+        console.log(chalk.dim('Choose an action:'));
+        choices.forEach((c) => console.log(`  ${c.name}`));
+    }
     getActionWithKeyboardShortcut() {
+        this.renderMenu();
         return new Promise((resolve) => {
-            const keyHandler = (_, key) => {
-                if (key) {
-                    switch (key.name) {
-                        case 'o':
-                            resolve('open');
-                            break;
-                        case 'd':
-                            resolve('complete');
-                            break;
-                        case 's':
-                            resolve('skip');
-                            break;
-                        case 'x':
-                            resolve('omit');
-                            break;
-                        case 'right':
-                            resolve('next');
-                            break;
-                        case 'left':
-                            resolve('prev');
-                            break;
-                        case 'q':
-                            resolve('exit');
-                            break;
-                    }
-                }
+            const actionMap = {
+                o: 'open',
+                a: 'autofix',
+                d: 'complete',
+                s: 'skip',
+                x: 'omit',
+                right: 'next',
+                left: 'prev',
+                q: 'exit',
             };
-            readline__namespace.emitKeypressEvents(process.stdin);
-            process.stdin.on('keypress', keyHandler);
-            this.promptAction().then((action) => {
-                process.stdin.removeListener('keypress', keyHandler);
+            const handleKeypress = (_, key) => {
+                const action = key ? actionMap[key.name] : undefined;
+                if (!action)
+                    return;
+                process.stdin.removeListener('keypress', handleKeypress);
                 resolve(action);
-            });
+            };
+            process.stdin.on('keypress', handleKeypress);
         });
     }
     async displaySummary() {
@@ -13766,6 +13952,9 @@ const REVIEW_PROMPT = new prompts$1.PromptTemplate({
     inputVariables,
 });
 
+// Some review prompts still produce a single feedback object. Normalize that shape
+// so the parser always returns an array for the rest of the review flow.
+const ReviewFeedbackResponseSchema = preprocess((value) => (Array.isArray(value) ? value : [value]), ReviewFeedbackItemArraySchema);
 const handler = async (argv, logger) => {
     const git = getRepo();
     const config = loadConfig(argv);
@@ -13866,7 +14055,7 @@ const handler = async (argv, logger) => {
         factory,
         parser,
         agent: async (context, options) => {
-            const parser = createSchemaParser(ReviewFeedbackItemArraySchema, llm);
+            const parser = createSchemaParser(ReviewFeedbackResponseSchema, llm);
             const formatInstructions = "Respond with a valid JSON object, containing four fields:'title' a string, 'summary' a short summary of the problem (include line number if big file), 'severity' a numeric enum up to ten, 'category' an enum string, and 'filePath' a relative filepath to file as string.";
             const prompt = getPrompt({
                 template: options.prompt,
@@ -13898,7 +14087,7 @@ const handler = async (argv, logger) => {
             process.exit(0);
         },
     });
-    const reviewer = new TaskList(recap);
+    const reviewer = new TaskList(recap, { ...config, apiKey: key ?? undefined });
     await reviewer.start();
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "git-coco",
-  "version": "0.30.0",
+  "version": "0.31.0",
   "description": "zero-effort git commits with coco.",
   "author": "gfargo <ghfargo@gmail.com>",
   "license": "MIT",