fixed

zhonghui · zhonghui · commit 4e8e2c7f5d78 · 2026-04-15T11:57:05.000+08:00
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -58,19 +58,28 @@ test: 测试
 ## 项目结构
 
 ```
-├── INSTRUCTIONS.md     ← 核心交付物（用户放入项目的文件）
+├── src/                ← TypeScript 源码
+│   ├── cli.ts          ← CLI 入口
+│   ├── mcp-server.ts   ← MCP Server 入口
+│   └── skills/         ← 技能实现
+│       ├── init.ts     ← 工作区初始化
+│       ├── plan.ts     ← 创建变更计划
+│       ├── checkpoint.ts ← 阶段审计与门禁
+│       ├── status.ts   ← 工作区状态查询
+│       ├── release.ts  ← 归档与记忆整合
+│       ├── agents.ts   ← 领域专家列表
+│       └── types.ts    ← 公共类型定义
+├── templates/          ← 初始化时复制到项目的模板
+│   ├── INSTRUCTIONS.md ← 核心协议文档
+│   └── doc-formats/    ← 文档格式模板 (proposal/design/impl-log 等)
 ├── agents/             ← 领域专家定义
 │   ├── README.md       ← 专家目录速查表
 │   └── *.md            ← 各专家的 prompt
 ├── skills/             ← Skills（可通过 skills.sh 安装）
 │   └── crew-*/SKILL.md ← 各 Skill 定义
-└── docs/               ← 用户文档
-    ├── quick-start.md  ← 快速开始
-    ├── guide.md        ← 使用指南
-    ├── scenarios.md    ← 使用场景 + 常见问题
-    ├── concepts.md     ← 核心概念
-    ├── specialists.md  ← 领域专家
-    └── examples/       ← 最佳实践
+├── docs/               ← VitePress 用户文档站点
+├── package.json        ← 包配置与脚本
+└── tsconfig.json       ← TypeScript 编译配置
 ```
 
 ## 许可证
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@lordmos/dev-crew",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "description": "AI-powered dev team orchestration framework — Skills + CLI + MCP Server",
   "keywords": [
     "ai",
@@ -35,7 +35,8 @@
   "files": [
     "dist/",
     "templates/",
-    "agents/"
+    "agents/",
+    "skills/"
   ],
   "scripts": {
     "build": "tsc",
diff --git a/skills/crew-agents/SKILL.md b/skills/crew-agents/SKILL.md
@@ -4,7 +4,7 @@ user-invocable: true
 description: List available domain specialists that can be activated for the DevCrew team — covering game dev, security, DevOps, AI/ML, and more.
 metadata:
   author: lordmos
-  version: 0.5.0
+  version: 0.5.1
 ---
 
 # crew agents — 列出可用领域专家
diff --git a/skills/crew-checkpoint/SKILL.md b/skills/crew-checkpoint/SKILL.md
@@ -4,7 +4,7 @@ user-invocable: true
 description: Run a phase audit and consistency check on the current change. Validates all checklist items before advancing to the next PDEVI phase.
 metadata:
   author: lordmos
-  version: 0.5.0
+  version: 0.5.1
 ---
 
 # crew checkpoint — 阶段审计与一致性检查
diff --git a/skills/crew-init/SKILL.md b/skills/crew-init/SKILL.md
@@ -4,7 +4,7 @@ user-invocable: true
 description: Initialize a DevCrew workspace and activate the full AI team orchestration protocol — PDEVI workflow, persistent memory, checkpoint-based quality control.
 metadata:
   author: lordmos
-  version: 0.5.0
+  version: 0.5.1
 ---
 
 # crew.init — 初始化 DevCrew 工作区并激活协议
@@ -187,6 +187,17 @@ DevCrew 提供以下 Skill，AI Agent 通过 MCP 工具调用执行，或人类
 - **Standard** Iterate ≤5 轮 → 超过创建 `[OPEN]` blocker，**暂停迭代**等待用户介入
 - **Prototype→Standard**: 用户说"正式做吧" → 审查 design.md 补测试策略 → 进入 Verify
 
+### 变更计划修改
+
+用户可能在 Plan 确认后发现计划有误（"计划不对"、"需求变了"、"改一下方案"）。PjM 处理流程：
+
+1. **重置状态**：resume.md `phase` 回退至 Plan，`plan_confirmed` 设 false；proposal.md `plan_confirmed` 设 false
+2. **标记下游影响**：若已有 design.md，在 resume.md 记录"计划修改后 Design 需重新审视"
+3. **分派 PdM**：更新 proposal.md
+4. **重新确认**：展示修改摘要，等待用户确认后继续 PDEVI 流程
+
+> **关键**：PjM **必须先更新 resume.md 状态**再分派工作，确保所有 Agent 读到一致的阶段信息。
+
 ## 通信规则
 
 - proposal.md 是需求的 **single source of truth**，**禁止跨文件重复长篇内容**——用 `参考 {文件} § {章节}` 引用
diff --git a/skills/crew-plan/SKILL.md b/skills/crew-plan/SKILL.md
@@ -4,7 +4,7 @@ user-invocable: true
 description: Create a new change proposal and enter the Plan phase of the PDEVI workflow. Supports Standard, Express, and Prototype modes.
 metadata:
   author: lordmos
-  version: 0.5.0
+  version: 0.5.1
 ---
 
 # crew plan — 创建变更计划
diff --git a/skills/crew-release/SKILL.md b/skills/crew-release/SKILL.md
@@ -4,7 +4,7 @@ user-invocable: true
 description: Archive completed changes and consolidate agent memory. Moves change artifacts to archive and updates each agent's long-term knowledge base.
 metadata:
   author: lordmos
-  version: 0.5.0
+  version: 0.5.1
 ---
 
 # crew release — 归档变更与记忆整合
diff --git a/skills/crew-status/SKILL.md b/skills/crew-status/SKILL.md
@@ -4,7 +4,7 @@ user-invocable: true
 description: View current DevCrew workspace status — active changes, current phase, progress, and open blockers.
 metadata:
   author: lordmos
-  version: 0.5.0
+  version: 0.5.1
 ---
 
 # crew status — 查看工作区状态
diff --git a/src/cli.ts b/src/cli.ts
@@ -13,7 +13,7 @@ const program = new Command();
 program
   .name("crew")
   .description("DevCrew — AI Agent 软件开发团队编排工具")
-  .version("0.5.0");
+  .version("0.5.1");
 
 program
   .command("init")
@@ -38,7 +38,9 @@ program
       description: opts.description,
       mode: opts.mode as "standard" | "express" | "prototype" | undefined,
     });
