miii-cli 1.2.1 → 1.2.2

package/dist/__tests__/integration.test.js

@@ -3,7 +3,7 @@ import { writeFileSync, unlinkSync, readFileSync } from 'fs';
 import { join } from 'path';
 import { looksCodeRelated } from '../tui/git-context.js';
 import { tools } from '../tools/index.js';
-// patch_file uses guardPath which restricts to CWD — use a local scratch file
+// update_file uses guardPath which restricts to CWD — use a local scratch file
 const SCRATCH = join(process.cwd(), '.miii-test-scratch.txt');
 // ─── looksCodeRelated ─────────────────────────────────────────────────────────
 describe('looksCodeRelated', () => {
@@ -23,9 +23,9 @@ describe('looksCodeRelated', () => {
         expect(looksCodeRelated('what is the weather like in london today')).toBe(false);
     });
 });
-// ─── patch_file ───────────────────────────────────────────────────────────────
-describe('patch_file', () => {
-    const patchTool = tools.find(t => t.name === 'patch_file');
+// ─── update_file ───────────────────────────────────────────────────────────────
+describe('update_file', () => {
+    const updateTool = tools.find(t => t.name === 'update_file');
     afterEach(() => {
         try {
             unlinkSync(SCRATCH);
@@ -34,17 +34,17 @@ describe('patch_file', () => {
     });
     it('applies a unique patch correctly', async () => {
         writeFileSync(SCRATCH, 'hello world\ngoodbye world\n');
-        await patchTool.execute({ path: SCRATCH, old: 'hello world', new: 'hello earth' });
+        await updateTool.execute({ path: SCRATCH, old: 'hello world', new: 'hello earth' });
         expect(readFileSync(SCRATCH, 'utf-8')).toBe('hello earth\ngoodbye world\n');
     });
     it('throws when old text not found', async () => {
         writeFileSync(SCRATCH, 'hello world\n');
-        await expect(patchTool.execute({ path: SCRATCH, old: 'no such text', new: 'x' }))
+        await expect(updateTool.execute({ path: SCRATCH, old: 'no such text', new: 'x' }))
             .rejects.toThrow('old text not found');
     });
     it('throws on ambiguous match (2+ occurrences)', async () => {
         writeFileSync(SCRATCH, 'hello world\nhello world\n');
-        await expect(patchTool.execute({ path: SCRATCH, old: 'hello world', new: 'hi' }))
+        await expect(updateTool.execute({ path: SCRATCH, old: 'hello world', new: 'hi' }))
             .rejects.toThrow('ambiguous');
     });
 });

package/dist/memory/extractor.js

@@ -6,7 +6,7 @@ Skip: trivial exchanges, transient state, tool output noise.
 Max 8 facts. Be specific and concrete.
 
 Example output:
-["User prefers patch_file over full rewrites","entry point is src/index.ts","decided to use Zod for validation"]`;
+["User prefers update_file over full rewrites","entry point is src/index.ts","decided to use Zod for validation"]`;
 export function extractFacts(messages, config, model) {
     const lines = messages
         .filter(m => m.role !== 'system')

package/dist/parser/stream-parser.js

@@ -98,7 +98,7 @@ function extractFileToolArgs(text, toolName) {
                 .replace(/\\"/g, '"').replace(/\\\\/g, '\\');
         }
     }
-    // For patch_file: extract old/new fields
+    // For update_file: extract old/new fields
     const oldM = text.match(/"old"\s*:\s*"((?:[^"\\]|\\[\s\S])*)"/);
     if (oldM)
         args.old = oldM[1].replace(/\\n/g, '\n').replace(/\\"/g, '"').replace(/\\\\/g, '\\');
@@ -127,7 +127,7 @@ export function extractBareToolCall(text) {
         pos = start + 1;
     }
     // Fallback: content-aware extraction for file-writing tools (immune to unescaped chars)
-    for (const name of ['edit_file', 'create_file', 'patch_file']) {
+    for (const name of ['edit_file', 'create_file', 'update_file']) {
         const args = extractFileToolArgs(text, name);
         if (args)
             return { name, args };

package/dist/skills/loader.js

@@ -79,6 +79,11 @@ const builtin = [
             return `moved: ${from} → ${to}`;
         },
     },
+    {
+        name: 'test',
+        ns: 'builtin',
+        description: 'Run test suite — usage: /test [path]',
+    },
     {
         name: 'touch',
         ns: 'default',

package/dist/tools/index.js

@@ -46,13 +46,13 @@ export const tools = [
     },
     {
         name: 'edit_file',
-        description: 'Write a new file — only for files that do not exist yet. Use patch_file to modify existing files.',
+        description: 'Write a new file — only for files that do not exist yet. Use update_file to modify existing files.',
         params: '{"path": "string", "content": "string"}',
         execute: async ({ path, content }) => {
             const safe = guardPath(path);
             if (existsSync(safe)) {
                 throw new Error(`edit_file cannot overwrite existing file: ${path}\n` +
-                    `Use patch_file with <old> and <new> blocks to make targeted edits.\n` +
+                    `Use update_file with <old> and <new> blocks to make targeted edits.\n` +
                     `Call read_file first to get the exact current text.`);
             }
             const text = content;
@@ -62,7 +62,7 @@ export const tools = [
         },
     },
     {
-        name: 'patch_file',
+        name: 'update_file',
         description: 'Replace an exact unique string in an existing file. Always call read_file first to get the exact text.',
         params: '{"path": "string", "old": "string", "new": "string"}',
         execute: async ({ path, old: oldStr, new: newStr }) => {
@@ -295,9 +295,9 @@ full file content here
 </content>
 </tool_call>
 
-For patch_file use <old> and <new> blocks:
+For update_file use <old> and <new> blocks:
 <tool_call>
-{"name": "patch_file", "args": {"path": "src/foo.ts"}}
+{"name": "update_file", "args": {"path": "src/foo.ts"}}
 <old>
 exact text to replace
 </old>
@@ -312,9 +312,9 @@ ${deepThinkDoc}
 
 Rules:
 - edit_file only works on NEW files — it throws an error if the file exists. Never call it on existing files
-- To modify any existing file: call read_file first, then patch_file with the exact text from that read as the <old> block
+- To modify any existing file: call read_file first, then update_file with the exact text from that read as the <old> block
 - Never guess or reuse old text from earlier in the conversation — always re-read immediately before patching
-- If patch_file reports "old text not found", call read_file again and retry with the exact current text
+- If update_file reports "old text not found", call read_file again and retry with the exact current text
 - Never delete without confirming
 - Use git_status and git_diff before any refactor to understand what has already changed
 - Use git_log to understand recent history before suggesting changes
@@ -322,7 +322,7 @@ Rules:
 - Be concise
 - Output plain text only — never use markdown formatting in your responses
 - No headers (no #, ##), no bold (**text**), no italic (*text*), no bullet points with *, no horizontal rules (---)
-- NEVER show file content or code in your text response — always use edit_file, patch_file, or create_file tools to write code to files
+- NEVER show file content or code in your text response — always use edit_file, update_file, or create_file tools to write code to files
 - If you want to show the user code, write it to the file with a tool call instead
 - No fenced code blocks (no \`\`\`). If you find yourself about to write a code block, use a tool call instead
 - Use plain indentation and labels for structure. This is a terminal, not a chat UI

package/dist/tui/InputBar.js

@@ -64,7 +64,7 @@ function diffHunks(diff) {
     return diff.filter((_, i) => inHunk.has(i));
 }
 function DiffPreview({ toolName, args }) {
-    if (toolName === 'patch_file' && (args.old != null || args.new != null)) {
+    if (toolName === 'update_file' && (args.old != null || args.new != null)) {
         const path = String(args.path ?? '');
         const diff = diffHunks(lineDiff(String(args.old ?? ''), String(args.new ?? '')));
         const visible = diff.slice(0, MAX_DIFF_LINES);

package/dist/tui/hooks/useRunLoop.js

@@ -6,10 +6,10 @@ import { StreamParser, extractBareToolCall } from '../../parser/stream-parser.js
 import { shouldCompact, compactContext, contextSize } from '../../tasks/compactor.js';
 import * as printer from '../printer.js';
 const MAX_TOOL_DEPTH = 10;
-const FILE_EDIT_TOOLS = new Set(['edit_file', 'create_file', 'patch_file', 'delete_file']);
+const FILE_EDIT_TOOLS = new Set(['edit_file', 'create_file', 'update_file', 'delete_file']);
 const SHOW_RESULT_TOOLS = new Set(['run_tests', 'git_commit']);
-const PERMISSION_TOOLS = new Set(['edit_file', 'patch_file', 'delete_file', 'create_file', 'move_file', 'run_command', 'git_commit']);
-const CHECKPOINT_TOOLS = new Set(['edit_file', 'patch_file', 'create_file', 'delete_file']);
+const PERMISSION_TOOLS = new Set(['edit_file', 'update_file', 'delete_file', 'create_file', 'move_file', 'run_command', 'git_commit']);
+const CHECKPOINT_TOOLS = new Set(['edit_file', 'update_file', 'create_file', 'delete_file']);
 // Tool result messages that are ephemeral — never worth storing in memory or compact summaries
 const EPHEMERAL_PATTERN = /^Tool (read_file|list_files|run_tests) result:|^\[current state of|^\[Context compacted/;
 export function stripEphemeral(messages) {
@@ -115,7 +115,7 @@ export function useRunLoop(config, currentModelRef, pushHistory, extraTools = []
                     if (hasFencedCode && depth < MAX_TOOL_DEPTH - 1) {
                         const nudge = {
                             role: 'user',
-                            content: 'You showed code in your response but did not use any file tools. Use edit_file or patch_file to actually write the changes to disk.',
+                            content: 'You showed code in your response but did not use any file tools. Use edit_file or update_file to actually write the changes to disk.',
                         };
                         await runLoop([...msgs, { role: 'assistant', content: fullText }, nudge], depth + 1, goal);
                         return;
@@ -165,9 +165,9 @@ export function useRunLoop(config, currentModelRef, pushHistory, extraTools = []
                         }
                         if (tool) {
                             try {
-                                // Guard: for patch_file, verify old text still matches before executing.
+                                // Guard: for update_file, verify old text still matches before executing.
                                 // If stale, inject fresh file content and skip — model will retry.
-                                if (tc.name === 'patch_file') {
+                                if (tc.name === 'update_file') {
                                     const filePath = tc.args.path;
                                     const oldText = tc.args.old;
                                     if (filePath && oldText && existsSync(filePath)) {
@@ -176,13 +176,13 @@ export function useRunLoop(config, currentModelRef, pushHistory, extraTools = []
                                         if (occurrences === 0) {
                                             printer.errorMsg(`patch stale: old text not found in ${filePath} — injecting fresh content`);
                                             next.push({ role: 'user', content: `Tool read_file result:\n${current}` });
-                                            next.push({ role: 'user', content: `patch_file failed: old text not found in ${filePath}. The file content above is the current state. Retry patch_file with the correct exact text.` });
+                                            next.push({ role: 'user', content: `update_file failed: old text not found in ${filePath}. The file content above is the current state. Retry update_file with the correct exact text.` });
                                             continue;
                                         }
                                         if (occurrences > 1) {
                                             printer.errorMsg(`patch ambiguous: old text matches ${occurrences} locations in ${filePath} — injecting fresh content`);
                                             next.push({ role: 'user', content: `Tool read_file result:\n${current}` });
-                                            next.push({ role: 'user', content: `patch_file failed: old text matches ${occurrences} locations in ${filePath}. Use more surrounding context to make old text unique, then retry.` });
+                                            next.push({ role: 'user', content: `update_file failed: old text matches ${occurrences} locations in ${filePath}. Use more surrounding context to make old text unique, then retry.` });
                                             continue;
                                         }
                                     }
@@ -217,33 +217,9 @@ export function useRunLoop(config, currentModelRef, pushHistory, extraTools = []
                 finally {
                     setCurrentTool(undefined);
                 }
-                // Auto-run tests after file edits
-                const didEditFiles = pendingTools.some(tc => FILE_EDIT_TOOLS.has(tc.name));
-                if (didEditFiles) {
-                    const testTool = staticTools.find(t => t.name === 'run_tests');
-                    if (testTool) {
-                        setCurrentTool('run_tests');
-                        try {
-                            printer.toolCallStart('run_tests', {});
-                            const testResult = await testTool.execute({});
-                            if (testResult && !testResult.startsWith('(no test script') && !testResult.startsWith('(no package.json')) {
-                                printer.toolResultSummary('run_tests', {}, testResult);
-                                printer.toolMsg('run_tests', testResult);
-                                next.push({ role: 'user', content: `Test results after edits:\n${testResult}` });
-                            }
-                        }
-                        catch (e) {
-                            const err = `run_tests error: ${e}`;
-                            printer.errorMsg(err);
-                            next.push({ role: 'user', content: err });
-                        }
-                        finally {
-                            setCurrentTool(undefined);
-                        }
-                    }
-                }
                 // For file-edit turns: slim context (system + goal + fresh file states + recent results)
                 // For non-edit turns: full next (model needs full conversational context)
+                const didEditFiles = pendingTools.some(tc => FILE_EDIT_TOOLS.has(tc.name));
                 if (didEditFiles) {
                     const systemMsg = msgs.find(m => m.role === 'system');
                     const goalMsg = msgs.find(m => m.role === 'user' && !m.content.startsWith('[') && !m.content.startsWith('Tool '));

package/dist/tui/hooks/useSubmit.js

@@ -431,6 +431,30 @@ export function useSubmit(deps) {
             printer.systemMsg('usage: /index build | /index status | /index search <query> | /index clear');
             return;
         }
+        if (cmd === '/test' || cmd.startsWith('/test ')) {
+            const testPath = cmd.slice(5).trim();
+            const testTool = (await import('../../tools/index.js')).tools.find(t => t.name === 'run_tests');
+            if (!testTool) {
+                printer.errorMsg('run_tests tool not found');
+                return;
+            }
+            setStatus('tool');
+            setCurrentTool('run_tests');
+            try {
+                printer.toolCallStart('run_tests', testPath ? { path: testPath } : {});
+                const result = await testTool.execute(testPath ? { path: testPath } : {});
+                printer.toolResultSummary('run_tests', {}, result);
+                printer.toolMsg('run_tests', result);
+            }
+            catch (e) {
+                printer.errorMsg(`run_tests: ${e}`);
+            }
+            finally {
+                setCurrentTool(undefined);
+                setStatus('idle');
+            }
+            return;
+        }
         if (cmd === '/watch' || cmd.startsWith('/watch ')) {
             const sub = cmd.slice(6).trim();
             if (sub === 'stop') {

package/dist/tui/printer.js

@@ -1,4 +1,5 @@
 // ANSI-formatted stdout output — goes into terminal scrollback
+import { readFileSync, existsSync } from 'fs';
 let _inkWrite = null;
 export function setInkInstance(inkWrite) {
     _inkWrite = inkWrite;
@@ -158,7 +159,7 @@ export function assistantMsg(text) {
     const tail = lines.slice(idx + 1).join('\n');
     write(`\n${blue('●')} ${head}${tail ? '\n' + tail : ''}\n`);
 }
-const EDIT_TOOLS = new Set(['edit_file', 'patch_file', 'create_file', 'write_file']);
+const EDIT_TOOLS = new Set(['edit_file', 'update_file', 'create_file', 'write_file']);
 const DELETE_TOOLS = new Set(['delete_file', 'remove_file']);
 function toolLabel(name, args) {
     const a = args;
@@ -168,7 +169,7 @@ function toolLabel(name, args) {
         case 'list_files': return `Listing ${a.path || '.'}`;
         case 'create_file': return `Creating ${a.path ?? ''}`;
         case 'edit_file': return `Writing ${a.path ?? ''}`;
-        case 'patch_file': return `Editing ${a.path ?? ''}`;
+        case 'update_file': return `Updating ${a.path ?? ''}`;
         case 'delete_file': return `Deleting ${a.path ?? ''}`;
         case 'move_file': return `Moving ${a.from} → ${a.to}`;
         case 'create_folder': return `Creating folder ${a.path ?? ''}`;
@@ -199,9 +200,63 @@ export function planSummary(tools) {
         write(`  ${dot} ${gray(label)}\n`);
     }
 }
+const DIFF_CTX = 2;
+const DIFF_MAX = 40;
+function printUpdateDiff(filePath, oldText, newText) {
+    const oldLines = oldText.split('\n');
+    const newLines = newText.split('\n');
+    write(gray(`  └ Added ${newLines.length} line${newLines.length !== 1 ? 's' : ''}, removed ${oldLines.length} line${oldLines.length !== 1 ? 's' : ''}\n`));
+    let fileLines = [];
+    let lineOffset = 0;
+    try {
+        if (existsSync(filePath)) {
+            const content = readFileSync(filePath, 'utf-8');
+            fileLines = content.split('\n');
+            const idx = content.indexOf(oldText);
+            if (idx >= 0)
+                lineOffset = content.slice(0, idx).split('\n').length - 1;
+            else
+                fileLines = []; // old text not in file — skip context lines
+        }
+    }
+    catch { }
+    let shown = 0;
+    const ctxStart = Math.max(0, lineOffset - DIFF_CTX);
+    for (let i = ctxStart; i < lineOffset && shown < DIFF_MAX; i++, shown++) {
+        write(gray(`  ${String(i + 1).padStart(4)}    ${fileLines[i] ?? ''}\n`));
+    }
+    for (let i = 0; i < oldLines.length && shown < DIFF_MAX; i++, shown++) {
+        write(`  ${gray(String(lineOffset + i + 1).padStart(4))} ${red('- ')}${red(oldLines[i])}\n`);
+    }
+    for (let i = 0; i < newLines.length && shown < DIFF_MAX; i++, shown++) {
+        write(`  ${gray(String(lineOffset + i + 1).padStart(4))} ${green('+ ')}${green(newLines[i])}\n`);
+    }
+    const ctxEnd = Math.min(fileLines.length, lineOffset + oldLines.length + DIFF_CTX);
+    for (let i = lineOffset + oldLines.length; i < ctxEnd && shown < DIFF_MAX; i++, shown++) {
+        write(gray(`  ${String(i + 1).padStart(4)}    ${fileLines[i] ?? ''}\n`));
+    }
+}
+function printEditPreview(content) {
+    const lines = content.split('\n');
+    const visible = lines.slice(0, DIFF_MAX);
+    const hidden = lines.length - visible.length;
+    write(gray(`  └ ${lines.length} line${lines.length !== 1 ? 's' : ''}\n`));
+    visible.forEach((line, i) => {
+        write(`  ${gray(String(i + 1).padStart(4))} ${green('+ ')}${green(line)}\n`);
+    });
+    if (hidden > 0)
+        write(gray(`  …${hidden} more line${hidden !== 1 ? 's' : ''}\n`));
+}
 export function toolCallStart(name, args) {
     const dot = DELETE_TOOLS.has(name) ? red('●') : EDIT_TOOLS.has(name) ? green('●') : blue('●');
     write(`\n${dot} ${bold(toolLabel(name, args))}\n`);
+    const a = args;
+    if (name === 'update_file' && a.old && a.new && a.path) {
+        printUpdateDiff(a.path, a.old, a.new);
+    }
+    else if (name === 'edit_file' && a.content && a.path) {
+        printEditPreview(a.content);
+    }
 }
 export function toolResultSummary(name, args, result) {
     const a = args;
@@ -219,7 +274,7 @@ export function toolResultSummary(name, args, result) {
             summary = `Created file · ${n} line${n === 1 ? '' : 's'}`;
             break;
         }
-        case 'patch_file':
+        case 'update_file':
             summary = lines[0] ?? 'Applied patch';
             break;
         case 'delete_file':

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "miii-cli",
-  "version": "1.2.1",
+  "version": "1.2.2",
   "type": "module",
   "description": "The high-performance local AI coding agent for your terminal. Automate complex workflows with local LLMs.",
   "license": "MIT",