@kendoo.agentdesk/agentdesk 0.29.1 → 0.29.3

package/CHANGELOG.md

@@ -8,6 +8,11 @@ All user-facing changes to AgentDesk. Each entry is tagged:
 
 Internal refactors, infrastructure changes, and architectural notes are not listed here.
 
+## [0.29.3] — 2026-09-21
+
+### Fixed
+- `[CLI]` The dashboard's new task-lookup (which recommends resuming, continuing, or starting fresh for a task id) could miss a workspace that already had unfinished local work, then try to check out its branch a second time and fail with `fatal: '...' is already used by worktree at ...`. Lookups now match by the repo's actual path instead of a locally-derived project label that could go stale. (Published as 0.29.3 — 0.29.2 got stuck mid-publish on the registry.)
+
 ## [0.29.1] — 2026-09-20
 
 ### Fixed

package/cli/daemon.mjs

@@ -15,7 +15,7 @@ import { getRegisteredProjects, registerLocalProject, claimLocalProjects } from
 import { buildTrackerUrl } from "./tracker-url.mjs";
 import { fileURLToPath } from "url";
 import { dirname } from "path";
-import { repositoryInfo, listWorkspaces, inspectWorkspace, cleanupWorkspace, recoverWorkspaceLocks } from "./worktrees.mjs";
+import { repositoryInfo, listWorkspaces, findWorkspacesByTask, inspectWorkspace, cleanupWorkspace, recoverWorkspaceLocks } from "./worktrees.mjs";
 import { createSessionQueue, sessionLimit, confirmWithAbort } from "./session-queue.mjs";
 
 const __dirname_daemon = dirname(fileURLToPath(import.meta.url));
@@ -419,7 +419,7 @@ async function confirmIncomingSession({ project, taskId, prompt, signal }) {
 
   // 4. Session handling
 
-  function handleWorkspaceRequest({ requestId, action, projectId, workspaceId, discard, confirmation }) {
+  function handleWorkspaceRequest({ requestId, action, projectId, workspaceId, discard, confirmation, taskId }) {
     try {
       const projectPaths = projects.map(p => p.path);
       let result;
@@ -435,6 +435,15 @@ async function confirmIncomingSession({ project, taskId, prompt, signal }) {
           }
           return inspectWorkspace(record);
         });
+      } else if (action === "lookup") {
+        const project = projects.find(p => p.id === projectId);
+        if (!project) throw new Error("Unknown project");
+        result = findWorkspacesByTask({ projectPath: project.path, taskId }).map(record => {
+          if (record.archived && !record.removedAt) {
+            try { return cleanupWorkspace(record.id, { projectPaths }); } catch {}
+          }
+          return inspectWorkspace(record);
+        });
       } else if (action === "cleanup" || action === "archive") {
         result = cleanupWorkspace(workspaceId, { projectPaths, discard: discard === true, confirmation, archive: action === "archive" });
       } else throw new Error("Unknown workspace action");

package/cli/worktrees.mjs

@@ -77,6 +77,18 @@ export function listWorkspaces({ root, projectPaths } = {}) {
   });
 }
 
