fix(ci): decouple current GeoJSON receipt validation - #2107
Merged
bartytime4life merged 3 commits intoAug 7, 2026
Conversation
4 tasks
bartytime4life
marked this pull request as ready for review
August 7, 2026 14:01
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
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.
Goal
Repair the post-merge
spec-hashreceipt gate for the structural GeoJSON digest implementation merged through PR #2099, without rewriting its historical authoring receipt.Failure record
The PR #2099
spec-hashrun completed dependency installation and the deterministic no-network test step successfully. The final receipt step failed because the workflow also revalidated the pre-existing historicalgenrec-spec-hash-validator-20260806.jsonagainst the workflow's newly changed current bytes. That receipt accurately describes its earlier artifact state and should remain immutable; using it as a perpetual current-state assertion created stale receipt coupling.During this follow-up, the first exact-head attempts also exposed an over-broad receipt artifact set and a one-character workflow job-name byte mismatch (
validate-commmon-spec-hash). Those were corrected before the final exact-head run.Base and exact head
mainat8ee05f3738c29344d055025bf47448150429e05ec220b776e9bde12ba1fc15c109e8069fd8b43260Changes
data/receipts/generated/genrec-geojson-feature-digests-ci-repair-20260807.jsonas a current follow-up receipt..github/workflows/spec-hash.ymlto validate the new current-state receipt instead of treating the unrelated historical receipt as a mutable assertion about current workflow bytes.tools/spec_hash/GEOJSON_FEATURE_DIGESTS.mdso its status reflects that PR feat(hashing): add deterministic GeoJSON feature digests #2099 is merged rather than pending on a feature branch.Directory Rules basis
This repair stays within existing responsibility roots:
.github/workflows/owns GitHub Actions orchestration.data/receipts/generated/owns immutable AI-authoring process memory.tools/spec_hash/owns operator-facing documentation for the existing hashing tool.No new root or parallel schema, contract, policy, source, registry, proof, release, or publication authority is created.
Exact-head validation
PASS —
spec-hashrun 31147952118At exact head
c220b776e9bde12ba1fc15c109e8069fd8b43260:Job:
validate-common-spec-hash, job ID92771333160.PENDING / outside the bounded repair
Trust boundary
This change repairs CI process-memory validation only. It does not alter GeoJSON digest semantics, reproject or validate topology, admit a source, resolve evidence, evaluate policy, approve review, promote lifecycle state, release, deploy, publish, or create public-use authority.
Commits
e41e1f655d809028190341885ec1241d652e835a— initial historical-receipt decouplingde0850c99f6cf4df578d70f04e564341e8f3722f— narrow receipt to follow-up-authored artifactsc220b776e9bde12ba1fc15c109e8069fd8b43260— align workflow bytes and correct job-name typoRollback
Close this draft and delete the branch before merge, or revert the eventual merge/squash commit after an authorized merge. Rollback restores the prior workflow coupling and stale documentation but requires no source deactivation, data migration, release withdrawal, cache invalidation, or public correction.