Skip to content

fix(agent): make Controller own canonical submit admission - #2439

Open
rv4no wants to merge 1 commit into
mainfrom
codex/refactor-submit-ledger
Open

fix(agent): make Controller own canonical submit admission#2439
rv4no wants to merge 1 commit into
mainfrom
codex/refactor-submit-ledger

Conversation

@rv4no

@rv4no rv4no commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Make Controller the sole owner of canonical user-message admission.
  • Add durable submit intent and provenance updates keyed by clientSubmitId, with provider/session/turn delivery facts kept separate from canonical content.
  • Separate composer intent, attachment preparation, and presentation state; add revision-based draft clearing and stale upload guards.
  • Preserve prompt.id as queue identity distinct from clientSubmitId.

Why

The Controller, Provider runtime, and Host could all replay the same canonical user message. This makes admission durable before provider dispatch and makes retries idempotent state updates instead of payload rewrites.

Tests

  • git commit pre-commit checks passed.
  • pnpm check:changed -- --push-ready passed all 25 lanes.
  • Targeted Agent GUI and Go tests passed during iteration.

Notes

  • This is the upstream contract change consumed by the companion TSH PR.
  • The branch intentionally remains behind main; no rebase was performed.

@cursor

cursor Bot commented Aug 17, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant