erpnext-queue-client 2.5.0 → 2.5.2

package/dist/erpnext/doctypes/shipment.js

@@ -5,28 +5,11 @@ const doctypeSubmittableResourceRequest_1 = require("../doctypeSubmittableResour
 const methodRequest_1 = require("../methodRequest");
 const Shipment_1 = require("../model/Shipment");
 const zodContextOptionals_1 = require("../../utils/zodContextOptionals");
-class ERPNextShipment {
-    temporalClient;
-    baseRequest;
+class ERPNextShipment extends doctypeSubmittableResourceRequest_1.ERPNextDoctypeSubmittableResourceRequest {
     methodRequest;
-    getList;
-    getById;
-    updateById;
-    deleteById;
-    create;
-    cancel;
-    submit;
     constructor(temporalClient) {
-        this.temporalClient = temporalClient;
-        this.baseRequest = new doctypeSubmittableResourceRequest_1.ERPNextDoctypeSubmittableResourceRequest(temporalClient, "Shipment", Shipment_1.Shipment);
+        super(temporalClient, "Shipment", Shipment_1.Shipment);
         this.methodRequest = new methodRequest_1.ERPNextMethodRequest(temporalClient);
-        this.getList = this.baseRequest.getList;
-        this.getById = this.baseRequest.getById;
-        this.updateById = this.baseRequest.updateById;
-        this.deleteById = this.baseRequest.deleteById;
-        this.create = this.baseRequest.create;
-        this.cancel = this.baseRequest.cancel;
-        this.submit = this.baseRequest.submit;
     }
     async createShipmentDraft(deliveryNoteName) {
         const draft = await this.methodRequest.request({
@@ -47,7 +30,7 @@ class ERPNextShipment {
         return shipmentInfo;
     }
     async addShippingLabelAndSubmit(shipmentErpNextName, body) {
-        const shipment = await this.baseRequest.updateById({
+        const shipment = await this.updateById({
             resourceId: shipmentErpNextName,
             inputValidationModel: Shipment_1.ShipmentAddShippingLabelAndSubmitBody,
             body,

package/dist/erpnext/item.getList.alias-parsing.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/erpnext/item.getList.alias-parsing.test.js

@@ -0,0 +1,66 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const item_1 = require("./doctypes/item");
+const zod_1 = require("zod");
+const zodUtils_1 = require("../utils/zodUtils");
+describe("Item.getList field alias parsing", () => {
+    test("does not treat field names containing 'as' as aliases", async () => {
+        const temporalClientStub = {
+            executeERPNextRequestWorkflow: async (_requestName, options) => {
+                const { responseValidationModel } = options;
+                if (!responseValidationModel) {
+                    throw new Error("Missing responseValidationModel in stub options");
+                }
+                // Intentionally omit the wrongly-selected key "undefined" that results from
+                // the buggy alias parsing logic in pickFromSchema.
+                const data = {
+                    data: [
+                        {
+                            name: "ITEM-0001",
+                            item_code: "ITEM-0001",
+                            item_name: "Test Item",
+                            ean: "EAN-1",
+                            disabled: 0,
+                            has_variants: 1,
+                            is_purchase_item: 1,
+                            is_stock_item: 1,
+                            custom_product_version: "v1",
+                            custom_supplier: "SUP-1",
+                        },
+                    ],
+                };
+                return (0, zodUtils_1.validateData)(data, responseValidationModel);
+            },
+        };
+        const itemApi = new item_1.ERPNextItem(temporalClientStub);
+        const fields = [
+            "name",
+            "item_code",
+            "item_name",
+            "ean",
+            "disabled",
+            "has_variants",
+            "is_purchase_item",
+            "is_stock_item",
+            "custom_product_version",
+            "custom_supplier",
+        ];
+        const result = await itemApi.getList({
+            fields: fields,
+            limit: 1,
+        });
+        expect(result).toHaveLength(1);
+        expect(result.at(0)?.name).toBeTypeOf("string");
+        expect(result.at(0)?.has_variants).toBeTypeOf("number");
+        expect(result.at(0)?.is_purchase_item).toBeTypeOf("number");
+        expect(result.at(0)?.custom_product_version).toBeTypeOf("string");
+    });
+    test("supports uppercase AS delimiter in alias expressions", () => {
+        const schema = zod_1.z.object({
+            "item_name AS item_title": zod_1.z.string(),
+        });
+        const picked = (0, zodUtils_1.pickFromSchema)(schema, ["item_name AS item_title"]);
+        const parsed = picked.parse({ item_title: "x" });
+        expect(parsed.item_title).toBe("x");
+    });
+});

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

@@ -35,7 +35,7 @@ export declare const Shipment: z.ZodObject<{
     delivery_to_type: z.ZodString;
     delivery_to: z.ZodNullable<z.ZodOptional<z.ZodString>>;
     description_of_content: z.ZodString;
-    incoterm: z.ZodString;
+    incoterm: z.ZodOptional<z.ZodNullable<z.ZodString>>;
     pallets: z.ZodString;
     parcel_template: z.ZodNullable<z.ZodOptional<z.ZodString>>;
     pickup_address_name: z.ZodString;
@@ -107,7 +107,6 @@ export declare const Shipment: z.ZodObject<{
     custom_export_status: z.ZodNullable<z.ZodOptional<z.ZodDefault<z.ZodEnum<["Open", "Customs Declaration in Progress", "Customs Declaration Submitted", "Customs Declaration Confirmed"]>>>>;
 }, "strip", z.ZodTypeAny, {
     status: string;
-    incoterm: string;
     shipment_id: string;
     delivery_address_name: string;
     delivery_address: string;
@@ -143,6 +142,7 @@ export declare const Shipment: z.ZodObject<{
     tracking_status: string;
     value_of_goods: number;
     custom_delivery_country: string;
+    incoterm?: string | null | undefined;
     shipping_provider?: string | null | undefined;
     awb_number?: string | null | undefined;
     carrier_service?: string | null | undefined;
@@ -168,7 +168,6 @@ export declare const Shipment: z.ZodObject<{
     custom_export_status?: "Open" | "Customs Declaration in Progress" | "Customs Declaration Submitted" | "Customs Declaration Confirmed" | null | undefined;
 }, {
     status: string;
-    incoterm: string;
     shipment_id: string;
     delivery_address_name: string;
     delivery_address: string;
@@ -204,6 +203,7 @@ export declare const Shipment: z.ZodObject<{
     tracking_status: string;
     value_of_goods: number;
     custom_delivery_country: string;
+    incoterm?: string | null | undefined;
     shipping_provider?: string | null | undefined;
     awb_number?: string | null | undefined;
     carrier_service?: string | null | undefined;
@@ -244,7 +244,7 @@ export declare const ShipmentAddShippingLabelAndSubmitBody: z.ZodObject<Pick<{
     delivery_to_type: z.ZodString;
     delivery_to: z.ZodNullable<z.ZodOptional<z.ZodString>>;
     description_of_content: z.ZodString;
-    incoterm: z.ZodString;
+    incoterm: z.ZodOptional<z.ZodNullable<z.ZodString>>;
     pallets: z.ZodString;
     parcel_template: z.ZodNullable<z.ZodOptional<z.ZodString>>;
     pickup_address_name: z.ZodString;

package/dist/erpnext/model/Shipment.js

@@ -59,7 +59,7 @@ exports.Shipment = zod_1.z
     delivery_to_type: zod_1.z.string(),
     delivery_to: zod_1.z.string().optional().nullable(),
     description_of_content: zod_1.z.string(),
-    incoterm: zod_1.z.string(),
+    incoterm: zod_1.z.string().nullish(),
     pallets: zod_1.z.string(),
     parcel_template: zod_1.z.string().optional().nullable(),
     pickup_address_name: zod_1.z.string(),

package/dist/index.test.js

@@ -113,8 +113,9 @@ describe.skipIf(constants_1.constants.TEMPORAL_HOST === "test")("Type tests", as
     });
     test("Item List with invalid field list", async () => {
         // don't execute, this is just for type testing statically
-        () => erp.item.getList({
-            // @ts-expect-error
+        () => 
+        // @ts-expect-error barcode is not a valid Item list field
+        erp.item.getList({
             fields: ["name", "barcode"],
             limit: 10,
         });

package/dist/utils/zodUtils.js

@@ -39,16 +39,24 @@ function validateData(data, ValidationModel, isDebug) {
 }
 function pickFromSchema(schema, keys) {
     const pickedShape = keys?.reduce((acc, key) => {
-        if (typeof key === "string" && key.includes("as")) {
-            const [field, alias] = key.split(" as ");
-            if (key in schema.shape) {
-                acc[alias] = schema.shape[key];
+        // ERPNext alias fields use: `<field> as <alias>` (case-insensitive).
+        // Don't treat substring matches like `has_variants` as aliases.
+        if (typeof key === "string") {
+            const match = key.match(/^(.*?)\s+as\s+(.*?)$/i);
+            if (match) {
+                const alias = match[2].trim();
+                if (key in schema.shape) {
+                    acc[alias] = schema.shape[key];
+                }
+                return acc;
             }
+            // Not an alias expression, treat it as a direct field key.
+            if (key in schema.shape)
+                acc[key] = schema.shape[key];
         }
         else {
-            if (key in schema.shape) {
+            if (key in schema.shape)
                 acc[key] = schema.shape[key];
-            }
         }
         return acc;
     }, {});

package/package.json

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