feat(mcp): coordinator agent with task visibility, pilot/co-pilot control, and expanded options#100

Open

brooksc wants to merge 116 commits into

johannesjo:mainfrom

brooksc:feature/orchestrator-control-v2

Commits on May 14, 2026

feat(remote): add coordinator task API routes to remote server

brooksc
and
claude
committed
chore: remove pre-commit hook

brooksc
and
claude
committed
Squash merge
brooksc
committed
fix(coordinator): four minor bug fixes — taskId guard, REST sentinel, diff truncation metadata, gitIsolation schema

brooksc
and
claude
committed
feat(coordinator): MCP backend improvements and coordinator core

brooksc
and
claude
committed
feat(ctrl): Take Control / Release Control for coordinator and sub-tasks

brooksc
and
claude
committed
docs: coordinator design document, TODOS, and known open issues

brooksc
and
claude
committed
test(tasks): add unit tests for MCP_TaskCreated IPC handler

brooksc
and
claude
committed
test(autofire): unit tests for controlledBy pause behavior in interval tick

brooksc
and
claude
committed
docs: add CONTRIBUTING.md
brooksc
committed
test(TerminalView): add unit tests for disableStdin behavior

brooksc
and
claude
committed
docs: fix merge_task description to say base branch instead of main

brooksc
and
claude
committed
fix: eliminate double coordinator preamble by removing lib version

brooksc
and
claude
committed
fix(mcp): add uncommitted-file marker to get_task_diff and review_and_merge_task

brooksc
and
claude
committed
fix(coordinator): throw on auto-commit failure with uncommitted changes

brooksc
and
claude
committed
fix: reflect needsReview flag in task attention state and dot status

brooksc
and
claude
committed
fix(SubTaskStrip): include collapsed sub-tasks in coordinator strip

brooksc
and
claude
committed
fix(coordinator): restore modified preamble files on createTask failure

brooksc
and
claude
committed
fix(tasks): disallow collapsing coordinator tasks

brooksc
and
claude
committed
fix(projects): close coordinator children before coordinator in removeProjectWithTasks

brooksc
and
claude
committed
fix(coordinator): strip preamble files before auto-commit in mergeTask()

brooksc
and
claude
committed
chore: remove completed TODO items, keep remaining edge cases and hard items
brooksc
committed
feat(coordinator): sliding-window preamble, max concurrent UI, and sub-task preamble cleanup

brooksc
and
claude
committed
fix(coordinator): preserve pre-existing empty preamble files on strip
brooksc
committed
fix: update review_and_merge_task description to use get_task_diff->merge_task->close_task flow
brooksc
committed
fix: reject gitIsolation in REST API instead of silently ignoring it
brooksc
committed
fix: split MCP_CoordinatorOrphanedNotification channel
brooksc
committed
fix: hydrate backend Coordinator.tasks on app restart
brooksc
committed
fix: correct sidebar drag index translation with coordinator children
brooksc
committed
fix: restore backend controlMap on app restart
brooksc
committed
fix: uncollapse coordinator when clicking collapsed child in sidebar
brooksc
committed
chore: clean up TODOS.md — all actionable items resolved

brooksc
and
claude
committed
chore: revert README.md changes and remove CONTRIBUTING.md

brooksc
and
claude
committed
fix(coordinator): address five lifecycle/correctness issues from review

brooksc
and
claude
committed
fix(mcp): two lifecycle gaps vs test plan + regression tests

brooksc
and
claude
committed
docs: add three test-plan coverage gaps to TODOS

brooksc
and
claude
committed
fix(docker-coordinator): mount main .git dir + bind MCP server to 0.0.0.0

