Skip to content

Test manifest schema compatibility with a bounded same-input probe - #226

Merged
magnus919 merged 1 commit into
mainfrom
codex/oct7-schema-compatibility-probe-226
Oct 7, 2026
Merged

magnus919 merged 1 commit into
mainfrom
codex/oct7-schema-compatibility-probe-226

Conversation

@magnus919

Copy link
Copy Markdown
Contributor

The v17 review reached the configured inference endpoint, but all 19 specialist requests returned the same HTTP 400 signature. Its whole-object target enum is a compatibility hypothesis, not a proven cause. The opt-in target schema now uses nested per-kind anyOf objects with primitive string enums; empty manifests use a valid item schema with zero allowed proposals. Server validation still checks the exact kind/value/evidence-kind binding before spending retrieval quota. Legacy request bytes, profile bytes, limits, and adapter identities remain unchanged.

Adds a main-only, default-off diagnostic that captures one fixed public PR305 task from bare Git objects and sends at most two same-input, zero-retry requests: the actual archived b075 schema and the candidate. It verifies only the target-schema subtree differs, binds both to the same fresh preparation receipt, and uploads only bounded hashes/status/error categories. It never checks out or executes target files, exposes no raw provider body or credentials, and cannot publish reviews. Historical request hashes are comparisons, not eligibility gates.

Validation: 16 root focused tests passed; independent review CLEAR with an actual eligible bare-source preparation and 15 independently run tests. Standard JSON Schema checks cover allowed and cross-kind pairs and the empty case; the existing blocker/quota regression passed. Fake provider tests passed 29 tests plus six subtests. Ruff, YAML/TOML, and diff checks passed. Endpoint compatibility remains unknown until the paired probe.

Reference for the candidate construction: OpenAI Structured Outputs supported schema subset. This does not establish third-party endpoint compatibility.

@magnus919
magnus919 merged commit 816c348 into main Oct 7, 2026
24 checks passed
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