dev-playbooks-cn 2.2.1 → 2.3.0

package/skills/devbooks-docs-consistency/scripts/rules-engine.sh

@@ -0,0 +1,255 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+usage() {
+  cat <<'EOF'
+Usage: rules-engine.sh [--rules <path>] [--once "remove:@pattern"] --input <file>
+
+Options:
+  --rules <path>     YAML rules file
+  --once <action>    One-time action (e.g. remove:@augment)
+  --input <file>     Document file to check
+EOF
+}
+
+RULES_PATH=""
+ONCE_ACTION=""
+INPUT_PATH=""
+
+while [[ $# -gt 0 ]]; do
+  case "$1" in
+    --rules)
+      RULES_PATH="$2"
+      shift 2
+      ;;
+    --once)
+      ONCE_ACTION="$2"
+      shift 2
+      ;;
+    --input)
+      INPUT_PATH="$2"
+      shift 2
+      ;;
+    -h|--help)
+      usage
+      exit 0
+      ;;
+    *)
+      echo "Unknown argument: $1" >&2
+      usage >&2
+      exit 2
+      ;;
+  esac
+done
+
+if [[ -z "$INPUT_PATH" ]]; then
+  echo "Missing --input" >&2
+  usage >&2
+  exit 2
+fi
+
+if [[ ! -f "$INPUT_PATH" ]]; then
+  echo "Input file not found: $INPUT_PATH" >&2
+  exit 2
+fi
+
+report_violation() {
+  local rule_id="$1"
+  local pattern="$2"
+  local file_path="$3"
+  echo "rule_id=${rule_id} file=${file_path} forbidden=${pattern}"
+}
+
+run_once_action() {
+  local action="$1"
+  local file_path="$2"
+
+  if [[ -z "$action" ]]; then
+    return 0
+  fi
+
+  if [[ "$action" != remove:* ]]; then
+    echo "Unsupported once action: $action" >&2
+    return 2
+  fi
+
+  local pattern="${action#remove:}"
+  if grep -q "$pattern" "$file_path"; then
+    echo "once_action=remove pattern=$pattern file=$file_path"
+  else
+    echo "once_action=remove pattern=$pattern file=$file_path (not found)"
+  fi
+}
+
+validate_yaml() {
+  local file_path="$1"
+  local has_rules=0
+  local has_invalid=0
+  local indent_error=0
+
+  while IFS= read -r line; do
+    if [[ "$line" =~ ^rules:[[:space:]]*(\[\])?[[:space:]]*$ ]]; then
+      has_rules=1
+      continue
+    fi
+
+    if [[ "$line" =~ ^[[:space:]]*-[[:space:]]*id:[[:space:]]* ]]; then
+      has_rules=1
+    fi
+
+    if [[ "$line" =~ ^[[:space:]]*pattern[[:space:]]+[^:]+$ ]]; then
+      has_invalid=1
+    fi
+  done < "$file_path"
+
+  if [[ "$has_invalid" -eq 1 ]]; then
+    return 1
+  fi
+  if [[ "$has_rules" -eq 0 ]]; then
+    return 1
+  fi
+  return 0
+}
+
+extract_rules() {
+  local file_path="$1"
+
+  python3 - "$file_path" <<'PY'
+import sys
+
+file_path = sys.argv[1]
+rules = []
+current = None
+
+with open(file_path, "r", encoding="utf-8") as fh:
+    for raw in fh:
+        line = raw.rstrip("\n")
+        if line.strip().startswith("rules:"):
+            continue
+        if line.lstrip().startswith("- "):
+            if current:
+                rules.append(current)
+            current = {}
+            line = line.lstrip()[2:]
+        if current is None:
+            continue
+        if ":" in line:
+            key, value = line.split(":", 1)
+            key = key.strip()
+            value = value.strip().strip('"')
+            current[key] = value
+    if current:
+        rules.append(current)
+
+for rule in rules:
+    rule_id = rule.get("id", "")
+    rule_type = rule.get("type", "")
+    rule_target = rule.get("target", "")
+    rule_action = rule.get("action", "")
+    rule_pattern = rule.get("pattern", "")
+    rule_replacement = rule.get("replacement", "")
+    if rule_id:
+        print("|".join([rule_id, rule_type, rule_target, rule_action, rule_pattern, rule_replacement]))
+PY
+}
+
+detect_conflicts() {
+  local rules_file="$1"
+  local conflict=0
+  local seen=""
+
+  while IFS='|' read -r rule_id rule_type rule_target rule_action rule_pattern rule_replacement; do
+    if [[ -z "$rule_id" ]]; then
+      continue
+    fi
+    if [[ "$rule_action" == "replace" ]]; then
+      local key="${rule_target}|${rule_pattern}"
+      local existing
+      existing=$(printf "%s\n" "$seen" | awk -F'=' -v k="$key" '$1==k {print $2}' | tail -n 1)
+      if [[ -n "$existing" && "$existing" != "$rule_replacement" ]]; then
+        echo "conflict detected: rule_id=$rule_id conflicts_with=$key"
+        conflict=1
+      else
+        seen+="${key}=${rule_replacement}\n"
+      fi
+    fi
+  done < <(extract_rules "$rules_file")
+
+  if [[ "$conflict" -eq 1 ]]; then
+    return 1
+  fi
+  return 0
+}
+
+apply_rules() {
+  local rules_file="$1"
+  local file_path="$2"
+  local violations=0
+  local matched_rules=0
+
+  while IFS='|' read -r rule_id rule_type rule_target rule_action rule_pattern rule_replacement; do
+    if [[ -z "$rule_id" ]]; then
+      continue
+    fi
+    matched_rules=1
+    case "$rule_action" in
+      check)
+        if grep -q "$rule_pattern" "$file_path"; then
+          report_violation "$rule_id" "$rule_pattern" "$file_path"
+          violations=1
+        fi
+        ;;
+      remove)
+        if grep -q "$rule_pattern" "$file_path"; then
+          report_violation "$rule_id" "$rule_pattern" "$file_path"
+          violations=1
+        fi
+        ;;
+      replace)
+        if grep -q "$rule_pattern" "$file_path"; then
+          report_violation "$rule_id" "$rule_pattern" "$file_path"
+          violations=1
+        fi
+        ;;
+      *)
+        echo "unsupported action: $rule_action" >&2
+        return 2
+        ;;
+    esac
+  done < <(extract_rules "$rules_file")
+
+  if [[ "$matched_rules" -eq 0 ]]; then
+    return 0
+  fi
+
+  if [[ "$violations" -eq 1 ]]; then
+    return 1
+  fi
+  return 0
+}
+
+if [[ -n "$ONCE_ACTION" ]]; then
+  run_once_action "$ONCE_ACTION" "$INPUT_PATH"
+  exit 0
+fi
+
+if [[ -z "$RULES_PATH" ]]; then
+  echo "no rules file provided"
+  exit 0
+fi
+
+if [[ ! -f "$RULES_PATH" ]]; then
+  echo "rules file not found: $RULES_PATH" >&2
+  exit 2
+fi
+
+if ! validate_yaml "$RULES_PATH"; then
+  echo "invalid yaml rules: $RULES_PATH" >&2
+  exit 2
+fi
+
+if ! detect_conflicts "$RULES_PATH"; then
+  exit 2
+fi
+
+apply_rules "$RULES_PATH" "$INPUT_PATH"