+// Matches on the resolved repo path, not the workspace record's stored
+// projectId — that field is client-derived (falls back to package.json's
+// name when a project isn't fully configured) and can silently drift from
+// the server's project id, which would make an unfinished workspace
+// invisible to lookups and risk a second `git worktree add` colliding with
+// the branch it already holds.
+export function findWorkspacesByTask({ root, projectPath, taskId } = {}) {
+  if (!taskId || !projectPath) return [];
+  const resolved = realpathSync(projectPath);
+  return listWorkspaces({ root }).filter(record => record.sourceCwd === resolved && record.taskId === taskId);
+}
+
 export function getWorkspace(id, { root, projectPaths } = {}) {
   const record = read(join(rootPath(root), `${hash(id)}.json`));
   if (record.id !== id || (projectPaths && !projectPaths.some(p => realpathSync(p) === record.sourceCwd))) throw new Error("Workspace not found for this project.");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kendoo.agentdesk/agentdesk",
-  "version": "0.29.1",
+  "version": "0.29.3",
   "description": "AI team orchestrator for Claude Code — run collaborative agent sessions from your terminal",
   "type": "module",
   "bin": {
@@ -22,7 +22,7 @@
     "server": "node server/index.mjs",
     "build": "vite build",
     "preview": "vite preview",
-    "test": "node --test tests/server.test.mjs tests/agents.test.mjs tests/homepage.test.mjs tests/sessionUtils.test.mjs tests/tracker-url.test.mjs tests/session-preflight.test.mjs tests/access.test.mjs tests/project-ownership.test.mjs tests/random-hex.test.mjs tests/projects-registry.test.mjs tests/phase-loop.test.mjs tests/proc.test.mjs tests/crypto.test.mjs tests/dotenv.test.mjs tests/update-check.test.mjs tests/setup-helpers.test.mjs tests/project-key.test.mjs tests/tracker-project.test.mjs tests/tracker-check.test.mjs tests/config.test.mjs tests/engine-env.test.mjs tests/engine-events.test.mjs tests/engine-verdict.test.mjs tests/engine-hooks.test.mjs tests/engine-agents.test.mjs tests/session-isolation.test.mjs tests/engine-session.test.mjs tests/engine-prompts.test.mjs tests/engine-schemas.test.mjs tests/engine-claude-auth.test.mjs tests/worktrees.test.mjs tests/workspaces-api.test.mjs tests/engine-outcome.test.mjs tests/session-outcomes.test.mjs tests/outcome-hydration.test.mjs tests/mobile-outcomes.test.mjs tests/session-queue.test.mjs tests/useFollowScroll.test.mjs tests/session-usage.test.mjs",
+    "test": "node --test tests/server.test.mjs tests/agents.test.mjs tests/homepage.test.mjs tests/sessionUtils.test.mjs tests/tracker-url.test.mjs tests/session-preflight.test.mjs tests/access.test.mjs tests/project-ownership.test.mjs tests/random-hex.test.mjs tests/projects-registry.test.mjs tests/phase-loop.test.mjs tests/proc.test.mjs tests/crypto.test.mjs tests/dotenv.test.mjs tests/update-check.test.mjs tests/setup-helpers.test.mjs tests/project-key.test.mjs tests/tracker-project.test.mjs tests/tracker-check.test.mjs tests/config.test.mjs tests/engine-env.test.mjs tests/engine-events.test.mjs tests/engine-verdict.test.mjs tests/engine-hooks.test.mjs tests/engine-agents.test.mjs tests/session-isolation.test.mjs tests/engine-session.test.mjs tests/engine-prompts.test.mjs tests/engine-schemas.test.mjs tests/engine-claude-auth.test.mjs tests/worktrees.test.mjs tests/workspaces-api.test.mjs tests/engine-outcome.test.mjs tests/session-outcomes.test.mjs tests/outcome-hydration.test.mjs tests/mobile-outcomes.test.mjs tests/session-queue.test.mjs tests/useFollowScroll.test.mjs tests/session-usage.test.mjs tests/task-lookup.test.mjs",
     "test:coverage": "node --test --experimental-test-coverage --test-coverage-include='cli/**' --test-coverage-include='server/**' --test-coverage-lines=60 --test-coverage-branches=62 tests/*.test.mjs",
     "lint": "eslint .",
     "lint:fix": "eslint . --fix",

package/shared/task-lookup.mjs

@@ -0,0 +1,41 @@
+// Pure decision logic for "what should agentdesk do with this task id" —
+// kept here so the server route and its tests share one source of truth.
+const PR_OUTCOME_KIND = "pr_created";
+
+function toCandidate(workspace) {
+  return { workspaceId: workspace.id, branch: workspace.branch, reason: workspace.reason };
+}
+
+export function recommendTaskAction({ workspaces = [], sessions = [] } = {}) {
+  const liveWorkspaces = workspaces.filter(w => !w.removedAt);
+
+  const inUse = liveWorkspaces.filter(w => w.active);
+  if (inUse.length === 1) return { action: "in_use", workspace: inUse[0], reason: inUse[0].reason };
+  if (inUse.length > 1) return { action: "multiple", candidates: inUse.map(toCandidate) };
+
+  const unfinished = liveWorkspaces.filter(w => !w.safe);
+  if (unfinished.length === 1) return { action: "resume", workspace: unfinished[0], reason: unfinished[0].reason };
+  if (unfinished.length > 1) return { action: "multiple", candidates: unfinished.map(toCandidate) };
+
+  const merged = liveWorkspaces.filter(w => w.safe);
+
+  const prOutcomes = sessions.flatMap(s => (s.outcomes || [])
+    .filter(o => o.kind === PR_OUTCOME_KIND)
+    .map(o => ({ ...o, sessionId: s.sessionId })));
+  if (prOutcomes.length > 0) {
+    const latest = prOutcomes.reduce((a, b) => (a.observedAt >= b.observedAt ? a : b));
+    return { action: "continue_pr", url: latest.url, branch: latest.branch || null, sessionId: latest.sessionId };
+  }
+
+  if (merged.length > 0) return { action: "already_merged", workspace: merged[0] };
+
+  const branches = [...new Set(sessions.map(s => s.branch || s.workspace?.branch).filter(Boolean))];
+  if (branches.length === 1) {
+    const branch = branches[0];
+    const session = sessions.find(s => (s.branch || s.workspace?.branch) === branch);
+    return { action: "continue_branch", branch, sessionId: session?.sessionId || null };
+  }
+  if (branches.length > 1) return { action: "multiple", candidates: branches.map(branch => ({ branch })) };
+
+  return { action: "start_new" };
+}