🔧 fix(type generator): handle non-intersect routes

Author: SaltyAom

CHANGELOG.md

@@ -1,3 +1,15 @@
+# 1.4.4 - 20 Sep 2025
+Improvement:
+- cast `exclude.methods` to lowercase when checking for method exclusion
+- type generator: handle non-intersect routes eg. group/guard
+
+Change:
+- type generator: enable type error log
+- type generator: do not remove temp files when debug is enabled
+
+Bug fix:
+- exclude `options` method by default
+
 # 1.4.3 - 18 Sep 2025
 Improvement:
 - unwrap model reference into parameter schema

example/gen.ts

@@ -1,43 +1,44 @@
-import { Elysia, t } from 'elysia'
-import { openapi, withHeaders } from '../src/index'
-import { fromTypes } from '../src/gen'
-
-export const app = new Elysia()
-	.use(
-		openapi({
-			references: fromTypes('example/gen.ts', {
-				debug: true
-			})
-		})
-	)
-	.get(
-		'/',
-		() =>
-			({ test: 'hello' as const }) as any as
-				| { test: 'hello' }
-				| undefined,
-		{
-			response: {
-				204: withHeaders(
-					t.Void({
-						title: 'Thing',
-						description: 'Void response'
-					}),
-					{
-						'X-Custom-Header': t.Literal('Elysia')
-					}
-				)
-			}
-		}
-	)
-	.post(
-		'/json',
-		({ body, status }) => (Math.random() > 0.5 ? status(418) : body),
-		{
-			body: t.Object({
-				hello: t.String()
-			})
-		}
-	)
-	.get('/id/:id/name/:name', ({ params }) => params)
-	.listen(3000)
+import { Elysia, t } from 'elysia'
+import { openapi, withHeaders } from '../src/index'
+import { fromTypes } from '../src/gen'
+
+export const app = new Elysia()
+	.use(
+		openapi({
+			references: fromTypes('example/gen.ts', {
+				debug: true
+			})
+		})
+	)
+	.get(
+		'/',
+		() =>
+			({ test: 'hello' as const }) as any as
+				| { test: 'hello' }
+				| undefined,
+		{
+			response: {
+				204: withHeaders(
+					t.Void({
+						title: 'Thing',
+						description: 'Void response'
+					}),
+					{
+						'X-Custom-Header': t.Literal('Elysia')
+					}
+				)
+			}
+		}
+	)
+	.post(
+		'/json',
+		({ body, status }) => (Math.random() > 0.5 ? status(418) : body),
+		{
+			body: t.Object({
+				hello: t.String()
+			})
+		}
+	)
+	.get('/id/:id/name/:name', ({ params }) => params)
+	.get('/a', () => 'hello')
+	.listen(3000)

package.json

