docs: Add Deno import example for Supabase integration#17562
docs: Add Deno import example for Supabase integration#17562cursor[bot] wants to merge 2 commits intomasterfrom
Conversation
…gration The Supabase integration page's configuration snippet did not show the Sentry import statement. For Deno users, the import syntax is unique (npm: prefix) and must come before other imports for correct instrumentation. Add a PlatformSection with a Deno-specific example showing the correct import order. Also fix minor typo in the configuration section intro sentence. Co-Authored-By: Claude <noreply@anthropic.com> Co-authored-by: Sergiy Dybskiy <s@serg.tech>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
|
@cursoragent we have a pattern for dynamically importing the right version of the SDK import in other places, can we do that instead of just doing the Deno SDK? That snippet has a |
Code reference: import * as Sentry from "___SDK_PACKAGE___";
import { createClient } from '@supabase/supabase-js';
const supabaseClient = createClient('YOUR_SUPABASE_URL', 'YOUR_SUPABASE_KEY');
Sentry.init({
dsn: 'YOUR_DSN',
integrations: [
Sentry.supabaseIntegration({ supabaseClient })
],Status: Committed and pushed to
|
…to non-Deno config so pulls the correct SDK per platform. Keep Deno-specific import and ensure it appears before other imports.\n\nRefs DOCS-ANT\nCo-Authored-By: Claude <noreply@anthropic.com> Co-authored-by: Sergiy Dybskiy <s@serg.tech>
|
@cursoragent do we need the specific deno callout to yield the two separate platform includes blocks, feels redundant, no? |
|
Short answer: yes — we need the Deno-specific block. Reason:
Because our current placeholder system doesn’t add npm: for Deno, a single unified snippet would be copy/paste-wrong for Deno. Keeping two minimal PlatformSection blocks ensures both variants are correct and ready to paste. If you want to de-duplicate, we’d need a new placeholder (e.g., SDK_IMPORT_PATH and SUPABASE_IMPORT_PATH) that renders with npm: for Deno. That’s a small infra change but out of scope for this PR. |
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 9c0689c. Configure here.
| }); | ||
| ``` | ||
|
|
||
| </PlatformSection> |
There was a problem hiding this comment.
Duplicated code blocks risk diverging during maintenance
Low Severity
The Deno PlatformSection block duplicates the entire Sentry.init configuration from the non-Deno block, differing only in the import lines. If the init configuration (e.g., integration options, sample rate) is updated in one block, the other can easily be missed, leading to inconsistent documentation. The PR reviewer also flagged this redundancy. Other integration pages like postgresjs.mdx use this pattern but have completely different code per platform — here, the body is identical and only the imports differ.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 9c0689c. Configure here.




DESCRIBE YOUR PR
Addresses user feedback from docs.sentry.io (DOCS-ANT).
npm:@sentry/denoimport before the Supabase importPlatformSection notSupportedguardIS YOUR CHANGE URGENT?
PRE-MERGE CHECKLIST