agenthub-multiagent-mcp 1.64.0 → 1.65.0

package/dist/tools/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/tools/index.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,SAAS,EAAE,OAAO,EAAqD,MAAM,cAAc,CAAC;AAOrG,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,oCAAoC,CAAC;AAkG/D,MAAM,WAAW,WAAW;IAC1B,KAAK,EAAE,WAAW,EAAE,CAAC;IACrB,QAAQ,EAAE,OAAO,EAAE,CAAC;CACrB;AAED,MAAM,WAAW,WAAW;IAC1B,iBAAiB,EAAE,MAAM,MAAM,CAAC;IAChC,iBAAiB,EAAE,CAAC,EAAE,EAAE,MAAM,KAAK,IAAI,CAAC;IACxC,aAAa,EAAE,MAAM,IAAI,CAAC;IAC1B,aAAa,EAAE,MAAM,MAAM,CAAC;IAC5B,aAAa,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,IAAI,CAAC;IACrC;;;OAGG;IACH,cAAc,EAAE,MAAM,WAAW,CAAC;IAClC,gDAAgD;IAChD,eAAe,CAAC,EAAE,MAAM,WAAW,CAAC;IACpC,oEAAoE;IACpE,iBAAiB,CAAC,EAAE,CAAC,QAAQ,EAAE;QAC7B,OAAO,CAAC,EAAE,MAAM,EAAE,CAAC;QACnB,UAAU,CAAC,EAAE,MAAM,EAAE,CAAC;KACvB,KAAK,IAAI,CAAC;CACZ;AAED,UAAU,WAAW;IACnB,EAAE,EAAE,MAAM,CAAC;IACX,WAAW,EAAE,MAAM,CAAC;IACpB,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,MAAM,CAAC;IACjB,UAAU,EAAE,MAAM,CAAC;CACpB;AAyLD,wBAAgB,aAAa,IAAI,IAAI,EAAE,CAk2FtC;AA2BD,wBAAsB,cAAc,CAClC,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC7B,MAAM,EAAE,SAAS,EACjB,OAAO,EAAE,WAAW,GACnB,OAAO,CAAC,OAAO,CAAC,CAyuFlB"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/tools/index.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,SAAS,EAAE,OAAO,EAAqD,MAAM,cAAc,CAAC;AAOrG,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,oCAAoC,CAAC;AAmG/D,MAAM,WAAW,WAAW;IAC1B,KAAK,EAAE,WAAW,EAAE,CAAC;IACrB,QAAQ,EAAE,OAAO,EAAE,CAAC;CACrB;AAED,MAAM,WAAW,WAAW;IAC1B,iBAAiB,EAAE,MAAM,MAAM,CAAC;IAChC,iBAAiB,EAAE,CAAC,EAAE,EAAE,MAAM,KAAK,IAAI,CAAC;IACxC,aAAa,EAAE,MAAM,IAAI,CAAC;IAC1B,aAAa,EAAE,MAAM,MAAM,CAAC;IAC5B,aAAa,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,IAAI,CAAC;IACrC;;;OAGG;IACH,cAAc,EAAE,MAAM,WAAW,CAAC;IAClC,gDAAgD;IAChD,eAAe,CAAC,EAAE,MAAM,WAAW,CAAC;IACpC,oEAAoE;IACpE,iBAAiB,CAAC,EAAE,CAAC,QAAQ,EAAE;QAC7B,OAAO,CAAC,EAAE,MAAM,EAAE,CAAC;QACnB,UAAU,CAAC,EAAE,MAAM,EAAE,CAAC;KACvB,KAAK,IAAI,CAAC;CACZ;AAED,UAAU,WAAW;IACnB,EAAE,EAAE,MAAM,CAAC;IACX,WAAW,EAAE,MAAM,CAAC;IACpB,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,MAAM,CAAC;IACjB,UAAU,EAAE,MAAM,CAAC;CACpB;AAyLD,wBAAgB,aAAa,IAAI,IAAI,EAAE,CAk2FtC;AA2BD,wBAAsB,cAAc,CAClC,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC7B,MAAM,EAAE,SAAS,EACjB,OAAO,EAAE,WAAW,GACnB,OAAO,CAAC,OAAO,CAAC,CA+qFlB"}

package/dist/tools/index.js

@@ -10,6 +10,7 @@ import * as brainCapture from "../hooks/brainCapture.js";
 import { getBrain } from "../brain/backend.js";
 import open from "open";
 import { registerOpenSpecVizTools, handleOpenSpecVizToolCall } from "./openspec-viz/index.js";
+import { syncTaskGroups, describeTaskSync } from "./openspec-viz/taskSync.js";
 import { registerIntakeRequestTools, handleIntakeRequestToolCall, isIntakeRequestTool, } from "./intakeRequests.js";
 import { registerSessionControlTools, handleSessionControlToolCall, isSessionControlTool, } from "./sessionControl.js";
 import { registerGoshtiReadTools, handleGoshtiReadToolCall, isGoshtiReadTool, } from "./goshtiRead.js";
