nstantpage-agent 0.5.3 → 0.5.4

package/dist/cli.js

@@ -25,7 +25,7 @@ const program = new Command();
 program
     .name('nstantpage')
     .description('Local development agent for nstantpage.com — run projects on your machine, preview in the cloud')
-    .version('0.5.3');
+    .version('0.5.4');
 program
     .command('login')
     .description('Authenticate with nstantpage.com')

package/dist/commands/start.js

@@ -24,7 +24,7 @@ import { getConfig, getProjectConfig, setProjectConfig, clearProjectConfig, getD
 import { TunnelClient } from '../tunnel.js';
 import { LocalServer } from '../localServer.js';
 import { PackageInstaller } from '../packageInstaller.js';
-const VERSION = '0.5.3';
+const VERSION = '0.5.4';
 /**
  * Resolve the backend API base URL.
  * - If --backend is passed, use it
@@ -226,20 +226,20 @@ export async function startCommand(directory, options) {
     console.log(chalk.gray(`  Gateway:      ${options.gateway}`));
     console.log(chalk.gray(`  Backend:      ${backendUrl}\n`));
     // 1. Fetch project files from the backend
+    const installer = new PackageInstaller({ projectDir });
     try {
         const { fileCount, isNew } = await fetchProjectFiles(backendUrl, projectId, projectDir, token);
-        // 2. Install dependencies if this is a new download or node_modules is missing
-        const hasNodeModules = fs.existsSync(path.join(projectDir, 'node_modules'));
-        if (isNew || !hasNodeModules) {
+        // 2. Install dependencies if needed (verifies actual packages, not just folder)
+        if (!installer.areDependenciesInstalled()) {
             if (fs.existsSync(path.join(projectDir, 'package.json'))) {
                 console.log(chalk.gray('  Installing dependencies...'));
-                const installer = new PackageInstaller({ projectDir });
-                const result = await installer.install([], false);
-                if (result.success) {
+                try {
+                    await installer.ensureDependencies();
                     console.log(chalk.green(`  ✓ Dependencies installed`));
                 }
-                else {
-                    console.log(chalk.yellow(`  ⚠ Dependency installation had issues: ${result.output.slice(0, 200)}`));
+                catch (err) {
+                    console.log(chalk.yellow(`  ⚠ Install failed: ${err.message?.slice(0, 200)}`));
+                    console.log(chalk.gray(`    Will retry when dev server starts`));
                 }
             }
         }
@@ -376,15 +376,21 @@ export async function startCommand(directory, options) {
         console.log(chalk.green(`  ✓ API server on port ${apiPort}`));
         // Start dev server unless --no-dev flag
         if (!options.noDev) {
-            console.log(chalk.gray('  Starting dev server...'));
-            try {
-                const devServer = localServer.getDevServer();
-                await devServer.start();
-                console.log(chalk.green(`  ✓ Dev server on port ${devPort}`));
+            if (installer.areDependenciesInstalled()) {
+                console.log(chalk.gray('  Starting dev server...'));
+                try {
+                    const devServer = localServer.getDevServer();
+                    await devServer.start();
+                    console.log(chalk.green(`  ✓ Dev server on port ${devPort}`));
+                }
+                catch (err) {
+                    console.log(chalk.yellow(`  ⚠ Dev server failed to start: ${err.message}`));
+                    console.log(chalk.gray('    You can start it later from the editor'));
+                }
             }
-            catch (err) {
-                console.log(chalk.yellow(`  ⚠ Dev server failed to start: ${err.message}`));
-                console.log(chalk.gray('    You can start it later from the editor'));
+            else {
+                console.log(chalk.yellow(`  ⚠ Skipping dev server — dependencies not fully installed`));
+                console.log(chalk.gray('    Dev server will start when browser opens the project'));
             }
         }
         else {
@@ -557,14 +563,18 @@ async function startAdditionalProject(projectId, opts) {
         catch (err) {
             console.log(chalk.yellow(`  ⚠ Could not fetch files: ${err.message}`));
         }
-        // Install dependencies
-        const hasNodeModules = fs.existsSync(path.join(projectDir, 'node_modules'));
-        if (!hasNodeModules && fs.existsSync(path.join(projectDir, 'package.json'))) {
+        // Install dependencies (must complete before dev server can start)
+        const installer = new PackageInstaller({ projectDir });
+        if (!installer.areDependenciesInstalled() && fs.existsSync(path.join(projectDir, 'package.json'))) {
             console.log(chalk.gray(`  Installing dependencies...`));
-            const installer = new PackageInstaller({ projectDir });
-            const result = await installer.install([], false);
-            if (result.success)
+            try {
+                await installer.ensureDependencies();
                 console.log(chalk.green(`  ✓ Dependencies installed`));
+            }
+            catch (err) {
+                console.log(chalk.red(`  ✗ Install failed: ${err.message}`));
+                console.log(chalk.yellow(`  ⚠ Dev server may fail — retrying install on first request`));
+            }
         }
         // Start local server
         const localServer = new LocalServer({
@@ -573,14 +583,20 @@ async function startAdditionalProject(projectId, opts) {
         });
         await localServer.start();
         console.log(chalk.green(`  ✓ API server on port ${allocated.apiPort}`));
-        // Start dev server
+        // Start dev server (only if dependencies are installed)
         if (!opts.noDev) {
-            try {
-                await localServer.getDevServer().start();
-                console.log(chalk.green(`  ✓ Dev server on port ${allocated.devPort}`));
+            if (installer.areDependenciesInstalled()) {
+                try {
+                    await localServer.getDevServer().start();
+                    console.log(chalk.green(`  ✓ Dev server on port ${allocated.devPort}`));
+                }
+                catch (err) {
+                    console.log(chalk.yellow(`  ⚠ Dev server: ${err.message}`));
+                }
             }
-            catch (err) {
-                console.log(chalk.yellow(`  ⚠ Dev server: ${err.message}`));
+            else {
+                console.log(chalk.yellow(`  ⚠ Skipping dev server — dependencies not installed`));
+                console.log(chalk.gray(`    Dev server will start when browser opens the project`));
             }
         }
         // Connect project tunnel

package/dist/localServer.js

@@ -559,7 +559,7 @@ export class LocalServer {
             connected: true,
             projectId: this.options.projectId,
             agent: {
-                version: '0.5.3',
+                version: '0.5.4',
                 hostname: os.hostname(),
                 platform: `${os.platform()} ${os.arch()}`,
             },

package/dist/packageInstaller.d.ts

@@ -7,6 +7,8 @@ export interface PackageInstallerOptions {
 }
 export declare class PackageInstaller {
     private projectDir;
+    /** Serializes concurrent install calls */
+    private installLock;
     constructor(options: PackageInstallerOptions);
     /**
      * Install packages into the project.
@@ -18,8 +20,14 @@ export declare class PackageInstaller {
     }>;
     /**
      * Ensure all project dependencies are installed.
+     * Uses a lock to prevent concurrent installs.
      */
     ensureDependencies(): Promise<void>;
+    /**
+     * Check if dependencies are actually installed (not just that node_modules/ exists).
+     * Verifies that at least one key dependency from package.json is present.
+     */
+    areDependenciesInstalled(): boolean;
     /**
      * Detect which package manager is used (pnpm, yarn, npm).
      */

package/dist/packageInstaller.js

@@ -4,9 +4,11 @@
  */
 import { spawn } from 'child_process';
 import path from 'path';
-import { existsSync } from 'fs';
+import fs, { existsSync } from 'fs';
 export class PackageInstaller {
     projectDir;
+    /** Serializes concurrent install calls */
+    installLock = Promise.resolve();
     constructor(options) {
         this.projectDir = options.projectDir;
     }
@@ -18,7 +20,7 @@ export class PackageInstaller {
         const args = this.buildInstallArgs(pm, packages, dev);
         console.log(`  [Installer] ${pm} ${args.join(' ')}`);
         try {
-            const timeout = packages.length === 0 ? 120_000 : 60_000;
+            const timeout = packages.length === 0 ? 300_000 : 120_000;
             const output = await this.runCommand(pm, args, timeout);
             return {
                 success: true,
@@ -36,16 +38,51 @@ export class PackageInstaller {
     }
     /**
      * Ensure all project dependencies are installed.
+     * Uses a lock to prevent concurrent installs.
      */
     async ensureDependencies() {
-        const nodeModules = path.join(this.projectDir, 'node_modules');
-        if (existsSync(nodeModules))
+        // Serialize with any in-flight install
+        await this.installLock;
+        if (this.areDependenciesInstalled())
             return;
-        console.log(`  [Installer] Installing project dependencies...`);
-        const pm = this.detectPackageManager();
-        const args = pm === 'pnpm' ? ['install'] : ['install'];
-        await this.runCommand(pm, args, 120_000);
-        console.log(`  [Installer] Dependencies installed`);
+        // Acquire lock for this install
+        let releaseLock;
+        this.installLock = new Promise(resolve => { releaseLock = resolve; });
+        try {
+            console.log(`  [Installer] Installing project dependencies...`);
+            const pm = this.detectPackageManager();
+            const args = pm === 'pnpm' ? ['install'] : ['install'];
+            await this.runCommand(pm, args, 300_000);
+            console.log(`  [Installer] Dependencies installed`);
+        }
+        finally {
+            releaseLock();
+        }
+    }
+    /**
+     * Check if dependencies are actually installed (not just that node_modules/ exists).
+     * Verifies that at least one key dependency from package.json is present.
+     */
+    areDependenciesInstalled() {
+        const nodeModules = path.join(this.projectDir, 'node_modules');
+        if (!existsSync(nodeModules))
+            return false;
+        // Check that at least one real dependency is installed
+        try {
+            const pkgPath = path.join(this.projectDir, 'package.json');
+            if (!existsSync(pkgPath))
+                return true; // no package.json → nothing to install
+            const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf-8'));
+            const deps = Object.keys(pkg.dependencies || {});
+            if (deps.length === 0)
+                return true; // no deps → nothing needed
+            // Check if the first listed dependency has its folder in node_modules
+            const firstDep = deps[0];
+            return existsSync(path.join(nodeModules, firstDep));
+        }
+        catch {
+            return existsSync(nodeModules);
+        }
     }
     /**
      * Detect which package manager is used (pnpm, yarn, npm).

package/dist/tunnel.js

@@ -63,7 +63,7 @@ export class TunnelClient {
                 // Send enhanced agent info with capabilities and deviceId
                 this.send({
                     type: 'agent-info',
-                    version: '0.5.3',
+                    version: '0.5.4',
                     hostname: os.hostname(),
                     platform: `${os.platform()} ${os.arch()}`,
                     deviceId: getDeviceId(),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nstantpage-agent",
-  "version": "0.5.3",
+  "version": "0.5.4",
   "description": "Local development agent for nstantpage.com — run your projects locally, preview in the cloud. Replaces cloud containers for faster builds.",
   "type": "module",
   "bin": {