Skip to content
This repository was archived by the owner on Jan 7, 2026. It is now read-only.

Remove all Sentry dependencies and references#613

Merged
bajtos merged 4 commits intomainfrom
copilot/fix-612
Aug 7, 2025
Merged

Remove all Sentry dependencies and references#613
bajtos merged 4 commits intomainfrom
copilot/fix-612

Conversation

Copy link
Contributor

Copilot AI commented Aug 5, 2025

This PR removes all Sentry error tracking dependencies from the spark-evaluate repository as requested. The changes include:

What was removed:

  • Dependency: Removed "@sentry/node": "^9.43.0" from package.json
  • Imports: Removed import * as Sentry from '@sentry/node' from 8 files across bin/ and lib/ directories
  • Initialization: Removed Sentry.init() calls from binary scripts
  • Error tracking: Removed all Sentry.captureException() calls while preserving console error logging
  • CI configuration: Removed '@sentry/' from Dependabot auto-approve workflow

Files modified:

  • package.json - Removed Sentry dependency
  • package-lock.json - Updated (59 packages removed)
  • .github/workflows/dependabot-auto-approve-minor.yml - Removed Sentry from auto-approve list
  • index.js - Removed Sentry import and 5 exception capture calls
  • bin/spark-evaluate.js - Removed Sentry import and initialization
  • bin/fetch-recent-miner-measurements.js - Removed Sentry import and initialization
  • bin/dry-run.js - Removed Sentry import, initialization, and exception capture
  • lib/platform-stats.js - Removed Sentry import and 2 exception captures
  • lib/provider-retrieval-result-stats.js - Removed Sentry import and exception capture
  • lib/evaluate.js - Removed Sentry import and 5 exception captures
  • lib/public-stats.js - Removed Sentry import and 2 exception captures

Testing:

  • All existing tests continue to pass (76 passing tests)
  • Linting passes without errors
  • Dependency checks pass
  • No Sentry references remain in the codebase

The error handling behavior is preserved - all error conditions that previously logged to Sentry now only log to the console, maintaining the same visibility for debugging while removing the external dependency.

Fixes #612.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

  • api.filspark.com
    • Triggering command: node /home/REDACTED/work/spark-evaluate/spark-evaluate/node_modules/.bin/mocha (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Co-authored-by: bajtos <1140553+bajtos@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove Sentry Remove all Sentry dependencies and references Aug 5, 2025
Copilot AI requested a review from bajtos August 5, 2025 15:24
@bajtos bajtos marked this pull request as ready for review August 6, 2025 08:25
@bajtos bajtos enabled auto-merge (squash) August 6, 2025 08:25
@bajtos bajtos self-requested a review August 6, 2025 08:26
@bajtos bajtos disabled auto-merge August 7, 2025 14:52
@bajtos bajtos merged commit dea2660 into main Aug 7, 2025
6 checks passed
@bajtos bajtos deleted the copilot/fix-612 branch August 7, 2025 15:14
@github-project-automation github-project-automation bot moved this to ✅ done in CheckerNetwork Aug 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Status: ✅ done

Development

Successfully merging this pull request may close these issues.

Remove Sentry

2 participants