ai-project-manage-cli 6.0.32 → 6.0.34

package/dist/index.js

@@ -500,11 +500,14 @@ import { join as join5 } from "path";
 import { stringify as yamlStringify } from "yaml";
 
 // src/session-messages-xml.ts
+function asXmlText(value) {
+  return value ?? "";
+}
 function escapeXmlAttr(value) {
-  return value.replace(/&/g, "&amp;").replace(/"/g, "&quot;").replace(/</g, "&lt;").replace(/>/g, "&gt;");
+  return asXmlText(value).replace(/&/g, "&amp;").replace(/"/g, "&quot;").replace(/</g, "&lt;").replace(/>/g, "&gt;");
 }
 function wrapCdata(value) {
-  return `<![CDATA[${value.replace(/]]>/g, "]]]]><![CDATA[>")}]]>`;
+  return `<![CDATA[${asXmlText(value).replace(/]]>/g, "]]]]><![CDATA[>")}]]>`;
 }
 function formatSessionMessagesXml(sessionId, messages) {
   const lines = [
@@ -1011,7 +1014,8 @@ var VALID_STATUSES = [
   "CREATED",
   "TYPING",
   "SUCCESS",
-  "FAILED"
+  "FAILED",
+  "CANCELLED"
 ];
 async function runUpdateMessageStatus(options) {
   const messageId = options.id?.trim();
@@ -1049,25 +1053,17 @@ function parseAgentWsMessage(raw) {
 function serializeAgentWsMessage(msg) {
   return JSON.stringify(msg);
 }
-function validateAgentWsMessage(value, kind) {
-  if (typeof value !== "object" || value === null) {
-    return { ok: false, reason: "\u6D88\u606F\u4F53\u4E0D\u662F JSON \u5BF9\u8C61" };
+function validateHeartbeat(o) {
+  if (o.type !== "heartbeat") {
+    return { ok: false, reason: "\u671F\u671B heartbeat" };
   }
-  const o = value;
-  const type = o.type;
-  if (type !== "heartbeat" && type !== "message") {
-    return { ok: false, reason: `\u672A\u77E5 type: ${String(type)}` };
+  if (!nonEmptyString(o.userId)) {
+    return { ok: false, reason: "heartbeat \u7F3A\u5C11 userId" };
   }
-  if (kind === "heartbeat" || type === "heartbeat") {
-    if (type !== "heartbeat") {
-      return { ok: false, reason: "\u671F\u671B heartbeat" };
-    }
-    if (!nonEmptyString(o.userId)) {
-      return { ok: false, reason: "heartbeat \u7F3A\u5C11 userId" };
-    }
-    return { ok: true, data: { type: "heartbeat", userId: o.userId.trim() } };
-  }
-  if (type !== "message") {
+  return { ok: true, data: { type: "heartbeat", userId: o.userId.trim() } };
+}
+function validateMessagePush(o) {
+  if (o.type !== "message") {
     return { ok: false, reason: "\u671F\u671B message" };
   }
   if (!nonEmptyString(o.messageId)) {
@@ -1105,6 +1101,35 @@ function validateAgentWsMessage(value, kind) {
     }
   };
 }
+function validateCancel(o) {
+  if (o.type !== "cancel") {
+    return { ok: false, reason: "\u671F\u671B cancel" };
+  }
+  if (!nonEmptyString(o.messageId)) {
+    return { ok: false, reason: "cancel \u7F3A\u5C11 messageId" };
+  }
+  return {
+    ok: true,
+    data: { type: "cancel", messageId: o.messageId.trim() }
+  };
+}
+function validateAgentWsMessage(value, kind) {
+  if (typeof value !== "object" || value === null) {
+    return { ok: false, reason: "\u6D88\u606F\u4F53\u4E0D\u662F JSON \u5BF9\u8C61" };
+  }
+  const o = value;
+  const type = o.type;
+  if (type !== "heartbeat" && type !== "message" && type !== "cancel") {
+    return { ok: false, reason: `\u672A\u77E5 type: ${String(type)}` };
+  }
+  if (kind === "heartbeat" || type === "heartbeat") {
+    return validateHeartbeat(o);
+  }
+  if (type === "cancel") {
+    return validateCancel(o);
+  }
+  return validateMessagePush(o);
+}
 
 // src/commands/connect/abort-signal-debug.ts
 import {
@@ -1506,62 +1531,84 @@ async function setMessageError(cfg, messageId, error) {
   console.log(`[apm] \u5DF2\u8BBE\u7F6E\u6D88\u606F\u9519\u8BEF: ${messageId}`);
 }
 var SHUTDOWN_DRAIN_MS = 3e3;
-async function handleInboundMessage(cfg, raw, signal) {
+function isUserCancelled(ctx) {
+  return ctx.perMessageSignal.aborted && !ctx.shutdownSignal.aborted;
+}
+async function handleInboundMessage(cfg, msg, signal, ctx) {
+  if (isUserCancelled(ctx)) return;
   if (signal.aborted) return;
-  const parsed = parseAgentWsMessage(raw);
-  if (parsed === null) {
-    console.error("[apm] \u6536\u5230\u65E0\u6548 JSON");
-    return;
-  }
-  if (typeof parsed === "object" && parsed !== null && parsed.type === "heartbeat") {
-    return;
-  }
-  const validated = validateAgentWsMessage(parsed, "outbound");
-  if (!validated.ok) {
-    console.error(`[apm] \u6536\u5230\u65E0\u6548 message: ${validated.reason}`);
-    return;
-  }
-  if (validated.data.type !== "message") {
-    return;
-  }
-  const msg = validated.data;
   const messageId = msg.messageId;
+  const runStep = async (step, fn) => {
+    try {
+      return await fn();
+    } catch (err) {
+      const detail = err instanceof Error ? err.message : String(err);
+      throw new Error(`[${step}] ${detail}`);
+    }
+  };
   try {
     if (signal.aborted) return;
-    await runBranch(msg.sessionId, { cwd: msg.workdir });
+    await runStep(
+      "branch",
+      () => runBranch(msg.sessionId, { cwd: msg.workdir })
+    );
     if (signal.aborted) return;
-    await runPull(msg.sessionId, workspaceApmDir(msg.workdir));
+    await runStep(
+      "pull",
+      () => runPull(msg.sessionId, workspaceApmDir(msg.workdir))
+    );
     if (signal.aborted) return;
-    await commitWorkingTreeIfDirty(msg.workdir, "fix: apm pull");
+    await runStep(
+      "commit-pull",
+      () => commitWorkingTreeIfDirty(msg.workdir, "fix: apm pull")
+    );
     if (signal.aborted) return;
-    await updateMessageStatus(cfg, messageId, "TYPING");
-    await runCursorAgent(
-      cfg,
-      {
-        messageId: msg.messageId,
-        sessionId: msg.sessionId,
-        prompt: msg.content,
-        model: msg.model,
-        apiKey: msg.apiKey,
-        workdir: msg.workdir
-      },
-      { signal }
+    await runStep(
+      "status-typing",
+      () => updateMessageStatus(cfg, messageId, "TYPING")
     );
-    await syncSessionDocuments(
-      cfg,
-      msg.sessionId,
-      workspaceApmDir(msg.workdir)
+    await runStep(
+      "cursor-agent",
+      () => runCursorAgent(
+        cfg,
+        {
+          messageId: msg.messageId,
+          sessionId: msg.sessionId,
+          prompt: msg.content,
+          model: msg.model,
+          apiKey: msg.apiKey,
+          workdir: msg.workdir
+        },
+        { signal }
+      )
     );
-    await commitWorkingTreeIfDirty(
-      msg.workdir,
-      "chore(apm): sync session documents"
+    await runStep(
+      "sync-documents",
+      () => syncSessionDocuments(cfg, msg.sessionId, workspaceApmDir(msg.workdir))
+    );
+    await runStep(
+      "commit-documents",
+      () => commitWorkingTreeIfDirty(
+        msg.workdir,
+        "chore(apm): sync session documents"
+      )
+    );
+    await runStep(
+      "status-success",
+      () => updateMessageStatus(cfg, messageId, "SUCCESS")
     );
-    await updateMessageStatus(cfg, messageId, "SUCCESS");
   } catch (err) {
+    if (isUserCancelled(ctx)) {
+      console.log(`[apm] \u6D88\u606F\u5DF2\u7EC8\u6B62 messageId=${messageId}`);
+      return;
+    }
     console.error(
       "[apm] \u5904\u7406\u6D88\u606F\u5931\u8D25:",
       err instanceof Error ? err.message : err
     );
+    if (err instanceof Error && err.stack) {
+      console.error(err.stack);
+    }
     try {
       await setMessageError(
         cfg,
@@ -1610,6 +1657,8 @@ async function runConnect(options) {
     let shuttingDown = false;
     const shutdownAbort = new AbortController();
     let inboundQueue = Promise.resolve();
+    const activeRuns = /* @__PURE__ */ new Map();
+    const pendingCancels = /* @__PURE__ */ new Set();
     const shutdown = async (code = 0) => {
       if (shuttingDown) return;
       shuttingDown = true;
@@ -1640,9 +1689,48 @@ async function runConnect(options) {
     ws.on("message", (data) => {
       if (shuttingDown) return;
       const text = Buffer.isBuffer(data) ? data.toString("utf8") : String(data);
-      inboundQueue = inboundQueue.then(
-        () => handleInboundMessage(cfg, text, shutdownAbort.signal)
-      );
+      const parsed = parseAgentWsMessage(text);
+      if (parsed === null) {
+        console.error("[apm] \u6536\u5230\u65E0\u6548 JSON");
+        return;
+      }
+      if (typeof parsed === "object" && parsed !== null && parsed.type === "heartbeat") {
+        return;
+      }
+      const validated = validateAgentWsMessage(parsed, "outbound");
+      if (!validated.ok) {
+        console.error(`[apm] \u6536\u5230\u65E0\u6548 WS \u5305: ${validated.reason}`);
+        return;
+      }
+      if (validated.data.type === "cancel") {
+        const { messageId } = validated.data;
+        pendingCancels.add(messageId);
+        activeRuns.get(messageId)?.abort();
+        return;
+      }
+      if (validated.data.type !== "message") {
+        return;
+      }
+      const msg = validated.data;
+      const perMessageController = new AbortController();
+      activeRuns.set(msg.messageId, perMessageController);
+      if (pendingCancels.has(msg.messageId)) {
+        activeRuns.delete(msg.messageId);
+        pendingCancels.delete(msg.messageId);
+        return;
+      }
+      const signal = AbortSignal.any([
+        shutdownAbort.signal,
+        perMessageController.signal
+      ]);
+      const ctx = {
+        shutdownSignal: shutdownAbort.signal,
+        perMessageSignal: perMessageController.signal
+      };
+      inboundQueue = inboundQueue.then(() => handleInboundMessage(cfg, msg, signal, ctx)).finally(() => {
+        activeRuns.delete(msg.messageId);
+        pendingCancels.delete(msg.messageId);
+      });
     });
     ws.on("close", (code, reason) => {
       console.log(

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ai-project-manage-cli",
-  "version": "6.0.32",
+  "version": "6.0.34",
   "description": "命令行工具：后续用于调用平台后端 API 完成运维与自动化操作",
   "type": "module",
   "private": false,

package/template/skills/apm-write-prd/SKILL.md

@@ -11,6 +11,8 @@
 内容要求如下：
 **不管是第几版需求，都要当成第一版来看，禁止有历史版本或者修订版本或者第几版更新的字样**
 
+**可读性**：信息完整保留，但避免整段只靠长句堆砌。对流程分支、状态条件、按钮对照、范围边界等多步骤/多条件内容，按模板中的「图示原则」**适当补充 Mermaid 图或简表**（每个需求点 0 ～ 1 张）；图示用于快速扫读，**可验收细节仍以 bullet 为准**，不得因配图而删减文字要求。
+
 ### 步骤 3: 同步 PRD 到远程
 
 执行命令：`apm sync-document <会话 ID> --file=PRD.md`

package/template/skills/apm-write-prd/template.md

@@ -1,17 +1,43 @@
-
 ## 需求模板
+
+### 图示原则
+
+信息量保持完整，**文字与图示互补**：bullet 写清可验收细节，图示帮助读者快速建立整体印象。优先用 **Mermaid**（Markdown 内嵌，无需额外图片文件）。
+
+| 场景                                        | 推荐图示                         | 作用                       |
+| ------------------------------------------- | -------------------------------- | -------------------------- |
+| 多步骤操作流程（审批、暂存/提交、办理路径） | `flowchart`                      | 一眼看清先后与分支         |
+| 字段展示/必填随状态变化                     | `flowchart` 或 `stateDiagram-v2` | 替代「当 A 且 B 时…」长句  |
+| 弹窗/页面内区块与按钮布局                   | `block-beta` 或 ASCII 框图       | 标出按钮位置与对照关系     |
+| 多需求点关系、范围边界                      | `mindmap` 或简表                 | 开篇总览，再展开 bullet    |
+| 角色与系统交互时序                          | `sequenceDiagram`                | 仅步骤多、文字难扫读时使用 |
+
+**图示写法**：
+
+- 每个需求点 **0 ～ 1 张** 图即可；一张图只表达一个核心概念，节点用业务语言（如「一级审批」「暂存」）。
+- 图 **不替代** bullet：图后仍保留 2 ～ 5 条可验收 bullet，图中未写清的细节（文案、边界、校验时机）必须在文字中写明。
+- 图为辅助：若单条 bullet 已足够清楚，**不必强行配图**。
+
 ### 正文骨架
+
 ```markdown
 ## 背景
+
 （1 ～ 3 句：业务背景、要解决的问题、预期效果，如果没有可空着，不要强行编）
 
 ## 范围
+
 - **包含**：…
 - **不包含**：…
 
+（可选：范围较大时，用 mindmap 或简表做一页总览）
+
 ## 需求说明
 
 ### 需求点 1：[简短名称]
+
+（可选：流程/状态/布局类需求，在此放一张 Mermaid 图）
+
 - …
 - …
 
@@ -42,14 +68,67 @@
 
 ### 需求点 3：「科室医德考评小组人员名单」字段
 
+（本需求点适合在 bullet 前加「字段展示与校验」流程图，见下方「图示示例」）
+
 - 原字段展示名 **「科室医德考评人员名单」** 统一改为 **「科室医德考评小组人员名单」**；凡界面、提示、校验文案等涉及该名称处**一并修改**。
 - 当流程处于 **一级审批** 且该字段**展示**时，须校验为**必填**（在展示场景下触发，而非所有保存入口一律校验）。
 ```
 
+### 图示示例（按需选用）
+
+**字段展示与校验**（对应上文需求点 3）：
+
+```mermaid
+flowchart LR
+  A[进入表单] --> B{流程处于一级审批?}
+  B -->|是| C[展示该字段]
+  B -->|否| D[按原规则展示/隐藏]
+  C --> E{用户操作}
+  E -->|暂存或提交| F[校验必填]
+  E -->|仅浏览未保存| G[不触发必填校验]
+```
+
+**流程分支**（暂存 vs 提交、审批节点）：
+
+```mermaid
+flowchart TD
+  A[用户打开新增/编辑弹窗] --> B[填写表单]
+  B --> C{点击底部按钮}
+  C -->|暂存| D[沿用原「确定」逻辑：校验 + 保存，不推进流程]
+  C -->|提交| E[沿用原「办理」逻辑：校验 + 保存并推进流程]
+```
+
+**弹窗底部按钮布局**（对照既有按钮）：
+
+```text
+┌─────────────────────────────────────┐
+│           新增 / 编辑弹窗            │
+│  …表单字段…                          │
+├─────────────────────────────────────┤
+│              [ 取消 ]  [ 暂存 ]       │  ← 暂存：原「确定」逻辑，仅改文案
+└─────────────────────────────────────┘
+```
+
+**范围总览**（需求点较多时，放在「范围」之后）：
+
+```mermaid
+mindmap
+  root((本需求))
+    包含
+      弹窗底部操作
+      字段更名与校验
+    不包含
+      历史数据迁移
+      导出能力
+```
+
 ### 自检列表
+
 - [ ] 含 **背景与目标、范围、需求说明** 三章，叙述完整
 - [ ] 2 分钟内可通读；每个需求点 2 ～ 5 条 bullet，一层列表
+- [ ] 流程/状态/多分支类需求点有 **图示或简表** 辅助，且 bullet 仍覆盖图中未写清的细节
+- [ ] 图示节点使用业务语言，与正文术语一致；无仅看图无法验收的遗漏
 - [ ] 场景、按钮行为、校验时机表述清楚，前后一致
 - [ ] 「范围」与需求点中的 **「不考虑」** 口径一致
 - [ ] 全文使用业务语言，术语统一
-- [ ] 仅材料明确涉及时才有「非功能」「待确认」
+- [ ] 仅材料明确涉及时才有「非功能」「待确认」