@xano/cli 0.0.83 → 0.0.85

package/README.md

@@ -118,7 +118,7 @@ xano workspace pull ./my-workspace --draft              # Include draft changes
 xano workspace push ./my-workspace
 xano workspace push ./my-workspace -b dev
 xano workspace push ./my-workspace --dry-run             # Preview changes without pushing
-xano workspace push ./my-workspace --partial             # No workspace block required
+xano workspace push ./my-workspace --partial             # Push only changed files, workspace block not required
 xano workspace push ./my-workspace --delete              # Delete objects not in the push
 xano workspace push ./my-workspace --records             # Include table records
 xano workspace push ./my-workspace --env                 # Include environment variables

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

@@ -164,7 +164,7 @@ Truncate all table records before importing
         if (documentEntries.length === 0) {
             this.error(`All .xs files in ${args.directory} are empty`);
         }
-        const multidoc = documentEntries.map((d) => d.content).join('\n---\n');
+        let multidoc = documentEntries.map((d) => d.content).join('\n---\n');
         // Build lookup map from document key to file path (for GUID writeback)
         const documentFileMap = new Map();
         for (const entry of documentEntries) {
@@ -176,7 +176,9 @@ Truncate all table records before importing
         }
         // Determine branch from flag or profile
         const branch = flags.branch || profile.branch || '';
-        // --partial implies --no-delete
+        if (flags.partial && flags.delete) {
+            this.error('Cannot use --delete with --partial');
+        }
         const shouldDelete = flags.partial ? false : flags.delete;
         // Construct the API URL
         const queryParams = new URLSearchParams({
@@ -195,10 +197,13 @@ Truncate all table records before importing
             'Content-Type': 'text/x-xanoscript',
         };
         // Preview mode: show what would change before pushing
+        let dryRunPreview = null;
         if (flags['dry-run'] || !flags.force) {
             const dryRunParams = new URLSearchParams(queryParams);
-            // Always request delete info in dry-run so we can show remote-only items
-            dryRunParams.set('delete', 'true');
+            // Request delete info in dry-run so we can show remote-only items (skip for partial)
+            if (!flags.partial) {
+                dryRunParams.set('delete', 'true');
+            }
             const dryRunUrl = `${profile.instance_origin}/api:meta/workspace/${workspaceId}/multidoc/dry-run?${dryRunParams.toString()}`;
             try {
                 const dryRunResponse = await this.verboseFetch(dryRunUrl, {
@@ -247,9 +252,32 @@ Truncate all table records before importing
                 else {
                     const dryRunText = await dryRunResponse.text();
                     const preview = JSON.parse(dryRunText);
+                    dryRunPreview = preview;
                     // Check if the server returned a valid dry-run response
                     if (preview && preview.summary) {
-                        this.renderPreview(preview, shouldDelete, workspaceId, flags.verbose);
+                        this.renderPreview(preview, shouldDelete, workspaceId, flags.verbose, flags.partial);
+                        // Check for critical errors that must block the push
+                        const criticalOps = preview.operations.filter((op) => op.details?.includes('exception:') || op.details?.includes('mvp:placeholder'));
+                        if (criticalOps.length > 0) {
+                            this.log('');
+                            this.log(ux.colorize('red', ux.colorize('bold', '=== CRITICAL ERRORS ===')));
+                            this.log('');
+                            this.log(ux.colorize('red', 'The following items contain syntax errors or unresolved placeholder statements'));
+                            this.log(ux.colorize('red', 'that would corrupt data if pushed. These must be resolved first:'));
+                            this.log('');
+                            for (const op of criticalOps) {
+                                this.log(`  ${ux.colorize('red', 'BLOCKED'.padEnd(16))} ${op.type.padEnd(18)} ${op.name}`);
+                                if (op.details) {
+                                    this.log(`  ${' '.repeat(16)} ${' '.repeat(18)} ${ux.colorize('dim', op.details)}`);
+                                }
+                            }
+                            this.log('');
+                            this.log(ux.colorize('red', `Push blocked: ${criticalOps.length} critical error(s) found.`));
+                            if (!flags.force) {
+                                return;
+                            }
+                            this.log(ux.colorize('yellow', 'Proceeding anyway due to --force flag.'));
+                        }
                         // Check if there are any actual changes (exclude deletes when --delete is off)
                         const hasChanges = Object.values(preview.summary).some((c) => c.created > 0 || c.updated > 0 || (shouldDelete && c.deleted > 0) || c.truncated > 0);
                         // Detect if local files contain records that would be imported
@@ -345,6 +373,25 @@ Truncate all table records before importing
                 }
             }
         }
+        // For partial pushes, filter to only changed documents
+        if (flags.partial && dryRunPreview) {
+            const changedKeys = new Set(dryRunPreview.operations
+                .filter((op) => op.action !== 'unchanged' && op.action !== 'delete' && op.action !== 'cascade_delete')
+                .map((op) => `${op.type}:${op.name}`));
+            const filteredEntries = documentEntries.filter((entry) => {
+                const parsed = parseDocument(entry.content);
+                if (!parsed)
+                    return true;
+                // For queries, operation name includes verb (e.g., "path/{id} DELETE")
+                const opName = parsed.verb ? `${parsed.name} ${parsed.verb}` : parsed.name;
+                return changedKeys.has(`${parsed.type}:${opName}`);
+            });
+            if (filteredEntries.length === 0) {
+                this.log('No changes to push.');
+                return;
+            }
+            multidoc = filteredEntries.map((d) => d.content).join('\n---\n');
+        }
         const apiUrl = `${profile.instance_origin}/api:meta/workspace/${workspaceId}/multidoc?${queryParams.toString()}`;
         const startTime = Date.now();
         try {
@@ -452,7 +499,8 @@ Truncate all table records before importing
             }
         }
         const elapsed = ((Date.now() - startTime) / 1000).toFixed(1);
-        this.log(`Pushed ${documentEntries.length} documents from ${args.directory} in ${elapsed}s`);
+        const pushedCount = multidoc.split('\n---\n').length;
+        this.log(`Pushed ${pushedCount} documents from ${args.directory} in ${elapsed}s`);
     }
     async confirm(message) {
         const readline = await import('node:readline');
@@ -473,7 +521,7 @@ Truncate all table records before importing
             });
         });
     }
-    renderPreview(result, willDelete, workspaceId, verbose = false) {
+    renderPreview(result, willDelete, workspaceId, verbose = false, partial = false) {
         const typeLabels = {
             addon: 'Addons',
             agent: 'Agents',
@@ -571,8 +619,8 @@ Truncate all table records before importing
             this.log(ux.colorize('yellow', '  If this is intended to be a field rename, use the Xano Admin — renaming is not'));
             this.log(ux.colorize('yellow', '  currently available through the CLI or Metadata API.'));
         }
-        // Show remote-only items when not using --delete
-        if (!willDelete && deleteOps.length > 0) {
+        // Show remote-only items when not using --delete (skip for partial pushes)
+        if (!willDelete && !partial && deleteOps.length > 0) {
             this.log('');
             this.log(ux.colorize('dim', '--- Remote Only (not included in push) ---'));
             this.log('');