smart-review 1.0.3 → 1.2.0

package/README.en-US.md

@@ -7,7 +7,8 @@
 ## ✨ Features
 
 - Static rule checks for Security, Performance, Best Practices
-- AI analysis (OpenAI-compatible) with incremental Git Diff review
+- AI analysis with unified OpenAI / Anthropic / Gemini integration
+- Skill-driven review orchestration to enforce detailed risk and fix outputs
 - Smart batching and chunked processing for large files
 - Git Hook integration (pre-commit)
 - Highly configurable and multilingual output
@@ -124,6 +125,11 @@ Add to `package.json` if you use Husky:
 }
 ```
 
+#### Interrupt & Terminal Compatibility
+- Works in Git Bash, CMD, and PowerShell
+- Press `q` or `Esc` during review to interrupt and print completed results
+- Interruptions do not fail the review; only blocking risks stop the commit
+
 ## ⚙️ Config
 
 Main config `.smart-review/smart-review.json` example:
@@ -132,6 +138,7 @@ Main config `.smart-review/smart-review.json` example:
 {
   "ai": {
     "enabled": true,
+    "provider": "openai",
     "model": "deepseek-chat",
     "baseURL": "https://api.deepseek.com/v1",
     "reviewOnlyChanges": true,
@@ -141,6 +148,37 @@ Main config `.smart-review/smart-review.json` example:
     "useStaticHints": true,
     "maxRequestTokens": 8000,
     "temperature": 0,
+    "skills": {
+      "enabled": true,
+      "strict": false,
+      "maxSkillsPerRequest": 4,
+      "required": ["DIFF_RISK_GUARD", "EVIDENCE_ENFORCER"],
+      "optional": ["SECURITY_DEEP", "LOGIC_CORRECTNESS", "API_CONTRACT"],
+      "routes": [
+        {
+          "match": ["**/auth/**", "**/security/**"],
+          "modes": ["diff", "batch", "segment"],
+          "add": ["SECURITY_DEEP", "API_CONTRACT"]
+        }
+      ]
+    },
+    "tools": {
+      "enabled": false,
+      "maxCalls": 2,
+      "maxReadLines": 400,
+      "maxSearchMatches": 50,
+      "maxSearchFiles": 120,
+      "maxListFiles": 200,
+      "allow": [
+        "read_file",
+        "get_staged_diff",
+        "list_files",
+        "search_in_file",
+        "get_file_outline",
+        "search_in_repo",
+        "list_changed_files"
+      ]
+    },
     "concurrency": 3
   },
   "riskLevels": {
@@ -158,8 +196,9 @@ Main config `.smart-review/smart-review.json` example:
 
 #### AI (`ai`)
 - `enabled`: Enable AI analysis
-- `model`: OpenAI model name
-- `apiKey`: OpenAI API key
+- `provider`: Model provider, supports `openai`, `anthropic`, `gemini`
+- `model`: Model name for the selected provider
+- `apiKey`: Unified API key field (or use environment variables)
 - `baseURL`: API base URL
 - `reviewOnlyChanges`: Enable Git Diff incremental review; true analyzes only changed lines
 - `maxResponseTokens`: Max tokens in AI response
@@ -173,7 +212,28 @@ Main config `.smart-review/smart-review.json` example:
 - `includeStaticHints`: Include rule hints in AI analysis
 - `temperature`: Model creativity; 0 favors deterministic outputs
 - `concurrency`: Number of concurrent AI requests
-
+- `skills.enabled`: Enable review skill orchestration
+- `skills.strict`: Enforce output constraints (path/snippet/reason/suggestion)
+- `skills.maxSkillsPerRequest`: Max skills applied in one request
+- `skills.required`: Required skill list
+- `skills.optional`: Optional skill list (mode-based supplement)
+- `skills.routes`: Dynamically append skills by file path and mode (`match/modes/add`)
+- `tools.enabled`: Enable local read-only tool calling (see tool list below)
+- `tools.maxCalls`: Max tool-call rounds per request
+- `tools.maxReadLines`: Max lines for single `read_file` call
+- `tools.maxSearchMatches`: Max returned matches for `search_in_file` / `search_in_repo`
+- `tools.maxSearchFiles`: Max scanned files for one `search_in_repo` call
+- `tools.maxListFiles`: Max returned files for one `list_files` call
+- `tools.allow`: Tool allowlist (only listed tools can be called by the model)
+
+#### AI Read-Only Tool List (`ai.tools.allow`)
+- `read_file`: Read specific line ranges from one file
+- `get_staged_diff`: Get staged Git diff (optionally filtered by path)
+- `list_files`: Recursively list repository files (supports subdir and keyword/wildcard filtering)
+- `search_in_file`: Search text or regex in a single file
+- `get_file_outline`: Extract lightweight file outline (class/function/method signatures)
+- `search_in_repo`: Search text or regex across repository files (with scan and result caps)
+- `list_changed_files`: List changed Git files (supports staged/unstaged and status filtering)
 #### Risk Levels (`riskLevels`)
 - `critical` / `high` / `medium` / `low` / `suggestion`
 - Each level supports `block` to decide whether to block commits
@@ -530,7 +590,11 @@ To add a new language (e.g., `ja-JP`), create `templates/rules/ja-JP/` with the
 ## 🌍 Environment Variables
 
 ```bash
