Repository navigation
Conversation
Contributor
Author
|
UK hub (Claude session, for Max): independent review round 2 (GPT-6.1 Sol) at
Held unmerged on purpose: this ships in the same release as the weekend Microcosm UK cut certification, which will stack on this branch. |
The note was a Subfleet job artifact, not part of the change. The hub keeps a copy outside the repository. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Oct 23, 2026
This branch has not been deployed
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.
Fixes #703
New generation processes can certify from a local release manifest when Hugging Face is unavailable.
POLICYENGINE_<COUNTRY>_DATA_MANIFEST_FILEalways selects the configured file, avoiding an outage timeout. With the setting unset, normal fetch, authentication, caching, offline fallback and certification behavior matches main for consistent bundles.Certification invariants
A local manifest certifies only when all three conditions hold:
composition/1/artifact/data_release_manifestanywhere in the TRO must agree on that SHA-256. Missing, malformed, conflicting or stale pins refuse; no environment digest is accepted.The artifact-consistency check also applies to fetched manifests. This closes the pre-existing inconsistent-bundle case there without changing consistent bundles. Certification and runtime validation share the existing release-relative diagnostics/evidence path normalization.
Local files are read and reverified on every access, including after a cached fetch. Successful local certification appends
_local_manifestto the existing basis. Packaged receipts record optionalruntime.data_certification_basis, preserving the published bundle certificate basis and existing digest domains. Unset receipts omit the annotation. The loader's callable facade retains typedcache_clearandcache_infocontrols.Validation
3,322 unique test cases passed, 5 skipped across 56 provenance/certification/disclosure modules and focused release/bootstrap checks. All 82 tests in
tests/test_local_data_manifest.pyand all 12 TRO-pin tests passed. The final affected manifest/certification run passed 280 tests, and the release/bootstrap-focused run passed 189 tests. The broad run's only failure was an old error-message assertion for a missing default artifact; it was updated for the earlier refusal and its complete 40-test module passed on rerun.Regression coverage includes a direct
@graphconflicting hash, an 80-example Hypothesis property over duplicate placement and nesting, and the reviewer's digest-matching manifest atadversary/elsewhere/elsewhere.h5under an unchanged bundled dataset URI. Local and fetched cases cover artifact identity/hash conflicts, revision rewriting, inherited artifacts and release-relative evidence paths. Existing exact-pair and unset main-snapshot comparisons remain.make format,make lint, focusedmypy --follow-imports=silent,git diff --check, and generated receipt schema checks pass. Each pytest invocation checked load1 ≤110 and available disk ≥55,000,000,000 bytes; no pytest workers were used.Deployment and downstream adoption
The G0 example exports the setting only after successful download, raw-byte verification and atomic replacement. Fetch and verify the manifest for the exact installed certified bundle whenever that release changes, then start a fresh process and retain its receipt. Release coordination has been removed from permanent public docs.
tests/fixtures/w4/execution_receipt.v1.schema.jsonwhen upgrading; its schema equality test requires the updated optional runtime field.pipeline/uk_replay_variants.py:730preserves the new setting during offline isolation, while:750requires exact runtime bundle equality. ClearPOLICYENGINE_*_DATA_MANIFEST_FILEduring that isolation.No package versions, dependency pins, datasets or certified releases changed. Leave this PR draft for hub review; do not merge automatically.