Repository navigation
evidence(OMN-12829): re-point C1 escalation receipts to dev-targeted PR #1126 - #2362
Conversation
|
Warning Review limit reached
More reviews will be available in 35 minutes and 31 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
📝 WalkthroughWalkthroughThis PR updates two Definition of Done receipt evidence files for task OMN-12829. The escalation suite evidence is refreshed with new commit metadata and working directory path. The market PR evidence switches from PR ChangesDefinition of Done Receipt Updates
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 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 unit tests (beta)
Comment |
…a4b33d The original C1 receipts bound to omnimarket PR #1124 and commit d627327e. #1124 merged into the B3 feature branch (not dev) via the stacked-PR auto-merge trap, so the receipt-gate eligibility check on the re-land PR #1126 failed with pr_ticket_mismatch (no PASS receipt bound to #1126 or its commit SHAs). Re-points dod-market-pr-merged-green (PR #1126, commit 96a4b33d, real gh pr view probe) and dod-escalation-suite-green (commit 96a4b33d) so the required verify/verify receipt gate on #1126 binds correctly. Contract unchanged (hash dc9ee91b preserved); receipts remain status=PASS with real probe_stdout and verifier != runner. Evidence-Source: 2a23a9c Evidence-Ticket: OMN-12829
94d30f8 to
2df952e
Compare
OMN-12829 — re-point C1 OCC receipts to the dev-targeted re-land PR
The original C1 escalation work (omnimarket PR #1124, commit
d627327e) was orphaned by the stacked-PR auto-merge trap: #1124 merged into the B3 feature branchjonah/omn-12828-cloud-endpoints-key-refs(not dev), so the escalation code never reachedorigin/dev. It is being re-landed via omnimarket PR #1126 (feat(OMN-12829)… re-land onto dev, commit96a4b33d, base dev).The existing OMN-12829 OCC receipts bound to PR #1124 /
d627327e, so the requiredverify / verifyreceipt gate on #1126 failed withreason: pr_ticket_mismatch("no PASS receipt for one or more tickets binds to PR #1126 or one of its commit SHAs").Change
Re-points two receipts so eligibility binds to the dev-targeted PR:
dod-market-pr-merged-green: now probesgh pr view 1126(real probe_stdout captured),commit_sha: 96a4b33d, branchjonah/omn-12829-escalation-to-dev.dod-escalation-suite-green:commit_share-pointed to96a4b33d(suite re-run green, 6 passed — real probe_stdout retained).The OMN-12829 contract is unchanged (hash
dc9ee91b…preserved, so allcontract_sha256fields stay valid). Both receipts remainstatus: PASSwith realprobe_stdoutandverifier != runner(claude-occ-verifier vs claude). Note:deploy-gateis not a required check for the omnimarket dev branch (required = CI Summary, reject-skip-token, verify/verify, node-drift-gate, Runtime Sweep), so no deploy receipt is added here.dod_evidence
uv run pytest tests/ -m "not kafka"→ 6539 passed (189s); mypy --strict clean; pre-commit greengh pr view 1126,pytest …test_escalation_omn12829.py→ 6 passed)Evidence-Source: 2a23a9c
Evidence-Ticket: OMN-12829
Summary by CodeRabbit