+export AI_API_KEY="your-api-key"
 export OPENAI_API_KEY="your-api-key"
+export ANTHROPIC_API_KEY="your-api-key"
+export GEMINI_API_KEY="your-api-key"
+export GOOGLE_API_KEY="your-api-key"
 export DEBUG_SMART_REVIEW=true
 export SMART_REVIEW_LOCALE=en-US
 ```
@@ -539,7 +603,7 @@ To use a custom OpenAI-compatible endpoint, set `ai.baseURL` in `.smart-review/s
 
 ```json
 {
-  "ai": { "baseURL": "https://api.openai.com/v1" }
+  "ai": { "provider": "openai", "baseURL": "https://api.openai.com/v1" }
 }
 ```
 
@@ -577,4 +641,4 @@ smart-review --staged --debug
 4. Push the branch (`git push origin feature/amazing-feature`)
 5. Open a Pull Request
 
-⭐ If this project helps you, please star the repo!
+⭐ If this project helps you, please star the repo!

package/README.md

@@ -7,7 +7,8 @@
 ## ✨ 特性
 
 - 🔍 **静态规则检测** - 内置安全、性能、最佳实践规则
-- 🧠 **AI智能分析** - 基于 OpenAI GPT 的深度代码分析
+- 🧠 **AI智能分析** - 支持 OpenAI / Anthropic / Gemini 的统一对接
+- 🧩 **审查技能编排** - 支持 Skills 约束输出，强制细化风险分析与修复建议
 - ⚡ **Git Diff增量审查** - 智能识别变更内容，只审查修改的代码行，大幅提升审查效率
 - 🚀 **智能分批处理** - 自动优化大文件处理，支持分段分析
 - 📊 **大文件支持** - 智能分段处理超大文件，突破token限制
