primo-cli 0.1.19 → 0.1.20

package/dist/commands/deploy.js

@@ -479,18 +479,26 @@ async function wait_for_ready(url) {
 // Run the equivalent of `primo push` against the local workspace. push_site
 // resolves the server URL from server.yaml (which we just wrote), so no extra
 // flags are needed. We surface failures but don't re-throw — the deploy already
-// succeeded and the user can rerun push manually.
+// succeeded and the user can rerun push manually. Only a fully clean push
+// counts as success: a partial upload must not be reported as "complete".
 async function run_auto_push(inventory) {
     console.log('');
     console.log(chalk.cyan('Uploading your workspace...'));
     try {
-        await push_site({ dir: inventory.root_dir });
+        const failed = await push_site({ dir: inventory.root_dir });
+        if (failed.length > 0) {
+            // push_site already printed the per-item errors, the summary, and
+            // set the exit code. The server itself is fine, so finish the
+            // deploy flow — just not as "complete".
+            return false;
+        }
         return true;
     }
     catch (error) {
         console.log('');
         console.log(chalk.yellow(`Auto-push failed: ${error instanceof Error ? error.message : error}`));
         console.log(chalk.dim('  Your server is live — rerun `primo push` once the issue is sorted.'));
+        process.exitCode = 1;
         return false;
     }
 }

package/dist/commands/push.d.ts

@@ -7,5 +7,5 @@ interface PushOptions {
     preview?: boolean;
     dryRun?: boolean;
 }
-export declare function push_site(options: PushOptions): Promise<void>;
+export declare function push_site(options: PushOptions): Promise<string[]>;
 export {};

package/dist/commands/push.js

@@ -39,6 +39,8 @@ async function resolve_group_name(site_dir, group_id) {
     }
     return undefined;
 }
+// Returns the labels (site slugs / 'library') that failed to push so callers
+// like `primo deploy` can tell a clean run from a partial one. Empty = success.
 export async function push_site(options) {
     const root_dir = path.resolve(options.dir);
     const has_site_yaml = await path_exists(get_site_config_path(root_dir));
@@ -61,17 +63,27 @@ export async function push_site(options) {
         : options;
     if (options.dryRun) {
         await print_push_dry_run(root_dir, has_site_yaml, has_server_yaml, effective_options);
-        return;
+        return [];
     }
     // Server-folder mode: walk site subfolders + push library
     if (!has_site_yaml && has_server_yaml) {
-        await push_server(root_dir, effective_options);
-        return;
+        const failed = await push_server(root_dir, effective_options);
+        if (failed.length > 0) {
+            console.log('');
+            console.log(chalk.red(`Push incomplete — failed: ${failed.join(', ')}`));
+            console.log(chalk.dim('  Fix the errors above and rerun `primo push` (completed pushes are safe to repeat).'));
+            console.log('');
+            // exitCode (not process.exit) so an in-process caller like `primo
+            // deploy` can still finish its own reporting before the process ends.
+            process.exitCode = 1;
+        }
+        return failed;
     }
     // Single-site mode (cwd is a site folder, or --dir points at one)
     const spinner = ora('Reading local files...').start();
     try {
         await push_single_site(root_dir, effective_options, spinner);
+        return [];
     }
     catch (error) {
         spinner.fail(`Push failed: ${error instanceof Error ? error.message : error}`);
@@ -166,6 +178,9 @@ async function print_push_dry_run(root_dir, has_site_yaml, has_server_yaml, opti
     console.log(chalk.dim('  No requests sent. Run without --dry-run to push.'));
     console.log('');
 }
+// Pushes every site folder plus the library, continuing past individual
+// failures. Returns the labels that failed — the caller decides how loudly a
+// partial push should fail.
 async function push_server(root_dir, options) {
     // Sites live under sites/<slug>/
     const sites_root = path.join(root_dir, 'sites');
@@ -204,9 +219,10 @@ async function push_server(root_dir, options) {
                 print_auth_hint();
             process.exit(1);
         }
-        return;
+        return [];
     }
     let saw_auth_error = false;
+    const failed = [];
     // Push each site
     for (const site_dir of site_dirs) {
         const spinner = ora(`Pushing ${chalk.cyan(path.basename(site_dir))}...`).start();
@@ -217,6 +233,7 @@ async function push_server(root_dir, options) {
             spinner.fail(`${path.basename(site_dir)}: ${error instanceof Error ? error.message : error}`);
             if (is_auth_error(error))
                 saw_auth_error = true;
+            failed.push(path.basename(site_dir));
             // Continue to remaining sites rather than abort the whole push
         }
     }
@@ -231,10 +248,12 @@ async function push_server(root_dir, options) {
             spinner.fail(`library: ${error instanceof Error ? error.message : error}`);
             if (is_auth_error(error))
                 saw_auth_error = true;
+            failed.push('library');
         }
     }
     if (saw_auth_error)
         print_auth_hint();
+    return failed;
 }
 async function push_single_site(site_dir, options, spinner) {
     let config = null;

package/dist/index.js

@@ -95,7 +95,7 @@ ${chalk.bold('See also')}
   primo deploy  Stand up a new hosted Primo server
   primo login   Authenticate with a hosted Primo server
 `)
-    .action((server, options) => push_site({ ...options, server: server || options.server }));
+    .action(async (server, options) => { await push_site({ ...options, server: server || options.server }); });
 program
     .command('pull [server] [dir]')
     .description('Pull entire server (all sites + library) to local files (defaults to ./<server-hostname>)')
@@ -169,4 +169,13 @@ program.on('command:*', (operands) => {
     console.error('');
     process.exit(1);
 });
-program.parse();
+// parseAsync so async action handlers are awaited inside commander's
+// lifecycle — with plain parse() a rejected handler becomes an unhandled
+// rejection (ugly stack, engine-dependent exit) instead of the clean
+// message + exit(1) below.
+program.parseAsync().catch((error) => {
+    console.error('');
+    console.error(chalk.red(error instanceof Error ? error.message : String(error)));
+    console.error('');
+    process.exit(1);
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "primo-cli",
-  "version": "0.1.19",
+  "version": "0.1.20",
   "description": "Local development CLI for Primo",
   "type": "module",
   "bin": {