Repository navigation
Conversation
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
This branch has not been deployed
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.
Local context estimates could exhaust maintenance or recall allowance even when the provider could still serve the request. This change makes counts advisory, calibrates subsequent whole-request estimates from completed provider input usage, and preserves original/protected context when maintenance fails. Explicit provider input overflow can force bounded source-backed history recovery even when the local estimate fits; output truncation remains distinct.
Shared CodeAgent/model/context changes only. Native runtime integration follows in #4315 after this verified foundation. Saved model output budgets are unchanged.
Validation on 0241c3c (product revision b697eac; final test-only fixture correction):
Controlled overflow tests use an actual HTTP peer; hosted-model acceptance uses qwen3.8-flash. Full Nexent feature acceptance belongs to the stacked native branch and is still pending. Existing auxiliary summary billing-metric gaps and missing optional history HTTP500 are outside this change.
CI follow-up: the initial runner lost communication. Its retry exposed an eager model-package import in low-dependency runtime tests and mock serialization recursion. Moved the usage helper into neutral context_runtime and restricted structured serialization to Pydantic models. Reproduced then passed the affected 4 bootstrap and 59 history/observation tests; factory tests also pass. Rebuilt/redeployed corrected main image 5925767071ec and repeated actual Python/Agent execution, inspected in Phoenix trace 72caeea199cb3ff2ed2fe6b27a536aa4. Transport D3 65 cases and strict assets revalidated.
Final CI on 0241c3c: all configured checks pass, including unit tests, both image architectures, copyright and CodeQL. Codecov patch coverage 92.24806% (90% target), total 92.1%. The final correction updates test namespaces/imports only; product source matches the rebuilt and verified b697eac image.