Repository navigation
Finish transfer/client fixes and close integration CI regressions - #179
Merged
LIghtJUNction merged 6 commits intoSep 29, 2026
Merged
LIghtJUNction merged 6 commits into
LIghtJUNction merged 6 commits into
Conversation
Bound ASCII decimal offsets before numeric storage/projection use, including sealed/download states. Preserve generated cursors and normal continuation. Cloud RED 36560714465 verified 11 cursor failures; correct two null-omission test assumptions with stricter per-page checks. Refs #78, #80, #83, #85.
Validate positive non-bool integer part_bytes at upload/download entry, before source hashing, journal writes or network requests. Preserve all existing protocol and journal formats. Cloud RED 36561926956: 14 intended failures, 9 passes, no errors/skips; real zero-byte reads do not advance the resume offset. Refs #80, #82, #83, #85.
LIghtJUNction
merged commit Sep 29, 2026
de3cd87
into
integrate/all-open-20260929
12 of 24 checks passed
4 tasks
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.
Refs #78, #80, #81, #82, #83, #85. Follow-up to #176, targeting its integration branch without overwriting it.
Incorporated histories
Preserves all commits from #177 (
769c124, bounded transfer cursors) and #178 (517f7e2, client chunk-size validation). The merge89990ebis a verified disjoint-path union, not a reimplementation. #177's focused 24 cases are green; #178's final focused verification is in progress. Their original RED/GREEN evidence remains in their notes and discussions.Full-suite blockers
Base
45470b5run 36559785404 has 2259 unique cases, four failures, no errors or skipped test cases. Independently downloaded all four artifacts, verified each ZIP hash and exact JUnit-node-ID partition union. The failures are one read-effect ordering case and three real PostgreSQL webhook-overlap cases, not regressions from #177/#178.This regression-first commit adds GET/HEAD tests requiring effect refusal before resource lookup or executor invocation, plus a focused workflow that preserves the exact public checkout for inspection. The next runtime corrections will move the discovery read-effect check before lookups and correct webhook literal-prefix deduplication while retaining legacy scope-key recognition and pinned current authorization. No fix/green claim yet for these two blockers.
Gate
Do not merge until final focused and complete four-shard node-ID, conformance, build and applicable specialized workflows all pass on the final tree. No skipped checks, swallowed database conflicts, removed unique constraints or production operations. CI workflow has contents:read only; no project runtime/tests run locally.
Progress:
docs/INTEGRATION_REGRESSION_PROGRESS.md,docs/TRANSFER_STATUS_PROGRESS.md,docs/CLIENT_PART_BYTES_PROGRESS.md.Once this combined follow-up is accepted, #177/#178 can be closed as incorporated rather than merged separately; broad acceptance issues remain open.