Skip to content

evidence(OMN-10409): record final manual merge sweep - #1846

Merged
jonahgabriel merged 1 commit into
devfrom
jonah/merge-sweep-final-integration
May 29, 2026
Merged

jonahgabriel merged 1 commit into
devfrom
jonah/merge-sweep-final-integration

Conversation

@jonahgabriel

@jonahgabriel jonahgabriel commented May 29, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • records final manual merge sweep artifact for 2026-05-29
  • includes post-all-PR runtime/container health, omnimarket CI, dashboard local proof, and dashboard PR check proof
  • scope: manual-merge-sweep-post-all-prs-dashboard-included

Verification

Summary by CodeRabbit

  • Chores
    • Added integration record capturing system health metrics, including runtime health, container health, and CI verification status for enhanced monitoring.

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 29, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

@jonahgabriel, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 43 minutes and 5 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 @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

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 configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 49e21206-55c1-4be0-abf3-aa52e01abc6a

📥 Commits

Reviewing files that changed from the base of the PR and between f684392 and bc46cc9.

📒 Files selected for processing (8)
  • contracts/OMN-10409.yaml
  • drift/dod_receipts/OMN-10409/dod-deploy/command.yaml
  • drift/dod_receipts/OMN-10409/dod-lint/command.yaml
  • drift/dod_receipts/OMN-10409/dod-occ-pr-1844/command.yaml
  • drift/dod_receipts/OMN-10409/dod-occ-pr-1845/command.yaml
  • drift/dod_receipts/OMN-10409/dod-occ-pr-1846/command.yaml
  • drift/dod_receipts/OMN-10409/dod-tests/command.yaml
  • drift/integration/2026-05-29.yaml
📝 Walkthrough

Walkthrough

A new integration record file is added for May 29, 2026, documenting a single-day ModelIntegrationRecord. The record includes artifact metadata, multiple health and CI check surfaces (runtime, container, GitHub checks, dashboard PR checks), and a list of referenced ticket identifiers.

Changes

Integration Record Addition

Layer / File(s) Summary
Daily integration record with health and CI metrics
drift/integration/2026-05-29.yaml
Integration record for 2026-05-29 defines artifact type/version, timestamp, scope, status, runtime/container health surfaces, GitHub CI and dashboard check results, and ticket references.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 A new record hops into the fold,
May twenty-nine, with health metrics bold,
Surfaces shine, the checks all align,
Tickets are linked, integration so fine! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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.
Title check ✅ Passed The title 'evidence(OMN-10409): record final manual merge sweep' accurately reflects the main change—recording a final manual merge sweep integration artifact for 2026-05-29 with verification evidence.

✏️ 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/merge-sweep-final-integration

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

@jonahgabriel jonahgabriel changed the title evidence: record final manual merge sweep evidence(merge-sweep): record final manual sweep May 29, 2026
@jonahgabriel jonahgabriel changed the title evidence(merge-sweep): record final manual sweep evidence(OMN-10409): record final manual merge sweep May 29, 2026

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

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/integration/2026-05-29.yaml`:
- Around line 1-7: The YAML payload declared as ModelIntegrationRecord does not
match the model contract: add the required top-level fields (e.g.,
schema_version, run_id, timestamp/metadata fields expected by
ModelIntegrationRecord) and transform the tickets section from a list of ticket
IDs into the expected probe-result objects (each ticket entry should contain the
probe-result fields the model expects such as id, status, results/details, and
any metadata), so that ModelIntegrationRecord.model_validate(...) will succeed;
locate the top-level artifact_type: ModelIntegrationRecord and the tickets list
in the diff and adjust those entries to match the contract shape.
- Line 9: The embedded health "response" value contains a truncated JSON string
(the object and its "details" section are not closed) which will break JSON
parsing; fix by replacing the partial blob with either (a) a valid complete JSON
string (close the nested objects and the outer string with the necessary "}"
characters) for the "response" key or (b) refactor the entry to use structured
YAML fields (e.g., response: and then a mapping with status, version, details: {
healthy: true, ... }) so parsers can consume the health data reliably; locate
the "response" field in the YAML and apply the chosen correction to fully close
the JSON or convert it to YAML mappings.
🪄 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: f4f39166-1b48-45bb-b192-c94753b26600

📥 Commits

Reviewing files that changed from the base of the PR and between 22f7539 and f684392.

📒 Files selected for processing (1)
  • drift/integration/2026-05-29.yaml

Comment thread drift/integration/2026-05-29.yaml
Comment thread drift/integration/2026-05-29.yaml Outdated
@jonahgabriel
jonahgabriel force-pushed the jonah/merge-sweep-final-integration branch 2 times, most recently from 5c62f3d to 75dda3a Compare May 29, 2026 07:19
@jonahgabriel
jonahgabriel force-pushed the jonah/merge-sweep-final-integration branch from 75dda3a to bc46cc9 Compare May 29, 2026 07:23
@jonahgabriel
jonahgabriel added this pull request to the merge queue May 29, 2026
Merged via the queue into dev with commit 9cba093 May 29, 2026
34 checks passed
@jonahgabriel
jonahgabriel deleted the jonah/merge-sweep-final-integration branch May 29, 2026 07:27
jonahgabriel added a commit that referenced this pull request Jul 3, 2026
…3548

occ-preflight (required on OCC dev) pulls in every ticket cited by the PR title
and touched receipt dirs, then requires a receipt bound to THIS OCC PR per ticket.
The OMN-13899 acceptance-case supersession file made OMN-13899 a cited ticket
with no receipt bindable to #3548 (its evidence binds #1846/#3539, and binding it
to #3548 would mean editing merged OMN-13899 files). Defer the OMN-13899
supersession landing to a dedicated follow-up OCC PR; the acceptance case is fully
implemented + proven locally (apply_supersessions -> merged #1846 citation) and
documented. Bind dod-occ-self to pr_number 3548 so OMN-13888 has a PR-bound
receipt. The receipt-hardening gate supersession-awareness stays (forward support
+ tests).
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