@autobest-ui/agent 1.0.20 → 1.0.22

package/mcp/trace-mcp-recorder/README.md

@@ -1,6 +1,6 @@
 # trace-mcp-recorder
 
-`trace-mcp-recorder` 是基于 `@modelcontextprotocol/sdk` 和 `StdioServerTransport` 的 Node.js MCP Server。它打开非无头 Chromium，并在页面左下角注入半透明的“开始录制”“停止录制”和设备切换按钮供测试人员手动控制 Trace；默认是 PC 端，也可切换为 iPhone 14 Pro Max 移动端模拟。点击开始后先自动刷新一次页面，确保 DOM Snapshot 所引用的图片、CSS 和网络资源被 Trace 捕获；后续用户活动经过防抖后以 `步骤N | 页面标题 | 事件类型` 的 Trace Group 写入轻量 Playwright Action，空闲超过 2 分钟自动结束，然后上传原生 `trace.zip` 和浏览器元数据，并返回私有后端的 Trace Viewer 链接。同一进程同一时间只允许一个录制任务。
+`trace-mcp-recorder` 是基于 `@modelcontextprotocol/sdk` 和 `StdioServerTransport` 的 Node.js MCP Server。它打开非无头 Chromium，并在页面左下角注入半透明的“开始录制”“停止录制”和设备切换按钮供测试人员手动控制 Trace；默认是 PC 端，也可切换为 iPhone 14 Pro Max 移动端模拟。页面先正常打开并等待，点击开始后先启动 Trace、刷新一次当前 SPA 页面并等待 DOM、CSS、字体和图片稳定，再进入正式录制；后续用户活动经过防抖后以 `步骤N | 页面标题 | 事件类型` 的 Trace Group 写入轻量 Playwright Action，空闲超过 2 分钟自动结束，然后上传原生 `trace.zip` 和浏览器元数据，并返回私有后端的 Trace Viewer 链接。同一进程同一时间只允许一个录制任务。
 
 ## 安装与启动
 

package/mcp/trace-mcp-recorder/index.js

