@jskit-ai/json-rest-api-core 0.1.84 → 0.1.85

package/package.descriptor.mjs

@@ -1,7 +1,7 @@
 export default Object.freeze({
   packageVersion: 1,
   packageId: "@jskit-ai/json-rest-api-core",
-  version: "0.1.84",
+  version: "0.1.85",
   kind: "runtime",
   description: "Shared internal json-rest-api host runtime with autofilter, query-projection, and row-policy support.",
   dependsOn: [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jskit-ai/json-rest-api-core",
-  "version": "0.1.84",
+  "version": "0.1.85",
   "type": "module",
   "scripts": {
     "test": "node --test"
@@ -10,7 +10,7 @@
   },
   "dependencies": {
     "hooked-api": "1.x.x",
-    "json-rest-api": "^1.0.26",
-    "@jskit-ai/kernel": "0.1.140"
+    "json-rest-api": "^1.0.27",
+    "@jskit-ai/kernel": "0.1.141"
   }
 }

package/src/server/jsonRestApiHost.js

@@ -2,6 +2,7 @@ import { Api } from "hooked-api";
 import {
   AutoFilterPlugin,
   QueryProjectionsPlugin,
+  REST_API_FIELDSET_ERROR_CODE,
   RestApiKnexPlugin,
   RestApiPlugin,
   RowPolicyPlugin
@@ -647,9 +648,7 @@ async function returnNullWhenJsonRestResourceMissing(run) {
 }
 
 function isJsonRestSparseFieldError(error = null) {
-  return /^Unknown sparse field '.+' requested for '.+'$/u.test(
-    normalizeJsonRestText(error?.message)
-  );
+  return normalizeJsonRestText(error?.code) === REST_API_FIELDSET_ERROR_CODE;
 }
 
 async function returnBadRequestWhenJsonRestFieldsetInvalid(run) {

package/test/entrypoints.boundary.test.js

@@ -2,6 +2,7 @@ import assert from "node:assert/strict";
 import { readFile } from "node:fs/promises";
 import test from "node:test";
 import { normalizeRecordId } from "@jskit-ai/kernel/shared/support/normalize";
+import { RestApiFieldsetError } from "json-rest-api";
 
 import {
   INTERNAL_JSON_REST_API,
@@ -27,7 +28,7 @@ test("package exports include explicit server jsonRestApiHost entrypoint only",
   const exportsMap = packageJson && typeof packageJson === "object" ? packageJson.exports : {};
   assert.equal(exportsMap["./server/jsonRestApiHost"], "./src/server/jsonRestApiHost.js");
   assert.equal(exportsMap["./server"], undefined);
-  assert.equal(packageJson.dependencies?.["json-rest-api"], "^1.0.26");
+  assert.equal(packageJson.dependencies?.["json-rest-api"], "^1.0.27");
 });
 
 test("server jsonRestApiHost entrypoint no longer exports host-side JSON:API simplification helpers", async () => {
@@ -649,8 +650,11 @@ test("returnNullWhenJsonRestResourceMissing only swallows missing-resource error
   );
 });
 
-test("invalid sparse fields become a stable 400 without swallowing unrelated failures", async () => {
-  const sparseFieldError = new Error("Unknown sparse field 'passwordHash' requested for 'contacts'");
+test("typed invalid sparse fields become a stable 400 without message matching", async () => {
+  const sparseFieldError = new RestApiFieldsetError({
+    field: "passwordHash",
+    resourceType: "contacts"
+  });
 
   await assert.rejects(
     () => returnBadRequestWhenJsonRestFieldsetInvalid(async () => {
@@ -664,6 +668,14 @@ test("invalid sparse fields become a stable 400 without swallowing unrelated fai
     }
   );
 
+  const messageOnlyError = new Error(sparseFieldError.message);
+  await assert.rejects(
+    () => returnBadRequestWhenJsonRestFieldsetInvalid(async () => {
+      throw messageOnlyError;
+    }),
+    (error) => error === messageOnlyError
+  );
+
   const unrelatedError = new Error("database unavailable");
   await assert.rejects(
     () => returnBadRequestWhenJsonRestFieldsetInvalid(async () => {