Track Codex native Responses sessions#44
Merged
StoneHanaMori merged 3 commits intoMay 27, 2026
Conversation
90be393 to
a287619
Compare
1f0ed7e to
26ba3aa
Compare
The native Responses API path (_responses_native_enabled) bypassed _handle_request entirely, skipping session turn recording, skill injection tracking, and snapshot uploads. Codex sessions through the proxy were invisible to the evolve pipeline. Adds _record_responses_turn to capture prompt/response text and injected_skills for Responses API requests, enabling the full session -> evolve -> skill generation loop for Codex users.
Parse native Responses streaming events before forwarding each chunk so session tracking is not lost if the client closes after response.completed. Reconstruct assistant output from response.output_text and response.output_item events because real Responses streams may omit output from the final response.completed payload. Update Codex integration to use the current split profile file at ~/.codex/skillclaw.config.toml while keeping ~/.codex/config.toml free of legacy [profiles.skillclaw] tables.
26ba3aa to
0347fcb
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Stack
Tests