taskplane 0.9.3 → 0.10.1

package/README.md

@@ -19,10 +19,12 @@ The taskplane dashboard runs on a local port on your system and gives you elegan
 ### Key Features
 
 - **Task Orchestrator** — Parallel multi-task execution using git worktrees for full filesystem isolation. Dependency-aware wave scheduling. Automated merges into a dedicated orch branch — your working branch stays stable until you choose to integrate.
-- **Task Runner** — What the Orchestrator uses for autonomous single-task execution. Worker agents run in fresh-context loops with STATUS.md as persistent memory. Every checkbox gets a git checkpoint. Cross-model reviewer agents catch what the worker agents missed.
-- **Web Dashboard** — Live browser-based monitoring via `taskplane dashboard`. SSE streaming, lane/task progress, wave visualization, batch history.
+- **Persistent Worker Context** — Workers handle all steps in a single context, auto-detecting the model's context window (1M for Claude 4.6 Opus, 200K for Bedrock). Only iterates on context overflow. Dramatic reduction in spawn count and token cost.
+- **Worker-Driven Inline Reviews** — Workers invoke a `review_step` tool at step boundaries. Reviewer agents spawn in tmux sessions with full telemetry. REVISE feedback is addressed inline without losing context.
+- **Supervisor Agent** — Conversational supervisor monitors batch progress, handles failures, and can invoke orchestrator commands autonomously (resume, integrate, pause, abort).
+- **Web Dashboard** — Live browser-based monitoring via `taskplane dashboard`. SSE streaming, lane/task progress, reviewer activity, merge telemetry, batch history.
 - **Structured Tasks** — PROMPT.md defines the mission, steps, and constraints. STATUS.md tracks progress. Agents follow the plan, not vibes.
-- **Checkpoint Discipline** — Every completed checkbox item triggers a git commit. Work is never lost, even if a worker crashes mid-task.
+- **Checkpoint Discipline** — Step boundary commits ensure work is never lost, even if a worker crashes mid-task.
 - **Cross-Model Review** — Reviewer agent uses a different model than the worker agent (highly recommended, not enforced). Independent quality gate before merge.
 
 ## Install
@@ -117,27 +119,17 @@ Inside the pi session:
 
 `/orch` with no arguments is the universal entry point — it detects your project state and activates the supervisor for guided interaction (onboarding, batch planning, health checks, or retrospective). The default scaffold includes two independent example tasks, so `/orch all` gives you an immediate orchestrator + dashboard experience.
 
-### 4. Optional: run one task directly
+### 4. Run a single task with isolation
 
-`/task` is still useful for single-task execution and focused debugging:
-
-```
-/task taskplane-tasks/EXAMPLE-001-hello-world/PROMPT.md
-/task-status
-```
-
-Important distinction:
-
-- `/task` runs in your **current branch/worktree**.
-- `/orch` runs tasks in **isolated worktrees** on a dedicated orch branch — your working branch is never touched until you integrate.
-
-Because workers checkpoint with git commits, `/task` can capture unrelated local edits if you're changing files in parallel. For safer isolation (even with one task), prefer:
+For a single task with full worktree isolation, dashboard, and reviews:
 
 ```text
 /orch taskplane-tasks/EXAMPLE-001-hello-world/PROMPT.md
 ```
 
-Orchestrator lanes execute tasks through task-runner under the hood, so `/task` and `/orch` share the same core task execution model.
+This uses the same orchestrator infrastructure as a full batch — isolated worktree, orch branch, supervisor, dashboard, inline reviews — but for just one task.
+
+> **Note:** The `/task` command still exists for direct single-task execution in the current branch, but `/orch` is recommended for all workflows. `/task` does not provide worktree isolation, dashboard, or inline reviews.
 
 ## Commands
 