package/skills/devbooks-docs-consistency/scripts/scanner.sh

@@ -0,0 +1,93 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+usage() {
+  cat <<'EOF'
+Usage: scanner.sh --scan-mode <incremental|full> [--cwd <path>]
+EOF
+}
+
+SCAN_MODE=""
+CWD=""
+
+while [[ $# -gt 0 ]]; do
+  case "$1" in
+    --scan-mode)
+      SCAN_MODE="$2"
+      shift 2
+      ;;
+    --cwd)
+      CWD="$2"
+      shift 2
+      ;;
+    -h|--help)
+      usage
+      exit 0
+      ;;
+    *)
+      echo "Unknown argument: $1" >&2
+      usage >&2
+      exit 2
+      ;;
+  esac
+done
+
+if [[ -z "$SCAN_MODE" ]]; then
+  echo "Missing --scan-mode" >&2
+  usage >&2
+  exit 2
+fi
+
+if [[ -n "$CWD" ]]; then
+  cd "$CWD" || {
+    echo "invalid cwd: $CWD" >&2
+    exit 2
+  }
+fi
+
+list_full_files() {
+  if command -v git >/dev/null 2>&1; then
+    if git rev-parse --is-inside-work-tree >/dev/null 2>&1; then
+      {
+        git ls-files
+        git ls-files --others --exclude-standard
+      } | sort -u
+      return 0
+    fi
+  fi
+  find . -type f -print
+}
+
+if [[ "$SCAN_MODE" == "incremental" ]]; then
+  if ! command -v git >/dev/null 2>&1; then
+    echo "incremental scan failed, fallback to full scan" >&2
+    list_full_files
+    exit 0
+  fi
+
+  if ! git rev-parse --is-inside-work-tree >/dev/null 2>&1; then
+    echo "incremental scan failed, fallback to full scan" >&2
+    list_full_files
+    exit 0
+  fi
+
+  if ! git diff --name-only HEAD~1 >/dev/null 2>&1; then
+    echo "incremental scan failed, fallback to full scan" >&2
+    list_full_files
+    exit 0
+  fi
+
+  {
+    git diff --name-only HEAD~1
+    git ls-files --others --exclude-standard
+  } | sort -u
+  exit 0
+fi
+
+if [[ "$SCAN_MODE" == "full" ]]; then
+  list_full_files
+  exit 0
+fi
+
+echo "invalid scan mode: $SCAN_MODE" >&2
+exit 2

