Astro Integration throws error after upgrade to v8.3.0

[felixrichardt]

Is there an existing issue for this?

• I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
• I have reviewed the documentation https://docs.sentry.io/
• I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/astro

SDK Version

8.3.0

Framework Version

No response

Link to Sentry event

No response

SDK Setup

sentry({
            environment: process.env.NODE_ENV || 'development',
            dsn: __MY_DSN__,
            sourceMapsUploadOptions: {
                project: __MY_PROJECT__,
                authToken: process.env.SENTRY_AUTH_TOKEN,
            },
        }),

Steps to Reproduce

Upgrading @sentry/astro from v7.116.0 to v8.3.0

Expected Result

Upgrade from v7.116.0 to v8.3.0 should work as expected

Actual Result

After upgrading to v8.3.0 astro build/dev throws error:

[ERROR] vite_ssr_import_1.getClient(...).setupIntegrations is not a function

[mydea]

Hey, do you by chance have a longer error with a stack trace you could share with us?

[AbhiPrasad]

also @felixrichardt - could you share you package.json dependencies with us? Are you using any other @sentry/ packages?

[felixrichardt]

Hi, sry for delayed answer.

Stacktrace from astro just tells that there is a problem in generated chunk file without additional messages. Maybe I have to enable some additional options in astro config / vite build options to get more insights (?)

13:51:49 [ERROR] __vite_ssr_import_1__.getClient(...).setupIntegrations is not a function
  Stack trace:
    at /xxx/node_modules/vite/dist/node/chunks/dep-cNe07EU9.js:55058:9
    [...] See full stack trace in the browser, or rerun with --verbose.

affected line in chunk:

await initModule(context.global, ssrModule, ssrImportMeta, ssrImport, ssrDynamicImport, ssrExportAll);

package.json deps:

  "devDependencies": {
    "@astrojs/sitemap": "^3.1.5",
    "@cloudflare/workers-types": "^4.20240512.0",
    "autoprefixer": "^10.4.19",
    "cssnano": "^7.0.1",
    "postcss": "^8.4.38",
    "postcss-preset-env": "^9.5.13",
    "sass": "^1.77.2",
    "wrangler": "^3.57.1"
  },
  "dependencies": {
    "@astrojs/react": "^3.3.4",
    "@sentry/astro": "^8.3.0",
    "@spotlightjs/astro": "^1.0.20",
    "@types/react": "^18.3.2",
    "@types/react-dom": "^18.3.0",
    "astro": "^4.8.7",
    "bootstrap": "^5.3.3",
    "bootstrap-icons": "^1.11.3",
    "date-fns": "^3.6.0",
    "ical.js": "^2.0.1",
    "jquery": "^3.7.1",
    "medium-zoom": "^1.1.0",
    "react": "^18.3.1",
    "react-day-picker": "^8.10.1",
    "react-dom": "^18.3.1",
    "react-responsive": "^10.0.0"
  }

I'm just using astro package of sentry. No additional sentry packages are installed.

[AbhiPrasad]

If you remove @spotlightjs/astro does the error go away? If so, that might be the issue.

[felixrichardt]

Yeah indeed, removing spotlightjs solves it.
Are there any conflicts using both of them?

[AbhiPrasad]

1.x of spotlight only supports 7.x. We need to make this more clear in the migration docs.

There's a 2.x alpha of spotlight you can use in the meantime that will work with 8.x. We'll cut a stable release of that soon!

[AbhiPrasad]

getsentry/spotlight#387

https://www.npmjs.com/package/@spotlightjs/spotlight/v/2.0.0-alpha.1

[AbhiPrasad]

Closing this in favour of tracking spotlight v8 compat in the above linked issue.