fix: dev hooks not executing

Author: Julien-R44

src/dev_server.ts

@@ -300,6 +300,8 @@ export class DevServer {
    * Start the development server
    */
   async start() {
+    await this.#hooks.registerDevServerHooks()
+
     this.#clearScreen()
     this.#logger.info('starting HTTP server...')
     this.#startHTTPServer(String(await getPort(this.#cwd)), 'nonblocking')
@@ -310,6 +312,8 @@ export class DevServer {
    * Start the development server in watch mode
    */
   async startAndWatch(ts: typeof tsStatic, options?: { poll: boolean }) {
+    await this.#hooks.registerDevServerHooks()
+
     const port = String(await getPort(this.#cwd))
     this.#isWatching = true
 

src/hooks.ts

@@ -1,3 +1,12 @@
+/*
+ * @adonisjs/assembler
+ *
+ * (c) AdonisJS
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
 import {
   AssemblerHookNode,
   SourceFileChangedHookHandler,

tests/dev_server.spec.ts

@@ -0,0 +1,101 @@
+/*
+ * @adonisjs/assembler
+ *
+ * (c) AdonisJS
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+import ts from 'typescript'
+import { test } from '@japa/runner'
+import { DevServer } from '../index.js'
+import { setTimeout as sleep } from 'node:timers/promises'
+
+test.group('DevServer', () => {
+  test('start() execute onDevServerStarted hook', async ({ assert, fs }) => {
+    assert.plan(1)
+    await fs.create('bin/server.js', `process.send({ isAdonisJS: true, environment: 'web' })`)
+
+    const devServer = new DevServer(fs.baseUrl, {
+      assets: {
+        enabled: false,
+      },
+      nodeArgs: [],
+      scriptArgs: [],
+      hooks: {
+        onDevServerStarted: [
+          async () => ({
+            default: () => {
+              assert.isTrue(true)
+            },
+          }),
+        ],
+      },
+    })
+
+    await devServer.start()
+    await sleep(600)
+  })
+
+  test('startAndWatch() execute onDevServerStarted hook', async ({ assert, fs, cleanup }) => {
+    assert.plan(1)
+    await fs.create('bin/server.js', `process.send({ isAdonisJS: true, environment: 'web' })`)
+
+    const devServer = new DevServer(fs.baseUrl, {
+      assets: {
+        enabled: false,
+      },
+      nodeArgs: [],
+      scriptArgs: [],
+      hooks: {
+        onDevServerStarted: [
+          async () => ({
+            default: () => {
+              assert.isTrue(true)
+            },
+          }),
+        ],
+      },
+    })
+
+    await devServer.startAndWatch(ts)
+    cleanup(() => devServer.close())
+    await sleep(600)
+  })
+
+  test('execute onSourceFileChanged hook', async ({ assert, fs, cleanup }) => {
+    assert.plan(1)
+
+    await fs.createJson('tsconfig.json', {
+      include: ['**/*'],
+      exclude: [],
+    })
+    await fs.create('index.ts', 'console.log("hey")')
+    await fs.create('bin/server.js', `process.send({ isAdonisJS: true, environment: 'web' })`)
+
+    const devServer = new DevServer(fs.baseUrl, {
+      assets: {
+        enabled: false,
+      },
+      nodeArgs: [],
+      scriptArgs: [],
+      hooks: {
+        onSourceFileChanged: [
+          async () => ({
+            default: () => {
+              assert.isTrue(true)
+            },
+          }),
+        ],
+      },
+    })
+
+    await devServer.startAndWatch(ts)
+    cleanup(() => devServer.close())
+
+    await sleep(100)
+    await fs.create('index.ts', 'sdf')
+    await sleep(10)
+  })
+})