Repository navigation
docs(OMN-11513): 2026-06-04 sanctioned-live --agent evidence — first full PASS + OMN-12664 post-closure 404 observation - #2166
Merged
jonahgabriel merged 3 commits intoJun 4, 2026
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (13)
✅ Files skipped from review due to trivial changes (8)
📝 WalkthroughWalkthroughThis pull request extends the OMN-11513 evidence bundle with updated manifest references, a new integration test stability document, two 2026-06-04 live-agent runs (one pass, one forced-escalation), a deterministic asyncio repro test for OMN-12587, and multiple contract and drift receipt updates. All changes are documentation, evidence artifacts, or tests. ChangesOMN-11513 Evidence Collection
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| without a live broker. This is the same injection seam the shipped tests use. | ||
| """ | ||
|
|
||
| async def start(self) -> None: ... |
| """ | ||
|
|
||
| async def start(self) -> None: ... | ||
| async def close(self) -> None: ... |
| async def close(self) -> None: ... | ||
| async def arm_response( | ||
| self, *, response_topic: str, correlation_id: str | ||
| ) -> None: ... |
| ) -> None: ... | ||
| async def publish( | ||
| self, topic: str, key: bytes | None, value: bytes, headers: object = None | ||
| ) -> None: ... |
clone45
force-pushed
the
clone45/omn-11513-evidence-2026-06-04-agent-runs
branch
from
June 4, 2026 16:45
f32f913 to
bba4f12
Compare
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (1)
evidence/OMN-11513/proof-of-life/2026-06-04-93adf56-live-agent-forced-escalation/proof-packet.md (1)
36-42: ⚡ Quick winAdd language specifier to fenced code block.
The error message code block should specify a language identifier for proper syntax highlighting and documentation tooling compatibility.
📝 Suggested improvement
-``` +```text model registry drift: inference effect reported model-not-found for served_model_id 'gemini-2.5-flash' at https://generativelanguage.googleapis.com/v1beta/openai/chat/completions; error_message="Client error '404 Not Found' for url 'https://generativelanguage.googleapis.com/v1/chat/completions'"</details> <details> <summary>🤖 Prompt for AI Agents</summary>Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.In
@evidence/OMN-11513/proof-of-life/2026-06-04-93adf56-live-agent-forced-escalation/proof-packet.md
around lines 36 - 42, The fenced code block in proof-packet.md lacks a language
specifier; update the triple-backtick opening for the error message block to
include a language identifier (e.g.,text) so syntax highlighters and tooling recognize it—locate the fenced block that begins withand change it to
🤖 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.
Nitpick comments:
In
`@evidence/OMN-11513/proof-of-life/2026-06-04-93adf56-live-agent-forced-escalation/proof-packet.md`:
- Around line 36-42: The fenced code block in proof-packet.md lacks a language
specifier; update the triple-backtick opening for the error message block to
include a language identifier (e.g., ```text) so syntax highlighters and tooling
recognize it—locate the fenced block that begins with ``` and change it to
```text while keeping the existing block content unchanged.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 1a1caa3b-8988-4d61-bee8-da7dc1c13d71
⛔ Files ignored due to path filters (2)
evidence/OMN-11513/proof-of-life/2026-06-04-93adf56-live-agent-forced-escalation/run.logis excluded by!**/*.logevidence/OMN-11513/proof-of-life/2026-06-04-93adf56-live-agent-pass/run.logis excluded by!**/*.log
📒 Files selected for processing (11)
evidence/OMN-11513/EVIDENCE_MANIFEST.mdevidence/OMN-11513/integration-test-passes/2026-05-30-bret-obj1-token-registry-stability.mdevidence/OMN-11513/proof-of-life/2026-06-04-93adf56-live-agent-forced-escalation/event-chain.jsonevidence/OMN-11513/proof-of-life/2026-06-04-93adf56-live-agent-forced-escalation/kafka-readiness-evidence.jsonevidence/OMN-11513/proof-of-life/2026-06-04-93adf56-live-agent-forced-escalation/overlay-evidence.jsonevidence/OMN-11513/proof-of-life/2026-06-04-93adf56-live-agent-forced-escalation/proof-packet.mdevidence/OMN-11513/proof-of-life/2026-06-04-93adf56-live-agent-pass/event-chain.jsonevidence/OMN-11513/proof-of-life/2026-06-04-93adf56-live-agent-pass/kafka-readiness-evidence.jsonevidence/OMN-11513/proof-of-life/2026-06-04-93adf56-live-agent-pass/overlay-evidence.jsonevidence/OMN-11513/proof-of-life/2026-06-04-93adf56-live-agent-pass/proof-packet.mdevidence/OMN-11513/repros/OMN-12587-asyncio/test_publish_and_wait_in_running_loop.py
…full PASS + OMN-12664 post-closure 404 observation [OMN-12587]
jonahgabriel
force-pushed
the
clone45/omn-11513-evidence-2026-06-04-agent-runs
branch
from
June 4, 2026 17:37
bba4f12 to
adce219
Compare
jonahgabriel
added this pull request to the merge queue
Jun 4, 2026
This was referenced Jun 4, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this adds
Contractor evidence port (same shape as merged #1886), under
evidence/OMN-11513/:proof-of-life/2026-06-04-93adf56-live-agent-pass/— first full sanctioned-live--agentPASS from the contractor environment (generate → validate → register → invoke,SentimentClassifier), stability lane,transport=bus_backed(SEA chore(contracts): backfill ModelTicketContract for 50 done tickets [OMN-8637 wave 44] #206 readiness artifact). Generation served at ladder tier 1 (local Qwen, per SEA chore(contracts): backfill ModelTicketContract for 50 done tickets [OMN-8637 wave 43] #205); cloud tier not attempted. Classified diagnostic in the packet (missing: runtime identity, topic offsets, projection state).proof-of-life/2026-06-04-93adf56-live-agent-forced-escalation/— forced-escalation run (local tiers made unreachable via the documentedSEA_LOCAL_INFERENCE_BASE_URLseam) to exercise the cloud tier after OMN-12664 moved Done. Observation on SEA dev93adf56, 2026-06-04 16:09Z: escalated tocloud_gemini, runtime round-trip live, failed with the same two-URL 404 signature as the 2026-06-03 filing — registered…/v1beta/openai/chat/completions, called…/v1/chat/completions. TypedSeaGenerationError; full log inrun.log. The packet's Interpretation boundary section is explicit that this seat cannot distinguish fix-not-deployed from fix-not-on-live-path.integration-test-passes/2026-05-30-bret-obj1-token-registry-stability.md— Objective 1 evidence authored 2026-05-30, ported now (was sitting uncommitted in the contractor clone).repros/OMN-12587-asyncio/— the F1 asyncio repro script referenced from the OMN-12587 thread (fix verified in SEA chore(contracts): backfill ModelTicketContract for 50 done tickets [OMN-8637 wave 41] #203; retained as regression reference).EVIDENCE_MANIFEST.md— rows for the above + rows for the already-merged 2026-05-26/2026-05-29 files that were missing from the manifest + a 2026-06-04 addendum section.Verification thread
OMN-12587 (a comment citing these paths follows once this PR has a number). Umbrella: OMN-11513.
Process disclosure
The local omniclaude Plan Gate hook flagged this branch (ticket-pattern branch, no
docs/plans/file). This is a docs-only evidence port with no implementation changes, matching the merged precedent #1886, so the manifest edit was applied by script rather than adding a plan file for a docs port. Flagging here for transparency — happy to add a plan file if that is preferred for evidence ports going forward.No secrets: run logs verified free of key material before commit.
Summary by CodeRabbit