codemini-cli 0.2.5 → 0.2.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codemini-cli",
-  "version": "0.2.5",
+  "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.5';
+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/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