@@ -187,10 +187,11 @@ function createRecorderPageScript() {
       const render = (state, deviceMode = 'desktop') => {
         const recording = state === 'recording';
         const waiting = state === 'waiting_for_start';
+        const preparing = state === 'starting';
         startButton.disabled = !waiting;
         stopButton.disabled = !recording && !waiting;
         deviceButton.disabled = state === 'starting' || state === 'stopping' || state === 'uploading';
-        startButton.textContent = recording ? '录制中' : '开始录制';
+        startButton.textContent = preparing ? '准备页面' : recording ? '录制中' : '开始录制';
         stopButton.textContent = state === 'stopping' || state === 'uploading' ? '正在停止' : '停止录制';
         deviceButton.textContent = deviceMode === 'mobile' ? '切换 PC 端' : '切换移动端';
       };
@@ -409,7 +410,13 @@ export class TraceRecorder {
       throw new Error('当前没有正在录制的 trace 会话');
     }
     if (session.state === 'starting') {
-      throw new Error('录制会话正在启动，请等待 start_trace_recording 返回后再停止');
+      await session.activityPromise?.catch(error => {
+        throw error;
+      });
+      if (session.state === 'failed') {
+        throw new Error('录制初始化失败，无法停止当前 trace');
+      }
+      return this.stop({ trigger, confirm });
     }
     if (session.state === 'waiting_for_start') {
       session.state = 'stopping';
@@ -489,13 +496,49 @@ export class TraceRecorder {
 
   async beginRecording(session) {
     if (session.tracingStarted || session.state !== 'waiting_for_start') return;
+    // The initial page was opened before the user clicked the button. Start the
+    // trace first, then reload once so SPA CSS, fonts, images and DOM resources
+    // are captured by Trace Viewer instead of being only browser-cache state.
+    session.state = 'starting';
     await session.context.tracing.start({ screenshots: true, snapshots: true, sources: true });
     session.tracingStarted = true;
-    // The initial navigation happened before Trace started. Reload now so the
-    // DOM snapshot also has response bodies for images, CSS, and other assets.
-    if (typeof session.page.reload === 'function') {
-      await session.page.reload({ waitUntil: 'load' });
+
+    const currentUrl = typeof session.page.url === 'function' ? session.page.url() : session.metadata.targetUrl;
+    const scrollPosition = await session.page
+      .evaluate(() => ({ x: window.scrollX, y: window.scrollY }))
+      .catch(() => ({ x: 0, y: 0 }));
+
+    await session.page.reload({ waitUntil: 'domcontentloaded' });
+    // networkidle is intentionally best-effort: SPAs with polling/websockets
+    // may never become idle, while the resource checks below remain bounded.
+    if (typeof session.page.waitForLoadState === 'function') {
+      await session.page.waitForLoadState('networkidle', { timeout: 10_000 }).catch(() => {});
+    }
+    await session.page
+      .evaluate(async () => {
+        await document.fonts?.ready?.catch?.(() => {});
+        const pendingImages = Array.from(document.images).filter(image => !image.complete);
+        await Promise.race([
+          Promise.all(pendingImages.map(image => new Promise(resolve => {
+            image.addEventListener('load', resolve, { once: true });
+            image.addEventListener('error', resolve, { once: true });
+          }))),
+          new Promise(resolve => setTimeout(resolve, 10_000))
+        ]);
+      })
+      .catch(() => {});
+    if (currentUrl && typeof session.page.url === 'function' && session.page.url() !== currentUrl) {
+      await session.page.goto(currentUrl, { waitUntil: 'domcontentloaded' });
     }
+    await session.page
+      .evaluate(({ x, y }) => window.scrollTo(x, y), scrollPosition)
+      .catch(() => {});
+
+    session.metadata.targetUrl = currentUrl || session.metadata.targetUrl;
+    session.metadata.userAgent = await session.page
+      .evaluate(() => navigator.userAgent)
+      .catch(() => session.metadata.userAgent);
+    session.metadata.viewport = session.page.viewportSize?.() ?? session.metadata.viewport;
     session.state = 'recording';
     session.lastActivityAt = Date.now();
     session.timer = this.setTimer(() => {
@@ -566,7 +609,9 @@ export class TraceRecorder {
       if (!session.activityPromise) {
         session.activityPromise = this.beginRecording(session).catch(error => this.failSession(session, error));
       }
-      return session.activityPromise.then(() => ({ state: session.state, deviceMode: session.deviceMode }));
+      // Do not hold the page binding open while reload navigates the document.
+      // The toolbar polls status and will switch to recording after the reload.
+      return { state: 'starting', deviceMode: session.deviceMode };
     }
     if (action === 'stop') {
       if (session.state !== 'recording' && session.state !== 'waiting_for_start') {
@@ -716,7 +761,7 @@ export class TraceRecorder {
   }
 
   async failSession(session, error) {
-    if (session.state !== 'recording' && session.state !== 'waiting_for_start') return;
+    if (!['starting', 'recording', 'waiting_for_start'].includes(session.state)) return;
     session.state = 'failed';
     this.clearTimer(session.timer);
     this.clearTimer(session.idleTimer);

package/mcp/trace-mcp-recorder/index.test.js

@@ -108,10 +108,13 @@ test('page controls start recording, record activity, upload and clean up one se
   assert.equal(calls.some(call => call[0] === 'trace-start'), false);
   assert.match(calls.find(call => call[0] === 'init-script')[1].content, /开始录制/);
   assert.deepEqual(await bindings.__traceRecorderControl({}, 'start'), {
-    state: 'recording',
+    state: 'starting',
     deviceMode: 'desktop'
   });
-  assert.ok(calls.some(call => call[0] === 'reload' && call[1].waitUntil === 'load'));
+  await recorder.activeSession.activityPromise;
+  assert.equal(recorder.activeSession.state, 'recording');
+  assert.ok(calls.some(call => call[0] === 'reload' && call[1].waitUntil === 'domcontentloaded'));
+  assert.ok(calls.findIndex(call => call[0] === 'trace-start') < calls.findIndex(call => call[0] === 'reload'));
   recorder.handleUserActivity(recorder.activeSession, {
     type: 'click',
     title: '首页功能'
@@ -168,6 +171,7 @@ test('marks the session failed and removes the local trace when upload fails', a
   const { calls, recorder } = createFixture({ uploadError: new Error('upload failed') });
   await recorder.start({ url: 'https://app.example.test' });
   await recorder.handlePageControl(recorder.activeSession, 'start');
+  await recorder.activeSession.activityPromise;
   recorder.handleUserActivity(recorder.activeSession);
 
   await recorder.stop();
@@ -179,6 +183,7 @@ test('automatically stops on timeout and returns that result on the next stop ca
   const { recorder, timer } = createFixture();
   await recorder.start({ url: 'https://app.example.test' });
   await recorder.handlePageControl(recorder.activeSession, 'start');
+  await recorder.activeSession.activityPromise;
   timer.callback();
 
   while (recorder.activeSession) await new Promise(resolve => setImmediate(resolve));
@@ -191,6 +196,7 @@ test('page stop control immediately stops and uploads, then exposes the result t
   const { bindings, recorder } = createFixture();
   await recorder.start({ url: 'https://app.example.test' });
   await bindings.__traceRecorderControl({}, 'start');
+  await recorder.activeSession.activityPromise;
 
   assert.deepEqual(await bindings.__traceRecorderControl({}, 'stop'), { state: 'stopping' });
   while (recorder.activeSession) await new Promise(resolve => setImmediate(resolve));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autobest-ui/agent",
-  "version": "1.0.20",
+  "version": "1.0.22",
   "private": false,
   "description": "Autobest Agent skills/plugins/mcp assets + sync cli",
   "files": [

package/skills/common/code-quality/SKILL.md

@@ -1,39 +1,165 @@
 ---
 name: code-quality
-description: 对前端仓库执行只读的安全与工程质量检查，并在项目根目录 `.scratch` 下生成带证据和修改建议的 Markdown 报告。用户要求前端安全扫描、依赖漏洞检查、代码质量 Review 或工程风险审查时使用；不修改源码、依赖、锁文件或配置。
+description: 对前端与 Node.js 服务执行稳定、可重复的只读安全审计，按固定 Phase 1-7、OWASP/CWE 标准和证据规则检查依赖、认证、会话、数据流、注入、服务端风险及环境配置，并在项目根目录 `.scratch/code-quality-report.md` 生成固定格式报告。用户要求前端或 Node.js 安全扫描、依赖漏洞检查、代码安全 Review 或工程安全风险审查时使用；不修改源码、依赖、锁文件、构建配置或提交历史。
 ---
 
-# 前端代码质量检查
+# 前端与 Node.js 服务安全审计
 
-对当前前端项目执行安全与工程质量审查。最终必须生成 `.scratch/code-quality-report.md`；报告内容、严重级别和单条 finding 字段遵循 [report-format.md](references/report-format.md)。
+目标是在相同审查范围内重复执行时尽量得到一致的结果。必须依次完成 Phase 1-7；不能用单次 `grep`、单次 `npm audit` 或单个静态规则替代完整流程。所有结论必须由源码、配置、依赖树或工具输出证据支持，最终只生成项目根目录的 `.scratch/code-quality-report.md`。报告章节、finding 字段和状态遵循 [report-format.md](references/report-format.md)。
 
-## 边界
+## 审计标准
 
-- 从仓库根目录开始，先确认 `git rev-parse --show-toplevel`；扫描目标必须是当前仓库或用户明确指定的仓库内路径。
-- 只读源码、依赖清单、锁文件、构建配置、lint/类型/测试配置和 Git 元数据；跳过 `node_modules`、构建产物、缓存和二进制文件。
-- 报告是唯一允许写入的项目产物，写入前创建 `.scratch`；不得修改业务源码、依赖版本、锁文件、配置、提交历史或远程状态。
-- 不读取或复制 `.env`、凭据、私钥和其他秘密值；发现疑似秘密时只记录脱敏位置和类型。
-- 工具不可用、网络不可达或项目命令失败时，记录 `Incomplete`/`Blocked` 及原因，不把缺失证据当作通过。
+将以下标准作为审计依据，并在 finding 中注明适用的标准或 CWE（无法映射时说明原因）：
 
-## 执行流程
+- OWASP Top 10。
+- OWASP API Security Top 10。
+- OWASP ASVS：V2 Authentication、V3 Session Management、V4 Access Control、V5 Validation, Sanitization and Encoding、V6 Stored Cryptography、V7 Error Handling and Logging、V8 Data Protection、V9 Communications、V12 Files and Resources、V14 Configuration。
+- CWE-79 XSS、CWE-22/CWE-23 路径穿越、CWE-601 Open Redirect、CWE-200 敏感信息暴露、CWE-312/CWE-319 明文敏感数据、CWE-327/CWE-328 弱密码学、CWE-532 日志中的敏感信息、CWE-798 硬编码凭据、CWE-918 SSRF、CWE-400/CWE-1333 DoS/ReDoS、CWE-94 代码注入。
+- `npm audit`/OSV、Node.js Security Best Practices、供应链安全和 lockfile 完整性。
 
-1. 记录扫描时间、仓库根目录、当前分支/提交和工作树状态，明确用户指定的审查范围；保留既有改动。
-2. 读取项目 manifest、锁文件、源码类型和可用脚本，识别 npm/Yarn/pnpm、框架和构建工具；不要凭目录名猜测技术栈。
-3. 选择当前环境实际存在的检查命令：依赖审计、密钥扫描、静态规则、类型检查、lint、测试或构建。执行前确认命令为只读。
-4. 检查安全风险：依赖漏洞、XSS/DOM 注入、危险动态执行、不安全 URL/重定向、敏感数据存储、凭据暴露、第三方脚本和生产配置。
-5. 检查工程质量：类型逃逸、异常与 Promise 处理、边界条件、复杂度/重复、依赖一致性、构建配置和关键路径测试保护。
-6. 复核工具告警的源码上下文，区分 `Confirmed`、`Suspected`、`False Positive` 和 `Needs Manual Verification`；只有有证据的问题才进入结论。
-7. 按报告格式写入 `.scratch/code-quality-report.md`，记录未执行的工具、未覆盖范围、风险统计和逐项修改建议。
-8. 重新读取报告并检查 Markdown 结构、路径、行号、证据和建议是否对应；最后复查 `git status --short`，确认业务文件未被修改。
+工具或漏洞数据库不可用时，必须在工具执行记录和未执行检查中写明 `Incomplete` 或 `Blocked`，不能将缺失证据写成通过。
 
-## 发现要求
+## 只读边界
 
-- 安全问题优先按 `Critical`、`High`、`Medium`、`Low`、`Info` 分级；不要用普通风格偏好掩盖安全问题。
-- 每条问题必须包含影响、证据、修复优先级、修改建议和修复后的验证方式。
-- 依赖漏洞同时记录包名、受影响版本、漏洞编号（如 CVE/OSV）和可用修复版本；无法联网时说明数据库覆盖限制。
-- 修改建议只描述安全、可验证的改法，不直接编辑代码，也不承诺一次修复即可消除所有风险。
-- 报告结论只能表示本次范围内的扫描结果，不能宣称项目绝对安全或完成渗透测试。
+- 开始时执行 `git rev-parse --show-toplevel`，确认审查目标在仓库根目录内；记录当前分支、提交和 `git status --short`。
+- 唯一允许生成的项目产物是 `.scratch/code-quality-report.md`。可以创建 `.scratch`，但不得在其中生成其他文件。
+- 读取源码、manifest、锁文件、配置、Docker/nginx/启动脚本、测试和 CI 文件，以及 Git 元数据；跳过 `node_modules`、构建缓存、二进制和无关的大型产物。
+- 不读取 `.env` 内容、私钥、凭据或 secret store；只记录文件位置、变量名或脱敏类型。
+- 对 `.env`、secret store 和私钥文件只记录存在性、路径和脱敏变量名；不得读取、复制或输出其值。
+- 不修改业务源码、依赖、锁文件、构建配置、测试、Git 历史、远程状态或运行中的业务数据；不安装依赖或改变项目配置。
+- 运行现有只读检查命令前确认命令不会写入项目；命令失败、网络不可达或范围不可读时保留失败证据。
 
-## 完成条件
+## 固定审计流程
 
-只有以下条件全部满足才算完成：`.scratch/code-quality-report.md` 已生成并可读；报告包含扫描范围、工具结果、风险分级、证据、修改建议和限制；工作树中除报告目录外没有本次 Skill 产生的代码或配置改动。
+### Phase 1: 项目识别
+
+确认并记录：
+
+- 仓库根目录、当前分支、提交和工作树状态。
+- React、Vue、Angular、Svelte、Node、Express、Koa、Nest、Fastify 等已由依赖或源码证明的技术栈。
+- package manager、lockfile 类型和 workspace/package 边界。
+- `build`、`lint`、`typecheck`、`test`、`audit` 脚本及其实际存在性。
+- 前端 SPA、SSR、Node API、BFF、CLI、构建工具和开发服务器边界。
+- 所有环境配置入口和环境变量来源；只记录位置和脱敏变量类型，不读取秘密值。
+
+没有证据的技术栈或运行边界标记为“未确认”，不得猜测。
+
+### Phase 2: 依赖与供应链
+
+按项目实际工具执行适用的检查：
+
+- `npm audit --json`、OSV 或仓库已配置的等价审计。
+- `npm ls` 或对应 package manager 的实际依赖树。
+- lockfile 完整性和 manifest/lockfile 一致性。
+- direct/transitive dependency 区分、实际安装版本和 workspace 归属。
+- 维护状态、过期版本、危险安装脚本和 postinstall 风险。
+- 构建工具、开发服务器和可暴露服务的攻击面。
+
+每个依赖漏洞必须记录包名、实际安装版本、direct/transitive、CVE/OSV/GHSA 编号、受影响范围、修复版本或升级路径、运行时/构建时/开发时影响，以及是否实际进入生产 bundle（无法确认时标记待确认）。按攻击面和依赖链聚合重复漏洞，但保留代表性工具证据，不机械地把全部审计输出合成一个 finding。
+
+### Phase 3: 认证、会话和敏感数据路径
+
+这是强制步骤，即使仓库看起来只有前端也必须确认是否存在这些路径；不存在时在报告中记录“未发现实现证据”。追踪登录、登出、刷新 token、token/cookie/session 初始化、改密、重置密码、MFA/验证码、角色与权限判断，以及支付、退款、订单审批等高权限操作。
+
+对每条存在的路径追踪：
+
+```text
+用户输入
+-> UI state/form
+-> API client
+-> base URL
+-> transport
+-> retry/error handler
+-> logger/telemetry
+-> storage/cache/cookie
+-> redirect/navigation
+```
+
+必须明确区分：HTTPS 请求体中的密码通常不是传输漏洞；HTTP 发送密码是传输安全漏洞；密码进入日志、遥测、错误上报、`localStorage`、URL 或缓存是敏感信息泄露；token 使用 HttpOnly/Secure/SameSite cookie 还是 `localStorage`；是否存在 token 泄露、过期、重放或跨环境混用风险。
+
+敏感字段检查至少覆盖：`password`、`passwd`、`pwd`、`adminPassword`、`adminPassWord`、`oldPassword`、`newPassword`、`confirmPassword`、`secret`、`token`、`accessToken`、`refreshToken`、`authorization`、`cookie`。不能只搜索 `password`。
+
+### 敏感 key 和凭据检查
+
+必须单独执行敏感 key 检查，覆盖源码、配置模板、测试 fixture、文档示例、构建产物（如纳入审查范围）和 Git 可见文本中的硬编码值。字段名和格式至少包括：
+
+- `apiKey`、`api_key`、`API_KEY`、`openaiKey`、`OPENAI_API_KEY`、`accessKey`、`access_key`、`secretKey`、`clientSecret`、`privateKey`、`serviceAccountKey`、`webhookSecret`。
+- OpenAI、AWS、GitHub、Google、Azure、Stripe 等供应商的已知前缀或结构；例如 `sk-`、`AKIA`、`ASIA`、`ghp_`、`github_pat_`、`AIza`、`xoxb-`、`xoxp-`、`SG.` 等。具体模式必须结合上下文核验，不得只因前缀就确认泄露。
+- PEM 私钥/证书块、JWT、Bearer Token、Basic 凭据和明显的高熵长字符串。
+- 赋值、对象字段、JSON/YAML 配置、HTTP header、SDK 初始化、环境变量映射、日志和错误上报中的 key 值流转。
+
+检查结果必须区分：真实可用的密钥、占位符/示例值、测试假值、变量引用和仅有字段名没有值的配置。`key` 字段本身不是漏洞：React `key`、对象索引或业务标识只有在值呈现凭据特征、进入认证/请求/日志路径或具备敏感上下文时才报告。静态命中没有可验证值或数据流时标记 `Needs Manual Verification`。
+
+发现疑似 key 时，报告只写脱敏指纹（例如前缀、后缀、长度、SHA-256 截断值），不得写入完整密钥。若证据显示密钥已提交到 Git 历史、生产 bundle、日志、遥测、URL、客户端存储或公开接口，必须评估撤销/轮换、受影响权限、环境和暴露范围，并按严重级别报告；不因为密钥位于 `.env` 就默认安全。
+
+### Phase 4: 前端注入与浏览器安全
+
+审查以下 source-to-sink 数据流：
+
+- `dangerouslySetInnerHTML`、`innerHTML`、`outerHTML`、`insertAdjacentHTML`。
+- `eval`、`new Function`、动态 script、iframe、object、embed。
+- `window.open`、`location.assign`、`location.replace`、`location.href`。
+- URL 参数到导航、HTML、脚本或资源 URL 的流转。
+- `javascript:`、`data:`、协议相对 URL。
+- `postMessage` 的 origin 校验。
+- CORS、CSP、SRI、安全响应头、第三方脚本和 CDN 资源。
+
+每个 DOM XSS finding 必须确认：输入来自用户、URL、接口、数据库或第三方内容；是否经过 sanitizer/allowlist；是否只是固定常量 HTML；是否允许事件属性、危险 URL、SVG 或 `style`；是否确实到达浏览器执行点。仅凭出现 `dangerouslySetInnerHTML` 或某个危险 API 不得判定为 Confirmed XSS。
+
+### Phase 5: Node.js 服务端风险
+
+如果存在 Node.js 服务端代码，必须检查：
+
+- 路由鉴权、越权、参数校验和 schema。
+- 原型污染、SSRF、路径穿越、文件上传、解压和临时文件。
+- 命令执行、shell 拼接、模板注入、SQL/NoSQL/LDAP 注入。
+- HTTP 请求代理、WebSocket 和不安全反序列化。
+- ReDoS、资源耗尽、body size、timeout 和 rate limit。
+- 错误栈、敏感信息、日志注入和敏感字段记录。
+- CORS、CSRF、cookie、安全响应头及实际可达的 middleware 配置。
+
+必须区分服务端实际可达风险与仅构建工具/开发依赖风险；后者不能直接升级为生产服务漏洞。
+
+### Phase 6: 配置和环境
+
+逐环境审查 `local`、`development`、`test`、`staging/uat`、`production`（只对仓库中有证据的环境下结论）。不能把 local/dev 配置直接判定为生产漏洞，也不能因为 production 看起来安全就忽略默认启动路径。
+
+检查 API HTTPS、HTTP 降级、mixed content、CORS、CSP、HSTS、cookie 属性、默认凭据、debug 模式、source map、错误页、开发服务器暴露、生产 bundle 中的测试地址/内网 IP/管理接口，以及 Dockerfile、nginx、启动脚本和构建阶段是否把错误环境打进生产产物。
+
+### Phase 7: 工程质量和安全测试保护
+
+检查 `typecheck`、`lint`、`test`、`build`、安全路径测试数量，以及鉴权、重定向、富文本、上传、支付和日志脱敏测试。检查是否存在 `test` script、CI 安全门禁、bundle/依赖/secret scanning 门禁。
+
+测试缺失只能作为工程风险，不能伪装成已确认安全漏洞；命令不可执行时记录原因和范围。
+
+## 证据和 finding 规则
+
+所有 High/Critical finding 必须具备：至少一个明确文件和行号、至少一个来源、至少一个危险汇点、明确中间调用链、受影响环境、攻击前置条件、用户/系统影响，以及修复后的验证命令或测试。每条 finding 必须标记 `Confirmed`、`Suspected`、`Needs Manual Verification` 或 `False Positive`，并提供置信度。
+
+静态关键词命中但没有 source-to-sink 证据时，只能标记 `Needs Manual Verification`，不能作为 Confirmed High。相同根因在多个文件中出现时聚合为一个 finding，并列出代表性位置；只有攻击面不同才拆分。
+
+## 固定严重级别
+
+### Critical
+
+仅用于生产环境可远程利用的 RCE、认证绕过或任意管理员接管、生产密钥/密码/token 大规模暴露、可直接导致供应链执行或生产构建接管、严重未授权高权限操作。可直接接管生产构建或具备高权限的真实供应商 key 也属于此级别。
+
+### High
+
+用于已确认的 DOM XSS 或服务端注入、认证/授权绕过、生产密码/token/支付数据进入日志或客户端不安全存储、生产明文传输敏感数据、SSRF、路径穿越、任意文件读取、高权限操作的开放重定向或 CSRF、可用但权限受限的硬编码供应商 key，以及有明确攻击路径的高危依赖。
+
+### Medium
+
+用于需要额外前置条件的 XSS/注入、非生产但可能误部署的安全配置、低权限敏感数据泄露、DoS、弱安全头、宽松 CORS 和中等风险依赖漏洞。
+
+### Low/Info
+
+用于仅本地开发风险、构建维护性问题、需要人工确认但暂无利用证据的问题、浏览器兼容性和测试覆盖不足。
+
+## 确定性与完成条件
+
+- 每次执行固定 Phase 1-7、固定严重级别、固定 finding 编号前缀和固定报告章节。
+- 不能因为发现数量多而省略认证、日志或环境配置审查。
+- 不能把“未扫描”写成“通过”，也不能把工具成功运行等同于项目安全。
+- 完成前重新读取 `.scratch/code-quality-report.md`，检查章节、行号、命令结果、证据和 Git 状态。
+- 只有报告已生成且可读、所有 Phase 都有结果（含未发现/未执行原因）、High/Critical 满足证据门槛、工作树除报告外没有本次 Skill 产生的项目文件时才算完成。

package/skills/common/code-quality/agents/openai.yaml

@@ -1,7 +1,7 @@
 interface:
   display_name: "Code Quality"
-  short_description: "检查前端安全、依赖漏洞、代码与工程质量"
-  default_prompt: "使用 $code-quality 对当前前端仓库执行只读安全与工程质量检查，并生成 .scratch/code-quality-report.md。"
+  short_description: "审计前端与 Node.js 服务安全和工程质量"
+  default_prompt: "使用 $code-quality 按固定 Phase 1-7 审计当前前端与 Node.js 服务，并生成 .scratch/code-quality-report.md。"
 
 policy:
   allow_implicit_invocation: true

package/skills/common/code-quality/references/report-format.md

@@ -1,47 +1,126 @@
 # Code Quality 报告格式
 
-报告路径固定为项目根目录 `.scratch/code-quality-report.md`。报告使用简体中文；JSON、命令、包名、规则编号、CVE/OSV 编号、路径和代码标识符保留原始格式。
+唯一输出路径是项目根目录 `.scratch/code-quality-report.md`。Skill 执行时不得在 `.scratch` 生成其他项目产物。报告使用简体中文；命令、包名、规则编号、CVE/OSV/GHSA、路径、指标和代码标识符保留原始格式。
 
 ## 固定章节
 
+报告必须按以下顺序出现；没有发现或没有适用项时保留章节并写明“未发现实现证据”或“不适用”。
+
 ```md
-# Code Quality Report
+# Code Quality Security Report
 
 ## 扫描摘要
 ## 扫描范围
 ## 项目与环境
 ## 工具执行记录
+## 认证与敏感数据路径
 ## 风险统计
 ## Critical 问题
 ## High 问题
 ## Medium/Low/Info 问题
 ## 工程质量问题
+## 未执行检查
 ## 修改建议与验证方式
 ## 未覆盖范围与残余风险
 ```
 
+## 摘要和统计
+
+摘要必须说明审计状态、审计时间、审计目标、是否存在阻断项和结论边界。风险统计必须分别列出：
+
+- Confirmed 漏洞数量。
+- Needs Manual Verification 数量。
+- Suspected 数量。
+- False Positive 数量（如有复核记录）。
+- 工程质量问题数量。
+- 依赖漏洞数量。
+- 生产环境问题数量。
+- 非生产环境问题数量。
+- 敏感 key/凭据发现数量，并区分真实值、占位符、测试假值和待确认值。
+- 未执行检查数量。
+
+状态使用：
+
+- `Passed`：声明范围内已完成所有可执行 Phase，未发现 Confirmed Critical/High 阻断项；仍必须列出残余风险。
+- `Findings`：流程完成并发现一个或多个 finding。
+- `Incomplete`：部分工具、目录、数据源或环境不可用，但仍完成其余 Phase。
+- `Blocked`：目标不可读、范围不明确或只读边界无法满足，无法完成可靠审计。
+
+`Passed` 不表示项目绝对安全、不表示完成渗透测试，也不覆盖后端、运行时基础设施或未声明环境。
+
+## 认证与敏感数据路径
+
+对每条发现的认证/会话/敏感数据路径使用表格或固定小节，至少记录：
+
+| 路径 | 来源 | API/传输 | 日志/遥测 | 存储/缓存/Cookie | 重定向 | 环境 | 结论 |
+| --- | --- | --- | --- | --- | --- | --- | --- |
+| 登录 | `src/...:line` | `src/...:line` | `src/...:line` | `src/...:line` | `src/...:line` | production | ... |
+
+必须明确记录未发现的登录、登出、刷新 token、改密、重置密码、MFA、角色判断和高权限操作证据；不能用空白表格表示已检查。
+
 ## 单条 finding
 
+每条 finding 使用稳定编号：安全 finding 使用 `CQ-SEC-001`，依赖使用 `CQ-DEP-001`，工程质量使用 `CQ-ENG-001`。同一报告重跑时，按章节、严重级别和首次出现位置保持排序，避免无关的随机重排。
+
 ```md
-### CQ-001: 问题标题
+### CQ-SEC-001: 问题标题
 
 - 严重级别：High
 - 状态：Confirmed
 - 置信度：High
+- 标准映射：OWASP A03 / ASVS V5 / CWE-79
 - 文件位置：`src/example.ts:42`
-- 类型：DOM XSS / dependency / configuration / maintainability
-- 证据：简短、脱敏的源码或工具证据
-- 影响：说明攻击面、用户影响或故障后果
-- 修改建议：给出可执行但不直接改代码的建议
+- 来源：`src/input.ts:18`
+- 危险汇点：`src/render.ts:42`
+- 数据流：`URL 参数 -> form state -> API client -> logger -> innerHTML`
+- 类型：DOM XSS / authentication / dependency / configuration / Node.js server
+- 受影响环境：production
+- 攻击前置条件：说明攻击者能力、权限和部署条件
+- 证据：简短、脱敏的源码、配置、依赖树或工具输出证据
+- 敏感值处理：只记录前缀/后缀/长度或截断指纹，绝不记录完整 key/token/私钥
+- 影响：说明用户、系统、数据或供应链后果
+- 修复建议：给出可执行但不直接改代码的建议
 - 修复优先级：立即 / 本迭代 / 后续
-- 验证方式：修复后应运行的命令或测试
+- 验证方式：修复后应运行的命令、测试或人工复核
+- 误报排除说明：如适用，说明为何不是固定常量、不可达路径或开发专用代码
 ```
 
-## 结果状态
+`Suspected`、`Needs Manual Verification` 和 `False Positive` 也必须使用同样字段；无法提供来源、汇点或数据流时，在对应字段中明确写“未确认”，不能补写推测性证据。
+
+## 敏感 key finding 补充字段
+
+发现 API key、访问 key、client secret、私钥、JWT、Bearer Token 或其他疑似凭据时，必须额外记录：
+
+- 字段/变量名及文件位置；`.env`、secret store 和私钥文件只记录路径与脱敏类型，不读取值。
+- 凭据类型和供应商（如可确认）。
+- 脱敏指纹：前缀、后缀、长度或 SHA-256 截断值；不得写完整秘密。
+- 出现位置：源码、配置模板、测试 fixture、文档、构建产物、日志、URL、客户端存储或 Git 历史。
+- 是否具备真实格式、是否可能有效、权限范围和受影响环境；无法确认时使用 `Needs Manual Verification`。
+- 是否进入生产 bundle、请求 header、日志/遥测或公开接口。
+- 建议的撤销、轮换、历史清理、权限收缩和复测步骤。
+
+普通 `key` 字段、React `key`、对象索引和业务标识不是自动漏洞。若只有字段名而没有敏感值或可验证数据流，必须记录为未确认或不报告，不能作为 High/Critical。
+
+## 依赖 finding 补充字段
+
+依赖漏洞至少包含：
+
+- 包名和实际安装版本。
+- `direct` 或 `transitive`。
+- CVE/OSV/GHSA 编号。
+- 受影响范围和修复版本/升级路径。
+- 运行时、构建时或开发时影响。
+- 是否实际进入生产 bundle；无法确认时标记 `Needs Manual Verification`。
+- `npm audit`、OSV、`npm ls` 或其他工具的代表性输出位置。
+
+相同根因可以按攻击面和依赖链聚合，但必须保留代表性漏洞证据，不得把全部工具输出压成没有包名和版本的单条问题。
+
+## 工具执行记录
+
+每个计划工具都要记录：
 
-- `Passed`：已完成声明范围，未发现确认的阻断问题。
-- `Findings`：完成扫描并发现一个或多个问题。
-- `Incomplete`：部分工具、目录或数据源不可用。
-- `Blocked`：目标不可读、范围不明确或安全边界无法满足。
+| 工具/命令 | 适用范围 | 状态 | 版本 | 证据/输出 | 未执行或失败原因 |
+| --- | --- | --- | --- | --- | --- |
+| `npm audit --json` | workspace | Executed/Incomplete/Blocked | ... | ... | ... |
 
-`Passed` 不表示项目绝对安全；必须同时列出未覆盖的运行时、后端和人工验证范围。
+网络、数据库、命令权限、项目脚本失败或源码不可读都必须进入“未执行检查”或“未覆盖范围”，不能默认为通过。

package/skills/common/trace-recorder/SKILL.md

@@ -14,7 +14,7 @@ description: 通过 trace-mcp-recorder 启动或停止 Playwright Trace 人工
 1. 从原话提取 `url`、`version`、`title`、`operator`。支持 `key=value` 和自然语言表达，不改写用户给出的值。
 2. `url` 必填；缺少时只询问目标页面地址。其他字段缺失时不要阻断录制。
 3. 调用 `start_trace_recording`，只传入用户明确提供的可选字段。服务端会优先使用 MCP 上下文中的操作人身份。
-4. 告知用户浏览器已打开：默认是 PC 端；如需移动端测试，先点击左下角“切换移动端”（iPhone 14 Pro Max，430×932，DPR 3），再点击“开始录制”；点击开始后服务会先刷新一次当前页面以捕获图片等资源，再启动计时，用户正式操作前不要提交表单或进行其他操作；操作完成后点击“停止录制”。新打开的页面会继承当前设备模式。连续 2 分钟没有真实操作时服务会自动停止并上传。真实操作经防抖后会在官方 Trace Viewer 中以 `步骤N | 页面标题 | 事件类型` 的分组节点呈现。
+4. 告知用户浏览器已打开：默认是 PC 端；如需移动端测试，先点击左下角“切换移动端”（iPhone 14 Pro Max，430×932，DPR 3），再点击“开始录制”；点击开始后服务会先启动 Trace、刷新当前 SPA 页面一次并等待 DOM、CSS、字体和图片稳定，随后才进入正式录制计时。刷新期间不要操作页面；完成后点击“停止录制”。同一 BrowserContext 会保留 cookies、localStorage 和 sessionStorage，新打开的页面会继承当前设备模式。连续 2 分钟没有真实操作时服务会自动停止并上传。真实操作经防抖后会在官方 Trace Viewer 中以 `步骤N | 页面标题 | 事件类型` 的分组节点呈现。
 
 示例：`trace-mcp-recorder 启动Playwright trace 访问https://cpd.dev.autobestdevops.com，version=v2.1.0 title=bug3452 operator=张三`。这里 `title=bug3452` 是本次录制的标题/功能描述，页面由 `url=https://cpd.dev.autobestdevops.com` 确定。