NativeScript / NativeScript/nativescript-cli

Fix cleanup request removal presence check logging

Abierto
#6,090 0 comentarios 0 reacciones 1 asignado Ver en GitHub

@edusperoni ya está trabajando en esto.

Desde el 25/7/2026.

Lenguaje dominante
JavaScript
Estrellas
1.1k
Forks
204
Merge medio
1 d 9 h
PR fusionados (30 d)
8

Descripción

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

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.