v12: Remove the deprecated `withSentry` root export from `@sentry/solidstart`

Ouverte
#25,915 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
4/5
Temps estimé
3-5 jours
Accessibilité débutants
45/100
Type d'issue
Refactorisation
Clarté
Clairement spécifiée
Activité
Active
Stack technique
javascript, rollup, typescript, vite

Piste de recherche

Start at the @sentry/solidstart root entry and trace its withSentry export to identify the build-time types and runtime dependency path. Check the server runtime entry for imports of @sentry/bundler-plugins and the orchestrion Rollup transform. Update MIGRATION.md and the SolidStart guide at docs/platforms/javascript/guides/solidstart/index.mdx; done means the root export is removed, the documented import uses @sentry/solidstart/vite, and the runtime no longer pulls in build tooling.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

javascript

Problem

withSentry wraps the SolidStart config at build time. The root entry of @sentry/solidstart re-exports it. Because of this, the server runtime entry reaches @sentry/bundler-plugins and the orchestrion rollup transform. Any tool that traces or bundles the server entry then pulls in that build-time tooling.

getsentry/sentry-javascript#25725 adds withSentry to @sentry/solidstart/vite, next to the Vite plugin. The root export stays, deprecated, so existing app.config.ts files keep working until the next major version.

Proposal

In v12, remove withSentry (and its related build-time types) from the root entry of @sentry/solidstart. Users must import it from @sentry/solidstart/vite.

// before
import { withSentry } from "@sentry/solidstart";

// after
import { withSentry } from "@sentry/solidstart/vite";

Tasks

  • Remove the withSentry re-export from the root entry.
  • Confirm that the server runtime entry no longer imports @sentry/bundler-plugins or the orchestrion rollup transform.
  • Add the change to the v11 to v12 migration guide (sentry-javascript MIGRATION.md and sentry-docs).
  • In sentry-docs, remove the Alert in docs/platforms/javascript/guides/solidstart/index.mdxthat explains the old import path.

References

  • getsentry/sentry-javascript#25725
  • #25676
  • getsentry/sentry-javascript#25676
Langage dominant
TypeScript
Étoiles
8.8k
Forks
1.9k
Merge moyen
1 j 21 h
PR mergées (30 j)
617

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de getsentry/sentry-javascript

Toutes les issues de getsentry/sentry-javascript

Issues similaires

Plus d'issues TypeScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.