chore(deps): bump connectrpc/connect to v1.20.0 in sdk, service, otdfctl#3589
Conversation
Aligns these modules with protocol/go which is already on v1.20.0. Resolves internal version skew that was blocking downstream consumers (virtru-sdk-go, DSP SDK) from upgrading. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Mary Dickson <mary.dickson@virtru.com>
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request performs a dependency maintenance task to align the connectrpc/connect library version across multiple modules in the repository. By standardizing on v1.20.0, the change eliminates internal version mismatches that were previously blocking downstream SDK consumers from upgrading their own dependencies. Highlights
New Features🧠 You can now enable Memory (public preview) to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize the Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counterproductive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. The versions were split in the code, A burden that heavy did load. We bumped up the trace, To a unified space, And now we are back on the road. Footnotes
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Pro Run ID: ⛔ Files ignored due to path filters (3)
📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThis PR updates the ChangesConnect RPC Dependency Upgrade
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
There was a problem hiding this comment.
Code Review
This pull request updates the connectrpc.com/connect dependency from version 1.19.2 to 1.20.0 in the otdfctl, sdk, and service modules. There are no review comments, and I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Benchmark results, click to expandBenchmark authorization.GetDecisions Results:
Benchmark authorization.v2.GetMultiResourceDecision Results:
Benchmark Statistics
Bulk Benchmark Results
TDF3 Benchmark Results:
|
|
|
connect v1.20.0 changes (release notes):
No breaking API changes. All downstream repos (DSP at Go 1.25.9, virtru-sdk-go at 1.25.11) are also on Go 1.25+, so the floor bump is a non-issue. |
Summary
connectrpc.com/connectfrom v1.19.2 to v1.20.0 insdk/,service/, andotdfctl/protocol/gois already on v1.20.0 — this aligns the rest of the repoContext
Identified in the Dependency Version Audit — connect v1.19.2 vs v1.20.0 split within platform was preventing downstream repos from bumping without breaking compatibility.
Test plan
🤖 Generated with Claude Code
Summary by CodeRabbit