Skip to content

Handle file change patch updates#188

Open
benbrandt wants to merge 1 commit into
mainfrom
patch-updated-event
Open

Handle file change patch updates#188
benbrandt wants to merge 1 commit into
mainfrom
patch-updated-event

Conversation

@benbrandt

Copy link
Copy Markdown
Member

No description provided.

@SteffenDE

Copy link
Copy Markdown
Contributor

I may be misremembering, but I think Codex flagged my usage of patchUpdated quoting the docs

item/fileChange/patchUpdated - when features.apply_patch_streaming_events is enabled, streams structured file-change snapshots parsed from the model-generated patch before it is executed.

with something about patchUpdated being sent before the item, so it could happen that we create a tool_call_update before the initial tool_call.

I tried to see if I can get Codex to actually emit a patchUpdated, but I did not see it to verify...

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.

2 participants