@zeniai/client-epic-state 5.1.17 → 5.1.19

package/lib/entity/customer/customerPayload.d.ts

@@ -1,11 +1,17 @@
 import { URLPayload } from '../../commonPayloadTypes/urlPayload';
 import { Customer, CustomerBase } from './customerState';
 export interface CustomerBasePayload {
-    customer_id: string;
+    customer_id: string | null;
     customer_name: string;
     logo?: URLPayload;
     qbo_id?: string;
 }
+/**
+ * Resolves the effective customer identifier from a payload.
+ * For JE/Deposit transactions, customer_id is null at runtime — qbo_id is the
+ * actual identity.
+ */
+export declare const getCustomerPayloadIdentifier: (customer: CustomerBasePayload) => string | null;
 export declare const toCustomerBase: (payload: CustomerBasePayload, parseOutParentName: boolean) => CustomerBase;
 export declare const toCustomerBasePayload: (customer: CustomerBase | undefined, customerUpdates?: CustomerBase) => CustomerBasePayload;
 export interface CustomerPayload extends CustomerBasePayload {

package/lib/entity/customer/customerPayload.js

@@ -1,16 +1,39 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.toCustomer = exports.toCustomerBasePayload = exports.toCustomerBase = void 0;
+exports.toCustomer = exports.toCustomerBasePayload = exports.toCustomerBase = exports.getCustomerPayloadIdentifier = void 0;
 const urlPayload_1 = require("../../commonPayloadTypes/urlPayload");
 const zeniDayJS_1 = require("../../zeniDayJS");
-const toCustomerBase = (payload, parseOutParentName) => ({
-    id: payload.customer_id,
-    name: parseOutParentName
-        ? removeParentName(payload.customer_name)
-        : payload.customer_name,
-    qboId: payload.qbo_id,
-    logo: payload.logo != null ? (0, urlPayload_1.toURL)(payload.logo) : undefined,
-});
+/**
+ * Resolves the effective customer identifier from a payload.
+ * For JE/Deposit transactions, customer_id is null at runtime — qbo_id is the
+ * actual identity.
+ */
+const getCustomerPayloadIdentifier = (customer) => {
+    const cid = customer.customer_id;
+    if (cid != null && cid !== '') {
+        return cid;
+    }
+    const qid = customer.qbo_id;
+    if (qid != null && qid !== '') {
+        return qid;
+    }
+    return null;
+};
+exports.getCustomerPayloadIdentifier = getCustomerPayloadIdentifier;
+const toCustomerBase = (payload, parseOutParentName) => {
+    const id = (0, exports.getCustomerPayloadIdentifier)(payload);
+    if (id == null) {
+        throw new Error(`toCustomerBase: no identifier (customer_id/qbo_id) found for customer "${payload.customer_name}".`);
+    }
+    return {
+        id,
+        name: parseOutParentName
+            ? removeParentName(payload.customer_name)
+            : payload.customer_name,
+        qboId: payload.qbo_id,
+        logo: payload.logo != null ? (0, urlPayload_1.toURL)(payload.logo) : undefined,
+    };
+};
 exports.toCustomerBase = toCustomerBase;
 const toCustomerBasePayload = (customer, customerUpdates) => {
     if (customerUpdates != null && customerUpdates.id != null) {

package/lib/entity/reimbursement/reimbursementPayload.js

@@ -184,7 +184,8 @@ const toLineDetail = (payload) => ({
     account: payload.account?.account_id != null
         ? (0, accountPayload_1.mapAccountBasePayloadToAccountBase)(payload.account)
         : undefined,
-    customer: payload.customer?.customer_id != null
+    customer: payload.customer != null &&
+        (0, customerPayload_1.getCustomerPayloadIdentifier)(payload.customer) != null
         ? (0, customerPayload_1.toCustomerBase)(payload.customer, false)
         : undefined,
 });

package/lib/entity/transaction/payloadTypes/customerTransactionPayload.js

@@ -12,7 +12,9 @@ const toCustomerTransaction = (payload) => {
     const transaction = (0, transactionPayload_1.toTransaction)(payload);
     return {
         ...transaction,
-        customer: payload.customer_id != null ? (0, customerPayload_1.toCustomerBase)(payload, false) : undefined,
+        customer: (0, customerPayload_1.getCustomerPayloadIdentifier)(payload) != null
+            ? (0, customerPayload_1.toCustomerBase)(payload, false)
+            : undefined,
         documentId: payload.document_id,
         paymentType: (0, paymentType_1.toPaymentTypeStrict)(payload.payment_type),
         paymentTypeName: payload.payment_type_name,

package/lib/entity/transaction/payloadTypes/transactionLinePayload.js

@@ -39,7 +39,7 @@ const toTransactionWithAccountAndClassLine = (payload, currency, type = 'transac
     class: (0, classPayload_1.mapClassBasePayloadToClassBase)(payload.line_detail.class ?? {}), // Note: why bother about empty {} object ? because sometimes backend sends empty objects, sometimes it sends undefined or null
     account: (0, accountPayload_1.mapAccountBasePayloadToAccountBase)(payload.line_detail.account),
     customer: payload.line_detail.customer != null &&
-        payload.line_detail.customer.customer_id != null
+        (0, customerPayload_1.getCustomerPayloadIdentifier)(payload.line_detail.customer) != null
         ? (0, customerPayload_1.toCustomerBase)(payload.line_detail.customer, false)
         : undefined,
     vendor: payload.line_detail.vendor != null &&

package/lib/entity/transaction/payloadTypes/vendorTransactionPayload.d.ts

@@ -12,7 +12,7 @@ export interface VendorTransactionPayload extends VendorBasePayload, Omit<Transa
     payment_type_name?: string;
 }
 export interface VendorTransactionWithCustomerPayload extends VendorTransactionPayload {
-    customer_id: string;
+    customer_id: string | null;
     customer_name: string;
 }
 export declare const isVendorTransactionPayload: (payload: TransactionPayload) => boolean;

package/lib/entity/transactionActivityLog/transactionActivityLogPayload.js

@@ -18,7 +18,10 @@ const toTransactionActivityLogLine = (line) => {
         class: line.accounting_class
             ? (0, classPayload_1.mapClassBasePayloadToClassBase)(line.accounting_class)
             : undefined,
-        customer: line.customer ? (0, customerPayload_1.toCustomerBase)(line.customer, false) : undefined,
+        customer: line.customer != null &&
+            (0, customerPayload_1.getCustomerPayloadIdentifier)(line.customer) != null
+            ? (0, customerPayload_1.toCustomerBase)(line.customer, false)
+            : undefined,
         vendor: line.vendor != null && (0, vendorPayload_1.getVendorPayloadIdentifier)(line.vendor) != null
             ? (0, vendorPayload_1.mapVendorBasePayloadToVendorBase)(line.vendor)
             : undefined,

package/lib/esm/entity/customer/customerPayload.js

@@ -1,13 +1,35 @@
 import { toURL } from '../../commonPayloadTypes/urlPayload';
 import { date } from '../../zeniDayJS';
-export const toCustomerBase = (payload, parseOutParentName) => ({
-    id: payload.customer_id,
-    name: parseOutParentName
-        ? removeParentName(payload.customer_name)
-        : payload.customer_name,
-    qboId: payload.qbo_id,
-    logo: payload.logo != null ? toURL(payload.logo) : undefined,
-});
+/**
+ * Resolves the effective customer identifier from a payload.
+ * For JE/Deposit transactions, customer_id is null at runtime — qbo_id is the
+ * actual identity.
+ */
+export const getCustomerPayloadIdentifier = (customer) => {
+    const cid = customer.customer_id;
+    if (cid != null && cid !== '') {
+        return cid;
+    }
+    const qid = customer.qbo_id;
+    if (qid != null && qid !== '') {
+        return qid;
+    }
+    return null;
+};
+export const toCustomerBase = (payload, parseOutParentName) => {
+    const id = getCustomerPayloadIdentifier(payload);
+    if (id == null) {
+        throw new Error(`toCustomerBase: no identifier (customer_id/qbo_id) found for customer "${payload.customer_name}".`);
+    }
+    return {
+        id,
+        name: parseOutParentName
+            ? removeParentName(payload.customer_name)
+            : payload.customer_name,
+        qboId: payload.qbo_id,
+        logo: payload.logo != null ? toURL(payload.logo) : undefined,
+    };
+};
 export const toCustomerBasePayload = (customer, customerUpdates) => {
     if (customerUpdates != null && customerUpdates.id != null) {
         return {

package/lib/esm/entity/reimbursement/reimbursementPayload.js

@@ -5,7 +5,7 @@ import { date, dateNow } from '../../zeniDayJS';
 import { mapAccountBasePayloadToAccountBase, toAccountPayload, } from '../account/accountPayload';
 import { toBillPaymentMethod, toBillPaymentStatus, toBillStage, toBillStatus, toBulkProcessingDetail, toDeletionInfo, } from '../billPay/billTransaction/billTransactionPayload';
 import { mapClassBasePayloadToClassBase, toClassBasePayload, } from '../class/classPayload';
-import { toCustomerBase, toCustomerBasePayload, } from '../customer/customerPayload';
+import { getCustomerPayloadIdentifier, toCustomerBase, toCustomerBasePayload, } from '../customer/customerPayload';
 import { mapMerchantBasePayloadToMerchantBase, toMerchantBasePayload, } from '../merchant/merchantPayload';
 import { toAttachment, } from '../transaction/payloadTypes/attachmentPayload';
 const ALL_REIMBURSEMENT_TRANSACTION_TYPE = ['reimbursement'];
@@ -178,7 +178,8 @@ const toLineDetail = (payload) => ({
     account: payload.account?.account_id != null
         ? mapAccountBasePayloadToAccountBase(payload.account)
         : undefined,
-    customer: payload.customer?.customer_id != null
+    customer: payload.customer != null &&
+        getCustomerPayloadIdentifier(payload.customer) != null
         ? toCustomerBase(payload.customer, false)
         : undefined,
 });

package/lib/esm/entity/transaction/payloadTypes/customerTransactionPayload.js

@@ -1,5 +1,5 @@
 import { toAmountWC } from '../../../commonStateTypes/amount';
-import { toCustomerBase, toCustomerBasePayload, } from '../../customer/customerPayload';
+import { getCustomerPayloadIdentifier, toCustomerBase, toCustomerBasePayload, } from '../../customer/customerPayload';
 import { toPaymentTypeStrict } from '../stateTypes/paymentType';
 import { toCustomerTransactionTypeStrict } from '../stateTypes/transactionType';
 import { toTransaction, toTransactionPayload, } from './transactionPayload';
@@ -8,7 +8,9 @@ export const toCustomerTransaction = (payload) => {
     const transaction = toTransaction(payload);
     return {
         ...transaction,
-        customer: payload.customer_id != null ? toCustomerBase(payload, false) : undefined,
+        customer: getCustomerPayloadIdentifier(payload) != null
+            ? toCustomerBase(payload, false)
+            : undefined,
         documentId: payload.document_id,
         paymentType: toPaymentTypeStrict(payload.payment_type),
         paymentTypeName: payload.payment_type_name,

package/lib/esm/entity/transaction/payloadTypes/transactionLinePayload.js

@@ -1,7 +1,7 @@
 import { toAmountWC } from '../../../commonStateTypes/amount';
 import { mapAccountBasePayloadToAccountBase, toAccountPayload, } from '../../account/accountPayload';
 import { mapClassBasePayloadToClassBase, toClassBasePayload, } from '../../class/classPayload';
-import { toCustomerBase, toCustomerBasePayload, } from '../../customer/customerPayload';
+import { getCustomerPayloadIdentifier, toCustomerBase, toCustomerBasePayload, } from '../../customer/customerPayload';
 import { mapProjectBasePayloadToProjectBase, toProjectBasePayload, } from '../../project/projectPayload';
 import { getVendorPayloadIdentifier, mapVendorBasePayloadToVendorBase, toVendorBasePayload, } from '../../vendor/vendorPayload';
 import { toCategorizationStatusType, toPlatformLineDetailType, } from '../stateTypes/transactionLine';
@@ -35,7 +35,7 @@ export const toTransactionWithAccountAndClassLine = (payload, currency, type = '
     class: mapClassBasePayloadToClassBase(payload.line_detail.class ?? {}), // Note: why bother about empty {} object ? because sometimes backend sends empty objects, sometimes it sends undefined or null
     account: mapAccountBasePayloadToAccountBase(payload.line_detail.account),
     customer: payload.line_detail.customer != null &&
-        payload.line_detail.customer.customer_id != null
+        getCustomerPayloadIdentifier(payload.line_detail.customer) != null
         ? toCustomerBase(payload.line_detail.customer, false)
         : undefined,
     vendor: payload.line_detail.vendor != null &&

package/lib/esm/entity/transactionActivityLog/transactionActivityLogPayload.js

@@ -1,7 +1,7 @@
 import { date } from '../../zeniDayJS';
 import { mapAccountBasePayloadToAccountBase, } from '../account/accountPayload';
 import { mapClassBasePayloadToClassBase, } from '../class/classPayload';
-import { toCustomerBase } from '../customer/customerPayload';
+import { getCustomerPayloadIdentifier, toCustomerBase } from '../customer/customerPayload';
 import { toAISummary, } from '../transaction/payloadTypes/transactionLinePayload';
 import { getVendorPayloadIdentifier, mapVendorBasePayloadToVendorBase, } from '../vendor/vendorPayload';
 import { toActivityLogEventType, } from './transactionActivityLogState';
@@ -14,7 +14,10 @@ const toTransactionActivityLogLine = (line) => {
         class: line.accounting_class
             ? mapClassBasePayloadToClassBase(line.accounting_class)
             : undefined,
-        customer: line.customer ? toCustomerBase(line.customer, false) : undefined,
+        customer: line.customer != null &&
+            getCustomerPayloadIdentifier(line.customer) != null
+            ? toCustomerBase(line.customer, false)
+            : undefined,
         vendor: line.vendor != null && getVendorPayloadIdentifier(line.vendor) != null
             ? mapVendorBasePayloadToVendorBase(line.vendor)
             : undefined,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zeniai/client-epic-state",
-  "version": "5.1.17",
+  "version": "5.1.19",
   "description": "Shared module between Web & Mobile containing required abstractions for state management, async network communication. ",
   "main": "lib/index.js",
   "module": "lib/esm/index.js",
@@ -33,6 +33,42 @@
       "require": "./lib/view/*.js"
     }
   },
+  "scripts": {
+    "clean": "rimraf lib",
+    "test:typecheck": "echo 'Typechecking test files...' && NODE_OPTIONS=--max-old-space-size=4096 time tsc -p tsconfig.typecheck.json",
+    "test:typecheck-modified-files": "chmod +x ./scripts/tsc-modified-files.sh && SINCE_DAYS=2 ./scripts/tsc-modified-files.sh master tsconfig.typecheck.json",
+    "test:typecheck-mocks": "echo 'Typechecking mock files...' && NODE_OPTIONS=--max-old-space-size=4096 time tsc -p tsconfig.mocks.json",
+    "test:typecheck-mocks-modified-files": "chmod +x ./scripts/tsc-modified-files.sh && SINCE_DAYS=2 ./scripts/tsc-modified-files.sh master tsconfig.mocks.json",
+    "test": "pnpm lint-modified-files && pnpm test:typecheck-modified-files && pnpm test:typecheck-mocks-modified-files && vitest run --typecheck",
+    "test:full": "pnpm test:typecheck-modified-files && pnpm test:typecheck-mocks-modified-files && vitest run --typecheck",
+    "lint": "eslint . --ext .js,.jsx,.ts,.tsx --max-warnings 0 --fix",
+    "lint-modified-files": "chmod +x ./scripts/lint-modified-files.sh && ./scripts/lint-modified-files.sh",
+    "lint:fix-modified-files": "chmod +x ./scripts/lint-modified-files.sh && LINT_FIX=true ./scripts/lint-modified-files.sh",
+    "tsc-modified-files": "chmod +x ./scripts/tsc-modified-files.sh && ./scripts/tsc-modified-files.sh",
+    "typecheck": "tsc --noEmit",
+    "find-dead-code": "ts-prune | grep -v '(used in module)'",
+    "find-unused-exports": "ts-unused-exports ./tsconfig.json",
+    "circular-dependency": "npx madge --circular --extensions ts ./src",
+    "build": "concurrently --kill-others-on-fail --handle-input \"echo 'TypeScript build in progress...' && time tsc && echo 'ESM build...' && tsc -p tsconfig.esm.json\" \"echo 'Running tests and typecheck (incl. lint-modified-files)...' && pnpm test\" && rimraf \"lib/**/__mocks__\" \"lib/**/__testHelpers__\"",
+    "only-build": "eslint . --ext .js,.jsx,.ts,.tsx && time tsc && tsc -p tsconfig.esm.json && rimraf \"lib/**/__mocks__\" \"lib/**/__testHelpers__\"",
+    "only-build-dev": "eslint . --ext .js,.jsx,.ts,.tsx && time tsc -p tsconfig.dev.json && tsc -p tsconfig.esm.dev.json && rimraf \"lib/**/__mocks__\" \"lib/**/__testHelpers__\"",
+    "format": "prettier --write --ignore-unknown \"src/**/*\" && pnpm lint",
+    "format-watch": "(git diff --name-only --diff-filter=ACM && git ls-files --others --exclude-standard) | grep -E '\\.(ts|tsx)$' | xargs prettier --write",
+    "check-version": "node ./scripts/check_version.js",
+    "prepublishOnly": "chmod +x ./scripts/branch_validation.sh && ./scripts/branch_validation.sh && pnpm update-slack-group-topic && pnpm run check-version && pnpm clean && pnpm build",
+    "version": "git add -A src",
+    "postversion": "git push && git push --tags",
+    "check-dependencies": "node ./scripts/check_dependencies.js",
+    "clean-overrides": "node ./scripts/clean_overrides.js",
+    "create-release-branch": "chmod +x ./scripts/create_release_branch.sh && ./scripts/create_release_branch.sh",
+    "bump-update-web-app-cockpit-beta": "chmod +x ./scripts/bump_and_update_web_app_ui_beta.sh && ./scripts/bump_and_update_web_app_ui_beta.sh",
+    "send-release-notes": "chmod +x ./scripts/send_release_notes.sh && ./scripts/send_release_notes.sh",
+    "update-slack-group-topic": "chmod +x ./scripts/update-slack-group-topic.sh && ./scripts/update-slack-group-topic.sh",
+    "build-beta-and-copy": "chmod +x ./scripts/build_and_copy.sh && ./scripts/build_and_copy.sh",
+    "pr-approvals-update": "chmod +x ./scripts/pr_approvals.sh && ./scripts/pr_approvals.sh",
+    "raise-pr-automation": "chmod +x ./scripts/raise_pr_automation.sh && ./scripts/raise_pr_automation.sh",
+    "test:file": "scripts/test-file.sh"
+  },
   "files": [
     "lib/**/*"
   ],
@@ -103,39 +139,32 @@
     "dayjs": "^1.10.7",
     "url-parse": "^1.5.10"
   },
-  "scripts": {
-    "clean": "rimraf lib",
-    "test:typecheck": "echo 'Typechecking test files...' && NODE_OPTIONS=--max-old-space-size=4096 time tsc -p tsconfig.typecheck.json",
-    "test:typecheck-modified-files": "chmod +x ./scripts/tsc-modified-files.sh && SINCE_DAYS=2 ./scripts/tsc-modified-files.sh master tsconfig.typecheck.json",
-    "test:typecheck-mocks": "echo 'Typechecking mock files...' && NODE_OPTIONS=--max-old-space-size=4096 time tsc -p tsconfig.mocks.json",
-    "test:typecheck-mocks-modified-files": "chmod +x ./scripts/tsc-modified-files.sh && SINCE_DAYS=2 ./scripts/tsc-modified-files.sh master tsconfig.mocks.json",
-    "test": "pnpm lint-modified-files && pnpm test:typecheck-modified-files && pnpm test:typecheck-mocks-modified-files && vitest run --typecheck",
-    "test:full": "pnpm test:typecheck-modified-files && pnpm test:typecheck-mocks-modified-files && vitest run --typecheck",
-    "lint": "eslint . --ext .js,.jsx,.ts,.tsx --max-warnings 0 --fix",
-    "lint-modified-files": "chmod +x ./scripts/lint-modified-files.sh && ./scripts/lint-modified-files.sh",
-    "lint:fix-modified-files": "chmod +x ./scripts/lint-modified-files.sh && LINT_FIX=true ./scripts/lint-modified-files.sh",
-    "tsc-modified-files": "chmod +x ./scripts/tsc-modified-files.sh && ./scripts/tsc-modified-files.sh",
-    "typecheck": "tsc --noEmit",
-    "find-dead-code": "ts-prune | grep -v '(used in module)'",
-    "find-unused-exports": "ts-unused-exports ./tsconfig.json",
-    "circular-dependency": "npx madge --circular --extensions ts ./src",
-    "build": "concurrently --kill-others-on-fail --handle-input \"echo 'TypeScript build in progress...' && time tsc && echo 'ESM build...' && tsc -p tsconfig.esm.json\" \"echo 'Running tests and typecheck (incl. lint-modified-files)...' && pnpm test\" && rimraf \"lib/**/__mocks__\" \"lib/**/__testHelpers__\"",
-    "only-build": "eslint . --ext .js,.jsx,.ts,.tsx && time tsc && tsc -p tsconfig.esm.json && rimraf \"lib/**/__mocks__\" \"lib/**/__testHelpers__\"",
-    "only-build-dev": "eslint . --ext .js,.jsx,.ts,.tsx && time tsc -p tsconfig.dev.json && tsc -p tsconfig.esm.dev.json && rimraf \"lib/**/__mocks__\" \"lib/**/__testHelpers__\"",
-    "format": "prettier --write --ignore-unknown \"src/**/*\" && pnpm lint",
-    "format-watch": "(git diff --name-only --diff-filter=ACM && git ls-files --others --exclude-standard) | grep -E '\\.(ts|tsx)$' | xargs prettier --write",
-    "check-version": "node ./scripts/check_version.js",
-    "version": "git add -A src",
-    "postversion": "git push && git push --tags",
-    "check-dependencies": "node ./scripts/check_dependencies.js",
-    "clean-overrides": "node ./scripts/clean_overrides.js",
-    "create-release-branch": "chmod +x ./scripts/create_release_branch.sh && ./scripts/create_release_branch.sh",
-    "bump-update-web-app-cockpit-beta": "chmod +x ./scripts/bump_and_update_web_app_ui_beta.sh && ./scripts/bump_and_update_web_app_ui_beta.sh",
-    "send-release-notes": "chmod +x ./scripts/send_release_notes.sh && ./scripts/send_release_notes.sh",
-    "update-slack-group-topic": "chmod +x ./scripts/update-slack-group-topic.sh && ./scripts/update-slack-group-topic.sh",
-    "build-beta-and-copy": "chmod +x ./scripts/build_and_copy.sh && ./scripts/build_and_copy.sh",
-    "pr-approvals-update": "chmod +x ./scripts/pr_approvals.sh && ./scripts/pr_approvals.sh",
-    "raise-pr-automation": "chmod +x ./scripts/raise_pr_automation.sh && ./scripts/raise_pr_automation.sh",
-    "test:file": "scripts/test-file.sh"
+  "pnpm": {
+    "overrides": {
+      "@babel/helpers": ">=7.26.10",
+      "@babel/runtime": ">=7.26.10",
+      "@babel/traverse": ">=7.23.2",
+      "@eslint/eslintrc>ajv": ">=6.14.0 <7",
+      "@eslint/plugin-kit": ">=0.3.4",
+      "@modelcontextprotocol/sdk": ">=1.25.2",
+      "ajv": ">=8.18.0",
+      "body-parser": ">=2.2.1",
+      "brace-expansion": ">=5.0.6",
+      "cross-spawn": ">=7.0.5",
+      "eslint>ajv": ">=6.14.0 <7",
+      "flatted": ">=3.4.2",
+      "glob": ">=11.1.0",
+      "hono": ">=4.11.4",
+      "immutable": ">=4.3.8 <5",
+      "js-yaml": ">=4.1.1",
+      "minimatch": ">=10.2.3",
+      "picomatch": ">=4.0.4",
+      "postcss": ">=8.5.12",
+      "qs": "^6.14.1",
+      "rollup": ">=4.59.0",
+      "tough-cookie": "=4.1.3",
+      "vite": ">=7.3.2 <8",
+      "yaml": ">=1.10.3"
+    }
   }
 }