@spfn/core 0.1.0-alpha.68 → 0.1.0-alpha.72

package/dist/route/index.d.ts

@@ -1,11 +1,10 @@
 export { A as AutoRouteLoader, R as RouteInfo, a as RouteStats, l as loadRoutes } from '../auto-loader-JFaZ9gON.js';
 import { Context, Hono, MiddlewareHandler } from 'hono';
-import { a as RouteContract, R as RouteContext, b as RouteHandler } from '../types-CAON3Mmg.js';
-export { c as HeaderRecord, H as HttpMethod, I as InferContract, d as RouteMeta, i as isHttpMethod } from '../types-CAON3Mmg.js';
-import * as _sinclair_typebox from '@sinclair/typebox';
-import { TSchema } from '@sinclair/typebox';
-import { b as ErrorResponse } from '../error-handler-wjLL3v-a.js';
+import { a as RouteContract, R as RouteContext, b as RouteHandler } from '../types-Dzggq1Yb.js';
+export { f as ApiErrorResponse, c as ApiErrorSchema, g as ApiResponse, d as ApiResponseSchema, e as ApiSuccessResponse, A as ApiSuccessSchema, h as HeaderRecord, H as HttpMethod, I as InferContract, j as RouteMeta, i as isHttpMethod } from '../types-Dzggq1Yb.js';
 import 'hono/utils/http-status';
+import '@sinclair/typebox';
+import '../error-handler-wjLL3v-a.js';
 
 /**
  * Contract-based Route Handler Wrapper
@@ -38,155 +37,4 @@ type SPFNApp = Hono & {
  */
 declare function createApp(): SPFNApp;
 
