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

未关闭
#25,915 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
4/5
预计耗时
3-5 天
新手友好度
45/100
Issue 类型
重构
描述清晰度
描述清楚
活跃度
活跃
技术栈
javascript, rollup, typescript, vite

调研方向

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.

由索引模型根据 Issue 内容生成。

描述

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
主要语言
TypeScript
星标
8.8k
派生
1.9k
平均合并
1 天 21 小时
30 天内合并 PR
617

环境准备

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

getsentry/sentry-javascript 的其他 Issue

查看 getsentry/sentry-javascript 的全部 Issue

相似的 Issue

更多 TypeScript Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。