Build script fails on Windows due to CRLF line endings in pnpm-workspace.yaml
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 85/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- node.js, typescript, yaml
- 領域
- build-system, tooling
調査の方向性
scripts/verify-packages-versions.jsから始め、特にリンク先の行付近にあるYAMLの解析を確認してください。スクリプトがCRLFの改行を含むpnpm-workspace.yamlをどのように処理するかを確認し、その改行形式でビルドを実行してください。workspaceのパッケージ一覧が正しく比較され、Windowsでビルドが成功すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
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/browser
SDK Version
N/A (monorepo root)
Framework Version
Node.js 24 on Windows
Link to Sentry event
No response
Reproduction Example/SDK Setup
No response
Steps to Reproduce
- Clone the sentry-javascript repository on a Windows machine.
- Ensure Git checks out pnpm-workspace.yaml with CRLF (\r\n) line endings.
- Run pnpm install and then pnpm run build
Expected Result
The build processes the workspace packages and runs successfully.
Actual Result
The build crashes immediately with the following error:
[INCONSISTENT WORKSPACES]: The packageslist in pnpm-workspace.yaml and theworkspaces list in package.json must contain the same packages in the same order.
I guess this happens because scripts/verify-packages-versions.js rigidly splits the YAML file using .split('\n'). On Windows, this reads the target string as 'packages:\r', causing the match to fail, returning an empty array and triggering the error.
Additional Context
Suggested Solution:
Update scripts/verify-packages-versions.js to handle both line endings by using a regex split: .split(/\r?\n/)
I'm happy to open a PR though.
Priority
React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding +1 or me too, to help us triage it.
- 主要言語
- TypeScript
- スター
- 8.8k
- フォーク
- 1.9k
- 平均マージ
- 1日 19時間
- マージ済み PR(30日)
- 664
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
getsentry/sentry-javascript のほかの issue
-
Bug Package: node Spans vercel ai
難易度 2/5 1〜3時間 初心者へのやさしさ 73/100
getsentry/sentry-javascript#26079 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Bug Node.js
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
getsentry/sentry-javascript#25950 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
normalize() throws when a getter throws a Symbol or a null-prototype object対応中かも @rjmcgee53192 が 4 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
getsentry/sentry-javascript#25920 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
Only create Nuxt database and storage spans when there is a parent span対応中かも @nicohrubec が 5 日前に担当しました。 オープンImprovement Nuxt
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
getsentry/sentry-javascript#25805 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
Only create Cloudflare storage and D1 db spans when there is a parent span対応中かも @nicohrubec が 5 日前に担当しました。 オープンCloudflare Workers Improvement Spans
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
getsentry/sentry-javascript#25804 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
getsentry/sentry-javascript の issue をすべて見る
似ている issue
-
test: claim-conflict drop leaves both facts active and is never re-proposed (untested review cell)オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
メンテナーはふだん 1 日以内に返信
-
[Feature]: Usage page should show API-equivalent cost and say when a connected plan shows info onlyオープンenhancement from-x
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
nightly-labs/openbot#4867 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
egonSchiele/agency-lang#1377 ·
メンテナーはふだん 1 日以内に返信
-
normalizePath removes the Windows device namespace prefix対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
caprover/caprover-frontend#273 ·
メンテナーはふだん 1 日以内に返信