@@ -131,6 +132,11 @@ node bin/review.js --files test/src/large-test-file.js
 }
 ```
 
+#### 中断与终端兼容
+- 支持在 Git Bash、CMD、PowerShell 中进行交互中断
+- 审查过程中输入 `q` 或按 `Esc` 可中断审查并输出已完成结果
+- 中断不会被视为审查失败，只有存在阻断风险才会阻止提交
+
 ## ⚙️ 配置文件
 
 ### 主配置文件 `.smart-review/smart-review.json`
@@ -139,6 +145,7 @@ node bin/review.js --files test/src/large-test-file.js
 {
   "ai": {
     "enabled": true,
+    "provider": "openai",
     "model": "deepseek-chat",
     "apiKey": "your-api-key",
     "baseURL": "https://api.deepseek.com/v1",
@@ -153,6 +160,37 @@ node bin/review.js --files test/src/large-test-file.js
     "tokenRatio": 4,
     "chunkOverlapLines": 5,
     "includeStaticHints": true,
+    "skills": {
+      "enabled": true,
+      "strict": false,
+      "maxSkillsPerRequest": 4,
+      "required": ["DIFF_RISK_GUARD", "EVIDENCE_ENFORCER"],
+      "optional": ["SECURITY_DEEP", "LOGIC_CORRECTNESS", "API_CONTRACT"],
+      "routes": [
+        {
+          "match": ["**/auth/**", "**/security/**"],
+          "modes": ["diff", "batch", "segment"],
+          "add": ["SECURITY_DEEP", "API_CONTRACT"]
+        }
+      ]
+    },
+    "tools": {
+      "enabled": false,
+      "maxCalls": 2,
+      "maxReadLines": 400,
+      "maxSearchMatches": 50,
+      "maxSearchFiles": 120,
+      "maxListFiles": 200,
+      "allow": [
+        "read_file",
+        "get_staged_diff",
+        "list_files",
+        "search_in_file",
+        "get_file_outline",
+        "search_in_repo",
+        "list_changed_files"
+      ]
+    },
     "temperature": 0,
     "concurrency": 3
   },
@@ -183,8 +221,9 @@ node bin/review.js --files test/src/large-test-file.js
 
 #### AI 配置 (`ai`)
 - `enabled`: 是否启用AI分析
-- `model`: OpenAI模型名称
-- `apiKey`: OpenAI API密钥
+- `provider`: 模型提供方，支持 `openai`、`anthropic`、`gemini`
+- `model`: 对应提供方的模型名称
+- `apiKey`: 统一 API 密钥字段（也可通过环境变量注入）
 - `baseURL`: API基础URL
 - `reviewOnlyChanges`: 是否启用Git Diff增量审查模式。`true`时只审查变更的代码行，`false`时审查整个文件内容。默认为`true`，大幅提升审查效率
 - `maxResponseTokens`: AI响应最大token数
@@ -199,6 +238,28 @@ node bin/review.js --files test/src/large-test-file.js
 - `includeStaticHints`: 是否在AI分析中包含静态规则提示
 - `temperature`: AI模型的创造性参数，0表示最确定性的输出
 - `concurrency`: 并发AI请求数量，默认3个。设置为1或更小时使用串行处理，大于1时启用并发处理以提升性能
+- `skills.enabled`: 是否启用审查技能编排
+- `skills.strict`: 是否强制检查输出是否满足“路径/片段/原因/建议”约束
+- `skills.maxSkillsPerRequest`: 单次请求最多启用的技能数量
+- `skills.required`: 必选技能列表
+- `skills.optional`: 可选技能列表（按模式补充）
+- `skills.routes`: 按文件路径和模式动态追加技能（`match/modes/add`）
+- `tools.enabled`: 启用本地只读工具调用（见下方工具清单）
+- `tools.maxCalls`: 单次请求最多工具调用轮次
+- `tools.maxReadLines`: `read_file` 单次读取最大行数
+- `tools.maxSearchMatches`: `search_in_file` / `search_in_repo` 单次最多返回匹配条数
+- `tools.maxSearchFiles`: `search_in_repo` 单次最多扫描文件数
+- `tools.maxListFiles`: `list_files` 单次最多返回文件数
+- `tools.allow`: 工具白名单（仅允许模型调用白名单中的工具）
+
+#### AI 只读工具清单 (`ai.tools.allow`)
+- `read_file`: 读取指定文件的行区间
+- `get_staged_diff`: 获取暂存区 diff（可按文件路径过滤）
+- `list_files`: 递归列出仓库内文件（支持子目录和关键字/通配符过滤）
+- `search_in_file`: 在单个文件内按文本或正则搜索
+- `get_file_outline`: 提取文件结构轮廓（类/函数/方法等）
+- `search_in_repo`: 在仓库范围内按文本或正则搜索（支持扫描文件数和结果数上限）
+- `list_changed_files`: 获取 Git 变更文件列表（支持 staged/unstaged 和状态过滤）
 
 #### 风险等级配置 (`riskLevels`)
 - `critical`: 致命风险
@@ -560,8 +621,15 @@ const reviewer = new CodeReviewer(customConfig, defaultRules);
 可通过环境变量配置：
 
 ```bash
