@lynn123411/dsh-ask-user-grilling 0.1.1

package/README.md

@@ -0,0 +1,24 @@
+# @lynn123411/dsh-ask-user-grilling
+
+DSH 侧的 grilling 适配层（输送机制）：把 Matt Pocock 的 grilling 流程在 DSH 里的提问环节做成工具级硬约束。**不修改、不包装这组技能的任何文件**——技能按原设计运行，本插件只负责「在 DSH 里怎么问」。
+
+## 特性
+
+- **ask_user_grilling**：grilling 轮次专用提问工具。
+  - **子代理闸门**：后台有子代理运行时会拒绝提问，返回「阻塞 + 运行中名单」——agent 应**结束当前回合**，子代理结算通知自动唤醒后再调用，不在回合内反复重试；
+  - **强制多选**：所有问题一律多选（schema 不提供关闭开关）；
+  - **补充机制**：自动为每个问题追加「补充」选项，并自动在每轮末尾追加「本轮还有什么要补充或调整的吗？」；
+  - **题干引导**：要求题干只含问题本身、不重复选项文本（仅模型侧引导，不做硬校验——避免误伤自然提及选项名称的题干）。
+- **enter_plan_mode**：为当前 agent 激活 DSH 计划模式。grilling 最后一轮答完、用户确认共识后调用，让 agent 写方案供审阅而不是直接开始执行；可通过 `exit_plan_mode` 或 `/plan off` 退出。
+
+## 安装
+
+```bash
+dsh plugin --profile web add @lynn123411/dsh-ask-user-grilling
+```
+
+已发布至 npm，配合 `matt-standard` 预设使用（preset 的工具行直接消费本包）。
+
+**本地开发**：将本目录复制到 `~/.dsh/profiles/web/node_modules/@lynn123411/dsh-ask-user-grilling/` 即可，preset 的工具行会直接从 node_modules 解析。注意：**不要**把它加进 profile `package.json` 的 `dsh.profile.bundles`（bundle 层必须在包内声明 `dsh.bundle`，否则启动报错）。
+
+> 注意：profile 是 pnpm hoisted 布局：`pnpm install` / `dsh plugin add|remove` 重装后，手工同步的本地副本会被清掉，需重新同步。

package/lib/index.js

