Skip to content

evidence(OMN-12438): OCC contract for consumer.py max_tokens regression fix - #2012

Closed
jonahgabriel wants to merge 1 commit into
mainfrom
jonah/omn-12438-occ-rebased
Closed

jonahgabriel wants to merge 1 commit into
mainfrom
jonah/omn-12438-occ-rebased

Conversation

@jonahgabriel

Copy link
Copy Markdown
Contributor

evidence(OMN-12438): OCC contract for consumer.py max_tokens regression fix

Rebased cleanly onto main HEAD (cedf9db). Supersedes #1927 which had conflicts.

Evidence-Source

OMN-12438

…ens regression fix (#1923)

* evidence(OMN-12438): add contract and receipt for consumer.py max_tokens regression fix

Binds onex-self-extending-agent PR #168 (restore max_tokens=8192 in consumer.py,
add test_call_llm_sends_max_tokens_8192 regression guard) to central OCC evidence.

* fix(OMN-12438): correct receipt contract_sha256 to valid digest

Receipt had 'sha256:pending'; replaced with real digest of OMN-12438.yaml
so OCC Eligibility check passes.

* fix(OMN-12438): add dod-occ-pr receipt and fix contract_sha256 in all receipts

OCC Eligibility was failing because dod-occ-pr receipt was missing and
contract_sha256 values were stale. All three files now carry the correct
digest aa9861a0ce4b00e07c27317d75f95d7b63bf3b108a3142ba9fbb0de160ee9a5d.
@coderabbitai

coderabbitai Bot commented Jun 1, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d21b8f1e-9250-44be-9835-8933298e43a6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jonah/omn-12438-occ-rebased

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

@jonahgabriel

Copy link
Copy Markdown
Contributor Author

Closing as superseded, not queueable. origin/dev already contains the newer OMN-12438 evidence from #1951 (29578ea73 evidence(OMN-12438): bind SEA PR 176 replacement receipt), and local validation on current origin/dev passes:

uv run python scripts/ci/run_contract_compliance_check.py --pr 2012 --repo OmniNode-ai/onex_change_control --contracts-dir contracts -> OMN-12438: 4/4 PASS, 0 WARN, 0 BLOCK.

The #2012 head is based on stale main evidence and would remove the newer dod-sea-pr-176 / dod-occ-pr-1951 contract entries if retargeted to dev.

auto-merge was automatically disabled June 1, 2026 17:47

Pull request was closed

@jonahgabriel
jonahgabriel deleted the jonah/omn-12438-occ-rebased branch June 11, 2026 17:59
jonahgabriel added a commit that referenced this pull request Jun 17, 2026
…ay_history in inventory

The Migration Inventory Sync gate on this PR failed with UNLISTED_FILE:
omnibase_infra/docker/migrations/forward/086_create_dlq_replay_history.sql exists on
omnibase_infra@dev (added today by OMN-12633 / PR #2012, commit 65e6006cf retiring the
ServiceDlqTracking runtime DDL to migration-managed schema) but was not listed in
src/onex_change_control/boundaries/migration_inventory.yaml. This is fresh cross-repo
inventory drift unrelated to the version bump, but it blocks this PR via the CI Summary
required check. Adding the 086 entry (table dlq_replay_history) clears the gate.

Verified locally: a fresh GitHub clone of omnibase_infra@dev + `uv run
check-migration-inventory` produces no UNLISTED_FILE/MISSING_FILE error for infra;
tests/unit/scripts/test_check_migration_inventory.py 4 passed.
jonahgabriel added a commit that referenced this pull request Aug 3, 2026
…6026)

* evidence(OMN-15651): OCC companion for omnimarket #2014 (follow-up to #2012)

Adds two dod_evidence entries binding OMN-15651 to omnimarket PR #2014, a
follow-up to squash-merged #2012 that closes an adversarial-review finding
(missing curated _SYSTEM_PROMPTS entries for the two new task classes,
documentation + validator_generation). Existing entries from OCC#6013/#6020
are untouched.

* evidence(OMN-15651): PASS receipts for omnimarket #2014 dod_evidence entries

Live probe results for dod-OmniNode-ai-omnimarket-pr-2014 and
dod-market-2014-curated-prompts-added, both PASS at PR #2014 head
62c1f1ee8e2fdf84c438d072b3e7e97b4d4a5485.
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