githubnext/chopinPublic

Let's compose rich plans together

Build reviewed plans through the shared local ACP connector #426

OpenMaggieAppletonwants to merge 0 commits intomainfromrefs/pull/426/head
  1. MA

    Builds on [#419](https://github.com/githubnext/chopin/pull/419). A teammate can review a settled plan’s tasks, choose their paired workspace, and select **Approve and build this plan**. Approval durably binds the reviewed document, graph and checkout; the shared ACP connector starts an isolated local session and claims the graph before prompting the orchestrator. The interface shows dependencies, task states, blockers, session identity and per-task PR links after refresh.

    The connector instructs the orchestrator to use sub-agents and produce small reviewable PRs. Its MCP credential fixes the document and run and exposes only implementation reads and lifecycle reports. Investigations and implementations share pairing, worktrees and workspace reservation. HTTP is selected when advertised by the ACP agent, with a stdio bridge otherwise; this handles the installed Copilot ACP version’s rejection of stdio MCP configuration.

    A real Copilot session read the approved graph and reported the blocker shown below through MCP. Scripted browser tests cover both transports, persisted blockers, two reported PRs and successful verification. Real coding, parallel sub-agent delivery and actual multi-PR creation remain unvalidated. Automatic connector startup, implementation-decision resolution/resume, a graphical DAG and browser PR-creation controls remain follow-up slices.

    [Discussion outline in Chopin](https://chopin.githubnext.com/documents/githubnext/chopin/build-plans-with-local-coding-agents) · [Setup and recovery](https://github.com/githubnext/chopin/blob/feat/local-launcher-tracer/docs/local-launcher.md)

    Validation: focused task/MCP/connector tests; workspace and E2E type checks; repository CI; production browser pairing/build tests; opt-in real Copilot ACP tracer. No PRs were merged.

    The earlier tracer screenshot below predates the Build-dialog review changes; it shows a real session’s saved blocker and its dependent task waiting.

    ![Real Copilot ACP session reporting a task blocker to Chopin](https://github.com/user-attachments/assets/43b64c49-5ea6-4f60-bc1d-74a860e78f46)

    ## Rebase and design-review follow-up Rebased onto `2182c95` after #419 merged. The branch now contains only the implementation increment and its follow-up fixes.

    - Move implementation review into the explicit **Build** dialog, preserving the ordinary document layout. - Replace permanent two-second polling with initial snapshots and committed change notifications. Editing locks remain current when the dialog is closed and after reconnect. - Add explicit, idempotent retry for failed startup and **Return plan for changes** for a stopped claimed run; no automatic replay. - Wake the connector after durable enqueue, reject duplicate approvals that silently change owner/checkout, and exit cleanly on intentional disconnect. - Local verification: 4,294 unit tests passed, 4 skipped; the formerly failing viewport/comment cases and focus-restoration case passed; both MCP transports, startup retry and stopped-run recovery passed; types and repository CI checks passed.

    A fresh deployed-preview report will follow. The live test will use the existing sandbox permissions and validate blocker/recovery behavior; real multi-PR creation remains outside that validation.

  2. The preview deployment for **chopin** failed. 🔴

    [Open Build Logs](https://next-appserver.tail49836a.ts.net/project/l9buznc9ua1if0vflxrjwajs/environment/lxqsvczv19eopxk29vfb46mj/application/hb85gdd9mwqqlfwcteflvpbj/deployment/koecmxebseoo8f4u7rnbrhh7) | [Open Application Logs](https://next-appserver.tail49836a.ts.net/project/l9buznc9ua1if0vflxrjwajs/environment/lxqsvczv19eopxk29vfb46mj/application/hb85gdd9mwqqlfwcteflvpbj/logs)

    Last updated at: 2026-10-09 16:29:48 CET

  3. Unknown GitHub usercommitted ac8e9e2ea12blast month on GitHub

  4. Unknown GitHub usercommitted aa81aa8283bflast month on GitHub

  5. Unknown GitHub usercommitted 05bf19a1928clast month on GitHub

  6. Unknown GitHub usercommitted bbdb41971fe7last month on GitHub

  7. Unknown GitHub usercommitted 706cd66f7beflast month on GitHub

  8. Unknown GitHub usercommitted 70a150759260last month on GitHub

  9. Unknown GitHub usercommitted 8ca86ff9345flast month on GitHub

  10. MaggieAppletonbase-ref-changedlast month on GitHub

  11. Unknown GitHub usercommitted d9e0bc1242b7last month on GitHub

  12. Unknown GitHub usercommitted 9453df44edeblast month on GitHub

  13. Unknown GitHub usercommitted cdd50149bb59last month on GitHub

  14. Krzysztof-Cieslakbase-ref-force-pushedlast month on GitHub

  15. Unknown GitHub usercommitted 4b874f2bb32blast month on GitHub

  16. MaggieAppletonhead-ref-force-pushed 4b874f2bb32blast month on GitHub

  17. MaggieAppletonadded-to-stacklast month on GitHub

  18. MaggieAppletonrenamed Launch local Codex implementations from reviewed task graphs to Build reviewed plans through the shared local ACP connectorlast month on GitHub

  19. Krzysztof-Cieslakautomatic-base-change-succeededlast month on GitHub

  20. Unknown GitHub usercommitted e3bf2970fadalast month on GitHub

  21. Unknown GitHub usercommitted 89b4b7fa283flast month on GitHub

  22. Unknown GitHub usercommitted a3f533642d28last month on GitHub

  23. Krzysztof-Cieslakhead-ref-force-pushed a3f533642d28last month on GitHub