@aderis-erp/cli 1.0.3 → 1.0.7

package/dist/commands/workflows.js

@@ -8,9 +8,9 @@ const HAS_CAMT053 = false;
 const NESTED_IMPORTS = [];
 const SUBCOMMANDS = [...["list", "status", "find"], ...(HAS_IMPORT ? ['import'] : []), ...(HAS_EXPORT ? ['export'] : []), ...NESTED_IMPORTS.map(n => `import-${n}`)];
 const ENDPOINTS = {
-    'list': { method: 'GET', path: '/v1/workflows', summary: 'Get all workflows with their summary', pathParams: [], hasBody: false, isMultipart: false },
-    'status': { method: 'POST', path: '/v1/workflows/status', summary: 'Create a new workflow status update', pathParams: [], hasBody: true, isMultipart: false },
-    'find': { method: 'GET', path: '/v1/workflows/{workflowId}', summary: 'Get workflow details including status updates and entity changes', pathParams: ['workflowId'], hasBody: false, isMultipart: false },
+    'list': { method: 'GET', path: '/v1/workflows', summary: 'Get all workflows with their summary', pathParams: [], hasBody: false, isMultipart: false, isBinaryResponse: false, bodyArrayFields: [] },
+    'status': { method: 'POST', path: '/v1/workflows/status', summary: 'Create a new workflow status update', pathParams: [], hasBody: true, isMultipart: false, isBinaryResponse: false, bodyArrayFields: [] },
+    'find': { method: 'GET', path: '/v1/workflows/{workflowId}', summary: 'Get workflow details including status updates and entity changes', pathParams: ['workflowId'], hasBody: false, isMultipart: false, isBinaryResponse: false, bodyArrayFields: [] },
 };
 export default class Workflows extends Command {
     static description = 'Manage workflows';
@@ -75,11 +75,6 @@ export default class Workflows extends Command {
         const endpoint = ENDPOINTS[args.action];
         if (!endpoint)
             this.error(`Unknown action: ${args.action}. Available: ${SUBCOMMANDS.join(', ')}`);
-        const context = flags.context ? getContext(flags.context) : getCurrentContext();
-        if (!context)
-            this.error('No context configured. Run: aderis auth login --new');
-        if (!context.tokens)
-            this.error('Not logged in. Run: aderis auth login');
         let apiPath = endpoint.path;
         for (let i = 0; i < endpoint.pathParams.length; i++) {
             const param = endpoint.pathParams[i];
@@ -98,6 +93,11 @@ export default class Workflows extends Command {
             }
             apiPath = apiPath.replace(`{${param}}`, String(value));
         }
+        const context = flags.context ? getContext(flags.context) : getCurrentContext();
+        if (!context)
+            this.error('No context configured. Run: aderis auth login --new');
+        if (!context.tokens)
+            this.error('Not logged in. Run: aderis auth login');
         const queryParams = {};
         if (flags.page)
             queryParams.page = flags.page;
@@ -124,15 +124,7 @@ export default class Workflows 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 Workflows 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 Workflows 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 Workflows 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 Workflows 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 Workflows 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 Workflows 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 Workflows 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 Workflows 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 Workflows 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,6 +851,21 @@ export default class Workflows 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) => {

package/dist/lib/api-client.d.ts

@@ -25,13 +25,18 @@ export declare class ApiError extends Error {
     data?: unknown | undefined;
     constructor(message: string, status: number, data?: unknown | undefined);
 }
-export declare function createApiClient(contextName?: string, acceptAny?: boolean): Promise<AxiosInstance>;
+export declare function createApiClient(contextName?: string, options?: boolean | {
+    acceptAny?: boolean;
+    binary?: boolean;
+}): Promise<AxiosInstance>;
 export declare function apiRequest<T = unknown>(method: string, path: string, options?: {
     acceptAny?: boolean;
+    binary?: boolean;
     contentType?: string;
     contextName?: string;
     data?: unknown;
     params?: Record<string, unknown>;
+    responseType?: 'arraybuffer' | 'json' | 'stream' | 'text';
 }): Promise<ApiResponse<T>>;
 export declare function apiGet<T = unknown>(path: string, params?: Record<string, unknown>, contextName?: string): Promise<ApiResponse<T>>;
 export declare function apiPost<T = unknown>(path: string, data?: unknown, contextName?: string): Promise<ApiResponse<T>>;

package/dist/lib/api-client.js

@@ -11,7 +11,10 @@ export class ApiError extends Error {
         this.name = 'ApiError';
     }
 }
-export async function createApiClient(contextName, acceptAny = false) {
+export async function createApiClient(contextName, options = false) {
+    // Backwards compatibility: a boolean argument means `acceptAny`.
+    const opts = typeof options === 'boolean' ? { acceptAny: options } : options;
+    const { acceptAny = false, binary = false } = opts;
     const context = contextName ? (await import('./config.js')).getContext(contextName) : getCurrentContext();
     if (!context) {
         throw new Error('No context configured. Run: aderis auth login --new');
@@ -21,14 +24,14 @@ export async function createApiClient(contextName, acceptAny = false) {
     const client = axios.create({
         baseURL,
         headers: {
-            Accept: acceptAny ? '*/*' : 'application/json',
+            Accept: binary ? '*/*' : (acceptAny ? '*/*' : 'application/json'),
             Authorization: `Bearer ${token}`,
             'Content-Type': 'application/json',
         },
         // Serialize arrays as repeated params (key=v1&key=v2) instead of bracket format (key[]=v1)
         paramsSerializer: { indexes: null },
-        // Allow any response type when acceptAny is true
-        responseType: acceptAny ? 'text' : 'json',
+        // Binary downloads come back as raw bytes; otherwise honour acceptAny.
+        responseType: binary ? 'arraybuffer' : (acceptAny ? 'text' : 'json'),
     });
     // Response interceptor for error handling
     client.interceptors.response.use((response) => response, (error) => {
@@ -45,6 +48,14 @@ export async function createApiClient(contextName, acceptAny = false) {
     return client;
 }
 function extractErrorMessage(data) {
+    // Binary requests buffer the response as ArrayBuffer / Buffer / Uint8Array.
+    // Decode it before parsing so error payloads aren't reported as "Unknown error".
+    if (data instanceof ArrayBuffer) {
+        return extractErrorMessage(new TextDecoder().decode(new Uint8Array(data)));
+    }
+    if (data instanceof Uint8Array || (typeof Buffer !== 'undefined' && Buffer.isBuffer(data))) {
+        return extractErrorMessage(new TextDecoder().decode(data));
+    }
     // If data is a JSON string, try to parse it first
     if (typeof data === 'string') {
         try {
@@ -75,7 +86,10 @@ function extractErrorMessage(data) {
     return 'Unknown error';
 }
 export async function apiRequest(method, path, options = {}) {
-    const client = await createApiClient(options.contextName, options.acceptAny);
+    const client = await createApiClient(options.contextName, {
+        acceptAny: options.acceptAny,
+        binary: options.binary,
+    });
     const config = {
         method,
         url: path,
@@ -95,6 +109,10 @@ export async function apiRequest(method, path, options = {}) {
     if (options.contentType) {
         config.headers = { ...config.headers, 'Content-Type': options.contentType };
     }
+    // Override responseType if specified (takes precedence over client-level default)
+    if (options.responseType) {
+        config.responseType = options.responseType;
+    }
     const response = await client.request(config);
     return {
         data: response.data,

package/dist/lib/auth.js

@@ -160,6 +160,7 @@ export async function login(context, options = {}) {
     const authorizationUrl = buildAuthorizeUrl(context, pkce, state, options);
     return new Promise((resolve, reject) => {
         const connections = new Set();
+        // eslint-disable-next-line prefer-const -- assigned after server creation; closure captures the reference safely
         let timeoutId;
         const server = http.createServer(async (req, res) => {
             const url = new URL(req.url || '', `http://localhost:${CALLBACK_PORT}`);
@@ -220,7 +221,17 @@ export async function login(context, options = {}) {
             }
             connections.clear();
             server.close();
+            process.off('SIGINT', onSigint);
         };
+        const onSigint = () => {
+            closeServer();
+            // Exit with the standard SIGINT code so the parent shell sees the
+            // expected termination status instead of a hung process. This is a CLI;
+            // process.exit is the correct way to bail from a SIGINT.
+            // eslint-disable-next-line unicorn/no-process-exit, n/no-process-exit
+            process.exit(130);
+        };
+        process.once('SIGINT', onSigint);
         server.listen(CALLBACK_PORT, () => {
             console.log(`Opening browser for authentication...`);
             console.log(`If the browser doesn't open, visit: ${authorizationUrl}`);
@@ -454,7 +465,14 @@ export async function loginAndSwitchTenant(context, tenantId) {
             }
             connections.clear();
             server.close();
+            process.off('SIGINT', onSigint);
+        };
+        const onSigint = () => {
+            closeServer();
+            // eslint-disable-next-line unicorn/no-process-exit, n/no-process-exit
+            process.exit(130);
         };
+        process.once('SIGINT', onSigint);
         const server = http.createServer(async (req, res) => {
             const url = new URL(req.url || '', `http://localhost:${CALLBACK_PORT}`);
             if (url.pathname !== '/oauth/callback') {

package/dist/lib/config.js

@@ -20,7 +20,7 @@ export const BASE_URLS = {
 export const DEFAULT_CLIENT_IDS = {
     dev: '74fd96f8-cf4b-49d8-b713-2a121ea22910',
     live: '970d9568-84c9-4613-b966-30d7e1d93828',
-    test: '', // TODO: Add test client ID
+    test: 'bf0e5dd6-419f-4d64-8951-2c385533651f',
 };
 const CONFIG_DIR = path.join(os.homedir(), '.aderis');
 // Ensure config directory exists