@open-mercato/shared 0.6.8-develop.7091.1.15ffbe30ce → 0.6.8-develop.7093.1.700c5c8d96

package/dist/lib/crud/factory.js

@@ -54,6 +54,8 @@ import { createGenericOptimisticLockReader } from "./optimistic-lock.js";
 import { registerOptimisticLockReaderIfAbsent } from "./optimistic-lock-store.js";
 import { createLogger } from "../logger/index.js";
 import { isTransientDbError } from "../db/pg-errors.js";
+import { getTelemetryRuntime } from "../telemetry/runtime.js";
+import { randomUUID } from "node:crypto";
 const logger = createLogger("shared").child({ component: "crud" });
 function resolveSortParams(queryParams, defaultSort) {
   const rawSortField = queryParams.sortField ?? queryParams.sort;
@@ -298,7 +300,13 @@ function attachOperationHeader(res, logEntry) {
   }
   return res;
 }
-function handleError(err) {
+const INBOUND_REQUEST_ID_PATTERN = /^[A-Za-z0-9._-]{1,128}$/;
+function resolveRequestId(request) {
+  const inbound = request?.headers.get("x-request-id")?.trim();
+  if (inbound && INBOUND_REQUEST_ID_PATTERN.test(inbound)) return inbound;
+  return randomUUID();
+}
+function handleError(err, request) {
   if (err instanceof Response) return err;
   if (isCrudHttpError(err)) return json(err.body, { status: err.status });
   const interceptorRejection = getCommandInterceptorHttpRejection(err);
@@ -317,12 +325,19 @@ function handleError(err) {
   }
   const message = err instanceof Error ? err.message : void 0;
   const stack = err instanceof Error ? err.stack : void 0;
-  logger.error("Unexpected CRUD error", { message, stack, err });
+  const errorName = err instanceof Error ? err.name : void 0;
+  const requestId = resolveRequestId(request);
+  logger.error("Unexpected CRUD error", { message, stack, err, requestId });
+  getTelemetryRuntime()?.reportError(err, {
+    module: "crud",
+    attributes: { requestId, errorName }
+  });
   const body = {
     error: "Internal server error",
-    message: "Something went wrong. Please try again later."
+    message: "Something went wrong. Please try again later.",
+    requestId
   };
-  return json(body, { status: 500 });
+  return json(body, { status: 500, headers: { "x-request-id": requestId } });
 }
 const LIFECYCLE_ACTION_MAP = {
   created: { before: "creating", after: "created" },
@@ -1569,7 +1584,7 @@ function makeCrudRoute(opts) {
       return response;
     } catch (e) {
       finishProfile({ result: "error" });
-      return handleError(e);
+      return handleError(e, request);
     }
   }
   async function POST(request) {
@@ -1854,7 +1869,7 @@ function makeCrudRoute(opts) {
       payload = await enrichSingleRecord(payload, ctx);
       return json(payload, { status: 201 });
     } catch (e) {
-      return handleError(e);
+      return handleError(e, request);
     }
   }
   async function PUT(request) {
@@ -2159,7 +2174,7 @@ function makeCrudRoute(opts) {
       }
       return json(payload);
     } catch (e) {
-      return handleError(e);
+      return handleError(e, request);
     }
   }
   async function DELETE(request) {
@@ -2422,7 +2437,7 @@ function makeCrudRoute(opts) {
       }
       return json(payload);
     } catch (e) {
-      return handleError(e);
+      return handleError(e, request);
     }
   }
   return { metadata, GET, POST, PUT, DELETE };