Repository navigation
evidence(OMN-16041): promotion receipt on OCC main for omnibase_infra#2744 (v0.38.5) - #6490
Closed
jonahgabriel wants to merge 2 commits into
Closed
jonahgabriel wants to merge 2 commits into
jonahgabriel wants to merge 2 commits into
Conversation
… main (v0.38.5 dev->main)
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
Contributor
Author
|
Superseded by #6491 — same commit content, head renamed to hotfix/* so this repo's main-target-guard admits it (it accepts only dev or hotfix/* against main). |
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.
OMN-16041 — promotion evidence on OCC
mainfor omnibase_infra#2744 (v0.38.5 dev→main)Net-new files only. This is the
main-side half of the evidence for theomnibase_infradev→main promotion OmniNode-ai/omnibase_infra#2744.Why this targets
mainand notdevThe receipt gate runs in
main-releasepolicy mode for any PR whose base ismain, and that mode rejects evidence that is not yet on OCCmain. Verbatim, from the promotion PR'sverify / verifyrun:The dev-side companion is open as #6489. OCC
devis 3457 commits ahead ofmain, so a dev merge does not reachmainin any useful timeframe — hence this directmainPR, matching the existing pattern for main-release evidence in this repo (#3595, #3732, #4394).Why the contract file here is not a copy of the dev-side one
contracts/OMN-16041.yamldoes not exist onmain. Copying the dev-side file wholesale failslint_contract_check_valuesonmain, because that file also carries three autobind-generated entries with hardcoded PR numbers (gh pr view 1989 ...,gh pr view 6463 ...) whichmain's linter rejects aslegacy-gh-pr. Those entries belong to the ticket's omniclaude-side work and have nothing to do with this promotion.So this file carries only the promotion entry. That is not a loss of binding:
contract_entry_sha256hashes the entry plus a pinned header subset (ticket_id,schema_version), not the whole file, so the entry hash is identical on both branches —sha256:7274b75d…. Onlycontract_sha256(a raw-byte digest) differs between the two, and each receipt copy carries the digest of the file it actually sits next to.The probe
Reads three files from
omnibase_infraat the exact promotion head3dd848d31857669592f528958b9fd1bd45a7c0b2viagh api .../contents/...?ref=<sha>— an immutable ref — and asserts:pyproject.tomldeclaresversion = "0.38.5"and pinsomnibase-core==0.46.8,omnibase-spi==0.23.1,omnibase-compat==0.5.6(all published; the currently-publishedomnibase-infra0.36.1 pinsomnibase-spi >=0.21,<0.22, which cannot resolve against the published 0.23.1 — that unresolvable pin chain is the defect this promotion exists to clear).os.environoros.getenv. Those two files were the promotion's only merge conflicts; this is the property that distinguishes the forward-port that was performed from the dev-wins resolution thatcheck-env-readsrejects.Fail-closed: a failed
gh apifetch exits non-zero via|| exit 1rather than leaving an empty file that would trivially satisfy the absence half.Recorded run — real output, exit 0:
lint_contract_check_values.py→ exit 0 on this branch. Hashes were recomputed after the repo'syamlfmthook reflowed the contract, so they bind the landed bytes and not a pre-format draft.