fix(todos): settle durable Next Action on completion - #3414
Merged
huangruiteng merged 1 commit intoAug 21, 2026
Conversation
huangruiteng
force-pushed
the
codex/todo-next-action-settlement-main-refresh-20260821
branch
from
August 21, 2026 14:08
b6b08c6 to
bba3f19
Compare
Owner
Author
|
No actionable findings. Open questions: none. Product/architecture judgment:
Latest-main reconciliation:
Validation performed:
Merge decision: approved for owner-authorized self-merge after the refreshed GitHub checks, including the native Windows lifecycle job, pass. |
huangruiteng
force-pushed
the
codex/todo-next-action-settlement-main-refresh-20260821
branch
3 times, most recently
from
August 21, 2026 14:37
ec874d3 to
d01bdb2
Compare
Signed-off-by: huangruiteng <huangrt01@163.com>
huangruiteng
force-pushed
the
codex/todo-next-action-settlement-main-refresh-20260821
branch
from
August 21, 2026 14:47
d01bdb2 to
c83f659
Compare
huangruiteng
deleted the
codex/todo-next-action-settlement-main-refresh-20260821
branch
August 21, 2026 14:57
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Problem and correctness model
A completed Todo could remain projected as the durable Next Action. That left lifecycle truth and routing truth contradictory, so consumers could continue steering toward already-completed work.
The fix treats Todo state as the source of truth and Next Action as an owned projection. System-generated projections carry a stable
todo_id; legacy state may migrate only when its single visible Next Action is exactly equal to the completed Todo text. Unbound owner-authored text is never rewritten. A successor completion does not reproject until every declared successor identity is materialized.The latest main-line TypeScript runtime refactor also exposed a Windows lifecycle bug. That bug is now fixed upstream by #3423; this branch is rebased on that fix and does not carry a duplicate implementation.
Architecture
loopx/control_plane/todos/next_action.tsis the single semantic owner for binding, typed decoding, completion fencing, route matching, and next-open-advancement selection.todo.next_action.transitionhandler. No new daemon or generic command executor is introduced.complete_goal_todoorsupersede_goal_todo, after all Todo mutations are present, so Todo and Next Action still commit through one file write.Validation
origin/mainatad77a0487185aaf3fa26e04a6772a427053e0770, including the upstream Windows runtime repair in fix(runtime): probe Windows Effect process without signals #3423npm run typecheck:control-planenpm run test:control-plane— 25 passedpython -m mypygit diff --check origin/main...HEADBoundary
This changes the built-in Todo completion and Next Action projection path. It does not launch benchmark jobs or change scoring, task semantics outside this projection, permissions, submissions, credentials, or third-party writes. No private evidence, local paths, raw logs, or generated artifacts are included.