Skip to content

fix(telegram): preserve native Codex Ultra effort#48

Open
violetaini wants to merge 1 commit into
xiaotonng:mainfrom
violetaini:fix/telegram-codex-ultra-effort
Open

fix(telegram): preserve native Codex Ultra effort#48
violetaini wants to merge 1 commit into
xiaotonng:mainfrom
violetaini:fix/telegram-codex-ultra-effort

Conversation

@violetaini

Copy link
Copy Markdown

Fixes #46

Summary

  • route Telegram chat effort selection through the shared splitEffortForAgent logic
  • preserve native Codex max / ultra values instead of folding ultra to max
  • retain Claude's ultra -> max + workflow behavior
  • add a regression test covering runtime state and persisted codexReasoningEffort

Testing

  • npm test -- test/bot.unit.test.ts test/effort-label.unit.test.ts test/command-ui.unit.test.ts (28 passed)
  • npm run build

The full test suite was also attempted on Windows. Unrelated existing tests that assume Unix paths, symlinks, process signals, or locally installed agent CLIs fail in that environment; the affected bot suite passes both independently and during the full run.

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.

Telegram: selecting native Codex Ultra in /models is persisted as Max

1 participant