perf(compile): avoid blocking fs reads in the async compile path for the package-config check
Maintainer thường phản hồi trong vòng 2 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
- 72/100
Hướng nghiên cứu
Tìm package_config_check::collect_warnings và kiểm tra cách nó được gọi từ luồng compile bất đồng bộ. Quyết định có nên chuyển tác vụ quét hệ thống tệp có giới hạn ra khỏi executor bằng tokio::fs hoặc spawn_blocking hay không, hoặc ghi lại lý do I/O chặn là chấp nhận được. Công việc hoàn tất khi một trong các lựa chọn đó được triển khai mà không làm thay đổi hành vi và các bài kiểm thử hiện có đều đạt.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Follow-up from the #2315 review (Rust reviewer, unposted theme).
package_config_check::collect_warnings runs on the async compile path but uses blocking std::fs (read_dir, metadata, read_to_string) to scan at most nine small files at the repository root.
The cost is negligible today, and compile does other synchronous I/O. Even so, the scan should either use tokio::fs or spawn_blocking, or document why blocking I/O is acceptable here, so the pattern doesn't spread to larger scans.
Acceptance criteria
- Either the scan runs off the async executor, or a comment justifies the bounded blocking I/O.
- No behaviour change, and the existing tests pass.
- Ngôn ngữ chính
- Rust
- Star
- 27
- Fork
- 8
- Merge trung bình
- 3 ngày 18 giờ
- Pull request đã merge (30 ngày)
- 10
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Không 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 githubnext/ado-aw
-
bug rust
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
githubnext/ado-aw#2359 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
rust test
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
githubnext/ado-aw#2357 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
docs documentation
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
githubnext/ado-aw#1766 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
agentic-workflows
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
githubnext/ado-aw#1041 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
security
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
githubnext/ado-aw#855 ·
Maintainer thường phản hồi trong vòng 2 ngày
Tất cả issue của githubnext/ado-aw
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
home-assistant/installer#308 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
swc-project/swc-next#1053 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
What happened to `check.py`?Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
linebender/resvg#1179 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
enhancement triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
zizmorcore/zizmor#2468 ·
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 64/100
Maintainer thường phản hồi trong vòng 2 ngày