@happyvertical/smrt-web 0.40.60 → 0.40.61

package/dist/index.d.ts

@@ -12,7 +12,8 @@ export declare function buildListQuery(params?: Record<string, unknown>): string
  * Build CRUD fetchers from a generated collection definition — the same URL
  * scheme and payload handling as the generated REST client
  * (`basePath + endpoint`), with one improvement: HTTP error statuses reject
- * with the server's `{ error }` body instead of resolving with it.
+ * instead of resolving with them. Typed/legacy 4xx detail is retained; 5xx
+ * failures stay opaque and payload-free.
  */
 export declare function createDefinitionFetchers(definition: SmrtWebCollectionDefinition<object>, basePath?: string, fetchFn?: typeof fetch): SmrtCrudFetchers;
 
@@ -699,6 +700,13 @@ export declare interface SmrtWebCollection<TData extends object> {
 export declare interface SmrtWebCollectionDefinition<TData extends object = object> {
     /** REST collection name (e.g. `products`). */
     name: string;
+    /**
+     * Canonical qualified model identity (e.g.
+     * `@happyvertical/smrt-products:Product`). Generated definitions always
+     * provide it; optional here for source compatibility with manual literals
+     * authored before policy-aware web collections.
+     */
+    objectRef?: string;
     /** Source class name (e.g. `Product`). */
     className: string;
     /** Path under the API base path (e.g. `/products`). */
@@ -907,11 +915,14 @@ export declare type SmrtWebRelationshipKind = 'foreignKey' | 'crossPackageRef' |
  * Raised when a generated-client call resolved with an error payload
  * (`{ error: string }` from the generated REST routes) or an unexpected shape.
  * Thrown inside a mutation handler, this triggers the automatic rollback of
- * optimistic state.
+ * optimistic state. HTTP 5xx failures are always opaque: their server payload
+ * is neither parsed nor retained because it may contain internal detail.
  */
 export declare class SmrtWebRequestError extends Error {
     readonly payload: unknown;
-    constructor(message: string, payload?: unknown);
+    readonly status?: number;
+    readonly code?: string;
+    constructor(message: string, payload?: unknown, status?: number, code?: string);
 }
 
 /** A row as stored in the client collection: the DTO plus a required key. */

package/dist/index.js

@@ -1529,12 +1529,32 @@ async function dispatch(fetchers, definition, action, args) {
 //#region src/index.ts
 var SmrtWebRequestError = class extends Error {
 	payload;
-	constructor(message, payload) {
+	status;
+	code;
+	constructor(message, payload, status, code) {
 		super(message);
 		this.name = "SmrtWebRequestError";
 		this.payload = payload;
+		if (status !== void 0) this.status = status;
+		if (code !== void 0) this.code = code;
 	}
 };
+function getWebErrorDetail(payload) {
+	if (!payload || typeof payload !== "object") return {};
+	const error = payload.error;
+	if (typeof error === "string") return { message: error };
+	if (!error || typeof error !== "object" || Array.isArray(error)) return {};
+	const failure = error;
+	return {
+		message: typeof failure.message === "string" ? failure.message : void 0,
+		code: typeof failure.code === "string" ? failure.code : void 0
+	};
+}
+function createHttpRequestError(collectionName, status, payload) {
+	if (status >= 500) return new SmrtWebRequestError(`[smrt-web] ${collectionName} request failed: server error`, void 0, status);
+	const detail = getWebErrorDetail(payload);
+	return new SmrtWebRequestError(`[smrt-web] ${collectionName} request failed: ${detail.message ?? `HTTP ${status}`}`, payload, status, detail.code);
+}
 function unwrapListResult(result, collectionName) {
 	if (Array.isArray(result)) return result;
 	if (result && typeof result === "object") {
@@ -1584,12 +1604,12 @@ function createDefinitionFetchers(definition, basePath = "/api/v1", fetchFn = (.
 	const collectionUrl = `${basePath}${definition.endpoint}`;
 	const headers = { "Content-Type": "application/json" };
 	const parse = async (response) => {
-		const payload = await response.json().catch(() => null);
 		if (!response.ok) {
-			const message = payload && typeof payload === "object" && typeof payload.error === "string" ? String(payload.error) : `HTTP ${response.status}`;
-			throw new SmrtWebRequestError(`[smrt-web] ${definition.name} request failed: ${message}`, payload);
+			if (response.status >= 500) throw createHttpRequestError(definition.name, response.status);
+			const payload = await response.json().catch(() => null);
+			throw createHttpRequestError(definition.name, response.status, payload);
 		}
-		return payload;
+		return response.json().catch(() => null);
 	};
 	return {
 		list: async (params) => parse(await fetchFn(`${collectionUrl}${buildListQuery(params)}`, { headers })),
@@ -1609,7 +1629,11 @@ function createDefinitionFetchers(definition, basePath = "/api/v1", fetchFn = (.
 				method: "DELETE",
 				headers
 			});
-			if (!response.ok) throw new SmrtWebRequestError(`[smrt-web] delete(${definition.name}) failed: HTTP ${response.status}`);
+			if (!response.ok) {
+				if (response.status >= 500) throw createHttpRequestError(definition.name, response.status);
+				const payload = await response.json().catch(() => null);
+				throw createHttpRequestError(definition.name, response.status, payload);
+			}
 			return true;
 		}
 	};