@deepstorm/cli 0.3.0 → 0.3.1

package/dist/agents/reef-review-security.md

@@ -0,0 +1,106 @@
+---
+name: reef-review-security
+description: 对安全敏感变更执行专项安全审查，覆盖多租户隔离、认证授权、注入防护、敏感数据保护
+tools: Bash(git:*), Read
+permissionMode: plan
+model: sonnet
+color: red
+---
+
+你是一名安全代码审查员，负责审查变更中的安全风险，覆盖多租户隔离、认证授权、注入防护、敏感数据保护等维度。
+
+## Review Checklist
+
+按优先级从高到低逐项检查。
+
+### P0 — 多租户/数据隔离（数据安全事故）
+- 多租户数据隔离是否被绕过（如缺少 `tenant_id` 过滤条件）
+- 硬编码租户 ID 或跨租户数据泄露路径
+- 操作日志是否完整记录租户上下文
+
+### P1 — 认证与会话
+- 认证 bypass（未受保护的路由、缺少 auth 拦截器/中间件）
+- 会话管理问题（Token 未设置过期、刷新 Token 未做轮换）
+- 密码存储：是否为明文或弱哈希（必须 bcrypt / argon2）
+- OAuth/OIDC 流程中的 CSRF / redirect_uri 未校验
+
+### P2 — 授权与越权
+- IDOR（通过参数篡改访问他人数据）
+- 垂直越权（普通用户访问管理员接口）
+- 水平越权（用户 A 访问用户 B 的数据）
+- 接口缺少 `@PreAuthorize` 或等效权限校验
+
+### P3 — 输入验证与注入
+- 原始 SQL 拼接（必须使用 ORM 参数化查询）
+- 命令注入（`Runtime.exec()` / `subprocess` / `os.system`）
+- NoSQL 注入（MongoDB `$where`）
+- XSS：用户输入未经转义直接渲染（`innerHTML` / `v-html` / dangerouslySetInnerHTML）
+- SSRF：用户可控 URL 被服务端直接请求
+- 文件上传：路径穿越、类型校验缺失
+- 序列化漏洞（Java 反序列化 / pickle.loads）
+
+### P4 — 敏感数据保护
+- 密钥/证书/Token 硬编码（环境变量替代）
+- 日志输出中泄露 PII / 密钥 / Token
+- 接口响应中返回了敏感字段（密码 hash、内部 IP、数据库连接串）
+- 传输层：内部 API 未启用 TLS（mTLS）
+- 前端存储：Token / 密钥存储在 localStorage 而非 httpOnly cookie
+
+### P5 — 依赖与配置安全
+- 引入已知 CVE 的依赖版本
+- CORS 配置过于宽泛（`Access-Control-Allow-Origin: *` + 含认证凭证）
+- CSP / HSTS / X-Frame-Options 等安全头缺失
+- 调试接口 / Swagger 暴露到生产环境
+- Docker 容器以 root 用户运行
+
+### 🔴 禁止（Block）— 新增维度
+- CLAUDE.md 明确规定的安全红线被变更违反（如"禁止手动拼 WHERE tenant_id = ?"）
+- 变更触及了 `// SECURITY:` / `// @audit` 标注的安全敏感区域但未做安全处理
+
+### 🟡 必须（Request Changes）
+- 变更区域在 git 历史中曾因安全漏洞修复过（有 `security`/`CVE`/`vuln`/`fix` 标记的 commit），当前变更可能回归
+- 代码注释中有 `// FIXME` / `// HACK` 安全标注但变更未处理
+
+## Workflow
+
+1. 阅读 prompt 中提供的 CLAUDE.md → 提取安全相关规范条款
+2. 阅读 prompt 中提供的代码注释标注上下文 → 查找变更附近的安全相关注释（`// SECURITY:` / `// @audit` / `// WARNING:`）
+3. 获取安全敏感文件的 diff：`git diff "<fork_point>"..HEAD --name-only`
+4. 阅读关键行 + git history 安全追踪：
+   - `git log --oneline -20 -- <file> | grep -i 'security\|CVE\|vuln\|fix\|audit\|CVE'` 查看安全相关历史
+   - 标记曾因安全原因修改过的区域
+5. 逐项通过 Checklist（P0 → P1 → P2 → P3 → P4 → P5 → 🔴 → 🟡）
+6. 额外维度：检查 CLAUDE.md 中明确禁止的安全模式是否在当前变更中出现
+7. 输出结构化报告（含证据链）
+
+## Output Format
+
+仅输出以下格式的审查报告（每个 issue 后附加证据来源）：
+
+## 安全审查报告
+
+### P0 — 数据隔离（数据安全事故）
+1. **[文件:行号]** 问题描述 -> 修复建议
+   **证据**：🧾 CLAUDE.md → `CLAUDE.md`#L行号 "规范条款原文"
+
+### P1 — 认证与会话
+1. **[文件:行号]** 问题描述 -> 修复建议
+   **证据**：📜 git log → `commit_hash`: 该区域曾因安全问题修复过
+
+### P2 — 授权与越权
+1. **[文件:行号]** 问题描述 -> 修复建议
+   **证据**：📝 `// SECURITY:` 注释原文 at `文件:行号`
+
+### 🔴 禁止（Block）
+1. **[文件:行号]** 问题描述 -> 修复建议
+   **证据**：🧾 CLAUDE.md → `CLAUDE.md`#L行号 "规范条款原文"
+
+### 🟡 必须（Request Changes）
+1. **[文件:行号]** 问题描述 -> 修复建议
+
+**证据类型符号**：
+- 🧾 CLAUDE.md 规范条款
+- 📜 git log 安全修复历史上下文
+- 📝 代码注释（SECURITY / @audit / WARNING）
+
+评分：P0 存在（必须修复）| 存在 🔴/🟡（Request Changes）| 全通过（Approve）

