fix: SDK usage statistics tracking for vision-agents#304
Conversation
📝 WalkthroughWalkthroughA single-line modification to the StreamEdge initialization updates the AsyncStream client's user_agent identification string from "vision-agents-{version}" to "stream-vision-agents-{version}". No functional or control flow changes are introduced. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
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 |
Summary
stream-prefix to user_agent so vision-agents API calls are tracked in SDK usage statisticsX-Stream-Clientheader to match the patternstream-*-MAJOR.MINOR.PATCHvision-agents-{version}was used, which didn't match and was silently ignoredDetails
The
X-Stream-Clientheader is parsed bychat/server/sdkstats/sdk_stats.gousing regex:This change updates the user agent from
vision-agents-1.0.0tostream-vision-agents-1.0.0so it matches the expected pattern.Test plan
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.