Skip to content

Commit

Permalink
feat: Update nextjs, remix, sveltekit and nuxt wizards to ins…
Browse files Browse the repository at this point in the history
…tall v9 (#794)

* Update nextjs wizard to v9

* Update remix wizard to v9

* Update sveltekit wizard to v9

* Update nuxt wizard to v9

* Add changelog entry
  • Loading branch information
andreiborza authored Feb 17, 2025
1 parent 116b0b6 commit 50ddd0b
Show file tree
Hide file tree
Showing 8 changed files with 11 additions and 15 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# Changelog

## Unreleased

- feat: Update `nextjs`, `remix`, `sveltekit` and `nuxt` wizards to install v9 ([#794](https://github.com/getsentry/sentry-wizard/pull/794))

## 3.41.0

- feat: Add `forceInstall` option to NPM-based wizards ([#791](https://github.com/getsentry/sentry-wizard/pull/791))
Expand Down
3 changes: 1 addition & 2 deletions e2e-tests/tests/remix.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -184,8 +184,7 @@ function checkRemixProject(
'import * as Sentry from "@sentry/remix";',
`Sentry.init({
dsn: "${TEST_ARGS.PROJECT_DSN}",
tracesSampleRate: 1,
autoInstrumentRemix: true
tracesSampleRate: 1
})`,
]);
});
Expand Down
2 changes: 1 addition & 1 deletion src/nextjs/nextjs-wizard.ts
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ export async function runNextjsWizardWithTelemetry(

const { packageManager: packageManagerFromInstallStep } =
await installPackage({
packageName: '@sentry/nextjs@^8',
packageName: '@sentry/nextjs@^9',
packageNameDisplayLabel: '@sentry/nextjs',
alreadyInstalled: !!packageJson?.dependencies?.['@sentry/nextjs'],
forceInstall,
Expand Down
2 changes: 1 addition & 1 deletion src/nuxt/nuxt-wizard.ts
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,7 @@ export async function runNuxtWizardWithTelemetry(
Sentry.setTag('sdk-already-installed', sdkAlreadyInstalled);

await installPackage({
packageName: '@sentry/nuxt',
packageName: '@sentry/nuxt@^9',
alreadyInstalled: sdkAlreadyInstalled,
packageManager,
forceInstall,
Expand Down
2 changes: 1 addition & 1 deletion src/remix/remix-wizard.ts
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ async function runRemixWizardWithTelemetry(
await getOrAskForProjectData(options, 'javascript-remix');

await installPackage({
packageName: '@sentry/remix@^8',
packageName: '@sentry/remix@^9',
packageNameDisplayLabel: '@sentry/remix',
alreadyInstalled: hasPackageInstalled('@sentry/remix', packageJson),
forceInstall,
Expand Down
5 changes: 0 additions & 5 deletions src/remix/sdk-setup.ts
Original file line number Diff line number Diff line change
Expand Up @@ -124,11 +124,6 @@ function getInitCallArgs(
}
}

// Adding autoInstrumentRemix option only for server
if (type === 'server') {
initCallArgs.autoInstrumentRemix = true;
}

return initCallArgs;
}

Expand Down
2 changes: 1 addition & 1 deletion src/sveltekit/sveltekit-wizard.ts
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ export async function runSvelteKitWizardWithTelemetry(
Sentry.setTag('sdk-already-installed', sdkAlreadyInstalled);

await installPackage({
packageName: '@sentry/sveltekit@^8',
packageName: '@sentry/sveltekit@^9',
packageNameDisplayLabel: '@sentry/sveltekit',
alreadyInstalled: sdkAlreadyInstalled,
forceInstall,
Expand Down
6 changes: 2 additions & 4 deletions test/remix/server-instrumentation.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,7 @@ describe('generateServerInstrumentationFile', () => {
Sentry.init({
dsn: "https://sentry.io/123",
tracesSampleRate: 1,
autoInstrumentRemix: true
tracesSampleRate: 1
})"
`);
});
Expand All @@ -30,8 +29,7 @@ describe('generateServerInstrumentationFile', () => {
"import * as Sentry from "@sentry/remix";
Sentry.init({
dsn: "https://sentry.io/123",
autoInstrumentRemix: true
dsn: "https://sentry.io/123"
})"
`);
});
Expand Down

0 comments on commit 50ddd0b

Please sign in to comment.