Repository navigation
Sync decisions contract for InferenceInfra #5208 (Eve format 4) - #208
Conversation
…e format 4) prompt_format_version gains 4 (pplx-decider-v1.2's wording) in the request and response enums. Vendored from InferenceInfra ecd738b257; the binding is regenerated (only its contract hash moves: the reader takes the version as an integer) and versions.toml pins the new commit and sha256.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Walkthrough
Merge Risk: ⚪ Minimal · up to
|
Sync of the vendored decisions contract for InferenceInfra #5208: Eve (pplx-decider) moves to v1.2, which answers
prompt_format_version4.contracts/wally-decisions-public-v1.openapi.json: re-vendored from InferenceInfraecd738b257. Theprompt_format_versionenums gain 4, and their descriptions change.src/account/decisions_contract.rs: regenerated. OnlyCONTRACT_SHA256moves, because the reader takes the version as an integer with no enum check.versions.toml: thedecisions_source_commitanddecisions_contract_sha256pins.There's no behaviour change for Wally. It never pins
prompt_format_version, and it already reads any integer.Why now: InferenceInfra's
consumer-impactcheck keeps #5208 red until this pin matches it. This PR merges first, then #5208. The monorepo has a sibling PR that does the same.python3 contracts/sync_from_inferenceinfra.py --checkpasses. The cargo tests need the kit, so CI runs them.Summary by cubic
Syncs the vendored InferenceInfra decisions contract so
prompt_format_version4 (Eve v1.2) is accepted.ecd738b257, adding 4 to the request and responseprompt_format_versionenums and updating their descriptions.src/account/decisions_contract.rs; onlyCONTRACT_SHA256changes because the reader takes the version as an integer with no enum check.decisions_source_commitanddecisions_contract_sha256pins inversions.toml.No behavior change for Wally: it never pins
prompt_format_versionand already reads any integer. This merge unblocks InferenceInfra #5208, so merge it first.Written for commit 9ce1c27. Summary will update on new commits.
Summary by CodeRabbit