@ranger1/dx 0.1.98 → 0.1.100

package/README.md

@@ -192,53 +192,6 @@ target（端）不写死，由 `env-policy.jsonc.targets` 定义；`commands.jso
 
 查看 `example/`：包含一个最小可读的 `dx/config` 配置示例，以及如何在一个 pnpm+nx monorepo 中接入 dx。
 
-## PR Review Loop（自动评审-修复闭环）
-
-仓库内提供了基于 Codex Skill 的 PR 评审自动化工作流：并行评审 -> 聚合结论 -> 生成修复清单 -> 自动修复 -> 再评审，最多循环 3 轮，用于让 PR 更快收敛。
-
-### 什么时候用
-
-- PR 变更较大、想要更系统地覆盖安全/性能/可维护性问题
-- 希望在 CI 通过前提下，把评审建议落成可执行修复清单（fixFile）
-- 希望避免同一个问题在不同轮次被反复提出（Decision Log）
-
-### 如何运行
-
-在 Codex 会话中触发该技能：
-
-```text
-使用 $pr-review-loop 对 PR #<PR_NUMBER> 执行审核闭环
-```
-
-技能入口与说明见：
-
-- `skills/pr-review-loop/SKILL.md`
-- `skills/pr-review-loop/references/agents/*.md`
-
-### 工作流概览
-
-- 预检（`pr-precheck`）：先做编译/基础 gate，不通过则终止流程
-- 获取上下文（`pr-context`）：生成本轮上下文缓存 `contextFile` 与 `runId`
-- 并行评审（reviewers）：按 `./reviewer/*-reviewer.md` 并行审查并产出 reviewFile
-- 聚合（`pr-review-aggregate` 模式 A）：合并评审结果、去重、发布 Review Summary、生成 `fixFile`
-- 修复（`fixer`）：按 `fixFile` 执行修复并产出 `fixReportFile`
-- 发布修复报告（`pr-review-aggregate` 模式 B）
-
-### 缓存文件（项目内 `./.cache/`）
-
-该流程中间产物写入 `./.cache/`，并在各阶段传递相对路径：
-
-- `./.cache/pr-context-pr<PR>-r<ROUND>-<RUN_ID>.md`（contextFile）
-- `./.cache/review-<ROLE_CODE>-pr<PR>-r<ROUND>-<RUN_ID>.md`（reviewFile）
-- `./.cache/fix-pr<PR>-r<ROUND>-<RUN_ID>.md`（fixFile）
-- `./.cache/fix-report-pr<PR>-r<ROUND>-<RUN_ID>.md`（fixReportFile）
-
-### Decision Log（跨轮次决策日志）
-
-- 文件：`./.cache/decision-log-pr<PR_NUMBER>.md`
-- 作用：记录每轮 Fixed/Rejected 结论，后续轮次用于过滤重复问题
-- 规则：默认 append-only，保留历史决策用于收敛
-
 ## 命令
 
 dx 的命令由 `dx/config/commands.json` 驱动，并且内置了一些 internal runner（避免项目侧依赖任何 `scripts/lib/*.js`）：
@@ -266,6 +219,12 @@ dx test e2e backend apps/backend/e2e/auth
 dx test e2e quantify apps/quantify/e2e/health/health.e2e-spec.ts
 ```
 
+关于 `dx initial`：
+
+- `dx initial` 会把 npm 包内置的 `skills/` 同步到 `~/.codex/skills` 与 `~/.claude/skills`。
+- 包内管理的同名 skill 目录会按当前包内容替换；旧版本中已删除的官方 skill 会被清理。
+- 不属于包内管理的其他用户自有 skill 目录会保留。
+
 关于 `help`：
 
 - `dx --help`

package/lib/codex-initial.js

@@ -4,18 +4,14 @@ import os from 'node:os'
 
 import { logger } from './logger.js'
 
-const DEPRECATED_SKILL_DIRS = ['pr-review-loop', 'git-commit-and-pr']
+const DEPRECATED_SKILL_DIRS = ['pr-review-loop', 'git-commit-and-pr', 'autospec']
+const TEMP_DIR_PATTERN = /^\..+\.(tmp|backup)-\d+-\d+$/
 
 async function collectAllFiles(dir) {
   const out = []
 
   async function walk(current) {
-    let entries
-    try {
-      entries = await fs.readdir(current, { withFileTypes: true })
-    } catch {
-      return
-    }
+    const entries = await fs.readdir(current, { withFileTypes: true })
 
     for (const entry of entries) {
       const full = join(current, entry.name)
@@ -42,6 +38,15 @@ async function ensureDir(path) {
   await fs.mkdir(path, { recursive: true })
 }
 
+async function pathExists(path) {
+  try {
+    await fs.access(path)
+    return true
+  } catch {
+    return false
+  }
+}
+
 async function assertDirExists(path, label) {
   try {
     const st = await fs.stat(path)
@@ -55,18 +60,53 @@ async function assertDirExists(path, label) {
 }
 
 async function copyDirMerge({ srcDir, dstDir }) {
-  const files = await collectAllFiles(srcDir)
-
-  for (const file of files) {
-    const rel = relative(srcDir, file)
-    const topLevelDir = rel.split('/')[0]
-    if (DEPRECATED_SKILL_DIRS.includes(topLevelDir)) continue
-    const target = join(dstDir, rel)
-    await ensureDir(dirname(target))
-    await fs.copyFile(file, target)
+  const entries = await fs.readdir(srcDir, { withFileTypes: true })
+  let fileCount = 0
+
+  for (const entry of entries) {
+    if (!entry.isDirectory()) continue
+    if (DEPRECATED_SKILL_DIRS.includes(entry.name)) continue
+
+    const srcSkillDir = join(srcDir, entry.name)
+    const dstSkillDir = join(dstDir, entry.name)
+    const token = `${process.pid}-${Date.now()}`
+    const tmpSkillDir = join(dstDir, `.${entry.name}.tmp-${token}`)
+    const backupSkillDir = join(dstDir, `.${entry.name}.backup-${token}`)
+    let hasBackup = false
+
+    try {
+      const files = await collectAllFiles(srcSkillDir)
+      fileCount += files.length
+      await ensureDir(tmpSkillDir)
+      for (const file of files) {
+        const rel = relative(srcSkillDir, file)
+        const target = join(tmpSkillDir, rel)
+        await ensureDir(dirname(target))
+        await fs.copyFile(file, target)
+      }
+
+      if (await pathExists(dstSkillDir)) {
+        await fs.rename(dstSkillDir, backupSkillDir)
+        hasBackup = true
+      }
+      await fs.rename(tmpSkillDir, dstSkillDir)
+      if (hasBackup) {
+        await fs.rm(backupSkillDir, { recursive: true, force: true })
+      }
+    } catch (error) {
+      await fs.rm(tmpSkillDir, { recursive: true, force: true })
+      if (hasBackup && !(await pathExists(dstSkillDir))) {
+        await fs.rename(backupSkillDir, dstSkillDir)
+        hasBackup = false
+      }
+      if (hasBackup) {
+        await fs.rm(backupSkillDir, { recursive: true, force: true })
+      }
+      throw error
+    }
   }
 
-  return { fileCount: files.length }
+  return { fileCount }
 }
 
 async function removeDeprecatedSkillDirs(skillsDir) {
@@ -75,6 +115,15 @@ async function removeDeprecatedSkillDirs(skillsDir) {
   }
 }
 
+async function removeStaleTempDirs(skillsDir) {
+  const entries = await fs.readdir(skillsDir, { withFileTypes: true })
+  for (const entry of entries) {
+    if (!entry.isDirectory()) continue
+    if (!TEMP_DIR_PATTERN.test(entry.name)) continue
+    await fs.rm(join(skillsDir, entry.name), { recursive: true, force: true })
+  }
+}
+
 export async function runCodexInitial(options = {}) {
   const packageRoot = options.packageRoot
   if (!packageRoot) throw new Error('runCodexInitial: 缺少 packageRoot')
@@ -91,8 +140,10 @@ export async function runCodexInitial(options = {}) {
   const stats = []
   for (const target of targets) {
     await ensureDir(target.dir)
+    await removeStaleTempDirs(target.dir)
     await removeDeprecatedSkillDirs(target.dir)
     const copyStats = await copyDirMerge({ srcDir: srcSkillsDir, dstDir: target.dir })
+    await removeStaleTempDirs(target.dir)
     await removeDeprecatedSkillDirs(target.dir)
     stats.push({ ...target, ...copyStats })
   }

package/lib/validate-env.js

@@ -245,15 +245,31 @@ function isAllowedBySimpleGlob(path, globs) {
       if (glob === path) return true
       continue
     }
-    // Only support a single trailing '*' for now.
-    if (glob.endsWith('*')) {
-      const prefix = glob.slice(0, -1)
-      if (path.startsWith(prefix)) return true
-    }
+    if (globToRegex(glob).test(path)) return true
   }
   return false
 }
 
+function globToRegex(glob) {
+  let pattern = ''
+  for (let i = 0; i < glob.length; i++) {
+    const ch = glob[i]
+    if (ch === '*') {
+      if (glob[i + 1] === '*') {
+        pattern += '.*'
+        i += 1
+      } else {
+        pattern += '[^/]*'
+      }
+    } else if ('\\^$+?.()|[]{}'.includes(ch)) {
+      pattern += `\\${ch}`
+    } else {
+      pattern += ch
+    }
+  }
+  return new RegExp(`^${pattern}$`)
+}
+
 function findOverlaps(namedSets) {
   const seen = new Map()
   const overlaps = []

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ranger1/dx",
-  "version": "0.1.98",
+  "version": "0.1.100",
   "type": "module",
   "license": "MIT",
   "repository": {

package/skills/doctor/SKILL.md

@@ -1,63 +1,66 @@
 ---
 name: doctor
-description: Use when 需要在本机一次性体检并修复 Codex 开发环境，包括 python3/python 别名、dx 初始化、agent-browser+Chromium 与 ripgrep。
+description: 仅在用户显式调用 $doctor 或明确要求使用 doctor 技能时使用；不要通过关键词自动触发。
 ---
 
 # Doctor
 
 ## 概览
 
-执行本技能时，优先运行 `scripts/doctor.sh`，由脚本完成并行检测、自动修复、最多三轮重试与最终报告。
-并且在任何本地检测/修复动作前，必须先拉取并完整执行一次远程 INSTALL 指令（不是只下载）。
+本技能用于把当前机器调整到可稳定运行 Codex 开发工作流的状态。
 
-## 适用场景
+不要把执行路径写死。模型应先识别当前系统、shell、包管理器、已有工具版本与用户权限，再自行选择最合适的检测和修复方式。完成修复后必须复检，并给出可读报告。
 
-- 新机器初始化 Codex 开发环境。
-- 发现命令缺失或版本漂移，希望一次性修复。
+## 目标状态
 
-## 执行步骤
+最终环境应尽量满足：
 
-1. 强制先执行并落地（不可跳过）：
+- `python3` 可用。
+- `python` 可调用 Python 3，或有清晰说明当前系统无需/不应创建该别名。
+- `node`、`npm`、`pnpm` 可用，且满足当前工作流需要。
+- `dx` 可用，并已完成必要初始化。
+- `agent-browser` 可用，且 Chromium/浏览器依赖已安装到可运行状态。
+- `rg`（ripgrep）可用。
+- 常用 PATH 配置在当前 shell 中可生效；若需要持久化，说明写入了哪个 shell 配置文件。
 
-```bash
-INSTALL_URL="https://raw.githubusercontent.com/obra/superpowers/refs/heads/main/.codex/INSTALL.md"
-INSTALL_MD="$(mktemp -t codex-install.XXXXXX.md)"
+## 执行原则
 
-echo "Fetch and follow instructions from ${INSTALL_URL}"
-curl -fsSL "${INSTALL_URL}" -o "${INSTALL_MD}"
+- 先诊断，再修复；不要未经确认就重复安装已经健康的工具。
+- 优先使用系统已有的包管理器和用户态安装路径。
+- 遇到多个可行方案时，选择对系统影响最小、最容易回滚的方案。
+- 对需要管理员权限、网络下载或会修改 shell 配置的动作，执行前简短说明影响。
+- 不要求固定轮次；根据实际结果迭代，直到通过验收或明确无法继续。
+- 不要强制执行远程安装文档。只有当诊断显示确实需要外部安装指引时，才读取可信来源并按实际情况采用。
 
-# 必须阅读并执行 INSTALL.md 中要求执行的命令；不可只拉取不执行
-cat "${INSTALL_MD}"
-```
+## 建议工作流
 
-执行要求（强制）：
-- 拉取后，按 `INSTALL.md` 的步骤顺序执行一遍。
-- 对其中出现的安装/初始化命令，必须实际执行，不可仅展示或解释。
-- 若某步失败，先修复前置条件后重试该步；完成后再继续 `doctor.sh`。
+1. 收集上下文：
+   - 操作系统与架构
+   - 当前 shell 与 PATH
+   - `python3`、`python`、`node`、`npm`、`pnpm`、`dx`、`agent-browser`、`rg` 的存在性与版本
+   - `CODEX_HOME` 与相关配置目录是否存在
+2. 对照目标状态判断缺口。
+3. 制定最小修复动作并执行。
+4. 每次修复后重新验证相关项。
+5. 所有项目完成后运行一次最终复检。
+6. 输出报告。
 
-2. 直接运行：
+## 验证要求
 
-```bash
-CODEX_HOME="${CODEX_HOME:-$HOME/.codex}"
-bash "$CODEX_HOME/skills/doctor/scripts/doctor.sh"
-```
+最终复检至少覆盖：
 
-3. 若需限制轮次（默认 3）：
+- 每个目标命令是否可被当前 shell 找到。
+- 每个目标命令的版本或基本健康输出。
+- `dx` 的初始化结果或当前初始化状态。
+- `agent-browser` 是否能找到并使用已安装的浏览器依赖。
+- 对未通过项给出失败原因、已尝试动作和下一步建议。
 
-```bash
-CODEX_HOME="${CODEX_HOME:-$HOME/.codex}"
-bash "$CODEX_HOME/skills/doctor/scripts/doctor.sh" --max-rounds 3
-```
+## 报告格式
 
-## 脚本职责
+最终报告使用中文，至少包含：
 
-- 并行检测：`python3`、`python` 别名、`pnpm`、`dx`、`agent-browser`、`rg`。
-- 自动修复：按平台选择安装器修复缺失项。
-- 强制执行：每轮都运行 `pnpm add -g @ranger1/dx@latest && dx initial`。
-- agent-browser：安装/升级并执行 Chromium 安装。
-- 结果输出：展示每项状态、版本、关键信息；全部通过则退出 0，否则最多三轮后退出 1。
-
-## 注意
-
-- 某些安装步骤可能需要管理员权限（例如 `sudo` 或 Homebrew 写权限）。
-- 若系统缺少包管理器，脚本会给出明确失败原因。
+- 环境摘要：系统、shell、关键 PATH 变更。
+- 检查结果表：检查项、状态、版本/证据、说明。
+- 已执行修复：实际执行过的安装、链接、初始化或配置变更。
+- 未完成项：若存在，说明阻塞原因和用户需要做什么。
+- 结论：通过 / 部分通过 / 未通过。

package/skills/doctor/agents/openai.yaml

@@ -1,4 +1,7 @@
 interface:
   display_name: "Doctor"
-  short_description: "并行检测并自动修复 Codex 开发环境关键依赖与特性开关"
-  default_prompt: "使用 $doctor 对当前系统执行并行体检、自动修复与三轮内收敛验证，并输出最终版本报告。"
+  short_description: "按目标状态自主诊断、修复并验证 Codex 开发环境"
+  default_prompt: "使用 $doctor 对当前系统进行 Codex 开发环境体检；根据实际缺口自主选择修复方式，完成后复检并输出报告。"
+
+policy:
+  allow_implicit_invocation: false

package/skills/git-pr-ship/SKILL.md

@@ -15,6 +15,7 @@ description: PR 交付流程（仅限显式调用）
 - 禁止在 `main/master` 直接提交。
 - 每修复一个问题立即 commit 一次，禁止攒到最后一次性提交。
 - AI 自主判断是否拒绝修复某个问题，拒绝必须写明理由。
+- 扫描范围内顺便发现的历史遗留问题（非本次 PR 引入），视同本次问题一并修复，不以"历史遗留"或"超出本 PR 范围"为由跳过。但不主动扩大扫描范围去寻找无关问题。
 - 上次跑完测试/Lint 后如果改过代码，必须重跑验证。
 - 使用 heredoc 写 commit message 和 gh 命令的 body（禁止 `\n`）。
 - 零参数设计：所有信息从仓库状态、当前分支、gh CLI 自动获取，不接受手动参数。
@@ -281,10 +282,12 @@ Step 3: 运行关联测试（根据以下改动范围判断需要跑哪些）
 
 审查者 A — 代码质量与架构（派独立 subagent）：
 - 关注：架构合理性、SOLID 原则、错误处理、性能、安全
+- 审查 PR diff 涉及的文件时，如果顺便发现同一文件中的历史遗留问题，也一并报告（不主动扩大到 diff 之外的文件）
 - 输入：PR diff
 
 审查者 B — 逻辑缺陷与规范（派独立 subagent）：
 - 关注：逻辑缺陷、边界条件、命名规范、类型安全、测试覆盖
+- 审查 PR diff 涉及的文件时，如果顺便发现同一文件中的历史遗留问题，也一并报告（不主动扩大到 diff 之外的文件）
 - 输入：PR diff
 
 > **禁止使用 `code-review` 技能或 `oh-my-claudecode:code-reviewer` agent 来执行审查。** 这些工具内置了自动发 PR 评论的行为，会导致 PR 上出现多条重复审核报告。必须派独立 subagent，并在 prompt 中明确要求"只返回审查结果文本，禁止调用 gh pr comment 或以任何方式发布 PR 评论"。
@@ -292,12 +295,14 @@ Step 3: 运行关联测试（根据以下改动范围判断需要跑哪些）
 Subagent A prompt：
 ```
 作为资深架构师审查此 PR diff，关注架构、性能、安全问题。
+审查 diff 涉及的文件时，如果顺便发现同一文件中已有的历史遗留问题（非本次 diff 引入），也一并报告——不要因为是历史代码就忽略。但不要主动扩大到 diff 未涉及的文件。
 【重要】只返回审查结果文本给调用者。禁止调用 gh pr comment 或以任何方式直接往 PR 发评论——评论由主 agent 统一发布。
 ```
 
 Subagent B prompt：
 ```
 作为质量工程师审查此 PR diff，关注逻辑缺陷、边界条件、规范遵从。
+审查 diff 涉及的文件时，如果顺便发现同一文件中已有的历史遗留问题（非本次 diff 引入），也一并报告——不要因为是历史代码就忽略。但不要主动扩大到 diff 未涉及的文件。
 【重要】只返回审查结果文本给调用者。禁止调用 gh pr comment 或以任何方式直接往 PR 发评论——评论由主 agent 统一发布。
 ```
 
@@ -379,8 +384,8 @@ MSG
 对每个问题：
 
 1. **判断是否修复**：
-   - 修复：执行修改
-   - 拒绝：记录理由（如：误报、设计意图、超出本 PR 范围）
+   - 修复：执行修改（包括扫描过程中顺便发现的历史遗留问题，不以"非本次引入"为由跳过）
+   - 拒绝：记录理由（如：误报、设计意图如此）。"历史遗留"或"超出本 PR 范围"不是合法的拒绝理由
 
 2. **修复后立即 commit**（一个问题一个 commit）：
 

package/skills/git-release/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: git-release
-description: 在 Git 仓库中执行标准化版本发布流程并自动生成高质量中文发行说明。用于以下场景：需要从 release 分支发布新版本；需要从分支名提取并校验语义化版本（含 alpha/beta/rc 预发布）；需要批量更新多个 package.json 的 version 字段并提交；需要基于最近 GitHub Release 汇总提交与 PR 信息、分类变更、生成发布摘要；需要创建 annotated tag、推送远端并创建 GitHub Release。
+description: 仅在用户显式调用 $git-release 或明确要求使用 git-release 技能时使用；不要通过关键词自动触发。
 ---
 
 # Git Release
@@ -15,6 +15,7 @@ description: 在 Git 仓库中执行标准化版本发布流程并自动生成
 - 严格执行前置校验，任何硬性条件不满足时立即终止。
 - 发行说明必须结构化、可读、可追溯。
 - 命令默认在仓库根目录执行。
+- 若能从当前 release 分支或自动建分支流程唯一推断出合法版本号，直接使用该版本继续发布，不要询问用户确认。
 
 ## 流程
 
@@ -33,7 +34,8 @@ description: 在 Git 仓库中执行标准化版本发布流程并自动生成
 - `release/v1.2.3` -> `v1.2.3` -> `1.2.3`
 - `release/v1.2.3-beta.2` -> `v1.2.3-beta.2` -> `1.2.3-beta.2`
 7. 检查目标 tag 是否已存在：`git tag -l "v<VERSION>"`。
-8. 向用户确认版本号；若用户修改版本号，重新校验格式与 tag 冲突。
+8. 输出推断出的版本号和推断来源，直接使用该版本号继续执行；不要向用户请求确认。
+9. 仅当无法从分支名或自动建分支流程唯一推断出合法版本号时，终止并要求用户显式指定目标版本。
 
 ### 二、更新版本号
 

package/skills/git-release/agents/openai.yaml

@@ -2,3 +2,6 @@ interface:
   display_name: "Git Release"
   short_description: "在 release 分支上自动完成版本发布与发行说明生成"
   default_prompt: "使用 $git-release 按规范执行一次发布并生成高质量发行说明。"
+
+policy:
+  allow_implicit_invocation: false