dsh-tree-task-flow 0.3.4 → 0.3.5

package/lib/tools.js

@@ -19,8 +19,10 @@
 
 import {
   addChildren,
+  addGoal,
   complete,
   createPlan,
+  doneReply,
   dropNode,
   renderContinue,
   renderTree,
@@ -76,8 +78,10 @@ export function registerTools(ctx, { store, cfg }) {
   ctx.tools.register({
     name: "tree_task_create",
     description:
-      "建立一个新计划：一个目标 + 若干任务。这会替换掉现有计划，所以只在开始一个新目标时调用。" +
-      "建完记得用 tree_task_plan 给目标下的第一个任务拆子任务。",
+      "新增一个目标：这个目标 + 它下面的若干任务。目标是要交付的东西，一个会话可以有多个目标，" +
+      "按调用顺序排在计划里——已经建过的目标和它们的历史都留着，不会被这次调用替换掉。" +
+      "这里的任务是交付节点——一件能独立验收的交付物，不是步骤本身。" +
+      "建完记得用 tree_task_plan 给这个目标下的第一个任务拆子任务。",
     parameters: {
       type: "object",
       additionalProperties: false,
@@ -86,7 +90,7 @@ export function registerTools(ctx, { store, cfg }) {
         detail: { type: "string", description: "目标的补充说明" },
         tasks: {
           type: "array",
-          description: "目标下的任务列表，按建议的执行顺序排列",
+          description: "目标下的任务列表（交付节点），按建议的执行顺序排列",
           items: {
             type: "object",
             additionalProperties: false,
@@ -102,15 +106,25 @@ export function registerTools(ctx, { store, cfg }) {
     },
     output: OUTPUT,
     async execute(args, exec) {
+      const sessionId = sessionOf(exec);
+      const existing = store.readPlan(sessionId);
       let plan;
+      let goal;
       try {
-        plan = createPlan(args);
+        if (existing === null) {
+          plan = createPlan(args);
+          [goal] = plan.goals;
+        } else {
+          plan = existing;
+          goal = addGoal(plan, args);
+        }
       } catch (error) {
-        return ok(`建立计划失败：${String(error?.message ?? error)}`);
+        return ok(`建立目标失败：${String(error?.message ?? error)}`);
       }
-      store.writePlan(sessionOf(exec), plan);
+      store.writePlan(sessionId, plan);
       return ok(
-        `计划已建立。\n\n${renderTree(plan)}\n\n下一步：调用 tree_task_plan 给目标下的第一个任务拆子任务。`,
+        `${existing === null ? "计划已建立。" : `已新增目标「${goal.title}」。`}\n\n${renderTree(plan)}\n\n` +
+          "下一步：调用 tree_task_plan 给这个目标下的第一个任务拆子任务。",
       );
     },
   });
@@ -118,8 +132,11 @@ export function registerTools(ctx, { store, cfg }) {
   ctx.tools.register({
     name: "tree_task_plan",
     description:
-      "给一个节点追加子节点：给目标追加任务，给任务追加子任务。子任务是最低一级，不能再往下拆。" +
-      "子任务全部结束后父节点不会自动完成，而是转入检查点——那时再决定继续拆还是提交结果。",
+      "给一个节点追加子节点：给目标追加任务，给任务追加子任务。" +
+      "给任务追加的是步骤（一两次工具调用能做完的事），别把任务拆到只剩一两个步骤。" +
+      "子任务是最低一级，不能再往下拆。" +
+      "子任务全部结束后父节点不会自动完成：插件会接着告诉你这一层的结果够不够，" +
+      "够了就用 tree_task_done 提交，不够再补步骤。",
     parameters: {
       type: "object",
       additionalProperties: false,
@@ -127,7 +144,7 @@ export function registerTools(ctx, { store, cfg }) {
         parentId: { type: "string", description: "父节点 id（目标或任务），例如 t-abc123" },
         children: {
           type: "array",
-          description: "要追加的子节点，按执行顺序；父节点是目标时是任务，是任务时是子任务",
+          description: "要追加的子节点，按执行顺序；父节点是目标时是任务（交付节点），是任务时是子任务（步骤）",
           items: {
             type: "object",
             additionalProperties: false,
@@ -163,9 +180,12 @@ export function registerTools(ctx, { store, cfg }) {
     name: "tree_task_done",
     description:
       "完成一个节点并提交它的结果（目标 / 任务 / 子任务都给它的 id）。**必须写 result**：" +
-      "插件会把这个节点从开始到现在的执行过程从上下文里折叠掉，换成一条只携带这段结果的汇总消息，" +
-      "所以 result 要写清它究竟产出了什么，写成能独立看懂的样子。" +
-      "子任务全部结束后父节点不会自动完成，而是转入检查点；那时再决定继续拆子任务还是提交父节点的结果。",
+      "插件会把这个节点从开始到现在的执行过程从上下文里折叠掉，换成一条只携带这段结果的汇总消息。" +
+      "任务的 result 写交付物与验收方式（文件在哪、怎么跑、验出什么、已知限制）；" +
+      "子任务的 result 一句话说清这步做了什么就够。" +
+      "子任务全部结束后父节点不会自动完成：插件会在返回里告诉你这一层的结果够不够，" +
+      "够就用它自己的 id 再调一次。" +
+      "返回内容就是下一步该干什么；这次调用与它的返回都留在上下文里，被收起的只是它前面的执行过程。",
     parameters: {
       type: "object",
       additionalProperties: false,
@@ -173,7 +193,9 @@ export function registerTools(ctx, { store, cfg }) {
         id: { type: "string", description: "要完成的节点 id" },
         result: {
           type: "string",
-          description: "这个节点产出了什么。它整段执行过程会被这条结果顶替，所以要能独立看懂。",
+          description:
+            "这个节点产出了什么。它整段执行过程会被这条结果顶替，所以要能独立看懂：" +
+            "任务写交付物与验收方式，子任务写清这步做了什么。",
         },
       },
       required: ["id", "result"],
@@ -185,9 +207,9 @@ export function registerTools(ctx, { store, cfg }) {
       const result = complete(plan, args?.id, args?.result);
       if (!result.ok) return ok(`没有完成：${result.reason}`);
       store.writePlan(sessionOf(exec), plan);
-      return ok(
-        [...result.events, "", renderTree(plan), "", renderContinue(plan)].join("\n"),
-      );
+      // 返回的就是"下一步该干什么"：这条结果连同这次调用都留在表层上，
+      // 折叠只收走它前面的执行过程。不重复节点状态，也不回整棵树。
+      return ok(doneReply(plan));
     },
   });
 
@@ -218,7 +240,8 @@ export function registerTools(ctx, { store, cfg }) {
   ctx.tools.register({
     name: "tree_task_drop",
     description:
-      "丢弃一个节点。丢弃任务会连同它的子任务一起丢弃。改需求用这个，不要重建整棵树——" +
+      "丢弃一个节点：目标、任务、子任务都能丢。丢弃目标会连同它下面的任务与子任务一起走，" +
+      "丢弃任务会带上它的子任务。改需求用这个，不要重建整棵树——" +
       "已丢弃的节点会留在树里可查。",
     parameters: {
       type: "object",

package/package.json

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