codemini-cli 0.2.4 → 0.2.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codemini-cli",
-  "version": "0.2.4",
+  "version": "0.2.6",
   "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.4';
+const VERSION = '0.2.6';
 
 function printHelp() {
   console.log(`codemini ${VERSION}

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

@@ -1840,11 +1840,11 @@ export function getBuiltinTools({ workspaceRoot = process.cwd(), config, onSyste
       function: {
         name: 'write',
         description:
-          'Create a new file or overwrite a file. Use this for new files or explicit full rewrites. Prefer edit for existing code changes.',
+          '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.',
         parameters: {
           type: 'object',
           properties: {
-            path: { type: 'string', description: 'File path to create or overwrite' },
+            path: { type: 'string', description: 'Required file path like src/app.js or pages/index.html. Never omit this.' },
             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' }

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