@hdwebsoft/hdcode-agent-team 0.2.1 → 0.2.2

package/dist/index.js

@@ -236,36 +236,38 @@ function createTeamTools(directory) {
         description: tool.schema.string().describe("Team description")
       },
       async execute(args) {
-        const dir = teamDir(directory, args.teamName);
-        if (await exists(join4(dir, "config.json"))) {
-          return `Error: Team "${args.teamName}" already exists`;
-        }
-        await ensureDir(join4(dir, "tasks"));
-        await ensureDir(join4(dir, "inboxes"));
-        await ensureDir(join4(dir, "reports"));
-        const now = Date.now();
-        const leadId = `team-lead@${args.teamName}`;
-        const config = {
-          name: args.teamName,
-          description: args.description,
-          createdAt: now,
-          leadAgentId: leadId,
-          members: [
-            {
-              agentId: leadId,
-              name: "team-lead",
-              agentType: "team-lead",
-              model: "unknown",
-              planModeRequired: false,
-              joinedAt: now,
-              cwd: directory,
-              isActive: true
-            }
-          ]
-        };
-        await writeJsonAtomic(join4(dir, "config.json"), config);
-        await writeJsonAtomic(join4(dir, "inboxes", "team-lead.json"), []);
-        return JSON.stringify(config, null, 2);
+        return withTeamLock(directory, args.teamName, async () => {
+          const dir = teamDir(directory, args.teamName);
+          if (await exists(join4(dir, "config.json"))) {
+            return `Error: Team "${args.teamName}" already exists`;
+          }
+          await ensureDir(join4(dir, "tasks"));
+          await ensureDir(join4(dir, "inboxes"));
+          await ensureDir(join4(dir, "reports"));
+          const now = Date.now();
+          const leadId = `team-lead@${args.teamName}`;
+          const config = {
+            name: args.teamName,
+            description: args.description,
+            createdAt: now,
+            leadAgentId: leadId,
+            members: [
+              {
+                agentId: leadId,
+                name: "team-lead",
+                agentType: "team-lead",
+                model: "unknown",
+                planModeRequired: false,
+                joinedAt: now,
+                cwd: directory,
+                isActive: true
+              }
+            ]
+          };
+          await writeJsonAtomic(join4(dir, "config.json"), config);
+          await writeJsonAtomic(join4(dir, "inboxes", "team-lead.json"), []);
+          return JSON.stringify(config, null, 2);
+        });
       }
     }),
     team_delete: tool({
@@ -274,12 +276,14 @@ function createTeamTools(directory) {
         teamName: tool.schema.string().describe("Team name to delete")
       },
       async execute(args) {
-        const dir = teamDir(directory, args.teamName);
-        if (!await exists(join4(dir, "config.json"))) {
-          return `Error: Team "${args.teamName}" not found`;
-        }
-        await rm2(dir, { recursive: true, force: true });
-        return `Team "${args.teamName}" deleted successfully`;
+        return withTeamLock(directory, args.teamName, async () => {
+          const dir = teamDir(directory, args.teamName);
+          if (!await exists(join4(dir, "config.json"))) {
+            return `Error: Team "${args.teamName}" not found`;
+          }
+          await rm2(dir, { recursive: true, force: true });
+          return `Team "${args.teamName}" deleted successfully`;
+        });
       }
     }),
     team_status: tool({
@@ -791,6 +795,24 @@ function createTaskTools(directory) {
             return `Error: Task "${args.taskId}" not found in team "${args.teamName}"`;
           }
           const task = await readJson(tPath);
+          const originalOwner = task.owner;
+          if (args.subject !== undefined)
+            task.subject = args.subject;
+          if (args.description !== undefined)
+            task.description = args.description;
+          if (args.activeForm !== undefined)
+            task.activeForm = args.activeForm;
+          if (args.owner !== undefined)
+            task.owner = args.owner;
+          if (args.metadata) {
+            let newMeta;
+            try {
+              newMeta = JSON.parse(args.metadata);
+            } catch {
+              return "Error: Invalid metadata JSON";
+            }
+            task.metadata = { ...task.metadata, ...newMeta };
+          }
           if (args.status) {
             const validTransitions = {
               pending: ["in_progress"],
@@ -819,7 +841,7 @@ function createTaskTools(directory) {
             if (args.status === "in_progress") {
               const fileScope = task.metadata?.fileScope;
               if (fileScope) {
-                const agentId = args.owner || task.owner || "unknown";
+                const agentId = task.owner || "unknown";
                 const reserveResult = await reserveFiles(directory, args.teamName, {
                   agentId,
                   taskId: task.id,
@@ -832,36 +854,22 @@ function createTaskTools(directory) {
               }
             }
             if (args.status === "completed") {
-              const agentId = args.owner || task.owner;
-              if (agentId) {
-                await releaseByTask(directory, args.teamName, agentId, task.id);
+              if (originalOwner) {
+                await releaseByTask(directory, args.teamName, originalOwner, task.id);
               }
             }
             task.status = args.status;
           }
-          if (args.subject !== undefined)
-            task.subject = args.subject;
-          if (args.description !== undefined)
-            task.description = args.description;
-          if (args.activeForm !== undefined)
-            task.activeForm = args.activeForm;
-          if (args.owner !== undefined)
-            task.owner = args.owner;
-          if (args.metadata) {
-            let newMeta;
-            try {
-              newMeta = JSON.parse(args.metadata);
-            } catch {
-              return "Error: Invalid metadata JSON";
-            }
-            task.metadata = { ...task.metadata, ...newMeta };
-          }
           if (args.addBlocks) {
             const result = parseStringArray(args.addBlocks, "addBlocks");
             if (typeof result === "string")
               return result;
             const allTasks = await loadAllTasks(directory, args.teamName);
             const taskMap = new Map(allTasks.map((t) => [t.id, t]));
+            for (const targetId of result) {
+              if (!taskMap.has(targetId))
+                return `Error: Task "${targetId}" not found — cannot add to blocks`;
+            }
             if (hasCycleViaBlocks(task.id, result, task.blockedBy, taskMap)) {
               return `Error: Circular dependency detected — cannot add blocks`;
             }
@@ -870,12 +878,10 @@ function createTaskTools(directory) {
                 task.blocks.push(targetId);
               }
               const targetPath = taskFilePath(directory, args.teamName, targetId);
-              if (await exists(targetPath)) {
-                const target = await readJson(targetPath);
-                if (!target.blockedBy.includes(task.id)) {
-                  target.blockedBy.push(task.id);
-                  await writeJsonAtomic(targetPath, target);
-                }
+              const target = await readJson(targetPath);
+              if (!target.blockedBy.includes(task.id)) {
+                target.blockedBy.push(task.id);
+                await writeJsonAtomic(targetPath, target);
               }
             }
           }
@@ -885,6 +891,10 @@ function createTaskTools(directory) {
               return result;
             const allTasks = await loadAllTasks(directory, args.teamName);
             const taskMap = new Map(allTasks.map((t) => [t.id, t]));
+            for (const targetId of result) {
+              if (!taskMap.has(targetId))
+                return `Error: Task "${targetId}" not found — cannot add to blockedBy`;
+            }
             if (hasCycle(task.id, result, taskMap)) {
               return `Error: Circular dependency detected — cannot add blockedBy`;
             }
@@ -893,12 +903,10 @@ function createTaskTools(directory) {
                 task.blockedBy.push(targetId);
               }
               const targetPath = taskFilePath(directory, args.teamName, targetId);
-              if (await exists(targetPath)) {
-                const target = await readJson(targetPath);
-                if (!target.blocks.includes(task.id)) {
-                  target.blocks.push(task.id);
-                  await writeJsonAtomic(targetPath, target);
-                }
+              const target = await readJson(targetPath);
+              if (!target.blocks.includes(task.id)) {
+                target.blocks.push(task.id);
+                await writeJsonAtomic(targetPath, target);
               }
             }
           }
@@ -1015,6 +1023,7 @@ function createMessageTools(directory) {
         const inboxesDir = join7(dir, "inboxes");
         await ensureDir(inboxesDir);
         const message = {
+          type: args.type,
           from: sender,
           text: args.content,
           ...args.summary && { summary: args.summary },
@@ -1061,11 +1070,17 @@ function createMessageTools(directory) {
       },
       async execute(args) {
         const dir = teamDir(directory, args.teamName);
-        if (!await exists(join7(dir, "config.json"))) {
+        const configPath = join7(dir, "config.json");
+        if (!await exists(configPath)) {
           return `Error: Team "${args.teamName}" not found`;
         }
         const agentName = args.agent.includes("@") ? args.agent.split("@")[0] : args.agent;
-        const inboxPath = join7(dir, "inboxes", `${agentName}.json`);
+        const config = await readJson(configPath);
+        const member = config.members.find((m) => m.name === agentName || m.agentId === args.agent);
+        if (!member) {
+          return `Error: Agent "${args.agent}" not found in team "${args.teamName}"`;
+        }
+        const inboxPath = join7(dir, "inboxes", `${member.name}.json`);
         if (args.markRead) {
           return withTeamLock(directory, args.teamName, async () => {
             const allMessages = await readInbox(inboxPath);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hdwebsoft/hdcode-agent-team",
-  "version": "0.2.1",
+  "version": "0.2.2",
   "description": "OpenCode plugin for multi-agent team coordination — per-agent inboxes, task management with dependency tracking, file reservation/locking, and atomic mkdir-based concurrency control.",
   "type": "module",
   "main": "./dist/index.js",