Skip to content

evidence(OMN-14498): rebind infra PR 2672 current head - #6181

Closed
jonahgabriel wants to merge 4 commits into
devfrom
codex/omn-14498-occ-pr2672-rehead
Closed

jonahgabriel wants to merge 4 commits into
devfrom
codex/omn-14498-occ-pr2672-rehead

Conversation

@jonahgabriel

@jonahgabriel jonahgabriel commented Aug 8, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Rebinds OMN-14498 central evidence for OmniNode-ai/omnibase_infra#2672 after the product branch moved from cd4966c0d to 923b7db1be for the Effect-Assertion Gate credential repair.

Evidence

  • Adds an append-only successor receipt for the live evidence(OMN-13100): OCC receipt — transport-mock baseline drained to zero #2672 current head/file set at 923b7db1beac9ffdf3dc0b5e8801e4359f4761eb.
  • Adds an append-only source-level seam proof receipt that supersedes the older OCC-validator item which executed in the wrong workspace.
  • Adds OCC#6181 self-bind receipt.
  • Local product proof: uv run pytest -q tests/unit/runtime/auto_wiring/test_result_applier_publish_failure_seam_omn14498.py -> 5 passed, 3 warnings.
  • Local OCC gates: pre-commit passed, receipt hardening passed, evidence-admissibility 76 passed, receipt gate for omnibase_infra#2672 PASS (3/10 PASS, 7 WARN, 0 BLOCK), receipt gate for onex_change_control#6181 PASS (3/10 PASS, 7 WARN, 0 BLOCK).

Evidence-Ticket: OMN-14498
Evidence-Head: e8137b0

Summary by CodeRabbit

  • Documentation
    • Updated verification records with refreshed evidence for the latest implementation and test coverage.
    • Added confirmation that required checks passed successfully across supported revisions.
    • Recorded corresponding product validation and companion change verification for improved auditability.

@coderabbitai

coderabbitai Bot commented Aug 8, 2026 •

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The change adds exact-head evidence for omnibase_infra PR #2672, command receipts for source and test verification, and an OCC companion binding for PR #6181.

Changes

OMN-14498 evidence

Layer / File(s) Summary
Contract evidence updates
contracts/OMN-14498.yaml
The contract records exact-head validation for commits 923b7db1 and d3316ce1, including files, handlers, tests, product proof, and companion PR #6181.
Upstream PR verification receipts
drift/dod_receipts/OMN-14498/dod-OmniNode-ai-omnibase_infra-pr-2672-923b7db1/command.yaml, drift/dod_receipts/OMN-14498/dod-OmniNode-ai-omnibase_infra-pr-2672-d3316ce1/command.yaml
The receipts record passing checks for PR state, branch, commit, exact file sets, command output, and execution metadata.
Product proof and companion binding
drift/dod_receipts/OMN-14498/dod-product-proof-pr-2672-923b7db1/command.yaml, drift/dod_receipts/OMN-14498/dod-product-proof-pr-2672-d3316ce1/command.yaml, drift/dod_receipts/OMN-14498/occ-self-bind-pr-6181/command.yaml
The receipts verify required handler symbols and regression tests at both commits, and record the passing self-binding check for PR #6181.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

Suggested labels: occ:machine-minted

🚥 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 summarizes the main change: rebinding OMN-14498 evidence to the current head of infrastructure PR 2672.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/omn-14498-occ-pr2672-rehead

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

@jonahgabriel
jonahgabriel enabled auto-merge (squash) August 8, 2026 07:28

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
drift/dod_receipts/OMN-14498/dod-OmniNode-ai-omnibase_infra-pr-2672-923b7db1/command.yaml (1)

1-29: 🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win

Use the required receipt locations and dual hash bindings.

Each receipt uses command.yaml and omits contract_sha256. The configured receipt gate requires <run_timestamp>.yaml, the raw-contract SHA-256, and the canonical per-entry SHA-256.

  • drift/dod_receipts/OMN-14498/dod-OmniNode-ai-omnibase_infra-pr-2672-923b7db1/command.yaml#L1-L29: Move the receipt to 2026-08-08T01:25:00Z.yaml and add the final raw-contract contract_sha256.
  • drift/dod_receipts/OMN-14498/dod-OmniNode-ai-omnibase_infra-pr-2672-d3316ce1/command.yaml#L1-L36: Move the receipt to 2026-08-08T04:55:00Z.yaml and add the final raw-contract contract_sha256.
  • drift/dod_receipts/OMN-14498/dod-product-proof-pr-2672-923b7db1/command.yaml#L1-L34: Move the receipt to 2026-08-08T01:28:00Z.yaml and add the final raw-contract contract_sha256.
  • drift/dod_receipts/OMN-14498/dod-product-proof-pr-2672-d3316ce1/command.yaml#L1-L32: Move the receipt to 2026-08-08T04:55:00Z.yaml and add the final raw-contract contract_sha256.
  • drift/dod_receipts/OMN-14498/occ-self-bind-pr-6181/command.yaml#L1-L19: Move the receipt to 2026-08-08T01:31:00Z.yaml and add the final raw-contract contract_sha256.