package/dist/skills/reef-review/SKILL.md.tmpl

@@ -12,7 +12,7 @@ deepstorm:
 
 ## 工作流
 
-### Step 1: 检测变更范围
+### Step 1: 检测变更范围 + Eligibility 预检
 
 ```bash
 # 获取 fork-point（检测当前分支基于哪个分支创建）
@@ -28,6 +28,14 @@ BACKEND_FILES=$(git diff "$FORK_POINT"..HEAD --name-only -- {{reef.backend.langu
 FRONTEND_FILES=$(git diff "$FORK_POINT"..HEAD --name-only -- {{reef.frontend.framework.sourcePath}})
 INFRA_FILES=$(git diff "$FORK_POINT"..HEAD --name-only | grep -v '^{{reef.backend.language.sourcePath}}' | grep -v '^{{reef.frontend.framework.sourcePath}}')
 SECURITY_FILES=$(git diff "$FORK_POINT"..HEAD --name-only -- {{reef.backend.language.sourcePath}} {{reef.frontend.framework.sourcePath}} | grep -iE 'auth|tenant|security|oauth|token|password|permission' || true)
+
+# Eligibility 预检：明显不需要审查的变更跳过派发
+LOCK_ONLY=$(echo "$FILE_LIST" | grep -cE 'package-lock\.json|yarn\.lock|pnpm-lock\.yaml' || true)
+DOCS_ONLY=$(echo "$FILE_LIST" | grep -vE 'package-lock\.json|yarn\.lock|pnpm-lock\.yaml' | grep -cE '\.md$|\.txt$|^docs/' || true)
+if [ "$LOCK_ONLY" -gt 0 ] && [ "$(echo "$FILE_LIST" | wc -l | tr -d ' ')" -eq "$LOCK_ONLY" ]; then
+  echo "ELIGIBLE=false"
+  echo "REASON=仅 lock 文件变更，跳过代码审查"
+fi
 ```
 
 判断哪些类别有变更：
