gent-cli 26.0.0 → 28.0.0

package/README.md

@@ -24,7 +24,7 @@ Beyond a faithful git-like workflow, Gent adds:
 - **`gent undo` / `gent redo`** — a one-command safety net over an operation journal (friendlier than `git reflog`).
 - **`gent resolve`** — an interactive conflict resolver (ours / theirs / both / edit / AI).
 - **`gent summary`** — a repository health dashboard, plus **`gent log --graph`**.
-- **Direct local AI** (`gent chat`, `gent review`, `gent merge --ai`, `gent resolve --ai`) — low-latency OpenAI calls from the CLI with task-specific prompts and no per-command key prompt.
+- **Direct local AI** (`gent chat`, `gent review`, `gent merge --ai`, `gent resolve --ai`) — low-latency OpenAI calls from the CLI with task-specific prompts. Configure once with `gent ai configure`.
 - **Genti, your terminal mascot** — a mint one-eyed sky-jelly that *acts out* your workflow: it floats a file crate to the cloud on `gent push`, carries one home on `gent pull`, and reconciles two branches on `gent merge`. It plays once (in place, no scrollback spam) after a successful command. Meet it directly with `gent pet` (add `--loop` to keep it running; try `gent pet push|pull|merge|auth`). Set `GENT_NO_PET=1` (or run in CI / a non-interactive shell) to turn the celebrations off.
 
 See [docs/COMMANDS.md](docs/COMMANDS.md) for the full reference and
@@ -500,14 +500,16 @@ gent explain --staged             # explain currently staged changes
 
 ### Optional AI features
 
