NativeScript / NativeScript/nativescript-cli

Fix cleanup request removal presence check logging

Aberta
#6,090 0 comentários 0 reações 1 responsável Ver no GitHub

@edusperoni já está trabalhando nisso.

Desde 25/7/2026.

Linguagem predominante
JavaScript
Estrelas
1.1k
Forks
204
Merge médio
1d 9h
PRs com merge (30d)
8

Descrição

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.tsremoveRequest

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

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.