@dunnewold-labs/mr-manager 0.4.25 → 0.4.27

package/dist/index.mjs

@@ -185,7 +185,7 @@ import { fileURLToPath } from "url";
 // cli/package.json
 var package_default = {
   name: "@dunnewold-labs/mr-manager",
-  version: "0.4.25",
+  version: "0.4.27",
   description: "Mr. Manager - Task and project management CLI",
   bin: {
     mr: "./dist/index.mjs"
@@ -2498,7 +2498,8 @@ ${systemPrompt}` : prompt2;
   if (mode === "plan") {
     return { bin: "claude", args: [...sessionArgs, ...nameArgs, ...systemArgs, ...turnsArgs, ...modelArgs, "--permission-mode", "plan", "-p", prompt2] };
   }
-  return { bin: "claude", args: [...sessionArgs, ...nameArgs, ...systemArgs, ...turnsArgs, ...modelArgs, "--dangerously-skip-permissions", "-p", prompt2] };
+  const permissionArgs = ["--dangerously-skip-permissions"];
+  return { bin: "claude", args: [...sessionArgs, ...nameArgs, ...systemArgs, ...turnsArgs, ...modelArgs, ...permissionArgs, "-p", prompt2] };
 }
 function commandExists(cmd) {
   return new Promise((resolve9) => {
@@ -3011,14 +3012,19 @@ var watchCommand = new Command8("watch").description(
                   await postTaskUpdate(task.id, `Agent finished \u2014 no ${prLabel} found for branch ${branchName}`, "system");
                 }
               }
-              await api.patch(`/api/tasks/${task.id}`, {
-                status: "review",
-                claudeSessionId: activeEntry.currentAgent === "claude" ? void 0 : null,
-                ...prUrl ? { link: prUrl } : {}
-              });
-              logSuccess(prefix, `"${paint("bold", task.title)}" marked ready for review`);
-              if (noMrRequested) {
-                await postTaskUpdate(task.id, `No ${prLabel} required: ${noMrDescription}`, "system");
+              const currentTask = await api.get(`/api/tasks/${task.id}`);
+              if (currentTask.status === "completed" || currentTask.status === "review") {
+                logWarn(prefix, `Task already in "${currentTask.status}" state \u2014 skipping status update`);
+              } else {
+                await api.patch(`/api/tasks/${task.id}`, {
+                  status: "review",
+                  claudeSessionId: activeEntry.currentAgent === "claude" ? void 0 : null,
+                  ...prUrl ? { link: prUrl } : {}
+                });
+                logSuccess(prefix, `"${paint("bold", task.title)}" marked ready for review`);
+                if (noMrRequested) {
+                  await postTaskUpdate(task.id, `No ${prLabel} required: ${noMrDescription}`, "system");
+                }
               }
             } catch (err) {
               if (isLikelyNetworkError(err)) {
@@ -3745,6 +3751,7 @@ ${divider}`);
         if (taskId.startsWith("proto-") || taskId.startsWith("repo-") || taskId.startsWith("scan-")) continue;
         if (!activeTaskIds.has(taskId)) {
           logWarn(watchTag(), `Task ${paint("yellow", taskId.slice(0, 8))} no longer active, terminating\u2026`);
+          entry.terminatedForError = true;
           entry.process.kill("SIGTERM");
           active.delete(taskId);
           queued.delete(taskId);
@@ -4121,20 +4128,20 @@ ${divider}`);
         const prefix = taskTag(sid);
         const prLabel = task.link.includes("gitlab") ? "MR" : "PR";
         const vcs = task.link.includes("gitlab") ? "gitlab" : "github";
-        const repoDir = findDirectoryForProject(config, task.projectId, rootDir) ?? rootDir;
+        const repoDir = findDirectoryForProject(config, task.projectId, rootDir);
+        if (!repoDir) continue;
         const status = await checkPrStatus(task.link, repoDir, vcs);
         if (!status) continue;
         if (status.merged) {
-          logSuccess(prefix, `${prLabel} merged \u2014 auto-completing "${paint("bold", task.title)}"`);
+          if (task.autoMergedAt) continue;
+          logSuccess(prefix, `${prLabel} merged \u2014 marking for acknowledgment "${paint("bold", task.title)}"`);
           try {
             await api.patch(`/api/tasks/${task.id}`, {
-              status: "completed",
-              completedAt: (/* @__PURE__ */ new Date()).toISOString(),
-              inProgressSince: null
+              autoMergedAt: (/* @__PURE__ */ new Date()).toISOString()
             });
-            await postTaskUpdate(task.id, `${prLabel} merged \u2014 task automatically completed`, "system");
+            await postTaskUpdate(task.id, `${prLabel} merged \u2014 waiting for your acknowledgment to complete the task`, "system");
           } catch (err) {
-            logError(prefix, `Failed to auto-complete task: ${err.message}`);
+            logError(prefix, `Failed to mark task as auto-merged: ${err.message}`);
           }
           continue;
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dunnewold-labs/mr-manager",
-  "version": "0.4.25",
+  "version": "0.4.27",
   "description": "Mr. Manager - Task and project management CLI",
   "bin": {
     "mr": "./dist/index.mjs"