package/skills/devbooks-docs-consistency/scripts/style-checker.sh

@@ -0,0 +1,123 @@
+#!/usr/bin/env bash
+set -euo pipefail
+
+usage() {
+  cat <<'EOF'
+Usage: style-checker.sh [--meta <docs-maintenance.md>] [--use-emoji true|false] [--use-fancy-words true|false] --input <file>
+EOF
+}
+
+META_PATH=""
+INPUT_PATH=""
+OVERRIDE_EMOJI=""
+OVERRIDE_FANCY=""
+
+while [[ $# -gt 0 ]]; do
+  case "$1" in
+    --meta)
+      META_PATH="$2"
+      shift 2
+      ;;
+    --input)
+      INPUT_PATH="$2"
+      shift 2
+      ;;
+    --use-emoji)
+      OVERRIDE_EMOJI="$2"
+      shift 2
+      ;;
+    --use-fancy-words)
+      OVERRIDE_FANCY="$2"
+      shift 2
+      ;;
+    -h|--help)
+      usage
+      exit 0
+      ;;
+    *)
+      echo "Unknown argument: $1" >&2
+      usage >&2
+      exit 2
+      ;;
+  esac
+done
+
+if [[ -z "$INPUT_PATH" ]]; then
+  echo "Missing --input" >&2
+  usage >&2
+  exit 2
+fi
+
+if [[ ! -f "$INPUT_PATH" ]]; then
+  echo "input not found: $INPUT_PATH" >&2
+  exit 2
+fi
+
+read_meta_flag() {
+  local key="$1"
+  local file_path="$2"
+  if [[ -z "$file_path" || ! -f "$file_path" ]]; then
+    echo ""
+    return
+  fi
+  awk -v key="$key" '
+    $0 ~ key":" {gsub(/^.*: /, "", $0); gsub(/^[[:space:]]+|[[:space:]]+$/, "", $0); print $0; exit}
+  ' "$file_path"
+}
+
+use_emoji=""
+use_fancy=""
+
+if [[ -n "$OVERRIDE_EMOJI" ]]; then
+  use_emoji="$OVERRIDE_EMOJI"
+else
+  use_emoji=$(read_meta_flag "use_emoji" "$META_PATH")
+fi
+
+if [[ -n "$OVERRIDE_FANCY" ]]; then
+  use_fancy="$OVERRIDE_FANCY"
+else
+  use_fancy=$(read_meta_flag "use_fancy_words" "$META_PATH")
+fi
+
+if [[ -z "$use_emoji" ]]; then
+  use_emoji="true"
+fi
+
+if [[ -z "$use_fancy" ]]; then
+  use_fancy="true"
+fi
+
+exit_code=0
+
+if [[ "$use_emoji" == "false" ]]; then
+  if python3 - "$INPUT_PATH" <<'PY'
+import sys
+
+path = sys.argv[1]
+try:
+    with open(path, "r", encoding="utf-8", errors="ignore") as fh:
+        data = fh.read()
+except OSError:
+    sys.exit(1)
+
+for ch in data:
+    codepoint = ord(ch)
+    if 0x1F300 <= codepoint <= 0x1FAFF:
+        sys.exit(0)
+sys.exit(1)
+PY
+  then
+    echo "emoji detected in $INPUT_PATH"
+    exit_code=1
+  fi
+fi
+
+if [[ "$use_fancy" == "false" ]]; then
+  if grep -qE "(最强大脑|智能|高效|强大|优雅|完美|革命性|颠覆性)" "$INPUT_PATH"; then
+    echo "fancy words detected in $INPUT_PATH"
+    exit_code=1
+  fi
+fi
+
+exit "$exit_code"

