@jaypie/express 1.2.20 → 1.2.22

package/dist/cjs/index.d.ts

@@ -3,6 +3,7 @@ export type { ApiGatewayV1Event, CreateLambdaHandlerOptions, FunctionUrlEvent, L
 export { EXPRESS } from "./constants.js";
 export { default as cors } from "./cors.helper.js";
 export type { CorsConfig } from "./cors.helper.js";
+export { default as fabricApiResponse } from "./fabricApiResponse.js";
 export { default as expressHandler } from "./expressHandler.js";
 export type { ExpressHandlerLocals, ExpressHandlerOptions, JaypieHandlerSetup, JaypieHandlerTeardown, JaypieHandlerValidate, } from "./expressHandler.js";
 export { default as expressHttpCodeHandler } from "./http.handler.js";

package/dist/esm/expressHandler.d.ts

@@ -5,6 +5,13 @@ export type JaypieHandlerValidate = (req: Request, res: Response) => Promise<boo
 export type ExpressHandlerLocals = (req: Request, res: Response) => Promise<unknown> | unknown;
 export interface ExpressHandlerOptions {
     chaos?: string;
+    /**
+     * When true, the handler's return value is wrapped with
+     * `fabricApiResponse` before `res.json()`. Plain objects become
+     * `{ data: value }`; pre-wrapped `{ data }` or `{ errors }` payloads
+     * pass through unchanged. `null` / `undefined` become `{ data: null }`.
+     */
+    fabric?: boolean;
     locals?: Record<string, unknown | ExpressHandlerLocals>;
     name?: string;
     secrets?: string[];

package/dist/esm/fabricApiResponse.d.ts

@@ -0,0 +1,14 @@
+/**
+ * Wrap a service return value in the canonical Jaypie API envelope.
+ *
+ * Rules:
+ * - `null` / `undefined` → `{ data: null }`
+ * - `{ data }` with exactly one key → passthrough
+ * - `{ errors }` with exactly one key → passthrough
+ * - everything else (including `{ data, other }`) → `{ data: value }`
+ *
+ * The "only one key, and that key is `data` or `errors`" check prevents
+ * false positives where a domain object happens to contain a `data` field.
+ */
+export declare function fabricApiResponse(result: unknown): unknown;
+export default fabricApiResponse;

package/dist/esm/index.d.ts

@@ -3,6 +3,7 @@ export type { ApiGatewayV1Event, CreateLambdaHandlerOptions, FunctionUrlEvent, L
 export { EXPRESS } from "./constants.js";
 export { default as cors } from "./cors.helper.js";
 export type { CorsConfig } from "./cors.helper.js";
+export { default as fabricApiResponse } from "./fabricApiResponse.js";
 export { default as expressHandler } from "./expressHandler.js";
 export type { ExpressHandlerLocals, ExpressHandlerOptions, JaypieHandlerSetup, JaypieHandlerTeardown, JaypieHandlerValidate, } from "./expressHandler.js";
 export { default as expressHttpCodeHandler } from "./http.handler.js";

package/dist/esm/index.js

@@ -1355,6 +1355,48 @@ var cors_helper = (config) => {
     };
 };
 
+//
+//
+// Helpers
+//
+function isWrappedData(value) {
+    return (typeof value === "object" &&
+        value !== null &&
+        !Array.isArray(value) &&
+        Object.keys(value).length === 1 &&
+        "data" in value);
+}
+function isWrappedErrors(value) {
+    return (typeof value === "object" &&
+        value !== null &&
+        !Array.isArray(value) &&
+        Object.keys(value).length === 1 &&
+        "errors" in value);
+}
+//
+//
+// Main
+//
+/**
+ * Wrap a service return value in the canonical Jaypie API envelope.
+ *
+ * Rules:
+ * - `null` / `undefined` → `{ data: null }`
+ * - `{ data }` with exactly one key → passthrough
+ * - `{ errors }` with exactly one key → passthrough
+ * - everything else (including `{ data, other }`) → `{ data: value }`
+ *
+ * The "only one key, and that key is `data` or `errors`" check prevents
+ * false positives where a domain object happens to contain a `data` field.
+ */
+function fabricApiResponse(result) {
+    if (result === null || result === undefined)
+        return { data: null };
+    if (isWrappedData(result) || isWrappedErrors(result))
+        return result;
+    return { data: result };
+}
+
 //
 //
 // Helper Functions
@@ -1717,7 +1759,7 @@ function expressHandler(handlerOrOptions, optionsOrHandler) {
     //
     // Validate
     //
-    let { chaos, locals, name, secrets, setup = [], teardown = [], unavailable, validate, } = options;
+    let { chaos, fabric, locals, name, secrets, setup = [], teardown = [], unavailable, validate, } = options;
     if (typeof handler !== "function") {
         throw new BadRequestError(`Argument "${handler}" doesn't match type "function"`);
     }
@@ -1871,6 +1913,9 @@ function expressHandler(handlerOrOptions, optionsOrHandler) {
             // Process
             //
             response = (await jaypieFunction(req, res, ...params));
+            if (fabric) {
+                response = fabricApiResponse(response);
+            }
             //
             //
             // Error Handling
@@ -1987,7 +2032,11 @@ function expressHandler(handlerOrOptions, optionsOrHandler) {
         // Replace UUIDs with :id for better aggregation
         path = path.replace(/[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}/gi, ":id");
         // Add request data to session report
-        logger$1.report({ path, status: String(res.statusCode) });
+        logger$1.report({
+            method: req.method,
+            path,
+            status: String(res.statusCode),
+        });
         // Submit metric if Datadog is configured
         if (hasDatadogEnv()) {
             let metricPrefix = "project";
@@ -2268,7 +2317,11 @@ function expressStreamHandler(handlerOrOptions, optionsOrHandler) {
             }
             path = path.replace(/[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}/gi, ":id");
             // Add request data to session report
-            logger.report({ path, status: String(res.statusCode) });
+            logger.report({
+                method: req.method,
+                path,
+                status: String(res.statusCode),
+            });
             // Submit metric if Datadog is configured
             if (hasDatadogEnv()) {
                 let metricPrefix = "project";
@@ -2348,5 +2401,5 @@ const noContentRoute = routes.noContentRoute;
 const notFoundRoute = routes.notFoundRoute;
 const notImplementedRoute = routes.notImplementedRoute;
 
-export { EXPRESS, LambdaRequest, LambdaResponseBuffered, LambdaResponseStreaming, badRequestRoute, cors_helper as cors, createLambdaHandler, createLambdaStreamHandler, echoRoute, expressHandler, httpHandler as expressHttpCodeHandler, expressStreamHandler, forbiddenRoute, getCurrentInvoke, getCurrentInvokeUuid, goneRoute, methodNotAllowedRoute, noContentRoute, notFoundRoute, notImplementedRoute };
+export { EXPRESS, LambdaRequest, LambdaResponseBuffered, LambdaResponseStreaming, badRequestRoute, cors_helper as cors, createLambdaHandler, createLambdaStreamHandler, echoRoute, expressHandler, httpHandler as expressHttpCodeHandler, expressStreamHandler, fabricApiResponse, forbiddenRoute, getCurrentInvoke, getCurrentInvokeUuid, goneRoute, methodNotAllowedRoute, noContentRoute, notFoundRoute, notImplementedRoute };
 //# sourceMappingURL=index.js.map