Repository navigation
Conversation
867fc4b to
bfa64eb
Compare
b4e6f56 to
8a48aa0
Compare
8a48aa0 to
748fa42
Compare
748fa42 to
838f0f3
Compare
838f0f3 to
35fb505
Compare
35fb505 to
e6bce41
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit e6bce41. Configure here.
| reason = report.ReasonQueueOverflow | ||
| } | ||
| a.recorder.RecordForEnvelope(reason, envelope) | ||
| } |
There was a problem hiding this comment.
Queue overflow outcomes not recorded
Medium Severity
AsyncTransport.SendEnvelope no longer records a full queue, and internalAsyncTransportAdapter only does so when recorder is set. The telemetry processor installs that adapter without a recorder, so SendEvent drops on ErrTransportQueueFull (and other send errors) never reach client reports.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit e6bce41. Configure here.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit e6bce41. Configure here.
This folds the ratelimited and capacity functionalities under transport so that users should only use SendEnvelope as public API
e6bce41 to
706a711
Compare


Description
Establish a common envelope-based contract for transports and loss accounting. This removes
IsRateLimitedfrom the public transport API and encapsulates it's functionality inside the transport.#skip-changelog
Issues
Changelog Entry Instructions
To add a custom changelog entry, uncomment the section above. Supports:
For more details: custom changelog entries
Reminders
feat:,fix:,ref:,meta:)Stack created with GitHub Stacks CLI • Give Feedback 💬