Skip to content

rollout: coalesce thread updated_at touches#21367

Merged
jif-oai merged 2 commits into
mainfrom
jif/thread-updated-touch
May 6, 2026
Merged

rollout: coalesce thread updated_at touches#21367
jif-oai merged 2 commits into
mainfrom
jif/thread-updated-touch

Conversation

@jif-oai
Copy link
Copy Markdown
Collaborator

@jif-oai jif-oai commented May 6, 2026

Why

Metadata-irrelevant rollout events currently refresh threads.updated_at on every flush. That keeps thread recency accurate, but it also turns high-frequency agent output into unnecessary SQLite writes. Recency only needs to advance periodically during an active session, while the final suppressed touch still needs to be persisted before shutdown.

What changed

  • coalesce touch-only updated_at writes in the rollout writer, with a short production interval between persisted touches
  • retain the latest suppressed touch and flush it during shutdown so the thread is not left stale
  • extend rollout recorder coverage for coalesced touches, delayed refresh, shutdown flushing, and the existing missing-thread fallback path

Verification

  • Added regression coverage in rollout/src/recorder_tests.rs for coalescing and shutdown flushing behavior.

@jif-oai jif-oai changed the title nit: threads updated at touch rollout: coalesce thread updated_at touches May 6, 2026
Co-authored-by: Codex <noreply@openai.com>
@jif-oai jif-oai merged commit 0e821b3 into main May 6, 2026
26 checks passed
@jif-oai jif-oai deleted the jif/thread-updated-touch branch May 6, 2026 17:32
@github-actions github-actions Bot locked and limited conversation to collaborators May 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants