fix: calling next after loading API once

.changeset/witty-glasses-fail.md

@@ -0,0 +1,5 @@
+---
+"@kopflos-cms/express": patch
+---
+
+After first request, the server would stop responding

packages/express/index.ts

@@ -12,15 +12,18 @@ import asyncMiddleware from 'middleware-async'
 export default (options: KopflosConfig): RequestHandler => {
   const kopflos = new Kopflos(options)
 
+  const loadApiGraphs = onetime(async (graphs: Required<KopflosConfig>['apiGraphs']) => {
+    await Kopflos.fromGraphs(kopflos, ...graphs)
+  })
+
   return Router()
-    .use(onetime(async (req, res, next) => {
-      if (options.apiGraphs) {
-        await Kopflos.fromGraphs(kopflos, ...options.apiGraphs)
-      } else {
+    .use((req, res, next) => {
+      if (!options.apiGraphs) {
         return next(new Error('No API graphs configured. In future release it will be possible to select graphs dynamically.'))
       }
-      next()
-    }))
+
+      loadApiGraphs(options.apiGraphs).then(next).catch(next)
+    })
     .use(rdfHandler({
       factory,
     }))

packages/express/test/index.test.ts

@@ -37,6 +37,16 @@ describe('@kopflos-cms/express', () => {
         .expect(200, 'not found')
     })
 
+    it('must not stall after first request', async () => {
+      await request(app)
+        .get('/not-found')
+        .expect(404)
+
+      await request(app)
+        .get('/not-found')
+        .expect(404)
+    })
+
     context('request to resource without explicit handler', () => {
       it('should should return Core representation', async () => {
         const response = await request(app)