-/**
- * Success response wrapper
- */
-interface ApiSuccessResponse<T = any> {
-    success: true;
-    data: T;
-    meta?: {
-        timestamp?: string;
-        requestId?: string;
-        pagination?: {
-            page: number;
-            limit: number;
-            total: number;
-            totalPages: number;
-        };
-        [key: string]: any;
-    };
-}
-/**
- * Error response type (re-exported from ErrorHandler for consistency)
- */
-type ApiErrorResponse = ErrorResponse;
-/**
- * Unified API response type
- */
-type ApiResponse<T = any> = ApiSuccessResponse<T> | ApiErrorResponse;
-/**
- * Creates a TypeBox schema for ApiSuccessResponse<T>
- *
- * @example
- * ```ts
- * const UserSchema = Type.Object({
- *   id: Type.String(),
- *   name: Type.String(),
- * });
- *
- * const contract = {
- *   response: ApiSuccessSchema(UserSchema),
- * };
- * ```
- */
-declare function ApiSuccessSchema<T extends TSchema>(dataSchema: T): _sinclair_typebox.TObject<{
-    success: _sinclair_typebox.TLiteral<true>;
-    data: T;
-    meta: _sinclair_typebox.TOptional<_sinclair_typebox.TObject<{
-        timestamp: _sinclair_typebox.TOptional<_sinclair_typebox.TString>;
-        requestId: _sinclair_typebox.TOptional<_sinclair_typebox.TString>;
-        pagination: _sinclair_typebox.TOptional<_sinclair_typebox.TObject<{
-            page: _sinclair_typebox.TNumber;
-            limit: _sinclair_typebox.TNumber;
-            total: _sinclair_typebox.TNumber;
-            totalPages: _sinclair_typebox.TNumber;
-        }>>;
-    }>>;
-}>;
-/**
- * Creates a TypeBox schema for ApiErrorResponse
- */
-declare function ApiErrorSchema(): _sinclair_typebox.TObject<{
-    success: _sinclair_typebox.TLiteral<false>;
-    error: _sinclair_typebox.TObject<{
-        message: _sinclair_typebox.TString;
-        type: _sinclair_typebox.TString;
-        statusCode: _sinclair_typebox.TNumber;
-        stack: _sinclair_typebox.TOptional<_sinclair_typebox.TString>;
-        details: _sinclair_typebox.TOptional<_sinclair_typebox.TAny>;
-    }>;
-}>;
-/**
- * Creates a TypeBox union schema for ApiResponse<T>
- *
- * Use this in your route contract's response field for standardized responses.
- *
- * @example
- * ```ts
- * const contract = {
- *   method: 'GET',
- *   path: '/users/:id',
- *   response: ApiResponseSchema(Type.Object({
- *     id: Type.String(),
- *     name: Type.String(),
- *   })),
- * };
- * ```
- */
-declare function ApiResponseSchema<T extends TSchema>(dataSchema: T): _sinclair_typebox.TUnion<[_sinclair_typebox.TObject<{
-    success: _sinclair_typebox.TLiteral<true>;
-    data: T;
-    meta: _sinclair_typebox.TOptional<_sinclair_typebox.TObject<{
-        timestamp: _sinclair_typebox.TOptional<_sinclair_typebox.TString>;
-        requestId: _sinclair_typebox.TOptional<_sinclair_typebox.TString>;
-        pagination: _sinclair_typebox.TOptional<_sinclair_typebox.TObject<{
-            page: _sinclair_typebox.TNumber;
-            limit: _sinclair_typebox.TNumber;
-            total: _sinclair_typebox.TNumber;
-            totalPages: _sinclair_typebox.TNumber;
-        }>>;
-    }>>;
-}>, _sinclair_typebox.TObject<{
-    success: _sinclair_typebox.TLiteral<false>;
-    error: _sinclair_typebox.TObject<{
-        message: _sinclair_typebox.TString;
-        type: _sinclair_typebox.TString;
-        statusCode: _sinclair_typebox.TNumber;
-        stack: _sinclair_typebox.TOptional<_sinclair_typebox.TString>;
-        details: _sinclair_typebox.TOptional<_sinclair_typebox.TAny>;
-    }>;
-}>]>;
-/**
- * Creates a success response
- *
- * @example
- * ```ts
- * // Simple success
- * return success(c, { id: '123', name: 'John' });
- *
- * // With metadata
- * return success(c, user, { timestamp: new Date().toISOString() });
- *
- * // With custom status
- * return success(c, newUser, undefined, 201);
- * ```
- */
-declare function success<T, TContract extends RouteContract = any>(c: RouteContext<TContract>, data: T, meta?: ApiSuccessResponse<T>['meta'], status?: number): Response;
-/**
- * Creates an error response
- *
- * Note: This is a convenience wrapper. ValidationErrors thrown by bind()
- * are automatically handled by ErrorHandler middleware.
- *
- * @example
- * ```ts
- * // Not found
- * return error(c, 'User not found', 404);
- *
- * // Validation error with details
- * return error(c, 'Invalid input', 400, { fields: ['email', 'password'] });
- * ```
- */
-declare function error<TContract extends RouteContract = any>(c: RouteContext<TContract>, message: string, statusCode?: number, details?: any): Response;
-/**
- * Creates a paginated success response
- *
- * @example
- * ```ts
- * const { users, total } = await db.listUsers(page, limit);
- * return paginated(c, users, page, limit, total);
- * ```
- */
-declare function paginated<T, TContract extends RouteContract = any>(c: RouteContext<TContract>, data: T[], page: number, limit: number, total: number): Response;
-
-export { type ApiErrorResponse, ApiErrorSchema, type ApiResponse, ApiResponseSchema, type ApiSuccessResponse, ApiSuccessSchema, RouteContext, RouteContract, RouteHandler, type SPFNApp, bind, createApp, error, paginated, success };
+export { RouteContext, RouteContract, RouteHandler, type SPFNApp, bind, createApp };

package/dist/route/index.js

@@ -119,11 +119,11 @@ function formatTimestampHuman(date) {
   const ms = String(date.getMilliseconds()).padStart(3, "0");
   return `${year}-${month}-${day} ${hours}:${minutes}:${seconds}.${ms}`;
 }
