Skip to content

ci(OMN-17427): run CI on pushes to dev - #3222

Merged
jonahgabriel merged 1 commit into
devfrom
jonah/omn-17427-omnimarket-dev-push-ci
Oct 2, 2026
Merged

jonahgabriel merged 1 commit into
devfrom
jonah/omn-17427-omnimarket-dev-push-ci

Conversation

@jonahgabriel

@jonahgabriel jonahgabriel commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Summary

This change adds dev to the push trigger branches of .github/workflows/ci.yml. Previously the workflow ran on pushes to main and hotfix/** only, so no CI ran on the dev landing branch after a merge. This matches omnibase_infra, omnibase_core and omniclaude. One docstring in tests/unit/scripts/ci/test_l4_event_scope_omn18963.py is updated to match.

Cause

dev is not strict and has no merge queue. #3208 merged at 2026-10-01T23:59:26Z and widened the projection row-source protocol. #3212 merged at 00:03:54Z on CI graded against the older base.

As a result, dev head d471dcb96 broke typecheck and Tests (Split 14/20), and no check ran on it until the release fast-forward to main at 01:18Z. For 75 minutes the red base read as per-PR reds, and the landing controller sent PR-level fix workers to omnimarket#3214 to #3219.

Verification

  • test_l4_event_scope_omn18963.py: 9 passed.
  • actionlint findings unchanged at 5 pre-existing.
  • check_release_identity.py: OK, no packaged src change.
  • The push concurrency group is ref-scoped and never cancels, so dev pushes queue at most one pending run.
  • CI-only change, not runtime-affecting, so no lab pass applies.
  • The edit was drafted by Codex thread 01a0fa6e-8cbf-7771-9ee4-ab75223553cf and reviewed; this body was drafted on the lab (onex delegate run 1869005e-6b4f-4654-bfbd-a459820f6c94) and checked against the facts.

Evidence-Ticket: OMN-17427

Evidence-Source: OCC#12271

@jonahgabriel
jonahgabriel enabled auto-merge (squash) October 2, 2026 02:30
@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown

✅ Hostile Reviewer — PASSED

Blocking findings (critical/error): 0
Total findings: 4
Models succeeded: qwen3-review,gpt-oss-review


Gate semantics

Verdict Meaning Blocks merge?
passed >=2 models succeeded, no critical/error findings No
blocked CRITICAL or ERROR findings found Yes
degraded Fewer than 2 models succeeded (infra unavailable/timeout) Yes (OMN-15110)

Powered by omniintelligence.review_pairing.cli_review — node-based adversarial review via HandlerLlmCliSubprocess (OMN-8524)

@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown

✅ Architectural Review — PASSED

Errors: 0
Warnings: 0

What this checks

Rule Description
ARCH-TOPIC-001 No hardcoded Kafka topic strings in handler code
ARCH-DI-001 No event_bus=None bypass
ARCH-DI-002 No direct Handler instantiation outside workflow_runner/adapters
ARCH-DI-003 No reinvented DI containers
ARCH-TOPIC-002 contract.yaml topics follow `onex.{cmd

No architectural violations found.

Static architectural lint — no model inference (OMN-14176).

jonahgabriel added a commit to OmniNode-ai/onex_change_control that referenced this pull request Oct 2, 2026
)

* evidence(OMN-17427): hand-authored companion for omnimarket#3222

Hand-authored change-control companion for OmniNode-ai/omnimarket#3222 (OMN-17427).
occ-autobind declined with DEFER_HAND_AUTHORED.
Adds two dod_evidence entries to contracts/OMN-17427.yaml and two PASS receipts under drift/dod_receipts/OMN-17427/ bound to pr_number 3222.
Probes: diff-scope check (gh pr view 3222) and branch line count in ci.yml.
yamlfmt pre-commit passed.

* evidence(OMN-18921): mint the owed OCC self-bind for PR #12271

---------

Co-authored-by: onexbot-occ-writer[bot] <onexbot-occ-writer[bot]@users.noreply.github.com>
dev was excluded from the CI push triggers, and dev is not strict and has no
merge queue. #3212 merged on CI graded against a base older than #3208, so dev
head d471dcb broke typecheck and Tests (Split 14/20) with no check run on it
until the release fast-forward to main. This adds dev to push.branches, matching
omnibase_infra, omnibase_core and omniclaude; the push concurrency group is
ref-scoped and never cancels.
@jonahgabriel
jonahgabriel force-pushed the jonah/omn-17427-omnimarket-dev-push-ci branch from 0d33533 to c215218 Compare October 2, 2026 03:38
@jonahgabriel
jonahgabriel merged commit 7508089 into dev Oct 2, 2026
127 checks passed
@jonahgabriel
jonahgabriel deleted the jonah/omn-17427-omnimarket-dev-push-ci branch October 2, 2026 04:14
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