@deepstorm/cli 0.2.4 → 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/cli.js

@@ -6757,6 +6757,10 @@ function cleanInstalled(targetDir) {
       }
     }
   }
+  const hooksJson = path2.join(targetDir, ".claude", "hooks.json");
+  if (fs6.existsSync(hooksJson)) {
+    fs6.rmSync(hooksJson, { force: true });
+  }
   const hooksDir = path2.join(targetDir, ".claude", "hooks");
   if (fs6.existsSync(hooksDir)) {
     fs6.rmSync(hooksDir, { recursive: true, force: true });
@@ -7003,12 +7007,11 @@ async function runSetup(reader, targetDir, cliDir2, options, registry2) {
   }
   if (shouldInstallGlobalHooks(tools, reader)) {
     const hooksSrcDir = path6.join(cliDir2, "hooks");
-    const targetHooksDir = path6.join(targetDir, ".claude", "hooks");
-    const destHooksJson = path6.join(targetHooksDir, "hooks.json");
+    const destHooksJson = path6.join(targetDir, ".claude", "hooks.json");
     for (const tool of tools) {
       const toolHooksJsonPath = path6.join(hooksSrcDir, `${tool}-hooks.json`);
       if (fs10.existsSync(toolHooksJsonPath)) {
-        ensureDir(targetHooksDir);
+        ensureDir(path6.join(targetDir, ".claude"));
         const incoming = JSON.parse(fs10.readFileSync(toolHooksJsonPath, "utf-8"));
         mergeHooks(destHooksJson, incoming);
       }
@@ -8338,12 +8341,11 @@ function loadRegistryFromCliDir(cliDir2) {
 }
 function mergeToolHooksJson(toolNames, cliDir2, targetDir) {
   const hooksSrcDir = path18.join(cliDir2, "hooks");
-  const targetHooksDir = path18.join(targetDir, ".claude", "hooks");
-  const destHooksJson = path18.join(targetHooksDir, "hooks.json");
+  const destHooksJson = path18.join(targetDir, ".claude", "hooks.json");
   for (const tool of toolNames) {
     const toolHooksJsonPath = path18.join(hooksSrcDir, `${tool}-hooks.json`);
     if (fs20.existsSync(toolHooksJsonPath)) {
-      ensureDir(targetHooksDir);
+      ensureDir(path18.join(targetDir, ".claude"));
       const incoming = JSON.parse(fs20.readFileSync(toolHooksJsonPath, "utf-8"));
       mergeHooks(destHooksJson, incoming);
     }

package/dist/hooks/reef-hooks.json

@@ -5,7 +5,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "bash ./reef-intent-detect.sh"
+            "command": "bash .claude/hooks/reef-intent-detect.sh"
           }
         ]
       }
@@ -16,7 +16,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "bash ./reef-block-dangerous.sh"
+            "command": "bash .claude/hooks/reef-block-dangerous.sh"
           }
         ]
       },
@@ -25,7 +25,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "bash ./reef-protect-files.sh"
+            "command": "bash .claude/hooks/reef-protect-files.sh"
           },
           {
             "type": "prompt",
@@ -41,7 +41,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "bash ./reef-auto-format.sh"
+            "command": "bash .claude/hooks/reef-auto-format.sh"
           }
         ]
       },
@@ -50,7 +50,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "bash ./reef-run-tests.sh",
+            "command": "bash .claude/hooks/reef-run-tests.sh",
             "if": "Edit(*.ts)|Write(*.ts)",
             "async": true,
             "statusMessage": "Running tests..."
@@ -63,7 +63,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "bash ./reef-run-tests.sh"
+            "command": "bash .claude/hooks/reef-run-tests.sh"
           }
         ]
       }

package/dist/hooks/sweep-hooks.json

@@ -5,7 +5,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "bash ./sweep-mcp-hook.sh"
+            "command": "bash .claude/hooks/sweep-mcp-hook.sh"
           }
         ]
       }

package/dist/hooks/tide-hooks.json

@@ -5,7 +5,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "bash ./tide-session-preload.sh"
+            "command": "bash .claude/hooks/tide-session-preload.sh"
           }
         ]
       }

package/dist/skills/reef-commit/SKILL.md

@@ -39,7 +39,82 @@ git status --short
 
 若无任何变更则提示用户并退出。
 