@@ -200,9 +192,7 @@ Orchestrator lanes execute tasks through task-runner under the hood, so `/task`
           └─────────────┘
 ```
 
-**Single task** (`/task`): Worker iterates in fresh-context loops. STATUS.md is persistent memory. Each checkbox → git checkpoint. Reviewer validates on completion.
-
-**Parallel batch** (`/orch`): Tasks are sorted into dependency waves. Each wave runs in parallel across lanes (git worktrees). Completed lanes merge into a dedicated orch branch. When the batch completes, use `/orch-integrate` to bring the results into your working branch (or configure auto-integration).
+**How it works:** Tasks are sorted into dependency waves. Each wave runs in parallel across lanes (git worktrees). Workers handle all steps in a single context, calling `review_step` at step boundaries for inline reviews. Completed lanes merge into a dedicated orch branch. A supervisor agent monitors progress and can autonomously resume, integrate, or abort. When the batch completes, use `/orch-integrate` to bring the results into your working branch (or configure auto-integration).
 
 ## Documentation
 

package/dashboard/public/app.js

@@ -78,15 +78,17 @@ function tokenSummaryFromLaneState(ls) {
 /** Build compact telemetry badge HTML for retry/compaction indicators.
  *  Only shows badges when telemetry data has meaningful values.
  *  @param {object|null} tel - Telemetry data for a lane (from currentData.telemetry[prefix])
+ *  @param {boolean} [suppressRetry=false] - When true, hide the retrying badge
+ *    (used when reviewer is active — long tool calls trigger false retry signals)
  *  @returns {string} HTML string with badges, or "" if nothing to show
  */
-function telemetryBadgesHtml(tel) {
+function telemetryBadgesHtml(tel, suppressRetry) {
   if (!tel) return "";
   let badges = "";
-  if (tel.retryActive) {
+  if (tel.retryActive && !suppressRetry) {
     const err = tel.lastRetryError ? ` — ${tel.lastRetryError}` : "";
     badges += `<span class="telem-badge telem-retry-active" title="Retry in progress${escapeHtml(err)}">🔄 retrying</span>`;
-  } else if (tel.retries > 0) {
+  } else if (tel.retries > 0 && !suppressRetry) {
     badges += `<span class="telem-badge telem-retry" title="${tel.retries} auto-retry event(s)">🔄 ${tel.retries}</span>`;
   }
   if (tel.compactions > 0) {
@@ -529,7 +531,7 @@ function renderLanesTasks(batch, tmuxSessions) {
 
       // Worker stats from lane state sidecar + telemetry badges
       let workerHtml = "";
-      const telemBadges = task.status !== "pending" ? telemetryBadgesHtml(tel) : "";
+      const telemBadges = task.status !== "pending" ? telemetryBadgesHtml(tel, reviewerActive) : "";
       // Reviewer sub-row should only appear under the task currently being reviewed,
       // not all tasks in the lane. The lane-state sidecar is per-lane (shared by all
       // tasks in the lane), so check that the sidecar's current taskId matches this task.

package/extensions/taskplane/execution.ts

@@ -12,43 +12,81 @@ import type { AllocatedLane, AllocatedTask, DependencyGraph, LaneExecutionResult
 import { allocateLanes } from "./waves.ts";
 import { runGit } from "./git.ts";
 
-// ── Task Runner Extension Path Resolution ────────────────────────────
+// ── Taskplane Package File Resolution ────────────────────────────────
 
 /**
- * Find the task-runner extension path for lane sessions.
+ * Cached result of `npm root -g` to avoid repeated child process spawns.
+ * null = not yet resolved, "" = resolution failed.
+ */
+let _npmGlobalRoot: string | null = null;
+
+/**
+ * Get the global npm root directory via `npm root -g`.
+ * Result is cached for the process lifetime.
+ */
+function getNpmGlobalRoot(): string {
+	if (_npmGlobalRoot !== null) return _npmGlobalRoot;
+	try {
+		const result = spawnSync("npm", ["root", "-g"], {
+			encoding: "utf-8",
+			timeout: 5000,
+			shell: true,
+		});
+		_npmGlobalRoot = result.stdout?.trim() || "";
+	} catch {
+		_npmGlobalRoot = "";
+	}
+	return _npmGlobalRoot;
+}
+
+/**
+ * Resolve a file path within the taskplane package.
  *
  * Resolution order:
- *   1. Local project: {repoRoot}/extensions/task-runner.ts (for taskplane dev)
- *   2. Global npm (Windows): {APPDATA}/npm/node_modules/taskplane/extensions/task-runner.ts
- *   3. Global npm (Unix): /usr/local/lib/node_modules/taskplane/extensions/task-runner.ts
- *   4. npm peer: resolve from pi's location
- *
- * @throws ExecutionError if task-runner.ts cannot be found anywhere
+ *   1. Local project: {repoRoot}/{relPath} (for taskplane development)
+ *   2. `npm root -g` based: {npmGlobalRoot}/taskplane/{relPath}
+ *      (covers Homebrew, nvm, volta, pnpm, and any custom npm prefix)
+ *   3. Well-known global npm paths (Windows/macOS/Linux):
+ *      - {APPDATA}/npm/node_modules/taskplane/{relPath}
+ *      - {HOME}/.npm-global/lib/node_modules/taskplane/{relPath}
+ *      - /usr/local/lib/node_modules/taskplane/{relPath}
+ *      - /opt/homebrew/lib/node_modules/taskplane/{relPath}
+ *   4. Peer of pi's package: resolve from pi's binary location
+ *
+ * @param repoRoot - Absolute path to the project root
+ * @param relPath  - Relative path within the taskplane package (e.g., "bin/rpc-wrapper.mjs")
+ * @returns Absolute path to the resolved file
  */
-function resolveTaskRunnerExtensionPath(repoRoot: string): string {
-	const extFile = join("extensions", "task-runner.ts");
-
+function resolveTaskplanePackageFile(repoRoot: string, relPath: string): string {
 	// 1. Local project (taskplane development)
-	const localPath = join(resolve(repoRoot), extFile);
+	const localPath = join(resolve(repoRoot), relPath);
 	if (existsSync(localPath)) return localPath;
 
-	// 2. Global npm install paths
-	const home = process.env.HOME || process.env.USERPROFILE || "";
 	const candidates: string[] = [];
+
+	// 2. Dynamic: `npm root -g` (covers ALL npm setups: nvm, Homebrew, volta, etc.)
+	const npmRoot = getNpmGlobalRoot();
+	if (npmRoot) {
+		candidates.push(join(npmRoot, "taskplane", relPath));
+	}
+
+	// 3. Well-known static paths
+	const home = process.env.HOME || process.env.USERPROFILE || "";
 	if (process.env.APPDATA) {
-		candidates.push(join(process.env.APPDATA, "npm", "node_modules", "taskplane", extFile));
+		candidates.push(join(process.env.APPDATA, "npm", "node_modules", "taskplane", relPath));
 	}
 	if (home) {
-		candidates.push(join(home, "AppData", "Roaming", "npm", "node_modules", "taskplane", extFile));
-		candidates.push(join(home, ".npm-global", "lib", "node_modules", "taskplane", extFile));
+		candidates.push(join(home, "AppData", "Roaming", "npm", "node_modules", "taskplane", relPath));
+		candidates.push(join(home, ".npm-global", "lib", "node_modules", "taskplane", relPath));
 	}
-	candidates.push(join("/usr", "local", "lib", "node_modules", "taskplane", extFile));
+	candidates.push(join("/usr", "local", "lib", "node_modules", "taskplane", relPath));
+	candidates.push(join("/opt", "homebrew", "lib", "node_modules", "taskplane", relPath));
 
-	// 3. Peer of pi's package
+	// 4. Peer of pi's package
 	try {
 		const piPath = process.argv[1] || "";
 		const piPkgDir = resolve(piPath, "..", "..");
-		candidates.push(join(piPkgDir, "..", "taskplane", extFile));
+		candidates.push(join(piPkgDir, "..", "taskplane", relPath));
 	} catch { /* ignore */ }
 
 	for (const candidate of candidates) {
@@ -59,51 +97,24 @@ function resolveTaskRunnerExtensionPath(repoRoot: string): string {
 	return localPath;
 }
 
+// ── Task Runner Extension Path Resolution ────────────────────────────
+
+/**
+ * Find the task-runner extension path for lane sessions.
+ * @see resolveTaskplanePackageFile for resolution order
+ */
+function resolveTaskRunnerExtensionPath(repoRoot: string): string {
+	return resolveTaskplanePackageFile(repoRoot, join("extensions", "task-runner.ts"));
+}
+
 // ── RPC Wrapper Path Resolution ──────────────────────────────────────
 
 /**
  * Find the rpc-wrapper.mjs path for lane sessions.
- *
- * Resolution order mirrors resolveTaskRunnerExtensionPath:
- *   1. Local project: {repoRoot}/bin/rpc-wrapper.mjs (for taskplane dev)
- *   2. Global npm (Windows): {APPDATA}/npm/node_modules/taskplane/bin/rpc-wrapper.mjs
- *   3. Global npm (Unix): /usr/local/lib/node_modules/taskplane/bin/rpc-wrapper.mjs
- *   4. npm peer: resolve from pi's location
- *
- * @throws ExecutionError if rpc-wrapper.mjs cannot be found anywhere
+ * @see resolveTaskplanePackageFile for resolution order
  */
 export function resolveRpcWrapperPath(repoRoot: string): string {
-	const wrapperFile = join("bin", "rpc-wrapper.mjs");
-
-	// 1. Local project (taskplane development)
-	const localPath = join(resolve(repoRoot), wrapperFile);
-	if (existsSync(localPath)) return localPath;
-
-	// 2. Global npm install paths
-	const home = process.env.HOME || process.env.USERPROFILE || "";
-	const candidates: string[] = [];
-	if (process.env.APPDATA) {
-		candidates.push(join(process.env.APPDATA, "npm", "node_modules", "taskplane", wrapperFile));
-	}
-	if (home) {
-		candidates.push(join(home, "AppData", "Roaming", "npm", "node_modules", "taskplane", wrapperFile));
-		candidates.push(join(home, ".npm-global", "lib", "node_modules", "taskplane", wrapperFile));
-	}
-	candidates.push(join("/usr", "local", "lib", "node_modules", "taskplane", wrapperFile));
-
-	// 3. Peer of pi's package
-	try {
-		const piPath = process.argv[1] || "";
-		const piPkgDir = resolve(piPath, "..", "..");
-		candidates.push(join(piPkgDir, "..", "taskplane", wrapperFile));
-	} catch { /* ignore */ }
-
-	for (const candidate of candidates) {
-		if (existsSync(candidate)) return candidate;
-	}
-
-	// Fallback: return the local path (will fail at spawn time with a clear error)
-	return localPath;
+	return resolveTaskplanePackageFile(repoRoot, join("bin", "rpc-wrapper.mjs"));
 }
 
 // ── Telemetry Helpers ────────────────────────────────────────────────