@xano/cli 0.0.95-beta.23 → 0.0.95-beta.25

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

@@ -1,36 +1,106 @@
-import { Args, Flags, ux } from '@oclif/core';
+import { Flags } from '@oclif/core';
 import * as fs from 'node:fs';
-import * as path from 'node:path';
+import { resolve } from 'node:path';
+import open from 'open';
 import BaseCommand from '../../../base-command.js';
-import { findFilesWithGuid } from '../../../utils/document-parser.js';
-import { checkReferences, checkTableIndexes } from '../../../utils/reference-checker.js';
+import { executePush } from '../../../utils/multidoc-push.js';
 export default class SandboxPush extends BaseCommand {
-    static args = {
-        directory: Args.string({
-            description: 'Directory containing documents to push (as produced by sandbox pull or workspace pull)',
-            required: true,
-        }),
-    };
-    static description = 'Push local documents to your sandbox environment via multidoc import';
+    static description = 'Push local documents to your sandbox environment via multidoc import. By default, only changed files are pushed (partial mode). Use --sync to push all files. Shows a preview of changes before pushing unless --force is specified. Use --dry-run to preview only.';
     static examples = [
-        `$ xano sandbox push ./my-workspace
-Pushed 42 documents to sandbox environment from ./my-workspace
+        `$ xano sandbox push
+Push from current directory (default partial mode)
+`,
+        `$ xano sandbox push -d ./my-workspace
+Push from a specific directory
+`,
+        `$ xano sandbox push --sync
+Push all files to the sandbox
+`,
+        `$ xano sandbox push --sync --delete
+Push all files and delete remote objects not included
+`,
+        `$ xano sandbox push --dry-run
+Preview changes without pushing
+`,
+        `$ xano sandbox push --force
+Skip preview and push immediately
+`,
+        `$ xano sandbox push --records --env`,
+        `$ xano sandbox push --truncate`,
+        `$ xano sandbox push -i "**/func*"
+Push only files matching the glob pattern
+`,
+        `$ xano sandbox push -i "function/*" -i "table/*"
+Push files matching multiple patterns
+`,
+        `$ xano sandbox push -e "table/*"
+Push all files except tables
+`,
+        `$ xano sandbox push --review
+Push and open sandbox review in the browser
 `,
-        `$ xano sandbox push ./backup --records --env`,
-        `$ xano sandbox push ./my-workspace --truncate`,
     ];
     static flags = {
         ...BaseCommand.baseFlags,
+        directory: Flags.string({
+            char: 'd',
+            default: '.',
+            description: 'Directory containing documents to push (defaults to current directory)',
+            required: false,
+        }),
+        delete: Flags.boolean({
+            default: false,
+            description: 'Delete sandbox objects not included in the push (requires --sync)',
+            required: false,
+        }),
+        'dry-run': Flags.boolean({
+            default: false,
+            description: 'Show preview of changes without pushing (exit after preview)',
+            required: false,
+        }),
         env: Flags.boolean({
             default: false,
             description: 'Include environment variables in import',
             required: false,
         }),
+        exclude: Flags.string({
+            char: 'e',
+            description: 'Glob pattern to exclude files (e.g. "table/*", "**/test*"). Matched against relative paths from the push directory.',
+            multiple: true,
+            required: false,
+        }),
+        force: Flags.boolean({
+            default: false,
+            description: 'Skip preview and confirmation prompt (for CI/CD pipelines)',
+            required: false,
+        }),
+        guids: Flags.boolean({
+            allowNo: true,
+            default: true,
+            description: 'Write server-assigned GUIDs back to local files (use --no-guids to skip)',
+            required: false,
+        }),
+        include: Flags.string({
+            char: 'i',
+            description: 'Glob pattern to include files (e.g. "**/func*", "table/*.xs"). Matched against relative paths from the push directory.',
+            multiple: true,
+            required: false,
+        }),
         records: Flags.boolean({
             default: false,
             description: 'Include records in import',
             required: false,
         }),
+        review: Flags.boolean({
+            default: false,
+            description: 'Open sandbox review in the browser after pushing',
+            required: false,
+        }),
+        sync: Flags.boolean({
+            default: false,
+            description: 'Full push — send all files, not just changed ones. Required for --delete.',
+            required: false,
+        }),
         transaction: Flags.boolean({
             allowNo: true,
             default: true,
@@ -44,140 +114,83 @@ Pushed 42 documents to sandbox environment from ./my-workspace
         }),
     };
     async run() {
-        const { args, flags } = await this.parse(SandboxPush);
+        const { flags } = await this.parse(SandboxPush);
         const { profile } = this.resolveProfile(flags);
-        const inputDir = path.resolve(args.directory);
+        const inputDir = resolve(flags.directory);
         if (!fs.existsSync(inputDir)) {
             this.error(`Directory not found: ${inputDir}`);
         }
         if (!fs.statSync(inputDir).isDirectory()) {
             this.error(`Not a directory: ${inputDir}`);
         }
-        const files = this.collectFiles(inputDir);
-        if (files.length === 0) {
-            this.error(`No .xs files found in ${args.directory}`);
-        }
-        const documentEntries = [];
-        for (const filePath of files) {
-            const content = fs.readFileSync(filePath, 'utf8').trim();
-            if (content) {
-                documentEntries.push({ content, filePath });
-            }
-        }
-        if (documentEntries.length === 0) {
-            this.error(`All .xs files in ${args.directory} are empty`);
+        const baseUrl = `${profile.instance_origin}/api:meta/sandbox`;
+        const target = {
+            buildDryRunUrl: (params) => `${baseUrl}/multidoc/dry-run?${params.toString()}`,
+            buildPushUrl: (params) => `${baseUrl}/multidoc?${params.toString()}`,
+            cliVersion: this.config.version,
+            instanceOrigin: profile.instance_origin,
+            label: 'sandbox environment',
+            supportsBranches: false,
+            supportsPartial: false,
+        };
+        const pushFlags = {
+            delete: flags.delete,
+            'dry-run': flags['dry-run'],
+            env: flags.env,
+            exclude: flags.exclude,
+            force: flags.force,
+            guids: flags.guids,
+            include: flags.include,
+            records: flags.records,
+            sync: flags.sync,
+            transaction: flags.transaction,
+            truncate: flags.truncate,
+            verbose: flags.verbose,
+        };
+        await executePush({
+            accessToken: profile.access_token,
+            branch: '',
+            command: this,
+            inputDir,
+            verboseFetch: this.verboseFetch.bind(this),
+        }, target, pushFlags);
+        if (flags.review) {
+            await this.openReview(profile.instance_origin, profile.access_token, flags.verbose);
         }
-        // Check for bad cross-references within the local file set
-        const badRefs = checkReferences(documentEntries);
-        if (badRefs.length > 0) {
-            this.renderBadReferences(badRefs);
+    }
+    async openReview(instanceOrigin, accessToken, verbose) {
+        const response = await this.verboseFetch(`${instanceOrigin}/api:meta/sandbox/impersonate`, {
+            headers: {
+                accept: 'application/json',
+                Authorization: `Bearer ${accessToken}`,
+            },
+            method: 'GET',
+        }, verbose, accessToken);
+        if (!response.ok) {
+            const message = await this.parseApiError(response, 'Failed to open sandbox review');
+            this.error(message);
         }
-        // Check for indexes referencing non-existent schema fields
-        const badIndexes = checkTableIndexes(documentEntries);
-        if (badIndexes.length > 0) {
-            this.renderBadIndexes(badIndexes);
+        const result = (await response.json());
+        if (!result._ti) {
+            this.error('No one-time token returned from impersonate API');
         }
-        const multidoc = documentEntries.map((d) => d.content).join('\n---\n');
-        const queryParams = new URLSearchParams({
-            env: flags.env.toString(),
-            records: flags.records.toString(),
-            transaction: flags.transaction.toString(),
-            truncate: flags.truncate.toString(),
-        });
-        const apiUrl = `${profile.instance_origin}/api:meta/sandbox/multidoc?${queryParams.toString()}`;
-        const startTime = Date.now();
-        try {
-            const response = await this.verboseFetch(apiUrl, {
-                body: multidoc,
-                headers: {
-                    accept: 'application/json',
-                    Authorization: `Bearer ${profile.access_token}`,
-                    'Content-Type': 'text/x-xanoscript',
-                },
-                method: 'POST',
-            }, flags.verbose, profile.access_token);
-            if (!response.ok) {
-                const errorText = await response.text();
-                let errorMessage = `Push failed (${response.status})`;
-                try {
-                    const errorJson = JSON.parse(errorText);
-                    errorMessage += `: ${errorJson.message}`;
-                    if (errorJson.payload?.param) {
-                        errorMessage += `\n  Parameter: ${errorJson.payload.param}`;
-                    }
-                }
-                catch {
-                    errorMessage += `\n${errorText}`;
-                }
-                // Provide guidance when sandbox access is denied (free plan restriction)
-                if (response.status === 500 && errorMessage.includes('Access Denied')) {
-                    this.error('Sandbox is not available on the Free plan. Upgrade your plan to use sandbox features.');
-                }
-                const guidMatch = errorMessage.match(/Duplicate \w+ guid: (\S+)/);
-                if (guidMatch) {
-                    const dupeFiles = findFilesWithGuid(documentEntries, guidMatch[1]);
-                    if (dupeFiles.length > 0) {
-                        const relPaths = dupeFiles.map((f) => path.relative(inputDir, f));
-                        errorMessage += `\n  Local files with this GUID:\n${relPaths.map((f) => `    ${f}`).join('\n')}`;
-                    }
-                }
-                this.error(errorMessage);
-            }
-            const responseText = await response.text();
-            if (responseText && responseText !== 'null' && flags.verbose) {
-                this.log(responseText);
-            }
-        }
-        catch (error) {
-            if (error instanceof Error && 'oclif' in error)
-                throw error;
-            if (error instanceof Error) {
-                this.error(`Failed to push multidoc: ${error.message}`);
-            }
-            else {
-                this.error(`Failed to push multidoc: ${String(error)}`);
-            }
-        }
-        const elapsed = ((Date.now() - startTime) / 1000).toFixed(1);
-        this.log(`Pushed ${documentEntries.length} documents to sandbox environment from ${args.directory} in ${elapsed}s`);
+        const frontendUrl = this.getFrontendUrl(instanceOrigin);
+        const params = new URLSearchParams({ _ti: result._ti });
+        const reviewUrl = `${frontendUrl}/impersonate?${params.toString()}`;
+        this.log('Opening sandbox review...');
+        await open(reviewUrl);
     }
-    collectFiles(dir) {
-        const files = [];
-        const entries = fs.readdirSync(dir, { withFileTypes: true });
-        for (const entry of entries) {
-            const fullPath = path.join(dir, entry.name);
-            if (entry.isDirectory()) {
-                files.push(...this.collectFiles(fullPath));
-            }
-            else if (entry.isFile() && entry.name.endsWith('.xs')) {
-                files.push(fullPath);
+    getFrontendUrl(instanceOrigin) {
+        try {
+            const url = new URL(instanceOrigin);
+            if (url.hostname === 'localhost' || url.hostname === '127.0.0.1') {
+                url.port = '4200';
+                return url.origin;
             }
         }
-        return files.sort();
-    }
-    renderBadIndexes(badIndexes) {
-        this.log('');
-        this.log(ux.colorize('red', ux.colorize('bold', '=== CRITICAL: Invalid Indexes ===')));
-        this.log('');
-        this.log(ux.colorize('red', 'The following tables have indexed referencing fields that do not exist in the schema, which may cause related issues.'));
-        this.log('');
-        for (const idx of badIndexes) {
-            this.log(`  ${ux.colorize('red', 'CRITICAL'.padEnd(16))} ${'table'.padEnd(18)} ${idx.table}`);
-            this.log(`  ${' '.repeat(16)} ${' '.repeat(18)} ${ux.colorize('dim', `${idx.indexType} index → field "${idx.field}" does not exist in schema`)}`);
-        }
-        this.log('');
-    }
-    renderBadReferences(badRefs) {
-        this.log('');
-        this.log(ux.colorize('yellow', ux.colorize('bold', '=== Unresolved References ===')));
-        this.log('');
-        this.log(ux.colorize('yellow', "The following references point to objects that don't exist in this push or on the server."));
-        this.log(ux.colorize('yellow', 'These will become placeholder statements after import.'));
-        this.log('');
-        for (const ref of badRefs) {
-            this.log(`  ${ux.colorize('yellow', 'WARNING'.padEnd(16))} ${ref.sourceType.padEnd(18)} ${ref.source}`);
-            this.log(`  ${' '.repeat(16)} ${' '.repeat(18)} ${ux.colorize('dim', `${ref.statementType} → ${ref.targetType} "${ref.target}" does not exist`)}`);
+        catch {
+            // fall through
         }
-        this.log('');
+        return instanceOrigin;
     }
 }

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

@@ -1,11 +1,9 @@
 import BaseCommand from '../../../base-command.js';
 export default class Pull extends BaseCommand {
-    static args: {
-        directory: import("@oclif/core/interfaces").Arg<string, Record<string, unknown>>;
-    };
     static description: string;
     static examples: string[];
     static flags: {
+        directory: import("@oclif/core/interfaces").OptionFlag<string, import("@oclif/core/interfaces").CustomOptions>;
         draft: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         env: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         records: import("@oclif/core/interfaces").BooleanFlag<boolean>;

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

@@ -1,4 +1,4 @@
-import { Args, Flags } from '@oclif/core';
+import { Flags } from '@oclif/core';
 import * as yaml from 'js-yaml';
 import * as fs from 'node:fs';
 import * as path from 'node:path';
@@ -6,29 +6,30 @@ import snakeCase from 'lodash.snakecase';
 import BaseCommand from '../../../base-command.js';
 import { buildApiGroupFolderResolver, parseDocument } from '../../../utils/document-parser.js';
 export default class Pull extends BaseCommand {
-    static args = {
-        directory: Args.string({
-            description: 'Output directory for pulled documents',
-            required: true,
-        }),
-    };
     static description = 'Pull a tenant multidoc from the Xano Metadata API and split into individual files';
     static examples = [
-        `$ xano tenant pull ./my-tenant -t my-tenant
+        `$ xano tenant pull -t my-tenant
+Pulled 42 documents from tenant my-tenant to current directory
+`,
+        `$ xano tenant pull -d ./my-tenant -t my-tenant
 Pulled 42 documents from tenant my-tenant to ./my-tenant
 `,
-        `$ xano tenant pull ./output -t my-tenant -w 40
+        `$ xano tenant pull -d ./output -t my-tenant -w 40
 Pulled 15 documents from tenant my-tenant to ./output
 `,
-        `$ xano tenant pull ./backup -t my-tenant --profile production --env --records
-Pulled 58 documents from tenant my-tenant to ./backup
-`,
-        `$ xano tenant pull ./my-tenant -t my-tenant --draft
-Pulled 42 documents from tenant my-tenant to ./my-tenant
+        `$ xano tenant pull -t my-tenant --profile production --env --records
+Pulled 58 documents from tenant my-tenant
 `,
+        `$ xano tenant pull -t my-tenant --draft`,
     ];
     static flags = {
         ...BaseCommand.baseFlags,
+        directory: Flags.string({
+            char: 'd',
+            default: '.',
+            description: 'Output directory for pulled documents (defaults to current directory)',
+            required: false,
+        }),
         draft: Flags.boolean({
             default: false,
             description: 'Include draft versions',
@@ -56,7 +57,7 @@ Pulled 42 documents from tenant my-tenant to ./my-tenant
         }),
     };
     async run() {
-        const { args, flags } = await this.parse(Pull);
+        const { flags } = await this.parse(Pull);
         // Get profile name (default or from flag/env)
         const profileName = flags.profile || this.getDefaultProfile();
         // Load credentials
@@ -84,7 +85,7 @@ Pulled 42 documents from tenant my-tenant to ./my-tenant
         }
         else {
             this.error(`Workspace ID is required. Either:\n` +
-                `  1. Provide it as a flag: xano tenant pull <directory> -t <tenant_name> -w <workspace_id>\n` +
+                `  1. Provide it as a flag: xano tenant pull -t <tenant_name> -w <workspace_id>\n` +
                 `  2. Set it in your profile using: xano profile:edit ${profileName} -w <workspace_id>`);
         }
         const tenantName = flags.tenant;
@@ -140,7 +141,7 @@ Pulled 42 documents from tenant my-tenant to ./my-tenant
             return;
         }
         // Resolve the output directory
