@xano/cli 1.0.6-beta.9 → 1.2.0

package/dist/commands/tenant/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, buildChannelServerResolver, parseDocument, resolveDocumentPath, } from '../../../utils/document-parser.js';
+import { buildApiGroupFolderResolver, parseDocument } 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 = [
@@ -129,19 +129,96 @@ 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) {
-            const { baseName, typeDir } = resolveDocumentPath(doc, outputDir, {
-                getApiGroupFolder,
-                getChannelServer,
-                join: path.join,
-                snakeCase,
-            });
+            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}`;
+                }
+            }
             fs.mkdirSync(typeDir, { recursive: true });
             // Track duplicates per directory
             const dirKey = path.relative(outputDir, typeDir);
@@ -160,4 +237,12 @@ Pulled 58 documents from tenant my-tenant
         }
         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}`);
             }

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

@@ -82,7 +82,15 @@ Results: 2 passed, 1 failed (2.691s total)
                 },
             });
             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: [], total_timing: 0 }, null, 2));
+                }
+                else {
+                    this.log('No workflow tests found');
+                }
                 return;
             }
             if (flags.output === 'summary') {
@@ -134,9 +142,9 @@ Results: 2 passed, 1 failed (2.691s total)
                         timing: runResult.timing,
                     };
                     results.push(result);
-                    totalTiming += runResult.timing;
+                    totalTiming += runResult.timing ?? 0;
                     if (flags.output === 'summary') {
-                        const timing = `(${runResult.timing.toFixed(3)}s)`;
+                        const timing = `(${(runResult.timing ?? 0).toFixed(3)}s)`;
                         if (passed) {
                             log(`PASS  ${test.name} ${timing}`);
                         }
@@ -149,6 +157,8 @@ Results: 2 passed, 1 failed (2.691s total)
                     }
                 }
                 catch (error) {
+                    if (error instanceof Error && 'oclif' in error)
+                        throw error;
                     const message = error instanceof Error ? error.message : String(error);
                     results.push({
                         message,
@@ -189,6 +199,8 @@ Results: 2 passed, 1 failed (2.691s total)
             }
         }
         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/workspace/git/pull/index.js

@@ -5,7 +5,7 @@ import * as os from 'node:os';
 import * as path from 'node:path';
 import snakeCase from 'lodash.snakecase';
 import BaseCommand, { buildUserAgent } from '../../../../base-command.js';
-import { buildApiGroupFolderResolver, buildChannelServerResolver, channelPathSegments, parseDocument, } from '../../../../utils/document-parser.js';
+import { buildApiGroupFolderResolver, parseDocument } from '../../../../utils/document-parser.js';
 export default class GitPull extends BaseCommand {
     static description = 'Pull XanoScript files from a git repository into a local directory';
     static examples = [
@@ -97,13 +97,10 @@ export default class GitPull extends BaseCommand {
             // Write documents to output directory using the same file tree logic as workspace pull
             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) {
-                const { baseName, typeDir } = this.resolveOutputPath(outputDir, doc, getApiGroupFolder, getChannelServer);
+                const { baseName, typeDir } = this.resolveOutputPath(outputDir, doc, getApiGroupFolder);
                 fs.mkdirSync(typeDir, { recursive: true });
                 // Track duplicates per directory
                 const dirKey = path.relative(outputDir, typeDir);
@@ -328,7 +325,7 @@ export default class GitPull extends BaseCommand {
      * Resolve the output directory and base filename for a parsed document.
      * Uses the same type-to-directory mapping as workspace pull.
      */
-    resolveOutputPath(outputDir, doc, getApiGroupFolder, getChannelServer) {
+    resolveOutputPath(outputDir, doc, getApiGroupFolder) {
         let typeDir;
         let baseName;
         if (doc.type === 'workspace') {
@@ -376,53 +373,6 @@ export default class GitPull extends BaseCommand {
             typeDir = path.join(outputDir, 'realtime', 'trigger');
             baseName = this.sanitizeFilename(doc.name);
         }
-        else if (doc.type === 'realtime_server') {
-            // Realtime v2 — see workspace/pull for the full rationale. Its own
-            // document is named after itself, mirroring api_group.
-            // realtime_server "chat" → realtime/server/chat/chat.xs
-            typeDir = path.join(outputDir, 'realtime', 'server', this.sanitizeFilename(doc.name));
-            baseName = this.sanitizeFilename(doc.name);
-        }
-        else if (doc.type === 'channel') {
-            // Realtime v2. The channel owns a directory named after itself (the full
-            // channel name, snake_cased into a single flat segment — like every other
-            // object), and its messages live in a message/ subfolder inside it. It
-            // nests under its owning realtime_server (from `server = "..."`):
-            //   channel "rooms/{room_id}" (server "chat")
-            //     → realtime/server/chat/channel/rooms_room_id/rooms_room_id.xs
-            //
-            // The channel name is snake_cased as a whole (rooms/{room_id} →
-            // rooms_room_id), so there is no path nesting and no bracket escaping —
-            // the on-disk name is a display form; the .xs content keeps the real
-            // name. A channel with no resolvable server (e.g. a pre-server export)
-            // falls back to the legacy flat channel/<path>/_channel.xs layout.
-            if (doc.server) {
-                typeDir = path.join(outputDir, 'realtime', 'server', this.sanitizeFilename(doc.server), 'channel', snakeCase(doc.name));
-                baseName = snakeCase(doc.name);
-            }
-            else {
-                typeDir = path.join(outputDir, 'channel', ...channelPathSegments(doc.name, snakeCase));
-                baseName = '_channel';
-            }
-        }
-        else if (doc.type === 'message' && doc.channel) {
-            // Realtime v2 message → nests in a message/ subfolder under its channel,
-            // under that channel's server. The message names only its channel; the
-            // server is resolved by looking that channel up in the same multidoc
-            // (two-pass resolve).
-            //   message "post" on channel "rooms/{room_id}" (server "chat")
-            //     → realtime/server/chat/channel/rooms_room_id/message/post.xs
-            //
-            // Nesting matters beyond tidiness: message names are unique only WITHIN
-            // a channel, so a flat message/ directory would collide when two
-            // channels both define e.g. "say". A channel whose server can't be
-            // resolved falls back to the legacy flat channel/ layout.
-            const messageServer = getChannelServer(doc.channel);
-            typeDir = messageServer
-                ? path.join(outputDir, 'realtime', 'server', this.sanitizeFilename(messageServer), 'channel', snakeCase(doc.channel), 'message')
-                : path.join(outputDir, 'channel', ...channelPathSegments(doc.channel, snakeCase));
-            baseName = this.sanitizeFilename(doc.name);
-        }
         else if (doc.type === 'api_group') {
             const groupFolder = getApiGroupFolder(doc.name);
             typeDir = path.join(outputDir, 'api', groupFolder);

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

@@ -14,4 +14,10 @@ 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;
 }