@jskit-ai/http-runtime 0.1.111 → 0.1.113

package/package.descriptor.mjs

@@ -1,7 +1,7 @@
 export default Object.freeze({
   "packageVersion": 1,
   "packageId": "@jskit-ai/http-runtime",
-  "version": "0.1.111",
+  "version": "0.1.113",
   "kind": "runtime",
   "dependsOn": [],
   "capabilities": {
@@ -67,7 +67,7 @@ export default Object.freeze({
   "mutations": {
     "dependencies": {
       "runtime": {
-        "@jskit-ai/kernel": "0.1.113"
+        "@jskit-ai/kernel": "0.1.114"
       },
       "dev": {}
     },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jskit-ai/http-runtime",
-  "version": "0.1.111",
+  "version": "0.1.113",
   "type": "module",
   "scripts": {
     "test": "node --test"
@@ -18,7 +18,7 @@
     "./shared/validators/operationValidation": "./src/shared/validators/operationValidation.js"
   },
   "dependencies": {
-    "@jskit-ai/kernel": "0.1.113",
+    "@jskit-ai/kernel": "0.1.114",
     "json-rest-schema": "1.x.x"
   }
 }

package/src/shared/clientRuntime/errors.js

@@ -1,14 +1,22 @@
 import { isRecord, resolveFieldErrors } from "../support/fieldErrors.js";
 import { createJsonApiClientErrorPayload } from "./jsonApiResourceTransport.js";
 
+function standardErrorMessage(payload = {}) {
+  const nestedMessage = Array.isArray(payload.errors)
+    ? payload.errors.find((entry) => isRecord(entry) && String(entry.message || "").trim())?.message
+    : "";
+  return String(payload.error || payload.message || nestedMessage || "").trim();
+}
+
 function createHttpError(response, data = {}) {
   const payload = isRecord(data) ? data : {};
-  const jsonApiPayload = createJsonApiClientErrorPayload(payload);
+  const message = standardErrorMessage(payload);
+  const jsonApiPayload = message ? null : createJsonApiClientErrorPayload(payload);
   if (jsonApiPayload) {
     return createHttpError(response, jsonApiPayload);
   }
 
-  const error = new Error(payload.error || `Request failed with status ${response.status}.`);
+  const error = new Error(message || `Request failed with status ${response.status}.`);
   const normalizedFieldErrors = resolveFieldErrors(payload);
   error.status = Number(response?.status || 0);
   error.code = String(payload.code || "").trim() || null;

package/test/validationErrors.test.js

@@ -99,6 +99,56 @@ test("createHttpError normalizes code and fieldErrors", () => {
   });
 });
 
+test("createHttpError preserves standard error envelopes that also contain an errors list", () => {
+  const error = createHttpError(
+    {
+      status: 409
+    },
+    {
+      code: "vibe64_repository_flip_active_sessions",
+      details: {
+        code: "vibe64_repository_flip_active_sessions"
+      },
+      error: "Finish or abandon active sessions before changing this project's repository mode.",
+      errors: [
+        {
+          code: "vibe64_repository_flip_active_sessions",
+          message: "Finish or abandon active sessions before changing this project's repository mode."
+        }
+      ],
+      ok: false
+    }
+  );
+
+  assert.equal(error.status, 409);
+  assert.equal(error.code, "vibe64_repository_flip_active_sessions");
+  assert.equal(
+    error.message,
+    "Finish or abandon active sessions before changing this project's repository mode."
+  );
+  assert.deepEqual(error.details, {
+    code: "vibe64_repository_flip_active_sessions"
+  });
+});
+
+test("createHttpError reads errors-list messages when a standard envelope has no top-level message", () => {
+  const error = createHttpError(
+    {
+      status: 400
+    },
+    {
+      errors: [
+        {
+          code: "invalid_request",
+          message: "The request is invalid."
+        }
+      ]
+    }
+  );
+
+  assert.equal(error.message, "The request is invalid.");
+});
+
 test("createHttpError decodes json:api error documents into the existing fieldErrors shape", () => {
   const error = createHttpError(
     {