fix(dev): do not end the dev session when a replaced fork exits non-zero

Author: danielroe

packages/nuxt-cli/src/dev/pool.ts

@@ -244,6 +244,9 @@ export class ForkPool {
 privatekillFork(fork: PooledFork,signal: NodeJS.Signals|number='SIGTERM'): Promise<void>{
 constwasAlive=fork.state!=='dead'&&!!fork.process&&fork.process.exitCode===null
 fork.state='dead'
+// A fork we are shutting down on purpose must not end the session, however
+// it exits on the way out.
+fork.serving=false
 if(fork.process){
 // signal 0 only probes for liveness, so map the `exit` case onto a real signal
 fork.process.kill(signal===0 ? 'SIGTERM' : signal)

packages/nuxt-cli/test/unit/pool.spec.ts

@@ -91,6 +91,20 @@ describe('fork pool', () => {
 exit.mockRestore()
 })
 
+it('should not end the session when the serving fork is closed deliberately',async()=>{
+constexit=vi.spyOn(process,'exit').mockImplementation(()=>undefinedasnever)
+constactive=awaitcreatePool().getFork(context)
+active.promote()
+
+constchild=forks.find(f=>f.pid===active.pid)!
+constclosing=active.close()
+child.emit('close',1,null)
+awaitclosing
+
+expect(exit).not.toHaveBeenCalled()
+exit.mockRestore()
+})
+
 it('should reject when the fork exits before it starts',async()=>{
 fork.mockImplementation(()=>{
 constchild=newFakeFork()