Skip to content

Remove Sentry error tracking integration#1038

Open
r33drichards wants to merge 1 commit intomainfrom
claude/remove-sentry-qp1fD
Open

Remove Sentry error tracking integration#1038
r33drichards wants to merge 1 commit intomainfrom
claude/remove-sentry-qp1fD

Conversation

@r33drichards
Copy link
Collaborator

Summary

This PR removes Sentry error tracking integration from the CUA SDK, consolidating telemetry to use only PostHog for analytics and OpenTelemetry for operational metrics.

Key Changes

  • Removed Sentry module: Deleted libs/python/core/core/telemetry/sentry.py containing all Sentry SDK initialization and error tracking functionality
  • Removed Sentry imports: Removed all imports of Sentry functions (add_breadcrumb, capture_exception, set_context, etc.) from:
    • libs/python/agent/agent/callbacks/otel.py
    • libs/python/computer/computer/computer.py
    • libs/python/computer/computer/otel_wrapper.py
  • Removed Sentry calls: Eliminated all breadcrumb tracking and exception capture calls throughout the codebase
  • Updated dependencies: Removed sentry-sdk from optional dependencies in pyproject.toml and uv.lock
  • Updated exports: Removed Sentry-related functions from libs/python/core/core/telemetry/__init__.py public API
  • Updated documentation: Updated module docstring to reflect that telemetry now uses only PostHog and OpenTelemetry

Implementation Details

  • All error tracking now relies solely on OpenTelemetry's record_error function
  • Breadcrumb logging for debugging context has been removed (can be added back via OpenTelemetry spans if needed)
  • The OtelErrorCallback class docstring was updated to remove reference to Sentry
  • No functional changes to OpenTelemetry instrumentation or PostHog analytics

https://claude.ai/code/session_01LwHaNaCED7DW1QbD7f1D79

Sentry SDK dependency and all related code removed:
- Delete core/telemetry/sentry.py module
- Remove sentry imports/exports from telemetry __init__.py
- Remove sentry-sdk from pyproject.toml optional dependencies
- Remove add_breadcrumb/capture_exception calls from computer and agent modules
- Remove sentry-sdk package from uv.lock

https://claude.ai/code/session_01LwHaNaCED7DW1QbD7f1D79
@vercel
Copy link
Contributor

vercel bot commented Feb 8, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Feb 8, 2026 7:12am

Request Review

@coderabbitai
Copy link

coderabbitai bot commented Feb 8, 2026

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • rabbit

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch claude/remove-sentry-qp1fD

Comment @coderabbitai help to get the list of available commands and usage tips.

@sentry
Copy link

sentry bot commented Feb 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants