@aderis-erp/cli 1.0.15 → 1.1.0

package/dist/commands/audit-log.d.ts

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

package/dist/commands/audit-log.js

@@ -1,5 +1,5 @@
 import { Command, Args, Flags } from '@oclif/core';
-import { apiRequest, ApiError, normalizeSortBy } from '../lib/api-client.js';
+import { apiRequest, ApiError, isTextualMediaType, normalizeSortBy } from '../lib/api-client.js';
 import { getCurrentContext, getContext, getBaseUrls, hasCredentials, missingCredentialsMessage } from '../lib/config.js';
 // Add 'import' action if both create and update exist
 const HAS_IMPORT = false;
@@ -8,8 +8,8 @@ const HAS_CAMT053 = false;
 const NESTED_IMPORTS = [];
 const SUBCOMMANDS = [...["list", "get"], ...(HAS_IMPORT ? ['import'] : []), ...(HAS_EXPORT ? ['export'] : []), ...NESTED_IMPORTS.map(n => `import-${n}`)];
 const ENDPOINTS = {
-    'list': { method: 'GET', path: '/v1/audit-logs', summary: 'Get audit log entries', pathParams: [], hasBody: false, isMultipart: false, isBinaryResponse: false, bodyArrayFields: [] },
-    'get': { method: 'GET', path: '/v1/audit-logs/{id}', summary: 'Get a single audit log entry', pathParams: ['id'], hasBody: false, isMultipart: false, isBinaryResponse: false, bodyArrayFields: [] },
+    'list': { method: 'GET', path: '/v1/audit-logs', summary: 'Get audit log entries', pathParams: [], hasBody: false, isMultipart: false, isBinaryResponse: false, hasBinaryVariant: false, bodyArrayFields: [] },
+    'get': { method: 'GET', path: '/v1/audit-logs/{id}', summary: 'Get a single audit log entry', pathParams: ['id'], hasBody: false, isMultipart: false, isBinaryResponse: false, hasBinaryVariant: false, bodyArrayFields: [] },
 };
 export default class AuditLog extends Command {
     static description = 'Manage audit log';
@@ -37,7 +37,7 @@ export default class AuditLog extends Command {
         'sort-by': Flags.string({ description: 'Sort field(s), comma-separated. Direction (:ASC/:DESC) is optional; date-like fields default to DESC, others to ASC.' }),
         filter: Flags.string({ description: 'Filter (e.g. --filter status=ACTIVE). Operators: in:A,B, like:%, gt:, lt:, null:', multiple: true }),
         query: Flags.string({ description: 'Additional query params as JSON' }),
-        fields: Flags.string({ description: 'Comma-separated list of fields to show (e.g. --fields=id,name,status)' }),
+        fields: Flags.string({ description: 'Comma-separated list of fields to return, applied by the API (e.g. --fields=id,name,status)' }),
         sanitize: Flags.boolean({ description: 'Remove null values, audit fields, and duplicate UUIDs from output' }),
         'show-http-request': Flags.boolean({ description: 'Show HTTP request details (method, URL, body)' }),
     };
@@ -109,6 +109,8 @@ export default class AuditLog extends Command {
             queryParams.search = flags.search;
         if (flags['sort-by'])
             queryParams.sortBy = normalizeSortBy(flags['sort-by']);
+        if (flags.fields)
+            queryParams.fields = flags.fields;
         if (flags.query) {
             try {
                 Object.assign(queryParams, JSON.parse(flags.query));
@@ -256,10 +258,11 @@ export default class AuditLog extends Command {
             process.stderr.write('\n');
         }
         try {
-            // Binary response: download to file
+            // Binary response: to -f / --file, or to stdout when it is redirected.
             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 toStdout = !flags.file;
+                if (toStdout && process.stdout.isTTY) {
+                    this.error(`Endpoint returns a binary file. Specify the output path with -f / --file (e.g. -f report.xlsx), or redirect stdout to a file.`);
                 }
                 const binResponse = await apiRequest(endpoint.method, apiPath, {
                     params: Object.keys(queryParams).length > 0 ? queryParams : undefined,
@@ -267,24 +270,55 @@ export default class AuditLog extends Command {
                     contextName: flags.context,
                     binary: true,
                 });
-                const fsModule = await import('node:fs');
                 const binData = binResponse.data;
                 const buffer = Buffer.isBuffer(binData)
                     ? binData
                     : Buffer.from(binData);
+                if (toStdout) {
+                    // Nothing else may reach stdout here — it is the file.
+                    process.stdout.write(buffer);
+                    return;
+                }
+                const fsModule = await import('node:fs');
                 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,
-                contextName: flags.context,
-                acceptAny: true,
-            });
+            let response;
+            if (endpoint.hasBinaryVariant) {
+                // This route answers JSON or a file depending on the request body, so the
+                // bytes are read first and the response's own content-type decides. Asking
+                // for text here would corrupt an xlsx/pdf before anything could inspect it.
+                const raw = await apiRequest(endpoint.method, apiPath, {
+                    params: Object.keys(queryParams).length > 0 ? queryParams : undefined,
+                    data,
+                    contextName: flags.context,
+                    binary: true,
+                });
+                const rawType = String(raw.headers['content-type'] || '').toLowerCase();
+                const rawData = raw.data;
+                const buffer = Buffer.isBuffer(rawData) ? rawData : Buffer.from(rawData);
+                if (!isTextualMediaType(rawType)) {
+                    if (process.stdout.isTTY) {
+                        this.error(`The API returned ${rawType || 'binary data'}. Redirect stdout to a file, e.g. ... > report.xlsx`);
+                    }
+                    // Nothing else may reach stdout here — it is the file.
+                    process.stdout.write(buffer);
+                    return;
+                }
+                response = { data: buffer.toString('utf8'), headers: raw.headers, status: raw.status };
+            }
+            else {
+                response = await apiRequest(endpoint.method, apiPath, {
+                    params: Object.keys(queryParams).length > 0 ? queryParams : undefined,
+                    data,
+                    contextName: flags.context,
+                    acceptAny: true,
+                });
+            }
             // Check content-type for non-JSON responses (XML, text, etc.)
             const contentType = (response.headers['content-type'] || '').toLowerCase();
-            if (contentType.includes('xml') || contentType.includes('text/')) {
+            if (isTextualMediaType(contentType) && !contentType.startsWith('application/json')) {
                 // Output raw content for XML/text responses (CSV, plain, HTML, etc.)
                 this.log(String(response.data));
                 return;
@@ -311,11 +345,6 @@ export default class AuditLog extends Command {
                     output = obj[arrayKeys[0]];
                 }
             }
-            // Filter fields if --fields flag is provided
-            if (flags.fields) {
-                const fieldList = flags.fields.split(',').map(f => f.trim());
-                output = this.filterFields(output, fieldList);
-            }
             // Sanitize output if requested
             if (flags.sanitize) {
                 output = this.sanitizeOutput(output);
@@ -570,6 +599,8 @@ export default class AuditLog extends Command {
             baseParams.sortBy = normalizeSortBy(flags['sort-by']);
         if (flags.include)
             baseParams.include = flags.include;
+        if (flags.fields)
+            baseParams.fields = flags.fields;
         if (flags.query) {
             try {
                 Object.assign(baseParams, JSON.parse(flags.query));
@@ -624,12 +655,7 @@ export default class AuditLog extends Command {
             }
         }
         process.stderr.write(' done\n');
-        // Filter fields if specified
         let output = allItems;
-        if (flags.fields) {
-            const fieldList = flags.fields.split(',').map(f => f.trim());
-            output = this.filterFields(allItems, fieldList);
-        }
         // Sanitize output if requested
         if (flags.sanitize) {
             output = this.sanitizeOutput(output);
@@ -958,56 +984,6 @@ export default class AuditLog extends Command {
             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) => {
-            // 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('.')) {
-                    const [first, ...rest] = key.split('.');
-                    if (isUnsafeKey(first))
-                        continue;
-                    const list = nestedKeys.get(first);
-                    if (list) {
-                        list.push(rest.join('.'));
-                    }
-                    else {
-                        nestedKeys.set(first, [rest.join('.')]);
-                    }
-                }
-                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;
-        };
-        if (Array.isArray(data)) {
-            return data.map(item => pick(item, fields));
-        }
-        return pick(data, fields);
-    }
     sanitizeOutput(data) {
         const auditFields = ['_type', 'tenantId', 'createdAt', 'updatedAt', 'createdBy', 'updatedBy'];
         const uuidRegex = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;

package/dist/commands/auth/login.d.ts

@@ -1,4 +1,5 @@
 import { Command } from '@oclif/core';
+import { type FlowHooks } from '../../lib/auth/index.js';
 export default class AuthLogin extends Command {
     static description: string;
     static examples: string[];
@@ -6,9 +7,17 @@ export default class AuthLogin extends Command {
         'client-id': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         context: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         environment: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        'keycloak-url': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         name: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         new: import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
     run(): Promise<void>;
+    /** Prompts for a tenant; throws {@link NoTerminalError} without a terminal, which `run()` turns into the hint. */
+    private chooseTenant;
+    /** Browser-flow hooks: the defaults here, overridden by tests. */
+    protected flowHooks(): FlowHooks;
+    private printSession;
+    /** Validates `--keycloak-url` before the context is stored. */
+    private resolveKeycloakBaseUrl;
     private createNewContext;
 }

package/dist/commands/auth/login.js

@@ -1,14 +1,15 @@
 import { input, select } from '@inquirer/prompts';
 import { Command, Flags } from '@oclif/core';
 import ora from 'ora';
-import { login, testAuth } from '../../lib/auth.js';
-import { addContext, DEFAULT_CLIENT_IDS, getAuthType, getContext, getCurrentContext, } from '../../lib/config.js';
+import { completeTenantSelection, login, parseKeycloakBaseUrl, tenantLabel, testAuth, } from '../../lib/auth/index.js';
+import { addContext, DEFAULT_KEYCLOAK_CLIENT_ID, getAuthType, getContext, getCurrentContext, } from '../../lib/config.js';
 export default class AuthLogin extends Command {
     static description = 'Login to an Aderis environment using OAuth';
     static examples = [
         '<%= config.bin %> auth login',
         '<%= config.bin %> auth login --context my-client',
         '<%= config.bin %> auth login --new --name my-client --environment dev',
+        '<%= config.bin %> auth login --new --name test-kc --environment test',
     ];
     static flags = {
         'client-id': Flags.string({
@@ -22,8 +23,11 @@ export default class AuthLogin extends Command {
         }),
         environment: Flags.string({
             char: 'e',
-            description: 'Environment (dev, staging, live)',
-            options: ['dev', 'staging', 'live'],
+            description: 'Environment (dev, test, staging, live)',
+            options: ['dev', 'test', 'staging', 'live'],
+        }),
+        'keycloak-url': Flags.string({
+            description: 'Keycloak origin for a new context instead of the default host for the environment (--new only; https, allowlisted)',
         }),
         name: Flags.string({
             description: 'Name for the new context (required with --new)',
@@ -37,6 +41,9 @@ export default class AuthLogin extends Command {
     async run() {
         const { flags } = await this.parse(AuthLogin);
         let context = null;
+        if (flags['keycloak-url'] && !flags.new) {
+            this.error('--keycloak-url only applies when creating a context with --new; an existing context keeps its stored Keycloak URL.');
+        }
         if (flags.new) {
             context = await this.createNewContext(flags);
         }
@@ -62,22 +69,61 @@ export default class AuthLogin extends Command {
         const spinner = ora(`Logging in to ${context.name} (${context.environment})...`).start();
         try {
             spinner.text = 'Waiting for browser authentication...';
-            await login(context);
-            spinner.text = 'Verifying authentication...';
+            await login(context, this.flowHooks());
+            // The tenant prompt needs the terminal, so the spinner pauses around it.
+            spinner.stop();
+            await completeTenantSelection(context.name, (tenants) => this.chooseTenant(tenants));
+            spinner.start('Verifying authentication...');
             const user = await testAuth(context.name);
             spinner.succeed(`Successfully logged in as ${user.email || user.name || 'user'}`);
-            this.log(`\nContext: ${context.name}`);
-            this.log(`Environment: ${context.environment}`);
-            if (context.tenant_id) {
-                this.log(`Tenant ID: ${context.tenant_id}`);
+            this.printSession(context);
+            // `context` predates the stored tokens; re-read it for the tenant they carry.
+            const tenantId = getContext(context.name)?.tenant_id;
+            if (tenantId) {
+                this.log(`Tenant ID: ${tenantId}`);
             }
         }
         catch (error) {
-            spinner.fail('Login failed');
-            if (error instanceof Error) {
-                this.error(error.message);
+            if (error instanceof NoTerminalError) {
+                // The login is stored and only the tenant pick is deferred; the session
+                // check is skipped because a token without a tenant is always rejected.
+                spinner.stop();
+                this.printSession(context);
+                this.warn(`Your user reaches ${error.tenants.length} tenants and none is selected yet. Pick one with: aderis auth tenant switch <tenant-id> --context ${context.name}\n` +
+                    error.tenants.map((t) => `  ${tenantLabel(t)}`).join('\n'));
+                return;
             }
-            throw error;
+            spinner.fail('Login failed');
+            this.error(error instanceof Error ? error.message : String(error));
+        }
+    }
+    /** Prompts for a tenant; throws {@link NoTerminalError} without a terminal, which `run()` turns into the hint. */
+    async chooseTenant(tenants) {
+        if (!process.stdin.isTTY) {
+            throw new NoTerminalError(tenants);
+        }
+        return select({
+            choices: tenants.map((t) => ({ name: tenantLabel(t), value: t.tenantId })),
+            message: 'Your user reaches several tenants. Select the one to work in:',
+        });
+    }
+    /** Browser-flow hooks: the defaults here, overridden by tests. */
+    flowHooks() {
+        return {};
+    }
+    printSession(context) {
+        this.log(`\nContext: ${context.name}`);
+        this.log(`Environment: ${context.environment}`);
+    }
+    /** Validates `--keycloak-url` before the context is stored. */
+    resolveKeycloakBaseUrl(flag) {
+        if (flag === undefined)
+            return undefined;
+        try {
+            return parseKeycloakBaseUrl(flag);
+        }
+        catch (error) {
+            this.error(`Invalid --keycloak-url: ${error instanceof Error ? error.message : String(error)}`);
         }
     }
     async createNewContext(flags) {
@@ -101,20 +147,18 @@ export default class AuthLogin extends Command {
         const environment = flags.environment ? flags.environment : (await select({
             choices: [
                 { description: 'Development environment', name: 'dev', value: 'dev' },
+                { description: 'QA environment', name: 'test', value: 'test' },
                 { description: 'Staging environment', name: 'staging', value: 'staging' },
                 { description: 'Production environment', name: 'live', value: 'live' },
             ],
             message: 'Environment:',
         }));
-        const defaultClientId = DEFAULT_CLIENT_IDS[environment];
-        const clientId = flags['client-id'] || defaultClientId;
-        if (!clientId) {
-            this.error(`No default client ID configured for ${environment} environment. Please provide --client-id`);
-        }
+        const keycloakBaseUrl = this.resolveKeycloakBaseUrl(flags['keycloak-url']);
         const context = {
             auth_type: 'oauth2',
-            client_id: clientId,
+            client_id: flags['client-id'] || DEFAULT_KEYCLOAK_CLIENT_ID,
             environment,
+            keycloak_base_url: keycloakBaseUrl,
             name: contextName,
         };
         addContext(context);
@@ -122,3 +166,12 @@ export default class AuthLogin extends Command {
         return context;
     }
 }
+/** Thrown by the tenant chooser when there is no terminal; the login stays stored. */
+class NoTerminalError extends Error {
+    tenants;
+    constructor(tenants) {
+        super('No terminal available to select a tenant');
+        this.tenants = tenants;
+        this.name = 'NoTerminalError';
+    }
+}

package/dist/commands/auth/logout.d.ts

@@ -1,4 +1,8 @@
 import { Command } from '@oclif/core';
+/**
+ * Deletes the stored credential only; the provider's browser session
+ * (`end_session_endpoint`) is left in place.
+ */
 export default class AuthLogout extends Command {
     static description: string;
     static examples: string[];

package/dist/commands/auth/logout.js

@@ -1,5 +1,9 @@
 import { Command, Flags } from '@oclif/core';
 import { clearApiKey, clearTokens, getAuthType, getContext, getCurrentContext, hasCredentials, } from '../../lib/config.js';
+/**
+ * Deletes the stored credential only; the provider's browser session
+ * (`end_session_endpoint`) is left in place.
+ */
 export default class AuthLogout extends Command {
     static description = 'Logout from the current or specified context';
     static examples = [

package/dist/commands/auth/status.js

@@ -1,5 +1,5 @@
 import { Command, Flags } from '@oclif/core';
-import { testAuth } from '../../lib/auth.js';
+import { keycloakBaseUrlOf, testAuth } from '../../lib/auth/index.js';
 import { getAuthType, getContext, getCurrentContext, hasCredentials, listContexts, maskApiKey, missingCredentialsMessage, } from '../../lib/config.js';
 export default class AuthStatus extends Command {
     static description = 'Show authentication status for contexts';
@@ -78,6 +78,7 @@ export default class AuthStatus extends Command {
         this.log(`Environment: ${context.environment}`);
         this.log(`Auth: ${authType === 'api_key' ? 'API key (x-api-key)' : 'oauth2'}`);
         if (authType === 'oauth2') {
+            this.log(`Keycloak URL: ${keycloakBaseUrlOf(context)}`);
             this.log(`Client ID: ${context.client_id ?? 'N/A'}`);
         }
         if (!hasCredentials(context)) {
@@ -85,30 +86,38 @@ export default class AuthStatus extends Command {
             this.log(`\n${missingCredentialsMessage(context)}`);
             return;
         }
+        // `testAuth` refreshes an access token that is within the expiry buffer, so
+        // it runs before the expiry is printed — otherwise a session that was just
+        // renewed still shows the stale timestamp, labelled (EXPIRED).
+        let checked;
+        if (detailed) {
+            try {
+                checked = await testAuth(name);
+            }
+            catch (error) {
+                checked = error instanceof Error ? error : new Error('Authentication error');
+            }
+        }
+        // Re-read: a refresh during the check replaced the stored tokens.
+        const fresh = getContext(name) ?? context;
         if (authType === 'api_key') {
-            this.log(`API key: ${maskApiKey(context.api_key)}`);
+            this.log(`API key: ${maskApiKey(fresh.api_key)}`);
         }
         else {
-            this.log(`Tenant ID: ${context.tenant_id || 'N/A'}`);
-            if (context.tokens?.expires_at) {
-                const expiresAt = new Date(context.tokens.expires_at);
+            this.log(`Tenant ID: ${fresh.tenant_id || 'N/A'}`);
+            if (fresh.tokens?.expires_at) {
+                const expiresAt = new Date(fresh.tokens.expires_at);
                 const isExpired = expiresAt < new Date();
                 this.log(`Token expires: ${expiresAt.toLocaleString()}${isExpired ? ' (EXPIRED)' : ''}`);
             }
         }
-        if (detailed) {
-            try {
-                const user = await testAuth(name);
-                this.log(`\nLogged in as: ${user.email || user.name || 'Unknown'}`);
-                this.log('Status: ✓ Authenticated');
+        if (checked !== undefined) {
+            if (checked instanceof Error) {
+                this.log(`\nStatus: ✗ ${checked.message}`);
             }
-            catch (error) {
-                if (error instanceof Error) {
-                    this.log(`\nStatus: ✗ ${error.message}`);
-                }
-                else {
-                    this.log('\nStatus: ✗ Authentication error');
-                }
+            else {
+                this.log(`\nLogged in as: ${checked.email || checked.name || 'Unknown'}`);
+                this.log('Status: ✓ Authenticated');
             }
         }
     }

package/dist/commands/auth/tenant.js

@@ -1,6 +1,6 @@
 import { select } from '@inquirer/prompts';
 import { Args, Command, Flags } from '@oclif/core';
-import { getCurrentTenant, listTenants, switchTenant } from '../../lib/auth.js';
+import { getCurrentTenant, listTenants, switchTenant, tenantLabel } from '../../lib/auth/index.js';
 import { getAuthType, getContext, getCurrentContext } from '../../lib/config.js';
 export default class AuthTenant extends Command {
     static args = {
@@ -107,7 +107,7 @@ export default class AuthTenant extends Command {
                 tags.push('default');
             const tagStr = tags.length > 0 ? ` (${tags.join(', ')})` : '';
             return {
-                name: `${t.name}${tagStr}`,
+                name: `${tenantLabel(t)}${tagStr}`,
                 value: t.tenantId,
             };
         });
@@ -184,10 +184,7 @@ export default class AuthTenant extends Command {
             this.log(`Successfully switched to tenant ${display}`);
         }
         catch (error) {
-            if (error instanceof Error) {
-                this.error(`Failed to switch tenant: ${error.message}`);
-            }
-            throw error;
+            this.error(`Failed to switch tenant: ${error instanceof Error ? error.message : String(error)}`);
         }
     }
 }

package/dist/commands/auth.d.ts

@@ -24,7 +24,6 @@ export default class Auth extends Command {
         table: import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
     run(): Promise<void>;
-    private filterFields;
     private getColumns;
     private getValue;
     private printCsv;

package/dist/commands/auth.js

@@ -48,7 +48,7 @@ export default class Auth extends Command {
         }),
         csv: Flags.boolean({ description: 'Output as CSV (shortcut for --output csv)' }),
         data: Flags.string({ char: 'd', description: 'JSON data for request body' }),
-        fields: Flags.string({ description: 'Comma-separated list of fields to show (e.g. --fields=id,name,status)' }),
+        fields: Flags.string({ description: 'Comma-separated list of fields to return, applied by the API (e.g. --fields=id,name,status)' }),
         file: Flags.string({ char: 'f', description: 'Read request body from file' }),
         include: Flags.string({ description: 'Relations to include' }),
         limit: Flags.string({ description: 'Items per page' }),
@@ -138,6 +138,8 @@ export default class Auth extends Command {
             queryParams.search = flags.search;
         if (flags['sort-by'])
             queryParams.sortBy = flags['sort-by'];
+        if (flags.fields)
+            queryParams.fields = flags.fields;
         if (flags.query) {
             try {
                 Object.assign(queryParams, JSON.parse(flags.query));
@@ -222,11 +224,6 @@ export default class Auth extends Command {
             if (args.action === 'list' && output && typeof output === 'object' && 'data' in output) {
                 output = output.data;
             }
-            // Filter fields if --fields flag is provided
-            if (flags.fields) {
-                const fieldList = flags.fields.split(',').map((f) => f.trim());
-                output = this.filterFields(output, fieldList);
-            }
             // Sanitize output if requested
             if (flags.sanitize) {
                 output = this.sanitizeOutput(output);
@@ -248,29 +245,6 @@ export default class Auth extends Command {
             throw error;
         }
     }
-    filterFields(data, fields) {
-        const pick = (obj, keys) => {
-            const result = {};
-            for (const key of keys) {
-                if (key.includes('.')) {
-                    // Handle nested fields like 'customer.name'
-                    const [first, ...rest] = key.split('.');
-                    if (obj[first] && typeof obj[first] === 'object') {
-                        const nested = pick(obj[first], [rest.join('.')]);
-                        result[first] = { ...result[first], ...nested };
-                    }
-                }
-                else if (key in obj) {
-                    result[key] = obj[key];
-                }
-            }
-            return result;
-        };
-        if (Array.isArray(data)) {
-            return data.map((item) => pick(item, fields));
-        }
-        return pick(data, fields);
-    }
     getColumns(data, fields, flatten = false) {
         const items = Array.isArray(data) ? data : [data];
         if (items.length === 0)
@@ -475,6 +449,8 @@ export default class Auth extends Command {
             baseParams.sortBy = flags['sort-by'];
         if (flags.include)
             baseParams.include = flags.include;
+        if (flags.fields)
+            baseParams.fields = flags.fields;
         if (flags.query) {
             try {
                 Object.assign(baseParams, JSON.parse(flags.query));
@@ -511,12 +487,7 @@ export default class Auth extends Command {
             }
         }
         process.stderr.write(' done\n');
-        // Filter fields if specified
         let output = allItems;
-        if (flags.fields) {
-            const fieldList = flags.fields.split(',').map((f) => f.trim());
-            output = this.filterFields(allItems, fieldList);
-        }
         // Sanitize output if requested
         if (flags.sanitize) {
             output = this.sanitizeOutput(output);

package/dist/commands/bank-accounts.d.ts

@@ -33,7 +33,6 @@ export default class BankAccounts extends Command {
     private runCamt053Import;
     private runNestedImport;
     private parseFilterValue;
-    private filterFields;
     private sanitizeOutput;
     private getColumns;
     private getValue;