Skip to content

fix: exclude sysprompt etc from context left % - #2446

Merged
ae-openai merged 3 commits into
mainfrom
ae/pct-4
Aug 19, 2025
Merged

fix: exclude sysprompt etc from context left %#2446
ae-openai merged 3 commits into
mainfrom
ae/pct-4

Conversation

@ae-openai

Copy link
Copy Markdown
Contributor
  • Prevents the % left indicator from immediately decrementing to ~97%.
  • Tested by prompting "hi" and noting it only decremented to 99%. And by adding a bunch of debug logs and observing numbers.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Codex Review: Here are some suggestions.

Reply with @codex fix comments to fix any unresolved comments.

About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request that is ready for review, or mark a draft as ready for review. You can also ask for a review by commenting "@codex review".

Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".

Comment thread codex-rs/tui/src/bottom_pane/chat_composer.rs Outdated
Comment thread codex-rs/tui/src/bottom_pane/chat_composer.rs
Comment thread codex-rs/tui/src/bottom_pane/chat_composer.rs
@ae-openai
ae-openai merged commit 096bca2 into main Aug 19, 2025
15 checks passed
@ae-openai
ae-openai deleted the ae/pct-4 branch August 19, 2025 15:20
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 19, 2025
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