Repository navigation
THROWAWAY(OMN-15178): DO-NOT-MERGE - seeded vector-5 skip-vector RED proof (omnimarket X4 parity) - #1914
THROWAWAY(OMN-15178): DO-NOT-MERGE - seeded vector-5 skip-vector RED proof (omnimarket X4 parity)#1914jonahgabriel wants to merge 1 commit into
Conversation
…ity RED proof DO NOT MERGE. Mirrors omniclaude#1947 (closed unmerged 2026-07-26): removes if: always() from the needs-gated job producing the REQUIRED Hostile Review Gate context, reproducing the exact vector-5 shape (OMN-15057).
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 26 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
✅ Architectural Review — PASSEDErrors: 0 What this checks
No architectural violations found. Static architectural lint — no model inference (OMN-14176). |
|
Seeded-proof captured — closing UNMERGED per plan. Guard went RED as designed: run https://github.com/OmniNode-ai/omnimarket/actions/runs/30269831694 named |
✅ Hostile Reviewer — PASSEDBlocking findings (critical/error): 0 Gate semantics
Powered by omniintelligence.review_pairing.cli_review — node-based adversarial review via HandlerLlmCliSubprocess (OMN-8524) |
DO NOT MERGE — THROWAWAY SEEDED-PROOF PR
Closed unmerged as soon as the guard result is captured; branch deleted after. Mirrors the omniclaude precedent omniclaude#1947 (closed unmerged 2026-07-26).
Purpose
OMN-15178 X4 residual: omnimarket seeded-proof parity. The
required-check-skip-guard / check-skip-vectorscontext was flipped to REQUIRED on omnimarketdev(operator ruling 2026-07-26T07:25Z) on the strength of green runs alone. Green runs prove the guard runs; they do not prove it fires. This PR seeds a real violation to prove RED.Seeded vector
.github/workflows/hostile-reviewer.yml, jobhostile-review-gate(produces the REQUIREDHostile Review Gatecontext, manifestskip_semantics: never): removedif: always()while keepingneeds: [hostile-review].That is the exact vector-5-ungated-needs-cascade shape (OMN-15057) — GitHub's implicit job-level
if:issuccess()overneeds:, so a failed upstream SKIPS the job, and a skipped job satisfies branch protection. This is the very regression OMN-15057 fixed on this file (live proof: omnimarket#1880, where 18 required gates wentskipped).Expected result
required-check-skip-guard / check-skip-vectorsRED, naming[vector-5-ungated-needs-cascade] required context 'Hostile Review Gate'.A local pre-run of the same validator against this branch already returns
FAIL: 1 required-check skip vector(s) foundwith that exact finding.Ticket
OMN-15178 — evidence comment posted on the ticket. No status flips.