@withpica/mcp-server-directory 1.5.0 → 1.6.0

package/src/utils/errors.ts

@@ -19,6 +19,24 @@ export class ToolExecutionError extends McpServerError {
   }
 }
 
+/**
+ * Status → code for a route refusal that carried no code of its own.
+ *
+ * ⚠️ A port of `STATUS_REFUSAL_CODES` in
+ * `mcp-server-shared/mcp-utils/src/errors.ts`. This package does not depend on
+ * `@withpica/mcp-utils` (it publishes on its own), so it cannot import the
+ * shared formatter; `scripts/__tests__/format-error-status-parity.test.ts`
+ * fails if the two tables disagree.
+ */
+export const STATUS_REFUSAL_CODES: Record<number, string> = {
+  400: "VALIDATION_ERROR",
+  404: "NOT_FOUND",
+  409: "CONFLICT",
+  413: "PAYLOAD_TOO_LARGE",
+  415: "UNSUPPORTED_MEDIA_TYPE",
+  422: "VALIDATION_ERROR",
+};
+
 export function formatError(error: any): { type: string; text: string } {
   if (error instanceof McpServerError) {
     return {
@@ -78,6 +96,28 @@ export function formatError(error: any): { type: string; text: string } {
     };
   }
 
+  // A code-less 4xx is still a refusal; its status names the kind. Without
+  // this it read as UNKNOWN_ERROR, the bucket for an unclassified crash (ops
+  // issue 55510879 — the same defect in the shared formatter).
+  const status = (error as { status?: unknown })?.status;
+  const statusCode =
+    typeof status === "number" ? STATUS_REFUSAL_CODES[status] : undefined;
+  if (statusCode) {
+    return {
+      type: "text",
+      text: JSON.stringify(
+        {
+          error: statusCode,
+          message: bodyMessage || rawMessage,
+          status,
+          ...(bodyDetails !== undefined ? { details: bodyDetails } : {}),
+        },
+        null,
+        2,
+      ),
+    };
+  }
+
   return {
     type: "text",
     text: JSON.stringify(