-    console.log(result.ok ? `\n✅ ${result.summary}` : `\n❌ ${result.summary}`);
+    console.log(
+      result.ok ? `\n✅ ${result.summary}` : `\n❌ ${result.summary}`,
+    );
     for (const line of result.details) console.log(`  ${line}`);
     console.log();
     if (!result.ok) process.exitCode = 1;
@@ -61,7 +63,9 @@ program
   .argument("[name]", "要归档的变更名称（不指定则归档全部）")
   .action((name?: string) => {
     const result = release({ cwd: process.cwd(), name });
-    console.log(result.ok ? `\n✅ ${result.summary}` : `\n❌ ${result.summary}`);
+    console.log(
+      result.ok ? `\n✅ ${result.summary}` : `\n❌ ${result.summary}`,
+    );
     for (const line of result.details) console.log(`  ${line}`);
     console.log();
     if (!result.ok) process.exitCode = 1;
@@ -78,7 +82,9 @@ program
   .argument("[change]", "变更名称（不指定则自动选择第一个活跃变更）")
   .action((change?: string) => {
     const result = checkpoint({ cwd: process.cwd(), change });
-    console.log(result.ok ? `\n✅ ${result.summary}` : `\n⚠️  ${result.summary}`);
+    console.log(
+      result.ok ? `\n✅ ${result.summary}` : `\n⚠️  ${result.summary}`,
+    );
     for (const line of result.details) console.log(line);
     console.log();
     if (!result.ok) process.exitCode = 1;
diff --git a/src/mcp-server.ts b/src/mcp-server.ts
@@ -24,7 +24,7 @@ function formatResult(result: SkillResult) {
 
 const server = new McpServer({
   name: "dev-crew",
-  version: "0.5.0",
+  version: "0.5.1",
 });
 
 // --- Skill: init ---
@@ -33,7 +33,10 @@ server.tool(
   "初始化 DevCrew 工作区（创建 dev-crew/ 目录、INSTRUCTIONS.md、配置文件、Agent 记忆文件，可指定 AI 平台自动配置）",
   {
     cwd: z.string().describe("项目根目录的绝对路径"),
-    name: z.string().optional().describe("项目名称（默认从 package.json 或目录名推断）"),
+    name: z
+      .string()
+      .optional()
+      .describe("项目名称（默认从 package.json 或目录名推断）"),
     platform: z
       .array(z.enum(["copilot", "cursor", "claude"]))
       .optional()
@@ -79,7 +82,10 @@ server.tool(
   "归档已完成的变更，触发记忆整合",
   {
     cwd: z.string().describe("项目根目录的绝对路径"),
-    name: z.string().optional().describe("要归档的变更名称（不指定则归档全部）"),
+    name: z
+      .string()
+      .optional()
+      .describe("要归档的变更名称（不指定则归档全部）"),
   },
   async (input) => formatResult(skills.release(input)),
 );
@@ -98,7 +104,10 @@ server.tool(
   "阶段审计：运行审计清单 + 一致性检查 + 记忆同步提醒，确认当前阶段可推进",
   {
     cwd: z.string().describe("项目根目录的绝对路径"),
-    change: z.string().optional().describe("变更名称（不指定则自动选择第一个活跃变更）"),
+    change: z
+      .string()
+      .optional()
+      .describe("变更名称（不指定则自动选择第一个活跃变更）"),
   },
   async (input) => formatResult(skills.checkpoint(input)),
 );
diff --git a/src/skills/checkpoint.ts b/src/skills/checkpoint.ts
@@ -99,7 +99,7 @@ export function checkpoint(input: CheckpointInput): SkillResult {
   // Run audit checklist for current phase
   const checklist = AUDIT_CHECKLIST[currentPhase] ?? [];
   details.push(`=== ${currentPhase} 阶段审计清单 ===`);
-  const auditResults = runAudit(changeDir, currentPhase, proposal);
+  const auditResults = runAudit(changeDir, crewDir, currentPhase, proposal);
   for (let i = 0; i < checklist.length; i++) {
     const passed = auditResults[i];
     details.push(`  ${passed ? "[PASS]" : "[TODO]"} ${checklist[i]}`);
@@ -150,9 +150,11 @@ function detectPhase(changeDir: string, mode: string): Phase {
   const hasImplLog = existsSync(join(changeDir, "impl-log.md"));
   const hasDesign = existsSync(join(changeDir, "design.md"));
 
-  if (hasTestReport) return "Verify";
+  const flow = PHASE_FLOW[mode] ?? PHASE_FLOW.standard;
+
+  if (hasTestReport && flow.includes("Verify")) return "Verify";
   if (hasImplLog) return "Execute";
-  if (hasDesign) return "Design";
+  if (hasDesign && flow.includes("Design")) return "Design";
   return "Plan";
 }
 
@@ -161,7 +163,7 @@ function extractField(content: string, field: string): string | null {
   return match?.[1]?.trim() ?? null;
 }
 
-function runAudit(changeDir: string, phase: Phase, proposal: string): boolean[] {
+function runAudit(changeDir: string, crewDir: string, phase: Phase, proposal: string): boolean[] {
   switch (phase) {
     case "Plan": {
       const hasObjective = /## 目标/.test(proposal);
@@ -199,7 +201,13 @@ function runAudit(changeDir: string, phase: Phase, proposal: string): boolean[]
       const reviewPath = join(changeDir, "review-report.md");
       const hasTest = existsSync(testPath);
       const hasReview = existsSync(reviewPath);
-      const verifyConfirmed = /verify_confirmed:\s*true/.test(proposal);
+      // verify_confirmed lives in resume.md, not proposal.md
+      const resumePath = join(crewDir, "resume.md");
+      let verifyConfirmed = false;
+      if (existsSync(resumePath)) {
+        const resume = readFileSync(resumePath, "utf-8");
+        verifyConfirmed = /verify_confirmed:\s*true/.test(resume);
+      }
       return [hasTest, hasReview, true, verifyConfirmed];
     }
     default:
@@ -231,7 +239,7 @@ function runConsistencyAudit(changeDir: string, crewDir: string): string[] {
   }
 
   // Check for oversized files (file pointer discipline)
-  const filesToCheck = ["proposal.md", "design.md", "impl-log.md"];
+  const filesToCheck = ["proposal.md", "design.md", "impl-log.md", "test-report.md", "review-report.md"];
   for (const file of filesToCheck) {
     const filePath = join(changeDir, file);
     if (existsSync(filePath)) {
diff --git a/src/skills/init.ts b/src/skills/init.ts
@@ -1,4 +1,4 @@
-import { resolve, join, dirname } from "node:path";
+import { resolve, join, dirname, basename } from "node:path";
 import {
   existsSync,
   mkdirSync,
@@ -223,7 +223,7 @@ function inferProjectName(dir: string): string {
       /* ignore */
     }
   }
-  return dir.split("/").pop() ?? "my-project";
+  return basename(dir) || "my-project";
 }
 
 function generateYaml(projectName: string): string {
diff --git a/src/skills/plan.ts b/src/skills/plan.ts
@@ -1,5 +1,5 @@
 import { join } from "node:path";
-import { existsSync, mkdirSync, writeFileSync } from "node:fs";
+import { existsSync, mkdirSync, writeFileSync, readFileSync } from "node:fs";
 import { success, failure, type SkillResult } from "./types.js";
 
 export interface PlanInput {
@@ -20,6 +20,14 @@ export function plan(input: PlanInput): SkillResult {
     mkdirSync(changesDir, { recursive: true });
   }
 
+  // Validate name: must be kebab-case, no path traversal
+  if (!/^[a-z0-9]+(?:-[a-z0-9]+)*$/.test(input.name)) {
+    return failure(`变更名称 "${input.name}" 无效`, [
+      "名称仅允许小写字母、数字和连字符（kebab-case）",
+      "示例: add-login, fix-nav-bug, redesign-dashboard",
+    ]);
+  }
+
   const changeDir = join(changesDir, input.name);
   if (existsSync(changeDir)) {
     return failure(`变更 "${input.name}" 已存在`, [
@@ -32,17 +40,20 @@ export function plan(input: PlanInput): SkillResult {
   mkdirSync(changeDir, { recursive: true });
 
   const proposalContent = `---
-name: ${input.name}
 mode: ${mode}
-stage: Plan
+plan_confirmed: false
 ---
 
 # ${input.name}
 
-## 需求概述
+## 目标
 
 ${input.description ?? "（待填写）"}
 
+## 需求
+
+（待细化）
+
 ## 验收标准
 
 - [ ] （待定义）
@@ -59,18 +70,38 @@ ${input.description ?? "（待填写）"}
   if (mode !== "express") {
     writeFileSync(
       join(changeDir, "design.md"),
-      `---
-change: ${input.name}
----
-
-# 设计文档
+      `# 设计文档
 
 （Architect 在 Design 阶段填写）
 `,
     );
     details.push(`创建 dev-crew/changes/${input.name}/design.md`);
   }
 
+  // Sync resume.md with new change
+  const resumePath = join(crewDir, "resume.md");
+  if (existsSync(resumePath)) {
+    let resume = readFileSync(resumePath, "utf-8");
+    const entry = [
+      `  - name: ${input.name}`,
+      `    mode: ${mode}`,
+      `    phase: Plan`,
+      `    plan_confirmed: false`,
+      `    iterate_count: 0`,
+    ].join("\n");
+
+    if (/active_changes:\s*\[\]/.test(resume)) {
+      resume = resume.replace(/active_changes:\s*\[\]/, `active_changes:\n${entry}`);
+    } else if (/active_changes:/.test(resume)) {
+      const fmEnd = resume.indexOf("\n---", 4);
+      if (fmEnd !== -1) {
+        resume = resume.slice(0, fmEnd) + "\n" + entry + resume.slice(fmEnd);
+      }
+    }
+    writeFileSync(resumePath, resume);
+    details.push("更新 dev-crew/resume.md");
+  }
+
   return success(`变更 "${input.name}" 已创建 (${mode})`, details);
 }
 
diff --git a/src/skills/release.ts b/src/skills/release.ts
diff --git a/src/skills/status.ts b/src/skills/status.ts
diff --git a/templates/INSTRUCTIONS.md b/templates/INSTRUCTIONS.md
