Skip to content

Add optional Sentry error reporting (env-driven, opt-in) #2139

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.

Sign up for GitHub

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

Merged
kevincodex1 merged 5 commits into from
Aug 19, 2026
Merged

Add optional Sentry error reporting (env-driven, opt-in) #2139

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
3199e79
Add optional Sentry error reporting (env-driven, opt-in)
anushkadas-coder Aug 17, 2026
e7fec01
Fix Sentry init to use dynamic import instead of require (ESM compati…
anushkadas-coder Aug 17, 2026
dd4ba7d
Document SENTRY_DSN setup in advanced-setup docs
anushkadas-coder Aug 17, 2026
040f7f7
Disable Sentry default integrations; document runtime install require…
anushkadas-coder Aug 17, 2026
6454dec
Wire reportErrorToSentry into top-level error handlers; add sentry.te…
anushkadas-coder Aug 19, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
63 changes: 63 additions & 0 deletions bun.lock
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,7 @@
"devDependencies": {
"@alcalzone/ansi-tokenize": "0.3.0",
"@anthropic-ai/bedrock-sdk": "0.29.1",
"@sentry/node": "^10.70.0",
"@anthropic-ai/foundry-sdk": "0.2.3",
"@anthropic-ai/sandbox-runtime": "0.0.55",
"@anthropic-ai/sdk": "0.94.0",
Expand Down
7 changes: 7 additions & 0 deletions scripts/externals.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,10 @@
export const COMMON_EXTERNALS: string[] = [
// Native image processing
'sharp',
// Optional Sentry error reporting — dynamically required in utils/sentry.ts
// only when SENTRY_DSN is set. Not shipped by default; kept external so
// esbuild doesn't try to inline it.
'@sentry/node',
// Cloud provider SDKs
'@aws-sdk/client-bedrock',
'@aws-sdk/client-bedrock-runtime',
Expand Down Expand Up @@ -88,6 +92,9 @@ export const OPTIONAL_RUNTIME_EXTERNALS: string[] = [
// Optional: only image reads need it, and it carries a native install
// script. Kept opt-in so default installs run no install scripts.
'sharp',
// Sentry error reporting — loaded via require() in utils/sentry.ts only
// when SENTRY_DSN is set. Optional: most users never enable this.
'@sentry/node',
]

// OPTIONAL_RUNTIME_EXTERNALS that are loaded ONLY through the runtime importer
Expand Down
Loading