codemini-cli 0.1.12 → 0.1.14

package/src/core/config-store.js

@@ -1,6 +1,7 @@
 import fs from 'node:fs/promises';
 import path from 'node:path';
 import { getConfigFilePath, getLegacyConfigDir } from './paths.js';
+import { normalizeReplyLanguage } from './reply-language.js';
 import { normalizeShellName } from './shell-profile.js';
 
 function normalizeUiLanguage(value) {
@@ -32,7 +33,28 @@ const DEFAULT_CONFIG = {
   },
   execution: {
     mode: 'auto',
-    always_allow_tools: ['run_command', 'read_file', 'write_file'],
+    always_allow_tools: [
+      'locate',
+      'open_target',
+      'edit_target',
+      'search_code',
+      'read_block',
+      'read_symbol_context',
+      'validate_edit',
+      'replace_block',
+      'replace_text',
+      'insert_before',
+      'insert_after',
+      'generate_diff',
+      'start_service',
+      'list_services',
+      'get_service_status',
+      'get_service_logs',
+      'stop_service',
+      'run_command',
+      'read_file',
+      'write_file'
+    ],
     max_steps: 16
   },
   sessions: {
@@ -44,7 +66,8 @@ const DEFAULT_CONFIG = {
     timeout_ms: 120000
   },
   ui: {
-    language: 'zh'
+    language: 'zh',
+    reply_language: 'zh'
   },
   soul: {
     preset: 'default',
@@ -110,10 +133,33 @@ function normalizePolicyLists(config) {
     ? next.execution.always_allow_tools
     : [];
   next.execution.always_allow_tools = uniqueStrings(
-    ['run_command', 'read_file', 'write_file', ...rawTools].filter((name) => String(name) !== 'list_files')
+    [
+      'locate',
+      'open_target',
+      'edit_target',
+      'search_code',
+      'read_block',
+      'read_symbol_context',
+      'validate_edit',
+      'replace_block',
+      'replace_text',
+      'insert_before',
+      'insert_after',
+      'generate_diff',
+      'start_service',
+      'list_services',
+      'get_service_status',
+      'get_service_logs',
+      'stop_service',
+      'run_command',
+      'read_file',
+      'write_file',
+      ...rawTools
+    ].filter((name) => String(name) !== 'list_files')
   );
   next.ui = next.ui || {};
   next.ui.language = normalizeUiLanguage(next.ui.language);
+  next.ui.reply_language = normalizeReplyLanguage(next.ui.reply_language);
   next.policy = next.policy || {};
   next.policy.command_allowlist = uniqueStrings(
     Array.isArray(next.policy.command_allowlist) ? next.policy.command_allowlist : []

package/src/core/reply-language.js

@@ -0,0 +1,25 @@
+export function normalizeReplyLanguage(value) {
+  const raw = String(value || '').trim().toLowerCase();
+  if (!raw) return 'zh';
+  if (['en', 'en-us', 'en_us', 'english'].includes(raw)) return 'en';
+  if (['zh', 'zh-cn', 'zh_cn', 'cn', 'chinese', '中文', '简体中文'].includes(raw)) return 'zh';
+  return 'zh';
+}
+
+export function buildSystemPromptWithReplyLanguage(baseSystemPrompt, config = {}) {
+  const replyLanguage = normalizeReplyLanguage(config?.ui?.reply_language);
+  const directive =
+    replyLanguage === 'en'
+      ? [
+          '[Reply language]',
+          'Respond in English.',
+          'Write generated documentation, user-facing text, and code comments in English unless the user explicitly asks for a different language.'
+        ].join('\n')
+      : [
+          '[Reply language]',
+          'Respond in Simplified Chinese.',
+          'Write generated documentation, user-facing text, and code comments in Simplified Chinese unless the user explicitly asks for a different language.'
+        ].join('\n');
+
+  return `${String(baseSystemPrompt || '').trim()}\n\n${directive}`.trim();
+}

package/src/core/shell-profile.js

@@ -118,5 +118,5 @@ export function getEffectivePolicy(config) {
 
 export function getShellSystemPrompt(value) {
   const profile = getShellProfile(value);
-  return `You are CodeMini CLI working in a ${profile.label} shell environment. For codebase exploration, use the allowed shell search and context commands that best fit the task, then use read_file only when command output is not enough. Use write_file for edits and always provide a concrete file path, not a directory. Avoid unnecessary tool calls.`;
+  return `You are CodeMini CLI working in a ${profile.label} shell environment. Prefer the high-level structured workflow first: use locate to find candidates, open_target to inspect the smallest useful block and receive edit metadata, and edit_target to apply minimal edits. When you need lower-level control, use search_code, read_block, read_symbol_context, validate_edit, replace_block, replace_text, insert_before, insert_after, and generate_diff. Use start_service, list_services, get_service_status, get_service_logs, and stop_service for long-running servers or watchers. Use run_command only for one-shot commands that should exit on their own. Use read_file only when structured reads are not enough. Use write_file only for full-file writes and always provide a concrete file path, not a directory. Avoid unnecessary tool calls.`;
 }

package/src/core/shell.js

@@ -1,6 +1,79 @@
-import { spawn } from 'node:child_process';
+import { spawn, spawnSync } from 'node:child_process';
 
-function resolveShell(defaultShell) {
+const LONG_RUNNING_COMMAND_RE =
+  /\b(npm\s+(?:run\s+)?(?:start|dev)\b|pnpm\s+(?:run\s+)?(?:start|dev)\b|yarn\s+(?:start|dev)\b|vite\b|next\s+dev\b|webpack\s+serve\b|python\s+-m\s+http\.server\b|serve\b|java\s+-jar\b|mvn(?:w)?\s+spring-boot:run\b|gradle(?:w)?\s+bootRun\b|gradle(?:w)?\s+run\b|java\b.*\bserver\b|dotnet\s+run\b|go\s+run\b.*\b(server|cmd\/server|main\.go)\b|air\b|cargo\s+run\b.*\b(server|api|web)\b|cargo\s+watch\s+-x\s+run\b)/i;
+const GENERIC_LONG_RUNNING_HINT_RE = /\b(start|serve|server|dev|preview|watch)\b/i;
+const READY_OUTPUT_PATTERNS = [
+  /\bready\b/i,
+  /\bcompiled successfully\b/i,
+  /\blocal:\s*https?:\/\//i,
+  /\blistening on\b/i,
+  /\bserver running\b/i,
+  /\brunning at\b/i,
+  /\bserving at\b/i,
+  /\bstarted\b.*\bin\b/i,
+  /\bstarted\s+[A-Za-z0-9_$.-]+\s+in\b/i,
+  /\btomcat started on port\(s\):/i,
+  /\bnetty started on port/i,
+  /\bnow listening on:\s*https?:\/\//i,
+  /\bapplication started\./i,
+  /\bserving http on\b/i,
+  /\bstarting development server at\b/i,
+  /\bactix web server running on\b/i,
+  /\bhttp:\/\/127\.0\.0\.1\b/i,
+  /\bhttp:\/\/localhost\b/i
+];
+const AUTO_STOP_GRACE_MS = 150;
+const LONG_RUNNING_STARTUP_WINDOW_MS = 1500;
+
+export function isLikelyLongRunningCommand(command) {
+  const value = String(command || '');
+  return LONG_RUNNING_COMMAND_RE.test(value) || GENERIC_LONG_RUNNING_HINT_RE.test(value);
+}
+
+export function hasReadyOutput(text) {
+  const value = String(text || '');
+  return READY_OUTPUT_PATTERNS.some((pattern) => pattern.test(value));
+}
+
+function collectDescendantPids(rootPid, seen = new Set()) {
+  const pid = Number(rootPid);
+  if (!Number.isInteger(pid) || pid <= 0 || seen.has(pid) || process.platform === 'win32') {
+    return [];
+  }
+  seen.add(pid);
+  const result = spawnSync('ps', ['-o', 'pid=', '--ppid', String(pid)], { encoding: 'utf8' });
+  if (result.status !== 0 || !result.stdout) return [];
+  const directChildren = result.stdout
+    .split('\n')
+    .map((value) => Number(String(value || '').trim()))
+    .filter((value) => Number.isInteger(value) && value > 0);
+  const descendants = [];
+  for (const childPid of directChildren) {
+    if (seen.has(childPid)) continue;
+    descendants.push(childPid);
+    descendants.push(...collectDescendantPids(childPid, seen));
+  }
+  return descendants;
+}
+
+export function terminateChild(child, signal = 'SIGTERM') {
+  if (!child) return;
+  const pid = Number(child.pid);
+  if (process.platform !== 'win32' && Number.isInteger(pid) && pid > 0) {
+    const descendants = collectDescendantPids(pid);
+    for (const targetPid of descendants.reverse()) {
+      try {
+        process.kill(targetPid, signal);
+      } catch {}
+    }
+  }
+  try {
+    child.kill(signal);
+  } catch {}
+}
+
+export function resolveShell(defaultShell) {
   if (process.platform === 'win32') {
     if (defaultShell === 'cmd') {
       return { command: 'cmd.exe', args: ['/d', '/s', '/c'] };
@@ -30,9 +103,13 @@ export function runShellCommand({
   timeoutMs = 120000
 }) {
   const shellSpec = resolveShell(shell);
+  const shellCommand =
+    process.platform !== 'win32' && /(?:^|\/)bash(?:\.exe)?$/i.test(shellSpec.command)
+      ? `exec ${command}`
+      : command;
 
   return new Promise((resolve, reject) => {
-    const child = spawn(shellSpec.command, [...shellSpec.args, command], {
+    const child = spawn(shellSpec.command, [...shellSpec.args, shellCommand], {
       cwd,
       stdio: ['ignore', 'pipe', 'pipe']
     });
@@ -40,32 +117,81 @@ export function runShellCommand({
     let stdout = '';
     let stderr = '';
     let timedOut = false;
+    let autoStopped = false;
+    let stopReason = '';
+    let finalized = false;
+    const longRunningCommand = isLikelyLongRunningCommand(command);
+    const autoStopWindowMs = longRunningCommand
+      ? Math.min(LONG_RUNNING_STARTUP_WINDOW_MS, Math.max(350, Math.floor(timeoutMs * 0.6)))
+      : 0;
+
+    const finalizeResolve = (value) => {
+      if (finalized) return;
+      finalized = true;
+      clearTimeout(timer);
+      if (autoStopTimer) clearTimeout(autoStopTimer);
+      resolve(value);
+    };
+
+    const finalizeReject = (error) => {
+      if (finalized) return;
+      finalized = true;
+      clearTimeout(timer);
+      if (autoStopTimer) clearTimeout(autoStopTimer);
+      reject(error);
+    };
 
     const timer = setTimeout(() => {
       timedOut = true;
-      child.kill('SIGTERM');
+      terminateChild(child, 'SIGTERM');
     }, timeoutMs);
+    const autoStopTimer =
+      autoStopWindowMs > 0
+        ? setTimeout(() => {
+            finalizeAutoStop('startup_window');
+          }, autoStopWindowMs)
+        : null;
+
+    const finalizeAutoStop = (reason) => {
+      if (timedOut || autoStopped || finalized) return;
+      autoStopped = true;
+      stopReason = reason;
+      terminateChild(child, 'SIGTERM');
+      setTimeout(() => {
+        terminateChild(child, 'SIGKILL');
+      }, AUTO_STOP_GRACE_MS);
+      finalizeResolve({ code: 0, stdout, stderr, auto_stopped: true, stop_reason: stopReason });
+    };
 
     child.stdout.on('data', (chunk) => {
       stdout += chunk.toString();
+      if (longRunningCommand && hasReadyOutput(stdout)) {
+        finalizeAutoStop('ready_output');
+      }
     });
 
     child.stderr.on('data', (chunk) => {
       stderr += chunk.toString();
+      if (longRunningCommand && hasReadyOutput(stderr)) {
+        finalizeAutoStop('ready_output');
+      }
     });
 
     child.on('error', (err) => {
-      clearTimeout(timer);
-      reject(err);
+      finalizeReject(err);
     });
 
     child.on('close', (code) => {
-      clearTimeout(timer);
+      if (finalized) return;
       if (timedOut) {
-        reject(new Error(`Command timed out after ${timeoutMs}ms`));
+        finalizeReject(new Error(`Command timed out after ${timeoutMs}ms`));
+        return;
+      }
+      if (autoStopped) {
+        finalizeResolve({ code: 0, stdout, stderr, auto_stopped: true, stop_reason: stopReason });
         return;
       }
-      resolve({ code, stdout, stderr });
+      finalizeResolve({ code, stdout, stderr });
     });
   });
 }

package/src/core/soul.js

@@ -2,6 +2,7 @@ import fs from 'node:fs/promises';
 import path from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { getBaseConfigDir } from './paths.js';
+import { buildSystemPromptWithReplyLanguage } from './reply-language.js';
 
 const MODULE_DIR = path.dirname(fileURLToPath(import.meta.url));
 const BUNDLED_SOULS_DIR = path.resolve(MODULE_DIR, '..', '..', 'souls');
@@ -45,11 +46,12 @@ export async function loadSoulPrompt(config = {}) {
 }
 
 export async function buildSystemPromptWithSoul(baseSystemPrompt, config = {}) {
+  const promptWithReplyLanguage = buildSystemPromptWithReplyLanguage(baseSystemPrompt, config);
   const soulPrompt = await loadSoulPrompt(config);
   const guard = [
     '[Soul guard]',
     'Apply this soul to response tone only.',
     'Response tone only: do not change plans, code, tests, file formats, or technical decisions.'
   ].join('\n');
-  return `${String(baseSystemPrompt || '').trim()}\n\n${guard}\n\n${soulPrompt}`.trim();
+  return `${String(promptWithReplyLanguage || '').trim()}\n\n${guard}\n\n${soulPrompt}`.trim();
 }