Repository navigation
Fix sparse baseline preflight for selected explicit inventories - #4631
Merged
Mister-Meeseeks merged 1 commit intoOct 22, 2026
Merged
Conversation
Preserve repository-relative manifest paths and select the baseline from the productive-helper certificate schema. Cover full and sparse checkouts with committed Git fixtures. Developed with substantial OpenAI Codex assistance.
This was referenced Oct 22, 2026
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.
With selected #4470 at main
6044f37b1f75a57f5fea6d17077b6985893062c1, runningpython3 -B scripts/research_baseline_sparse.pyfails withValueError: Invalid selected file: certificates/LICENSE. Its explicitrepository-files-sha256-v1inventory contains repository-relative paths, but the preflight treats them as relative to the manifest directory. It also directs productive-helper certificates to the historical balanced-transfer baseline.This focused change:
research_selected_baseline.pyfor its supported productive-helper schema, including when the certificate is sparsely omitted, and exposesnextin both preflight outcomes;Three new real-Git regression cases cover a complete explicit inventory, a non-cone sparse checkout with both metadata files omitted and successful materialization, and a productive certificate using the older package inventory format. All three failed before the implementation change.
Validation on macOS, Python 3.12.13 / Git 2.50.1:
python3.12 -B -m unittest discover -s tests -p 'test_research_baseline_sparse.py' -v: 8 passed.python3.12 -B -m unittest discover -s tests -p 'test_research_selected_baseline.py' -v: 13 passed.python3.12 -B -m unittest discover -s tests -p 'test_research_*.py' -v: 334 tests, 316 passed and 18 platform-dependent skips, in 239.227 seconds.certificates/LICENSE.git diff --checkpassed.python3 -B scripts/research_package.py --changed-since 6044f37b1f75a57f5fea6d17077b6985893062c1 --matrix-batch-size 256selects zero registered replay packages.Scope: this is a read-only onboarding preflight, not source-hash verification, a scientific replay or a new κ claim. The selected record, pinned baseline implementations and frozen evidence are unchanged. The full selected mathematical replay and native Windows cases were not run.
Task context:
portable-replay, priority revision 7, narrowed to selected-baseline onboarding compatibility. Started this fix from canonical main6044f37b1f75a57f5fea6d17077b6985893062c1; refreshed upstream tofbc9ea0249746acca4f504d4fda2b84e0e36cf0dbefore submission and checked its sole integration-receipt change. Refreshed overlap evidence covers 405 open PR bodies and 3,574 Discussion records; the 404-PR path snapshot found no matching change, and the newly opened #4629 was checked separately. Historical #376/#473 file inventories are incomplete and 5,824 sparse local notes were outside coverage, so this is not an exhaustive novelty claim.Credit: Douglas Colkitt and Claude Opus 5.5 for the existing preflight; Arham-Tahir64's #3102 sparse-omission contract; Mihir Srivastava with OpenAI Codex assistance for the productive-helper baseline. The selected #4470 supplier and all inherited notices retain their attribution. This contribution was prepared by lee-phys with substantial OpenAI Codex assistance under Apache-2.0.