Repository navigation
evidence(OMN-11423): hash-bind deploy-gate receipts - #1298
Conversation
📝 WalkthroughWalkthroughAdds ChangesOMN-11423 Evidence & Contract Updates
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
25dd692 to
cc19409
Compare
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@drift/dod_receipts/OMN-11423/dod-ci-surface/command.yaml`:
- Line 7: The contract_sha256 value is incorrect; update the contract_sha256
entry to the required hash by replacing
"sha256:6a13b912c1f2d84d59360e241a28faddde386e8e23cc1568dbfb21104f638d4f" with
the correct
"sha256:53cbe870e016338d948fcbbb346500fa71612d4afe8b7f73c484630036290c72" so the
receipt binds to the PR's required contract.
In `@drift/dod_receipts/OMN-11423/dod-omniclaude-pr-1627/command.yaml`:
- Line 7: Update the contract_sha256 value in the command.yaml to match the PR's
declared contract digest: replace the current contract_sha256
"sha256:6a13b912c1f2d84d59360e241a28faddde386e8e23cc1568dbfb21104f638d4f" with
the correct
"sha256:53cbe870e016338d948fcbbb346500fa71612d4afe8b7f73c484630036290c72" so the
receipt is bound to the correct contract digest.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 36f7d7bc-4491-4c5b-838e-2626890289b5
📒 Files selected for processing (5)
contracts/OMN-11423.yamldrift/dod_receipts/OMN-11423/dod-ci-surface/command.yamldrift/dod_receipts/OMN-11423/dod-deploy-gate-proof/command.yamldrift/dod_receipts/OMN-11423/dod-occ-self/command.yamldrift/dod_receipts/OMN-11423/dod-omniclaude-pr-1627/command.yaml
✅ Files skipped from review due to trivial changes (2)
- drift/dod_receipts/OMN-11423/dod-occ-self/command.yaml
- drift/dod_receipts/OMN-11423/dod-deploy-gate-proof/command.yaml
Summary
contract_sha256binding to all OMN-11423 DoD receipts.sha256:6a13b912c1f2d84d59360e241a28faddde386e8e23cc1568dbfb21104f638d4f, the current hash ofcontracts/OMN-11423.yamlafter binding the OCC self receipt to PR 1298.Verification
LC_ALL=C shasum -a 256 contracts/OMN-11423.yaml->6a13b912c1f2d84d59360e241a28faddde386e8e23cc1568dbfb21104f638d4fuv run validate-yaml contracts/OMN-11423.yamlCONTRACT_REPO_DIR=$PWD uv run python scripts/ci/run_contract_compliance_check.py --pr 1627 --repo OmniNode-ai/omniclaude --contracts-dir contracts->OMN-11423: 4/4 PASS, 0 WARN, 0 BLOCKEvidence-Ticket: OMN-11423