NativeScript / NativeScript/nativescript-cli
Fix cleanup request removal presence check logging
Đang mở
@edusperoni đang làm issue này rồi.
Từ ngày 25/7/2026.
- Ngôn ngữ chính
- JavaScript
- Star
- 1.1k
- Fork
- 204
- Merge trung bình
- 1 ngày 9 giờ
- Pull request đã merge (30 ngày)
- 8
Mô tả
Summary
removeRequest in lib/detached-processes/cleanup-process.ts uses a self-comparison in its existence check:
- The predicate compares
currentRequestInfowith itself, so it evaluates to true whenever the request queue is non-empty. - The subsequent
_.removecall already correctly compares each queued request against the suppliedrequestInfo.
As a result, removing a request that was never queued can log that it was removed when there are other queued requests. The request list itself is not incorrectly modified; the impact is limited to the success-versus-absent-request log branch.
Required change
Update the removeRequest existence predicate to compare currentRequestInfo with the incoming requestInfo, matching the predicate used by _.remove.
Affected area
lib/detached-processes/cleanup-process.ts—removeRequest
Acceptance criteria
removeRequestselects the successful-removal branch only when an equal queued request exists.- Removing an absent request logs the existing “cannot remove … has not been added before” message.
- Removing an existing request continues to remove it and log the existing successful-removal message.
Context
- Reported during PR #6089: https://github.com/NativeScript/nativescript-cli/pull/6089
- Original review discussion: https://github.com/NativeScript/nativescript-cli/pull/6089#discussion_r3650504137
- Requested by @edusperoni.
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.
Đánh giá
Issue này chưa được đánh giá.