FE-1089: Sync planning docs at orchestrator-cutover close#284
FE-1089: Sync planning docs at orchestrator-cutover close#284kostandinang wants to merge 1 commit into
Conversation
ln-sync at arc close: record the completed D101-L port cutover in SPEC (D101-L row, I56-L rewrite, requirement 26, ExecutionPorts lexicon row, worker content-quality blind spot), thin the five D101-L refinement blockquotes to src/executor/TOPOLOGY.md pointers, archive done frontier definitions to PLAN_HISTORY, close the orchestrator-cutover arc in PLAN, acknowledge in-flight FE-1116/FE-1115 (PRs #280/#281), reconcile executor topology headers, fix the stale I52-L comment to I56-L, and delete 11 exhausted scope cards.
PR SummaryLow Risk Overview
Removes 11 exhausted Reviewed by Cursor Bugbot for commit a83bbae. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |

Stack Context
Final PR of the executor cutover stack (#274 → #282). It updates only the planning documents to match what the stack actually built. No product behavior changes — #282 below it stays a pure feature diff.
What?
memory/SPEC.md— the spec still described real execution as "inactive until the real-execution stack lands". That stack has now landed, so the executor decision (D101-L) and invariant (I56-L) are rewritten to describe reality: allexecute_*tools are active in CODE mode, real git/subprocess/agent work enters only through the injectedExecutionPorts, and host changes are file-only behind an accepted commit SHA. Five per-PR implementation notes were collapsed into a pointer tosrc/executor/TOPOLOGY.md, which owns that detail. Also adds anExecutionPortsglossary entry and names a known gap: the agent-runner tests prove wiring, not real-model output quality.memory/PLAN.md— marks the orchestrator-cutover arc done, moves the six finished work-item definitions todocs/archive/PLAN_HISTORY.md, and records that FE-1116 (FE-1116: Session-local elicitation gaps from a graph-derived seed #280) and FE-1115 (Add standalone .pi/components preview harness #281) are in flight on parallel branches, so their own PLAN/SPEC edits stay authoritative when merges meet.memory/cards/— deletes 11 scope cards whose work is finished.src/executor/TOPOLOGY.md, and a code comment inactive-tools.tsthat cited the wrong invariant id (I52-L → I56-L).Why?
The planning docs lagged the code: they described the executor as mostly inactive and listed six finished work items as live. Syncing them now gives the next piece of work (executor observability) accurate docs to plan against, and keeping the cleanup out of #282 keeps that PR reviewable on its own.
Verification
npm run verify— 1156 tests passed / 1 skipped; build and markdown-link checks green.