Skip to content

contract(OMN-10865): bind infra PR 1591 receipt - #1003

Merged
jonahgabriel merged 3 commits into
mainfrom
jonah/omn-10958-omn-10865-pr1591-receipt
May 14, 2026
Merged

jonahgabriel merged 3 commits into
mainfrom
jonah/omn-10958-omn-10865-pr1591-receipt

Conversation

@jonahgabriel

@jonahgabriel jonahgabriel commented May 14, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Verification

  • uv run python -m onex_change_control.scripts.validate_yaml contracts/OMN-10865.yaml
  • receipt YAML parse assertion for drift/dod_receipts/OMN-10865/dod-omnibase-infra-pr-1591/command.yaml
  • contract check_value grep for dod-omnibase-infra-pr-1591

Downstream

Summary by CodeRabbit

  • Chores
    • Added generated verification receipts recording PASS results for multiple pull-request status checks and command probes, including timestamps, commit identifiers, runner/verifier metadata, and captured command outputs.
    • Updated evidence fingerprints for several existing receipts to ensure the audit trail and verification records remain consistent and traceable.

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 14, 2026 •

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 82be3eb2-b94f-40fb-896e-40193d9d9fab

📥 Commits

Reviewing files that changed from the base of the PR and between f6d8e5e and 818cc61.

📒 Files selected for processing (8)
  • drift/dod_receipts/OMN-10865/dod-arch-lint-fix/command.yaml
  • drift/dod_receipts/OMN-10865/dod-nodes-migrated/command.yaml
  • drift/dod_receipts/OMN-10865/dod-occ-pr-979/command.yaml
  • drift/dod_receipts/OMN-10865/dod-occ-pr/command.yaml
  • drift/dod_receipts/OMN-10865/dod-omnibase-infra-pr-1569/command.yaml
  • drift/dod_receipts/OMN-10865/dod-omnimarket-pr-614/command.yaml
  • drift/dod_receipts/OMN-10865/dod-omnimarket-pr/command.yaml
  • drift/dod_receipts/OMN-10865/dod-unit-tests/command.yaml
✅ Files skipped from review due to trivial changes (4)
  • drift/dod_receipts/OMN-10865/dod-omnibase-infra-pr-1569/command.yaml
  • drift/dod_receipts/OMN-10865/dod-nodes-migrated/command.yaml
  • drift/dod_receipts/OMN-10865/dod-omnimarket-pr/command.yaml
  • drift/dod_receipts/OMN-10865/dod-arch-lint-fix/command.yaml

📝 Walkthrough

Walkthrough

Adds two generated DoD receipts to ticket OMN-10865 and their corresponding drift evidence command records asserting PASS and containing PR-open marker text for omnibase_infra PR #1591 and OCC PR #1003; also updates contract_sha256 values in several existing receipt files.

Changes

OMN-10865 DoD receipts

Layer / File(s) Summary
DoD contract entry and evidence for omnibase_infra PR #1591
contracts/OMN-10865.yaml, drift/dod_receipts/OMN-10865/dod-omnibase-infra-pr-1591/command.yaml
Inserted a generated DoD receipt (dod-omnibase-infra-pr-1591) in the contract and added the corresponding command.yaml evidence file capturing the command check, probe output, run metadata, PASS status, actual_output, exit_code, pr_number, and branch.
Self-binding DoD evidence for OCC PR #1003
contracts/OMN-10865.yaml, drift/dod_receipts/OMN-10865/dod-occ-pr-1003/command.yaml
Inserted a generated self-binding DoD receipt (dod-occ-pr-1003) in the contract and added the command.yaml evidence file with the gh pr view probe output, run metadata, PASS status, actual_output, exit_code, pr_number, and branch.
Updated contract_sha256 in existing receipts
drift/dod_receipts/OMN-10865/*/command.yaml
Updated contract_sha256 fingerprints in multiple existing receipt command.yaml files (arch-lint-fix, nodes-migrated, occ-pr-979, occ-pr, omnibase-infra-pr-1569, omnimarket-pr-614, omnimarket-pr, unit-tests).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Poem

🐰 I hopped through YAML fields with cheer,
Two receipts recorded, tidy and clear,
PRs marked OPEN, status set to PASS,
Checks and hashes updated en masse,
A rabbit's nibble left — hop, hop, hooray!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: adding a DoD evidence entry binding the omnibase_infra PR 1591 receipt to contract OMN-10865.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jonah/omn-10958-omn-10865-pr1591-receipt

Comment @coderabbitai help to get the list of available commands and usage tips.

@jonahgabriel
jonahgabriel enabled auto-merge (squash) May 14, 2026 12:48
@jonahgabriel
jonahgabriel merged commit 7b6e353 into main May 14, 2026
29 checks passed
@jonahgabriel
jonahgabriel deleted the jonah/omn-10958-omn-10865-pr1591-receipt branch May 14, 2026 12:57
andywu42 pushed a commit to andywu42/onex_change_control that referenced this pull request Jun 10, 2026
…c PRs (OmniNode-ai#579)

* docs(OMN-10376): add DoD contracts and receipts for swarm pattern epic PRs

Adds 11 missing ticket contracts + dod receipts for omnibase_core PRs
blocked by the Receipt Gate verify/verify check:

- OMN-10347 (PR OmniNode-ai#986): harden receipt gate skip tokens
- OMN-10351 (PR OmniNode-ai#987): EnumFileZone
- OMN-10357 (PR OmniNode-ai#991): ModelCompletionVerifyResult
- OMN-10358 (PR OmniNode-ai#990): extract_identifiers
- OMN-10363 (PR OmniNode-ai#1001): static import graph
- OMN-10367 (PR OmniNode-ai#996): trajectory store
- OMN-10368 (PR OmniNode-ai#1000): PRM detectors
- OMN-10369 (PR OmniNode-ai#1002): PRM escalation
- OMN-10371 (PR OmniNode-ai#1003): EnumDiffSeverity + AST diff models
- OMN-10372 (PR OmniNode-ai#998): tree-sitter symbol extractor
- OMN-10381 (PR OmniNode-ai#992): dispatch eval models

All contracts validated against ModelTicketContract schema.

* docs(OMN-10376): add contracts for newly opened swarm pattern PRs

Adds 4 more missing ticket contracts + dod receipts for omnibase_core PRs
opened after the initial batch:

- OMN-10360 (PR OmniNode-ai#1005): verify() + receipt-gate hook
- OMN-10356 (PR OmniNode-ai#1006): path-filtered CI matrix docs-only short-circuit
- OMN-10375 (PR OmniNode-ai#1009): AST semantic diff CLI + PR risk labeler
- OMN-10365 (PR OmniNode-ai#1010): co-change dark matter CLI + JSON output

All contracts validated against ModelTicketContract schema.

* docs(OMN-10376): add contract and receipts for proof-of-life integration tests

* docs(OMN-10376): resolve OCC receipt review comments

* ci: kick CI Summary runner (GHA queue stall)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant