@supacloud/compiler 0.34.0 → 0.35.0

package/dist/cli.js

@@ -2509,7 +2509,7 @@ function renderClient(graph, options) {
     "  readonly path: string;",
     "  readonly status: number;",
     "  readonly requestId: string | undefined;",
-    "  /** Undeclared HTTP response for explicit inspection; excluded from JSON serialization. */",
+    "  /** HTTP response for inspection; successful response bodies may already be consumed. */",
     "  readonly response: Response | undefined;",
     "  constructor(message: string, details: ApiClientErrorDetails) {",
     "    super(message);",
@@ -2856,25 +2856,27 @@ function renderClient(graph, options) {
     "    const declaredSchema = responseSchemas ? selectResponseSchema(response.status, responseSchemas) : undefined;",
     "    const failure = (code: ApiClientErrorCode, message: string) => new ApiClientError(message, {",
     '      code, method, path, status: response.status, requestId: response.headers.get("x-request-id"),',
-    '      ...(code === "API_HTTP_ERROR" ? { response } : {}),',
+    "      response,",
     "    });",
     "    if (!response.ok && declaredSchema === undefined) {",
     '      throw failure("API_HTTP_ERROR", `API request failed: ${method} ${path} -> ${response.status}`);',
     "    }",
+    "    // Preserve the original HTTP error response if a declared error body fails validation.",
+    "    const bodyResponse = response.ok ? response : response.clone();",
     '    const contentType = response.headers?.get("content-type") ?? "";',
     "    let value: unknown;",
     "    if ([204, 205, 304].includes(response.status)) {",
     "      value = undefined;",
     '    } else if (contentType.includes("application/json") || contentType.includes("+json")) {',
-    "      const responseText = await response.text();",
+    "      const responseText = await bodyResponse.text();",
     "      try { value = JSON.parse(responseText); }",
     '      catch { throw failure("API_RESPONSE_INVALID", "Response is not valid JSON"); }',
     '    } else if (responseKind === "binary" || contentType.includes("application/octet-stream")) {',
-    "      value = await response.arrayBuffer();",
+    "      value = await bodyResponse.arrayBuffer();",
     '    } else if (responseKind === "stream") {',
-    "      value = response.body;",
+    "      value = bodyResponse.body;",
     "    } else {",
-    "      value = await response.text();",
+    "      value = await bodyResponse.text();",
     "    }",
     "    if (!responseSchemas) return decode ? decode(value) : value;",
     "    if (declaredSchema === undefined) {",
@@ -2901,9 +2903,10 @@ function renderClient(graph, options) {
     "  ): ClientRequestOptions {",
     "    const headers = { ...(options.headers as Record<string, string> | undefined) };",
     "    if (execution?.idempotencyKey !== undefined) {",
-    "      if (!/^[A-Za-z0-9._:-]{1,512}$/.test(execution.idempotencyKey)) {",
+    '      if (typeof execution.idempotencyKey !== "string" || !/^[A-Za-z0-9._:-]{1,512}$/.test(execution.idempotencyKey)) {',
     '        throw new TypeError("idempotencyKey must contain only safe header characters and be at most 512 characters");',
     "      }",
+    '      for (const key of Object.keys(headers)) if (key.toLowerCase() === "idempotency-key") delete headers[key];',
     '      headers["idempotency-key"] = execution.idempotencyKey;',
     "    }",
     '    if (idempotency === "required" && !hasHeader(headers, "idempotency-key")) {',

package/dist/index.js

@@ -2508,7 +2508,7 @@ function renderClient(graph, options) {
     "  readonly path: string;",
     "  readonly status: number;",
     "  readonly requestId: string | undefined;",
-    "  /** Undeclared HTTP response for explicit inspection; excluded from JSON serialization. */",
+    "  /** HTTP response for inspection; successful response bodies may already be consumed. */",
     "  readonly response: Response | undefined;",
     "  constructor(message: string, details: ApiClientErrorDetails) {",
     "    super(message);",
@@ -2855,25 +2855,27 @@ function renderClient(graph, options) {
     "    const declaredSchema = responseSchemas ? selectResponseSchema(response.status, responseSchemas) : undefined;",
     "    const failure = (code: ApiClientErrorCode, message: string) => new ApiClientError(message, {",
     '      code, method, path, status: response.status, requestId: response.headers.get("x-request-id"),',
-    '      ...(code === "API_HTTP_ERROR" ? { response } : {}),',
+    "      response,",
     "    });",
     "    if (!response.ok && declaredSchema === undefined) {",
     '      throw failure("API_HTTP_ERROR", `API request failed: ${method} ${path} -> ${response.status}`);',
     "    }",
+    "    // Preserve the original HTTP error response if a declared error body fails validation.",
+    "    const bodyResponse = response.ok ? response : response.clone();",
     '    const contentType = response.headers?.get("content-type") ?? "";',
     "    let value: unknown;",
     "    if ([204, 205, 304].includes(response.status)) {",
     "      value = undefined;",
     '    } else if (contentType.includes("application/json") || contentType.includes("+json")) {',
-    "      const responseText = await response.text();",
+    "      const responseText = await bodyResponse.text();",
     "      try { value = JSON.parse(responseText); }",
     '      catch { throw failure("API_RESPONSE_INVALID", "Response is not valid JSON"); }',
     '    } else if (responseKind === "binary" || contentType.includes("application/octet-stream")) {',
-    "      value = await response.arrayBuffer();",
+    "      value = await bodyResponse.arrayBuffer();",
     '    } else if (responseKind === "stream") {',
-    "      value = response.body;",
+    "      value = bodyResponse.body;",
     "    } else {",
-    "      value = await response.text();",
+    "      value = await bodyResponse.text();",
     "    }",
     "    if (!responseSchemas) return decode ? decode(value) : value;",
     "    if (declaredSchema === undefined) {",
@@ -2900,9 +2902,10 @@ function renderClient(graph, options) {
     "  ): ClientRequestOptions {",
     "    const headers = { ...(options.headers as Record<string, string> | undefined) };",
     "    if (execution?.idempotencyKey !== undefined) {",
-    "      if (!/^[A-Za-z0-9._:-]{1,512}$/.test(execution.idempotencyKey)) {",
+    '      if (typeof execution.idempotencyKey !== "string" || !/^[A-Za-z0-9._:-]{1,512}$/.test(execution.idempotencyKey)) {',
     '        throw new TypeError("idempotencyKey must contain only safe header characters and be at most 512 characters");',
     "      }",
+    '      for (const key of Object.keys(headers)) if (key.toLowerCase() === "idempotency-key") delete headers[key];',
     '      headers["idempotency-key"] = execution.idempotencyKey;',
     "    }",
     '    if (idempotency === "required" && !hasHeader(headers, "idempotency-key")) {',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@supacloud/compiler",
-  "version": "0.34.0",
+  "version": "0.35.0",
   "description": "Static compiler for @supacloud/app metadata: builds the application graph from AST, validates it, and generates reflection-free factory code",
   "type": "module",
   "main": "./dist/index.js",