Repository navigation
[COVAL-7144] Reconcile the CLI with the Oct 5 API catalog and release 0.9.0 - #131
Merged
Merged
Conversation
callumreid
force-pushed
the
chore/weekly-api-parity
branch
from
September 28, 2026 10:09
a6b2188 to
286e1be
Compare
….9.0 - Run-template test_case_ids, metric aggregation_method and unit, and the review-annotation structured ground-truth fields now forward an explicit null, which the API uses to clear them. Plain Option dropped the null, so a test-case subset could never be removed (the API rejects an empty list). - Use caller data for the initialization_parameters test value; it is a persona setting, not an account attribute. - Bump to 0.9.0: review-annotations create --priority is removed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
kdmelon
approved these changes
Oct 5, 2026
kdmelon
left a comment
Contributor
There was a problem hiding this comment.
APPROVE — first review.
Severity counts: P0 0, P1 0, P2 0, P3 0.
Validation: inspected the complete current-head diff and request serialization paths; all current-head CI, CodeQL, and cross-platform builds are passing. No existing review threads.
Verification limitations: this is a client contract-parity change; review used the PR's mock-server serialization coverage and current-head CI rather than a live authenticated API round trip.
This was referenced Oct 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Reconciles the CLI with the 2026-10-05 public API catalog (183 operations, 437 request fields) and releases 0.9.0.
Changes
judge_mode/enabled_tools; SQL and custom-traceaggregation_method/unit/value_source/span_name/metric_attribute; audio presets and thresholds;metric_metadata.silent_mode,multi_phone_config,initialization_parameters,custom_persona_data,voice,custom_voice_id.project_typeon update.test_case_ids.nullinstead of dropping it:test_case_ids(the API rejects[], so null is the only way to remove a test-case subset);aggregation_method/unit;annotations/ground_truth_json/ground_truth_set_value.review-annotations create --priority. The create contract no longer accepts priority and the server ignored it, so the flag did nothing. Scripts that pass it now get a clap error.update --priorityis unchanged.persona_metricslaunch field. The CLI always sentNone, so the wire format doesn't change.complete-conversationandsofia/delegation-tokenas reviewed gaps.Release on version bump.Validation
cargo fmt --checkandcargo clippy --all-targets -- -D warnings: clean.cargo test -- --test-threads=2: 9 + 194 passed.test_update_check_notifies_when_outdatedcan trip its 500 ms budget. That flake predates this PR.api-coverage-report.mdis byte-identical to the committed one.release_version.pyreportsv0.9.0.api.coval.dev/v1/openapiis byte-identical to docsorigin/mainfor all 23 specs. Every added field's name and optionality matches the spec.Follow-ups
review-annotations create --priority(cli/human-review.mdx,llms-full.txt). Remove it when 0.9.0 ships.Part of COVAL-2079 (the CLI/API lockstep umbrella stays open).
🤖 Generated with Claude Code