-AI calls run directly from the CLI using the credential provisioned for the
-installation. Commands never prompt users for a provider key. Fast task-specific
-prompts keep review, merge resolution, chat, and summaries concise.
+AI calls run directly from the CLI. Run `gent ai configure` once to enter the
+OpenAI key through a masked prompt; Gent stores it locally with owner-only
+permissions. Fast task-specific prompts keep review, merge resolution, chat,
+and summaries concise.
 
 ```bash
+gent ai configure                 # configure OpenAI once on this computer
 gent commit --ai                  # suggest a commit message from the staged diff
 gent explain                      # narrate a diff instead of just printing it
-gent resolve                      # adds an "Ask AI" choice per conflict hunk
+gent resolve                      # ours/theirs/both/AI/edit/skip per conflict hunk
 ```
 
 ### Tags

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gent-cli",
-  "version": "26.0.0",
+  "version": "28.0.0",
   "description": "A modern, Git-like version control CLI with cloud sync, AI-powered superpowers (ask/review/docs/changelog), and zero-friction setup (gent setup/doctor/config).",
   "main": "src/index.js",
   "bin": {
@@ -8,8 +8,8 @@
   },
   "scripts": {
     "start": "node src/index.js",
-    "test": "node --check src/index.js && node --test tests/diff.test.js tests/merge.test.js tests/hash.test.js tests/merge-base.test.js tests/file-system.test.js tests/web-urls.test.js tests/repos.test.js tests/ai-service.test.js tests/auth-storage.test.js && node tests/offline-e2e.js && node tests/ai-merge.e2e.js && node --test tests/git-compat/engine.test.js tests/git-compat/cli.test.js tests/git-compat/transport.test.js tests/git-compat/migrate.test.js",
-    "test:unit": "node --test tests/diff.test.js tests/merge.test.js tests/hash.test.js tests/merge-base.test.js tests/file-system.test.js tests/web-urls.test.js tests/repos.test.js tests/ai-service.test.js tests/auth-storage.test.js",
+    "test": "node --check src/index.js && node --test tests/diff.test.js tests/merge.test.js tests/hash.test.js tests/merge-base.test.js tests/file-system.test.js tests/web-urls.test.js tests/repos.test.js tests/ai-service.test.js tests/auth-storage.test.js tests/local-ai-config.test.js tests/resolve-options.test.js && node tests/offline-e2e.js && node tests/ai-merge.e2e.js && node --test tests/git-compat/engine.test.js tests/git-compat/cli.test.js tests/git-compat/transport.test.js tests/git-compat/migrate.test.js",
+    "test:unit": "node --test tests/diff.test.js tests/merge.test.js tests/hash.test.js tests/merge-base.test.js tests/file-system.test.js tests/web-urls.test.js tests/repos.test.js tests/ai-service.test.js tests/auth-storage.test.js tests/local-ai-config.test.js tests/resolve-options.test.js",
     "test:e2e": "node tests/offline-e2e.js",
     "test:remote:e2e": "node tests/remote-e2e.js",
     "demo": "bash demo.sh",

package/src/commands/ai.js

@@ -1,34 +1,56 @@
 /**
  * AI Command - Manage and verify AI integration.
  *
- *   gent ai status            → show the managed service status
+ *   gent ai configure         → securely save a key for this computer
+ *   gent ai status            → show local AI status
  *   gent ai test              → make a tiny live request to confirm it works
  *   gent ai models            → show provider management information
  */
 
 const chalk = require('chalk');
+const inquirer = require('inquirer');
 const ora = require('ora');
 const ai = require('../utils/ai-service');
+const localAiConfig = require('../utils/local-ai-config');
 
 async function aiCommand(subcommand) {
     const sub = (subcommand || 'status').toLowerCase();
     switch (sub) {
+        case 'configure': return configure();
         case 'status': return status();
         case 'test': return test();
         case 'models': return models();
         default:
             console.error(chalk.red(`Unknown subcommand '${sub}'`));
-            console.log(chalk.gray('Usage: gent ai <status|test|models>'));
+            console.log(chalk.gray('Usage: gent ai <configure|status|test|models>'));
             process.exit(1);
     }
 }
 
+async function configure() {
+    if (!process.stdin.isTTY || !process.stdout.isTTY) {
+        console.error(chalk.red('Run `gent ai configure` in an interactive terminal.'));
+        process.exitCode = 1;
+        return;
+    }
+    const { apiKey } = await inquirer.prompt([{
+        type: 'password',
+        name: 'apiKey',
+        message: 'OpenAI API key:',
+        mask: '*',
+        validate: value => localAiConfig.validateApiKey(value) || 'Enter a valid key beginning with sk-.',
+    }]);
+    const savedPath = await localAiConfig.saveApiKey(apiKey);
+    console.log(chalk.green('✓ Gent AI configured for this computer.'));
+    console.log(chalk.gray(`  Stored locally in ${savedPath} with owner-only permissions.`));
+}
+
 async function status() {
     const { value: available, source } = await ai.resolveKey();
 
     console.log(chalk.bold.cyan('\nGent AI status\n'));
     if (available) {
-        console.log(`  ${chalk.green('●')} Service:   ${chalk.white(`direct OpenAI [${source}]`)}`);
+        console.log(`  ${chalk.green('●')} Service:   ${chalk.white(source)}`);
     } else {
         console.log(`  ${chalk.gray('○')} Service:   ${chalk.gray('unavailable')}`);
         console.log(chalk.gray('             ↳ ' + ai.disabledHint()));

package/src/commands/canonical.js

@@ -216,7 +216,7 @@ const handlers = {
                 continue;
             }
             try {
-                const suggestion = await ai.resolveConflictHunk({
+                const resolution = await ai.resolveConflictHunk({
                     base: sides.base?.toString('utf8') || '',
                     ours: sides.ours?.toString('utf8') || '',
                     theirs: sides.theirs?.toString('utf8') || '',
@@ -226,10 +226,11 @@ const handlers = {
                 await worktree.assertNoSymlinkParent(repo, name);
                 const absolute = path.join(repo.worktree, name);
                 await fs.mkdir(path.dirname(absolute), { recursive: true });
-                await fs.writeFile(absolute, suggestion, 'utf8');
+                await fs.writeFile(absolute, resolution.merged, 'utf8');
                 await merge.markResolved(repo, name);
                 resolved++;
                 console.log(`Resolved and staged ${name}`);
+                console.log(`  AI: ${resolution.summary}`);
             } catch (error) {
                 console.log(`AI did not resolve ${name}: ${error.message}`);
             }

package/src/commands/doctor.js

@@ -137,7 +137,7 @@ async function checkAiKey(probe) {
             name: 'AI service',
             status: 'warn',
             detail: 'unavailable in this CLI installation',
-            hint: 'Ask the Gent distributor to provision the local AI credential.',
+            hint: 'Run `gent ai configure` once on this computer.',
         };
     }
 
@@ -145,7 +145,7 @@ async function checkAiKey(probe) {
         return {
             name: 'AI service',
             status: 'pass',
-            detail: `direct OpenAI [${source}], model: ${await ai.resolveModel()} (use --ai to live-test)`,
+            detail: `${source}, model: ${await ai.resolveModel()} (use --ai to live-test)`,
         };
     }
 
@@ -161,7 +161,7 @@ async function checkAiKey(probe) {
             name: 'AI service',
             status: 'fail',
             detail: err.message,
-            hint: 'Retry later or ask the Gent distributor to check AI quota and credentials.',
+            hint: 'Run `gent ai configure` to replace the key, or check the OpenAI project quota.',
         };
     }
 }

package/src/commands/resolve.js

@@ -65,19 +65,6 @@ async function resolve(options = {}) {
             return;
         }
 
-        if (!options.ai && ai.isEnabled() && process.stdin.isTTY && process.stdout.isTTY) {
-            const { mode } = await inquirer.prompt([{
-                type: 'list',
-                name: 'mode',
-                message: 'How should Gent resolve this merge?',
-                choices: [
-                    { name: 'Merge with AI (fast) — resolve, commit, then review', value: 'ai' },
-                    { name: 'Resolve manually — choose each conflict', value: 'manual' },
-                ],
-            }]);
-            options.ai = mode === 'ai';
-        }
-
         console.log(chalk.bold.cyan(`\nResolving merge of '${mergeState.sourceBranch}' — ${markerFiles.length} file(s)\n`));
 
         // Working copy of merged tree entries (we patch hashes as files resolve).
@@ -106,6 +93,7 @@ async function resolve(options = {}) {
             let idx = 0;
             let aborted = false;
             const out = [];
+            const aiSummaries = [];
 
             for (const seg of segments) {
                 if (seg.type === 'text') {
@@ -113,7 +101,7 @@ async function resolve(options = {}) {
                     continue;
                 }
                 idx++;
-                const resolvedLines = await resolveHunk(seg, file, idx, conflictCount, options);
+                const resolvedLines = await resolveHunk(seg, file, idx, conflictCount, options, aiSummaries);
                 if (resolvedLines === null) { aborted = true; break; }
                 out.push(...resolvedLines);
             }
@@ -133,6 +121,9 @@ async function resolve(options = {}) {
             } else {
                 await stageResolved(gentPath, staging, entriesByName, file, resolvedContent);
                 console.log(chalk.green(`  ✓ resolved ${file}`));
+                if (aiSummaries.length) {
+                    console.log(chalk.gray(`    AI: ${summarizeFileChanges(aiSummaries)}`));
+                }
             }
         }
 
@@ -177,26 +168,17 @@ async function resolve(options = {}) {
  * Prompt for one conflict hunk. Returns the chosen lines, or null to abort
  * (leave the rest of the file as-is with markers).
  */
-async function resolveHunk(seg, file, idx, total, options = {}) {
+async function resolveHunk(seg, file, idx, total, options = {}, aiSummaries = []) {
     console.log(chalk.gray(`  Conflict ${idx}/${total}:`));
     console.log(chalk.green('    <<< ours'));
     seg.ours.forEach(l => console.log(chalk.green(`      ${l}`)));
     console.log(chalk.red('    >>> theirs'));
     seg.theirs.forEach(l => console.log(chalk.red(`      ${l}`)));
 
-    const choices = [
-        { name: 'Keep ours', value: 'ours' },
-        { name: 'Keep theirs', value: 'theirs' },
-        { name: 'Keep both (ours then theirs)', value: 'both' },
-        { name: 'Edit manually', value: 'edit' }
-    ];
-    if (ai.isEnabled()) {
-        choices.splice(3, 0, { name: `Ask AI (${ai.getModel()})`, value: 'ai' });
-    }
-    choices.push({ name: 'Skip the rest of this file', value: 'skip' });
+    const choices = resolutionChoices();
 
     if (options.ai) {
-        const suggestion = await askAiForHunk(seg, file, true);
+        const suggestion = await askAiForHunk(seg, file, true, aiSummaries);
         if (suggestion !== null) return suggestion;
         return null;
     }
@@ -223,40 +205,64 @@ async function resolveHunk(seg, file, idx, total, options = {}) {
             return text.replace(/\n$/, '').split('\n');
         }
         case 'ai': {
-            const suggestion = await askAiForHunk(seg, file);
+            const suggestion = await askAiForHunk(seg, file, false, aiSummaries);
             if (suggestion !== null) return suggestion;
-            return resolveHunk(seg, file, idx, total, { ai: false });
+            return resolveHunk(seg, file, idx, total, { ai: false }, aiSummaries);
         }
         default: return seg.ours;
     }
 }
 
-async function askAiForHunk(seg, file, autoAccept = false) {
+function resolutionChoices() {
+    return [
+        { name: 'Keep ours', value: 'ours' },
+        { name: 'Keep theirs', value: 'theirs' },
+        { name: 'Keep both (ours then theirs)', value: 'both' },
+        { name: `Resolve with AI (${ai.getModel()})`, value: 'ai' },
+        { name: 'Edit manually', value: 'edit' },
+        { name: 'Skip the rest of this file', value: 'skip' },
+    ];
+}
+
+async function askAiForHunk(seg, file, autoAccept = false, aiSummaries = []) {
     try {
-        const suggestion = await ai.resolveConflictHunk({
+        const resolution = await ai.resolveConflictHunk({
             ours: seg.ours.join('\n'),
             theirs: seg.theirs.join('\n'),
             fileName: file
         });
         if (autoAccept) {
-            console.log(chalk.green(`    ✓ AI resolved ${file}`));
-            return suggestion.split('\n');
+            aiSummaries.push(resolution.summary);
+            return resolution.merged.split('\n');
         }
         console.log(chalk.cyan('    AI suggestion (review before accepting):'));
-        suggestion.split('\n').forEach(line => console.log(chalk.cyan(`      ${line}`)));
+        resolution.merged.split('\n').forEach(line => console.log(chalk.cyan(`      ${line}`)));
         const { accept } = await inquirer.prompt([{
             type: 'confirm',
             name: 'accept',
             message: 'Use this AI suggestion?',
             default: false,
         }]);
-        return accept ? suggestion.split('\n') : null;
+        if (!accept) return null;
+        aiSummaries.push(resolution.summary);
+        return resolution.merged.split('\n');
     } catch (error) {
         console.log(chalk.yellow(`    AI failed (${error.message}); no file was changed.`));
         return null;
     }
 }
 
+function summarizeFileChanges(summaries) {
+    const summary = [...new Set(summaries)].join('; ');
+    const words = summary.split(/\s+/);
+    const wordLimited = words.length <= 32
+        ? summary
+        : `${words.slice(0, 32).join(' ')}...`;
+    return wordLimited.length <= 220
+        ? wordLimited
+        : `${wordLimited.slice(0, 217).trimEnd()}...`;
+}
+
 /** Store the resolved file as a blob, patch the tree entry, and stage it. */
 async function stageResolved(gentPath, staging, entriesByName, file, content) {
     const hash = await storeBlob(gentPath, content);
@@ -324,3 +330,4 @@ async function finalizeMerge(gentPath, staging, mergeState, entriesByName) {
 }
 
 module.exports = resolve;
+module.exports.resolutionChoices = resolutionChoices;

package/src/index.js

@@ -370,7 +370,7 @@ program
 
 program
     .command('ai [subcommand]')
-    .description('Inspect AI integration (status|test|models)')
+    .description('Configure or inspect local AI (configure|status|test|models)')
     .action(aiCommand);
 
 // ─── Platform-special (AI-powered) ──────────────────────

package/src/utils/ai-service.js

@@ -8,7 +8,7 @@ const DEFAULT_MODEL = 'gpt-4.1-mini';
 const PROMPTS = Object.freeze({
     chat: 'Answer as a concise senior engineer. Use the repository context. Give the direct answer first.',
     review: 'Review fast. Return only concrete correctness, security, or regression risks, then brief fixes. If none, say "No blocking issues."',
-    merge: 'Resolve this merge conflict fast. Preserve both intended behaviors. Return only the final merged text with no markdown fence or explanation.',
+    merge: 'Resolve this merge conflict fast. Preserve both intended behaviors. Return only valid JSON with this shape: {"merged":"final merged text","summary":"one sentence, at most 18 words, saying what was kept or combined"}. Do not use markdown fences.',
     commit: 'Write one concise conventional commit message. Return only the message.',
     explain: 'Explain this change briefly and concretely. Return short bullets only.',
     docs: 'Write concise, accurate repository documentation from only the supplied context.',
@@ -30,7 +30,7 @@ function getApiUrl() {
 
 async function resolveKey() {
     const value = getApiKey();
-    return { value, source: value ? 'local Gent installation' : 'unset' };
+    return { value, source: value ? 'local CLI configuration' : 'unset' };
 }
 
 async function resolveModel() {
@@ -46,7 +46,7 @@ function isEnabled() {
 }
 
 function disabledHint() {
-    return 'Gent AI is unavailable in this CLI installation.';
+    return 'Gent AI is unavailable. Run `gent ai configure` once on this computer.';
 }
 
 function extractText(payload) {
@@ -63,7 +63,6 @@ function extractText(payload) {
 async function complete({ prompt, system, profile = 'chat', maxTokens = 1024 }) {
     const apiKey = getApiKey();
     if (!apiKey) throw new Error(disabledHint());
-
     const instructions = [PROMPTS[profile], system].filter(Boolean).join('\n\n');
     try {
         const response = await axios.post(getApiUrl(), {
@@ -90,10 +89,13 @@ async function complete({ prompt, system, profile = 'chat', maxTokens = 1024 })
 function enrichAiError(error) {
     const status = error?.response?.status;
     const apiError = error?.response?.data?.error;
-    if (status === 401 || status === 403) return new Error('Gent AI credential was rejected.');
-    if (apiError?.code === 'insufficient_quota') return new Error('Gent AI quota is exhausted.');
+    const apiCode = typeof apiError === 'object' ? apiError?.code : null;
+    if (status === 401 || status === 403) return new Error('OpenAI rejected the configured CLI credential.');
+    if (status === 404) return new Error(`OpenAI rejected model "${getModel()}".`);
+    if (apiCode === 'insufficient_quota') return new Error('Gent AI quota is exhausted.');
     if (status === 429) return new Error('Gent AI is busy. Retry in a moment.');
     if (apiError?.message) return new Error(`Gent AI failed: ${apiError.message}`);
+    if (typeof apiError === 'string') return new Error(apiError);
     return error;
 }
 
@@ -120,7 +122,42 @@ async function resolveConflictHunk({ base, ours, theirs, fileName }) {
         `BASE:\n${base || '(none)'}\n\n` +
         `OURS:\n${ours}\n\n` +
         `THEIRS:\n${theirs}`;
-    return complete({ profile: 'merge', prompt, maxTokens: 1400 });
+    const response = await complete({ profile: 'merge', prompt, maxTokens: 1400 });
+    return parseMergeResolution(response);
+}
+
+function parseMergeResolution(response) {
+    const cleaned = response
+        .replace(/^```(?:json)?\s*/i, '')
+        .replace(/\s*```$/, '')
+        .trim();
+    try {
+        const parsed = JSON.parse(cleaned);
+        if (typeof parsed.merged !== 'string') throw new Error('missing merged text');
+        return {
+            merged: parsed.merged,
+            summary: briefSummary(parsed.summary),
+        };
+    } catch {
+        return {
+            merged: response,
+            summary: 'Combined the conflicting changes.',
+        };
+    }
+}
+
+function briefSummary(value) {
+    const summary = typeof value === 'string'
+        ? value.replace(/\s+/g, ' ').trim()
+        : '';
+    if (!summary) return 'Combined the conflicting changes.';
+    const words = summary.split(' ');
+    const wordLimited = words.length <= 18
+        ? summary
+        : `${words.slice(0, 18).join(' ')}...`;
+    return wordLimited.length <= 160
+        ? wordLimited
+        : `${wordLimited.slice(0, 157).trimEnd()}...`;
 }
 
 module.exports = {
@@ -138,5 +175,6 @@ module.exports = {
     explainChanges,
     reviewChanges,
     resolveConflictHunk,
+    parseMergeResolution,
     extractText,
 };

package/src/utils/local-ai-config.js

@@ -0,0 +1,39 @@
+const fs = require('fs').promises;
+const os = require('os');
+const path = require('path');
+const { GENT_DIR } = require('./constants');
+
+function getEnvPath() {
+    return path.join(os.homedir(), GENT_DIR, '.env');
+}
+
+function validateApiKey(value) {
+    return typeof value === 'string' && /^sk-[A-Za-z0-9_-]{20,}$/.test(value.trim());
+}
+
+async function saveApiKey(value) {
+    const apiKey = value.trim();
+    if (!validateApiKey(apiKey)) throw new Error('Enter a valid OpenAI API key beginning with sk-.');
+
+    const envPath = getEnvPath();
+    const directory = path.dirname(envPath);
+    const existing = await fs.readFile(envPath, 'utf8').catch(error => {
+        if (error.code === 'ENOENT') return '';
+        throw error;
+    });
+    const lines = existing
+        .split(/\r?\n/)
+        .filter(line => !/^\s*OPENAI_API_KEY\s*=/.test(line));
+    while (lines.length && lines[lines.length - 1] === '') lines.pop();
+    lines.push(`OPENAI_API_KEY=${apiKey}`);
+
+    await fs.mkdir(directory, { recursive: true, mode: 0o700 });
+    const temporary = `${envPath}.tmp-${process.pid}`;
+    await fs.writeFile(temporary, lines.join('\n') + '\n', { encoding: 'utf8', mode: 0o600 });
+    await fs.rename(temporary, envPath);
+    await fs.chmod(envPath, 0o600);
+    process.env.OPENAI_API_KEY = apiKey;
+    return envPath;
+}
+
+module.exports = { getEnvPath, validateApiKey, saveApiKey };