@pixelbyte-software/pixcode 1.54.7 → 1.54.9

package/dist-server/server/daemon-manager.js

@@ -221,22 +221,22 @@ function buildDaemonServiceUnit({ appRoot, serverPort, databasePath, nodeExecPat
         nodeExecPath,
         cliEntry,
     });
-    return `[Unit]
-Description=Pixcode Server
-After=network.target
-
-[Service]
-Type=simple
-WorkingDirectory=${appRoot}
-ExecStart=${execStart}
-Environment=HOST=0.0.0.0
-Environment=CI=true
-Environment=PIXCODE_DAEMON_MANAGED=1
-Restart=always
-RestartSec=2
-
-[Install]
-WantedBy=default.target
+    return `[Unit]
+Description=Pixcode Server
+After=network.target
+
+[Service]
+Type=simple
+WorkingDirectory=${appRoot}
+ExecStart=${execStart}
+Environment=HOST=0.0.0.0
+Environment=CI=true
+Environment=PIXCODE_DAEMON_MANAGED=1
+Restart=always
+RestartSec=2
+
+[Install]
+WantedBy=default.target
 `;
 }
 function resolveFrontendDaemonEntry(appRoot, cliEntry) {
@@ -277,22 +277,22 @@ function buildFrontendDaemonServiceUnit({ appRoot, frontendPort, nodeExecPath, c
         nodeExecPath,
         cliEntry,
     });
-    return `[Unit]
-Description=Pixcode Frontend (Vite Dev Server)
-After=network.target
-
-[Service]
-Type=simple
-WorkingDirectory=${appRoot}
-ExecStart=${execStart}
-Environment=HOST=0.0.0.0
-Environment=CI=true
-Environment=PIXCODE_DAEMON_MANAGED=1
-Restart=always
-RestartSec=2
-
-[Install]
-WantedBy=default.target
+    return `[Unit]
+Description=Pixcode Frontend (Vite Dev Server)
+After=network.target
+
+[Service]
+Type=simple
+WorkingDirectory=${appRoot}
+ExecStart=${execStart}
+Environment=HOST=0.0.0.0
+Environment=CI=true
+Environment=PIXCODE_DAEMON_MANAGED=1
+Restart=always
+RestartSec=2
+
+[Install]
+WantedBy=default.target
 `;
 }
 function normalizeState(value) {
@@ -489,38 +489,38 @@ function showDaemonHelp(c, context = {}) {
     const statusExample = buildDaemonCliCommand({ subcommand: 'status', mode: 'system' }, context);
     const doctorExample = buildDaemonCliCommand({ subcommand: 'doctor', mode: 'auto' }, context);
     const logsExample = buildDaemonCliCommand({ subcommand: 'logs', mode: 'system' }, context);
-    console.log(`
-${c.bright('Pixcode Daemon')} - Persistent Linux service manager
-
-Usage:
-  ${usagePrefix} daemon [subcommand] [options]
-
-Subcommands:
-  ${c.bright('install')}      Install/update unit, reload daemon, enable and start now
-  ${c.bright('start')}        Start the service
-  ${c.bright('stop')}         Stop the service (temporary; auto-start remains enabled)
-  ${c.bright('restart')}      Restart the service
-  ${c.bright('status')}       Show active/enabled state
-  ${c.bright('logs')}         Show recent service logs
-  ${c.bright('enable')}       Enable auto-start at boot
-  ${c.bright('disable')}      Disable auto-start at boot
-  ${c.bright('uninstall')}    Stop, disable, and remove the service unit
-  ${c.bright('doctor')}       Run diagnostics (bus, linger, units, state, port, logs)
-  ${c.bright('help')}         Show this help
-
-Options:
-  -p, --port <port>           Set service server port (default: 3001)
-  --frontend-port <port>      Start a separate frontend Vite service on this port (legacy/dev)
-  --single-port, --no-frontend
-                              Serve the built UI from the backend port only (default)
-  -m, --mode <mode>           Service mode: user | system | auto (default: system)
-  --database-path <path>      Set service database path
-
-Examples:
-  $ ${installExample}
-  $ ${statusExample}
-  $ ${doctorExample}
-  $ ${logsExample}
+    console.log(`
+${c.bright('Pixcode Daemon')} - Persistent Linux service manager
+
+Usage:
+  ${usagePrefix} daemon [subcommand] [options]
+
+Subcommands:
+  ${c.bright('install')}      Install/update unit, reload daemon, enable and start now
+  ${c.bright('start')}        Start the service
+  ${c.bright('stop')}         Stop the service (temporary; auto-start remains enabled)
+  ${c.bright('restart')}      Restart the service
+  ${c.bright('status')}       Show active/enabled state
+  ${c.bright('logs')}         Show recent service logs
+  ${c.bright('enable')}       Enable auto-start at boot
+  ${c.bright('disable')}      Disable auto-start at boot
+  ${c.bright('uninstall')}    Stop, disable, and remove the service unit
+  ${c.bright('doctor')}       Run diagnostics (bus, linger, units, state, port, logs)
+  ${c.bright('help')}         Show this help
+
+Options:
+  -p, --port <port>           Set service server port (default: 3001)
+  --frontend-port <port>      Start a separate frontend Vite service on this port (legacy/dev)
+  --single-port, --no-frontend
+                              Serve the built UI from the backend port only (default)
+  -m, --mode <mode>           Service mode: user | system | auto (default: system)
+  --database-path <path>      Set service database path
+
+Examples:
+  $ ${installExample}
+  $ ${statusExample}
+  $ ${doctorExample}
+  $ ${logsExample}
 `);
 }
 export function hasInstalledDaemonUnit() {

package/dist-server/server/index.js

@@ -453,10 +453,20 @@ async function runRuntimeDirUpdateJob(job, runtimeDir, latestVersion, tarballUrl
     });
     return latestVersion;
 }
+function isNodeVersionSupported() {
+    const match = process.version?.match(/^v(\d+)\./);
+    if (!match)
+        return true; // can't detect, assume ok
+    return parseInt(match[1], 10) >= 20;
+}
 async function runCommandUpdateJob(job, updateCommand, updateCwd) {
-    appendUpdateJobLog(job, 'meta', `Running: ${updateCommand}\n`);
-    await new Promise((resolve, reject) => {
-        const child = spawn(updateCommand, {
+    if (!isNodeVersionSupported()) {
+        const msg = `Node.js ${process.version} is too old. Pixcode requires Node.js 20+. Please upgrade Node.js first.\n`;
+        appendUpdateJobLog(job, 'stderr', msg);
+        throw new Error(msg.trim());
+    }
+    const runCommand = (cmd) => new Promise((resolve, reject) => {
+        const child = spawn(cmd, {
             cwd: updateCwd,
             env: process.env,
             shell: true,
@@ -468,15 +478,43 @@ async function runCommandUpdateJob(job, updateCommand, updateCwd) {
         }
         catch { /* noop */ }
         child.stdout?.on('data', (data) => appendUpdateJobLog(job, 'stdout', data.toString()));
-        child.stderr?.on('data', (data) => appendUpdateJobLog(job, 'stderr', data.toString()));
+        let stderrBuf = '';
+        child.stderr?.on('data', (data) => {
+            const chunk = data.toString();
+            stderrBuf += chunk;
+            appendUpdateJobLog(job, 'stderr', chunk);
+        });
         child.on('error', reject);
         child.on('close', (code) => {
             if (code === 0)
                 resolve();
             else
-                reject(new Error(`Update command exited with code ${code}`));
+                reject({ code, stderr: stderrBuf });
         });
     });
+    appendUpdateJobLog(job, 'meta', `Running: ${updateCommand}\n`);
+    try {
+        await runCommand(updateCommand);
+    }
+    catch (err) {
+        // On Linux/macOS, retry with sudo if the initial attempt failed with EACCES.
+        // npm install -g needs write access to /usr/local/lib/node_modules/,
+        // which the running user may not have.
+        if (process.platform !== 'win32' && (err.stderr?.includes('EACCES') || err.stderr?.includes('permission denied'))) {
+            const sudoCmd = `sudo -n ${updateCommand}`;
+            appendUpdateJobLog(job, 'meta', `Permission denied. Retrying with sudo...\n`);
+            try {
+                await runCommand(sudoCmd);
+                return;
+            }
+            catch (sudoErr) {
+                appendUpdateJobLog(job, 'stderr', `Sudo update also failed: ${sudoErr.stderr || sudoErr.code}\n`);
+                appendUpdateJobLog(job, 'meta', 'Tip: Run the update manually with: sudo npm install -g @pixelbyte-software/pixcode@latest\n');
+                throw new Error(`Update command exited with code ${err.code}`);
+            }
+        }
+        throw new Error(`Update command exited with code ${err.code}`);
+    }
 }
 function readCurrentPackageVersion() {
     try {
@@ -1070,6 +1108,10 @@ const FILE_TREE_MAX_ITEMS = Number.parseInt(process.env.PIXCODE_FILE_TREE_MAX_IT
 const FILE_TREE_MAX_DIRECTORIES = Number.parseInt(process.env.PIXCODE_FILE_TREE_MAX_DIRECTORIES || '', 10) || 1200;
 const FILE_TREE_SCAN_MAX_MS = Number.parseInt(process.env.PIXCODE_FILE_TREE_SCAN_MAX_MS || '', 10) || 4000;
 const FILE_TREE_MAX_ENTRIES_PER_DIRECTORY = Number.parseInt(process.env.PIXCODE_FILE_TREE_MAX_ENTRIES_PER_DIRECTORY || '', 10) || 1500;
+const FILE_TREE_MAX_DEPTH_DEFAULT = 5; // was 10, reduced to avoid deep traversal on huge projects
+const FILE_TREE_CACHE_TTL_MS = 3000; // short-lived cache for rapid successive requests
+const FILE_TREE_CACHE_MAX_ENTRIES = 50;
+const fileTreeCache = new Map();
 const FILE_TREE_EXCLUDED_ENTRY_NAMES = new Set([
     '.cache',
     '.git',
@@ -1606,6 +1648,7 @@ app.get('/health', (req, res) => {
         timestamp: new Date().toISOString(),
         installMode,
         version: SERVER_VERSION,
+        nodeVersion: process.version,
         pendingRestart: updateState.pendingRestart,
         lastAppliedUpdate: updateState.lastAppliedUpdate,
     });
@@ -2319,6 +2362,14 @@ app.post('/api/system/update', authenticateToken, requireAdmin, requireApiScope(
             console.warn('[update] Registry precheck failed:', (err && err.message) || err);
         }
     }
+    // Check Node.js version before starting update
+    if (!isNodeVersionSupported()) {
+        send('log', { stream: 'stderr', chunk: `Node.js ${process.version} is too old. Pixcode requires Node.js 20+.\n` });
+        send('log', { stream: 'meta', chunk: 'Please upgrade Node.js and try again.\n' });
+        send('done', { success: false, error: `Node.js ${process.version} is too old. Pixcode requires Node.js 20+. Please upgrade Node.js first.` });
+        endStream();
+        return;
+    }
     send('log', { stream: 'meta', chunk: `Running: ${updateCommandLabel}\n` });
     // Cross-platform shell invocation. `detached: true` + `unref()` below
     // means the install child survives if this server process gets killed
@@ -2327,7 +2378,7 @@ app.post('/api/system/update', authenticateToken, requireAdmin, requireApiScope(
     // can segfault or the supervisor kills it). Without detachment, a
     // killed parent tears down the npm child too and users end up with
     // a half-installed package and no server at all.
-    const child = spawn(updateCommand, {
+    let child = spawn(updateCommand, {
         cwd: updateCwd,
         env: process.env,
         shell: true,
@@ -2353,7 +2404,9 @@ app.post('/api/system/update', authenticateToken, requireAdmin, requireApiScope(
     child.stdout?.on('data', (data) => {
         send('log', { stream: 'stdout', chunk: data.toString() });
     });
+    let stderrBuf = '';
     child.stderr?.on('data', (data) => {
+        stderrBuf += data.toString();
         send('log', { stream: 'stderr', chunk: data.toString() });
     });
     child.on('error', (error) => {
@@ -2363,7 +2416,7 @@ app.post('/api/system/update', authenticateToken, requireAdmin, requireApiScope(
         send('done', { success: false, error: error.message });
         endStream();
     });
-    child.on('close', (code) => {
+    child.on('close', async (code) => {
         if (ended)
             return;
         if (clientAborted) {
@@ -2376,13 +2429,61 @@ app.post('/api/system/update', authenticateToken, requireAdmin, requireApiScope(
                 version: SERVER_VERSION,
                 message: 'Update completed. Restart the server to apply changes.',
             });
+            endStream();
+            return;
         }
-        else {
-            send('done', {
-                success: false,
-                error: `Update command exited with code ${code}`,
-            });
+        // On Linux/macOS, retry with sudo on EACCES (npm install -g permission error)
+        if (code !== 0 && process.platform !== 'win32' && (stderrBuf.includes('EACCES') || stderrBuf.includes('permission denied'))) {
+            const sudoCmd = `sudo -n ${updateCommand}`;
+            send('log', { stream: 'meta', chunk: 'Permission denied. Retrying with sudo...\n' });
+            try {
+                await new Promise((resolveSudo, rejectSudo) => {
+                    let sudoEnded = false;
+                    const sudoChild = spawn(sudoCmd, {
+                        cwd: updateCwd,
+                        env: process.env,
+                        shell: true,
+                        detached: true,
+                        stdio: ['ignore', 'pipe', 'pipe'],
+                    });
+                    try {
+                        sudoChild.unref();
+                    }
+                    catch { /* noop */ }
+                    sudoChild.stdout?.on('data', (d) => send('log', { stream: 'stdout', chunk: d.toString() }));
+                    let sudoStderr = '';
+                    sudoChild.stderr?.on('data', (d) => { sudoStderr += d.toString(); send('log', { stream: 'stderr', chunk: d.toString() }); });
+                    sudoChild.on('error', (e) => { if (!sudoEnded) {
+                        sudoEnded = true;
+                        rejectSudo(e);
+                    } });
+                    sudoChild.on('close', (sudoCode) => {
+                        if (sudoEnded)
+                            return;
+                        sudoEnded = true;
+                        if (sudoCode === 0)
+                            resolveSudo();
+                        else
+                            rejectSudo(new Error(`Sudo update exited with code ${sudoCode}`));
+                    });
+                });
+                send('done', {
+                    success: true,
+                    version: SERVER_VERSION,
+                    message: 'Update completed (with sudo). Restart the server to apply changes.',
+                });
+                endStream();
+                return;
+            }
+            catch (sudoErr) {
+                send('log', { stream: 'stderr', chunk: `Sudo update failed: ${sudoErr.message}\n` });
+                send('log', { stream: 'meta', chunk: 'Tip: Run the update manually with: sudo npm install -g @pixelbyte-software/pixcode@latest\n' });
+            }
         }
+        send('done', {
+            success: false,
+            error: `Update command exited with code ${code}`,
+        });
         endStream();
     });
 });
@@ -2867,7 +2968,7 @@ app.get('/api/projects/:projectName/files', authenticateToken, requireProjectAcc
         catch (e) {
             return res.status(404).json({ error: `Project path not found: ${actualPath}` });
         }
-        const files = await getFileTree(actualPath, 10, 0, true);
+        const files = await getFileTree(actualPath, FILE_TREE_MAX_DEPTH_DEFAULT, 0, true);
         res.json(filterFileTreeForUser(files, req.user, {
             name: req.params.projectName,
             projectName: req.params.projectName,
@@ -4697,6 +4798,26 @@ function permToRwx(perm) {
     const x = perm & 1 ? 'x' : '-';
     return r + w + x;
 }
+// File tree result cache (keyed by path:depth:showHidden, TTL 3s)
+function readFileTreeCache(cacheKey) {
+    const entry = fileTreeCache.get(cacheKey);
+    if (!entry || entry.expiresAt <= Date.now()) {
+        if (entry)
+            fileTreeCache.delete(cacheKey);
+        // Prune stale entries
+        if (fileTreeCache.size > FILE_TREE_CACHE_MAX_ENTRIES) {
+            const keys = [...fileTreeCache.keys()];
+            for (const k of keys.slice(0, keys.length - FILE_TREE_CACHE_MAX_ENTRIES)) {
+                fileTreeCache.delete(k);
+            }
+        }
+        return undefined;
+    }
+    return entry.tree;
+}
+function writeFileTreeCache(cacheKey, tree) {
+    fileTreeCache.set(cacheKey, { tree, expiresAt: Date.now() + FILE_TREE_CACHE_TTL_MS });
+}
 function createFileTreeScanContext() {
     return {
         startedAt: Date.now(),
@@ -4752,6 +4873,13 @@ async function readDirectoryEntriesBounded(dirPath, context) {
     return entries;
 }
 async function getFileTree(dirPath, maxDepth = 3, currentDepth = 0, showHidden = true, scanContext = null) {
+    // Check cache on top-level call
+    if (currentDepth === 0 && !scanContext) {
+        const cacheKey = `${path.resolve(dirPath)}:${maxDepth}:${showHidden}`;
+        const cached = readFileTreeCache(cacheKey);
+        if (cached)
+            return cached;
+    }
     const context = scanContext || createFileTreeScanContext();
     const items = [];
     if (!hasFileTreeBudget(context)) {
@@ -4798,12 +4926,18 @@ async function getFileTree(dirPath, maxDepth = 3, currentDepth = 0, showHidden =
         }
         items.push(item);
     }
-    return items.sort((a, b) => {
+    const sorted = items.sort((a, b) => {
         if (a.type !== b.type) {
             return a.type === 'directory' ? -1 : 1;
         }
         return a.name.localeCompare(b.name);
     });
+    // Store in cache on top-level call
+    if (currentDepth === 0 && !scanContext) {
+        const cacheKey = `${path.resolve(dirPath)}:${maxDepth}:${showHidden}`;
+        writeFileTreeCache(cacheKey, sorted);
+    }
+    return sorted;
 }
 const SERVER_PORT = process.env.SERVER_PORT || 3001;
 const HOST = process.env.HOST || '0.0.0.0';