An unset variable in path: puts the working directory on PATH
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
- 75/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ệ
- typescript
- Lĩnh vực
- cli, testing-qa
Hướng nghiên cứu
Tìm đoạn mã mở rộng các mục path: và kiểm tra cách các biến chưa được thiết lập trở thành các phần tử PATH. Thêm bài kiểm thử được đề xuất bằng cách sử dụng một $VAR đứng riêng, sau đó xác minh rằng phần mở rộng rỗng được bỏ qua và thư mục làm việc không được tìm kiếm; bài kiểm thử phải tạo ra kết quả command-not-found rõ ràng.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
An unset variable in a path: entry expands to an empty string, and an empty PATH
element means the current directory to POSIX. So this front matter:
path:
- $TOOL_DIR
silently puts the working directory on PATH when TOOL_DIR is not set, instead of
contributing nothing.
That is the wrong failure. A test that names a directory it cannot resolve should get a
clear "command not found", not a lookup that quietly searches somewhere the author never
named — and in a sandboxed run the working directory contains fixtures, which is exactly
where a stray executable could sit.
Proposal
Drop entries that expand to empty, rather than passing them through.
A test asserting that a bare $VAR does not put the working directory on PATH would pin
it: today that assertion fails.
Related
jlevy/tryscript#54 asks for requires:, which addresses the same underlying problem from
the other side — knowing which binary a run actually resolved rather than trusting PATH
composition to have gone as intended.
- Ngôn ngữ chính
- TypeScript
- Star
- 13
- Fork
- 1
- Merge trung bình
- 3 giờ 38 phút
- Pull request đã merge (30 ngày)
- 3
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
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 jlevy/tryscript
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 52/100
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 52/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
Tất cả issue của jlevy/tryscript
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
Eynzof/Hermes-CN-Desktop#610 ·
-
[Bug]: Matrix progress drafts fail with "Matrix runtime not initialized" during tool activity Đang mởbug clawsweeper:linked-pr-open clawsweeper:needs-live-repro clawsweeper:no-new-fix-pr impact:message-loss issue-rating: 🐚 platinum hermit P2 regression
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
-
Client support matrix inclusion Đang mởenhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
calcite-components needs triage refactor
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Esri/calcite-design-system#15203 ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 78/100
fullcalendar/fullcalendar#8106 ·