Release 6.3.7#213
Merged
Merged
Conversation
- Remove @sentry/node and @sentry/profiling-node dependencies - Remove Sentry initialization and error reporting code - Remove SENTRY_DSN constant and LANGTRACE_ERROR_REPORTING env var - Update README to remove error reporting documentation - Fixes TypeError: The URL must be of scheme file in Supabase Edge Functions Co-Authored-By: karthik@scale3labs.com <karthik@scale3labs.com>
Contributor
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Removes all Sentry integration from the TypeScript SDK to resolve Deno compatibility issues in Supabase Edge Functions. The original error
TypeError: The URL must be of scheme filewas caused by Sentry's profiling module (@sentry/profiling-node) attempting to use Node.js-specific file URL schemes that are incompatible with Deno runtime.Related Issue: User reported inability to use
@langtrase/typescript-sdk@6.3.5in Supabase Edge Functions due to Sentry profiling module failures.Changes Made
Core Changes
Removed Sentry imports from
src/init/init.ts:import * as Sentry from '@sentry/node'import { nodeProfilingIntegration } from '@sentry/profiling-node'import { SENTRY_DSN } from '@langtrace-constants/common'Removed Sentry initialization block including:
Removed SENTRY_DSN constant from
src/constants/common.tsRemoved LANGTRACE_ERROR_REPORTING environment variable handling
Dependencies
@sentry/node:^8.30.0@sentry/profiling-node:^8.30.0npm install(1619 lines removed)Documentation
LANGTRACE_ERROR_REPORTING=Falseenvironment variableVerification Steps Completed
✅
npm run lint- No linting errors✅
npm run build- TypeScript compilation successful✅ Comprehensive search for remaining Sentry references - None found in src/
✅ Git commit shows 5 files changed, 242 insertions(+), 1619 deletions(-)
Human Review Checklist
Critical Items:
Standard Review:
Link to Devin run: https://app.devin.ai/sessions/41ba048c88d14ed2928fc83b587cf5b9
Requested by: karthik@scale3labs.com
Note: This is a complete removal approach rather than conditional Deno detection. Consider if a more nuanced solution (e.g., runtime detection to disable only problematic Sentry features in Deno) would be preferable for maintaining error reporting in Node.js environments.