package/skills/devbooks-entropy-monitor/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: devbooks-entropy-monitor
 description: devbooks-entropy-monitor：定期采集系统熵度量（结构熵/变更熵/测试熵/依赖熵），生成量化报告，当指标超阈值时建议重构。用户说"熵度量/复杂度趋势/重构预警/代码健康/技术债务度量"等时使用。
+recommended_experts: ["Performance Engineer", "System Architect"]
 allowed-tools:
   - Glob
   - Grep
@@ -150,39 +151,6 @@ allowed-tools:
 
 ---
 
-## MCP 增强
-
-本 Skill 支持 MCP 运行时增强，自动检测并启用高级功能。
-
-MCP 增强规则参考：`skills/_shared/MCP增强模板.md`
-
-### 依赖的 MCP 服务
-
-| 服务 | 用途 | 超时 |
-|------|------|------|
-| `mcp__ckb__getHotspots` | 获取热点文件分析 | 2s |
-| `mcp__ckb__getStatus` | 检测 CKB 索引可用性 | 2s |
-
-### 检测流程
-
-1. 调用 `mcp__ckb__getStatus`（2s 超时）
-2. 若 CKB 可用 → 使用 `getHotspots` 获取精确热点分析
-3. 若超时或失败 → 降级到 Git 历史统计
-
-### 增强模式 vs 基础模式
-
-| 功能 | 增强模式 | 基础模式 |
-|------|----------|----------|
-| 热点分析 | CKB 实时分析（含复杂度） | Git log 变更频率统计 |
-| 耦合检测 | 调用图分析 | 文件共变分析 |
-| 趋势预测 | 基于复杂度变化率 | 基于变更频率 |
-
-### 降级提示
-
-当 MCP 不可用时，输出以下提示：
-
-```
-⚠️ CKB 不可用，使用 Git 历史进行熵度量。
-热点分析基于变更频率，不含代码复杂度数据。
-```
+## MCP 说明
 
+本 Skill 不依赖 MCP 服务，无需运行时检测。

