Skip to content

Conversation

@bradhe
Copy link
Contributor

@bradhe bradhe commented May 4, 2025

  • Bump version to v0.3.11
  • Modernized way of generating Rust clients
  • Customize generator to extract Tower Trace ID from each request
  • Add Tower Trace ID to debug output to make debugging easier
  • Fix broken --local runs (due to change in package encoding)
  • Update Python client
  • Update Rust client
  • Standardize how we interact with various commands in Tower

@bradhe bradhe requested review from Copilot and sankroh May 4, 2025 10:08
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This release bumps the OpenAPI document version to v0.5.6, modernizes the Rust client generation including updated debug output, and fixes a broken local run related to package encoding.

  • Updated documentation comments across multiple model files
  • Added Tower Trace ID to the API response struct for improved debug output
  • Updated user-agent string in the API configuration and adjusted Cargo.toml dependencies

Reviewed Changes

Copilot reviewed 154 out of 154 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
crates/tower-api/src/models/* Updated OpenAPI document version comments
crates/tower-api/src/apis/mod.rs Added tower_trace_id field for tracing in debug output
crates/tower-api/src/apis/configuration.rs Updated user-agent string
crates/tower-api/README.md Updated API version, generator version, and added new endpoints
crates/tower-api/Cargo.toml Added log dependency and updated reqwest dependency settings

use std::fmt;

#[derive(Debug, Clone)]
pub struct ResponseContent<T> {
Copy link

Copilot AI May 4, 2025

Choose a reason for hiding this comment

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

Consider adding a documentation comment for the new field 'tower_trace_id' to clarify its purpose and usage in debug output.

Suggested change
pub struct ResponseContent<T> {
pub struct ResponseContent<T> {
/// A unique identifier for tracing requests and responses, used for debugging purposes.

Copilot uses AI. Check for mistakes.
@bradhe bradhe merged commit f29cdfd into main May 4, 2025
22 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.

2 participants