code-abyss 1.7.0 → 1.7.1

package/skills/tools/lib/shared.js

@@ -0,0 +1,98 @@
+'use strict';
+
+/**
+ * 验证工具共享库
+ * 消灭 verify-* 脚本间的重复代码
+ */
+
+// --- CLI 参数解析 ---
+
+function parseCliArgs(argv, extraFlags) {
+  const args = argv.slice(2);
+  const result = { target: '.', verbose: false, json: false };
+  if (extraFlags) Object.assign(result, extraFlags);
+
+  for (let i = 0; i < args.length; i++) {
+    if (args[i] === '-v' || args[i] === '--verbose') result.verbose = true;
+    else if (args[i] === '--json') result.json = true;
+    else if (args[i] === '-h' || args[i] === '--help') { result.help = true; }
+    else if (args[i] === '--mode' && args[i + 1]) { result.mode = args[++i]; }
+    else if (args[i] === '--exclude') {
+      result.exclude = result.exclude || [];
+      while (i + 1 < args.length && !args[i + 1].startsWith('-')) result.exclude.push(args[++i]);
+    }
+    else if (!args[i].startsWith('-')) result.target = args[i];
+  }
+  return result;
+}
+
+// --- 报告格式化 ---
+
+const SEP = '='.repeat(60);
+const DASH = '-'.repeat(40);
+const ICONS = {
+  error: '\u2717', warning: '\u26A0', info: '\u2139',
+  critical: '\u{1F534}', high: '\u{1F7E0}', medium: '\u{1F7E1}', low: '\u{1F535}'
+};
+
+function reportHeader(title, fields) {
+  const lines = [SEP, title, SEP];
+  for (const [k, v] of Object.entries(fields)) {
+    lines.push(`\n${k}: ${v}`);
+  }
+  return lines;
+}
+
+function reportIssues(issues, verbose, groupBy) {
+  if (!issues.length) return [];
+  const lines = ['\n' + DASH, '问题列表:', DASH];
+
+  if (groupBy) {
+    const groups = {};
+    for (const i of issues) (groups[i[groupBy]] || (groups[i[groupBy]] = [])).push(i);
+    for (const cat of Object.keys(groups).sort()) {
+      const items = groups[cat];
+      lines.push(`\n【${cat}】(${items.length} 个)`);
+      for (const i of items.slice(0, 10)) {
+        lines.push(`  ${ICONS[i.severity] || '\u2139'} ` +
+          `${i.file_path || ''}${i.line_number ? ':' + i.line_number : ''}`);
+        lines.push(`    ${i.message}`);
+        if (verbose && i.suggestion) lines.push(`    \u{1F4A1} ${i.suggestion}`);
+        if (verbose && i.recommendation) lines.push(`    \u{1F4A1} ${i.recommendation}`);
+      }
+      if (items.length > 10) lines.push(`  ... 及其他 ${items.length - 10} 个问题`);
+    }
+  } else {
+    for (const i of issues) {
+      const icon = ICONS[i.severity] || '\u2139';
+      lines.push(`  ${icon} [${i.severity.toUpperCase()}] ${i.message}`);
+      if (i.path && verbose) lines.push(`    路径: ${i.path}`);
+    }
+  }
+  return lines;
+}
+
+function reportFooter() { return ['\n' + SEP]; }
+
+function buildReport(title, fields, issues, verbose, groupBy) {
+  return [...reportHeader(title, fields), ...reportIssues(issues, verbose, groupBy), ...reportFooter()].join('\n');
+}
+
+// --- 通用计数 ---
+
+function countBySeverity(issues, field) {
+  field = field || 'severity';
+  const counts = {};
+  for (const i of issues) counts[i[field]] = (counts[i[field]] || 0) + 1;
+  return counts;
+}
+
+function hasFatal(issues, fatalLevels) {
+  fatalLevels = fatalLevels || ['error'];
+  return issues.some(i => fatalLevels.includes(i.severity));
+}
+
+module.exports = {
+  parseCliArgs, buildReport, reportHeader, reportIssues,
+  reportFooter, countBySeverity, hasFatal, SEP, DASH, ICONS
+};

package/skills/tools/verify-change/scripts/change_analyzer.js

@@ -4,6 +4,7 @@
 const { execSync } = require("child_process");
 const path = require("path");
 const fs = require("fs");
