devassist-agent 1.0.4 → 1.0.5

package/bin/devassist.js

@@ -172,6 +172,7 @@ function printHelp() {
   console.log(`  ${c.green('watch')}                 实时监控文件变更并运行检查`);
   console.log(`  ${c.green('diff')} ${c.gray('--staged|--base <分支>')}  仅检查变更文件（CI 模式）`);
   console.log(`  ${c.green('report')} ${c.gray('[--open]')}          生成 HTML 检查报告`);
+  console.log(`  ${c.gray('         --watch-history [n]')}   查看最近 n 条监控复盘报告（默认 20）`);
   console.log(`  ${c.green('deploy')} ${c.gray('--check')}           部署前验证（端口 + 路由 + MIME + 清单）`);
   console.log(`  ${c.green('debt')} ${c.gray('--scan')}              查看技术债务看板（按优先级排序）`);
   console.log(`  ${c.green('convention')} ${c.gray('add|list|remove')}  管理项目约定`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "devassist-agent",
-  "version": "1.0.4",
+  "version": "1.0.5",
   "description": "AI Agent 开发质量守护工具 — 30+规则检测代码缺陷/安全漏洞/性能反模式，多引擎AI根因分析，零外部依赖",
   "main": "src/index.js",
   "bin": {

package/src/cli/commands/report.js

@@ -12,6 +12,7 @@
  *   devassist report              # Generate report
  *   devassist report --ai         # Include AI analysis
  *   devassist report --open       # Open in browser after generating
+ *   devassist report --watch-history [n]  # Show recent watch reports (default 20)
  */
 
 const fs = require('fs');
@@ -38,6 +39,11 @@ module.exports = async function report(projectRoot, args, ui) {
   const useAI = args.includes('--ai');
   const openBrowser = args.includes('--open');
 
+  // Watch history mode
+  if (args.includes('--watch-history')) {
+    return showWatchHistory(projectRoot, args, ui);
+  }
+
   printHeader('生成报告');
 
   // Collect files using shared utility
@@ -308,3 +314,101 @@ function findingHTML(f) {
 function escapeHTML(str) {
   return String(str).replace(/&/g, '&amp;').replace(/</g, '&lt;').replace(/>/g, '&gt;').replace(/"/g, '&quot;');
 }
+
+/**
+ * Show watch history for retrospective.
+ * Usage: devassist report --watch-history [n]
+ */
+function showWatchHistory(projectRoot, args, ui) {
+  const { printHeader, c } = ui;
+  printHeader('监控复盘报告');
+
+  const historyDir = path.join(projectRoot, '.devassist', 'history', 'watch');
+  const logPath = path.join(historyDir, 'watch-log.jsonl');
+
+  if (!fs.existsSync(logPath)) {
+    console.log(`  ${c.yellow('暂无监控历史记录。')}`);
+    console.log(`  ${c.gray('运行 devassist start 后，文件变更的检查/分析/修复记录会自动保存到这里。')}\n`);
+    return;
+  }
+
+  // Parse limit from args
+  let limit = 20;
+  const idx = args.indexOf('--watch-history');
+  if (idx >= 0 && args[idx + 1] && /^\d+$/.test(args[idx + 1])) {
+    limit = parseInt(args[idx + 1]);
+  }
+
+  // Read JSONL log
+  const lines = fs.readFileSync(logPath, 'utf-8').trim().split('\n').filter(Boolean);
+  const entries = lines.map(l => { try { return JSON.parse(l); } catch (_) { return null; } }).filter(Boolean);
+
+  if (entries.length === 0) {
+    console.log(`  ${c.yellow('暂无监控历史记录。')}\n`);
+    return;
+  }
+
+  // Show recent entries (newest first)
+  const recent = entries.slice(-limit).reverse();
+  console.log(`  ${c.gray(`共 ${entries.length} 条记录，显示最近 ${recent.length} 条：`)}\n`);
+
+  // Summary stats
+  const totalEvents = entries.length;
+  const totalFindings = entries.reduce((s, e) => s + (e.total || 0), 0);
+  const totalBlocks = entries.reduce((s, e) => s + (e.blocks || 0), 0);
+  const totalFixed = entries.reduce((s, e) => s + (e.fixed || 0), 0);
+  const aiEvents = entries.filter(e => e.aiUsed).length;
+
+  console.log(`  ${c.bold('汇总统计：')}`);
+  console.log(`    监控事件：${totalEvents} 次`);
+  console.log(`    发现问题：${totalFindings} 个（其中 ${c.red(totalBlocks + ' 阻断')}）`);
+  console.log(`    自动修复：${c.green(totalFixed + ' 处')}`);
+  console.log(`    AI 分析：${aiEvents} 次\n`);
+
+  // Detail table
+  console.log(`  ${c.bold('最近事件：')}\n`);
+  console.log(`  ${c.gray('时间'.padEnd(20))} ${c.gray('文件'.padEnd(30))} ${c.gray('问题'.padEnd(8))} ${c.gray('修复'.padEnd(6))} ${c.gray('引擎')}`);
+  console.log(`  ${'─'.repeat(90)}`);
+
+  for (const e of recent) {
+    const time = new Date(e.timestamp).toLocaleString('zh-CN', { month: '2-digit', day: '2-digit', hour: '2-digit', minute: '2-digit', second: '2-digit' });
+    const file = (e.file || '').length > 28 ? '...' + (e.file || '').slice(-25) : (e.file || '');
+    const issues = e.total > 0 ? c.yellow(String(e.total)) : c.green('0');
+    const blocks = e.blocks > 0 ? c.red(`(${e.blocks}阻断)`) : '';
+    const fixed = e.fixed > 0 ? c.green(String(e.fixed)) : '-';
+    const engines = e.engines || '-';
+
+    console.log(`  ${time.padEnd(20)} ${file.padEnd(30)} ${issues} ${blocks}`.padEnd(58) + ` ${fixed.toString().padEnd(6)} ${engines}`);
+  }
+
+  // Latest report detail
+  const latestPath = path.join(projectRoot, '.devassist', 'reports', 'latest.json');
+  if (fs.existsSync(latestPath)) {
+    console.log(`\n  ${c.gray('最近一次完整报告：')} ${latestPath}`);
+    console.log(`  ${c.gray('历史报告目录：')} ${historyDir}`);
+  }
+
+  // Watch files with most issues
+  const fileStats = {};
+  for (const e of entries) {
+    if (!fileStats[e.file]) fileStats[e.file] = { count: 0, issues: 0, blocks: 0, fixed: 0 };
+    fileStats[e.file].count++;
+    fileStats[e.file].issues += e.total || 0;
+    fileStats[e.file].blocks += e.blocks || 0;
+    fileStats[e.file].fixed += e.fixed || 0;
+  }
+
+  const hotFiles = Object.entries(fileStats)
+    .sort((a, b) => b[1].issues - a[1].issues)
+    .filter(([, s]) => s.issues > 0)
+    .slice(0, 5);
+
+  if (hotFiles.length > 0) {
+    console.log(`\n  ${c.bold('问题热点文件 TOP 5：')}\n`);
+    for (const [file, s] of hotFiles) {
+      console.log(`    ${file} — ${s.issues} 个问题，${s.blocks} 阻断，${s.fixed} 已修复（${s.count} 次变更）`);
+    }
+  }
+
+  console.log('');
+}

package/src/cli/commands/watch.js

@@ -171,9 +171,6 @@ module.exports = async function watch(projectRoot, args, ui) {
       // Apply inline ignores
       allFindings = filterIgnored(allFindings, [{ path: relPath, content }]);
 
-      // Save findings for ai --analyze
-      saveWatchFindings(projectRoot, allFindings);
-
       // Print header
       console.log(`\n  ${c.gray(`[${timestamp}]`)} ${c.bold(relPath)}`);
 
@@ -270,13 +267,37 @@ module.exports = async function watch(projectRoot, args, ui) {
       console.log(`  ${c.gray('—')} ${parts.join(c.gray(' · '))}`);
 
       // --- Auto Fix (if enabled) ---
+      let fixResult = null;
       if (autoFix && suspectFindings.length > 0) {
-        const fixedCount = await autoFixFile(filePath, content, projectRoot, c);
-        if (fixedCount > 0) {
-          console.log(`  ${c.green('自动修复')} ${fixedCount} 处问题`);
+        fixResult = await autoFixFile(filePath, content, projectRoot, c);
+        if (fixResult.count > 0) {
+          console.log(`  ${c.green('自动修复')} ${fixResult.count} 处问题`);
+          for (const d of fixResult.details) {
+            const typeLabel = d.type === 'fetch-catch' ? 'fetch 添加 .catch()' : '添加 await';
+            console.log(`         ${c.gray(`第${d.line}行`)} ${typeLabel}`);
+          }
         }
       }
 
+      // --- Save comprehensive report for retrospective ---
+      saveWatchReport(projectRoot, {
+        timestamp: new Date().toISOString(),
+        file: relPath,
+        findings: finalFindings,
+        aiAnalysisUsed: useAI && readyEngines.length > 0 && suspectFindings.length > 0,
+        compareMode: useCompare && readyEngines.length >= 2,
+        engines: readyEngines.map(e => ({ provider: e.provider, model: e.model, name: e.name })),
+        fixes: fixResult ? fixResult.details : [],
+        fixBackupPath: fixResult ? fixResult.backupPath : null,
+        stats: {
+          total: finalFindings.length,
+          blocks: blocks.length,
+          warnings: warns.length,
+          infos: infos.length,
+          fixed: fixResult ? fixResult.count : 0,
+        },
+      });
+
     } finally {
       analyzingFiles.delete(filePath);
     }
@@ -346,7 +367,7 @@ async function autoFixFile(filePath, content, projectRoot, c) {
       const trimmed = line.trimEnd();
       if (trimmed.endsWith(')') && !trimmed.endsWith('.catch()')) {
         const fixedLine = trimmed.replace(/\)\s*$/, ").catch(err => console.error('fetch error:', err))");
-        changes.push({ line: i, newLine: fixedLine, type: 'fetch-catch' });
+        changes.push({ line: i + 1, type: 'fetch-catch', before: line, after: fixedLine });
         fixCount++;
         continue;
       }
@@ -366,7 +387,7 @@ async function autoFixFile(filePath, content, projectRoot, c) {
           'await $1('
         );
         if (fixedLine !== line) {
-          changes.push({ line: i, newLine: fixedLine, type: 'add-await' });
+          changes.push({ line: i + 1, type: 'add-await', before: line, after: fixedLine });
           fixCount++;
           continue;
         }
@@ -374,11 +395,11 @@ async function autoFixFile(filePath, content, projectRoot, c) {
     }
   }
 
-  if (fixCount === 0) return 0;
+  if (fixCount === 0) return { count: 0, details: [] };
 
   // Apply changes
   for (const ch of changes) {
-    lines[ch.line] = ch.newLine;
+    lines[ch.line - 1] = ch.after;
   }
   const newContent = lines.join('\n');
 
@@ -391,22 +412,60 @@ async function autoFixFile(filePath, content, projectRoot, c) {
   // Write fixed content
   fs.writeFileSync(filePath, newContent, 'utf-8');
 
-  return fixCount;
+  return { count: fixCount, details: changes, backupPath };
 }
 
-function saveWatchFindings(projectRoot, findings) {
+/**
+ * Save comprehensive watch report for retrospective.
+ * - latest.json: always overwritten with the latest event
+ * - history/watch-<timestamp>.json: per-event detail, kept for 90 days
+ * - history/watch-log.jsonl: append-only summary log for quick querying
+ */
+function saveWatchReport(projectRoot, report) {
   const reportDir = path.join(projectRoot, '.devassist', 'reports');
   if (!fs.existsSync(reportDir)) fs.mkdirSync(reportDir, { recursive: true });
-  const reportPath = path.join(reportDir, 'latest.json');
-  fs.writeFileSync(reportPath, JSON.stringify({
-    generatedAt: new Date().toISOString(),
-    source: 'watch',
-    findings,
-    stats: {
-      total: findings.length,
-      blocks: findings.filter(f => f.severity === 'block').length,
-      warnings: findings.filter(f => f.severity === 'warn').length,
-      infos: findings.filter(f => f.severity === 'info').length,
-    },
-  }, null, 2), 'utf-8');
+
+  // 1. Save latest.json (full detail, always latest)
+  const latestPath = path.join(reportDir, 'latest.json');
+  fs.writeFileSync(latestPath, JSON.stringify(report, null, 2), 'utf-8');
+
+  // 2. Save timestamped history
+  const historyDir = path.join(projectRoot, '.devassist', 'history', 'watch');
+  if (!fs.existsSync(historyDir)) fs.mkdirSync(historyDir, { recursive: true });
+
+  const ts = new Date();
+  const tsStr = ts.toISOString().replace(/[:.]/g, '-');
+  const historyPath = path.join(historyDir, `watch-${tsStr}.json`);
+  fs.writeFileSync(historyPath, JSON.stringify(report, null, 2), 'utf-8');
+
+  // 3. Append summary to JSONL log
+  const logPath = path.join(historyDir, 'watch-log.jsonl');
+  const summary = {
+    timestamp: report.timestamp,
+    file: report.file,
+    total: report.stats.total,
+    blocks: report.stats.blocks,
+    warnings: report.stats.warnings,
+    infos: report.stats.infos,
+    fixed: report.stats.fixed,
+    aiUsed: report.aiAnalysisUsed,
+    compare: report.compareMode,
+    engines: report.engines.map(e => e.provider).join('+'),
+  };
+  fs.appendFileSync(logPath, JSON.stringify(summary) + '\n', 'utf-8');
+
+  // 4. Auto-cleanup: remove history files older than 90 days
+  try {
+    const files = fs.readdirSync(historyDir);
+    const now = Date.now();
+    const maxAge = 90 * 24 * 60 * 60 * 1000; // 90 days in ms
+    for (const f of files) {
+      if (!f.startsWith('watch-') || !f.endsWith('.json')) continue;
+      const filePath = path.join(historyDir, f);
+      const stat = fs.statSync(filePath);
+      if (now - stat.mtime.getTime() > maxAge) {
+        fs.unlinkSync(filePath);
+      }
+    }
+  } catch (_) {}
 }