ts-procedures 5.12.0 → 5.14.0

package/src/implementations/http/hono-api/index.ts

@@ -19,22 +19,7 @@ export type { APIConfig, APIHttpRouteDoc, APIInput, HttpMethod }
 
 export type QueryParser = (queryString: string) => Record<string, unknown>
 
-/** Lazy-loaded qs module (optional peer dependency) */
-let _qsModule: { parse: (str: string, opts?: any) => any } | false | undefined
-
-async function loadQs(): Promise<{ parse: (str: string, opts?: any) => any } | undefined> {
-  if (_qsModule === undefined) {
-    try {
-      const mod = await import('qs')
-      _qsModule = mod.default ?? mod
-    } catch {
-      _qsModule = false
-    }
-  }
-  return _qsModule || undefined
-}
-
-/** Fallback query parser using native URLSearchParams */
+/** Default query parser using native URLSearchParams. */
 function parseQueryNative(queryString: string): Record<string, unknown> {
   const searchParams = new URLSearchParams(queryString)
   const result: Record<string, unknown> = {}
@@ -45,21 +30,6 @@ function parseQueryNative(queryString: string): Record<string, unknown> {
   return result
 }
 
-/**
- * Resolves the query parser once. Called during build() so handlers use a sync parser.
- * Priority: custom queryParser > qs (optional peer dep) > native URLSearchParams
- */
-async function resolveQueryParser(custom?: QueryParser): Promise<QueryParser> {
-  if (custom) return custom
-
-  const qs = await loadQs()
-  if (qs) {
-    return (raw: string) => qs.parse(raw) as Record<string, unknown>
-  }
-
-  return parseQueryNative
-}
-
 /** Extract path parameter names from a route pattern (e.g., '/users/:id' → ['id']) */
 function extractPathParamNames(path: string): string[] {
   const matches = path.match(/:([a-zA-Z_][a-zA-Z0-9_]*)/g)
@@ -104,7 +74,20 @@ export type HonoAPIAppBuilderConfig = {
   pathPrefix?: string
   /**
    * Custom query string parser. Receives the raw query string (without '?').
-   * Default: uses `qs` (optional peer dependency) if available, otherwise native URLSearchParams.
+   *
+   * Default: native `URLSearchParams`. The default handles:
+   *   - flat keys:      `?page=2&limit=10` → `{ page: '2', limit: '10' }`
+   *   - repeated keys:  `?tag=a&tag=b`     → `{ tag: ['a', 'b'] }`
+   *
+   * The default does NOT parse any of these — the bracket/dot syntax is kept
+   * as part of the literal key name, not interpreted:
+   *   - bracket objects:    `?user[name]=John`       → `{ 'user[name]': 'John' }`
+   *   - bracket arrays:     `?tags[]=a&tags[]=b`     → `{ 'tags[]': ['a', 'b'] }`
+   *   - dot paths:          `?user.name=John`        → `{ 'user.name': 'John' }`
+   *   - comma-split arrays: `?tags=a,b,c`            → `{ tags: 'a,b,c' }`
+   *
+   * For any of the above, install `qs` and opt in explicitly:
+   *   `queryParser: (raw) => qs.parse(raw) as Record<string, unknown>`
    */
   queryParser?: QueryParser
   onRequestStart?: (c: Context) => void
@@ -216,11 +199,9 @@ export class HonoAPIAppBuilder {
 
   /**
    * Builds and returns the Hono application with registered API routes.
-   * Async because it resolves the query parser (qs optional peer dep) once at build time.
    */
-  async build(): Promise<Hono> {
-    // Resolve query parser once so handlers use it synchronously
-    const queryParser = await resolveQueryParser(this.config?.queryParser)
+  build(): Hono {
+    const queryParser = this.config?.queryParser ?? parseQueryNative
 
     this.factories.forEach(({ factory, factoryContext, extendProcedureDoc }) => {
       factory.getProcedures().map((procedure: TProcedureRegistration<any, APIConfig>) => {