Skip to content

Conversation

@priscilawebdev
Copy link
Member

@linear
Copy link

linear bot commented Nov 4, 2025

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 4, 2025
@priscilawebdev priscilawebdev marked this pull request as ready for review November 6, 2025 12:28
@priscilawebdev priscilawebdev requested a review from a team as a code owner November 6, 2025 12:28
textWithMarkupMatcher(
/const Sentry = require\("@sentry\/google-cloud-serverless"\);/
)
textWithMarkupMatcher(/import \* as Sentry from "@sentry\/google-cloud-serverless"/)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: ESM vs CJS Mismatch in SDK Snippet

The test expects an ESM import statement (import * as Sentry from "@sentry/google-cloud-serverless"), but the code generates a CJS require statement (const Sentry = require("@sentry/google-cloud-serverless");). The getSdkInitSnippet(params, 'gpc') call in onboarding.tsx does not pass a defaultMode parameter, so it defaults to CJS format. The test regex should be /const Sentry = require\("@sentry\/google-cloud-serverless"\);/ instead of /import \* as Sentry from "@sentry\/google-cloud-serverless"/.

Fix in Cursor Fix in Web

@priscilawebdev priscilawebdev merged commit 4fa11af into master Nov 7, 2025
48 checks passed
@priscilawebdev priscilawebdev deleted the priscila/ref/onboarding/split-gcpfunctions-onboarding-docs branch November 7, 2025 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants