-
Notifications
You must be signed in to change notification settings - Fork 295
Rust Client Implementation #2558
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
h4n0
wants to merge
37
commits into
canary
Choose a base branch
from
aomi-labs/rust-client
base: canary
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
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
- Add BAML-RUST.md: Comprehensive analysis of Rust client implementation approach - Add BAML-GO.md: Deep dive into Go client FFI architecture and performance - Document why OpenAPI-generated Rust clients are limited (~60% feature loss) - Outline implementation plan for native Rust client generator - Compare direct FFI clients vs HTTP-based clients 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Refactor Rust client generation templates for improved code structure - Remove redundant runtime.rs.j2 template - Update generator configuration to support Rust client output - Add Rust generator to BAML test configuration 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
➜ generators: ✗ cargo test --package generators-rust
…ample (failing) semantic_streaming and unions
- Keep newer protoc versions (v1.36.9, v6.31.1) from HEAD - Keep unsafe import and protobuf imports from HEAD - Use mise-based commands from upstream/canary for Go generator - Preserve lang_rust generator from HEAD
LGTM 👍 |
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🌿 Preview your docs: https://boundary-preview-939926e6-e031-4f21-b4fe-f25a5e7ce230.docs.buildwithfern.com |
5 tasks
1. adding TypeNonStreaming::Top and TypeStreaming::Top (which we my need to handle better) 2. Tags in CffiFunctionArguments comming from CffiMapEntry
🌿 Preview your docs: https://boundary-preview-0cf0e572-87b5-4141-bc7e-646c5cbe52cb.docs.buildwithfern.com |
🌿 Preview your docs: https://boundary-preview-88e282cc-966f-4e08-8ce7-9a5bc89a93dc.docs.buildwithfern.com |
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.
PR Summary
This pull request introduces integration tests for the Rust client generator in the
baml
repository. The primary goal is to ensure that the Rust client generator produces correct and reliable client code by testing its functionality against a suite of integration scenarios.Key Changes
Next Steps