Repository navigation
Validate late research context refusals for upstream #912 - #17
Draft
cwschroeder wants to merge 1 commit into
Draft
cwschroeder wants to merge 1 commit into
cwschroeder wants to merge 1 commit into
Conversation
8 tasks done
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.
A streamed request can fit initially but exceed its window after Cortheon adds research results. At upstream ashhart#912 head
8f874288, that late context refusal becomes HTTP 400 with no SSE error, while initial context refusals remain streamed. This patch routes only unopenedRefused + context_lengththrough the existing context-error stream. Capacity errors keep HTTP 503.Candidate
393e5c7dcontains a two-window regression using the actual scripted Cortheon path. It fails before the fix and passes afterward in Debug and Fast. The CPU server target passes 378/378 tests; whole-tree formatting, whitespace and lean pass (517/517 against main, zero new). Astra reviewed the exact two-file fix without another material finding.This validation PR isolates those two files on composition
f622d935, which merges upstream ashhart#912 unchanged onto main6ee828c2, preserving authorship. The patch also applies to8f874288. Complete candidate CI is green at393e5c7d: the Linux No GPU workflow ranzig build test,test-kernel-contract,test-golden,fuzz-cortheonand lean (517/517, zero new). Whole-tree format and both app/mobile workflows also pass. The draft validation PR isolates these two files onf622d935; it is an adoption aid for this PR.The first PR desktop run failed in the unchanged
guest.test.tsoutput-limit case withOperation not permitted; its failed-job rerun passed without source changes. The same commit’s push desktop run and the unchanged composition base’s app workflow had passed already. Thirty focused base probes did not reproduce that error, so its cause remains undiagnosed. No device/model, performance or deployment qualification is claimed. Upstream review.