server: wait for the stores to disconnect on stop

CHANGELOG.md

@@ -8,6 +8,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/).
 
 - fix: reject create conflicts and duplicate permission targets
 - fix(sql): seed ns export types coredns & native for 2.x upgrade
+- fix: server stop waits for the stores to disconnect
 
 ### [3.0.3] - 2026-07-27
 

routes/index.js

@@ -96,7 +96,7 @@
     },
     httpAuthScheme: 'Bearer',
     headerName: 'authorization',
     validate: (artifacts, request, h) => {
       return {
         isValid: true,
         credentials: artifacts.decoded.payload.nt,
@@ -155,7 +155,7 @@
     }
   })
 
-  server.events.on('stop', async () => {
+  server.ext('onPostStop', async () => {
     await User.disconnect()
     await Session.disconnect()
   })

routes/lifecycle.test.js

@@ -0,0 +1,34 @@
+import assert from 'node:assert/strict'
+import { test } from 'node:test'
+
+import User from '../lib/user/index.js'
+import { init } from './index.js'
+
+test('server stop waits for store disconnection', async (t) => {
+  const server = await init()
+  let enterDisconnect
+  let releaseDisconnect
+  const entered = new Promise((resolve) => {
+    enterDisconnect = resolve
+  })
+  const released = new Promise((resolve) => {
+    releaseDisconnect = resolve
+  })
+
+  t.mock.method(User, 'disconnect', async () => {
+    enterDisconnect()
+    await released
+  })
+
+  let stopped = false
+  const stopping = server.stop().then(() => {
+    stopped = true
+  })
+  await entered
+  await new Promise((resolve) => setImmediate(resolve))
+  assert.equal(stopped, false)
+
+  releaseDisconnect()
+  await stopping
+  assert.equal(stopped, true)
+})