codemini-cli 0.2.5 → 0.2.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codemini-cli",
-  "version": "0.2.5",
+  "version": "0.2.7",
   "description": "Coding CLI optimized for small-model workflows and Windows PowerShell",
   "keywords": [
     "cli",

package/src/cli.js

@@ -4,7 +4,7 @@ import { handleConfig } from './commands/config.js';
 import { handleDoctor } from './commands/doctor.js';
 import { handleSkill } from './commands/skill.js';
 
-const VERSION = '0.2.5';
+const VERSION = '0.2.7';
 
 function printHelp() {
   console.log(`codemini ${VERSION}

package/src/core/agent-loop.js

@@ -7,7 +7,10 @@ function safeJsonParse(raw) {
   try {
     return JSON.parse(raw);
   } catch {
-    return {};
+    return {
+      _raw: String(raw),
+      _invalid_json: true
+    };
   }
 }
 

package/src/core/provider/openai-compatible.js

@@ -36,6 +36,38 @@ function isMiniMaxModel(model) {
   return String(model || '').toLowerCase().includes('minimax');
 }
 
+function normalizeToolCallArguments(argumentsText) {
+  const raw = typeof argumentsText === 'string' ? argumentsText : JSON.stringify(argumentsText ?? {});
+  try {
+    const parsed = JSON.parse(raw);
+    if (parsed && typeof parsed === 'object' && !Array.isArray(parsed)) {
+      return JSON.stringify(parsed);
+    }
+  } catch {}
+  return '{}';
+}
+
+function sanitizeGatewayMessages(messages) {
+  const source = Array.isArray(messages) ? messages : [];
+  return source
+    .filter((message) => message && typeof message === 'object')
+    .map((message) => {
+      if (!Array.isArray(message.tool_calls) || message.tool_calls.length === 0) {
+        return message;
+      }
+      return {
+        ...message,
+        tool_calls: message.tool_calls.map((toolCall) => ({
+          ...toolCall,
+          function: {
+            ...toolCall?.function,
+            arguments: normalizeToolCallArguments(toolCall?.function?.arguments)
+          }
+        }))
+      };
+    });
+}
+
 function sanitizeMiniMaxMessages(messages) {
   const source = Array.isArray(messages) ? messages : [];
   const out = [];
@@ -64,10 +96,11 @@ function sanitizeMiniMaxMessages(messages) {
 }
 
 function buildPayload({ model, temperature, messages, tools, stream = false }) {
+  const sanitizedMessages = sanitizeGatewayMessages(messages);
   const payload = {
     model,
     temperature,
-    messages: isMiniMaxModel(model) ? sanitizeMiniMaxMessages(messages) : messages
+    messages: isMiniMaxModel(model) ? sanitizeMiniMaxMessages(sanitizedMessages) : sanitizedMessages
   };
   if (stream) {
     payload.stream = true;

package/src/core/shell-profile.js

@@ -142,6 +142,10 @@ Some tools are loaded on demand. If a needed tool is not listed, call tool_searc
 
 # Doing tasks
 
+- You are a terminal-first CLI coding agent, not a generic chat assistant
+- The user shares your workspace with you; prefer inspecting the project yourself before asking them to paste files that should be discoverable
+- Before substantial tool work, send a short progress update to the user about what you are about to inspect or do
+- Do not jump straight into tools without a brief user-facing note when the task is actionable
 - Search or read before editing unless the exact target is already known
 - If a command or tool is blocked or fails, inspect the error and retry with allowed commands or tools
 - For AST-scoped edits, if edit rejects due to missing or stale ast_target, fix arguments and retry

package/src/core/tools.js

@@ -755,7 +755,7 @@ async function readFile(root, args) {
 }
 
 async function writeFile(root, args) {
-  const rawPath = String(args?.path || '').trim();
+  const rawPath = String(args?.path || args?.file_path || '').trim();
   if (!rawPath) {
     throw new Error('write requires a file path like weather/WeatherForecast.js');
   }
@@ -1536,7 +1536,7 @@ async function openTarget(root, args) {
 }
 
 function normalizeEditTargetArgs(args = {}) {
-  const file = String(args?.file || args?.path || '').trim();
+  const file = String(args?.file || args?.path || args?.file_path || '').trim();
   const nestedEdit = args?.edit && typeof args.edit === 'object' ? args.edit : null;
   if (nestedEdit) {
     const normalizedEdit = { ...nestedEdit };
@@ -1561,6 +1561,8 @@ function normalizeEditTargetArgs(args = {}) {
       new_content: args?.new_content ?? args?.content,
       old_text: args?.old_text,
       new_text: args?.new_text,
+      old_string: args?.old_string,
+      new_string: args?.new_string,
       anchor_text: args?.anchor_text,
       content: args?.content
     }
@@ -1573,6 +1575,12 @@ async function editTarget(root, args) {
   const astTarget = normalized.ast_target;
   const edit = normalized.edit || {};
   let kind = String(edit.kind || '').trim();
+  if (edit.old_text == null && edit.old_string != null) {
+    edit.old_text = edit.old_string;
+  }
+  if (edit.new_text == null && edit.new_string != null) {
+    edit.new_text = edit.new_string;
+  }
   const hasContent = edit.new_content != null || edit.content != null;
   const hasTargetHint = Boolean(edit.symbol || args?.symbol || edit.line || args?.line || edit.target);
   if (!kind) {
@@ -1586,7 +1594,14 @@ async function editTarget(root, args) {
       kind = 'rewrite_file';
     }
   }
-  if (!file || !kind) throw new Error('edit requires file and edit.kind');
+  if (!file || !kind) {
+    const recentFile = String(args?.recent_file || '').trim();
+    const rawArgs = typeof args?._raw === 'string' && args._raw.trim() ? ` Raw tool arguments: ${args._raw.trim()}.` : '';
+    const hint = recentFile
+      ? ` If you meant the recently read file ${recentFile}, use edit with {file:"${recentFile}", old_text:"...", new_text:"..."} for a text replacement, or {file:"${recentFile}", edit:{kind:"rewrite_file", new_content:"..."}} for a full rewrite.`
+      : ' Use edit with {file:"path", old_text:"...", new_text:"..."} for a text replacement, or {file:"path", edit:{kind:"rewrite_file", new_content:"..."}} for a full rewrite.';
+    throw new Error(`edit requires file and edit.kind.${rawArgs}${hint}`);
+  }
   if (astTarget) {
     if (kind !== 'replace_block') {
       throw new Error('AST-scoped edit only supports replace_block');
@@ -1659,6 +1674,7 @@ export function getBuiltinTools({ workspaceRoot = process.cwd(), config, onSyste
   };
   const astSelectionCache = new Map();
   let lastAstTarget = null;
+  let lastReadPath = '';
   const rememberAstSelection = (filePath, astTarget) => {
     const key = String(filePath || '').trim();
     if (!key || !astTarget) return;
@@ -1812,15 +1828,18 @@ export function getBuiltinTools({ workspaceRoot = process.cwd(), config, onSyste
       function: {
         name: 'edit',
         description:
-          'Edit existing files. Use block edits, exact replacements, or anchored inserts. When ast_target is provided, keep the edit constrained to that node. Read first unless the exact target is already known. Prefer this over write for code changes.',
+          'Edit existing files. Prefer one of these shapes: 1) {file, old_text, new_text} for exact text replacement, 2) {file, symbol, edit:{kind:"replace_block", new_content:"..."}} for block replacement, 3) {file, anchor_text, position:"before"|"after", content:"..."} for inserts. Demo-style aliases {file_path, old_string, new_string} are also accepted. Read first unless the exact target is already known. Prefer this over write for existing code changes.',
         parameters: {
           type: 'object',
           properties: {
             file: { type: 'string', description: 'File path to edit' },
             path: { type: 'string', description: 'Alias for file' },
+            file_path: { type: 'string', description: 'Alias for file, compatible with simpler demo-style tool calls' },
             new_content: { type: 'string', description: 'Replacement content' },
             old_text: { type: 'string', description: 'Exact text to replace' },
             new_text: { type: 'string', description: 'Replacement text' },
+            old_string: { type: 'string', description: 'Alias for old_text' },
+            new_string: { type: 'string', description: 'Alias for new_text' },
             anchor_text: { type: 'string', description: 'Anchor text for inserts' },
             content: { type: 'string', description: 'Content to insert or append' },
             position: { type: 'string', description: 'before or after' },
@@ -1840,11 +1859,12 @@ export function getBuiltinTools({ workspaceRoot = process.cwd(), config, onSyste
       function: {
         name: 'write',
         description:
-          'Create a new file or overwrite a file. Always include path and content. Use this for new files or explicit full rewrites only. If the file path is not decided yet, do not call write yet. Prefer edit for existing code changes.',
+          'Create a new file or overwrite a file. Always include path (or file_path) and content. Use this for new files or explicit full rewrites only. Example: {path:"src/page.html", content:"..."} . If the file path is not decided yet, do not call write yet. Prefer edit for existing code changes.',
         parameters: {
           type: 'object',
           properties: {
             path: { type: 'string', description: 'Required file path like src/app.js or pages/index.html. Never omit this.' },
+            file_path: { type: 'string', description: 'Alias for path, compatible with simpler demo-style tool calls' },
             content: { type: 'string', description: 'Content to write' },
             append: { type: 'boolean', description: 'Append instead of overwrite' },
             full_file_rewrite: { type: 'boolean', description: 'Set true for whole-file rewrites' }
@@ -2049,6 +2069,10 @@ export function getBuiltinTools({ workspaceRoot = process.cwd(), config, onSyste
           typeof args?.max_chars === 'number'
             ? args.max_chars
             : config.context?.read_file_max_chars ?? 24000
+      }).then((result) => {
+        const readPath = String(result?.path || args?.path || '').trim();
+        if (readPath) lastReadPath = readPath;
+        return result;
       }),
     grep: (args) => grep(workspaceRoot, args),
     glob: (args) => glob(workspaceRoot, args),
@@ -2069,7 +2093,10 @@ export function getBuiltinTools({ workspaceRoot = process.cwd(), config, onSyste
       await ensureProjectIndex();
       const normalizedKind = String(args?.edit?.kind || args?.kind || '').trim();
       const astTarget = resolveCachedAstTarget(args, { requireAstScope: normalizedKind === 'replace_block' });
-      const result = await editTarget(workspaceRoot, astTarget ? { ...args, ast_target: astTarget } : args);
+      const result = await editTarget(
+        workspaceRoot,
+        astTarget ? { ...args, ast_target: astTarget, recent_file: lastReadPath } : { ...args, recent_file: lastReadPath }
+      );
       if (result?.path) await refreshProjectFile(result.path);
       return result;
     },

package/src/tui/chat-app.js

@@ -375,6 +375,39 @@ function isCodeGenerationActivityName(name) {
   return String(name || '').trim() === 'Code generation';
 }
 
+export function buildPreToolNotice(name, copy) {
+  const parsed = parseToolDisplayName(name);
+  const base = parsed.base;
+  const target = parsed.target ? trimText(parsed.target, 48) : '';
+  const isEnglish = String(copy?.roleLabels?.coder || '').trim() === 'CODER' && String(copy?.roleLabels?.you || '').trim() === 'YOU';
+
+  if (isEnglish) {
+    if (base === 'read') return target ? `I'll inspect ${target} first.` : `I'll inspect the relevant file first.`;
+    if (base === 'list' || base === 'glob') return target ? `I'll inspect the ${target} directory first.` : `I'll inspect the relevant directory first.`;
+    if (base === 'grep') return `I'll search the relevant code first.`;
+    if (base === 'edit' || base === 'write' || base === 'patch' || base === 'generate_diff') {
+      return `I'll inspect the current code first, then make the change.`;
+    }
+    if (base === 'run') return `I'll verify the current project state first.`;
+    return `I'll check the relevant project context first.`;
+  }
+
+  if (base === 'read') return target ? `我先查看 ${target} 的内容。` : '我先查看相关文件内容。';
+  if (base === 'list' || base === 'glob') return target ? `我先查看 ${target} 目录里的内容。` : '我先查看相关目录内容。';
+  if (base === 'grep') return '我先搜索相关代码位置。';
+  if (base === 'edit' || base === 'write' || base === 'patch' || base === 'generate_diff') return '我先确认当前代码上下文，再动手修改。';
+  if (base === 'run') return '我先检查当前项目状态。';
+  return '我先查看相关上下文。';
+}
+
+export function shouldInjectPreToolNotice(msg) {
+  if (!msg) return false;
+  const text = String(msg.text || '').trim();
+  const segments = Array.isArray(msg.segments) ? msg.segments : [];
+  const hasTextSegment = segments.some((segment) => segment?.type === 'text' && String(segment.text || '').trim());
+  return !text && !hasTextSegment;
+}
+
 function formatDurationMs(ms) {
   const safeMs = Math.max(0, Number(ms) || 0);
   return `${(safeMs / 1000).toFixed(1)}s`;
@@ -2280,11 +2313,12 @@ export function ChatApp({ runtime, sessionId, model, language = 'zh', shellName
         } else {
           segments.push({ type: 'text', text: delta });
         }
-        const nextText = `${m.text}${delta}`;
+        const nextText = m.syntheticPrelude ? delta : `${m.text}${delta}`;
         return {
           ...m,
           text: nextText,
-          segments
+          segments,
+          syntheticPrelude: false
         };
       })
     );
@@ -2634,8 +2668,15 @@ export function ChatApp({ runtime, sessionId, model, language = 'zh', shellName
               prev.map((m) => {
                 if (m.id !== targetId) return m;
                 const nextMessage = isCodeActivityName(event.name) ? finishCodeGeneration(m, finishedAt) : m;
+                const withPrelude = shouldInjectPreToolNotice(nextMessage)
+                  ? {
+                      ...nextMessage,
+                      text: buildPreToolNotice(event.name, copy),
+                      syntheticPrelude: true
+                    }
+                  : nextMessage;
                 return {
-                  ...nextMessage,
+                  ...withPrelude,
                   loading: true,
                   phase: 'tooling',
                   liveStatus: detail