package/skills/devbooks-impact-analysis/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: devbooks-impact-analysis
 description: devbooks-impact-analysis：跨模块/跨文件/对外契约变更前做影响分析，产出可直接写入 proposal.md 的 Impact 部分（Scope/Impacts/Risks/Minimal Diff/Open Questions）。用户说"做影响分析/改动面控制/引用查找/受影响模块/兼容性风险"等时使用。
+recommended_experts: ["System Architect"]
 allowed-tools:
   - Glob
   - Grep
@@ -129,42 +130,6 @@ AI：[使用 Grep/CKB 分析引用]
 
 ---
 
-## MCP 增强
-
-本 Skill 支持 MCP 运行时增强，自动检测并启用高级功能。
-
-MCP 增强规则参考：`skills/_shared/MCP增强模板.md`
-
-### 依赖的 MCP 服务
-
-| 服务 | 用途 | 超时 |
-|------|------|------|
-| `mcp__ckb__analyzeImpact` | 符号级影响分析 | 2s |
-| `mcp__ckb__findReferences` | 精确引用查找 | 2s |
-| `mcp__ckb__getCallGraph` | 调用图分析 | 2s |
-| `mcp__ckb__getStatus` | 检测 CKB 索引可用性 | 2s |
-
-### 检测流程
-
-1. 调用 `mcp__ckb__getStatus`（2s 超时）
-2. 若 CKB 可用 → 使用 `analyzeImpact` 和 `findReferences` 进行精确分析
-3. 若超时或失败 → 降级到基础模式（Grep 文本搜索）
-
-### 增强模式 vs 基础模式
-
-| 功能 | 增强模式 | 基础模式 |
-|------|----------|----------|
-| 引用查找 | 符号级精确匹配 | 文本 Grep 搜索 |
-| 影响范围 | 调用图传递分析 | 直接引用统计 |
-| 风险评估 | 基于调用深度量化 | 基于文件数量估算 |
-| 跨模块分析 | 自动识别模块边界 | 需手动指定范围 |
-
-### 降级提示
-
-当 MCP 不可用时，输出以下提示：
-
-```
-⚠️ CKB 不可用，使用 Grep 文本搜索进行影响分析。
-分析结果可能不够精确，建议手动生成 SCIP 索引后重新分析。
-```
+## MCP 说明
 
+本 Skill 不依赖 MCP 服务，无需运行时检测。

package/skills/devbooks-implementation-plan/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: devbooks-implementation-plan
 description: devbooks-implementation-plan：从设计文档推导编码计划（tasks.md），输出可跟踪的主线计划/临时计划/断点区，并绑定验收锚点。用户说"写编码计划/Implementation Plan/tasks.md/任务拆解/并行拆分/里程碑/验收锚点"等时使用。
+recommended_experts: ["System Architect", "Product Manager"]
 allowed-tools:
   - Glob
   - Grep
@@ -153,9 +154,6 @@ implementation-plan → test-owner (会话A) → coder (会话B)
 
 ---
 
-## MCP 增强
+## MCP 说明
 
 本 Skill 不依赖 MCP 服务，无需运行时检测。
-
-MCP 增强规则参考：`skills/_shared/MCP增强模板.md`
-

package/skills/devbooks-proposal-author/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: devbooks-proposal-author
 description: devbooks-proposal-author：撰写变更提案 proposal.md（Why/What/Impact + Debate Packet），作为后续 Design/Spec/Plan 的入口。对设计性决策会呈现选项给用户选择。用户说"写提案/proposal/为什么要改/影响范围/坏味道重构提案"等时使用。
+recommended_experts: ["Product Manager", "System Architect"]
 allowed-tools:
   - Glob
   - Grep
@@ -175,9 +176,12 @@ proposal-author → [impact-analysis] → design-doc → [spec-contract] → imp
 
 ---
 
-## MCP 增强
+## Challenger 审视
 
-本 Skill 不依赖 MCP 服务，无需运行时检测。
+在完成提案草案后，执行 Challenger 审视，重点检查遗漏的约束与不确定性，并参考完备性方法论。
+
+- 方法论参考：[完备性思维框架](../_shared/references/完备性思维框架.md)
 
