-
Notifications
You must be signed in to change notification settings - Fork 120
refactor: update dependencies and fix pointer types in schemas #553
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
refactor: update dependencies and fix pointer types in schemas #553
Conversation
|
Caution Review failedThe pull request is closed. 📝 WalkthroughSummary by CodeRabbit
WalkthroughThe diff updates error typing in Anthropic chat schema, adjusts request parameter extraction for Cohere, removes a string-slice pointer helper, applies pointer-to-slice refactors across tests, deletes Anthropic and OpenAI HTTP integration type/convert layers, expands plugin module dependencies, and switches UI tab elements from div to button. Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Poem
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (14)
Comment |
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
f8870a6 to
c01b5a7
Compare
4aba40f to
025f8a9
Compare

Summary
Updated Go module dependencies in the plugins/logging and plugins/otel packages, and fixed schemas usage in test files to accommodate changes in the core library.
Changes
github.com/bytedance/sonicfrom direct to indirect dependency in plugins/loggingType of change
Affected areas
How to test
Breaking changes
Related issues
Addresses dependency updates and schema compatibility issues.
Checklist