Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Improper handling of graceful exits with npx #8164

Open
2 tasks done
greguz opened this issue Mar 14, 2025 · 2 comments
Open
2 tasks done

[BUG] Improper handling of graceful exits with npx #8164

greguz opened this issue Mar 14, 2025 · 2 comments
Labels
Bug thing that needs fixing Needs Triage needs review for next steps

Comments

@greguz
Copy link

greguz commented Mar 14, 2025

Is there an existing issue for this?

  • I have searched the existing issues

This issue exists in the latest npm version

  • I am using the latest npm

Current Behavior

Sending a SIGINT signal using npx node anything.js will result in a double SIGINT received by the application code.

Expected Behavior

Just one signal to be received by the application code.

Steps To Reproduce

console.log('starting up')

process.on('SIGINT', () => {
  console.log('shutting down')

  // simulate cleanup
  setTimeout(() => {
    console.log('cleanly exiting')
    process.exit(0)
  }, 5000)
})

// run forever
setInterval(() => {}, 1 << 30)
╭─greguz@prunix ~/Development/issue 
╰─$ npx node z.js
starting up
^Cshutting down
shutting down
cleanly exiting

Environment

  • npm: v11.2.0
  • Node.js: v23.10.0
  • OS Name: macOS 15.3.2 (24D81)
  • System Model Name: Apple M2 Pro
  • npm config:
; "user" config from /Users/greguz/.npmrc

//registry.npmjs.org/:_authToken = (protected)

; node bin location = /Users/greguz/.local/share/fnm/node-versions/v23.10.0/installation/bin/node
; node version = v23.10.0
; npm local prefix = /Users/greguz/Development/evologi/orderpod-dlr
; npm version = 11.2.0
; cwd = /Users/greguz/Development/evologi/orderpod-dlr
; HOME = /Users/greguz
; Run `npm config ls -l` to show all defaults.
@greguz greguz added Bug thing that needs fixing Needs Triage needs review for next steps labels Mar 14, 2025
@greguz greguz changed the title [BUG] Improper handling of graceful exits with tsx [BUG] Improper handling of graceful exits with npx Mar 14, 2025
@milaninfy
Copy link
Contributor

We have similer issue raised and triaged in past where npm was handling signals like SIGINT twice #5021

@greguz
Copy link
Author

greguz commented Mar 24, 2025

The fact that #5021 is still open doesn't make me feel any better 🙃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug thing that needs fixing Needs Triage needs review for next steps
Projects
None yet
Development

No branches or pull requests

2 participants