-MCP 增强规则参考：`skills/_shared/MCP增强模板.md`
+## MCP 说明
 
+本 Skill 不依赖 MCP 服务，无需运行时检测。

package/skills/devbooks-proposal-challenger/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: devbooks-proposal-challenger
 description: devbooks-proposal-challenger：对 proposal.md 发起质疑（Challenger）+ 查漏补缺，指出风险/遗漏/不一致并给结论，发现缺失的验收标准和未覆盖场景。用户说"质疑提案/挑刺/风险评估/提案对辩 challenger/查漏补缺"等时使用。
+recommended_experts: ["Product Manager", "System Architect"]
 allowed-tools:
   - Glob
   - Grep
@@ -78,9 +79,6 @@ allowed-tools:
 
 ---
 
-## MCP 增强
+## MCP 说明
 
 本 Skill 不依赖 MCP 服务，无需运行时检测。
-
-MCP 增强规则参考：`skills/_shared/MCP增强模板.md`
-

package/skills/devbooks-proposal-judge/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: devbooks-proposal-judge
 description: devbooks-proposal-judge：对 proposal 阶段进行裁决（Judge），输出 Approved/Revise/Rejected 并写回 proposal.md 的 Decision Log。用户说"裁决提案/提案评审/Approved Revise Rejected/decision log"等时使用。
+recommended_experts: ["Product Manager", "System Architect"]
 allowed-tools:
   - Glob
   - Grep
@@ -70,9 +71,6 @@ allowed-tools:
 
 ---
 
-## MCP 增强
+## MCP 说明
 
 本 Skill 不依赖 MCP 服务，无需运行时检测。
-
-MCP 增强规则参考：`skills/_shared/MCP增强模板.md`
-

package/skills/devbooks-reviewer/SKILL.md

@@ -1,6 +1,7 @@
 ---
 name: devbooks-reviewer
 description: devbooks-reviewer：以 Reviewer 角色做可读性/一致性/依赖健康/坏味道审查，只输出审查意见与可执行建议，不讨论业务正确性。用户说"帮我做代码评审/review 可维护性/坏味道/依赖风险/一致性建议"，或在 DevBooks apply 阶段以 reviewer 执行时使用。
+recommended_experts: ["System Architect", "Security Expert"]
 allowed-tools:
   - Glob
   - Grep
@@ -206,40 +207,6 @@ Review 通过后，Reviewer 必须执行：
 
 ---
 
-## MCP 增强
-
-本 Skill 支持 MCP 运行时增强，自动检测并启用高级功能。
-
-MCP 增强规则参考：`skills/_shared/MCP增强模板.md`
-
-### 依赖的 MCP 服务
-
-| 服务 | 用途 | 超时 |
-|------|------|------|
-| `mcp__ckb__getHotspots` | 检测热点文件，优先审查 | 2s |
-| `mcp__ckb__getStatus` | 检测 CKB 索引可用性 | 2s |
-
-### 检测流程
-
-1. 调用 `mcp__ckb__getStatus`（2s 超时）
-2. 若 CKB 可用 → 调用 `mcp__ckb__getHotspots` 获取热点文件
-3. 对热点文件进行优先审查
-4. 若超时或失败 → 降级到基础模式
-
-### 增强模式 vs 基础模式
-
-| 功能 | 增强模式 | 基础模式 |
-|------|----------|----------|
-| 热点优先审查 | 自动识别高风险文件 | 按变更顺序审查 |
-| 依赖方向检查 | 基于模块图分析 | 基于文件路径推断 |
-| 循环依赖检测 | CKB 精确检测 | Grep 启发式检测 |
-
-### 降级提示
-
-当 MCP 不可用时，输出以下提示：
-
-```
-⚠️ CKB 不可用，无法进行热点优先审查。
-按变更文件顺序进行审查。
-```
+## MCP 说明
 
+本 Skill 不依赖 MCP 服务，无需运行时检测。