@zauru-sdk/common 6.3.0 → 6.3.2

package/CHANGELOG.md

@@ -3,6 +3,25 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [6.3.2](https://github.com/intuitiva/zauru-typescript-sdk/compare/@zauru-sdk/common@6.3.1...@zauru-sdk/common@6.3.2) (2026-09-09)
+
+**Note:** Version bump only for package @zauru-sdk/common
+
+
+
+
+
+## [6.3.1](https://github.com/intuitiva/zauru-typescript-sdk/compare/@zauru-sdk/common@6.3.0...@zauru-sdk/common@6.3.1) (2026-09-09)
+
+
+### Bug Fixes
+
+* read rejection percentage from memo objects ([3e1292a](https://github.com/intuitiva/zauru-typescript-sdk/commit/3e1292a2d77f1347a8ceef526019298618fe2ca0))
+
+
+
+
+
 # [6.3.0](https://github.com/intuitiva/zauru-typescript-sdk/compare/@zauru-sdk/common@6.2.5...@zauru-sdk/common@6.3.0) (2026-09-08)
 
 

package/dist/common.d.ts

@@ -1,13 +1,17 @@
 import "moment-timezone";
 import type { PayeeGraphQL, AxiosUtilsResponse, BasketSchema, SelectFieldOption, JsonMemoType } from "@zauru-sdk/types";
 export declare const DESTINOS_MUESTRA_OPTIONS: SelectFieldOption[];
-export declare const parseJsonMemo: (memo?: string) => JsonMemoType;
+export declare const parseJsonMemo: (memo?: string | object) => JsonMemoType;
 export declare const stringifyJsonMemo: (memo: JsonMemoType) => string;
-export declare const mergeJsonMemo: (memo: string | undefined, patch: Partial<JsonMemoType>) => string;
+export declare const mergeJsonMemo: (memo: string | object | undefined, patch: Partial<JsonMemoType>) => string;
+/**
+ * Reads rejection % from memo.rejectionPercentage.
+ * Does not use purchase_orders.discount (monetary Baculo field).
+ */
 export declare const getRejectionPercentage: (source: string | {
-    memo?: string;
+    memo?: string | object;
 } | JsonMemoType | undefined) => number;
-export declare const setRejectionPercentage: (memo: string | undefined, percentage: number) => string;
+export declare const setRejectionPercentage: (memo: string | object | undefined, percentage: number) => string;
 /**
  * Obtener el objeto de canastas en base al memo
  * @param memo

package/dist/esm/common.js

@@ -31,8 +31,11 @@ exports.DESTINOS_MUESTRA_OPTIONS = [
 moment_1.default.locale("es");
 // Helper function to parse JSON memo safely
 const parseJsonMemo = (memo) => {
-    if (!memo)
+    if (memo == null || memo === "")
         return {};
+    if (typeof memo === "object") {
+        return memo;
+    }
     try {
         return JSON.parse(memo);
     }
@@ -48,21 +51,31 @@ const stringifyJsonMemo = (memo) => JSON.stringify(memo);
 exports.stringifyJsonMemo = stringifyJsonMemo;
 const mergeJsonMemo = (memo, patch) => (0, exports.stringifyJsonMemo)({ ...(0, exports.parseJsonMemo)(memo), ...patch });
 exports.mergeJsonMemo = mergeJsonMemo;
+const toFiniteNumber = (value) => {
+    if (value == null || value === "")
+        return undefined;
+    const parsed = Number(value);
+    return Number.isFinite(parsed) ? parsed : undefined;
+};
+/**
+ * Reads rejection % from memo.rejectionPercentage.
+ * Does not use purchase_orders.discount (monetary Baculo field).
+ */
 const getRejectionPercentage = (source) => {
     if (source == null || source === "")
         return 0;
-    let parsed;
     if (typeof source === "string") {
-        parsed = (0, exports.parseJsonMemo)(source);
+        return toFiniteNumber((0, exports.parseJsonMemo)(source).rejectionPercentage) ?? 0;
     }
-    else if (typeof source.memo === "string") {
-        parsed = (0, exports.parseJsonMemo)(source.memo);
-    }
-    else {
-        parsed = source;
+    if (typeof source !== "object")
+        return 0;
+    const record = source;
+    if (record.memo != null && record.memo !== "") {
+        const fromMemo = toFiniteNumber((0, exports.parseJsonMemo)(record.memo).rejectionPercentage);
+        if (fromMemo !== undefined)
+            return fromMemo;
     }
-    const value = Number(parsed.rejectionPercentage);
-    return Number.isFinite(value) ? value : 0;
+    return toFiniteNumber(record.rejectionPercentage) ?? 0;
 };
 exports.getRejectionPercentage = getRejectionPercentage;
 const setRejectionPercentage = (memo, percentage) => (0, exports.mergeJsonMemo)(memo, { rejectionPercentage: percentage });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zauru-sdk/common",
-  "version": "6.3.0",
+  "version": "6.3.2",
   "description": "Funciones de uso común en el desarrollo de webapps en Zauru",
   "main": "./dist/esm/index.js",
   "module": "./dist/esm/index.js",
@@ -21,9 +21,9 @@
   },
   "dependencies": {
     "@remix-run/node": "^2.8.1",
-    "@zauru-sdk/types": "^6.9.0",
+    "@zauru-sdk/types": "^6.10.0",
     "moment": "^2.30.1",
     "moment-timezone": "^0.5.45"
   },
-  "gitHead": "2cc49202c68f74f1ef031322b108b1461b9c9ad7"
+  "gitHead": "bde6df12ceda8636075e0adb3b79a563076d51bf"
 }

package/src/common.ts

@@ -18,8 +18,11 @@ export const DESTINOS_MUESTRA_OPTIONS: SelectFieldOption[] = [
 moment.locale("es");
 
 // Helper function to parse JSON memo safely
-export const parseJsonMemo = (memo?: string): JsonMemoType => {
-  if (!memo) return {};
+export const parseJsonMemo = (memo?: string | object): JsonMemoType => {
+  if (memo == null || memo === "") return {};
+  if (typeof memo === "object") {
+    return memo as JsonMemoType;
+  }
   try {
     return JSON.parse(memo);
   } catch (error) {
@@ -34,30 +37,48 @@ export const stringifyJsonMemo = (memo: JsonMemoType): string =>
   JSON.stringify(memo);
 
 export const mergeJsonMemo = (
-  memo: string | undefined,
+  memo: string | object | undefined,
   patch: Partial<JsonMemoType>,
 ): string => stringifyJsonMemo({ ...parseJsonMemo(memo), ...patch });
 
+const toFiniteNumber = (value: unknown): number | undefined => {
+  if (value == null || value === "") return undefined;
+  const parsed = Number(value);
+  return Number.isFinite(parsed) ? parsed : undefined;
+};
+
+/**
+ * Reads rejection % from memo.rejectionPercentage.
+ * Does not use purchase_orders.discount (monetary Baculo field).
+ */
 export const getRejectionPercentage = (
-  source: string | { memo?: string } | JsonMemoType | undefined,
+  source: string | { memo?: string | object } | JsonMemoType | undefined,
 ): number => {
   if (source == null || source === "") return 0;
 
-  let parsed: JsonMemoType;
   if (typeof source === "string") {
-    parsed = parseJsonMemo(source);
-  } else if (typeof (source as { memo?: unknown }).memo === "string") {
-    parsed = parseJsonMemo((source as { memo: string }).memo);
-  } else {
-    parsed = source as JsonMemoType;
+    return toFiniteNumber(parseJsonMemo(source).rejectionPercentage) ?? 0;
+  }
+
+  if (typeof source !== "object") return 0;
+
+  const record = source as {
+    memo?: string | object;
+    rejectionPercentage?: number | string;
+  };
+
+  if (record.memo != null && record.memo !== "") {
+    const fromMemo = toFiniteNumber(
+      parseJsonMemo(record.memo).rejectionPercentage,
+    );
+    if (fromMemo !== undefined) return fromMemo;
   }
 
-  const value = Number(parsed.rejectionPercentage);
-  return Number.isFinite(value) ? value : 0;
+  return toFiniteNumber(record.rejectionPercentage) ?? 0;
 };
 
 export const setRejectionPercentage = (
-  memo: string | undefined,
+  memo: string | object | undefined,
   percentage: number,
 ): string => mergeJsonMemo(memo, { rejectionPercentage: percentage });