+const { parseCliArgs, buildReport, hasFatal, DASH } = require(path.join(__dirname, '..', '..', 'lib', 'shared.js'));
 
 const CODE_EXT = new Set([".py",".go",".rs",".ts",".js",".jsx",".tsx",".java",".c",".cpp",".h",".hpp"]);
 const DOC_EXT = new Set([".md",".rst",".txt",".adoc"]);
@@ -61,7 +62,7 @@ function parsePorcelainLine(line) {
 }
 
 function git(args) {
-  try { return execSync(`git ${args}`, { encoding: "utf8", stdio: ["pipe","pipe","pipe"] }); }
+  try { return execSync('git ' + args, { encoding: "utf8", stdio: ["pipe","pipe","pipe"] }); }
   catch { return ""; }
 }
 
@@ -146,14 +147,22 @@ function checkDocSync(changes, modules) {
     if (!modCode.length) continue;
     const total = modCode.reduce((s, c) => s + c.additions + c.deletions, 0);
     if (total > 50 && !docPaths.has(design)) {
-      issues.push({ severity: "warning", message: `模块 ${mod} 有较大代码变更 (${total} 行)，但 DESIGN.md 未更新`, related_files: modCode.map(c => c.path) });
+      issues.push({
+        severity: "warning",
+        message: `模块 ${mod} 有较大代码变更 (${total} 行)，但 DESIGN.md 未更新`,
+        related_files: modCode.map(c => c.path)
+      });
       docStatus[`${mod}/DESIGN.md`] = false;
     } else {
       docStatus[`${mod}/DESIGN.md`] = true;
     }
     const newFiles = modCode.filter(c => c.type === "added");
     if (newFiles.length && !docPaths.has(readme)) {
-      issues.push({ severity: "info", message: `模块 ${mod} 新增了文件，建议更新 README.md`, related_files: newFiles.map(c => c.path) });
+      issues.push({
+        severity: "info",
+        message: `模块 ${mod} 新增了文件，建议更新 README.md`,
+        related_files: newFiles.map(c => c.path)
+      });
     }
   }
   return { docStatus, issues };
@@ -165,12 +174,30 @@ function analyzeImpact(changes) {
   const tests = changes.filter(c => c.is_test);
   if (code.length && !tests.length) {
     const total = code.reduce((s, c) => s + c.additions + c.deletions, 0);
-    if (total > 30) issues.push({ severity: "warning", message: `代码变更 ${total} 行，但没有对应的测试更新`, related_files: code.map(c => c.path) });
+    if (total > 30) {
+      issues.push({
+        severity: "warning",
+        message: `代码变更 ${total} 行，但没有对应的测试更新`,
+        related_files: code.map(c => c.path)
+      });
+    }
   }
   const configs = changes.filter(c => c.is_config);
-  if (configs.length) issues.push({ severity: "info", message: "配置文件有变更，请确认是否需要更新文档", related_files: configs.map(c => c.path) });
+  if (configs.length) {
+    issues.push({
+      severity: "info",
+      message: "配置文件有变更，请确认是否需要更新文档",
+      related_files: configs.map(c => c.path)
+    });
+  }
   const deleted = changes.filter(c => c.type === "deleted");
-  if (deleted.length) issues.push({ severity: "info", message: `删除了 ${deleted.length} 个文件，请确认相关引用已清理`, related_files: deleted.map(c => c.path) });
+  if (deleted.length) {
+    issues.push({
+      severity: "info",
+      message: `删除了 ${deleted.length} 个文件，请确认相关引用已清理`,
+      related_files: deleted.map(c => c.path)
+    });
+  }
   return issues;
 }
 
