Skip to content

fix: recover recorded reasoning effort from Codex thread settings - #54

Merged
capisoft-lib merged 1 commit into
mainfrom
codex/fix-reasoning-effort
Sep 28, 2026
Merged

capisoft-lib merged 1 commit into
mainfrom
codex/fix-reasoning-effort

Conversation

@capisoft-lib

Copy link
Copy Markdown
Owner

Summary

  • Read reasoning_effort from Codex thread_settings_applied events when turn_context.effort is absent.
  • Stop carrying a prior turn's effort into a turn without recorded effort.
  • Invalidate the analysis cache and add regressions for both cases.

Verification

  • npm run check with Node 24.19.0: 313 tests passed.
  • git diff --check: passed.

Data limit

The screenshot's Conversation cba6dc20 has no effort in its two call records, turn context, applied settings, or local Codex thread state. This change leaves its effort unknown rather than inventing a value. It improves cases where the applied settings event does record the effort.

@capisoft-lib
capisoft-lib merged commit 4ded258 into main Sep 28, 2026
2 checks passed
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.

1 participant