@@ -0,0 +1,247 @@
+import { defineTool } from "@deepseek-ai/dsh-tools";
+import "@deepseek-ai/dsh-user-questions";
+
+/**
+ * @lynn123411/dsh-ask-user-grilling — DSH delivery adaptations for grilling
+ * rounds (mattpocock/skills). No skill file is modified; this package only
+ * hardens how rounds are asked inside DSH.
+ *
+ * ask_user_grilling:
+ *   - gate: refuses while background subagents are running (R1)
+ *   - forces multi-select on every question (R2)
+ *   - appends a per-question "Supplement" option and a round-end question (R3)
+ *   - stem/option separation is guidance only — never rejects stems (R4
+ *     relaxed: substring checks false-positive on legitimate stems)
+ *
+ * enter_plan_mode:
+ *   - activates DSH plan mode for the current agent (planMode.set)
+ */
+const name = "tool-ask-user-grilling";
+const inject = ["tools", "userQuestions"];
+
+const SUPPLEMENT_OPTION = {
+  label: "Supplement",
+  description: "Select this and write your supplement for this question in your reply.",
+};
+
+const ROUND_END_QUESTION = {
+  id: "__grill_round_supplement__",
+  question: "Anything else to add or adjust this round?",
+  header: "Round supplement",
+  options: [
+    { label: "Nothing to add" },
+    { label: "I have something to add (write it in my reply)", description: "Select this and write any additions or adjustments for this round in your reply." },
+  ],
+  multiSelect: true,
+};
+
+function displayName(entry) {
+  return entry.label ?? entry.id;
+}
+
+function apply(ctx) {
+  ctx.tools.register(defineTool({
+    name: "ask_user_grilling",
+    description: "Ask the user a ROUND of grilling questions (decision-tree interview). Use for grilling sessions (grill-me / grill-with-docs / triage / wayfinder / architecture grilling): send ALL frontier questions of the current round in ONE call. The tool forces multi-select, appends a per-question 'Supplement' option, and appends a round-end supplement question automatically. If background subagents are running, this tool returns blocked — do NOT retry within the same turn: end your turn and wait; the settlement notice will wake you automatically, then call again. Keep the question stem to the question itself, without repeating option labels (style guidance only — stems are never rejected). Example of one round with two questions: questions: [{ id: 'q1', question: 'Which issue tracker?', options: [{ label: 'GitHub' }, { label: 'Local markdown' }] }, { id: 'q2', question: 'Any deadline?', options: [{ label: 'This week' }, { label: 'Next month' }] }].",
+    parameters: {
+      questions: {
+        type: "array",
+        required: true,
+        description: "All frontier questions of the current round, each with a stable id, the question text (stem) and options.",
+        items: {
+          type: "object",
+          additionalProperties: true,
+          properties: {
+            id: {
+              type: "string",
+              required: true,
+              description: "Stable id for this question; echoed in the answer. Must NOT start with __grill_ (reserved prefix).",
+            },
+            question: {
+              type: "string",
+              required: true,
+              description: "The question stem only — never include option text here.",
+            },
+            header: {
+              type: "string",
+              description: "Optional short heading for the question, such as \"Round 3\".",
+            },
+            options: {
+              type: "array",
+              description: "Choices to show the user. If you recommend one, put it first and append \"(Recommended)\" to that label.",
+              items: {
+                type: "object",
+                additionalProperties: true,
+                properties: {
+                  label: {
+                    type: "string",
+                    required: true,
+                    description: "Short user-facing option label.",
+                  },
+                  description: {
+                    type: "string",
+                    description: "One sentence explaining the tradeoff or impact.",
+                  },
+                },
+              },
+            },
+          },
+        },
+      },
+    },
+    output: {
+      schema: {
+        type: "object",
+        additionalProperties: false,
+        properties: {
+          blocked: {
+            type: "boolean",
+            description: "True when background subagents are running and the round was not asked.",
+          },
+          waiting: {
+            type: "array",
+            items: { type: "string" },
+            description: "Running subagent display names when blocked.",
+          },
+          rejected: {
+            type: "boolean",
+            description: "True when input validation failed and the round was not asked.",
+          },
+          violations: {
+            type: "array",
+            items: { type: "string" },
+            description: "Validation violations when rejected (reserved id prefix only).",
+          },
+          error: { type: "string" },
+          answers: {
+            type: "array",
+            items: {
+              type: "object",
+              additionalProperties: false,
+              properties: {
+                id: { type: "string", required: true },
+                selected: { type: "array", required: true, items: { type: "string" } },
+                custom: { type: "string" },
+              },
+            },
+          },
+        },
+      },
+      render: (_args, value) => [{ type: "text", text: JSON.stringify(value) }],
+    },
+    async execute(args, exec) {
+      // 1. gate: refuse while background subagents are running (R1)
+      const subagents = ctx.get("subagents");
+      const agent = exec.agent;
+      const waiting = [];
+      if (subagents !== undefined && agent !== undefined) {
+        try {
+          const entries = await subagents.listDescendants(agent.id, exec.signal);
+          for (const entry of entries) {
+            if (entry.kind === "child" && entry.activity === "running") waiting.push(displayName(entry));
+          }
+        } catch (error) {
+          // Fail-closed: R1 says never enter the decision tree unless we know
+          // no subagents are running. Only an aborted call (user cancelled)
+          // propagates; any other listing failure blocks the round.
+          if (exec.signal.aborted) throw error;
+          console.error("ask_user_grilling: subagent listing failed: %o", error);
+          return {
+            blocked: true,
+            waiting: [],
+            error: "Cannot confirm background subagent status (subagents query failed). Please call this tool again later.",
+          };
+        }
+      }
+      if (waiting.length > 0) {
+        return {
+          blocked: true,
+          waiting,
+          error: `Still ${waiting.length} subagent(s) running (${waiting.join(", ")}). End your turn and wait — the settlement notice will wake you automatically, then call again.`,
+        };
+      }
+
+      // 2. input validation: reserved id prefix guard only (the round-end
+      //    question owns __grill_). Stem/option separation (R4) is guidance,
+      //    NOT enforced: substring matching rejected legitimate stems (e.g. a
+      //    stem that naturally mentions an option name), so no stem check may
+      //    refuse a round — a bad stem is preferable to a false rejection.
+      const violations = [];
+      for (const question of args.questions) {
+        if (typeof question.id === "string" && question.id.startsWith("__grill_")) {
+          violations.push(`Question id "${question.id}" uses the reserved prefix __grill_`);
+        }
+      }
+      if (violations.length > 0) {
+        return {
+          rejected: true,
+          violations,
+          error: "Question ids must not use the reserved prefix __grill_ (reserved for the round-end supplement question). Fix the ids and call this tool again.",
+        };
+      }
+
+      // 3. transform: force multi-select (R2), append supplement option (R3)
+      const questions = args.questions.map((question) => ({
+        id: question.id,
+        question: question.question,
+        ...(question.header !== undefined ? { header: question.header } : {}),
+        options: [
+          ...(question.options ?? []).map((option) => ({
+            label: option.label,
+            ...(option.description !== undefined ? { description: option.description } : {}),
+          })),
+          SUPPLEMENT_OPTION,
+        ],
+        multiSelect: true,
+      }));
+
+      // 4. round-end supplement question (R3)
+      questions.push(ROUND_END_QUESTION);
+
+      // 5. ask through the userQuestions seam (UI renders from the service)
+      const answer = await ctx.userQuestions.ask({
+        questions,
+        ...(exec.agent !== undefined ? { agent: exec.agent } : {}),
+        signal: exec.signal,
+      });
+      return {
+        answers: answer.answers.map((entry) => ({
+          id: entry.id,
+          selected: [...entry.selected],
+          ...(entry.custom !== undefined ? { custom: entry.custom } : {}),
+        })),
+      };
+    },
+  }));
+
+  ctx.tools.register(defineTool({
+    name: "enter_plan_mode",
+    description: "Enter plan mode for the current agent. Call it after the final round of a grilling session once the user has confirmed shared understanding, so a plan is written for review instead of executing directly. Plan mode ends via exit_plan_mode or the /plan off command.",
+    parameters: {},
+    output: {
+      schema: {
+        type: "object",
+        additionalProperties: false,
+        properties: {
+          ok: { type: "boolean" },
+          result: { type: "string" },
+          error: { type: "string" },
+        },
+      },
+      render: (_args, value) => [{ type: "text", text: JSON.stringify(value) }],
+    },
+    async execute(_args, exec) {
+      const planMode = ctx.get("planMode");
+      if (planMode === undefined) return { ok: false, error: "planMode service unavailable" };
+      const agent = exec.agent;
+      if (agent === undefined) return { ok: false, error: "current agent unavailable" };
+      const outcome = planMode.set(agent, true);
+      // cancelled = an opposite pending selection was cleared and the logged
+      // state already matches the requested active state — plan mode IS active.
+      const ok = outcome === "committed" || outcome === "queued" || outcome === "cancelled" || outcome === "noop";
+      return { ok, result: `plan mode ${outcome}` };
+    },
+  }));
+}
+
+export { apply, inject, name };

package/package.json

@@ -0,0 +1,24 @@
+{
+  "name": "@lynn123411/dsh-ask-user-grilling",
+  "version": "0.1.1",
+  "description": "DSH delivery adaptations for grilling rounds: ask_user_grilling (subagent gate / forced multi-select / supplement option / stem guidance) and enter_plan_mode. Does not modify any skill file.",
+  "type": "module",
+  "main": "lib/index.js",
+  "exports": {
+    ".": "./lib/index.js",
+    "./package.json": "./package.json"
+  },
+  "files": [
+    "lib",
+    "README.md"
+  ],
+  "license": "MIT",
+  "peerDependencies": {
+    "@deepseek-ai/cordis": "^4.0.1",
+    "@deepseek-ai/dsh-tools": "^0.1.2-alpha.1",
+    "@deepseek-ai/dsh-user-questions": "^0.1.2-alpha.1"
+  },
+  "publishConfig": {
+    "access": "public"
+  }
+}