feat: add durable Activity history and print compatibility - #546
feat: add durable Activity history and print compatibility#546lidge-jun wants to merge 38 commits into
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Activity history now survives reconnects and server restarts through a bounded journal in the existing trace store. Trace admission captures the owning chat and execution scope; replay and raw reads enforce that ownership, including historical backfills and forked pointers.
Print runtimes observe accepted message/tool updates while the existing lifecycle continues to select and deliver final answers. Journal, link, and observer failures preserve successful message writes and existing send/retry behavior. Display-only settings default to Activity and retain explicit legacy mode without execution side effects.
Native Cursor main now uses the same captured chat/scope ownership at its fifth trace admission. The published parent is integrated by normal merges, including its gateway behavior regression follow-up.
Validation: 792 focused tests across 60 explicitly selected files passed after cascade, plus the parent follow-up checks. Root/frontend typechecks and builds, assets, strict baseline and 487 documentation counts passed. Owned Node Cursor fixtures exercise real factory/protocol/pool/lifecycle -> SQLite journal -> loopback HTTP with multi-session on/off, exact-owner reads, copied-fork denial, fixed-through replay, and storage failure containment. Removing the owner fields makes the regression fail; restoring them passes. Independent Astra review found no remaining blockers.
Current head
43d05fc5bddd1eb941b5b1374effbc96c52eb262includes parentef1fb23bf125513fd9a595d46cded24c18107689. Its Tests and Postinstall Platform Checks both passed on this exact head. Final-head regression: 54 tests, typecheck and 487 documentation counts passed; broader 792 tests plus 24 parent follow-up checks observed the identical production tree. Earlier CI remains historical. The existing instance authentication policy remains unchanged.Stack: this backend layer targets #542 (
codex/native-activity-02); dependent Web/TUI consumers reuse the isolated owner patch and fixture commit. No PR merge, release or deployment.