@rest-vir/define-service 1.4.0 → 1.5.0

package/dist/endpoint/endpoint.d.ts

@@ -104,7 +104,7 @@ export declare const endpointInitShape: Shape<{
     }> | import("@sinclair/typebox").TUnsafe<RegExp> | import("@sinclair/typebox").TString | import("@sinclair/typebox").TUndefined | import("@sinclair/typebox").TUnsafe<() => void> | import("@sinclair/typebox").TArray<import("@sinclair/typebox").TUnion<(import("@sinclair/typebox").TUnsafe<{
         anyOrigin: boolean;
     }> | import("@sinclair/typebox").TUnsafe<RegExp> | import("@sinclair/typebox").TString | import("@sinclair/typebox").TUnsafe<() => void>)[]>>)[]>>>;
-    methods: import("@sinclair/typebox").TUnsafe<Partial<Record<HttpMethod, boolean>>>;
+    methods: Shape<import("@sinclair/typebox").TUnsafe<Partial<Record<HttpMethod, boolean>>>>;
     bypassResponseValidation: Shape<import("@sinclair/typebox").TOptional<import("@sinclair/typebox").TUnion<(import("@sinclair/typebox").TBoolean | import("@sinclair/typebox").TNull | import("@sinclair/typebox").TUndefined)[]>>>;
     customProps: Shape<import("@sinclair/typebox").TOptional<import("@sinclair/typebox").TUnion<(import("@sinclair/typebox").TUndefined | import("@sinclair/typebox").TUnsafe<Record<never, unknown>>)[]>>>;
 }>;

package/dist/frontend-connect/fetch-endpoint.d.ts

@@ -95,7 +95,10 @@ export type FetchEndpointOutput<EndpointToFetch extends Readonly<SelectFrom<Endp
     response: Readonly<Response>;
 }> | Readonly<{
     ok: false;
-    data: string | undefined;
+    data: EndpointToFetch extends SelectFrom<EndpointDefinition, {
+        requestDataShape: true;
+        responseDataShape: true;
+    }> ? EndpointExecutorData<EndpointToFetch>['response'] | string | undefined : any;
     response: Readonly<Response>;
 }>;
 /**

package/dist/frontend-connect/fetch-endpoint.js

@@ -1,6 +1,6 @@
 import { check } from '@augment-vir/assert';
 import { addPrefix, filterMap, getObjectTypedEntries, HttpMethod, mapObject, } from '@augment-vir/common';
-import { assertValidShape } from 'object-shape-tester';
+import { assertValidShape, checkWrapValidShape } from 'object-shape-tester';
 import { buildUrl } from 'url-vir';
 import { parseJsonWithUndefined } from '../augments/json.js';
 function defaultFetch(...[url, requestInit,]) {
@@ -64,10 +64,11 @@ export async function fetchEndpoint(endpoint, ...params) {
     const { requestInit, url } = buildEndpointRequestInit(endpoint, ...params);
     /* node:coverage ignore next: all tests mock fetch so we're never going to have a fallback here. */
     const response = await (fetch || defaultFetch)(url, requestInit, endpoint);
+    const responseText = await response.clone().text();
+    const responseData = endpoint.responseDataShape
+        ? parseJsonWithUndefined(responseText)
+        : undefined;
     if (response.ok) {
-        const responseData = endpoint.responseDataShape
-            ? parseJsonWithUndefined(await response.text())
-            : undefined;
         if (endpoint.responseDataShape && !bypassResponseValidation) {
             assertValidShape(responseData, endpoint.responseDataShape, { allowExtraKeys: true });
         }
@@ -78,10 +79,16 @@ export async function fetchEndpoint(endpoint, ...params) {
         };
     }
     else {
+        const validResponseData = endpoint.responseDataShape
+            ? bypassResponseValidation
+                ? responseData
+                : checkWrapValidShape(responseData, endpoint.responseDataShape, {
+                    allowExtraKeys: true,
+                })
+            : undefined;
         return {
             ok: false,
-            /** This will be an error message. */
-            data: (await response.text()) || undefined,
+            data: validResponseData || responseText || undefined,
             response,
         };
     }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@rest-vir/define-service",
-    "version": "1.4.0",
+    "version": "1.5.0",
     "description": "Define an connect to a declarative and type safe REST and WebSocket service.",
     "keywords": [
         "rest",
@@ -40,14 +40,14 @@
         "test:update": "npm test update"
     },
     "dependencies": {
-        "@augment-vir/assert": "^31.55.0",
-        "@augment-vir/common": "^31.55.0",
+        "@augment-vir/assert": "^31.59.1",
+        "@augment-vir/common": "^31.59.1",
         "date-vir": "^8.1.0",
-        "type-fest": "^5.3.1",
+        "type-fest": "^5.4.3",
         "url-vir": "^2.1.7"
     },
     "devDependencies": {
-        "@augment-vir/test": "^31.55.0",
+        "@augment-vir/test": "^31.59.1",
         "@web/dev-server-esbuild": "^1.0.4",
         "@web/test-runner": "^0.20.2",
         "@web/test-runner-commands": "^0.9.0",
@@ -55,9 +55,9 @@
         "@web/test-runner-visual-regression": "^0.10.0",
         "istanbul-smart-text-reporter": "^1.1.5",
         "markdown-code-example-inserter": "^3.0.3",
-        "object-shape-tester": "^6.10.1",
-        "typedoc": "^0.28.15",
-        "ws": "^8.18.3"
+        "object-shape-tester": "^6.11.0",
+        "typedoc": "^0.28.16",
+        "ws": "^8.19.0"
     },
     "peerDependencies": {
         "object-shape-tester": ">=5"