Build script fails on Windows due to CRLF line endings in pnpm-workspace.yaml
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 85/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- node.js, typescript, yaml
- Lĩnh vực
- build-system, tooling
Hướng nghiên cứu
Bắt đầu với scripts/verify-packages-versions.js, đặc biệt là phần phân tích YAML gần dòng được liên kết. Kiểm tra cách tập lệnh xử lý pnpm-workspace.yaml với ký tự kết thúc dòng CRLF, sau đó chạy bản dựng với định dạng kết thúc dòng đó. Hoàn thành khi các danh sách gói của workspace được so sánh chính xác và bản dựng thành công trên Windows.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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.
- Ngôn ngữ chính
- TypeScript
- Star
- 8.8k
- Fork
- 1.9k
- Merge trung bình
- 1 ngày 19 giờ
- Pull request đã merge (30 ngày)
- 664
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của getsentry/sentry-javascript
-
Vercel AI: `gen_ai.system_instructions` is missing when the system prompt is a message objectĐang mởBug Package: node Spans vercel ai
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 73/100
getsentry/sentry-javascript#26079 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Bug Node.js
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
getsentry/sentry-javascript#25950 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
normalize() throws when a getter throws a Symbol or a null-prototype objectCó thể đã có người làm @rjmcgee53192 đã nhận 4 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
getsentry/sentry-javascript#25920 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Only create Nuxt database and storage spans when there is a parent spanCó thể đã có người làm @nicohrubec đã nhận 5 ngày trước. Đang mởImprovement Nuxt
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 83/100
getsentry/sentry-javascript#25805 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Only create Cloudflare storage and D1 db spans when there is a parent spanCó thể đã có người làm @nicohrubec đã nhận 5 ngày trước. Đang mởCloudflare Workers Improvement Spans
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
getsentry/sentry-javascript#25804 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của getsentry/sentry-javascript
Issue tương tự
-
Add: Yichang TVĐang mởcheck:passed streams:add
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
Maintainer thường phản hồi trong vòng 1 ngày
-
vtselect: data race in Tempo search tags APIs for invalid TraceQL queriesCó thể đã có người làm @Shubham19032004 đã nhận hôm nay. Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
VictoriaMetrics/VictoriaTraces#304 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
test: claim-conflict drop leaves both facts active and is never re-proposed (untested review cell)Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 83/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
microsoft/vscode-pull-request-github#9159 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[Feature]: Usage page should show API-equivalent cost and say when a connected plan shows info onlyĐang mởenhancement from-x
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
nightly-labs/openbot#4867 ·
Maintainer thường phản hồi trong vòng 1 ngày