fix: improve binary download handling for npx usage

- Remove circular dependency from package.json
- Make doplan.js attempt to download binary if missing
- Improve error messages for missing binaries
- Allow postinstall to succeed in development/CI even if download fails
- Binary will be downloaded on first CLI execution if missing

Author: iDorgham

bin/doplan.js

@@ -21,10 +21,31 @@ if (platform === 'win32') {
 const binaryPath = path.join(binDir, binaryName);
 
 if (!fs.existsSync(binaryPath)) {
-  console.error('Error: DoPlan CLI binary not found.');
-  console.error(`Expected: ${binaryPath}`);
-  console.error('Please run: npm install');
-  process.exit(1);
+  console.log('Binary not found. Attempting to download...');
+  const downloadScript = path.join(__dirname, '..', 'scripts', 'download.js');
+  if (fs.existsSync(downloadScript)) {
+    try {
+      require(downloadScript).main();
+      // Check again after download attempt
+      if (!fs.existsSync(binaryPath)) {
+        console.error('Error: DoPlan CLI binary not found.');
+        console.error(`Expected: ${binaryPath}`);
+        console.error('Please ensure the GitHub release exists with binaries.');
+        console.error('Or build from source: https://github.com/DoPlan-dev/CLI');
+        process.exit(1);
+      }
+    } catch (error) {
+      console.error('Error downloading binary:', error.message);
+      console.error('Please ensure the GitHub release exists with binaries.');
+      console.error('Or build from source: https://github.com/DoPlan-dev/CLI');
+      process.exit(1);
+    }
+  } else {
+    console.error('Error: DoPlan CLI binary not found.');
+    console.error(`Expected: ${binaryPath}`);
+    console.error('Please run: npm install');
+    process.exit(1);
+  }
 }
 
 // Make executable on Unix systems

package-lock.json

@@ -28,4 +28,3 @@
     "scripts"
   ]
 }
-

package.json

@@ -121,15 +121,25 @@ async function main() {
 
     console.log('Installation complete!');
   } catch (error) {
-    // In CI environments or when release doesn't exist yet, don't fail
+    // In CI environments or development, don't fail
     const isCI = process.env.CI === 'true' || process.env.GITHUB_ACTIONS === 'true' || process.env.CI === '1';
-    if (isCI || error.message.includes('404')) {
-      console.warn('Warning: Could not download binary (this is normal in CI or before first release):', error.message);
-      console.warn('Binary will be downloaded on first use or when release is available.');
+    const isDev = process.env.NODE_ENV === 'development' || !process.env.npm_config_user_config;
+    
+    if (isCI || isDev || error.message.includes('404')) {
+      console.warn('Warning: Could not download binary:', error.message);
+      if (error.message.includes('404')) {
+        console.warn('The GitHub release may not exist yet or binaries are not available.');
+        console.warn('The binary will be downloaded automatically when you run the CLI.');
+        console.warn('Or check: https://github.com/DoPlan-dev/CLI/releases');
+      } else {
+        console.warn('Binary will be downloaded on first use or when release is available.');
+      }
       process.exit(0);
     }
+    // For production npm installs, fail so they know something is wrong
     console.error('Installation failed:', error.message);
-    console.error('This may be normal if the release is not yet available.');
+    console.error('Please check: https://github.com/DoPlan-dev/CLI/releases');
+    console.error('Or build from source: https://github.com/DoPlan-dev/CLI');
     process.exit(1);
   }
 }