ai-helper-agent 1.14.0 → 1.14.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ai-helper-agent",
-  "version": "1.14.0",
+  "version": "1.14.2",
   "description": "AI Helper 代理服务 - 为 Chrome 扩展提供本地文件读写和系统命令执行能力",
   "type": "module",
   "main": "./src/server.js",

package/src/executor.js

@@ -86,6 +86,12 @@ const SAFE_ENV_KEYS = [
   'GIT_EDITOR', 'GIT_PAGER', 'GIT_SSH_COMMAND',
   // 平台相关
   'TMPDIR', 'TEMPDIR', 'TEMP', 'TMP',
+  // Windows 必要环境变量（cmd/powershell 及其子进程依赖）
+  // 缺失 COMSPEC/PATHEXT/SystemRoot 会导致 cmd 命令解析异常、npm 等工具找不到；
+  // 缺失 PSModulePath 会导致 PowerShell 找不到模块
+  'COMSPEC', 'PATHEXT', 'SystemRoot', 'WINDIR',
+  'APPDATA', 'LOCALAPPDATA', 'USERPROFILE', 'PROGRAMDATA',
+  'PSModulePath', 'NUMBER_OF_PROCESSORS', 'PROCESSOR_ARCHITECTURE',
   // 常用工具
   'NVM_DIR', 'NVM_BIN', 'JAVA_HOME', 'GOPATH', 'GOROOT', 'CARGO_HOME',
   'RUSTUP_HOME', 'PYTHONPATH', 'VIRTUAL_ENV', 'CONDA_PREFIX',
@@ -134,11 +140,23 @@ function executeCommand(command, cwd, wsClient, onComplete, collectOutput = fals
   const timeout = config.commandTimeout || 300000;
 
   const { shell, args } = getShellForExec();
+  // detached 的取舍（按平台 + shell 类型）：
+  // - Unix（macOS/Linux）：任何 shell（zsh/bash/fish 等）都不弹窗（无控制台窗口概念），
+  //   且 detached 才能让 killProcess 的 process.kill(-pid) 杀掉整个进程组 → 始终启用。
+  // - Windows：分两类 shell——
+  //   1) MSYS/Cygwin 系（Git Bash/MSYS2/Cygwin 的 bash/zsh/fish 等）：通过 pty 模拟层，
+  //      不依赖 Windows 控制台，detached 不会 AllocConsole → 不弹窗，可安全 detach。
+  //   2) Windows 原生控制台程序（cmd/powershell/pwsh 等）：detached 脱离父控制台后会
+  //      AllocConsole 创建可见窗口（黑窗），windowsHide 管不住 → 必须禁用 detached。
+  //      进程树终止改靠 taskkill /T /PID（killProcess Windows 分支），不依赖 detached。
+  //   用黑名单精确识别会弹窗的原生控制台程序，其余（含 MSMS/Cygwin 系）默认 detach。
+  const isWin = os.platform() === 'win32';
+  const isWinConsoleShell = isWin && /\b(cmd|powershell|pwsh)\b/i.test(shell);
   const proc = spawn(shell, [...args, command], {
     cwd: workdir,
     env: buildSafeEnv(),
     stdio: ['ignore', 'pipe', 'pipe'],
-    detached: true,       // 独立进程组，防止 kill 父进程后子进程成为孤儿
+    detached: !isWinConsoleShell,
     windowsHide: true
   });
 

package/src/security.js

@@ -6,14 +6,22 @@ import { loadConfig, MEMORY_DIR } from './config.js';
 
 const isWin32 = platform() === 'win32';
 
-function normalizePathFormat(pathStr) {
+/**
+ * 路径格式规整（跨平台）
+ * @param {string} pathStr - 待规整的路径
+ * @param {string} [platformOverride] - 平台覆盖（'win32' | 'darwin' | 'linux'），用于单元测试 mock；
+ *   不传则使用当前运行平台 os.platform()
+ * @returns {string} 规整后的路径
+ */
+function normalizePathFormat(pathStr, platformOverride) {
   if (!pathStr || typeof pathStr !== 'string') return pathStr;
+  const isWin = platformOverride !== undefined ? platformOverride === 'win32' : isWin32;
   // 1. 把异常的 "/C:/" 形式（带前导斜杠的盘符）规整为 "C:/"
   //    仅匹配 /<字母>:/ ，不会误伤 /home/ 等普通路径段
   pathStr = pathStr.replace(/\/([a-zA-Z]):(\/|$)/g, '$1:$2');
   // 2. MSYS/Git Bash 风格路径：/d/Users/... → D:/Users/...
   //    前导 / + 单字母盘符 + / ，无冒号，是 MSYS 特有的挂载点表示法
-  if (isWin32) {
+  if (isWin) {
     pathStr = pathStr.replace(/^\/([a-zA-Z])\//, '$1:/');
   }
   // 3. 统一为正斜杠（path.resolve/normalize 在所有平台都能正确处理正斜杠并归一化到平台分隔符）
@@ -21,7 +29,7 @@ function normalizePathFormat(pathStr) {
   // 4. Windows 盘符统一为大写：d:/ → D:/
   //    path.resolve 保留盘符原始大小写，而白名单前缀比较是大小写敏感的字符串比较，
   //    小写盘符路径与大写盘符白名单前缀比较会失败（d:\Users\.startsWith(D:\Users\) → false）
-  if (isWin32) {
+  if (isWin) {
     pathStr = pathStr.replace(/^([a-zA-Z]):/, m => m.toUpperCase());
   }
   return pathStr;
@@ -44,17 +52,22 @@ const HARD_BLOCKED_PATHS = [
 
 /**
  * 检查路径是否命中核心敏感文件
+ * 注意：Windows 文件系统不区分大小写，必须做大小写不敏感比较，
+ *       否则 .AI-HELPER-AGENT/config.JSON 等大小写变换可绕过硬阻止
  */
-function isHardBlocked(normalizedPath) {
+function isHardBlocked(normalizedPath, platformOverride) {
+  const isWin = platformOverride !== undefined ? platformOverride === 'win32' : isWin32;
+  const cmpPath = isWin ? normalizedPath.toLowerCase() : normalizedPath;
   for (const blocked of HARD_BLOCKED_PATHS) {
+    const cmpBlocked = isWin ? blocked.toLowerCase() : blocked;
     if (blocked.endsWith(sep)) {
       // 目录：检查是否在该目录下
-      if (normalizedPath.startsWith(blocked) || normalizedPath === blocked.slice(0, -1)) {
+      if (cmpPath.startsWith(cmpBlocked) || cmpPath === cmpBlocked.slice(0, -1)) {
         return true;
       }
     } else {
       // 文件：精确匹配
-      if (normalizedPath === blocked) {
+      if (cmpPath === cmpBlocked) {
         return true;
       }
     }
@@ -314,4 +327,4 @@ function checkCommand(command, force = false) {
   return { safe: true, level: 'allow' };
 }
 
-export { checkPath, checkCommand };
+export { checkPath, checkCommand, normalizePathFormat, isHardBlocked };

package/src/server.js

@@ -12,7 +12,7 @@ import { randomBytes } from 'crypto';
 import os from 'os';
 import { loadConfig, saveConfig } from './config.js';
 import { verifyToken, startPairCodeRotation, stopPairCodeRotation, handlePairRequest } from './auth.js';
-import { checkPath, checkCommand } from './security.js';
+import { checkPath, checkCommand, normalizePathFormat } from './security.js';
 import { moveToTrash, restoreFromTrash, listTrash, startPeriodicCleanup, stopPeriodicCleanup } from './trash.js';
 import { executeCommand, executeCommandSync, addWsClient, disconnectWsClient, killProcess, getRunningProcesses } from './executor.js';
 import { setConsoleOutput, logAuth, logFs, logExec, logSecurity, logSystem, logError, queryLogs, getLogDates } from './logger.js';
@@ -801,6 +801,9 @@ export function startServer() {
         } else if (resolvedWorkdir.startsWith('~/')) {
           resolvedWorkdir = join(homedir(), resolvedWorkdir.slice(2));
         }
+        // 规整路径格式（MSYS /d/Users/... → D:/Users/...、盘符大写统一），
+        // 必须在 isAbsolute/resolve 之前，否则 Windows 上 /d/Users 会被当作 Unix 绝对路径错误解析
+        resolvedWorkdir = normalizePathFormat(resolvedWorkdir);
 
         // 必须是绝对路径（跨平台：path.isAbsolute 覆盖 Unix /、Windows 盘符、UNC 路径）
         if (!isAbsolute(resolvedWorkdir)) {
@@ -857,6 +860,8 @@ export function startServer() {
         } else if (resolvedPath.startsWith('~/')) {
           resolvedPath = join(homedir(), resolvedPath.slice(2));
         }
+        // 规整 MSYS 路径格式，与 workdir 切换端点保持一致
+        resolvedPath = normalizePathFormat(resolvedPath);
         if (!isAbsolute(resolvedPath)) {
           return jsonResponse(res, 400, { success: false, error: 'path 必须是绝对路径' });
         }
@@ -1385,6 +1390,10 @@ export function startServer() {
         if (newName.includes('/') || newName.includes('\\')) {
           return jsonResponse(res, 400, { success: false, error: '新名称不能包含路径分隔符' });
         }
+        // Windows 文件名非法字符（仅 Windows 拦截；Unix 允许这些字符）
+        if (os.platform() === 'win32' && /[:*?"<>|]/.test(newName)) {
+          return jsonResponse(res, 400, { success: false, error: '新名称包含 Windows 非法字符 (: * ? " < > |)' });
+        }
         const check = await checkPath(oldPath);
         if (!check.allowed) {
           logSecurity('fs_rename_blocked', { path: oldPath, reason: check.reason });
@@ -1400,6 +1409,12 @@ export function startServer() {
           logSecurity('fs_rename_blocked', { path: newFullPath, reason: newCheck.reason });
           return jsonResponse(res, 403, { success: false, error: `目标路径不允许: ${newCheck.reason}` });
         }
+        // 防止覆盖：目标已存在时统一返回 409。
+        // macOS/Linux 的 fs.rename 会静默覆盖目标文件（仅 Windows 报 EEXIST），
+        // 主动检查避免跨平台数据丢失风险
+        if (await exists(newFullPath)) {
+          return jsonResponse(res, 409, { success: false, error: '目标名称已存在' });
+        }
         try {
           await rename(check.resolved, newFullPath);
           logFs('rename', { from: check.resolved, to: newFullPath });
@@ -1443,6 +1458,10 @@ export function startServer() {
           logSecurity('fs_move_blocked', { path: destPath, reason: destCheck2.reason });
           return jsonResponse(res, 403, { success: false, error: `移动目标路径不允许: ${destCheck2.reason}` });
         }
+        // 防止覆盖：目标已存在时统一返回 409（与 rename 端点一致，避免 macOS/Linux 静默覆盖）
+        if (await exists(destPath)) {
+          return jsonResponse(res, 409, { success: false, error: '目标位置已存在同名文件/目录' });
+        }
         try {
           await rename(srcCheck.resolved, destPath);
           logFs('move', { from: srcCheck.resolved, to: destPath });