@@ -195,18 +222,17 @@ function analyzeChanges(mode = "working") {
 }
 
 function formatReport(r, verbose) {
-  const L = [];
-  const sep = "=".repeat(60);
-  const sep2 = "-".repeat(40);
-  L.push(sep, "变更分析报告", sep);
-  L.push(`\n变更文件: ${r.changes.length}`);
-  L.push(`新增行数: +${r.totalAdd}`);
-  L.push(`删除行数: -${r.totalDel}`);
-  L.push(`受影响模块: ${[...r.modules].join(", ") || "无"}`);
-  L.push(`分析结果: ${r.passed ? "✓ 通过" : "✗ 需要关注"}`);
+  const fields = {
+    '变更文件': r.changes.length,
+    '新增行数': `+${r.totalAdd}`,
+    '删除行数': `-${r.totalDel}`,
+    '受影响模块': [...r.modules].join(", ") || "无",
+    '分析结果': r.passed ? "✓ 通过" : "✗ 需要关注",
+  };
+  let report = buildReport('变更分析报告', fields, r.issues, verbose);
 
   if (r.changes.length && verbose) {
-    L.push("\n" + sep2, "变更文件列表:", sep2);
+    const lines = ["\n" + DASH, "变更文件列表:", DASH];
     const icons = { added: "➕", modified: "📝", deleted: "➖", renamed: "📋" };
     for (const c of r.changes) {
       const tags = [];
@@ -215,56 +241,49 @@ function formatReport(r, verbose) {
       if (c.is_test) tags.push("测试");
       if (c.is_config) tags.push("配置");
       const t = tags.length ? ` [${tags.join(", ")}]` : "";
-      L.push(`  ${icons[c.type] || "📝"} ${c.path}${t} (+${c.additions}/-${c.deletions})`);
-    }
-  }
-
-  if (r.issues.length) {
-    L.push("\n" + sep2, "问题与建议:", sep2);
-    const si = { error: "✗", warning: "⚠", info: "ℹ" };
-    for (const i of r.issues) {
-      L.push(`\n  ${si[i.severity] || "ℹ"} [${i.severity.toUpperCase()}] ${i.message}`);
-      if (verbose && i.related_files.length) {
-        for (const f of i.related_files.slice(0, 5)) L.push(`      - ${f}`);
-        if (i.related_files.length > 5) L.push(`      ... 及其他 ${i.related_files.length - 5} 个文件`);
-      }
+      lines.push(`  ${icons[c.type] || "📝"} ${c.path}${t} (+${c.additions}/-${c.deletions})`);
     }
+    report += '\n' + lines.join('\n');
   }
 
   if (Object.keys(r.docStatus).length) {
-    L.push("\n" + sep2, "文档同步状态:", sep2);
+    const lines = ["\n" + DASH, "文档同步状态:", DASH];
     for (const [doc, synced] of Object.entries(r.docStatus)) {
-      L.push(`  ${synced ? "✓" : "✗"} ${doc}`);
+      lines.push(`  ${synced ? "✓" : "✗"} ${doc}`);
     }
+    report += '\n' + lines.join('\n');
   }
 
-  L.push("\n" + sep);
-  return L.join("\n");
+  return report;
 }
 
 // CLI
-const args = process.argv.slice(2);
-let mode = "working", verbose = false, jsonOut = false;
-for (let i = 0; i < args.length; i++) {
-  if (args[i] === "--mode" && args[i + 1]) { mode = args[++i]; }
-  else if (args[i] === "-v" || args[i] === "--verbose") { verbose = true; }
-  else if (args[i] === "--json") { jsonOut = true; }
-}
+if (require.main === module) {
+  const opts = parseCliArgs(process.argv);
+  const result = analyzeChanges(opts.mode || "working");
 
-const result = analyzeChanges(mode);
+  if (opts.json) {
+    console.log(JSON.stringify({
+      passed: result.passed,
+      total_additions: result.totalAdd,
+      total_deletions: result.totalDel,
+      affected_modules: [...result.modules],
+      changes: result.changes.map(c => ({
+        path: c.path, type: c.type, additions: c.additions,
+        deletions: c.deletions, is_code: c.is_code,
+        is_doc: c.is_doc, is_test: c.is_test
+      })),
+      issues: result.issues.map(i => ({
+        severity: i.severity, message: i.message,
+        related_files: i.related_files,
+      })),
+      doc_sync_status: result.docStatus,
+    }, null, 2));
+  } else {
+    console.log(formatReport(result, opts.verbose));
+  }
 
-if (jsonOut) {
-  console.log(JSON.stringify({
-    passed: result.passed,
-    total_additions: result.totalAdd,
-    total_deletions: result.totalDel,
-    affected_modules: [...result.modules],
-    changes: result.changes.map(c => ({ path: c.path, type: c.type, additions: c.additions, deletions: c.deletions, is_code: c.is_code, is_doc: c.is_doc, is_test: c.is_test })),
-    issues: result.issues.map(i => ({ severity: i.severity, message: i.message, related_files: i.related_files })),
-    doc_sync_status: result.docStatus,
-  }, null, 2));
-} else {
-  console.log(formatReport(result, verbose));
+  process.exit(result.passed ? 0 : 1);
 }
 
-process.exit(result.passed ? 0 : 1);
+module.exports = { normalizePath, classifyFile, parsePorcelainLine, parseNameStatusLine, identifyModules };

package/skills/tools/verify-module/scripts/module_scanner.js

@@ -3,11 +3,15 @@
 
 const fs = require('fs');
 const path = require('path');
+const { parseCliArgs, buildReport, hasFatal } = require(path.join(__dirname, '..', '..', 'lib', 'shared.js'));
 
 const REQUIRED_FILES = { 'README.md': '模块说明文档', 'DESIGN.md': '设计决策文档' };
 const ALT_SRC_DIRS = ['src', 'lib', 'pkg', 'internal', 'cmd', 'app'];
 const ALT_TEST_DIRS = ['tests', 'test', '__tests__', 'spec'];
-const ROOT_SCRIPT_FILES = new Set(['install.sh', 'uninstall.sh', 'install.ps1', 'uninstall.ps1', 'Dockerfile', 'Makefile']);
+const ROOT_SCRIPT_FILES = new Set([
+  'install.sh', 'uninstall.sh', 'install.ps1',
+  'uninstall.ps1', 'Dockerfile', 'Makefile'
+]);
 const CODE_EXTS = new Set(['.py', '.go', '.rs', '.ts', '.js', '.java', '.sh', '.ps1']);
 const TEST_PATTERNS = ['test_', '_test.', '.test.', 'spec_', '_spec.'];
 
@@ -45,8 +49,14 @@ function scanModule(target) {
   const issues = [];
   const add = (severity, message, p) => issues.push({ severity, message, path: p || null });
 
-  if (!fs.existsSync(modulePath)) { add('error', `路径不存在: ${modulePath}`); return { modulePath, issues, structure: {} }; }
-  if (!fs.statSync(modulePath).isDirectory()) { add('error', `不是目录: ${modulePath}`); return { modulePath, issues, structure: {} }; }
+  if (!fs.existsSync(modulePath)) {
+    add('error', `路径不存在: ${modulePath}`);
+    return { modulePath, issues, structure: {} };
+  }
+  if (!fs.statSync(modulePath).isDirectory()) {
+    add('error', `不是目录: ${modulePath}`);
+    return { modulePath, issues, structure: {} };
+  }
 
   const structure = scanStructure(modulePath);
 
@@ -58,15 +68,36 @@ function scanModule(target) {
   }
 
   // source dirs
-  let srcFound = ALT_SRC_DIRS.some(d => { try { return fs.statSync(path.join(modulePath, d)).isDirectory(); } catch { return false; } });
+  let srcFound = ALT_SRC_DIRS.some(d => {
+    try { return fs.statSync(path.join(modulePath, d)).isDirectory(); }
+    catch { return false; }
+  });
   const entries = fs.readdirSync(modulePath);
-  const rootCode = entries.filter(n => { try { const s = fs.statSync(path.join(modulePath, n)); return s.isFile() && CODE_EXTS.has(path.extname(n)); } catch { return false; } });
-  const rootScript = entries.filter(n => { try { return fs.statSync(path.join(modulePath, n)).isFile() && ROOT_SCRIPT_FILES.has(n); } catch { return false; } });
-  if (rootCode.length || rootScript.length) { srcFound = true; if (rootCode.length > 5) add('warning', `根目录代码文件过多 (${rootCode.length}个)，建议整理到 src/ 目录`); }
+  const rootCode = entries.filter(n => {
+    try {
+      const s = fs.statSync(path.join(modulePath, n));
+      return s.isFile() && CODE_EXTS.has(path.extname(n));
+    } catch { return false; }
+  });
+  const rootScript = entries.filter(n => {
+    try {
+      return fs.statSync(path.join(modulePath, n)).isFile()
+        && ROOT_SCRIPT_FILES.has(n);
+    } catch { return false; }
+  });
+  if (rootCode.length || rootScript.length) {
+    srcFound = true;
+    if (rootCode.length > 5) {
+      add('warning', `根目录代码文件过多 (${rootCode.length}个)，建议整理到 src/ 目录`);
+    }
+  }
   if (!srcFound) add('warning', '未找到源码目录或代码文件');
 
   // test dirs
-  let testFound = ALT_TEST_DIRS.some(d => { try { return fs.statSync(path.join(modulePath, d)).isDirectory(); } catch { return false; } });
+  let testFound = ALT_TEST_DIRS.some(d => {
+    try { return fs.statSync(path.join(modulePath, d)).isDirectory(); }
+    catch { return false; }
+  });
   if (!testFound) testFound = rglob(modulePath, n => TEST_PATTERNS.some(p => n.includes(p)));
   if (!testFound) add('warning', '未找到测试目录或测试文件');
 
@@ -75,12 +106,16 @@ function scanModule(target) {
   if (fs.existsSync(readme)) {
     const c = fs.readFileSync(readme, 'utf-8');
     if (!c.includes('#')) add('warning', 'README.md 缺少标题', readme);
-    if (!['usage', 'install', '使用', '安装', 'example', '示例'].some(k => c.toLowerCase().includes(k))) add('info', 'README.md 建议添加使用说明或示例', readme);
+    const docKeys = ['usage', 'install', '使用', '安装', 'example', '示例'];
+    if (!docKeys.some(k => c.toLowerCase().includes(k)))
+      add('info', 'README.md 建议添加使用说明或示例', readme);
   }
   const design = path.join(modulePath, 'DESIGN.md');
   if (fs.existsSync(design)) {
     const c = fs.readFileSync(design, 'utf-8');
-    if (!['决策', 'decision', '选择', 'choice', '权衡', 'trade'].some(k => c.toLowerCase().includes(k))) add('info', 'DESIGN.md 建议记录设计决策和权衡', design);
+    const designKeys = ['决策', 'decision', '选择', 'choice', '权衡', 'trade'];
+    if (!designKeys.some(k => c.toLowerCase().includes(k)))
+      add('info', 'DESIGN.md 建议记录设计决策和权衡', design);
   }
 
   return { modulePath, issues, structure };
@@ -98,40 +133,31 @@ function formatStructure(s, indent = 0) {
 }
 
 function formatReport(r, verbose) {
-  const passed = !r.issues.some(i => i.severity === 'error');
   const errs = r.issues.filter(i => i.severity === 'error').length;
   const warns = r.issues.filter(i => i.severity === 'warning').length;
-  const lines = [
-    '='.repeat(60), '模块完整性扫描报告', '='.repeat(60),
-    `\n模块路径: ${r.modulePath}`,
-    `扫描结果: ${passed ? '\u2713 通过' : '\u2717 未通过'}`,
-    `错误: ${errs} | 警告: ${warns}`
-  ];
-  if (r.issues.length) {
-    lines.push('\n' + '-'.repeat(40), '问题列表:', '-'.repeat(40));
-    for (const i of r.issues) {
-      const icon = { error: '\u2717', warning: '\u26A0', info: '\u2139' }[i.severity];
-      lines.push(`  ${icon} [${i.severity.toUpperCase()}] ${i.message}`);
-      if (i.path && verbose) lines.push(`    路径: ${i.path}`);
-    }
-  }
+  const passed = !hasFatal(r.issues);
+  const fields = {
+    '模块路径': r.modulePath,
+    '扫描结果': passed ? '\u2713 通过' : '\u2717 未通过',
+    '统计': `错误: ${errs} | 警告: ${warns}`,
+  };
+  const issues = r.issues.map(i => ({
+    severity: i.severity, message: i.message, path: i.path,
+    file_path: i.path || '', line_number: null,
+  }));
+  let report = buildReport('模块完整性扫描报告', fields, issues, verbose);
   if (verbose && r.structure.name) {
-    lines.push('\n' + '-'.repeat(40), '目录结构:', '-'.repeat(40), formatStructure(r.structure));
+    report += '\n' + '-'.repeat(40) + '\n目录结构:\n' + '-'.repeat(40) + '\n' + formatStructure(r.structure);
   }
-  lines.push('\n' + '='.repeat(60));
-  return lines.join('\n');
+  return report;
 }
 
 // CLI
-const args = process.argv.slice(2);
-const verbose = args.includes('-v') || args.includes('--verbose');
-const jsonOut = args.includes('--json');
-const target = args.find(a => !a.startsWith('-')) || '.';
-
-const result = scanModule(target);
-const passed = !result.issues.some(i => i.severity === 'error');
+const opts = parseCliArgs(process.argv);
+const result = scanModule(opts.target);
+const passed = !hasFatal(result.issues);
 
-if (jsonOut) {
+if (opts.json) {
   console.log(JSON.stringify({
     module_path: result.modulePath, passed,
     error_count: result.issues.filter(i => i.severity === 'error').length,
@@ -139,7 +165,7 @@ if (jsonOut) {
     issues: result.issues
   }, null, 2));
 } else {
-  console.log(formatReport(result, verbose));
+  console.log(formatReport(result, opts.verbose));
 }
 
 process.exit(passed ? 0 : 1);