-function formatError(error2) {
+function formatError(error) {
   const lines = [];
-  lines.push(`${error2.name}: ${error2.message}`);
-  if (error2.stack) {
-    const stackLines = error2.stack.split("\n").slice(1);
+  lines.push(`${error.name}: ${error.message}`);
+  if (error.stack) {
+    const stackLines = error.stack.split("\n").slice(1);
     lines.push(...stackLines);
   }
   return lines.join("\n");
@@ -309,7 +309,7 @@ var init_logger = __esm({
       /**
        * Log processing (internal)
        */
-      log(level, message, error2, context) {
+      log(level, message, error, context) {
         if (LOG_LEVEL_PRIORITY[level] < LOG_LEVEL_PRIORITY[this.config.level]) {
           return;
         }
@@ -318,7 +318,7 @@ var init_logger = __esm({
           level,
           message,
           module: this.module,
-          error: error2,
+          error,
           // Mask sensitive information in context to prevent credential leaks
           context: context ? maskSensitiveData(context) : void 0
         };
@@ -329,8 +329,8 @@ var init_logger = __esm({
        */
       processTransports(metadata) {
         const promises = this.config.transports.filter((transport) => transport.enabled).map((transport) => this.safeTransportLog(transport, metadata));
-        Promise.all(promises).catch((error2) => {
-          const errorMessage = error2 instanceof Error ? error2.message : String(error2);
+        Promise.all(promises).catch((error) => {
+          const errorMessage = error instanceof Error ? error.message : String(error);
           process.stderr.write(`[Logger] Transport error: ${errorMessage}
 `);
         });
@@ -341,8 +341,8 @@ var init_logger = __esm({
       async safeTransportLog(transport, metadata) {
         try {
           await transport.log(metadata);
-        } catch (error2) {
-          const errorMessage = error2 instanceof Error ? error2.message : String(error2);
+        } catch (error) {
+          const errorMessage = error instanceof Error ? error.message : String(error);
           process.stderr.write(`[Logger] Transport "${transport.name}" failed: ${errorMessage}
 `);
         }
@@ -432,11 +432,11 @@ var init_file = __esm({
         }
         if (this.currentStream) {
           return new Promise((resolve, reject) => {
-            this.currentStream.write(message + "\n", "utf-8", (error2) => {
-              if (error2) {
-                process.stderr.write(`[FileTransport] Failed to write log: ${error2.message}
+            this.currentStream.write(message + "\n", "utf-8", (error) => {
+              if (error) {
+                process.stderr.write(`[FileTransport] Failed to write log: ${error.message}
 `);
-                reject(error2);
+                reject(error);
               } else {
                 resolve();
               }
@@ -458,8 +458,8 @@ var init_file = __esm({
           encoding: "utf-8"
         });
         this.currentFilename = filename;
-        this.currentStream.on("error", (error2) => {
-          process.stderr.write(`[FileTransport] Stream error: ${error2.message}
+        this.currentStream.on("error", (error) => {
+          process.stderr.write(`[FileTransport] Stream error: ${error.message}
 `);
           this.currentStream = null;
           this.currentFilename = null;
@@ -473,9 +473,9 @@ var init_file = __esm({
           return;
         }
         return new Promise((resolve, reject) => {
-          this.currentStream.end((error2) => {
-            if (error2) {
-              reject(error2);
+          this.currentStream.end((error) => {
+            if (error) {
+              reject(error);
             } else {
               this.currentStream = null;
               this.currentFilename = null;
@@ -500,8 +500,8 @@ var init_file = __esm({
           if (stats.size >= this.maxFileSize) {
             await this.rotateBySize();
           }
-        } catch (error2) {
-          const errorMessage = error2 instanceof Error ? error2.message : String(error2);
+        } catch (error) {
+          const errorMessage = error instanceof Error ? error.message : String(error);
           process.stderr.write(`[FileTransport] Failed to check file size: ${errorMessage}
 `);
         }
@@ -527,8 +527,8 @@ var init_file = __esm({
             const newPath2 = join(this.logDir, `${baseName}.${newNum}.log`);
             try {
               renameSync(oldPath, newPath2);
-            } catch (error2) {
-              const errorMessage = error2 instanceof Error ? error2.message : String(error2);
+            } catch (error) {
+              const errorMessage = error instanceof Error ? error.message : String(error);
               process.stderr.write(`[FileTransport] Failed to rotate file: ${errorMessage}
 `);
             }
@@ -540,8 +540,8 @@ var init_file = __esm({
           if (existsSync(currentPath)) {
             renameSync(currentPath, newPath);
           }
-        } catch (error2) {
-          const errorMessage = error2 instanceof Error ? error2.message : String(error2);
+        } catch (error) {
+          const errorMessage = error instanceof Error ? error.message : String(error);
           process.stderr.write(`[FileTransport] Failed to rotate current file: ${errorMessage}
 `);
         }
@@ -568,15 +568,15 @@ var init_file = __esm({
               const filepath = join(this.logDir, file);
               try {
                 unlinkSync(filepath);
-              } catch (error2) {
-                const errorMessage = error2 instanceof Error ? error2.message : String(error2);
+              } catch (error) {
+                const errorMessage = error instanceof Error ? error.message : String(error);
                 process.stderr.write(`[FileTransport] Failed to delete old file "${file}": ${errorMessage}
 `);
               }
             }
           }
-        } catch (error2) {
-          const errorMessage = error2 instanceof Error ? error2.message : String(error2);
+        } catch (error) {
+          const errorMessage = error instanceof Error ? error.message : String(error);
           process.stderr.write(`[FileTransport] Failed to clean old files: ${errorMessage}
 `);
         }
@@ -635,8 +635,8 @@ function validateDirectoryWritable(dirPath) {
   if (!existsSync(dirPath)) {
     try {
       mkdirSync(dirPath, { recursive: true });
-    } catch (error2) {
-      const errorMessage = error2 instanceof Error ? error2.message : String(error2);
+    } catch (error) {
+      const errorMessage = error instanceof Error ? error.message : String(error);
       throw new Error(`Failed to create log directory "${dirPath}": ${errorMessage}`);
     }
   }
@@ -649,8 +649,8 @@ function validateDirectoryWritable(dirPath) {
   try {
     writeFileSync(testFile, "test", "utf-8");
     unlinkSync(testFile);
-  } catch (error2) {
-    const errorMessage = error2 instanceof Error ? error2.message : String(error2);
+  } catch (error) {
+    const errorMessage = error instanceof Error ? error.message : String(error);
     throw new Error(`Cannot write to log directory "${dirPath}": ${errorMessage}`);
   }
 }
@@ -727,11 +727,11 @@ function validateConfig() {
     validateFileConfig();
     validateSlackConfig();
     validateEmailConfig();
-  } catch (error2) {
-    if (error2 instanceof Error) {
-      throw new Error(`[Logger] Configuration validation failed: ${error2.message}`);
+  } catch (error) {
+    if (error instanceof Error) {
+      throw new Error(`[Logger] Configuration validation failed: ${error.message}`);
     }
-    throw error2;
+    throw error;
   }
 }
 var init_config = __esm({
@@ -885,12 +885,12 @@ function discoverFunctionRoutes(cwd = process.cwd()) {
             prefix: prefix || "(none)"
           });
         }
-      } catch (error2) {
+      } catch (error) {
       }
     }
-  } catch (error2) {
+  } catch (error) {
     routeLogger.warn("Failed to discover function routes", {
-      error: error2 instanceof Error ? error2.message : "Unknown error"
+      error: error instanceof Error ? error.message : "Unknown error"
     });
   }
   return functions;
@@ -924,8 +924,8 @@ var AutoRouteLoader = class {
     }
     let failureCount = 0;
     for (const file of files) {
-      const success2 = await this.loadRoute(app, file);
-      if (success2) ; else {
+      const success = await this.loadRoute(app, file);
+      if (success) ; else {
         failureCount++;
       }
     }
@@ -962,8 +962,8 @@ var AutoRouteLoader = class {
     let successCount = 0;
     let failureCount = 0;
     for (const file of files) {
-      const success2 = await this.loadRoute(app, file, prefix);
-      if (success2) {
+      const success = await this.loadRoute(app, file, prefix);
+      if (success) {
         successCount++;
       } else {
         failureCount++;
@@ -1061,8 +1061,8 @@ var AutoRouteLoader = class {
         }
       });
       return true;
-    } catch (error2) {
-      this.categorizeAndLogError(error2, relativePath);
+    } catch (error) {
+      this.categorizeAndLogError(error, relativePath);
       return false;
     }
   }
@@ -1118,9 +1118,9 @@ var AutoRouteLoader = class {
       }
     }
   }
-  categorizeAndLogError(error2, relativePath) {
-    const message = error2.message;
-    const stack = error2.stack;
+  categorizeAndLogError(error, relativePath) {
+    const message = error.message;
+    const stack = error.stack;
     if (message.includes("Cannot find module") || message.includes("MODULE_NOT_FOUND")) {
       routeLogger2.error("Missing dependency", {
         file: relativePath,
@@ -1183,10 +1183,10 @@ async function loadRoutes(app, options) {
             routesDir: func.routesDir,
             prefix: func.prefix || "/"
           });
-        } catch (error2) {
+        } catch (error) {
           routeLogger2.error("Failed to load function routes", {
             package: func.packageName,
-            error: error2 instanceof Error ? error2.message : "Unknown error"
+            error: error instanceof Error ? error.message : "Unknown error"
           });
         }
       }
@@ -1298,8 +1298,42 @@ function bind(contract, handler) {
         return body;
       },
       json: (data, status, headers) => {
+        const errorHandlerEnabled = rawContext.get("errorHandlerEnabled");
+        if (errorHandlerEnabled && process.env.NODE_ENV !== "production") {
+          const hasSuccessField = data && typeof data === "object" && "success" in data;
+          if (!hasSuccessField) {
+            console.warn(
+              "[SPFN] Warning: ErrorHandler is enabled but c.json() is being used with non-standard response format.\nConsider using c.success() for consistent API responses, or disable ErrorHandler if you prefer custom formats."
+            );
+          }
+        }
         return rawContext.json(data, status, headers);
       },
+      success: (data, meta, status = 200) => {
+        const response = {
+          success: true,
+          data
+        };
+        if (meta) {
+          response.meta = meta;
+        }
+        return rawContext.json(response, status);
+      },
+      paginated: (data, page, limit, total) => {
+        const response = {
+          success: true,
+          data,
+          meta: {
+            pagination: {
+              page,
+              limit,
+              total,
+              totalPages: Math.ceil(total / limit)
+            }
+          }
+        };
+        return rawContext.json(response, 200);
+      },
       raw: rawContext
     };
     return handler(routeContext);
@@ -1412,56 +1446,12 @@ function ApiResponseSchema(dataSchema) {
     ApiErrorSchema()
   ]);
 }
-function success(c, data, meta, status = 200) {
-  const response = {
-    success: true,
-    data
-  };
-  if (meta) {
-    response.meta = meta;
-  }
-  return c.json(response, status);
-}
-function error(c, message, statusCode = 400, details) {
-  const response = {
-    success: false,
-    error: {
-      message,
-      type: getErrorType(statusCode),
-      statusCode
-    }
-  };
-  if (details) {
-    response.error.details = details;
-  }
-  return c.json(response, statusCode);
-}
-function paginated(c, data, page, limit, total) {
-  return success(c, data, {
-    pagination: {
-      page,
-      limit,
-      total,
-      totalPages: Math.ceil(total / limit)
-    }
-  });
-}
-function getErrorType(statusCode) {
-  if (statusCode >= 500) return "InternalServerError";
-  if (statusCode === 404) return "NotFoundError";
-  if (statusCode === 401) return "UnauthorizedError";
-  if (statusCode === 403) return "ForbiddenError";
-  if (statusCode === 400) return "ValidationError";
-  if (statusCode === 409) return "ConflictError";
-  if (statusCode === 422) return "UnprocessableEntityError";
-  return "ClientError";
-}
 
 // src/route/types.ts
 function isHttpMethod(value) {
   return typeof value === "string" && ["GET", "POST", "PUT", "PATCH", "DELETE"].includes(value);
 }
 
-export { ApiErrorSchema, ApiResponseSchema, ApiSuccessSchema, AutoRouteLoader, bind, createApp, error, isHttpMethod, loadRoutes, paginated, success };
+export { ApiErrorSchema, ApiResponseSchema, ApiSuccessSchema, AutoRouteLoader, bind, createApp, isHttpMethod, loadRoutes };
 //# sourceMappingURL=index.js.map
 //# sourceMappingURL=index.js.map