Build script fails on Windows due to CRLF line endings in pnpm-workspace.yaml
維護者通常 1 天內回覆
還沒有人認領這個 Issue。
評估
- 難度
- 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 天 20 小時
- 30 天內合併 PR
- 682
環境準備
- 沒有 Dockerfile 或 Docker Compose 檔案
- 有 Pull Request 範本
- 閱讀貢獻指南
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
getsentry/sentry-javascript 的其他 Issue
-
Workers AI integration drops output of non-streaming chat responses in the OpenAI format (`choices`)未關閉Bug Cloudflare Workers Spans
難度 2/5 1-3 小時 新手友好度 78/100
getsentry/sentry-javascript#26120 · 1 則留言 ·
維護者通常 1 天內回覆
-
Bug Cloudflare Workers
難度 2/5 1-3 小時 新手友好度 78/100
getsentry/sentry-javascript#26091 · 1 則留言 ·
維護者通常 1 天內回覆
-
Vercel AI: `gen_ai.system_instructions` is missing when the system prompt is a message object可能已有人在做 @biggdawg320 於 1 天前認領。 未關閉Bug Package: node Spans vercel ai
難度 2/5 1-3 小時 新手友好度 73/100
getsentry/sentry-javascript#26079 · 1 則留言 ·
維護者通常 1 天內回覆
-
fsIntegration crashes in native ESM when patching the read-only fs namespace可能已有人在做 @chargome 於 1 天前認領。 未關閉Bug Node.js Waiting for: Product Owner
難度 2/5 1-3 小時 新手友好度 68/100
getsentry/sentry-javascript#25950 · 2 則留言 · 已指派 1 人 ·
維護者通常 1 天內回覆
-
normalize() throws when a getter throws a Symbol or a null-prototype object可能已有人在做 @rjmcgee53192 於 5 天前認領。 未關閉
難度 2/5 1-3 小時 新手友好度 78/100
getsentry/sentry-javascript#25920 · 2 則留言 ·
維護者通常 1 天內回覆
查看 getsentry/sentry-javascript 的全部 Issue
相似的 Issue
-
[fullsend] E2E: scorecard — filecheck GitLab entity metrics unavailable due to initialDelay timing未關閉e2e-failure ready-to-code
難度 2/5 1-3 小時 新手友好度 75/100
redhat-developer/rhdh-plugin-export-overlays#4698 · 1 則留言 ·
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 88/100
devswha/herdr-web-ui#2884 ·
-
難度 2/5 1-3 小時 新手友好度 78/100
維護者通常 1 天內回覆
-
b:minor bug
難度 2/5 1-3 小時 新手友好度 78/100
software-mansion/argent#1771 ·
維護者通常 1 天內回覆
-
Resources: New palettes of Xiamen可能已有人在做 @rmt-svc 今天認領。 未關閉resources
難度 1/5 1-3 小時 新手友好度 74/100
railmapgen/rmg-palette#2491 ·
維護者通常 4 天內回覆