dsh-tree-task-flow 0.2.0 → 0.3.1

package/lib/tools.js

@@ -1,15 +1,11 @@
 /**
  * 六个 `tree_task_*` 工具。
  *
- * 工具本身**只做状态读写**——不注入消息、不动 surface。
+ * 工具本身**只做状态读写**——不注入消息、不动 surface，也不拦任何别的工具。
  * 「完成时把这个节点的执行过程从上下文里折起来、换成一条携带它结果的汇总」
  * 那件事完全由 `agent/pre-step` 监听器负责（它比对游标与 `activePath` 就知道
  * 哪一层刚结束）。
  *
- * 这边唯一的 hook 是**检查点闸门**：它也不碰会话、不碰 surface，只是在
- * `tools/pre-execute` 上拦一道——检查点期间除了拆子节点和提交结果，别的工具一律
- * 拒掉。判断本身在纯函数 `checkpointGate` 里，方便单测。
- *
  * 工具能决定的只有一件事：**这个节点算不算做完了**。而它做完了就必须交出
  * `result`——插件正是拿这段文字去顶替它整段执行过程。没有结果的完成会让那段
  * 上下文白白消失，所以 `result` 是必填的。
@@ -23,11 +19,9 @@
 
 import {
   addChildren,
-  checkpointOf,
   complete,
   createPlan,
   dropNode,
-  levelLabel,
   renderContinue,
   renderTree,
   updateNode,
@@ -55,43 +49,6 @@ function childLabel(level) {
   return level === "goal" ? "任务" : "子任务";
 }
 
-/**
- * 检查点期间放行的工具，就这两个。
- *
- * 检查点是"这一层只剩一个判断"的时刻：继续往下拆，或者交出结果收尾。
- * 那一刻再多放行任何一个工具，模型都可以绕开这个判断继续埋头干活，
- * 检查点也就成了摆设。所以这里是白名单，不是黑名单。
- */
-export const CHECKPOINT_ALLOWED = Object.freeze(["tree_task_plan", "tree_task_done"]);
-
-/** 在检查点里被挡下时，给模型看的话：局面是什么、只剩哪两条路。 */
-function checkpointDenyReason(toolName, checkpoint) {
-  const label = levelLabel(checkpoint.level);
-  const id = checkpoint.node.id;
-  return [
-    `现在是检查点：${label}「${checkpoint.node.title}」(${id}) 的子节点已全部结束。`,
-    `此刻只放行两个工具：tree_task_plan 继续给 ${id} 拆子节点，` +
-      `或者 tree_task_done(id="${id}", result="…") 提交这个${label}的结果并完成它。`,
-    `工具 "${toolName}" 不在放行范围内，已被拒绝。`,
-  ].join("\n");
-}
-
-/**
- * 检查点闸门的判定。
- *
- * **纯函数**——不读磁盘、不碰会话，只看出入参，所以可以直接单测。
- * 注册处只负责把当前计划读出来、把 `checkpointOf` 的结果递进来。
- *
- * @param toolName - 模型这次要调的工具名。
- * @param checkpoint - `checkpointOf(plan)` 的结果；`null` 表示不在检查点。
- * @returns `{ kind: "allow" }`，或 `{ kind: "deny", reason }`。
- */
-export function checkpointGate(toolName, checkpoint) {
-  if (checkpoint === null || checkpoint === undefined) return { kind: "allow" };
-  if (CHECKPOINT_ALLOWED.includes(toolName)) return { kind: "allow" };
-  return { kind: "deny", reason: checkpointDenyReason(toolName, checkpoint) };
-}
-
 /**
  * 注册全部工具。
  *
@@ -101,24 +58,6 @@ export function checkpointGate(toolName, checkpoint) {
 export function registerTools(ctx, { store, cfg }) {
   const sessionOf = (exec) => exec?.agent?.session?.id ?? null;
 
-  /**
-   * 检查点的硬闸门。
-   *
-   * 先跑完下游，别人已经拒了就尊重别人的决定——那多半是审批或 hook；
-   * 只有一路放行到这里，才轮到用检查点收敛。
-   *
-   * 计划按会话取，所以闸门只对"正处于检查点的那个会话"生效，
-   * 别的会话照常使用全部工具。
-   */
-  ctx.on("tools/pre-execute", async (exec, next) => {
-    const decision = await next();
-    if (decision?.kind !== "allow") return decision;
-
-    const checkpoint = checkpointOf(store.readPlan(sessionOf(exec)));
-    const verdict = checkpointGate(exec?.name, checkpoint);
-    return verdict.kind === "allow" ? decision : verdict;
-  });
-
   // ---------------------------------------------------------------- 计划树
 
   ctx.tools.register({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-tree-task-flow",
-  "version": "0.2.0",
+  "version": "0.3.1",
   "description": "树形任务流：把多步任务组织成目标→任务→子任务三级树，节点完成时由 AI 提交结果，插件随即把该节点的执行过程从上下文里折叠掉，只留下结果。",
   "type": "module",
   "main": "lib/index.js",
@@ -13,7 +13,7 @@
     "lib",
     "client.js",
     "cordis.patch.yml",
-    "enable.example.yml",
+    "config.example.yml",
     "README.md"
   ],
   "dsh": {

package/enable.example.yml

@@ -1,4 +0,0 @@
-# 显式启用插件用。默认 enabled 是 false——装上不等于启用。
-- id: task-tree
-  config:
-    enabled: true