Conversation
* chore: openapi-python-client 0.24.3->0.28.1 * chore: regenerate python client * chore: fix test fixtures for new api client * chore: bump to deb 12 for newer python on arm * fix: don't globally install pip we run everything in a venv anyway * fix: try to get 32 bit ARM build to compile * chore: make builds run when stuff that affects the build changes * fix: workaround qemu bug with tmpfs for ~/.cargo * fix: jettisoning 32 bit ARM building - more trouble than they're worth * hack: workaround broken old dependency Could've updated the dependency, but figure if it's affecting us it's going to affect others too
* Add a wraper function to Tower SDK for packaging code for Tower * Add tests for this exposed functionality * Add the Tower CLI binary back in the mix * Upgrade Maturin usage to find interpreters for us * Change the way installation happens in the actions script * Don't limit the installations to just what's local * Final bits to make windows, linux, and mussllinux builds work/tested * chore: Yet another attempt to let maturin to do it's thing * Fix formatting of code in a test * Stub for mypy. Shouldn't interfere with other stuff * Updates to integration tests for the new target * Enable builds on ARM64 targets on Windows * Some cleanup, please deleting a .so * Share data between steps * A few fixes for the last set of failures * chore: Add separate runner type for Windows ARM * Addres some feedback from @konstantinoscs
* chore: Builds for all other python versions * Windows ARM64 only supports starting at 3.11. * macOS builds only support Python 3.11+
* Explicitly try to install wheels built when doing the tests * Drop Python 3.9 support
* chore: openapi-python-client 0.24.3->0.28.1 * chore: regenerate python client * chore: bump autogen rust oapi client * chore: bump generator + versions, fix API references * fix: update mock server to be compliant with latest API specs
* fix(mock): use RFC 7807 format for 422 validation response Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(mock): match real server warning event and log stream format Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix(mock): add name field to schedule update endpoint Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * test: update BDD assertions to match real server responses Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * feat(test-runner): support running tests against external server Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * chore: make CLI build location configurable in tests * fix: TOWER_JWT being set makes it ignore the actual mock one in the repo 🤦 --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
* Updated Readme for Tower CLI * Added an MCP guide * Restored dbt section under optional * Added pointer to full MCP docs for users of other AI assistants * Addressing Brad's comments * Update INSTALL-AND-REFERENCE.md Co-authored-by: Ben Lovell <ben@tower.dev> * Applying feedback * Added some additional case studies --------- Co-authored-by: Ben Lovell <ben@tower.dev>
* Move `build_package` into a namespace * chore: Fix test formatting
|
Important Review skippedToo many files! This PR contains 299 files, which is 149 over the limit of 150. ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (299)
You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
bradhe
requested review from
codingcyclist,
Copilot,
datancoffee,
giray123,
jo-sm,
konstantinoscs,
sammuti and
socksy
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the Tower SDK to version 0.3.49rc2 with significant changes to the Python client, new packaging functionality, and infrastructure improvements.
Changes:
- Modernized Python API client error handling and type annotations
- Added native package building capabilities via PyO3 bindings
- Updated OpenAPI client generator and API models to align with v0.9.29 spec
Reviewed changes
Copilot reviewed 297 out of 552 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| pyproject.toml | Updated version to 0.3.49rc2, added tower CLI entrypoint, changed bindings from bin to pyo3, bumped Python requirement to 3.10+ |
| src/tower/packages/init.py | New module exposing build_package function from native bindings |
| src/tower/cli.py | New CLI entrypoint that delegates to Rust implementation via _run_cli |
| src/tower/_native.pyi | Type stubs for native Rust functions (build_package, _run_cli) |
| crates/tower/src/lib.rs | New PyO3 bindings for package building and CLI execution |
| crates/tower/Cargo.toml | Added pyo3 feature and dependencies for Python bindings |
| crates/tower-uv/src/lib.rs | Added setuptools<82 workaround for legacy pkg_resources compatibility |
| scripts/generate-rust-api-client.sh | Bumped OpenAPI generator version from 7.13.0 to 7.19.0 |
| scripts/rust-api-client-generator-config.yaml | Added App→deprecated_app name mapping |
| src/tower/tower_api_client/api/default/*.py | Standardized error handling across all API endpoints, replacing Optional returns with ErrorModel unions |
| crates/tower-api/src/models/*.rs | Updated API models with new fields and OpenAPI spec version v0.9.29 |
| crates/tower-cmd/src/api.rs | Simplified schedule creation with Default::default() pattern |
| DEVELOPMENT.md | New development guide with setup instructions and workflow |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
konstantinoscs
approved these changes
Feb 24, 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
5 participants
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.
Uh oh!
There was an error while loading. Please reload this page.