erpnext-queue-client 2.4.4 → 2.5.0

package/dist/client.js

@@ -1,7 +1,11 @@
 "use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.TemporalClient = void 0;
 const client_1 = require("@temporalio/client");
+const zod_1 = __importDefault(require("zod"));
 const logger_1 = require("./utils/logger");
 const zodUtils_1 = require("./utils/zodUtils");
 const nanoid_1 = require("nanoid");
@@ -119,7 +123,9 @@ class TemporalClient {
             throw err;
         });
         logger_1.lg.info(`Started workflow ${runId}`);
-        return data;
+        // console.log("TESTTEST", JSON.stringify(responseValidationModel, null, 2));
+        const validatedData = (0, zodUtils_1.validateData)(data, responseValidationModel ?? zod_1.default.any());
+        return validatedData;
     }
 }
 exports.TemporalClient = TemporalClient;

package/dist/erpnext/methodRequest.js

@@ -10,8 +10,6 @@ class ERPNextMethodRequest {
         this.temporalClient = temporalClient;
     }
     async request({ methodName, requestMethod, body, inputValidationModel, responseValidationModel, isAutoWrapResponseValidationWithMessage = true, params, priority = 5, }) {
-        if (responseValidationModel && !responseValidationModel.description)
-            throw new Error("Resource model must have a description");
         const result = await this.temporalClient.executeERPNextRequestWorkflow(`${requestMethod}-${methodName}`, {
             methodName,
             requestMethod,

package/dist/erpnext/model/Waitlist.d.ts

@@ -1,20 +1,35 @@
 import { z } from "zod";
-export declare const WaitlistInput: z.ZodObject<{
+export declare const Waitlist: z.ZodObject<{
     sku: z.ZodString;
     e_mail: z.ZodString;
+    is_notified: z.ZodNumber & {
+        __optionalForInput: true;
+    };
+    requested_at: z.ZodString & {
+        __optionalForInput: true;
+    };
 }, "strip", z.ZodTypeAny, {
     sku: string;
     e_mail: string;
+    is_notified: number;
+    requested_at: string;
 }, {
     sku: string;
     e_mail: string;
+    is_notified: number;
+    requested_at: string;
 }>;
-export type WaitlistInputType = z.infer<typeof WaitlistInput>;
-export declare const Waitlist: z.ZodObject<{
+export type WaitlistType = z.infer<typeof Waitlist>;
+/** @deprecated Use `Waitlist` instead. */
+export declare const WaitlistInput: z.ZodObject<{
     sku: z.ZodString;
     e_mail: z.ZodString;
-    is_notified: z.ZodNumber;
-    requested_at: z.ZodString;
+    is_notified: z.ZodNumber & {
+        __optionalForInput: true;
+    };
+    requested_at: z.ZodString & {
+        __optionalForInput: true;
+    };
 }, "strip", z.ZodTypeAny, {
     sku: string;
     e_mail: string;
@@ -26,4 +41,5 @@ export declare const Waitlist: z.ZodObject<{
     is_notified: number;
     requested_at: string;
 }>;
-export type WaitlistType = z.infer<typeof Waitlist>;
+/** @deprecated Use `WaitlistType` instead. */
+export type WaitlistInputType = WaitlistType;

package/dist/erpnext/model/Waitlist.js

@@ -1,18 +1,15 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.Waitlist = exports.WaitlistInput = void 0;
+exports.WaitlistInput = exports.Waitlist = void 0;
 const zod_1 = require("zod");
-exports.WaitlistInput = zod_1.z
-    .object({
-    sku: zod_1.z.string(),
-    e_mail: zod_1.z.string(),
-})
-    .describe("WaitlistInput");
+const zodContextOptionals_1 = require("../../utils/zodContextOptionals");
 exports.Waitlist = zod_1.z
     .object({
     sku: zod_1.z.string(),
     e_mail: zod_1.z.string(),
-    is_notified: zod_1.z.number(),
-    requested_at: zod_1.z.string(),
+    is_notified: (0, zodContextOptionals_1.optionalForInput)(zod_1.z.number()),
+    requested_at: (0, zodContextOptionals_1.optionalForInput)(zod_1.z.string()),
 })
     .describe("Waitlist");
+/** @deprecated Use `Waitlist` instead. */
+exports.WaitlistInput = exports.Waitlist;

package/dist/erpnext/resourceRequest.js

@@ -34,8 +34,6 @@ class ERPNextResourceRequest {
         return allParams;
     };
     async getList({ resourceName, fields, filters, resourceModel, skip, limit, priority = 5, asDict, params, }) {
-        if (resourceModel && !resourceModel.description)
-            throw new Error("Resource model must have a description");
         const erpNextFields = fields?.length
             ? fields
             : ["name"]; // default field is name
@@ -57,7 +55,7 @@ class ERPNextResourceRequest {
                         data: zod_1.z.array((0, zodUtils_1.pickFromSchema)((0, DocTypeHelpers_1.DocModelListEntry)(resourceModel), erpNextFields.filter((f) => f !== "*") // filter out * when other fields are present
                         )),
                     })
-                        .describe(`${resourceModel.description} List with fields ${JSON.stringify(erpNextFields.filter((f) => f !== "*"))}`)
+                        .describe(`${resourceName} List with fields ${JSON.stringify(erpNextFields.filter((f) => f !== "*"))}`)
                 : zod_1.z.any().describe("Any response");
         const defaultPaginationSize = 500;
         let loopLimit = defaultPaginationSize;
@@ -92,8 +90,6 @@ class ERPNextResourceRequest {
         return results;
     }
     async getById({ resourceName, resourceId, resourceModel, priority = 5, }) {
-        if (resourceModel && !resourceModel.description)
-            throw new Error("Resource model must have a description");
         if (!resourceId)
             throw new Error("Resource id is required");
         const result = await this.temporalClient
@@ -134,8 +130,6 @@ class ERPNextResourceRequest {
         return result.data;
     }
     async create({ resourceName, inputValidationModel, resourceModel, body, params, priority = 5, }) {
-        if (resourceModel && !resourceModel.description)
-            throw new Error("Resource model must have a description");
         const result = await this.temporalClient.executeERPNextRequestWorkflow(`POST-${resourceName}`, {
             requestMethod: "POST",
             resourceName,
@@ -154,8 +148,6 @@ class ERPNextResourceRequest {
         return result.data;
     }
     async deleteById({ resourceName, resourceId, resourceModel, priority = 5, }) {
-        if (resourceModel && !resourceModel.description)
-            throw new Error("Resource model must have a description");
         return await this.temporalClient.executeERPNextRequestWorkflow(`DELETE-${resourceName}`, {
             requestMethod: "DELETE",
             resourceName,

package/dist/utils/zodUtils.d.ts

@@ -1,4 +1,4 @@
 import { z, ZodObject, ZodRawShape } from "zod";
-export declare function validateData<T extends z.ZodTypeAny>(data: unknown, ValidationModel: T): z.infer<T>;
+export declare function validateData<T extends z.ZodTypeAny>(data: unknown, ValidationModel: T, isDebug?: boolean): z.infer<T>;
 export declare function pickFromSchema<T extends ZodRawShape, K extends keyof T>(schema: ZodObject<T>, keys: readonly K[]): ZodObject<Pick<T, K>>;
 export type KeysOf<T> = T extends ZodObject<any> ? keyof z.infer<T> : string;

package/dist/utils/zodUtils.js

@@ -4,33 +4,36 @@ exports.validateData = validateData;
 exports.pickFromSchema = pickFromSchema;
 const zod_1 = require("zod");
 const constants_1 = require("../constants");
-const logger_1 = require("./logger");
-function validateData(data, ValidationModel) {
+function validateData(data, ValidationModel, isDebug) {
     if (!ValidationModel.description)
         throw new Error("ValidationModel.description is required");
     // writeFileSync("test.json", JSON.stringify(data, null, 2));
     if (!(ValidationModel instanceof zod_1.ZodObject) &&
         !(ValidationModel instanceof zod_1.ZodArray)) {
-        console.log(ValidationModel);
         throw new Error("Invalid Zod Model");
     }
     const validationResult = ValidationModel.safeParse(data);
     if (!validationResult.success) {
-        const errorObject = {
-            message: `Validation error in model ${ValidationModel.description}`,
-            error: validationResult.error,
-            input: constants_1.constants.DEBUG_MODE === "true"
-                ? data
-                : "set DEBUG_MODE to true to see the input",
-        };
-        logger_1.lg.error(errorObject);
-        if (data &&
-            typeof data === "object" &&
-            "message" in data &&
-            typeof data.message === "string") {
-            throw new Error(data.message);
-        }
-        throw new Error(errorObject.message, { cause: errorObject.error });
+        const isDebugSet = constants_1.constants.DEBUG_MODE === "true" || isDebug;
+        // ZodError.issues contains the most actionable information; render a readable list.
+        const issues = validationResult.error.issues;
+        const issueStrings = issues.map((issue) => {
+            const path = issue.path.length ? issue.path.join(".") : "(root)";
+            return `"${path}": ${issue.message}`;
+        });
+        const previewCount = 8;
+        const previewLines = "\n──────────────\n\n" +
+            issueStrings.slice(0, previewCount).join("\n") +
+            "\n\n──────────────\n";
+        const moreCount = Math.max(0, issueStrings.length - previewCount);
+        const moreSuffix = moreCount > 0 ? `\n- (+${moreCount} more)` : "";
+        const cleanMessage = `Validation failed for ${ValidationModel.description}\n${previewLines}${moreSuffix}`;
+        // Include the full error as a `cause` only when debugging; otherwise keep stack output clean.
+        if (isDebugSet)
+            console.dir(data, { depth: null });
+        const error = new Error(cleanMessage + "\n" + validationResult.error.toString());
+        delete error.stack;
+        throw error;
     }
     return validationResult.data;
 }

package/package.json

@@ -30,7 +30,7 @@
     "winston": "^3.15.0",
     "zod": "3.25.76"
   },
-  "version": "2.4.4",
+  "version": "2.5.0",
   "devDependencies": {
     "@types/crypto-js": "^4.2.2",
     "@types/lodash": "^4.17.13",