chore: 固化QQBot审查护栏

README.md

@@ -23,7 +23,7 @@
 - 生成验证进程清理计划：按项目路径和端口给出 PowerShell 检查命令，不直接杀进程。
 - 清理历史产物：统一治理 `.kt-workspace` 下的测试/验证产物，按目录最近修改时间只保留最近 3 轮，模板目录永久保留；CLI 默认 dry-run，真实清理必须显式传 `--execute`。
 - 检查 env 策略和变更风险：区分后端真实 env 与前端客户端 `.env*`，提醒锁文件、核心表格组件、部署链路和 Vue TSX 插槽写法等高风险改动。
-- 全局 CodeReview 只读扫描：汇总全部 KT 子仓库的 Git 状态、敏感文件跟踪、冲突标记、运行时调试输出、疑似凭据字面量、NapCat `latest` 镜像漂移风险、根目录生成产物、`TASKS.md` 最近记录字段结构和当前变更风险；默认只对变更文件做内容扫描，并放过短中文显示标签，避免历史误报污染上下文；任何文件改动后都要跑一遍。
+- 全局 CodeReview 只读扫描：汇总全部 KT 子仓库的 Git 状态、敏感文件跟踪、冲突标记、运行时调试输出、疑似凭据字面量、NapCat `latest` 镜像漂移风险、QQBot 重型服务单测导入风险、根目录生成产物、`TASKS.md` 最近记录字段结构和当前变更风险；默认只对变更文件做内容扫描，并放过短中文显示标签，避免历史误报污染上下文；任何文件改动后都要跑一遍。
 - 生成或写入 `TASKS.md` 最近记录：默认 `dryRun=true`，确认后再落盘。
 - 生成提交前检查清单：校验 KT commit message 约定。
 
@@ -106,7 +106,7 @@ pnpm run admin-login -- --url http://127.0.0.1:5999/#/auth/login
 | 脚本 | 用途 |
 | --- | --- |
 | `pnpm run admin-login` | 使用可见 Edge 打开 Admin 登录页，填写账号密码，拖动滑块，保存登录态和截图。默认账号来自初始化数据 `admin/123456`，生产或个人账号用 `KT_ADMIN_USERNAME` / `KT_ADMIN_PASSWORD` 或 CLI 参数覆盖。 |
-| `pnpm run global-review` | 对 KT 全部子仓库做只读全局 CodeReview 扫描，默认仅对变更文件做内容深扫，输出 JSON 复审报告，并校验 `TASKS.md` 最近记录只保留范围、关键词、验证字段；确认误报时优先升级 `src/tools/review.ts`，不要把误报沉积到上下文。 |
+| `pnpm run global-review` | 对 KT 全部子仓库做只读全局 CodeReview 扫描，默认仅对变更文件做内容深扫，输出 JSON 复审报告，并校验 `TASKS.md` 最近记录只保留范围、关键词、验证字段、NapCat `latest` 漂移和 QQBot 重型服务单测导入风险；确认误报时优先升级 `src/tools/review.ts`，不要把误报沉积到上下文。 |
 | `pnpm run obsidian-context` | 输出 Obsidian 索引上下文；可传 `--module Admin`、`--query QQBot`、`--max-documents 10`。 |
 | `pnpm run obsidian-validate` | 校验 KT Obsidian vault 结构和链接；默认 warning 不让脚本失败，需要严格模式时传 `--fail-on-warnings`。 |
 | `pnpm run obsidian-sync` | 审计 Obsidian 工作流入口是否连通，并联动执行 validate。 |

src/selfTest.ts

@@ -12,7 +12,7 @@ import { cleanupHistoryArtifacts, parseCliCleanupArgs } from './tools/cleanup.js
 import { buildWorkflowLoopAudit } from './tools/loop.js';
 import { inspectProject } from './tools/inspect.js';
 import { prepareTask, readWorkflowContext } from './tools/task.js';
-import { buildGlobalCodeReview, findNapcatImageGovernanceFindings, findTaskRecordGovernanceFindings, isBenignCredentialReviewValue } from './tools/review.js';
+import { buildGlobalCodeReview, findNapcatImageGovernanceFindings, findQqbotCommandServiceTestImportFindings, findTaskRecordGovernanceFindings, isBenignCredentialReviewValue } from './tools/review.js';
 import { buildBusinessTestPlan } from './tools/testing.js';
 import { buildVerificationPlan } from './tools/verification.js';
 import { buildWorkstreamCloseout } from './tools/closeout.js';
@@ -50,6 +50,21 @@ export async function runSelfTest(): Promise<void> {
     throw new Error('NapCat image governance self-check failed');
   }
 
+  const qqbotCommandServiceImportFindings = findQqbotCommandServiceTestImportFindings(
+    ["import { QqbotCommandService } from '@/qqbot/command/qqbot-command.service';"],
+    {
+      file: 'test/qqbot/command/qqbot-command-cooldown.service.spec.ts',
+      project: 'Node/kt-template-online-api',
+    },
+  );
+  if (
+    !qqbotCommandServiceImportFindings.some(
+      (item) => item.category === 'qqbot-command-service-heavy-test-import',
+    )
+  ) {
+    throw new Error('QQBot command service test import governance self-check failed');
+  }
+
   const reviewCliParser = {
     dashedAll: parseGlobalReviewCliArgs(['node', 'server', '--global-review', '--content-scan-all']).contentScanMode,
     keyValueAll: parseGlobalReviewCliArgs(['node', 'server', '--global-review', '--contentScanMode=all']).contentScanMode,

src/tools/review.ts

@@ -186,6 +186,39 @@ export function findNapcatImageGovernanceFindings(
   });
 }
 
+export function findQqbotCommandServiceTestImportFindings(
+  lines: string[],
+  context: {
+    file: string;
+    project: string;
+  },
+): ReviewFinding[] {
+  const normalized = context.file.replaceAll('\\', '/');
+  if (
+    context.project !== 'Node/kt-template-online-api' ||
+    !normalized.startsWith('test/qqbot/') ||
+    !normalized.endsWith('.ts')
+  ) {
+    return [];
+  }
+
+  return lines.flatMap((line, index) => {
+    if (!/from\s+['"][^'"]*qqbot-command\.service['"]/.test(line)) return [];
+
+    return [
+      {
+        category: 'qqbot-command-service-heavy-test-import',
+        file: context.file,
+        line: index + 1,
+        level: 'P2' as const,
+        message: 'QQBot 测试直接导入 QqbotCommandService，容易拖入插件注册和 BangDream runtime 导致单测卡住。',
+        project: context.project,
+        suggestion: '把纯策略抽到轻量函数测试，或通过 Nest TestingModule 明确 mock 掉插件 registry/runtime 依赖。',
+      },
+    ];
+  });
+}
+
 async function listGitTrackedFiles(projectPath: string): Promise<string[]> {
   const tracked = await tryExecFile('git', ['ls-files'], projectPath);
   if (!tracked.ok) return [];
@@ -363,6 +396,10 @@ function collectProjectContentFindings(
           file,
           project: project.relativePath,
         }),
+        ...findQqbotCommandServiceTestImportFindings(lines, {
+          file,
+          project: project.relativePath,
+        }),
       );
     }