Skip to content

Conversation

@jif-oai
Copy link
Collaborator

@jif-oai jif-oai commented Feb 3, 2026

Summary

  • add Cursor/ThreadsPage conversions so state DB listings can be mapped back into the rollout list model
  • make recorder list helpers query the state DB first (archived flag included) and only fall back to file traversal if needed, along with populating head bytes lazily
  • add extensive tests to ensure the DB path is honored for active and archived threads and that the fallback works

Testing

  • Not run (not requested)
Screenshot 2026-02-03 at 20 42 33

@jif-oai
Copy link
Collaborator Author

jif-oai commented Feb 3, 2026

@codex review

Copy link
Contributor

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 42ef7c4779

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

# External (non-OpenAI) Pull Request Requirements

Before opening this Pull Request, please read the dedicated
"Contributing" markdown file or your PR may be closed:
https://github.com/openai/codex/blob/main/docs/contributing.md

If your PR conforms to our contribution guidelines, replace this text
with a detailed and high quality description of your changes.

Include a link to a bug report or enhancement request.
Copy link
Collaborator

@aibrahim-oai aibrahim-oai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Should we add a todo to remove the fallback once things are stable?

@jif-oai
Copy link
Collaborator Author

jif-oai commented Feb 4, 2026

I have a Linear task

@jif-oai jif-oai merged commit 100eb6e into main Feb 4, 2026
32 checks passed
@jif-oai jif-oai deleted the jif/db-5 branch February 4, 2026 09:27
@github-actions github-actions bot locked and limited conversation to collaborators Feb 4, 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