@xano/cli 1.0.6-beta.1 → 1.0.6-beta.11

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

@@ -3,7 +3,7 @@ import * as fs from 'node:fs';
 import * as path from 'node:path';
 import snakeCase from 'lodash.snakecase';
 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 ReleasePull extends BaseCommand {
     static description = 'Pull a release multidoc from the Xano Metadata API and split into individual files';
     static examples = [
@@ -123,96 +123,19 @@ Pulled 58 documents from release 'v1.0'
         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);
@@ -257,12 +180,4 @@ Pulled 58 documents from release 'v1.0'
         }
         return match.id;
     }
-    /**
-     * 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/sandbox/env/list/index.js

@@ -1,5 +1,6 @@
 import { Flags } from '@oclif/core';
 import BaseCommand from '../../../../base-command.js';
+import { formatPagingFooter } from '../../../../utils/paging.js';
 export default class SandboxEnvList extends BaseCommand {
     static description = 'List environment variable keys for a sandbox environment';
     static examples = [
@@ -51,6 +52,9 @@ Environment variables for sandbox environment:
                         this.log(`  - ${envVar.name}`);
                     }
                 }
+                const footer = formatPagingFooter({ items: envVars }, { noun: 'environment variable', tier: 'none' });
+                if (footer)
+                    this.log(footer);
             }
         }
         catch (error) {

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

@@ -12,5 +12,4 @@ export default class SandboxPull extends BaseCommand {
         verbose: import("@oclif/core/interfaces").BooleanFlag<boolean>;
     };
     run(): Promise<void>;
-    private sanitizeFilename;
 }

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

@@ -1,7 +1,7 @@
 import { Flags } from '@oclif/core';
 import snakeCase from 'lodash.snakecase';
 import BaseCommand from '../../../base-command.js';
-import { buildApiGroupFolderResolver, parseDocument } from '../../../utils/document-parser.js';
+import { buildApiGroupFolderResolver, buildChannelServerResolver, parseDocument, resolveDocumentPath, } from '../../../utils/document-parser.js';
 import { fetchKnowledge, writeKnowledge } from '../../../utils/knowledge-sync.js';
 import * as fs from 'node:fs';
 import * as path from 'node:path';
@@ -92,82 +92,18 @@ Pulled 42 documents from sandbox environment to ./my-sandbox
         const outputDir = path.resolve(flags.directory);
         fs.mkdirSync(outputDir, { recursive: true });
         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);
         const filenameCounters = new Map();
         let writtenCount = 0;
         for (const doc of documents) {
-            let typeDir;
-            let baseName;
-            if (doc.type === 'workspace') {
-                typeDir = path.join(outputDir, 'workspace');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'workspace_trigger') {
-                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') {
-                typeDir = path.join(outputDir, 'ai', 'agent');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'mcp_server') {
-                typeDir = path.join(outputDir, 'ai', 'mcp_server');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'tool') {
-                typeDir = path.join(outputDir, 'ai', 'tool');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'agent_trigger') {
-                typeDir = path.join(outputDir, 'ai', 'agent', 'trigger');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'mcp_server_trigger') {
-                typeDir = path.join(outputDir, 'ai', 'mcp_server', 'trigger');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'table_trigger') {
-                typeDir = path.join(outputDir, 'table', 'trigger');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'realtime_channel') {
-                typeDir = path.join(outputDir, 'realtime', 'channel');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'realtime_trigger') {
-                typeDir = path.join(outputDir, 'realtime', 'trigger');
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'api_group') {
-                const groupFolder = getApiGroupFolder(doc.name);
-                typeDir = path.join(outputDir, 'api', groupFolder);
-                baseName = this.sanitizeFilename(doc.name);
-            }
-            else if (doc.type === 'query' && doc.apiGroup) {
-                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 {
-                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 });
             const dirKey = path.relative(outputDir, typeDir);
             if (!filenameCounters.has(dirKey)) {
@@ -194,7 +130,4 @@ Pulled 42 documents from sandbox environment to ./my-sandbox
             parts.push(`${knowledgeCount} knowledge file${knowledgeCount === 1 ? '' : 's'}`);
         this.log(`Pulled ${parts.join(' + ')} from sandbox environment to ${flags.directory}`);
     }
-    sanitizeFilename(name) {
-        return snakeCase(name.replaceAll('"', ''));
-    }
 }

package/dist/commands/sandbox/unit_test/list/index.d.ts

@@ -3,6 +3,8 @@ export default class SandboxUnitTestList extends BaseCommand {
     static description: string;
     static examples: string[];
     static flags: {
+        page: import("@oclif/core/interfaces").OptionFlag<number, import("@oclif/core/interfaces").CustomOptions>;
+        per_page: import("@oclif/core/interfaces").OptionFlag<number, import("@oclif/core/interfaces").CustomOptions>;
         branch: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         'obj-type': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         output: import("@oclif/core/interfaces").OptionFlag<string, import("@oclif/core/interfaces").CustomOptions>;

package/dist/commands/sandbox/unit_test/list/index.js

@@ -1,5 +1,6 @@
 import { Flags } from '@oclif/core';
 import BaseCommand from '../../../../base-command.js';
+import { buildPagingJson, buildPagingParams, formatPagingFooter, normalizeListResponse, pagingFlags } from '../../../../utils/paging.js';
 export default class SandboxUnitTestList extends BaseCommand {
     static description = 'List all unit tests for a sandbox environment';
     static examples = [
@@ -28,12 +29,12 @@ Unit tests:
             options: ['summary', 'json'],
             required: false,
         }),
+        ...pagingFlags('envelope', { maxPerPage: 10_000 }),
     };
     async run() {
         const { flags } = await this.parse(SandboxUnitTestList);
         const { profile } = this.resolveProfile(flags);
-        const params = new URLSearchParams();
-        params.set('per_page', '10000');
+        const params = new URLSearchParams(buildPagingParams(flags, 'envelope'));
         if (flags.branch)
             params.set('branch', flags.branch);
         if (flags['obj-type'])
@@ -51,19 +52,10 @@ Unit tests:
                 const message = await this.parseApiError(response, 'API request failed');
                 this.error(message);
             }
-            const data = (await response.json());
-            let tests;
-            if (Array.isArray(data)) {
-                tests = data;
-            }
-            else if (data && typeof data === 'object' && 'items' in data && Array.isArray(data.items)) {
-                tests = data.items;
-            }
-            else {
-                this.error('Unexpected API response format');
-            }
+            const list = normalizeListResponse(await response.json());
+            const tests = list.items;
             if (flags.output === 'json') {
-                this.log(JSON.stringify(tests, null, 2));
+                this.log(JSON.stringify(buildPagingJson(list, { page: flags.page, perPage: flags.per_page, tier: 'envelope' }), null, 2));
             }
             else {
                 if (tests.length === 0) {
@@ -75,6 +67,9 @@ Unit tests:
                         this.log(`  - ${test.name} (ID: ${test.id}) [${test.obj_type}: ${test.obj_name}]`);
                     }
                 }
+                const footer = formatPagingFooter(list, { noun: 'unit test', page: flags.page, tier: 'envelope' });
+                if (footer)
+                    this.log(footer);
             }
         }
         catch (error) {

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

@@ -46,24 +46,30 @@ Result: PASS
                 this.error(message);
             }
             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)

package/dist/commands/sandbox/unit_test/run_all/index.d.ts

@@ -4,6 +4,7 @@ export default class SandboxUnitTestRunAll extends BaseCommand {
     static examples: string[];
     static flags: {
         branch: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        concurrency: import("@oclif/core/interfaces").OptionFlag<number, import("@oclif/core/interfaces").CustomOptions>;
         'obj-type': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         output: import("@oclif/core/interfaces").OptionFlag<string, import("@oclif/core/interfaces").CustomOptions>;
         config: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;

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

@@ -1,5 +1,7 @@
 import { Flags } from '@oclif/core';
 import BaseCommand from '../../../../base-command.js';
+import { createOrderedEmitter, mapWithConcurrency } from '../../../../utils/concurrency.js';
+import { collectAllPages, INTERNAL_PAGE_SIZE, normalizeListResponse } from '../../../../utils/paging.js';
 export default class SandboxUnitTestRunAll extends BaseCommand {
     static description = 'Run all unit tests for a sandbox environment';
     static examples = [
@@ -21,6 +23,11 @@ Results: 4 passed, 1 failed
             description: 'Filter by branch name',
             required: false,
         }),
+        concurrency: Flags.integer({
+            default: 1,
+            description: 'Run this many tests in parallel. Tests share the workspace database, so raise this only when your tests do not depend on shared state.',
+            required: false,
+        }),
         'obj-type': Flags.string({
             description: 'Filter by object type',
             options: ['function', 'query', 'middleware'],
@@ -40,36 +47,46 @@ Results: 4 passed, 1 failed
         const baseUrl = `${profile.instance_origin}/api:meta/sandbox/unit_test`;
         try {
             // Step 1: List all unit tests
-            const listParams = new URLSearchParams();
-            listParams.set('per_page', '10000');
-            if (flags.branch)
-                listParams.set('branch', flags.branch);
-            if (flags['obj-type'])
-                listParams.set('obj_type', flags['obj-type']);
-            const listResponse = await this.verboseFetch(`${baseUrl}?${listParams}`, {
-                headers: {
-                    accept: 'application/json',
-                    Authorization: `Bearer ${profile.access_token}`,
+            // Page the list rather than asking for everything at once: 100 per
+            // request is plenty for typical suites and does not make the instance
+            // build one enormous response. Termination is the server's own nextPage,
+            // never "the page came back full".
+            const branchValue = flags.branch;
+            const tests = await collectAllPages(async (page) => {
+                const listParams = new URLSearchParams();
+                listParams.set('page', String(page));
+                listParams.set('per_page', String(INTERNAL_PAGE_SIZE));
+                if (branchValue)
+                    listParams.set('branch', branchValue);
+                if (flags['obj-type'])
+                    listParams.set('obj_type', flags['obj-type']);
+                const listResponse = await this.verboseFetch(`${baseUrl}?${listParams}`, {
+                    headers: {
+                        accept: 'application/json',
+                        Authorization: `Bearer ${profile.access_token}`,
+                    },
+                    method: 'GET',
+                }, flags.verbose, profile.access_token);
+                if (!listResponse.ok) {
+                    const errorText = await listResponse.text();
+                    this.error(`Failed to list unit tests: ${listResponse.status}: ${listResponse.statusText}\n${errorText}`);
+                }
+                return normalizeListResponse(await listResponse.json());
+            }, {
+                onTruncate: (collected) => {
+                    this.warn(`Stopped after ${collected} unit tests — the list did not terminate. Some tests may not have run.`);
                 },
-                method: 'GET',
-            }, flags.verbose, profile.access_token);
-            if (!listResponse.ok) {
-                const errorText = await listResponse.text();
-                this.error(`Failed to list unit tests: ${listResponse.status}: ${listResponse.statusText}\n${errorText}`);
-            }
-            const data = (await listResponse.json());
-            let tests;
-            if (Array.isArray(data)) {
-                tests = data;
-            }
-            else if (data && typeof data === 'object' && 'items' in data && Array.isArray(data.items)) {
-                tests = data.items;
-            }
-            else {
-                this.error('Unexpected API response format');
-            }
+            });
             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') {
@@ -77,7 +94,15 @@ Results: 4 passed, 1 failed
             }
             // Step 2: Run each test
             const results = [];
-            for (const test of tests) {
+            const runOne = async (test) => {
+                // Shadowed accumulators: the body below is the original sequential
+                // logic, but it records output instead of printing it so a concurrent
+                // run can still emit in input order.
+                const results = [];
+                const lines = [];
+                const log = (message) => {
+                    lines.push(message);
+                };
                 const runUrl = `${baseUrl}/${test.id}/run`;
                 try {
                     const runResponse = await this.verboseFetch(runUrl, {
@@ -98,10 +123,10 @@ Results: 4 passed, 1 failed
                             status: 'fail',
                         });
                         if (flags.output === 'summary') {
-                            this.log(`FAIL  ${test.name} [${test.obj_type}: ${test.obj_name}]`);
-                            this.log(`      Error: API error ${runResponse.status}`);
+                            log(`FAIL  ${test.name} [${test.obj_type}: ${test.obj_name}]`);
+                            log(`      Error: API error ${runResponse.status}`);
                         }
-                        continue;
+                        return { lines, results };
                     }
                     const runResult = (await runResponse.json());
                     const passed = runResult.status === 'ok';
@@ -115,13 +140,13 @@ Results: 4 passed, 1 failed
                     });
                     if (flags.output === 'summary') {
                         if (passed) {
-                            this.log(`PASS  ${test.name} [${test.obj_type}: ${test.obj_name}]`);
+                            log(`PASS  ${test.name} [${test.obj_type}: ${test.obj_name}]`);
                         }
                         else {
-                            this.log(`FAIL  ${test.name} [${test.obj_type}: ${test.obj_name}]`);
+                            log(`FAIL  ${test.name} [${test.obj_type}: ${test.obj_name}]`);
                             for (const expect of failedExpects) {
                                 if (expect.message) {
-                                    this.log(`      Error: ${expect.message}`);
+                                    log(`      Error: ${expect.message}`);
                                 }
                             }
                         }
@@ -139,11 +164,24 @@ Results: 4 passed, 1 failed
                         status: 'fail',
                     });
                     if (flags.output === 'summary') {
-                        this.log(`FAIL  ${test.name} [${test.obj_type}: ${test.obj_name}]`);
-                        this.log(`      Error: ${message}`);
+                        log(`FAIL  ${test.name} [${test.obj_type}: ${test.obj_name}]`);
+                        log(`      Error: ${message}`);
                     }
                 }
-            }
+                return { lines, results };
+            };
+            // Print in input order as each test settles, so concurrent output stays
+            // diffable against a sequential run.
+            const emitter = createOrderedEmitter((settled) => {
+                for (const line of settled.lines)
+                    this.log(line);
+            });
+            const perTest = await mapWithConcurrency(tests, flags.concurrency, async (test, index) => {
+                const settled = await runOne(test);
+                emitter.settle(index, settled);
+                return settled;
+            });
+            results.push(...perTest.flatMap((r) => r.results));
             // Step 3: Summary
             const passed = results.filter((r) => r.status === 'pass').length;
             const failed = results.filter((r) => r.status === 'fail').length;

package/dist/commands/sandbox/workflow_test/list/index.d.ts

@@ -3,6 +3,8 @@ export default class SandboxWorkflowTestList extends BaseCommand {
     static description: string;
     static examples: string[];
     static flags: {
+        page: import("@oclif/core/interfaces").OptionFlag<number, import("@oclif/core/interfaces").CustomOptions>;
+        per_page: import("@oclif/core/interfaces").OptionFlag<number, import("@oclif/core/interfaces").CustomOptions>;
         branch: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         output: import("@oclif/core/interfaces").OptionFlag<string, import("@oclif/core/interfaces").CustomOptions>;
         config: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;

package/dist/commands/sandbox/workflow_test/list/index.js

@@ -1,5 +1,6 @@
 import { Flags } from '@oclif/core';
 import BaseCommand from '../../../../base-command.js';
+import { buildPagingJson, buildPagingParams, formatPagingFooter, normalizeListResponse, pagingFlags } from '../../../../utils/paging.js';
 export default class SandboxWorkflowTestList extends BaseCommand {
     static description = 'List workflow tests for a sandbox environment';
     static examples = [
@@ -23,12 +24,12 @@ Workflow tests:
             options: ['summary', 'json'],
             required: false,
         }),
+        ...pagingFlags('envelope', { maxPerPage: 10_000 }),
     };
     async run() {
         const { flags } = await this.parse(SandboxWorkflowTestList);
         const { profile } = this.resolveProfile(flags);
-        const params = new URLSearchParams();
-        params.set('per_page', '10000');
+        const params = new URLSearchParams(buildPagingParams(flags, 'envelope'));
         if (flags.branch)
             params.set('branch', flags.branch);
         const apiUrl = `${profile.instance_origin}/api:meta/sandbox/workflow_test?${params}`;
@@ -44,19 +45,10 @@ Workflow tests:
                 const message = await this.parseApiError(response, 'API request failed');
                 this.error(message);
             }
-            const data = (await response.json());
-            let tests;
-            if (Array.isArray(data)) {
-                tests = data;
-            }
-            else if (data && typeof data === 'object' && 'items' in data && Array.isArray(data.items)) {
-                tests = data.items;
-            }
-            else {
-                this.error('Unexpected API response format');
-            }
+            const list = normalizeListResponse(await response.json());
+            const tests = list.items;
             if (flags.output === 'json') {
-                this.log(JSON.stringify(tests, null, 2));
+                this.log(JSON.stringify(buildPagingJson(list, { page: flags.page, perPage: flags.per_page, tier: 'envelope' }), null, 2));
             }
             else {
                 if (tests.length === 0) {
@@ -68,6 +60,9 @@ Workflow tests:
                         this.log(`  - ${test.name} (ID: ${test.id})`);
                     }
                 }
+                const footer = formatPagingFooter(list, { noun: 'workflow test', page: flags.page, tier: 'envelope' });
+                if (footer)
+                    this.log(footer);
             }
         }
         catch (error) {

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

@@ -46,22 +46,25 @@ Result: PASS (0.25s)
                 this.error(message);
             }
             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)