As per coding guidelines, DoD receipts must use the timestamped location and bind both contract hashes through omnibase_core.validation.validator_receipt_gate.

🤖 Prompt for 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.

In
`@drift/dod_receipts/OMN-14498/dod-OmniNode-ai-omnibase_infra-pr-2672-923b7db1/command.yaml`
around lines 1 - 29, Rename the receipts and add the final raw-contract
contract_sha256 while preserving the existing canonical contract_entry_sha256:
drift/dod_receipts/OMN-14498/dod-OmniNode-ai-omnibase_infra-pr-2672-923b7db1/command.yaml:1-29
to 2026-08-08T01:25:00Z.yaml;
drift/dod_receipts/OMN-14498/dod-OmniNode-ai-omnibase_infra-pr-2672-d3316ce1/command.yaml:1-36
to 2026-08-08T04:55:00Z.yaml;
drift/dod_receipts/OMN-14498/dod-product-proof-pr-2672-923b7db1/command.yaml:1-34
to 2026-08-08T01:28:00Z.yaml;
drift/dod_receipts/OMN-14498/dod-product-proof-pr-2672-d3316ce1/command.yaml:1-32
to 2026-08-08T04:55:00Z.yaml; and
drift/dod_receipts/OMN-14498/occ-self-bind-pr-6181/command.yaml:1-19 to
2026-08-08T01:31:00Z.yaml. Ensure every renamed receipt binds both hashes as
required by validator_receipt_gate.

Source: Coding guidelines

🤖 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.

Outside diff comments:
In
`@drift/dod_receipts/OMN-14498/dod-OmniNode-ai-omnibase_infra-pr-2672-923b7db1/command.yaml`:
- Around line 1-29: Rename the receipts and add the final raw-contract
contract_sha256 while preserving the existing canonical contract_entry_sha256:
drift/dod_receipts/OMN-14498/dod-OmniNode-ai-omnibase_infra-pr-2672-923b7db1/command.yaml:1-29
to 2026-08-08T01:25:00Z.yaml;
drift/dod_receipts/OMN-14498/dod-OmniNode-ai-omnibase_infra-pr-2672-d3316ce1/command.yaml:1-36
to 2026-08-08T04:55:00Z.yaml;
drift/dod_receipts/OMN-14498/dod-product-proof-pr-2672-923b7db1/command.yaml:1-34
to 2026-08-08T01:28:00Z.yaml;
drift/dod_receipts/OMN-14498/dod-product-proof-pr-2672-d3316ce1/command.yaml:1-32
to 2026-08-08T04:55:00Z.yaml; and
drift/dod_receipts/OMN-14498/occ-self-bind-pr-6181/command.yaml:1-19 to
2026-08-08T01:31:00Z.yaml. Ensure every renamed receipt binds both hashes as
required by validator_receipt_gate.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: daf44654-2d6c-43f9-bcf8-2878a91ea667

📥 Commits

Reviewing files that changed from the base of the PR and between 0da939e and 41743e5.

📒 Files selected for processing (6)
  • contracts/OMN-14498.yaml
  • drift/dod_receipts/OMN-14498/dod-OmniNode-ai-omnibase_infra-pr-2672-923b7db1/command.yaml
  • drift/dod_receipts/OMN-14498/dod-OmniNode-ai-omnibase_infra-pr-2672-d3316ce1/command.yaml
  • drift/dod_receipts/OMN-14498/dod-product-proof-pr-2672-923b7db1/command.yaml
  • drift/dod_receipts/OMN-14498/dod-product-proof-pr-2672-d3316ce1/command.yaml
  • drift/dod_receipts/OMN-14498/occ-self-bind-pr-6181/command.yaml

@jonahgabriel

Copy link
Copy Markdown
Contributor Author

Closed per operator-approved regenerate-class triage (2026-08-08 open-PR census, docs/tracking/2026-08-08-open-pr-drain-worklist.md @ 4679e53ae): many-red evidence append predating 2026-08-08 gate hardening. Regenerate against current dev; do not reopen this PR.

@jonahgabriel
jonahgabriel disabled auto-merge August 8, 2026 21:47
@jonahgabriel

Copy link
Copy Markdown
Contributor Author

Superseded by OCC#6266: appends a fresh rebind (dod-OmniNode-ai-omnibase_infra-pr-2672-rebind + occ-self-bind-pr-6266) on top of dev's already-merged #6139 entry, targeting infra PR #2672's current head. This PR stays closed per the 2026-08-08 regenerate-class triage; do not reopen.

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