-### 2. 审查待提交文件
+### 2. 分支名与任务相关性检查
+
+在审查待提交文件之前，先检查当前分支是否与待提交的任务相关。如果当前在 `main`/`master` 分支上则**必须**创建新分支；如果分支名与任务内容明显不匹配（如命名随意、与 OpenSpec 任务不符），建议创建新分支。
+
+```bash
+BRANCH=$(git branch --show-current)
+echo "当前分支: $BRANCH"
+
+# 检测是否需要创建新分支
+MUST_NEW_BRANCH=false
+
+# 条件一：在 main 或 master 上必须创建新分支
+if [ "$BRANCH" = "main" ] || [ "$BRANCH" = "master" ]; then
+  echo "⚠️ 当前在 $BRANCH 分支上，不允许直接提交，必须创建新分支"
+  MUST_NEW_BRANCH=true
+fi
+
+# 条件二：检查分支名是否包含 temp、wip、test、tmp、dev 等临时名称
+TEMP_PATTERN='^(temp|wip|test|tmp|dev)(/.*)?$'
+if echo "$BRANCH" | grep -qE "$TEMP_PATTERN"; then
+  echo "⚠️ 当前分支名 ($BRANCH) 看起来是临时分支，建议创建有意义的新分支"
+fi
+
+# 收集 OpenSpec 任务上下文
+for dir in openspec/changes/*/; do
+  if [ -f "$dir/proposal.md" ]; then
+    TASK_NAME=$(basename "$dir")
+    echo "发现 OpenSpec 任务: $TASK_NAME"
+  fi
+done
+```
+
+**判断规则（LLM 自行推理执行）：**
+
+1. **如果 MUST_NEW_BRANCH=true**（当前在 main/master）：
+   - 直接进入步骤 3「创建新分支」，无需询问用户
+2. **如果分支名明显不相关**（如 `temp-xxx`、`test-foo`、或随意命名与当前变更毫无关联）：
+   - 向用户说明：「当前分支 $BRANCH 与待提交的变更内容似乎不匹配，是否创建一个新分支？」
+   - 用户同意 → 进入步骤 3「创建新分支」
+   - 用户不同意 → 继续当前分支
+3. **如果分支名合理**（如与 OpenSpec 任务同名，或包含功能描述的 kebab-case）：
+   - 直接继续
+
+### 3. 创建新分支
+
+当需要创建新分支时，按以下逻辑确定分支名：
+
+**分支名生成规则（优先级从高到低）：**
+1. **OpenSpec 任务名**：如果检测到 `openspec/changes/<task>/proposal.md`，使用 `<task>` 作为分支名
+2. **用户输入**：询问用户想要的分支名
+3. **AI 推导**：根据变更内容总结生成 kebab-case 分支名（例如 `feat/add-user-auth`、`fix/login-timeout`）
+
+```bash
+# 暂存当前未提交变更
+STASHED=false
+if [ -n "$(git status --porcelain)" ]; then
+  git stash push -m "reef-commit-auto-stash"
+  STASHED=true
+fi
+
+# 创建并切换到新分支（基于 main）
+git checkout main
+git pull origin main 2>/dev/null || true
+git checkout -b <new-branch-name>
+
+# 恢复暂存的变更
+if [ "$STASHED" = true ]; then
+  git stash pop
+fi
+
+echo "✅ 已切换到新分支: <new-branch-name>"
+```
+
+> **注意**：创建新分支后，后续步骤（审查文件、范围检查、测试等）在新分支上继续执行。
+
+### 4. 审查待提交文件
 
 展示变更清单，检查敏感文件：
 
@@ -49,7 +124,7 @@ git diff --stat
 
 如包含 `.env`、凭据、证书或大型二进制文件，让用户确认或排除后再继续。
 
-### 2.4 分支范围检查
+### 5. 分支范围检查
 
 检查当前分支是否涉及多个业务领域。如果跨领域，给出拆分建议并中止提交。
 
@@ -68,7 +143,7 @@ if [ -f "$SCOPE_HOOK" ]; then
 fi
 ```
 
-### 2.5 运行单元测试
+### 6. 运行单元测试
 
 ```bash
 git status --short
@@ -84,7 +159,7 @@ git status --short
 - 测试全部通过 → 继续
 - 任一测试失败 → 提示用户修复后再提交
 
-### 3. 收集上下文
+### 7. 收集上下文
 
 ```bash
 BRANCH=$(git branch --show-current)
@@ -97,24 +172,24 @@ grep -iE '(issue|jira|lc-|proj-)' "openspec/changes/$BRANCH/proposal.md" 2>/dev/
 git log "$FORK_POINT"..HEAD --format="%B" 2>/dev/null | grep -ioP '[A-Z]+-\d+' | head -1
 ```
 
-### 4. 生成提交信息
+### 8. 生成提交信息
 
 **标题：** 优先 OpenSpec proposal 标题 → 分支名（kebab-case 转中文）→ 变更总结。依赖更新用 `更新 {依赖} 至 {版本}`，问题修复用 `解决{问题描述}的问题`。
 
 **正文：** 中/复杂变更用 `本次提交改动如下：` + 3-6 条要点。尾部隔空行加 `JIRA: {完整 URL}`（优先从 proposal.md/jira-start 元数据中获取 JIRA 链接），如有 PRD 链接加 `Ref: {URL}`。
 
-### 5. 展示并确认
+### 9. 展示并确认
 
 展示完整提交信息，用户确认即可提交；支持 `--amend` 合并；用户可要求修改后重新生成。
 
-### 6. 执行提交
+### 10. 执行提交
 
 ```bash
 git add -A && git commit -m "<完整提交信息>"
 # 用户要求 amend 时: git commit --amend -m "<完整提交信息>"
 ```
 
-### 7. 推送（仅在用户要求时）
+### 11. 推送（仅在用户要求时）
 
 ```bash
 # 新提交（有远程跟踪）：git push

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.2.4",
+  "version": "0.3.1",
   "description": "DeepStorm CLI — 一键配置项目开发环境",
   "license": "MIT",
   "author": "billkang",