@xano/cli 1.1.0 → 1.3.0

package/dist/commands/tenant/cluster/create/index.js

@@ -1,6 +1,6 @@
 import { Flags } from '@oclif/core';
 import * as fs from 'node:fs';
-import * as path from 'node:path';
+import path from 'node:path';
 import BaseCommand from '../../../../base-command.js';
 export default class TenantClusterCreate extends BaseCommand {
     static description = 'Create a new tenant cluster';

package/dist/commands/tenant/cluster/license/get/index.js

@@ -1,6 +1,6 @@
 import { Args, Flags } from '@oclif/core';
 import * as fs from 'node:fs';
-import * as path from 'node:path';
+import path from 'node:path';
 import BaseCommand from '../../../../../base-command.js';
 export default class TenantClusterLicenseGet extends BaseCommand {
     static args = {

package/dist/commands/tenant/cluster/license/set/index.js

@@ -1,6 +1,6 @@
 import { Args, Flags } from '@oclif/core';
 import * as fs from 'node:fs';
-import * as path from 'node:path';
+import path from 'node:path';
 import BaseCommand from '../../../../../base-command.js';
 export default class TenantClusterLicenseSet extends BaseCommand {
     static args = {

package/dist/commands/tenant/create/index.d.ts

@@ -13,8 +13,8 @@ export default class TenantCreate extends BaseCommand {
         license: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         output: import("@oclif/core/interfaces").OptionFlag<string, import("@oclif/core/interfaces").CustomOptions>;
         platform_id: import("@oclif/core/interfaces").OptionFlag<number | undefined, import("@oclif/core/interfaces").CustomOptions>;
-        type: import("@oclif/core/interfaces").OptionFlag<string, import("@oclif/core/interfaces").CustomOptions>;
         tasks: import("@oclif/core/interfaces").BooleanFlag<boolean>;
+        type: import("@oclif/core/interfaces").OptionFlag<string, import("@oclif/core/interfaces").CustomOptions>;
         workspace: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         config: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         profile: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;

package/dist/commands/tenant/create/index.js

@@ -1,7 +1,7 @@
 import { Args, Flags } from '@oclif/core';
 import * as yaml from 'js-yaml';
 import * as fs from 'node:fs';
-import * as path from 'node:path';
+import path from 'node:path';
 import BaseCommand from '../../../base-command.js';
 export default class TenantCreate extends BaseCommand {
     static args = {
@@ -54,17 +54,17 @@ Created tenant: Production (production) - ID: 42
             description: 'Platform ID to use',
             required: false,
         }),
+        tasks: Flags.boolean({
+            allowNo: true,
+            default: true,
+            description: 'Enable background tasks',
+        }),
         type: Flags.string({
             default: 'tier1',
             description: 'Tenant type',
             options: ['tier1', 'tier2', 'tier3'],
             required: false,
         }),
-        tasks: Flags.boolean({
-            allowNo: true,
-            default: true,
-            description: 'Enable background tasks',
-        }),
         workspace: Flags.string({
             char: 'w',
             description: 'Workspace ID (uses profile workspace if not provided)',

package/dist/commands/tenant/deploy_platform/index.js

@@ -1,6 +1,6 @@
 import { Args, Flags } from '@oclif/core';
 import * as fs from 'node:fs';
-import * as path from 'node:path';
+import path from 'node:path';
 import BaseCommand from '../../../base-command.js';
 export default class TenantDeployPlatform extends BaseCommand {
     static args = {

package/dist/commands/tenant/edit/index.js

@@ -91,16 +91,16 @@ Updated tenant: New Name (my-tenant) - ID: 42
             const current = await getResponse.json();
             // Merge in user-provided values
             const body = {
-                description: flags.description !== undefined ? flags.description : (current.description ?? ''),
-                display: flags.display !== undefined ? flags.display : (current.display ?? current.name),
-                domain: flags.domain !== undefined ? flags.domain : (current.domain ?? ''),
-                ingress: flags.ingress !== undefined ? flags.ingress : (current.ingress ?? true),
-                proxy: flags.proxy !== undefined ? flags.proxy : (current.proxy ?? ''),
+                description: flags.description === undefined ? (current.description ?? '') : flags.description,
+                display: flags.display === undefined ? (current.display ?? current.name) : flags.display,
+                domain: flags.domain === undefined ? (current.domain ?? '') : flags.domain,
+                ingress: flags.ingress === undefined ? (current.ingress ?? true) : flags.ingress,
+                proxy: flags.proxy === undefined ? (current.proxy ?? '') : flags.proxy,
                 rbac: {
-                    enabled: flags.rbac !== undefined ? flags.rbac : (current.rbac?.enabled ?? false),
+                    enabled: flags.rbac === undefined ? (current.rbac?.enabled ?? false) : flags.rbac,
                 },
                 tag: current.tag ?? [],
-                tasks: flags.tasks !== undefined ? flags.tasks : (current.tasks ?? true),
+                tasks: flags.tasks === undefined ? (current.tasks ?? true) : flags.tasks,
             };
             // Update tenant
             const putResponse = await this.verboseFetch(baseUrl, {

package/dist/commands/tenant/env/get_all/index.js

@@ -1,7 +1,7 @@
 import { Args, Flags } from '@oclif/core';
 import * as yaml from 'js-yaml';
 import * as fs from 'node:fs';
-import * as path from 'node:path';
+import path from 'node:path';
 import BaseCommand from '../../../../base-command.js';
 export default class TenantEnvGetAll extends BaseCommand {
     static args = {

package/dist/commands/tenant/env/set_all/index.js

@@ -1,7 +1,7 @@
 import { Args, Flags } from '@oclif/core';
 import * as yaml from 'js-yaml';
 import * as fs from 'node:fs';
-import * as path from 'node:path';
+import path from 'node:path';
 import BaseCommand from '../../../../base-command.js';
 export default class TenantEnvSetAll extends BaseCommand {
     static args = {

package/dist/commands/tenant/impersonate/index.d.ts

@@ -14,6 +14,6 @@ export default class TenantImpersonate extends BaseCommand {
         verbose: import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
     run(): Promise<void>;
-    private getImpersonateResponse;
     private getFrontendUrl;
+    private getImpersonateResponse;
 }

package/dist/commands/tenant/impersonate/index.js

@@ -76,13 +76,26 @@ Impersonation successful!
             }
         }
     }
+    getFrontendUrl(instanceOrigin) {
+        try {
+            const url = new URL(instanceOrigin);
+            if (url.hostname === 'localhost' || url.hostname === '127.0.0.1') {
+                url.port = '4200';
+                return url.origin;
+            }
+        }
+        catch {
+            // fall through
+        }
+        return instanceOrigin;
+    }
     async getImpersonateResponse(profile, workspaceId, tenantName) {
         const apiUrl = `${profile.instance_origin}/api:meta/workspace/${workspaceId}/tenant/${encodeURIComponent(tenantName)}/impersonate`;
         const { verbose } = await this.parse(TenantImpersonate).then((r) => r.flags);
         const response = await this.verboseFetch(apiUrl, {
             headers: {
-                Authorization: `Bearer ${profile.access_token}`,
                 accept: 'application/json',
+                Authorization: `Bearer ${profile.access_token}`,
             },
             method: 'GET',
         }, verbose, profile.access_token);
@@ -96,17 +109,4 @@ Impersonation successful!
         }
         return result;
     }
-    getFrontendUrl(instanceOrigin) {
-        try {
-            const url = new URL(instanceOrigin);
-            if (url.hostname === 'localhost' || url.hostname === '127.0.0.1') {
-                url.port = '4200';
-                return url.origin;
-            }
-        }
-        catch {
-            // fall through
-        }
-        return instanceOrigin;
-    }
 }

package/dist/commands/tenant/license/get/index.js

@@ -1,6 +1,6 @@
 import { Args, Flags } from '@oclif/core';
 import * as fs from 'node:fs';
-import * as path from 'node:path';
+import path from 'node:path';
 import BaseCommand from '../../../../base-command.js';
 export default class TenantLicenseGet extends BaseCommand {
     static args = {

package/dist/commands/tenant/license/set/index.js

@@ -1,6 +1,6 @@
 import { Args, Flags } from '@oclif/core';
 import * as fs from 'node:fs';
-import * as path from 'node:path';
+import path from 'node:path';
 import BaseCommand from '../../../../base-command.js';
 export default class TenantLicenseSet extends BaseCommand {
     static args = {

package/dist/commands/tenant/pull/index.d.ts

@@ -14,10 +14,4 @@ export default class Pull extends BaseCommand {
         verbose: import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
     run(): Promise<void>;
-    /**
-     * Sanitize a document name for use as a filename.
-     * Strips quotes, replaces spaces with underscores, and removes
-     * characters that are unsafe in filenames.
-     */
-    private sanitizeFilename;
 }

package/dist/commands/tenant/pull/index.js

@@ -1,9 +1,9 @@
 import { Flags } from '@oclif/core';
-import * as fs from 'node:fs';
-import * as path from 'node:path';
 import snakeCase from 'lodash.snakecase';
+import * as fs from 'node:fs';
+import path from 'node:path';
 import BaseCommand from '../../../base-command.js';
-import { buildApiGroupFolderResolver, parseDocument } from '../../../utils/document-parser.js';
+import { buildApiGroupFolderResolver, buildChannelServerResolver, parseDocument, resolveDocumentPath, } from '../../../utils/document-parser.js';
 export default class Pull extends BaseCommand {
     static description = 'Pull a tenant multidoc from the Xano Metadata API and split into individual files';
     static examples = [
@@ -57,7 +57,7 @@ Pulled 58 documents from tenant my-tenant
     };
     async run() {
         const { flags } = await this.parse(Pull);
-        const { profileName, profile } = this.resolveProfile(flags);
+        const { profile, profileName } = this.resolveProfile(flags);
         // Determine workspace_id from flag or profile
         let workspaceId;
         if (flags.workspace) {
@@ -129,96 +129,19 @@ Pulled 58 documents from tenant my-tenant
         fs.mkdirSync(outputDir, { recursive: true });
         // Resolve api_group names to unique folder names, disambiguating collisions
         const getApiGroupFolder = buildApiGroupFolderResolver(documents, snakeCase);
+        // Resolve a realtime v2 channel path -> owning realtime_server name, so
+        // messages can nest under their channel's server (see resolver docs).
+        const getChannelServer = buildChannelServerResolver(documents);
         // Track filenames per type to handle duplicates
         const filenameCounters = new Map();
         let writtenCount = 0;
         for (const doc of documents) {
-            let typeDir;
-            let baseName;
-            if (doc.type === 'workspace') {
-                // workspace → workspace/{name}.xs
-                typeDir = path.join(outputDir, 'workspace');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'workspace_trigger') {
-                // workspace_trigger → workspace/trigger/{name}.xs
-                typeDir = path.join(outputDir, 'workspace', 'trigger');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'error_trigger') {
-                // error_trigger → workspace/trigger/{name}.xs (singleton, colocated with workspace triggers)
-                typeDir = path.join(outputDir, 'workspace', 'trigger');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'agent') {
-                // agent → ai/agent/{name}.xs
-                typeDir = path.join(outputDir, 'ai', 'agent');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'mcp_server') {
-                // mcp_server → ai/mcp_server/{name}.xs
-                typeDir = path.join(outputDir, 'ai', 'mcp_server');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'tool') {
-                // tool → ai/tool/{name}.xs
-                typeDir = path.join(outputDir, 'ai', 'tool');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'agent_trigger') {
-                // agent_trigger → ai/agent/trigger/{name}.xs
-                typeDir = path.join(outputDir, 'ai', 'agent', 'trigger');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'mcp_server_trigger') {
-                // mcp_server_trigger → ai/mcp_server/trigger/{name}.xs
-                typeDir = path.join(outputDir, 'ai', 'mcp_server', 'trigger');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'table_trigger') {
-                // table_trigger → table/trigger/{name}.xs
-                typeDir = path.join(outputDir, 'table', 'trigger');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'realtime_channel') {
-                // realtime_channel → realtime/channel/{name}.xs
-                typeDir = path.join(outputDir, 'realtime', 'channel');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'realtime_trigger') {
-                // realtime_trigger → realtime/trigger/{name}.xs
-                typeDir = path.join(outputDir, 'realtime', 'trigger');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'api_group') {
-                // api_group "test" → api/{resolved_folder}/{name}.xs
-                const groupFolder = getApiGroupFolder(doc.name);
-                typeDir = path.join(outputDir, 'api', groupFolder);
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'query' && doc.apiGroup) {
-                // query in group "test" → api/{resolved_folder}/{query_name}.xs
-                const groupFolder = getApiGroupFolder(doc.apiGroup);
-                const nameParts = doc.name.split('/');
-                const leafName = nameParts.pop();
-                const folderParts = nameParts.map((part) => snakeCase(part));
-                typeDir = path.join(outputDir, 'api', groupFolder, ...folderParts);
-                baseName = this.sanitizeFilename(leafName);
-                if (doc.verb) {
-                    baseName = `${baseName}_${doc.verb}`;
-                }
-            }
-            else {
-                // Default: split folder path from name
-                const nameParts = doc.name.split('/');
-                const leafName = nameParts.pop();
-                const folderParts = nameParts.map((part) => snakeCase(part));
-                typeDir = path.join(outputDir, doc.type, ...folderParts);
-                baseName = this.sanitizeFilename(leafName);
-                if (doc.verb) {
-                    baseName = `${baseName}_${doc.verb}`;
-                }
-            }
+            const { baseName, typeDir } = resolveDocumentPath(doc, outputDir, {
+                getApiGroupFolder,
+                getChannelServer,
+                join: path.join,
+                snakeCase,
+            });
             fs.mkdirSync(typeDir, { recursive: true });
             // Track duplicates per directory
             const dirKey = path.relative(outputDir, typeDir);
@@ -229,20 +152,11 @@ Pulled 58 documents from tenant my-tenant
             const count = typeCounters.get(baseName) || 0;
             typeCounters.set(baseName, count + 1);
             // Append numeric suffix for duplicates
-            let filename;
-            filename = count === 0 ? `${baseName}.xs` : `${baseName}_${count + 1}.xs`;
+            const filename = count === 0 ? `${baseName}.xs` : `${baseName}_${count + 1}.xs`;
             const filePath = path.join(typeDir, filename);
             fs.writeFileSync(filePath, doc.content, 'utf8');
             writtenCount++;
         }
         this.log(`Pulled ${writtenCount} documents from tenant ${tenantName} to ${flags.directory}`);
     }
-    /**
-     * Sanitize a document name for use as a filename.
-     * Strips quotes, replaces spaces with underscores, and removes
-     * characters that are unsafe in filenames.
-     */
-    sanitizeFilename(name) {
-        return snakeCase(name.replaceAll('"', ''));
-    }
 }

package/dist/commands/tenant/unit_test/run/index.js

@@ -68,26 +68,34 @@ Result: PASS
                 this.error(`API request failed with status ${response.status}: ${response.statusText}\n${errorText}`);
             }
             const result = (await response.json());
+            // Anything other than 'ok' is a failure -- the API has been observed
+            // returning several distinct non-ok statuses, so never enumerate them.
+            const passed = result.status === 'ok';
             if (flags.output === 'json') {
                 this.log(JSON.stringify(result, null, 2));
             }
+            else if (passed) {
+                this.log('Result: PASS');
+            }
             else {
-                if (result.status === 'ok') {
-                    this.log('Result: PASS');
-                }
-                else {
-                    this.log('Result: FAIL');
-                    const failedExpects = result.results?.filter((r) => r.status === 'fail') ?? [];
-                    for (const expect of failedExpects) {
-                        if (expect.message) {
-                            this.log(`  Error: ${expect.message}`);
-                        }
+                this.log('Result: FAIL');
+                const failedExpects = result.results?.filter((r) => r.status === 'fail') ?? [];
+                for (const expect of failedExpects) {
+                    if (expect.message) {
+                        this.log(`  Error: ${expect.message}`);
                     }
-                    this.exit(1);
                 }
             }
+            // Set the code rather than throwing via this.exit(): the exit must apply
+            // in both output modes, and process.exitCode cannot be swallowed by the
+            // surrounding catch. Mirrors what run_all already does.
+            if (!passed) {
+                process.exitCode = 1;
+            }
         }
         catch (error) {
+            if (error instanceof Error && 'oclif' in error)
+                throw error;
             if (error instanceof Error) {
                 this.error(`Failed to run unit test: ${error.message}`);
             }

package/dist/commands/tenant/unit_test/run_all/index.js

@@ -95,7 +95,15 @@ Results: 4 passed, 1 failed
                 },
             });
             if (tests.length === 0) {
-                this.log('No unit tests found');
+                // An empty suite is a normal state (fresh branches have no tests) and
+                // is a success, not a failure. Under -o json it must still be parseable,
+                // with the same keys the populated path emits below.
+                if (flags.output === 'json') {
+                    this.log(JSON.stringify({ failed: 0, passed: 0, results: [] }, null, 2));
+                }
+                else {
+                    this.log('No unit tests found');
+                }
                 return;
             }
             if (flags.output === 'summary') {
@@ -162,6 +170,8 @@ Results: 4 passed, 1 failed
                     }
                 }
                 catch (error) {
+                    if (error instanceof Error && 'oclif' in error)
+                        throw error;
                     const message = error instanceof Error ? error.message : String(error);
                     results.push({
                         message,
@@ -202,6 +212,8 @@ Results: 4 passed, 1 failed
             }
         }
         catch (error) {
+            if (error instanceof Error && 'oclif' in error)
+                throw error;
             if (error instanceof Error) {
                 this.error(`Failed to run unit tests: ${error.message}`);
             }

package/dist/commands/tenant/workflow_test/run/index.js

@@ -61,24 +61,29 @@ Result: PASS (0.25s)
                 this.error(`API request failed with status ${response.status}: ${response.statusText}\n${errorText}`);
             }
             const result = (await response.json());
+            // Anything other than 'ok' is a failure -- the API has been observed
+            // returning several distinct non-ok statuses, so never enumerate them.
+            const passed = result.status === 'ok';
             if (flags.output === 'json') {
                 this.log(JSON.stringify(result, null, 2));
             }
             else {
                 const timing = result.timing ? ` (${result.timing}s)` : '';
-                if (result.status === 'ok') {
-                    this.log(`Result: PASS${timing}`);
-                }
-                else {
-                    this.log(`Result: FAIL${timing}`);
-                    if (result.message) {
-                        this.log(`  Error: ${result.message}`);
-                    }
-                    this.exit(1);
+                this.log(`Result: ${passed ? 'PASS' : 'FAIL'}${timing}`);
+                if (!passed && result.message) {
+                    this.log(`  Error: ${result.message}`);
                 }
             }
+            // Set the code rather than throwing via this.exit(): the exit must apply
+            // in both output modes, and process.exitCode cannot be swallowed by the
+            // surrounding catch. Mirrors what run_all already does.
+            if (!passed) {
+                process.exitCode = 1;
+            }
         }
         catch (error) {
+            if (error instanceof Error && 'oclif' in error)
+                throw error;
             if (error instanceof Error) {
                 this.error(`Failed to run workflow test: ${error.message}`);
             }

package/dist/commands/tenant/workflow_test/run_all/index.js

@@ -85,7 +85,15 @@ Results: 2 passed, 1 failed
                 },
             });
             if (tests.length === 0) {
-                this.log('No workflow tests found');
+                // An empty suite is a normal state (fresh branches have no tests) and
+                // is a success, not a failure. Under -o json it must still be parseable,
+                // with the same keys the populated path emits below.
+                if (flags.output === 'json') {
+                    this.log(JSON.stringify({ failed: 0, passed: 0, results: [] }, null, 2));
+                }
+                else {
+                    this.log('No workflow tests found');
+                }
                 return;
             }
             if (flags.output === 'summary') {
@@ -146,6 +154,8 @@ Results: 2 passed, 1 failed
                     }
                 }
                 catch (error) {
+                    if (error instanceof Error && 'oclif' in error)
+                        throw error;
                     const message = error instanceof Error ? error.message : String(error);
                     results.push({
                         message,
@@ -184,6 +194,8 @@ Results: 2 passed, 1 failed
             }
         }
         catch (error) {
+            if (error instanceof Error && 'oclif' in error)
+                throw error;
             if (error instanceof Error) {
                 this.error(`Failed to run workflow tests: ${error.message}`);
             }

package/dist/commands/unit_test/run/index.js

@@ -63,26 +63,34 @@ Result: PASS
                 this.error(`API request failed with status ${response.status}: ${response.statusText}\n${errorText}`);
             }
             const result = await response.json();
+            // Anything other than 'ok' is a failure -- the API has been observed
+            // returning several distinct non-ok statuses, so never enumerate them.
+            const passed = result.status === 'ok';
             if (flags.output === 'json') {
                 this.log(JSON.stringify(result, null, 2));
             }
+            else if (passed) {
+                this.log('Result: PASS');
+            }
             else {
-                if (result.status === 'ok') {
-                    this.log('Result: PASS');
-                }
-                else {
-                    this.log('Result: FAIL');
-                    const failedExpects = result.results?.filter(r => r.status === 'fail') ?? [];
-                    for (const expect of failedExpects) {
-                        if (expect.message) {
-                            this.log(`  Error: ${expect.message}`);
-                        }
+                this.log('Result: FAIL');
+                const failedExpects = result.results?.filter(r => r.status === 'fail') ?? [];
+                for (const expect of failedExpects) {
+                    if (expect.message) {
+                        this.log(`  Error: ${expect.message}`);
                     }
-                    this.exit(1);
                 }
             }
+            // Set the code rather than throwing via this.exit(): the exit must apply
+            // in both output modes, and process.exitCode cannot be swallowed by the
+            // surrounding catch. Mirrors what run_all already does.
+            if (!passed) {
+                process.exitCode = 1;
+            }
         }
         catch (error) {
+            if (error instanceof Error && 'oclif' in error)
+                throw error;
             if (error instanceof Error) {
                 this.error(`Failed to run unit test: ${error.message}`);
             }

package/dist/commands/unit_test/run_all/index.js

@@ -91,7 +91,15 @@ Results: 2 passed, 1 failed
                 },
             });
             if (tests.length === 0) {
-                this.log('No unit tests found');
+                // An empty suite is a normal state (fresh branches have no tests) and
+                // is a success, not a failure. Under -o json it must still be parseable,
+                // with the same keys the populated path emits below.
+                if (flags.output === 'json') {
+                    this.log(JSON.stringify({ failed: 0, passed: 0, results: [] }, null, 2));
+                }
+                else {
+                    this.log('No unit tests found');
+                }
                 return;
             }
             if (flags.output === 'summary') {
@@ -161,6 +169,8 @@ Results: 2 passed, 1 failed
                     }
                 }
                 catch (error) {
+                    if (error instanceof Error && 'oclif' in error)
+                        throw error;
                     const message = error instanceof Error ? error.message : String(error);
                     results.push({
                         message,
@@ -202,6 +212,8 @@ Results: 2 passed, 1 failed
             }
         }
         catch (error) {
+            if (error instanceof Error && 'oclif' in error)
+                throw error;
             if (error instanceof Error) {
                 this.error(`Failed to run unit tests: ${error.message}`);
             }

package/dist/commands/workflow_test/run/index.js

@@ -55,24 +55,29 @@ Result: PASS (1.234s)
                 this.error(`API request failed with status ${response.status}: ${response.statusText}\n${errorText}`);
             }
             const result = await response.json();
+            // Anything other than 'ok' is a failure -- the API has been observed
+            // returning several distinct non-ok statuses, so never enumerate them.
+            const passed = result.status === 'ok';
             if (flags.output === 'json') {
                 this.log(JSON.stringify(result, null, 2));
             }
             else {
-                const timing = `(${result.timing.toFixed(3)}s)`;
-                if (result.status === 'ok') {
-                    this.log(`Result: PASS ${timing}`);
-                }
-                else {
-                    this.log(`Result: FAIL ${timing}`);
-                    if (result.message) {
-                        this.log(`  Error: ${result.message}`);
-                    }
-                    this.exit(1);
+                const timing = typeof result.timing === 'number' ? ` (${result.timing.toFixed(3)}s)` : '';
+                this.log(`Result: ${passed ? 'PASS' : 'FAIL'}${timing}`);
+                if (!passed && result.message) {
+                    this.log(`  Error: ${result.message}`);
                 }
             }
+            // Set the code rather than throwing via this.exit(): the exit must apply
+            // in both output modes, and process.exitCode cannot be swallowed by the
+            // surrounding catch. Mirrors what run_all already does.
+            if (!passed) {
+                process.exitCode = 1;
+            }
         }
         catch (error) {
+            if (error instanceof Error && 'oclif' in error)
+                throw error;
             if (error instanceof Error) {
                 this.error(`Failed to run workflow test: ${error.message}`);
             }