Skip to content

chore(cli): reconcile weekly API parity - #128

Merged
callumreid merged 4 commits into
mainfrom
chore/weekly-api-parity
Sep 14, 2026
Merged

callumreid merged 4 commits into
mainfrom
chore/weekly-api-parity

Conversation

@callumreid

Copy link
Copy Markdown
Contributor

Automated comparison of the live public OpenAPI catalog with first-class CLI commands.

  • api-coverage-report.md is deterministic and changes only when API or CLI coverage changes.
  • This PR does not claim to generate the CLI's hand-written command UX.
  • If the report says ACTION REQUIRED, implement the missing commands or explicitly review the manifest exception, regenerate the report, and get CI green before merging.

The branch is stable, so future weekly runs update this PR instead of opening duplicates.

callumreid and others added 4 commits September 14, 2026 10:07
CreatePersonaRequest and UpdatePersonaRequest did not declare the published
interruption_rate field, so serde discarded it in silence. The API applies
the field through an is-not-None guard on both create and update, so it
stays a plain Option with no explicit-null clear.
The metrics create and update requests dropped ivr_flow and the runs
launch request dropped config_overrides, so serde discarded both. The
metrics update path clears ivr_flow when it arrives as an explicit null,
so the update request keeps absent and null distinguishable.

MetricType also lacked the METRIC_IVR_FLOW_ADHERENCE variant, which made
--ivr-flow unusable end to end: the type deserialized to the untagged
Unknown fallback and would have been sent back as "Unknown".
Cover the newly modeled request fields with forwarding tests and README
examples, refresh the reviewed snapshot, and regenerate the report. The
audit passes with no unreviewed gaps remaining.
@callumreid
callumreid merged commit 177fd67 into main Sep 14, 2026
6 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