-        const outputDir = path.resolve(args.directory);
+        const outputDir = path.resolve(flags.directory);
         // Create the output directory if it doesn't exist
         fs.mkdirSync(outputDir, { recursive: true });
         // Resolve api_group names to unique folder names, disambiguating collisions
@@ -246,7 +247,7 @@ Pulled 42 documents from tenant my-tenant to ./my-tenant
             fs.writeFileSync(filePath, doc.content, 'utf8');
             writtenCount++;
         }
-        this.log(`Pulled ${writtenCount} documents from tenant ${tenantName} to ${args.directory}`);
+        this.log(`Pulled ${writtenCount} documents from tenant ${tenantName} to ${flags.directory}`);
     }
     loadCredentials() {
         const credentialsPath = this.getCredentialsPath();

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

@@ -1,12 +1,10 @@
 import BaseCommand from '../../../../base-command.js';
 export default class GitPull extends BaseCommand {
-    static args: {
-        directory: import("@oclif/core/interfaces").Arg<string, Record<string, unknown>>;
-    };
     static description: string;
     static examples: string[];
     static flags: {
         branch: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        directory: import("@oclif/core/interfaces").OptionFlag<string, import("@oclif/core/interfaces").CustomOptions>;
         path: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         repo: import("@oclif/core/interfaces").OptionFlag<string, import("@oclif/core/interfaces").CustomOptions>;
         token: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;

package/dist/commands/workspace/git/pull/index.js

@@ -1,4 +1,4 @@
-import { Args, Flags } from '@oclif/core';
+import { Flags } from '@oclif/core';
 import { execSync } from 'node:child_process';
 import * as fs from 'node:fs';
 import * as os from 'node:os';
@@ -7,21 +7,16 @@ import snakeCase from 'lodash.snakecase';
 import BaseCommand, { buildUserAgent } from '../../../../base-command.js';
 import { buildApiGroupFolderResolver, parseDocument } from '../../../../utils/document-parser.js';
 export default class GitPull extends BaseCommand {
-    static args = {
-        directory: Args.string({
-            description: 'Output directory for imported files',
-            required: true,
-        }),
-    };
     static description = 'Pull XanoScript files from a git repository into a local directory';
     static examples = [
-        `$ xano workspace git pull ./output -r https://github.com/owner/repo`,
-        `$ xano workspace git pull ./output -r https://github.com/owner/repo/tree/main/path/to/dir`,
-        `$ xano workspace git pull ./output -r https://github.com/owner/repo/blob/main/path/to/file.xs`,
-        `$ xano workspace git pull ./output -r git@github.com:owner/repo.git`,
-        `$ xano workspace git pull ./output -r https://github.com/owner/private-repo -t ghp_xxx`,
-        `$ xano workspace git pull ./output -r https://gitlab.com/owner/repo/-/tree/master/path`,
-        `$ xano workspace git pull ./output -r https://gitlab.com/owner/repo -b main`,
+        `$ xano workspace git pull -r https://github.com/owner/repo`,
+        `$ xano workspace git pull -d ./output -r https://github.com/owner/repo`,
+        `$ xano workspace git pull -r https://github.com/owner/repo/tree/main/path/to/dir`,
+        `$ xano workspace git pull -r https://github.com/owner/repo/blob/main/path/to/file.xs`,
+        `$ xano workspace git pull -r git@github.com:owner/repo.git`,
+        `$ xano workspace git pull -r https://github.com/owner/private-repo -t ghp_xxx`,
+        `$ xano workspace git pull -r https://gitlab.com/owner/repo/-/tree/master/path`,
+        `$ xano workspace git pull -r https://gitlab.com/owner/repo -b main`,
     ];
     static flags = {
         ...BaseCommand.baseFlags,
@@ -30,6 +25,12 @@ export default class GitPull extends BaseCommand {
             description: 'Branch, tag, or ref to fetch (defaults to repository default branch)',
             required: false,
         }),
+        directory: Flags.string({
+            char: 'd',
+            default: '.',
+            description: 'Output directory for imported files (defaults to current directory)',
+            required: false,
+        }),
         path: Flags.string({
             description: 'Subdirectory within the repo to import from',
             required: false,
@@ -47,9 +48,9 @@ export default class GitPull extends BaseCommand {
         }),
     };
     async run() {
-        const { args, flags } = await this.parse(GitPull);
+        const { flags } = await this.parse(GitPull);
         const token = flags.token || '';
-        const outputDir = path.resolve(args.directory);
+        const outputDir = path.resolve(flags.directory);
         // Normalize the URL to extract owner/repo/ref/path from various formats
         const repoInfo = this.parseRepoUrl(flags.repo);
         // CLI flags override values extracted from the URL
@@ -115,7 +116,7 @@ export default class GitPull extends BaseCommand {
                 writtenCount++;
             }
             const source = subPath ? `${flags.repo} (${subPath})` : flags.repo;
-            this.log(`Pulled ${writtenCount} documents from ${source} to ${args.directory}`);
+            this.log(`Pulled ${writtenCount} documents from ${source} to ${flags.directory}`);
         }
         finally {
             // Clean up temp directory

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

@@ -1,12 +1,10 @@
 import BaseCommand from '../../../base-command.js';
 export default class Pull extends BaseCommand {
-    static args: {
-        directory: import("@oclif/core/interfaces").Arg<string, Record<string, unknown>>;
-    };
     static description: string;
     static examples: string[];
     static flags: {
         branch: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+        directory: import("@oclif/core/interfaces").OptionFlag<string, import("@oclif/core/interfaces").CustomOptions>;
         env: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         draft: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         records: import("@oclif/core/interfaces").BooleanFlag<boolean>;

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

@@ -1,4 +1,4 @@
-import { Args, Flags } from '@oclif/core';
+import { Flags } from '@oclif/core';
 import * as yaml from 'js-yaml';
 import * as fs from 'node:fs';
 import * as path from 'node:path';
@@ -6,29 +6,22 @@ import snakeCase from 'lodash.snakecase';
 import BaseCommand from '../../../base-command.js';
 import { buildApiGroupFolderResolver, parseDocument } from '../../../utils/document-parser.js';
 export default class Pull extends BaseCommand {
-    static args = {
-        directory: Args.string({
-            description: 'Output directory for pulled documents',
-            required: true,
-        }),
-    };
     static description = 'Pull a workspace multidoc from the Xano Metadata API and split into individual files';
     static examples = [
-        `$ xano workspace pull ./my-workspace
+        `$ xano workspace pull
+Pulled 42 documents to current directory
+`,
+        `$ xano workspace pull -d ./my-workspace
 Pulled 42 documents to ./my-workspace
 `,
-        `$ xano workspace pull ./output -w 40
+        `$ xano workspace pull -d ./output -w 40
 Pulled 15 documents to ./output
 `,
-        `$ xano workspace pull ./backup --profile production --env --records
-Pulled 58 documents to ./backup
-`,
-        `$ xano workspace pull ./my-workspace --draft
-Pulled 42 documents to ./my-workspace
-`,
-        `$ xano workspace pull ./my-workspace -b dev
-Pulled 42 documents to ./my-workspace
+        `$ xano workspace pull --profile production --env --records
+Pulled 58 documents
 `,
+        `$ xano workspace pull --draft`,
+        `$ xano workspace pull -b dev`,
     ];
     static flags = {
         ...BaseCommand.baseFlags,
@@ -37,6 +30,12 @@ Pulled 42 documents to ./my-workspace
             description: 'Branch name (optional if set in profile, defaults to live)',
             required: false,
         }),
+        directory: Flags.string({
+            char: 'd',
+            default: '.',
+            description: 'Output directory for pulled documents (defaults to current directory)',
+            required: false,
+        }),
         env: Flags.boolean({
             default: false,
             description: 'Include environment variables',
@@ -59,7 +58,7 @@ Pulled 42 documents to ./my-workspace
         }),
     };
     async run() {
-        const { args, flags } = await this.parse(Pull);
+        const { flags } = await this.parse(Pull);
         // Get profile name (default or from flag/env)
         const profileName = flags.profile || this.getDefaultProfile();
         // Load credentials
@@ -87,7 +86,7 @@ Pulled 42 documents to ./my-workspace
         }
         else {
             this.error(`Workspace ID is required. Either:\n` +
-                `  1. Provide it as a flag: xano workspace pull <directory> -w <workspace_id>\n` +
+                `  1. Provide it as a flag: xano workspace pull -w <workspace_id>\n` +
                 `  2. Set it in your profile using: xano profile:edit ${profileName} -w <workspace_id>`);
         }
         // Determine branch from flag or profile
@@ -145,7 +144,7 @@ Pulled 42 documents to ./my-workspace
             return;
         }
         // Resolve the output directory
-        const outputDir = path.resolve(args.directory);
+        const outputDir = path.resolve(flags.directory);
         // Create the output directory if it doesn't exist
         fs.mkdirSync(outputDir, { recursive: true });
         // Resolve api_group names to unique folder names, disambiguating collisions
@@ -252,7 +251,7 @@ Pulled 42 documents to ./my-workspace
             fs.writeFileSync(filePath, doc.content, 'utf8');
             writtenCount++;
         }
-        this.log(`Pulled ${writtenCount} documents to ${args.directory}`);
+        this.log(`Pulled ${writtenCount} documents to ${flags.directory}`);
     }
     loadCredentials() {
         const credentialsPath = this.getCredentialsPath();