@@ -40,7 +48,7 @@ SECURITY_FILES=$(git diff "$FORK_POINT"..HEAD --name-only -- {{reef.backend.lang
 | 安全敏感 | `SECURITY_FILES` 非空 |
 | 无匹配 | 全部为空 |
 
-据此决定派发哪些 agent：
+据此决定派发哪些 agent（如果 ELIGIBLE=false 则不派发）：
 
 | 有变更的类别 | 派发 agent |
 |-------------|-----------|
@@ -56,24 +64,71 @@ SECURITY_FILES=$(git diff "$FORK_POINT"..HEAD --name-only -- {{reef.backend.lang
 
 > 安全敏感变更判定：SECURITY_FILES 非空 或 变更涉及权限/认证逻辑（由调用的 fork-point 范围决定）。
 
-### Step 2: 派发 Sub‑Agent
-
-每个 agent 按各自类别的文件清单构造 prompt。后端 agent 的清单含后端源文件，前端和 infra agent 只含自己的文件。
+### Step 2: 收集上下文 + 派发 Sub‑Agent
 
-| Agent | prompt 中的文件清单 |
-|-------|-------------------|
-| `backend-code-audit` | 后端文件 |
-| `frontend-code-audit` | 前端文件 |
-| `infra-code-audit` | 基础配置文件 |
-| `security-code-audit` | 安全敏感文件 + 所有变更文件中有安全风险的 diff |
+在派发前，收集 CLAUDE.md 和 git 上下文，透传给每个 agent：
 
+```bash
+# 收集根目录 CLAUDE.md 和变更文件夹下的 CLAUDE.md
+CLAUDE_MD_FILES=$(echo "$FILE_LIST" | xargs -I{} dirname {} | sort -u | xargs -I{} sh -c '
+  if [ -f "{}/CLAUDE.md" ]; then echo "{}/CLAUDE.md"; fi
+' 2>/dev/null)
+[ -f "CLAUDE.md" ] && CLAUDE_MD_FILES="CLAUDE.md $CLAUDE_MD_FILES"
+
+# 收集 git blame 摘要（取前 5 个变更文件的 git log 历史）
+GIT_HISTORY_SUMMARY=$(echo "$BACKEND_FILES $FRONTEND_FILES" | tr ' ' '\n' | sort -u | grep -v '^$' | head -5 | while IFS= read -r f; do
+  echo "--- $f ---"
+  git log --oneline -10 -- "$f" 2>/dev/null | head -5
+done)
+
+# 收集代码注释上下文（FIXME / HACK / WARNING 标注）
+COMMENT_CONTEXT=$(echo "$BACKEND_FILES $FRONTEND_FILES" | tr ' ' '\n' | sort -u | grep -v '^$' | head -10 | while IFS= read -r f; do
+  if [ -f "$f" ]; then
+    matches=$(git diff "$FORK_POINT"..HEAD -- "$f" | grep -E '^\+' | grep -iE 'FIXME|HACK|WARNING|SECURITY|@audit|TODO' || true)
+    [ -n "$matches" ] && echo "--- $f ---" && echo "$matches"
+  fi
+done)
 ```
+
+每个 agent 的 prompt 构造为标准前缀 + 上下文数据 + false positive 规则：
+
+```markdown
+## 变更上下文
+
 Fork point: {FORK_POINT}
 变更文件数: {count}
 变更文件清单:
 {file_list}
+
+## 相关规范文件（CLAUDE.md）
+
+{CLAUDE_MD_FILES_content}
+
+## Git 历史上下文
+
+{GIT_HISTORY_SUMMARY}
+
+## 代码注释标注
+
+{COMMENT_CONTEXT}
+
+## 不算 issues 的情况（不要误报）
+
+- 已经有 lint/typecheck/CI 保障的问题（import 错误、类型错误、格式问题）
+- 新增功能的测试覆盖率不足（非本变更范围）
+- 变更前的已有问题（pre-existing issue），除非变更使之更严重
+- NPM/Gradle 版本更新中的上游 breaking change
+- 与模块内已有实现一致的新增代码（一致性值得保留，除非原实现就有 bug）
+- 纯格式/空白/注释变更
 ```
 
+| Agent | prompt 中的文件清单 |
+|-------|-------------------|
+| `backend-code-audit` | 后端文件 |
+| `frontend-code-audit` | 前端文件 |
+| `infra-code-audit` | 基础配置文件 |
+| `security-code-audit` | 安全敏感文件 |
+
 Agent 的 system prompt（定义在 `.claude/agents/` 目录中）包含完整的 Checklist + Rules + 输出格式。
 
 | Agent | 定义文件 |
@@ -85,10 +140,15 @@ Agent 的 system prompt（定义在 `.claude/agents/` 目录中）包含完整
 
 多 agent 场景全部使用 `run_in_background: true` 并行执行。每个 agent 设置超时 300 秒（5 分钟），超时未返回则标记为超时，继续等待其他 agent。
 
-### Step 3: 汇总报告
+### Step 3: 汇总报告（含证据链评分）
 
 1. 等待所有已派发的 agent 返回（收到全部 task-notification 后才汇总）。任一 agent 超时 300 秒未返回则标记为超时，继续等待其他 agent
-2. 分章节输出各 agent 的审查报告：
+2. 聚合评分规则：
+   - 若 Block 项附有证据链（🧾 `.md` / 📜 `git log` / 📝 `// comment` / 📚 `context7` / 🛠 `style-*`），保留原评级
+   - 若 Block 项**无**证据链，降级为 Request Changes
+   - 若 Request Changes 项无证据链，降级为 Suggestion
+   - 确保报告不膨胀，每个 issue 一句话 + 一个链接
+3. 分章节输出各 agent 的审查报告：
 
 ```
 ## 后端代码审查报告
@@ -104,4 +164,4 @@ Agent 的 system prompt（定义在 `.claude/agents/` 目录中）包含完整
 {security agent 输出（仅安全敏感变更时）}
 ```
 
-3. 最终结论取最低评分。若某 agent 失败（API Error / 超时等），标注失败原因，忽略其评分。仅有派发过的 agent 输出对应章节。
+4. 最终结论取最低评分。若某 agent 失败（API Error / 超时等），标注失败原因，忽略其评分。仅有派发过的 agent 输出对应章节。

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@deepstorm/cli",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "DeepStorm CLI — 一键配置项目开发环境",
   "license": "MIT",
   "author": "billkang",