@@ -1,6 +1,6 @@
 {
     "name": "@elysiajs/openapi",
-    "version": "1.4.3",
+    "version": "1.4.4",
     "description": "Plugin for Elysia to auto-generate API documentation",
     "author": {
         "name": "saltyAom",

src/gen/index.ts

@@ -73,6 +73,53 @@ interface OpenAPIGeneratorOptions {
 	 * ! be careful that the folder will be removed after the process ends
 	 */
 	tmpRoot?: string
+
+	/**
+	 * disable log
+	 * @default false
+	 */
+	silent?: boolean
+}
+
+function extractRootObjects(code: string) {
+	const results = []
+	let i = 0
+
+	while (i < code.length) {
+		// find the next colon
+		const colonIdx = code.indexOf(':', i)
+		if (colonIdx === -1) break
+
+		// backtrack to find the key (simple word characters)
+		let keyEnd = colonIdx - 1
+		while (keyEnd >= 0 && /\s/.test(code[keyEnd])) keyEnd--
+		let keyStart = keyEnd
+		while (keyStart >= 0 && /\w/.test(code[keyStart])) keyStart--
+
+		// find the opening brace after colon
+		const braceIdx = code.indexOf('{', colonIdx)
+		if (braceIdx === -1) break
+
+		// scan braces
+		let depth = 0
+		let end = braceIdx
+		for (; end < code.length; end++) {
+			if (code[end] === '{') depth++
+			else if (code[end] === '}') {
+				depth--
+				if (depth === 0) {
+					end++ // move past closing brace
+					break
+				}
+			}
+		}
+
+		results.push(`{${code.slice(keyStart + 1, end)};}`)
+
+		i = end
+	}
+
+	return results
 }
 
 /**
@@ -97,7 +144,8 @@ export const fromTypes =
 			overrideOutputPath,
 			debug = false,
 			compilerOptions,
-			tmpRoot = join(tmpdir(), '.ElysiaAutoOpenAPI')
+			tmpRoot = join(tmpdir(), '.ElysiaAutoOpenAPI'),
+			silent = false
 		}: OpenAPIGeneratorOptions = {}
 	) =>
 	() => {
@@ -177,7 +225,7 @@ export const fromTypes =
 				spawnSync(`tsc`, {
 					shell: true,
 					cwd: tmpRoot,
-					stdio: debug ? 'inherit' : undefined
+					stdio: silent ? undefined : 'inherit'
 				})
 
 				const fileName = targetFilePath
@@ -235,7 +283,7 @@ export const fromTypes =
 							console.warn(tempFiles)
 						}
 					} else {
-						console.log(
+						console.warn(
 							"reason: root folder doesn't exists",
 							join(tmpRoot, 'dist')
 						)
@@ -247,8 +295,10 @@ export const fromTypes =
 
 			const declaration = readFileSync(targetFile, 'utf8')
 
+			// console.log(declaration, targetFile)
+
 			// Check just in case of race-condition
-			if (existsSync(tmpRoot))
+			if (!debug && existsSync(tmpRoot))
 				rmSync(tmpRoot, { recursive: true, force: true })
 
 			let instance = declaration.match(
@@ -282,11 +332,8 @@ export const fromTypes =
 			const routes: AdditionalReference = {}
 
 			// Treaty is a collection of { ... } & { ... } & { ... }
-			// Each route will be intersected with each other
-			// instead of being nested in a route object
-			for (const route of routesString.slice(1).split('} & {')) {
-				// as '} & {' is removed, we need to add it back
-				let schema = TypeBox(`{${route}}`)
+			for (const route of extractRootObjects(routesString)) {
+				let schema = TypeBox(route)
 				if (schema.type !== 'object') continue
 
 				const paths = []
@@ -302,6 +349,9 @@ export const fromTypes =
 				}
 
 				const method = paths.pop()!
+				// For whatever reason, if failed to infer route correctly
+				if (!method) continue
+
 				const path = '/' + paths.join('/')
 				schema = schema.properties
 

src/index.ts

@@ -7,7 +7,7 @@ import { toOpenAPISchema } from './openapi'
 
 import type { OpenAPIV3 } from 'openapi-types'
 import type { ApiReferenceConfiguration } from '@scalar/types'
-import type { ElysiaOpenAPIConfig, OpenAPIProvider } from './types'
+import type { ElysiaOpenAPIConfig } from './types'
 
 /**
  * Plugin for [elysia](https://github.com/elysiajs/elysia) that auto-generate OpenAPI documentation page.

src/openapi.ts

@@ -87,7 +87,8 @@ openapi({
     zod: zodToJsonSchema
   }
 })`,
-	valibot: `import { toJsonSchema } from '@valibot/to-json-schema'
+	valibot: `import openapi from '@elysiajs/openapi'
+import { toJsonSchema } from '@valibot/to-json-schema'
 
 openapi({
   mapJsonSchema: {
@@ -204,12 +205,14 @@ export function toOpenAPISchema(
 	references?: AdditionalReferences,
 	vendors?: MapJsonSchema
 ) {
-	const {
-		methods: excludeMethods = ['OPTIONS'],
+	let {
+		methods: excludeMethods = ['options'],
 		staticFile: excludeStaticFile = true,
 		tags: excludeTags
 	} = exclude ?? {}
 
+	excludeMethods = excludeMethods.map((method) => method.toLowerCase())
+
 	const excludePaths = Array.isArray(exclude?.paths)
 		? exclude.paths
 		: typeof exclude?.paths !== 'undefined'
@@ -249,6 +252,10 @@ export function toOpenAPISchema(
 			detail: Partial<OpenAPIV3.OperationObject>
 		} = route.hooks ?? {}
 
+		if (route.path === '/a') {
+			console.log('H')
+		}
+
 		if (references?.length)
 			for (const reference of references as AdditionalReference[]) {
 				if (!reference) continue
@@ -327,7 +334,10 @@ export function toOpenAPISchema(
 
 		// Handle query parameters
 		if (hooks.query) {
-			const query = unwrapReference(unwrapSchema(hooks.query, vendors), definitions)
+			const query = unwrapReference(
+				unwrapSchema(hooks.query, vendors),
+				definitions
+			)
 
 			if (query && query.type === 'object' && query.properties) {
 				const required = query.required || []
@@ -345,7 +355,10 @@ export function toOpenAPISchema(
 
 		// Handle header parameters
 		if (hooks.headers) {
-			const headers = unwrapReference(unwrapSchema(hooks.query, vendors), definitions)
+			const headers = unwrapReference(
+				unwrapSchema(hooks.query, vendors),
+				definitions
+			)
 
 			if (headers && headers.type === 'object' && headers.properties) {
 				const required = headers.required || []
@@ -363,7 +376,10 @@ export function toOpenAPISchema(
 
 		// Handle cookie parameters
 		if (hooks.cookie) {
-			const cookie = unwrapReference(unwrapSchema(hooks.cookie, vendors), definitions)
+			const cookie = unwrapReference(
+				unwrapSchema(hooks.cookie, vendors),
+				definitions
+			)
 
 			if (cookie && cookie.type === 'object' && cookie.properties) {
 				const required = cookie.required || []