fix(openapi): handle same pattern for multiple endpoints

Julien-R44 · May 20, 2024 · cc62c62 · cc62c62
1 parent fb9e904
commit cc62c62
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 4 deletions.
diff --git a/packages/openapi/src/generator.ts b/packages/openapi/src/generator.ts
@@ -209,7 +209,7 @@ export class OpenApiGenerator {
     return {
       method,
       path: openApiPath,
-      spec: defu(this.metaStore.getComputed(options.path), {
+      spec: defu(this.metaStore.getComputed({ method, path: options.path }), {
         ...parameters,
         responses: objectMap(responses, (status, schema) => {
           return [

diff --git a/packages/openapi/src/meta_store.ts b/packages/openapi/src/meta_store.ts
@@ -12,13 +12,16 @@ export class MetaStore {
     return this.#store.get(key)
   }
 
-  getComputed(key: string) {
-    return this.#computedStore.get(key)
+  getComputed(options: { method: string; path: string }) {
+    return this.#computedStore.get(`${options.method.toUpperCase()}:${options.path}`)
   }
 
   compute() {
     for (const [route, value] of this.#store) {
-      this.#computedStore.set(route.toJSON().pattern, value)
+      const serialized = route.toJSON()
+      for (const method of serialized.methods) {
+        this.#computedStore.set(`${method}:${serialized.pattern}`, value)
+      }
     }
   }
 }

diff --git a/playgrounds/openapi/start/routes.ts b/playgrounds/openapi/start/routes.ts
@@ -34,6 +34,9 @@ router
     router.get('/random/:id', [MiscController, 'show']).openapi({
       description: 'Get a random thing by id',
     })
+    router.post('/random', [MiscController, 'store']).openapi({
+      description: 'Create a random thing',
+    })
   })
   .prefix('/misc')
   .openapi({ tags: ['misc'] })