From 225d3ea65cfc35d211fc66e30cf05cbc693d37e4 Mon Sep 17 00:00:00 2001 From: Adam Malczewski Date: Sat, 23 May 2026 05:25:03 +0900 Subject: feat: relative working directory support and subagent tab cwd propagation - Resolve relative cwd paths (e.g. ./subtask) against parent's working directory at runtime - check-dir endpoint resolves relative paths and returns the resolved absolute path - AgentBuilder shows resolved path below input for relative paths, updated helper text - tab-created event now includes workingDirectory so subagent tabs display their cwd in sidebar - Add workingDirectory to tab-created AgentEvent type definition - spawnChildAgent stores resolved absolute path instead of raw relative path --- packages/api/src/agent-manager.ts | 17 ++++++++++++++++- packages/api/src/routes/agents.ts | 9 +++++++-- 2 files changed, 23 insertions(+), 3 deletions(-) (limited to 'packages/api') diff --git a/packages/api/src/agent-manager.ts b/packages/api/src/agent-manager.ts index 74995b2..d473950 100644 --- a/packages/api/src/agent-manager.ts +++ b/packages/api/src/agent-manager.ts @@ -329,6 +329,15 @@ export class AgentManager { workingDirectory = join(homedir(), workingDirectory.slice(1)); } + // Resolve relative paths against the default working directory + // (e.g. subagent cwd "./subtask" resolves relative to the parent's effective dir) + { + const { isAbsolute, resolve } = await import("node:path"); + if (!isAbsolute(workingDirectory)) { + workingDirectory = resolve(defaultWorkDir, workingDirectory); + } + } + // Auto-create the working directory if it doesn't exist try { const { mkdirSync, existsSync } = await import("node:fs"); @@ -668,6 +677,8 @@ export class AgentManager { parentEffectiveDir = join(homedir(), parentEffectiveDir.slice(1)); } + // Resolve and validate child working directory against parent's effective dir + let resolvedWorkingDirectory = options.workingDirectory; if (options.workingDirectory) { const { isAbsolute, relative, resolve, join } = await import("node:path"); // Expand ~ in child working directory @@ -685,6 +696,9 @@ export class AgentManager { `Working directory "${options.workingDirectory}" is outside the parent's working directory "${parentDir}".`, ); } + // Store the resolved absolute path so downstream code doesn't + // re-resolve against the wrong base directory + resolvedWorkingDirectory = resolved; } // Intersect requested tools with parent's allowed tools to prevent privilege escalation @@ -696,7 +710,7 @@ export class AgentManager { // Create the tab agent entry with overrides const tabAgent = this._getOrCreateTabAgent(tabId); tabAgent.toolsOverride = childTools; - tabAgent.workingDirectoryOverride = options.workingDirectory; + tabAgent.workingDirectoryOverride = resolvedWorkingDirectory; tabAgent.keyId = options.parentKeyId ?? null; tabAgent.modelId = options.parentModelId ?? null; tabAgent.finalOutput = ""; @@ -727,6 +741,7 @@ export class AgentManager { keyId: tabAgent.keyId, modelId: tabAgent.modelId, parentTabId: options.parentTabId ?? null, + workingDirectory: resolvedWorkingDirectory ?? null, }, tabId, ); diff --git a/packages/api/src/routes/agents.ts b/packages/api/src/routes/agents.ts index d3ca23f..1627674 100644 --- a/packages/api/src/routes/agents.ts +++ b/packages/api/src/routes/agents.ts @@ -99,11 +99,16 @@ agentsRoutes.get("/check-dir", (c) => { if (dirPath === "~" || dirPath.startsWith("~/")) { dirPath = path.join(os.homedir(), dirPath.slice(1)); } + // Resolve relative paths against the project root + if (!path.isAbsolute(dirPath)) { + const projectDir = process.env.DISPATCH_WORKING_DIR || process.cwd(); + dirPath = path.resolve(projectDir, dirPath); + } try { const stat = fs.statSync(dirPath); - return c.json({ exists: stat.isDirectory() }); + return c.json({ exists: stat.isDirectory(), resolved: dirPath }); } catch { - return c.json({ exists: false }); + return c.json({ exists: false, resolved: dirPath }); } }); -- cgit v1.2.3