-
Notifications
You must be signed in to change notification settings - Fork 3.9k
feat: raindrop optional observability through env vars #9123
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
base: main
Are you sure you want to change the base?
Conversation
Learn moreAll Green is an AI agent that automatically: ✅ Addresses code review comments ✅ Fixes failing CI checks ✅ Resolves merge conflicts |
Learn moreAll Green is an AI agent that automatically: ✅ Addresses code review comments ✅ Fixes failing CI checks ✅ Resolves merge conflicts |
|
✅ Review Complete Code Review SummaryTroubleshooting
|
Documentation ReviewAfter reviewing PR #9123, no documentation updates are needed for the following reasons: Why No Docs Needed
Documentation Assessment
RecommendationIf Raindrop observability becomes a recommended or officially supported feature in the future (e.g., for enterprise customers or debugging workflows), then documentation should be added to explain:
Until then, keeping this as an undocumented internal feature is appropriate. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues found across 11 files
CI Check Failure AnalysisThe failing E2E test is not related to the PR changes. Root CauseThis is a flaky infrastructure issue where GitHub's CDN/download service returned a 403 error when trying to download the Evidence
RecommendationRe-run the failed job - This is a transient network/CDN issue that should resolve on retry. The E2E test failure is unrelated to the Raindrop observability changes in this PR. |
|
@sestinj The E2E test failure is a transient ripgrep download issue (403 error) unrelated to the PR code. Could you re-run the failed workflow? |
| import { serviceContainer } from "../services/ServiceContainer.js"; | ||
| import { | ||
| AgentFileServiceState, | ||
| AuthServiceState, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is a comment

Description
[ What changed? Feel free to be brief. ]
AI Code Review
@continue-reviewChecklist
Screen recording or screenshot
[ When applicable, please include a short screen recording or screenshot - this makes it much easier for us as contributors to review and understand your changes. See this PR as a good example. ]
Tests
[ What tests were added or updated to ensure the changes work as expected? ]
Summary by cubic
Adds optional Raindrop observability for CLI LLM calls. When RAINDROP_API_KEY or RAINDROP_WRITE_KEY is set, we emit OpenTelemetry traces for OpenAI and Anthropic calls with session metadata.
New Features
Dependencies
Written for commit a293cb6. Summary will update automatically on new commits.