fix(traceloop-sdk): add google-generativeai to span processor allowlist - #1033
fix(traceloop-sdk): add google-generativeai to span processor allowlist#1033nishmethuku wants to merge 1 commit into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe default Traceloop instrumentation library list now includes ChangesGoogle Generative AI instrumentation support
Estimated code review effort: 1 (Trivial) | ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint install timed out. The project may have too many dependencies for the sandbox. 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 |
Fixes #621
The
@traceloop/instrumentation-google-generativeaipackage instruments@google/genai(the replacement for the deprecated@google-cloud/vertexai),but it was missing from the
traceloopInstrumentationLibrariesallowlist inspan-processor.ts. This caused GenAI spans to be dropped by the defaultTraceloop span processor, while VertexAI spans were exported correctly.
This one-line fix adds the package to the allowlist so spans are exported as expected.
Summary by CodeRabbit