@maravilla-labs/cli 0.1.7 → 0.1.9

package/README.md

@@ -16,6 +16,11 @@ pnpm add -g @maravilla-labs/cli
 yarn global add @maravilla-labs/cli
 ```
 
+**Note:** To see installation progress with npm 7+, use the `--foreground-scripts` flag:
+```bash
+npm install -g @maravilla-labs/cli --foreground-scripts
+```
+
 Verify:
 
 ```bash

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@maravilla-labs/cli",
-  "version": "0.1.7",
+  "version": "0.1.9",
   "description": "NPM wrapper for the Maravilla CLI binary; downloads the right release for your platform.",
   "license": "SEE LICENSE IN LICENSE",
   "author": "Maravilla Labs",

package/scripts/postinstall.js

@@ -18,8 +18,17 @@ if (process.env.MARAVILLA_CLI_SKIP_INSTALL === '1' || process.env.CI) {
   console.log('✿ Skipping Maravilla CLI binary install (CI or skip flag set)');
   process.exit(0);
 }
+
 import { spawn } from 'child_process';
 
+// Detect if output is being suppressed (npm 7+ behavior)
+// Write to stderr as a fallback since it's more likely to be shown
+const isOutputSuppressed = !process.stdout.isTTY && !process.env.npm_config_foreground_scripts;
+if (isOutputSuppressed) {
+  console.error('✿ Installing Maravilla CLI binary...');
+  console.error('✿ To see detailed progress, run: npm install --foreground-scripts');
+}
+
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 
@@ -195,6 +204,7 @@ async function install() {
   // Decide whether to download based on current vs requested version
   const force = process.env.MARAVILLA_FORCE_INSTALL === '1';
   let requested = null;
+  // Handle both "cli-v0.1.9" and "v0.1.9" formats
   const m = tag.match(/v(\d+\.\d+\.\d+)/);
   if (m) requested = m[1];
 
@@ -241,6 +251,14 @@ async function install() {
 
   const innerDir = path.join(tmpDir, art);
   const srcPath = path.join(innerDir, EXECUTABLE);
+  
+  // Remove existing binary first to ensure clean replacement
+  try {
+    await fs.promises.unlink(INSTALL_PATH);
+  } catch (e) {
+    // File might not exist, that's okay
+  }
+  
   await fs.promises.copyFile(srcPath, INSTALL_PATH);
   await fs.promises.chmod(INSTALL_PATH, 0o755);
   log(chalk.green(`Installed to ${INSTALL_PATH}`));
@@ -273,7 +291,13 @@ async function sha256File(filePath) {
   });
 }
 
-install().catch(err => {
-  console.error(chalk.red(`${flower()} install failed: ${err.message}`));
-  process.exitCode = 1;
-});
+install()
+  .then(() => {
+    if (isOutputSuppressed) {
+      console.error('✿ Maravilla CLI binary installed successfully!');
+    }
+  })
+  .catch(err => {
+    console.error(chalk.red(`${flower()} install failed: ${err.message}`));
+    process.exitCode = 1;
+  });

package/scripts/update.js

@@ -111,11 +111,27 @@ async function main() {
 		log(`Updating to ${chalk.cyan(tag)}`);
 	}
 
-		return await new Promise((resolve) => {
-			const env = { ...process.env, MARAVILLA_FORCE_INSTALL: '1' };
-			const child = spawn(process.execPath, [postinstall], { stdio: 'inherit', env });
+	// Force the specific version we want
+	const env = { 
+		...process.env, 
+		MARAVILLA_FORCE_INSTALL: '1',
+		MARAVILLA_VERSION: tag  // Pass the tag to ensure we get the right version
+	};
+	
+	const result = await new Promise((resolve) => {
+		const child = spawn(process.execPath, [postinstall], { stdio: 'inherit', env });
 		child.on('exit', (code) => resolve(code ?? 0));
 	});
+	
+	// Verify the update worked
+	if (result === 0) {
+		const newVersion = await currentVersion();
+		if (newVersion) {
+			log(chalk.green(`✅ Successfully updated to v${newVersion}`));
+		}
+	}
+	
+	return result;
 }
 
 main().then((code) => process.exit(code)).catch((e) => {