Skip to content

build(deps): upgrade cli-engine to 0.9 and progenitor to 0.14 for reqwest 0.13 - #228

Merged
jpage-godaddy merged 1 commit into
mainfrom
progenitor-upgrade
Aug 24, 2026
Merged

build(deps): upgrade cli-engine to 0.9 and progenitor to 0.14 for reqwest 0.13#228
jpage-godaddy merged 1 commit into
mainfrom
progenitor-upgrade

Conversation

@jpage-godaddy

Copy link
Copy Markdown
Collaborator

Summary

  • Bumps cli-engine to 0.9.0, which moved to reqwest 0.13 (aws-lc-rs-backed rustls, rustls-tls feature renamed to rustls, form-encoded bodies split into a new form feature).
  • Bumps domains-client's progenitor/progenitor-client from 0.11 to 0.14 to match — the previous pin was progenitor's last release still on reqwest 0.12; 0.12+ requires reqwest 0.13. Keeps the whole workspace on a single reqwest major version, since domains-client's TransportObserver trait and cli_engine::transport::debug_log_reqwest_request/debug_log_reqwest_response both take reqwest::Request/Response by reference across the crate boundary.
  • Fixes a short-flag collision: cli-engine 0.9.0 added a global --interactive/-i flag, which collided with api call's and api graphql call's own --include/-i. Dropped the short form from --include (long flag only).

Test plan

  • cargo check
  • cargo clippy --all-targets -- -D warnings
  • cargo fmt --check
  • cargo test (693 passed)
  • cargo test -p domains-client (15 passed; generated-client doctests unaffected)
  • ./rust/scripts/check-module-size.sh

Copilot AI lite review requested due to automatic review settings August 24, 2026 21:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Upgrades core HTTP/client dependencies to align the workspace with reqwest 0.13 (via cli-engine 0.9.0 and progenitor/progenitor-client 0.14), and adjusts CLI flags to avoid a new global -i collision introduced by cli-engine.

Changes:

  • Bump cli-engine to 0.9.0 and update the workspace reqwest dependency to 0.13 with the renamed/split TLS/form features.
  • Bump domains-client’s progenitor/progenitor-client to 0.14 and move it to reqwest 0.13.
  • Remove the -i short flag from api call --include and api graphql call --include to avoid collision with the new global --interactive/-i.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
rust/src/api_explorer/graphql/call.rs Drops -i short flag from --include for GraphQL calls to avoid global -i collision.
rust/src/api_explorer/call.rs Drops -i short flag from --include for REST calls to avoid global -i collision.
rust/domains-client/Cargo.toml Updates progenitor* and reqwest versions/features to reqwest 0.13 compatibility.
rust/Cargo.toml Bumps cli-engine and moves the main crate to reqwest 0.13 with updated feature flags (rustls, form).
rust/Cargo.lock Lockfile updates reflecting the dependency upgrades and new transitive deps from reqwest/rustls changes.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

…west 0.13

cli-engine 0.9.0 moved to reqwest 0.13 (aws-lc-rs-backed rustls, renamed
rustls feature). domains-client follows suit: progenitor/progenitor-client
0.11 -> 0.14 to pick up the matching reqwest 0.13 generated client, keeping
the whole workspace on one reqwest major version.

cli-engine 0.9.0 also added a global --interactive/-i flag, which collided
with api call/graphql call's own --include/-i. Dropped the short form from
--include (long flag only) to resolve it.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 5 changed files in this pull request and generated no new comments.

@jpage-godaddy
jpage-godaddy merged commit 764adc1 into main Aug 24, 2026
5 checks passed
@jpage-godaddy
jpage-godaddy deleted the progenitor-upgrade branch August 24, 2026 23:16
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.

3 participants