Fix cleanup request removal presence check logging

Đang mở

#6,090 mở ngày 25/7/2026 0 bình luận 0 reaction 1 người được giao Xem trên GitHub

@edusperoni đang làm issue này rồi.

Từ ngày 25/7/2026.

Đánh giá

Issue này chưa được đánh giá.

Mô tả

Summary

removeRequest in lib/detached-processes/cleanup-process.ts uses a self-comparison in its existence check:

The predicate compares currentRequestInfo with itself, so it evaluates to true whenever the request queue is non-empty.
The subsequent _.remove call already correctly compares each queued request against the supplied requestInfo.

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

removeRequest selects 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.

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

Hướng dẫn đóng góp

Mở 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 NativeScript/nativescript-cli

cli does not recognize physical iOS devices (since cli 9.1.0) Đang mở

in-progress

NativeScript/nativescript-cli#6140 · 2 bình luận · 1 người được giao · Mở ngày 31/8/2026
ns 9 running test after running the app starts the app not the test runner. Đang mở

in progress

NativeScript/nativescript-cli#6015 · 1 reaction · 1 người được giao · Mở ngày 1/5/2026
[note]: SPM preferred over Cocoapods going forward with plugins Đang mở

question

Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100

NativeScript/nativescript-cli#5992 · 5 bình luận · Mở ngày 26/3/2026
feat: default output could suppress xcode build logs, enable via flag Đang mở

feature-pending-triage

Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100

NativeScript/nativescript-cli#5975 · Mở ngày 27/2/2026
ns build android fails in CI environment without emulator Đang mở

bug-pending-triage

Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100

NativeScript/nativescript-cli#5921 · Mở ngày 25/11/2025

Tất cả issue của NativeScript/nativescript-cli

Issue tương tự

XIAO nRF52840 Series: Broken tables in wiki Đang mở

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100

Seeed-Studio/wiki-documents#5655 · 2 bình luận · Mở ngày 21/9/2026
1.4.2: a failed transport probe now crashes the process with ERR_UNHANDLED_REJECTION Đang mở

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100

faye/faye#553 · Mở ngày 21/9/2026
Edit: CW+ Đang mở

channels:edit check:passed

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100

iptv-org/database#35803 · Mở ngày 21/9/2026
Username & password should not autocomplete on new user form Đang mở

01 type: bug 30 needs: triage 99 tag: UX Accessibility

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100

plone/volto#8433 · Mở ngày 21/9/2026
`--profile` missing from `quarto call engine`, `call build-ts-extension`, and `call typst-gather` --help Đang mở

documentation

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100

quarto-dev/quarto-cli#14915 · Mở ngày 21/9/2026

Thêm issue về JavaScript