-# OpenAI API配置
+# 统一 API 配置（最高优先级）
+export AI_API_KEY="your-api-key"
+
+# 按 Provider 配置
 export OPENAI_API_KEY="your-api-key"
+export ANTHROPIC_API_KEY="your-api-key"
+export GEMINI_API_KEY="your-api-key"
+# 或 Google 生态变量
+export GOOGLE_API_KEY="your-api-key"
 
 # 调试模式
 export DEBUG_SMART_REVIEW=true
@@ -578,7 +646,7 @@ export SMART_REVIEW_LOCALE=zh-CN  # 或 en-US
 
 ```json
 {
-  "ai": { "baseURL": "https://api.openai.com/v1" }
+  "ai": { "provider": "openai", "baseURL": "https://api.openai.com/v1" }
 }
 ```
 
@@ -740,13 +808,4 @@ smart-review --staged --debug
 4. 推送到分支 (`git push origin feature/amazing-feature`)
 5. 开启 Pull Request
 
-## 📞 支持
-
-- 📧 邮箱: zlife@vip.qq.com
-- 🐛 问题反馈: [GitHub Issues](https://github.com/vlinr/smart-review/issues)
-
----
-
 ⭐ 如果这个项目对你有帮助，请给个 Star！
-
-

package/bin/install.js

@@ -264,6 +264,8 @@ class Installer {
 
 echo "${t(loc, 'hook_start_review')}"
 
+trap 'echo "${t(loc, 'interrupt_cancelled')}"; exit 0' INT TERM
+
 # 获取暂存区文件
 STAGED_FILES=$(git diff --cached --name-only --diff-filter=ACM)
 
@@ -283,9 +285,14 @@ ROOT_BIN="$REPO_ROOT/node_modules/.bin/smart-review"
 
 FOUND_CMD=""
 FOUND_IS_ENTRY=0
+FOUND_ENTRY_CMD=""
 
 if [ -f "$ROOT_BIN" ]; then
   FOUND_CMD="$ROOT_BIN"
+  ROOT_ENTRY="$REPO_ROOT/node_modules/smart-review/bin/review.js"
+  if [ -f "$ROOT_ENTRY" ]; then
+    FOUND_ENTRY_CMD="$ROOT_ENTRY"
+  fi
 else
   MAX_ASCEND=6
   while IFS= read -r file; do
@@ -296,9 +303,13 @@ else
       candidate_bin="$REPO_ROOT/$dir/node_modules/.bin/smart-review"
       candidate_entry="$REPO_ROOT/$dir/node_modules/smart-review/bin/review.js"
       if [ -f "$candidate_bin" ]; then
-        FOUND_CMD="$candidate_bin"; FOUND_IS_ENTRY=0; break 2
+        FOUND_CMD="$candidate_bin"; FOUND_IS_ENTRY=0;
+        if [ -f "$candidate_entry" ]; then
+          FOUND_ENTRY_CMD="$candidate_entry"
+        fi
+        break 2
       elif [ -f "$candidate_entry" ]; then
-        FOUND_CMD="$candidate_entry"; FOUND_IS_ENTRY=1; break 2
+        FOUND_CMD="$candidate_entry"; FOUND_IS_ENTRY=1; FOUND_ENTRY_CMD="$candidate_entry"; break 2
       fi
       dir=$(dirname "$dir")
       depth=$((depth + 1))
@@ -319,13 +330,89 @@ if [ -z "$FOUND_CMD" ]; then
 fi
 
 echo "${t(loc, 'hook_use_command_prefix')} $FOUND_CMD --staged"
-if [ $FOUND_IS_ENTRY -eq 1 ]; then
-  node "$FOUND_CMD" --staged
+USE_WINPTY=0
+if command -v uname >/dev/null 2>&1; then
+  KERNEL=$(uname -s)
 else
-  "$FOUND_CMD" --staged
+  KERNEL=""
 fi
-
+IS_MSYS=0
+if [[ "$KERNEL" == MINGW* || "$KERNEL" == MSYS* || -n "$MSYSTEM" ]]; then
+  IS_MSYS=1
+fi
+HAS_TTY=0
+if [ -t 0 ] || [ -t 1 ]; then
+  HAS_TTY=1
+fi
+TTY_DEVICE=""
+if [ $HAS_TTY -eq 1 ] && [ -r /dev/tty ]; then
+  TTY_DEVICE="/dev/tty"
+elif [ $HAS_TTY -eq 1 ] && [ -r "CONIN$" ]; then
+  TTY_DEVICE="CONIN$"
+elif [ -r /dev/tty ]; then
+  TTY_DEVICE="/dev/tty"
+elif [ -r "CONIN$" ]; then
+  TTY_DEVICE="CONIN$"
+fi
+if [ -n "$TTY_DEVICE" ]; then
+  export SMART_REVIEW_TTY="$TTY_DEVICE"
+fi
+if [ $IS_MSYS -eq 1 ] && [ $HAS_TTY -eq 1 ]; then
+  if command -v winpty >/dev/null 2>&1; then
+    USE_WINPTY=1
+  fi
+fi
+run_direct() {
+  if [ $USE_WINPTY -eq 1 ]; then
+    if [ -n "$FOUND_ENTRY_CMD" ]; then
+      winpty node "$FOUND_ENTRY_CMD" --staged
+    else
+      winpty "$FOUND_CMD" --staged
+    fi
+  else
+    if [ -n "$FOUND_ENTRY_CMD" ]; then
+      node "$FOUND_ENTRY_CMD" --staged
+    else
+      "$FOUND_CMD" --staged
+    fi
+  fi
+}
+run_with_device() {
+  if [ -n "$TTY_DEVICE" ]; then
+    "$@" < "$TTY_DEVICE"
+  else
+    "$@"
+  fi
+}
+TMP_ERR=""
+if command -v mktemp >/dev/null 2>&1; then
+  TMP_ERR=$(mktemp -t smart-review-err.XXXXXX)
+else
+  TMP_ERR="/tmp/smart-review-err-$$"
+fi
+run_direct 2> "$TMP_ERR"
 EXIT_CODE=$?
+if [ $EXIT_CODE -ne 0 ] && [ -s "$TMP_ERR" ] && grep -qi "stdin is not a tty" "$TMP_ERR"; then
+  if [ $USE_WINPTY -eq 1 ]; then
+    if [ -n "$FOUND_ENTRY_CMD" ]; then
+      run_with_device winpty node "$FOUND_ENTRY_CMD" --staged
+    else
+      run_with_device winpty "$FOUND_CMD" --staged
+    fi
+  else
+    if [ -n "$FOUND_ENTRY_CMD" ]; then
+      run_with_device node "$FOUND_ENTRY_CMD" --staged
+    else
+      run_with_device "$FOUND_CMD" --staged
+    fi
+  fi
+  EXIT_CODE=$?
+fi
+rm -f "$TMP_ERR"
+if [ $EXIT_CODE -eq 130 ] || [ $EXIT_CODE -eq 143 ]; then
+  echo "${t(loc, 'interrupt_cancelled')}"
+  exit 0
+fi
 if [ $EXIT_CODE -ne 0 ]; then
   echo "${t(loc, 'hook_review_fail')}"
   exit 1
@@ -336,18 +423,98 @@ fi
 `;
 
     fs.writeFileSync(preCommitHook, hookContent);
-    // Windows 兼容：提供 CMD 包装器，调用 bash 执行同名脚本
+    const escapeCmdText = (value) => String(value).replace(/\\/g, '\\\\').replace(/'/g, "\\'");
+    const cmdMessages = {
+      start: escapeCmdText(t(loc, 'hook_start_review')),
+      noStaged: escapeCmdText(t(loc, 'hook_no_staged')),
+      foundHeader: escapeCmdText(t(loc, 'hook_found_staged_header')),
+      cdFail: escapeCmdText(t(loc, 'hook_cd_repo_fail')),
+      cmdNotFound1: escapeCmdText(t(loc, 'hook_cmd_not_found1')),
+      cmdNotFound2: escapeCmdText(t(loc, 'hook_cmd_not_found2')),
+      cmdMissingContinue: escapeCmdText(t(loc, 'hook_cmd_missing_continue')),
+      useCmdPrefix: escapeCmdText(t(loc, 'hook_use_command_prefix')),
+      reviewFail: escapeCmdText(t(loc, 'hook_review_fail')),
+      reviewPass: escapeCmdText(t(loc, 'hook_review_pass')),
+      interruptCancelled: escapeCmdText(t(loc, 'interrupt_cancelled'))
+    };
+    const cmdNodeScript = [
+      "const { execSync, spawnSync } = require('child_process');",
+      "const fs = require('fs');",
+      "const path = require('path');",
+      `const MSG = { start: '${cmdMessages.start}', noStaged: '${cmdMessages.noStaged}', foundHeader: '${cmdMessages.foundHeader}', cdFail: '${cmdMessages.cdFail}', cmdNotFound1: '${cmdMessages.cmdNotFound1}', cmdNotFound2: '${cmdMessages.cmdNotFound2}', cmdMissingContinue: '${cmdMessages.cmdMissingContinue}', useCmdPrefix: '${cmdMessages.useCmdPrefix}', reviewFail: '${cmdMessages.reviewFail}', reviewPass: '${cmdMessages.reviewPass}', interruptCancelled: '${cmdMessages.interruptCancelled}' };`,
+      "const log = (value) => { if (value) console.log(value); };",
+      "log(MSG.start);",
+      "let staged = '';",
+      "try { staged = execSync('git diff --cached --name-only --diff-filter=ACM', { encoding: 'utf8' }); } catch (e) {}",
+      "const stagedFiles = staged.split(/\\r?\\n/).filter(Boolean);",
+      "if (!stagedFiles.length) { log(MSG.noStaged); process.exit(0); }",
+      "log(MSG.foundHeader);",
+      "log(stagedFiles.join('\\n'));",
+      "let repoRoot = '';",
+      "try { repoRoot = execSync('git rev-parse --show-toplevel', { encoding: 'utf8' }).trim(); } catch (e) {}",
+      "if (!repoRoot) { log(MSG.cdFail); process.exit(1); }",
+      "const rootBin = path.join(repoRoot, 'node_modules', '.bin', 'smart-review');",
+      "const rootEntry = path.join(repoRoot, 'node_modules', 'smart-review', 'bin', 'review.js');",
+      "let foundCmd = '';",
+      "let foundEntry = '';",
+      "if (fs.existsSync(rootBin)) { foundCmd = rootBin; if (fs.existsSync(rootEntry)) { foundEntry = rootEntry; } }",
+      "if (!foundCmd) {",
+      "  const maxAscend = 6;",
+      "  outer: for (const file of stagedFiles) {",
+      "    let dir = path.dirname(file);",
+      "    let depth = 0;",
+      "    while (dir && dir !== '.' && depth < maxAscend) {",
+      "      const candidateBin = path.join(repoRoot, dir, 'node_modules', '.bin', 'smart-review');",
+      "      const candidateEntry = path.join(repoRoot, dir, 'node_modules', 'smart-review', 'bin', 'review.js');",
+      "      if (fs.existsSync(candidateBin)) { foundCmd = candidateBin; if (fs.existsSync(candidateEntry)) { foundEntry = candidateEntry; } break outer; }",
+      "      if (fs.existsSync(candidateEntry)) { foundCmd = candidateEntry; foundEntry = candidateEntry; break outer; }",
+      "      dir = path.dirname(dir);",
+      "      depth++;",
+      "    }",
+      "  }",
+      "}",
+      "if (!foundCmd) {",
+      "  try { execSync('where smart-review', { stdio: 'ignore' }); foundCmd = 'smart-review'; } catch (e) {}",
+      "}",
+      "if (!foundCmd) { log(MSG.cmdNotFound1); log(MSG.cmdNotFound2); log(MSG.cmdMissingContinue); process.exit(0); }",
+      "log(MSG.useCmdPrefix + ' ' + foundCmd + ' --staged');",
+      "const hasTty = !!(process.stdin && process.stdin.isTTY) || !!(process.stdout && process.stdout.isTTY);",
+      "const tryTty = (p) => { try { const fd = fs.openSync(p, 'r'); fs.closeSync(fd); return p; } catch (e) { return ''; } };",
+      "const ensureTty = () => {",
+      "  if (process.env.SMART_REVIEW_TTY) return;",
+      "  let tty = '';",
+      "  if (hasTty) { tty = tryTty('\\\\\\\\.\\\\CONIN$') || tryTty('CONIN$'); }",
+      "  if (!tty) { tty = tryTty('\\\\\\\\.\\\\CONIN$') || tryTty('CONIN$'); }",
+      "  if (tty) { process.env.SMART_REVIEW_TTY = tty; process.env.SMART_REVIEW_FORCE_TTY = '1'; }",
+      "};",
+      "const args = ['--staged'];",
+      "const runOnce = (capture) => {",
+      "  if (foundEntry) {",
+      "    return spawnSync(process.execPath, [foundEntry, ...args], { stdio: capture ? 'pipe' : 'inherit' });",
+      "  }",
+      "  return spawnSync(foundCmd, args, { stdio: capture ? 'pipe' : 'inherit', shell: true });",
+      "};",
+      "let result = runOnce(true);",
+      "if (result.stdout) process.stdout.write(result.stdout);",
+      "if (result.stderr) process.stderr.write(result.stderr);",
+      "let code = Number.isInteger(result.status) ? result.status : (Number.isInteger(result.code) ? result.code : 0);",
+      "const errText = result.stderr ? String(result.stderr) : '';",
+      "if (code !== 0 && /stdin is not a tty/i.test(errText)) {",
+      "  ensureTty();",
+      "  result = runOnce(false);",
+      "  code = Number.isInteger(result.status) ? result.status : (Number.isInteger(result.code) ? result.code : 0);",
+      "}",
+      "if (code === 130 || code === 143) { log(MSG.interruptCancelled); process.exit(0); }",
+      "if (code !== 0) { log(MSG.reviewFail); process.exit(1); }",
+      "log(MSG.reviewPass); process.exit(0);"
+    ].join('');
+    // Windows 兼容：提供 CMD 包装器，避免 bash 在 CMD 下的 TTY 问题
     try {
       const preCommitCmd = path.join(gitHooksDir, 'pre-commit.cmd');
       const cmdContent = [
         '@echo off',
         'SETLOCAL',
-        'set HOOK=%~dp0pre-commit',
-        'if not exist "%HOOK%" (',
-        '  echo [smart-review] pre-commit hook missing.',
-        '  exit /b 1',
-        ')',
-        'bash "%HOOK%"',
+        `node -e "${cmdNodeScript}"`,
         'exit /b %ERRORLEVEL%\r\n'
       ].join('\r\n');
       fs.writeFileSync(preCommitCmd, cmdContent);
@@ -416,4 +583,4 @@ fi
 
 // 运行安装
 const installer = new Installer();
-(async () => { await installer.install(); })();
+(async () => { await installer.install(); })();