Skip to content

ci(OMN-21003): omniintelligence's pr-title check runs omnimarket's node, not onex_change_control's reusable - #1057

Merged
jonahgabriel merged 1 commit into
devfrom
jonah/omn-21003-pr-title-omnimarket-node
Oct 11, 2026
Merged

jonahgabriel merged 1 commit into
devfrom
jonah/omn-21003-pr-title-omnimarket-node

Conversation

@jonahgabriel

@jonahgabriel jonahgabriel commented Oct 11, 2026 •

Copy link
Copy Markdown
Contributor

Ticket: OMN-21003

PR Title Check now installs published omnimarket==0.4.310 and runs node_pr_title_check_compute in an isolated Python 3.12 environment. The job remains pr-title, named pr-title / check-title, with the existing pull request triggers and effective runner expression. The manifest declares its local producer using the existing schema.

The child contract binds workflow shape, installed-node verdict parity and manifest criteria to executable checks in the focused test file.

The clean-root and naming hooks use uv run --frozen python to resolve the project interpreter when the host has no python executable on PATH. Their validator logic and enforcement are unchanged.

Lab: host=h201 lane=occ-ret-prtitleintel-i3-8e80 command=timeout 110 uv run --no-project --with pytest --with pytest-asyncio --with pytest-xdist --with pyyaml pytest tests/ci/test_pr_title_check_omnimarket_node.py -q -n 0 observed=33 passed(exit 0); merge-base control=3 failed,30 passed(exit 1).

  • Published-node parity: 29 title/author cases against the exact retired bash decision; malformed input returns 2. Manual OMN/bare/dependency titles return 0/1/0.
  • Local DoD dry run: implementation verifies 3/3; merge-base workflow and manifest refuse AC1 and AC3 (exit 1), with AC2 still verified.
  • Actionlint passes; an undefined-expression control refuses (exit 1). Focused ruff check and format pass. The full-project focused pytest invocation also passes after YAML formatting.
  • Offline advisory gate returns the same seven existing findings on the base and changed files. The logs are byte-identical; this change adds no finding. The validator-requirements consumer is baseline-clean.
  • Remote pre-commit is blocked by the sandbox's read-only .agents, .aws and .codex directories at the clean-root hook. The launching host must complete governed pre-commit in its normal worktree before publication; no validator was changed.

Lab results cover the staged implementation on base f500ad32f7f1c0259045b33fe35cbe426cb4b96f. Commit, push, PR creation and landing handoff remain with the launching host and landing controller.

Evidence-Ticket: OMN-21003
Evidence-Source: OCC#13695

onexbot-occ-writer Bot pushed a commit to OmniNode-ai/onex_change_control that referenced this pull request Oct 11, 2026
…omniintelligence#1057

OCC companion by node_pr_lifecycle_fix_effect (OMN-13317 F1 / OMN-13990 / OMN-14285). Product PR head daa6994b206ab9e1be4d2a1ad89f5b91fdf57b9e.
jonahgabriel pushed a commit to OmniNode-ai/onex_change_control that referenced this pull request Oct 11, 2026
…intelligence (#13695)

* evidence(OMN-21003, OMN-20940): author OCC companion for OmniNode-ai/omniintelligence#1057

OCC companion by node_pr_lifecycle_fix_effect (OMN-13317 F1 / OMN-13990 / OMN-14285). Product PR head daa6994b206ab9e1be4d2a1ad89f5b91fdf57b9e.

* evidence(OMN-21003, OMN-20940): self-bind OCC#13695 + rebind contract_sha256

---------

Co-authored-by: omnimarket-bot <bot@omninode.ai>
@jonahgabriel
jonahgabriel merged commit 5f6fe6b into dev Oct 11, 2026
96 of 103 checks passed
@jonahgabriel
jonahgabriel deleted the jonah/omn-21003-pr-title-omnimarket-node branch October 11, 2026 21:40
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