@aderis-erp/cli 1.0.2 → 1.0.4

package/dist/commands/payment-method-ledger-mappings.d.ts

@@ -31,6 +31,7 @@ export default class PaymentMethodLedgerMappings extends Command {
     private runExport;
     private runCamt053Import;
     private runNestedImport;
+    private parseFilterValue;
     private filterFields;
     private sanitizeOutput;
     private getColumns;

package/dist/commands/payment-method-ledger-mappings.js

@@ -8,11 +8,11 @@ const HAS_CAMT053 = false;
 const NESTED_IMPORTS = ["ledger-mappings"];
 const SUBCOMMANDS = [...["list-ledger-mappings", "create-ledger-mappings", "get-ledger-mappings", "update-ledger-mappings", "delete-ledger-mappings"], ...(HAS_IMPORT ? ['import'] : []), ...(HAS_EXPORT ? ['export'] : []), ...NESTED_IMPORTS.map(n => `import-${n}`)];
 const ENDPOINTS = {
-    'list-ledger-mappings': { method: 'GET', path: '/v1/payment-methods/{paymentMethodId}/ledger-mappings', summary: 'List ledger mappings for a payment method', pathParams: ['paymentMethodId'], hasBody: false, isMultipart: false },
-    'create-ledger-mappings': { method: 'POST', path: '/v1/payment-methods/{paymentMethodId}/ledger-mappings', summary: 'Create payment method ledger mapping', pathParams: ['paymentMethodId'], hasBody: true, isMultipart: false },
-    'get-ledger-mappings': { method: 'GET', path: '/v1/payment-methods/{paymentMethodId}/ledger-mappings/{id}', summary: 'Get payment method ledger mapping by ID', pathParams: ['paymentMethodId', 'id'], hasBody: false, isMultipart: false },
-    'update-ledger-mappings': { method: 'PATCH', path: '/v1/payment-methods/{paymentMethodId}/ledger-mappings/{id}', summary: 'Update payment method ledger mapping', pathParams: ['paymentMethodId', 'id'], hasBody: true, isMultipart: false },
-    'delete-ledger-mappings': { method: 'DELETE', path: '/v1/payment-methods/{paymentMethodId}/ledger-mappings/{id}', summary: 'Delete payment method ledger mapping', pathParams: ['paymentMethodId', 'id'], hasBody: false, isMultipart: false },
+    'list-ledger-mappings': { method: 'GET', path: '/v1/payment-methods/{paymentMethodId}/ledger-mappings', summary: 'List ledger mappings for a payment method', pathParams: ['paymentMethodId'], hasBody: false, isMultipart: false, isBinaryResponse: false, bodyArrayFields: [] },
+    'create-ledger-mappings': { method: 'POST', path: '/v1/payment-methods/{paymentMethodId}/ledger-mappings', summary: 'Create payment method ledger mapping', pathParams: ['paymentMethodId'], hasBody: true, isMultipart: false, isBinaryResponse: false, bodyArrayFields: [] },
+    'get-ledger-mappings': { method: 'GET', path: '/v1/payment-methods/{paymentMethodId}/ledger-mappings/{id}', summary: 'Get payment method ledger mapping by ID', pathParams: ['paymentMethodId', 'id'], hasBody: false, isMultipart: false, isBinaryResponse: false, bodyArrayFields: [] },
+    'update-ledger-mappings': { method: 'PATCH', path: '/v1/payment-methods/{paymentMethodId}/ledger-mappings/{id}', summary: 'Update payment method ledger mapping', pathParams: ['paymentMethodId', 'id'], hasBody: true, isMultipart: false, isBinaryResponse: false, bodyArrayFields: [] },
+    'delete-ledger-mappings': { method: 'DELETE', path: '/v1/payment-methods/{paymentMethodId}/ledger-mappings/{id}', summary: 'Delete payment method ledger mapping', pathParams: ['paymentMethodId', 'id'], hasBody: false, isMultipart: false, isBinaryResponse: false, bodyArrayFields: [] },
 };
 export default class PaymentMethodLedgerMappings extends Command {
     static description = 'Manage payment method ledger mappings';
@@ -124,15 +124,7 @@ export default class PaymentMethodLedgerMappings extends Command {
                 if (eqIndex === -1)
                     this.error(`Invalid --filter format: ${f}. Use: --filter key=value`);
                 const key = f.slice(0, eqIndex);
-                let value = f.slice(eqIndex + 1);
-                // Check for operator prefix (with or without $)
-                const opMatch = value.match(/^\$?(eq|ne|in|like|gt|gte|lt|lte|null):/);
-                if (opMatch) {
-                    value = `\$${opMatch[1]}:${value.slice(opMatch[0].length)}`;
-                }
-                else {
-                    value = `\$eq:${value}`;
-                }
+                const value = this.parseFilterValue(f.slice(eqIndex + 1));
                 const filterKey = `filter.${key}`;
                 if (filterKey in queryParams) {
                     // Multiple filters on same field: collect as array
@@ -165,7 +157,9 @@ export default class PaymentMethodLedgerMappings extends Command {
             data = form;
         }
         else if (endpoint.hasBody) {
-            if (flags.file) {
+            // For binary-response endpoints, --file is the output path, not a JSON
+            // input file — skip the read-as-body branch.
+            if (flags.file && !endpoint.isBinaryResponse) {
                 const fsModule = await import('node:fs');
                 try {
                     data = JSON.parse(fsModule.readFileSync(flags.file, 'utf-8'));
@@ -187,6 +181,7 @@ export default class PaymentMethodLedgerMappings extends Command {
                 if (!data)
                     data = {};
                 const body = data;
+                const arrayFields = new Set(endpoint.bodyArrayFields);
                 for (const pair of flags.set) {
                     const eqIndex = pair.indexOf('=');
                     if (eqIndex === -1)
@@ -198,6 +193,19 @@ export default class PaymentMethodLedgerMappings extends Command {
                         value = JSON.parse(value);
                     }
                     catch { /* keep as string */ }
+                    // For top-level keys typed as array in the API schema, split bare
+                    // comma-separated input (e.g. "1171,1170" -> ["1171","1170"]) and
+                    // wrap a lone scalar in a single-element array. Skips when --set
+                    // already provided a JSON array.
+                    const isArrayField = !key.includes('.') && arrayFields.has(key);
+                    if (isArrayField && !Array.isArray(value)) {
+                        if (typeof value === 'string') {
+                            value = value.split(',').map(v => v.trim()).filter(v => v.length > 0);
+                        }
+                        else if (value !== undefined && value !== null) {
+                            value = [value];
+                        }
+                    }
                     // Support nested keys like "address.city"
                     const keys = key.split('.');
                     if (keys.some(k => k === '__proto__' || k === 'constructor' || k === 'prototype')) {
@@ -248,6 +256,26 @@ export default class PaymentMethodLedgerMappings extends Command {
             process.stderr.write('\n');
         }
         try {
+            // Binary response: download to file
+            if (endpoint.isBinaryResponse) {
+                if (!flags.file) {
+                    this.error(`Endpoint returns a binary file. Specify the output path with -f / --file (e.g. -f report.xlsx).`);
+                }
+                const binResponse = await apiRequest(endpoint.method, apiPath, {
+                    params: Object.keys(queryParams).length > 0 ? queryParams : undefined,
+                    data,
+                    contextName: flags.context,
+                    binary: true,
+                });
+                const fsModule = await import('node:fs');
+                const binData = binResponse.data;
+                const buffer = Buffer.isBuffer(binData)
+                    ? binData
+                    : Buffer.from(binData);
+                fsModule.writeFileSync(flags.file, buffer);
+                this.log(`Saved ${buffer.length} bytes to ${flags.file}`);
+                return;
+            }
             const response = await apiRequest(endpoint.method, apiPath, {
                 params: Object.keys(queryParams).length > 0 ? queryParams : undefined,
                 data,
@@ -338,6 +366,7 @@ export default class PaymentMethodLedgerMappings extends Command {
         }
         else if (flags.set && flags.set.length > 0) {
             const item = {};
+            const arrayFields = new Set(ENDPOINTS['create']?.bodyArrayFields ?? []);
             for (const pair of flags.set) {
                 const eqIndex = pair.indexOf('=');
                 if (eqIndex === -1)
@@ -348,6 +377,15 @@ export default class PaymentMethodLedgerMappings extends Command {
                     value = JSON.parse(value);
                 }
                 catch { /* keep as string */ }
+                const isArrayField = !key.includes('.') && arrayFields.has(key);
+                if (isArrayField && !Array.isArray(value)) {
+                    if (typeof value === 'string') {
+                        value = value.split(',').map(v => v.trim()).filter(v => v.length > 0);
+                    }
+                    else if (value !== undefined && value !== null) {
+                        value = [value];
+                    }
+                }
                 const keys = key.split('.');
                 if (keys.some(k => k === '__proto__' || k === 'constructor' || k === 'prototype')) {
                     this.error(`Invalid --set key: ${key}`);
@@ -497,14 +535,7 @@ export default class PaymentMethodLedgerMappings extends Command {
                 if (eqIndex === -1)
                     this.error(`Invalid --filter format: ${f}. Use: --filter key=value`);
                 const key = f.slice(0, eqIndex);
-                let value = f.slice(eqIndex + 1);
-                const opMatch = value.match(/^\$?(eq|ne|in|like|gt|gte|lt|lte|null):/);
-                if (opMatch) {
-                    value = `\$${opMatch[1]}:${value.slice(opMatch[0].length)}`;
-                }
-                else {
-                    value = `\$eq:${value}`;
-                }
+                const value = this.parseFilterValue(f.slice(eqIndex + 1));
                 const filterKey = `filter.${key}`;
                 if (filterKey in baseParams) {
                     const existing = baseParams[filterKey];
@@ -681,6 +712,7 @@ export default class PaymentMethodLedgerMappings extends Command {
         }
         else if (flags.set && flags.set.length > 0) {
             const item = {};
+            const arrayFields = new Set(ENDPOINTS[`create-${suffix}`]?.bodyArrayFields ?? []);
             for (const pair of flags.set) {
                 const eqIndex = pair.indexOf('=');
                 if (eqIndex === -1)
@@ -691,6 +723,15 @@ export default class PaymentMethodLedgerMappings extends Command {
                     value = JSON.parse(value);
                 }
                 catch { /* keep as string */ }
+                const isArrayField = !key.includes('.') && arrayFields.has(key);
+                if (isArrayField && !Array.isArray(value)) {
+                    if (typeof value === 'string') {
+                        value = value.split(',').map(v => v.trim()).filter(v => v.length > 0);
+                    }
+                    else if (value !== undefined && value !== null) {
+                        value = [value];
+                    }
+                }
                 const keys = key.split('.');
                 if (keys.some(k => k === '__proto__' || k === 'constructor' || k === 'prototype')) {
                     this.error(`Invalid --set key: ${key}`);
@@ -810,35 +851,62 @@ export default class PaymentMethodLedgerMappings extends Command {
         this.log('');
         this.log(`Summary: ${created} created, ${updated} updated, ${errors.length} failed`);
     }
+    parseFilterValue(value) {
+        // Recognises the nestjs-paginate operator set (with or without the leading $)
+        // and recursively unwraps chained operators like 'not:null' -> '$not:$null'
+        // or 'not:like:John' -> '$not:$like:John'. Falls back to '$eq:<value>' when
+        // no operator prefix is present.
+        const opRe = /^\$?(and|or|not|eq|gt|gte|in|null|lt|lte|btw|ne|ilike|like|sw|contains)(?::|$)/;
+        const m = value.match(opRe);
+        if (!m)
+            return `\$eq:${value}`;
+        const op = m[1];
+        const rest = value.slice(m[0].length);
+        if (rest === '')
+            return `\$${op}`;
+        return opRe.test(rest) ? `\$${op}:${this.parseFilterValue(rest)}` : `\$${op}:${rest}`;
+    }
     filterFields(data, fields) {
         const isUnsafeKey = (k) => k === '__proto__' || k === 'constructor' || k === 'prototype';
         const pick = (obj, keys) => {
-            const result = {};
+            // Group keys by their first segment so sibling sub-paths
+            // (e.g. 'items.amount' + 'items._type') resolve in one recursive pass
+            // and don't clobber each other on merge.
+            const directKeys = new Set();
+            const nestedKeys = new Map();
             for (const key of keys) {
                 if (key.split('.').some(isUnsafeKey))
                     continue;
                 if (key.includes('.')) {
-                    // Handle nested fields like 'customer.name' or 'allocations.salesInvoiceId'
                     const [first, ...rest] = key.split('.');
-                    const val = obj[first];
-                    if (val && typeof val === 'object') {
-                        const restKey = rest.join('.');
-                        if (Array.isArray(val)) {
-                            // Map over array elements (e.g., allocations[].salesInvoiceId)
-                            const existing = Array.isArray(result[first]) ? result[first] : [];
-                            result[first] = val.map((item, i) => ({
-                                ...(existing[i] || {}),
-                                ...pick(item, [restKey]),
-                            }));
-                        }
-                        else {
-                            const nested = pick(val, [restKey]);
-                            result[first] = { ...(result[first] || {}), ...nested };
-                        }
+                    if (isUnsafeKey(first))
+                        continue;
+                    const list = nestedKeys.get(first);
+                    if (list) {
+                        list.push(rest.join('.'));
+                    }
+                    else {
+                        nestedKeys.set(first, [rest.join('.')]);
                     }
                 }
-                else if (key in obj) {
+                else {
+                    directKeys.add(key);
+                }
+            }
+            const result = {};
+            for (const key of directKeys) {
+                if (key in obj)
                     result[key] = obj[key];
+            }
+            for (const [first, restKeys] of nestedKeys) {
+                const val = obj[first];
+                if (!val || typeof val !== 'object')
+                    continue;
+                if (Array.isArray(val)) {
+                    result[first] = val.map(item => pick(item, restKeys));
+                }
+                else {
+                    result[first] = pick(val, restKeys);
                 }
             }
             return result;

package/dist/commands/payment-methods 3.d.ts

@@ -0,0 +1,40 @@
+import { Command } from '@oclif/core';
+export default class PaymentMethods extends Command {
+    static description: string;
+    static examples: string[];
+    static args: {
+        action: import("@oclif/core/interfaces").Arg<string, Record<string, unknown>>;
+        id: import("@oclif/core/interfaces").Arg<string | undefined, Record<string, unknown>>;
+    };
+    static flags: {
+        context: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        output: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        table: import("@oclif/core/interfaces").BooleanFlag<boolean>;
+        csv: import("@oclif/core/interfaces").BooleanFlag<boolean>;
+        data: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        file: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        set: import("@oclif/core/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        page: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        limit: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        include: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        search: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        'sort-by': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        filter: import("@oclif/core/interfaces").OptionFlag<string[] | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        query: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        fields: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        sanitize: import("@oclif/core/interfaces").BooleanFlag<boolean>;
+        'show-http-request': import("@oclif/core/interfaces").BooleanFlag<boolean>;
+    };
+    run(): Promise<void>;
+    private runImport;
+    private runExport;
+    private runCamt053Import;
+    private runNestedImport;
+    private parseFilterValue;
+    private filterFields;
+    private sanitizeOutput;
+    private getColumns;
+    private getValue;
+    private printTable;
+    private printCsv;
+}