Skip to content

Conversation

@celia-oai
Copy link
Collaborator

@celia-oai celia-oai commented Jan 29, 2026

Add dynamic tools to rollout file for persistence & read from rollout on resume. Ran a real example and spotted the following in the rollout file:

{"timestamp":"2026-01-29T01:27:57.468Z","type":"session_meta","payload":{"id":"019c075d-3f0b-77e3-894e-c1c159b04b1e","timestamp":"2026-01-29T01:27:57.451Z","...."dynamic_tools":[{"name":"demo_tool","description":"Demo dynamic tool","inputSchema":{"additionalProperties":false,"properties":{"city":{"type":"string"}},"required":["city"],"type":"object"}}],"git":{"commit_hash":"ebc573f15c01b8af158e060cfedd401f043e9dfa","branch":"dev/cc/dynamic-tools","repository_url":"https://github.com/openai/codex.git"}}}

@celia-oai celia-oai changed the title draft [feat] persist dynamic tools in session rollout file Jan 29, 2026
@celia-oai celia-oai requested review from jif-oai and owenlin0 January 29, 2026 01:30
@celia-oai celia-oai marked this pull request as ready for review January 29, 2026 01:31
@celia-oai celia-oai force-pushed the dev/cc/dynamic-tools branch from ebc573f to dc66ebd Compare January 29, 2026 01:44
@owenlin0
Copy link
Collaborator

approving for now to get teddy unblocked, but makes sense to store in sqlite

Copy link

@teddywyly-oai teddywyly-oai left a comment

Choose a reason for hiding this comment

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

Thank you!

🔨

@celia-oai celia-oai force-pushed the dev/cc/dynamic-tools branch 2 times, most recently from 0782bfe to dc66ebd Compare January 30, 2026 00:45
@celia-oai celia-oai enabled auto-merge (squash) January 30, 2026 00:49
@celia-oai celia-oai merged commit 7151387 into main Jan 30, 2026
83 of 85 checks passed
@celia-oai celia-oai deleted the dev/cc/dynamic-tools branch January 30, 2026 01:10
@github-actions github-actions bot locked and limited conversation to collaborators Jan 30, 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.

4 participants