brooksc
and
claude
committed
chore: mark TODO #10 complete (tmp config cleanup test added)
brooksc
committed
Squash merge
brooksc
committed
Squash merge
brooksc
committed
Squash merge
brooksc
committed
chore: mark TODO #9 complete (edit suppression tests landed)
brooksc
committed
chore: mark TODO #12 complete (selectMcpJsonDir test added)
brooksc
committed
chore: mark TODO #26 complete (MCP status reporting expanded)
brooksc
committed
chore: mark TODO #24 complete (agent def derived from agentCommand)
brooksc
committed
chore: merge task/todo-26-mcp-status-reporting — expand GetMCPStatus response (TODO #26)

brooksc
and
claude
committed
chore: merge task/todo-12-selectmcpjsondir-test — add unit tests for selectMcpJsonDir (TODO #12)
brooksc
committed
chore: mark TODO #22 complete (startRemoteServer awaits listen)
brooksc
committed
feat: await server listen before resolving startRemoteServer
brooksc
committed
chore: mark TODO #27 complete (deregistration cleans child backend resources)
brooksc
committed
chore: mark TODO #21 complete (detectStaleDockerMCPUrl called on startup)
brooksc
committed
fix: clean up child backend resources on coordinator deregistration (TODO #27)
brooksc
committed
feat: call detectStaleDockerMCPUrl on MCP server startup
brooksc
committed
fix: persist initialPrompt across app restarts (TODO #29)
brooksc
committed
chore: mark TODO #19 complete (Docker sub-tasks get isolated HOME dirs)
brooksc
committed
fix: isolate Docker sub-task HOME dirs to prevent config collisions (TODO #19)

brooksc
and
claude
committed
chore: mark TODO #20 complete (.claude.json writes serialized)
brooksc
committed
chore: mark TODO #28 complete (setTaskControl awaits IPC before committing UI)
brooksc
committed
fix: setTaskControl awaits IPC before updating UI state
brooksc
committed
chore: mark TODO #17 complete (MCP token passed via env var)
brooksc
committed
fix: pass MCP token via env var instead of CLI arg
brooksc
committed
chore: mark TODO #13 complete (tasks scoped to coordinator ID)
brooksc
committed
fix: scope MCP tool handlers to coordinator ID to prevent cross-contamination
brooksc
committed
chore: mark TODO #14 complete (coordinator routes registered lazily)
brooksc
committed
fix: coordinator routes use lazy getCoordinator() lookup
brooksc
committed
chore: mark TODO #15 complete (await StartMCPServer + hydrateTask restores signal state)
brooksc
committed
fix: await StartMCPServer before respawn; hydrateTask restores signalDoneAt
brooksc
committed
chore: mark TODO #25 complete (coordinator routes always registered lazily)
brooksc
committed
fix(remote): register coordinator routes lazily so remote-first start works

brooksc
and
claude
committed
chore: mark TODO #16 complete (MCP_CoordinatedTaskClosed IPC cleans backend)
brooksc
committed
fix: IPC notifies coordinator backend when coordinated child is closed from UI
brooksc
committed
fix: kill inner docker exec agent process on sub-task cleanup
brooksc
committed
chore: mark TODO #18 complete (docker exec inner process cleanup)
brooksc
committed
fix: block collapseTask for coordinated children to preserve agent identity
brooksc
committed
chore: mark TODO #23 complete (collapse blocked for coordinated children)
brooksc
committed
chore: remove TODO #13 and #14 sections (already landed)
brooksc
committed
fix(git): guard getWorktreeStatus against deleted worktree path

brooksc
and
claude
committed
fix(autofire): reset miss counter when agent produces output during tool call

brooksc
and
claude
committed
docs: add TODO #31 — one container per Docker sub-task

brooksc
and
claude
committed
fix(coordinator): three post-review bug fixes + TODOS additions

brooksc
and
claude
committed
fix: three post-review bug fixes (controlMap leak, TOCTOU port race, silent stop)

brooksc
and
claude
committed
fix: four post-review bug fixes (mcpServerInfo isolation, sentinel error, gitdir walk, removed reason)

brooksc
and
claude
committed
fix: stale MCP config on restart and Docker cleanup kills sibling agents

brooksc
and
claude
committed
fix: four lifecycle and cleanup bugs from review

brooksc
and
claude
committed
fix: branch restore regression, wrong task diff base, stale batch IDs

brooksc
and
claude
committed
chore: update package-lock.json to sync with package.json

brooksc
and
claude
committed
fix(security): write MCP config files with mode 0o600

brooksc
and
claude
committed
fix(coordinator): resolve signal/idle waiters on exit, persist preambleFileExistedBefore, use git add -u

brooksc
and
claude
committed
fix: signal_done ownership check, cleanupTask resolves signal waiters, readBody 413

brooksc
and
claude
committed
fix(coordinator): revert git add -u back to git add -A in mergeTask auto-commit

brooksc
and
claude
committed
fix: deregisterCoordinator resolves signal waiters; validate mcpConfigPath on hydrate

brooksc
and
claude
committed
fix(security): two-class token, mcpReady gate, propagateSkipPermissions, input validation

brooksc
and
claude
committed
docs(coordinator): clarify PR 100 follow-up TODOs
brooksc
committed
fix: restore MAX_DIFF_BYTES truncation, complete mcpReady finally, clear pendingStop on re-start

brooksc
and
claude
committed
fix(#43): surface MCP restore failures with error state and retry UI
brooksc
committed
test: add StartMCPServer input validation tests (TODO #35)

brooksc
and
claude
committed
test(coordinator): add missing tests for #33, #36, #39

brooksc
and
claude
committed
fix(coordinator): async preamble I/O with per-path write serialization (#32)
brooksc
committed
fix(coordinator): cleanup failure retained — MCP_TaskCleanupFailed, Docker kill awaited (#41)
brooksc
committed
feat(coordinator): Docker sub-tasks each run in their own container (#31)
brooksc
committed
chore: document TODO #44 (staged prompt visibility) and #45 (wait_for_signal_done retry on network error)
brooksc
committed
fix(P1): retry wait_for_signal_done on network error; preserve content after preamble block

brooksc
and
claude
committed
fix(P1): idempotent wait_for_signal_done retry; correct preamble diff base

brooksc
and
claude
committed
fix(security): shared validation, fail-closed JSON, staged prompt UI, tests for #34/#37/#40

brooksc
and
claude
committed
fix: validation parity, fail-closed state, and replay cache scoping

brooksc
and
claude
committed
fix: preamble fail-safe + restart race (coordinator/sub-task spawn deferral)

brooksc
and
claude
committed
refactor: async FS in detectPreambleFiles/stripPreamble, sequential git cache warm, temp file cleanup

brooksc
and
claude
committed
fix(security): mobile token, bind-address, route scoping, validator hardening

brooksc
and
claude
committed
refactor: extract atomic writes, replay cache, preamble module; coordinator lifecycle state machine

brooksc
and
claude
committed
chore: add static analysis tooling (Knip, dependency-cruiser, Semgrep, Gitleaks)

brooksc
and
claude
committed
fix(security): address round-2 review — token scoping, per-task done tokens, atomic EXDEV, hydration gating

brooksc
and
claude
committed
chore: remove internal working docs before merge

brooksc
and
claude
committed
test: fix navigation and focus tests after rebase

brooksc
and
claude
committed

Commits on May 15, 2026

fix(security): address round-3 review — UUID validation, race fix, fsync, branch hardening

brooksc
and
claude
committed
chore: remove ask-coordinator proposal.md from PR (keep as local working doc)

brooksc
and
claude
committed
fix(docker): tighten MCP bind address on Linux; warn on macOS when coordinator+Docker active

brooksc
and
claude
committed

Uh oh!

Commits

Commits on May 14, 2026

Commits on May 15, 2026