ci: enforce declared Python package compatibility - #259
Conversation
Move declared Python package compatibility into canonical Foundation ownership. Keep CPython 3.14 as the primary quality lane, add exact-head CPython 3.12/3.13 package compatibility on ubuntu-24.04, discover package contracts from pyproject metadata without package-name switchboards, retain package-owned 100% statement/branch coverage, and bind the compatibility test toolchain with reviewed hashes. Implements the source repair for #258 without closing it before protected integration.
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Distinguish a valid unsupported requires-python constraint from parser or metadata failure. Missing, blank, malformed, or invalid specifier metadata now fails the compatibility job instead of being converted into a package skip. Add executable Foundation hygiene coverage, update traceability, and reseal the exact Foundation workflow bytes.
|
Fresh central owner correction for the current CodeQL RCA: |
Finding
Protected
develop@eb9757f8649aaad026a9865508d9aad50c1a7a4fconsolidated repository-owned quality into Foundation CI, while merged Python packages already declare support below the primary CPython 3.14 runtime. Structured Interview Plan #40 is the concrete adopter: it declaresrequires-python = ">=3.12"and historically exercised 3.12/3.13/3.14. Deleting its retired package-local workflow without replacement evidence would weaken the declared runtime contract.Earlier heads repaired four distinct defects: metadata/parser failure being reclassified as an unsupported-package skip; patch-sensitive PEP 440 evaluation against fabricated
major.minor.0; owned packages escaping acceptance whensrc/ortests/disappeared; and a second compatibility matrix job that violated the protected one-job Foundation admission contract. Exact headf529086c...produced the real queue-shape RED in Foundation run34050838082.Exact head
79e8757515673144b68687517360cf493e93ccb8then produced Foundation GREEN in run34053906336: the singleRepository qualityjob passed exact checkout, runner-image proof, Foundation validation, dependency hygiene, primary package/service/PostgreSQL contracts, Python 3.12 compatibility, Python 3.13 compatibility, and clean-checkout proof.That GREEN exposed one remaining provenance finding on review:
.github/requirements/foundation-compatibility-test.txtwas hash-locked per dependency but its complete file bytes were not bound by the canonical manifest inventory. A reviewed compatibility dependency set could therefore change without changing the manifest unless a tracked artifact bound it.Current repair
Current successor
f1f152b0838e11cba1cf583706eb0983d56af373keeps the same one-job compatibility behavior and adds a fail-closed provenance check before compatibility installation:.github/workflows/foundation-ci.ymlremains exactly onequalityjob onubuntu-24.04;cebb36181e8ac995a36d73a02a45094a204ff5adb3cbcdc0c9eccff309ac6aab;--require-hashes --no-deps --only-binary=:all:;The final Foundation workflow is resealed in
manifest.jsonas SHA-25627fd3f2a69181e6a31b49ab2a3d394bfc8dca2b8ef5e9ce39324273f7962773d, 11,708 bytes, 251 lines. A manual manifest transcription error on an unrelated migration digest in intermediate commit581e1a3f...was immediately detected by exact diff review and corrected in normal fast-forward successordd89898a...; comparison against79e875...after correction shows only the intended Foundation workflow and its manifest entry changed. No migration source was modified.Traceability is synchronized at the current head and records the successful
34053906336predecessor evidence plus the provenance repair rationale.Exact-current-head authority
f1f152b0838e11cba1cf583706eb0983d56af373;develop@eb9757f8649aaad026a9865508d9aad50c1a7a4f;34057542130: terminal SUCCESS;34057542122: terminal SUCCESS;34057542151: terminal FAILURE only independency-review. Exact-head checkout/verification succeeded; Scorecard, OSV, and Trivy succeeded;Check dependency review supportfailed before the pinned Dependency Review action, which was skipped. Central owner.github#810records the current fail-closed GitHub dependency-comparison 403 incident. This is incomplete authoritative security evidence, not an Orgmetra source vulnerability and not permission for a local substitute or gate weakening;34057542155: terminal FAILURE. Language detection succeeded and both Python and Actions compatibility jobs successfully requested the current-head dispatch, then failed only atRelease runner or enforce current-head CodeQL verdict. Central owner.github#1925records the dispatched native-scan handler template defect;.github#1927records the subsequent dispatch-identity authorization mismatch that must be decided at the central security boundary. Do not synthesize a verdict or churn this source head;18156473still requires one approving review while naming no reviewer/code-owner/last-push authority..github#772owns that solo-maintainer governance deadlock; self/model approval and routine administrator bypass remain forbidden.Scope boundary
The HTTP services declare Python 3.11 support. This PR does not claim that service-runtime compatibility; #260 remains the separate successor after #258/#259 is normally integrated.
Acceptance
Keep Draft until the unchanged exact head has terminal authoritative central Security/CodeQL evidence and the then-live protected review governance is structurally satisfiable. Do not self-approve, use routine administrator bypass, manufacture a status, weaken a gate, create a no-op retrigger, or copy central workflow source into Orgmetra.
After protected integration, #40 and #42 can non-force adopt the canonical Foundation capability without restoring package-local workflow ownership.