fix(scripts): sync structured compat matrix (#35) + deterministic e2e venv install (#36)#37
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.
Summary
setuptools>=61backend.--allow-networkauthorization for build-dependency bootstrap and document/update the E2E CI invocation.Bug #35 — structured compatibility matrix drift
Root cause
scripts/update_orro_engine_lock.pyonly treateddocs/compatibility-matrix.mdas the compatibility matrix. It never loaded or wroterelease/compatibility-matrix.v0.json, and--self-testcopied/asserted only the Markdown file.RED
From an
origin/maintemporary fixture after a successful updater invocation with111...,222..., and333...:Fix and verification
depone-n-witnessd-nandorro-rc-locked-tripletwhile preserving all other structured metadata and boundary fields.ref_namevalues unless replacement refs are explicitly supplied; the issue's reproduction command therefore remains checker-valid.--self-testto copy/assert the structured matrix and runcheck_compatibility_matrix.py --metadata-onlyagainst the updated fixture.GREEN fixture result:
Bug #36 — fresh venv lacks setuptools backend
Root cause
The E2E smoke used
--no-build-isolationinside a new venv and relied onsystem_site_packages=Trueto exposesetuptools.build_meta. Fresh Python environments, including Homebrew Python 3.13, do not guarantee setuptools is present.RED (fresh venv)
Fix and verification
system_site_packages=False.pyproject.toml's declared backend requirement.--allow-network; pip may use its cache, otherwise a configured package index is required.Focused full-smoke evidence:
Validation
python3 scripts/update_orro_engine_lock.py --self-testpython3 scripts/check_compatibility_matrix.py --metadata-onlypython3 scripts/orro_e2e_smoke.py --self-test../witnessd, pinned../depone, and--allow-networkpython3 scripts/check_orro_repo_contract.pypython3 scripts/check_no_bidi_controls.pypython3 -m py_compile scripts/update_orro_engine_lock.py scripts/orro_e2e_smoke.py scripts/check_compatibility_matrix.pygit diff --check origin/main...HEADCloses #35.
Closes #36.