Repository navigation
fix(memory): avoid unnecessary input and exhausted provider calls - #93
Merged
Merged
Conversation
|
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.
Automatic memory extracts cited facts from a supplied exchange. Codex also reads the checkout's instruction files on every extraction, consuming input unrelated to that job. The automatic extractor could additionally start a provider whose usage snapshot already showed that the selected allowance was exhausted.
Exclude checkout instruction files for the Codex extraction operation while keeping the real working directory, provider settings, and selected model. Check the selected provider's model-specific allowance before launching an automatic extraction. A known reset re-enables subsequent jobs. Ordinary chat and checkout-aware generation such as PR writing still receive their usual context.
Verified with the installed Codex CLI's offline prompt renderer. A synthetic checkout instruction marker appeared 340 times before the change and zero times afterward. The regression test failed before the fix. The provider tests also cover exhausted windows with and without a reset time and an elapsed reset. Offline
vp run check:changed origin/mainpassed, including 41 tests and three affected package typechecks.GPT-6 family with a Sol reviewer, Codex desktop.