@@ -4750,89 +4751,29 @@ export async function handleToolCall(name, args, client, context) {
             catch {
                 // Non-fatal if cleanup fails
             }
-            // 5. Sync tasks to ticket system (create Stories + Tasks from spec task groups)
-            let taskSyncResult;
+            // 5. Sync tasks to ticket system (create Stories + Tasks from spec task groups).
+            //    The sync reports its own outcome. It used to be wrapped in a bare
+            //    `catch` here, which made a crashed sync byte-identical to one that
+            //    was never requested — the hierarchy was then discovered to be
+            //    missing only by independently listing the epic's children
+            //    (AGHUB-1469).
             const publishAgentState = workingDir ? state.loadState(workingDir) : undefined;
-            if (publishResult.sync_tasks && publishResult.taskGroups.length > 0 && publishAgentState?.project_id) {
-                taskSyncResult = { storiesCreated: 0, tasksCreated: 0, storiesSkipped: 0, tasksSkipped: 0 };
+            const taskSyncOutcome = await syncTaskGroups({
+                client,
+                projectId: publishAgentState?.project_id ?? "",
+                taskGroups: publishResult.taskGroups,
+                ticketType: publishResult.ticket_type,
+                ticketId: publishResult.ticket_id,
                 // The change id is the input arg `id` — in change/diff modes that's
-                // the openspec change name. Stamp newly-created stories and tasks
-                // with it so the hierarchy resolver can find them.
-                const changeID = typeof args.id === "string" ? args.id : undefined;
-                try {
-                    // Get existing stories to detect duplicates
-                    const existingStories = await client.listProjectStories(publishAgentState.project_id);
-                    const storyTitleMap = new Map(); // title → id
-                    for (const s of existingStories.stories || []) {
-                        storyTitleMap.set(s.title.toLowerCase().trim(), s.id);
-                    }
-                    for (const group of publishResult.taskGroups) {
-                        const groupTitle = group.title.trim();
-                        if (!groupTitle)
-                            continue;
-                        // Create or find story for this task group
-                        let storyId;
-                        const existingStoryId = storyTitleMap.get(groupTitle.toLowerCase());
-                        if (existingStoryId) {
-                            storyId = existingStoryId;
-                            taskSyncResult.storiesSkipped++;
-                        }
-                        else {
-                            const storyResult = await client.createStory({
-                                project_id: publishAgentState.project_id,
-                                epic_id: publishResult.ticket_type === "epic" ? publishResult.ticket_id : undefined,
-                                title: groupTitle,
-                                labels: ["openspec-sync"],
-                                openspec_change_id: changeID,
-                            });
-                            storyId = storyResult.id;
-                            storyTitleMap.set(groupTitle.toLowerCase(), storyId);
-                            taskSyncResult.storiesCreated++;
-                        }
-                        // Get existing tasks for this story — map titles to ticket info for dedup + mapping
-                        const existingTasks = await client.listProjectTasks(publishAgentState.project_id, storyId);
-                        const taskInfoMap = new Map();
-                        for (const t of existingTasks.tasks || []) {
-                            taskInfoMap.set(t.title.toLowerCase().trim(), { id: t.id, key: t.key });
-                        }
-                        // Create tasks for each item in the group
-                        for (const task of group.tasks) {
-                            const taskTitle = task.text.trim();
-                            if (!taskTitle)
-                                continue;
-                            const existingTask = taskInfoMap.get(taskTitle.toLowerCase());
-                            if (existingTask) {
-                                // Store the ticket mapping even for skipped (existing) tasks
-                                task.ticket_id = existingTask.id;
-                                task.ticket_key = existingTask.key;
-                                taskSyncResult.tasksSkipped++;
-                                continue;
-                            }
-                            const createdTask = await client.createTask({
-                                project_id: publishAgentState.project_id,
-                                story_id: storyId,
-                                title: taskTitle,
-                                labels: ["openspec-sync"],
-                            });
-                            // Store the ticket mapping for the newly created task. Tasks
-                            // created here go through story_id, so the discriminated
-                            // result is always ok:true; ignore the !ok branch silently
-                            // (the outer try/catch will swallow truly unexpected errors).
-                            if (createdTask.ok) {
-                                task.ticket_id = createdTask.id;
-                                task.ticket_key = createdTask.key;
-                                taskSyncResult.tasksCreated++;
-                            }
-                        }
-                    }
-                }
-                catch {
-                    // Task sync failure is non-fatal
-                    taskSyncResult = undefined;
-                }
-            }
-            // 6. Re-upload JSON with ticket ID mappings if task sync produced mappings
-            if (taskSyncResult) {
+                // the openspec change name. Stamp newly-created stories with it so the
+                // hierarchy resolver can find them.
+                changeId: typeof args.id === "string" ? args.id : undefined,
+                enabled: publishResult.sync_tasks,
+            });
+            // 6. Re-upload JSON with ticket ID mappings if task sync produced mappings.
+            //    A failed sync may still have mapped part of the hierarchy, so the
+            //    mappings it did write are preserved rather than discarded.
+            if (taskSyncOutcome.status !== "skipped") {
                 try {
                     const fs = await import("fs");
                     // Update the parsed JSON object's tasks with ticket mappings
@@ -4868,9 +4809,11 @@ export async function handleToolCall(name, args, client, context) {
                 sections: publishResult.sections,
                 requirementCount: publishResult.requirementCount,
                 gapCount: publishResult.gapCount,
-                taskSync: taskSyncResult,
-                message: `Published OpenSpec to ${publishResult.ticket_key} (HTML + JSON, ${publishResult.requirementCount} requirements)` +
-                    (taskSyncResult ? `. Task sync: ${taskSyncResult.storiesCreated} stories + ${taskSyncResult.tasksCreated} tasks created (${taskSyncResult.storiesSkipped + taskSyncResult.tasksSkipped} skipped as duplicates)` : ""),
+                taskSync: taskSyncOutcome,
+                // Always stated, so a caller reading only the message can tell a
+                // successful sync from a failed or skipped one.
+                message: `Published OpenSpec to ${publishResult.ticket_key} (HTML + JSON, ${publishResult.requirementCount} requirements). ` +
+                    describeTaskSync(taskSyncOutcome),
             }, context);
         }
         // Plan Review tools