@mulmoclaude/accounting-plugin 1.0.3 → 1.1.0

package/dist/server/http.d.ts

@@ -1,3 +1,35 @@
 import type { Request, Response, NextFunction } from "express";
-export declare function asyncHandler<TReq = Request, TRes = Response>(namespace: string, fallbackMessage: string, handler: (req: TReq, res: TRes) => Promise<void>): (req: TReq, res: TRes, next: NextFunction) => Promise<void>;
+/** The body the catch path sends. Declared here rather than imported from the
+ *  host's `httpError.ts` for the same reason the wrapper is copied: reaching
+ *  into the host would be an uphill import. */
+export interface ErrorBody {
+    error: string;
+}
+/** A route response that can carry either its success body or an error — use
+ *  this, not a bare `Response<T>`, for any handler wrapped by `asyncHandler`,
+ *  which can always emit `ErrorBody` on the failure path. */
+export type ApiResponse<T> = Response<T | ErrorBody>;
+/** The bounds below name exactly what the catch path dereferences.
+ *
+ *  They are NOT `extends Request` / `extends Response`: Express uses its type
+ *  parameters in mixed variance positions, so a nominal bound rejects valid
+ *  call sites like `Request<object, unknown, MyBody>`. Naming only the members
+ *  we touch keeps every concrete `Request<…>` / `Response<…>` assignable.
+ *
+ *  Structural bounds rather than unconstrained generics plus `as` casts,
+ *  because the casts were load-bearing in a way that hid a real contract: this
+ *  wrapper sends `{ error }` on the failure path, so a route declaring a
+ *  `ResBody` that cannot carry it was lying. That is now a compile error, and
+ *  the fix at such a site is `ApiResponse<T>`. */
+interface RoutePathBearing {
+    path: string;
+}
+interface ErrorSendableResponse {
+    headersSent: boolean;
+    status: (code: number) => {
+        json: (body: ErrorBody) => unknown;
+    };
+}
+export declare function asyncHandler<TReq extends RoutePathBearing = Request, TRes extends ErrorSendableResponse = Response>(namespace: string, fallbackMessage: string, handler: (req: TReq, res: TRes) => Promise<void>): (req: TReq, res: TRes, next: NextFunction) => Promise<void>;
+export {};
 //# sourceMappingURL=http.d.ts.map

package/dist/server/http.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"http.d.ts","sourceRoot":"","sources":["../../src/server/http.ts"],"names":[],"mappings":"AAaA,OAAO,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,YAAY,EAAE,MAAM,SAAS,CAAC;AAI/D,wBAAgB,YAAY,CAAC,IAAI,GAAG,OAAO,EAAE,IAAI,GAAG,QAAQ,EAC1D,SAAS,EAAE,MAAM,EACjB,eAAe,EAAE,MAAM,EACvB,OAAO,EAAE,CAAC,GAAG,EAAE,IAAI,EAAE,GAAG,EAAE,IAAI,KAAK,OAAO,CAAC,IAAI,CAAC,GAC/C,CAAC,GAAG,EAAE,IAAI,EAAE,GAAG,EAAE,IAAI,EAAE,IAAI,EAAE,YAAY,KAAK,OAAO,CAAC,IAAI,CAAC,CAkB7D"}
+{"version":3,"file":"http.d.ts","sourceRoot":"","sources":["../../src/server/http.ts"],"names":[],"mappings":"AAaA,OAAO,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,YAAY,EAAE,MAAM,SAAS,CAAC;AAI/D;;+CAE+C;AAC/C,MAAM,WAAW,SAAS;IACxB,KAAK,EAAE,MAAM,CAAC;CACf;AAED;;6DAE6D;AAC7D,MAAM,MAAM,WAAW,CAAC,CAAC,IAAI,QAAQ,CAAC,CAAC,GAAG,SAAS,CAAC,CAAC;AAErD;;;;;;;;;;;kDAWkD;AAClD,UAAU,gBAAgB;IACxB,IAAI,EAAE,MAAM,CAAC;CACd;AAED,UAAU,qBAAqB;IAC7B,WAAW,EAAE,OAAO,CAAC;IACrB,MAAM,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK;QAAE,IAAI,EAAE,CAAC,IAAI,EAAE,SAAS,KAAK,OAAO,CAAA;KAAE,CAAC;CAClE;AAED,wBAAgB,YAAY,CAAC,IAAI,SAAS,gBAAgB,GAAG,OAAO,EAAE,IAAI,SAAS,qBAAqB,GAAG,QAAQ,EACjH,SAAS,EAAE,MAAM,EACjB,eAAe,EAAE,MAAM,EACvB,OAAO,EAAE,CAAC,GAAG,EAAE,IAAI,EAAE,GAAG,EAAE,IAAI,KAAK,OAAO,CAAC,IAAI,CAAC,GAC/C,CAAC,GAAG,EAAE,IAAI,EAAE,GAAG,EAAE,IAAI,EAAE,IAAI,EAAE,YAAY,KAAK,OAAO,CAAC,IAAI,CAAC,CAgB7D"}

package/dist/server/router.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"router.d.ts","sourceRoot":"","sources":["../../src/server/router.ts"],"names":[],"mappings":"AAYA,OAAO,EAAE,MAAM,EAAqB,MAAM,SAAS,CAAC;AAyUpD;;;2BAG2B;AAC3B,wBAAgB,sBAAsB,IAAI,MAAM,CAsC/C"}
+{"version":3,"file":"router.d.ts","sourceRoot":"","sources":["../../src/server/router.ts"],"names":[],"mappings":"AAYA,OAAO,EAAE,MAAM,EAAqB,MAAM,SAAS,CAAC;AA2UpD;;;2BAG2B;AAC3B,wBAAgB,sBAAsB,IAAI,MAAM,CAsC/C"}

package/dist/server.cjs

@@ -1858,17 +1858,15 @@ function asyncHandler(namespace, fallbackMessage, handler) {
 		try {
 			await handler(req, res);
 		} catch (err) {
-			const expressReq = req;
-			const expressRes = res;
 			log.error(namespace, "handler threw", {
-				route: expressReq.path,
+				route: req.path,
 				error: require_shared.errorMessage(err)
 			});
-			if (expressRes.headersSent) {
+			if (res.headersSent) {
 				next(err);
 				return;
 			}
-			expressRes.status(500).json({ error: fallbackMessage });
+			res.status(500).json({ error: fallbackMessage });
 		}
 	};
 }