@pixelbyte-software/pixcode 1.47.2 → 1.47.4

package/scripts/smoke/git-install-update.mjs

@@ -21,10 +21,28 @@ assert.doesNotMatch(
   'Server update command should not use the brittle raw git checkout/pull/install chain.',
 );
 
+assert.match(
+  serverIndex,
+  /updateCommandLabel[\s\S]*Pixcode source update/,
+  'Server update stream should describe git installs with product language instead of an internal script command.',
+);
+
 assert.match(
+  modal,
+  /versionUpdate\.pixcodeUpgradeCommand/,
+  'Version modal should show the user-facing Pixcode update command.',
+);
+
+assert.doesNotMatch(
   modal,
   /node scripts\/update-git-install\.mjs/,
-  'Version modal should show the safe git updater command.',
+  'Version modal should not expose the internal git updater script as manual product guidance.',
+);
+
+assert.match(
+  fs.readFileSync('server/cli.js', 'utf8'),
+  /update-git-install\.mjs[\s\S]*installMode === 'git'[\s\S]*updateGitPackage/,
+  'pixcode update should drive the safe git updater for source installs.',
 );
 
 assert.match(
@@ -51,6 +69,12 @@ assert.match(
   'Safe updater should reinstall dependencies after updating source files.',
 );
 
+assert.match(
+  updater,
+  /npm[\s\S]*run[\s\S]*build/,
+  'Safe updater should rebuild source installs after updating source files.',
+);
+
 const tempRoot = fs.mkdtempSync(path.join(os.tmpdir(), 'pixcode-git-update-'));
 const origin = path.join(tempRoot, 'origin.git');
 const source = path.join(tempRoot, 'source');
@@ -81,7 +105,13 @@ function run(command, args, cwd) {
 function writePackage(version) {
   fs.writeFileSync(
     path.join(source, 'package.json'),
-    JSON.stringify({ name: 'pixcode-update-smoke', version }, null, 2),
+    JSON.stringify({
+      name: 'pixcode-update-smoke',
+      version,
+      scripts: {
+        build: 'node -e "require(\\"node:fs\\").writeFileSync(\\"built.txt\\", \\"built\\")"',
+      },
+    }, null, 2),
   );
   fs.writeFileSync(
     path.join(source, 'package-lock.json'),
@@ -137,5 +167,10 @@ assert.match(
   /pixcode-auto-update-/,
   'Safe updater should leave local dirty files recoverable in git stash.',
 );
+assert.equal(
+  fs.readFileSync(path.join(install, 'built.txt'), 'utf8'),
+  'built',
+  'Safe updater should run the repository build after installing dependencies.',
+);
 
 console.log('git install update smoke passed');

package/scripts/update-git-install.mjs

@@ -119,6 +119,13 @@ async function main() {
   log(`Repository updated to Pixcode ${packageVersion}.`);
 
   await run('npm', ['install', '--no-audit', '--no-fund']);
+  const updatedPackageJson = JSON.parse(fs.readFileSync(path.join(repoRoot, 'package.json'), 'utf8'));
+  if (updatedPackageJson.scripts?.build) {
+    log('Building Pixcode source install.');
+    await run('npm', ['run', 'build']);
+  } else {
+    log('No build script found; skipping build.');
+  }
   log('Pixcode git install update completed.');
 }
 

package/server/cli.js

@@ -249,10 +249,77 @@ async function checkForUpdates(silent = false) {
     }
 }
 
+function runInherited(command, args, options = {}) {
+    return new Promise((resolve, reject) => {
+        const child = spawn(command, args, {
+            cwd: options.cwd || APP_ROOT,
+            env: options.env || process.env,
+            stdio: 'inherit',
+            shell: false,
+            windowsHide: false,
+        });
+
+        child.on('error', reject);
+        child.on('close', (code) => {
+            if (code === 0) {
+                resolve();
+                return;
+            }
+
+            reject(new Error(`${command} ${args.join(' ')} exited with code ${code}`));
+        });
+    });
+}
+
+async function maybeRestartDaemonAfterUpdate(options = {}) {
+    if (options.restartDaemon) {
+        if (!hasInstalledDaemonUnit()) {
+            console.log(`${c.warn('[WARN]')} No daemon unit detected; skipping restart.`);
+            return;
+        }
+        console.log(`${c.info('[INFO]')} Restarting daemon service...`);
+        await handleDaemonCommand(['restart', '--mode=system'], {
+            appRoot: APP_ROOT,
+            defaultPort: process.env.SERVER_PORT || process.env.PORT || '3001',
+            color: c,
+        });
+        console.log(`${c.ok('[OK]')} Daemon restart completed.`);
+        return;
+    }
+
+    if (hasInstalledDaemonUnit()) {
+        const restartCommand = buildDaemonCliCommand(
+            { subcommand: 'restart', mode: 'system' },
+            DAEMON_COMMAND_CONTEXT
+        );
+        console.log(`${c.tip('[TIP]')} Daemon unit detected. Restart to apply update: ${c.bright(restartCommand)}`);
+        console.log(`${c.tip('[TIP]')} Or update + restart in one step: ${c.bright('pixcode update --restart-daemon')}`);
+    } else {
+        console.log(`${c.tip('[TIP]')} Restart pixcode to use the new version.`);
+    }
+}
+
+async function updateGitPackage(options = {}) {
+    const gitUpdateScript = path.join(APP_ROOT, 'scripts', 'update-git-install.mjs');
+    if (!fs.existsSync(gitUpdateScript)) {
+        throw new Error(`Git update script was not found: ${gitUpdateScript}`);
+    }
+
+    console.log(`${c.info('[INFO]')} Updating Pixcode source checkout...`);
+    await runInherited(process.execPath, [gitUpdateScript], { cwd: APP_ROOT });
+    console.log(`${c.ok('[OK]')} Source update complete!`);
+    await maybeRestartDaemonAfterUpdate(options);
+}
+
 // Update the package
 async function updatePackage(options = {}) {
     try {
         const { execSync } = await import('child_process');
+        if (installMode === 'git') {
+            await updateGitPackage(options);
+            return;
+        }
+
         console.log(`${c.info('[INFO]')} Checking for updates...`);
 
         const { hasUpdate, latestVersion, currentVersion } = await checkForUpdates(true);
@@ -265,32 +332,13 @@ async function updatePackage(options = {}) {
         console.log(`${c.info('[INFO]')} Updating from ${currentVersion} to ${latestVersion}...`);
         execSync('npm update -g @pixelbyte-software/pixcode', { stdio: 'inherit' });
         console.log(`${c.ok('[OK]')} Update complete!`);
-
-        if (options.restartDaemon) {
-            if (!hasInstalledDaemonUnit()) {
-                console.log(`${c.warn('[WARN]')} No daemon unit detected; skipping restart.`);
-                return;
-            }
-            console.log(`${c.info('[INFO]')} Restarting daemon service...`);
-            await handleDaemonCommand(['restart', '--mode=system'], {
-                appRoot: APP_ROOT,
-                defaultPort: process.env.SERVER_PORT || process.env.PORT || '3001',
-                color: c,
-            });
-            console.log(`${c.ok('[OK]')} Daemon restart completed.`);
-        } else if (hasInstalledDaemonUnit()) {
-            const restartCommand = buildDaemonCliCommand(
-                { subcommand: 'restart', mode: 'system' },
-                DAEMON_COMMAND_CONTEXT
-            );
-            console.log(`${c.tip('[TIP]')} Daemon unit detected. Restart to apply update: ${c.bright(restartCommand)}`);
-            console.log(`${c.tip('[TIP]')} Or update + restart in one step: ${c.bright('pixcode update --restart-daemon')}`);
-        } else {
-            console.log(`${c.tip('[TIP]')} Restart pixcode to use the new version.`);
-        }
+        await maybeRestartDaemonAfterUpdate(options);
     } catch (e) {
         console.error(`${c.error('[ERROR]')} Update failed: ${e.message}`);
-        console.log(`${c.tip('[TIP]')} Try running manually: npm update -g @pixelbyte-software/pixcode`);
+        const fallbackCommand = installMode === 'git'
+            ? 'pixcode update --restart-daemon'
+            : 'npm update -g @pixelbyte-software/pixcode';
+        console.log(`${c.tip('[TIP]')} Try running manually: ${fallbackCommand}`);
     }
 }
 

package/server/index.js

@@ -498,6 +498,11 @@ app.post('/api/system/update', authenticateToken, async (req, res) => {
         : installMode === 'git'
             ? `${JSON.stringify(process.execPath)} ${JSON.stringify(gitUpdateScript)}`
             : 'npm install -g @pixelbyte-software/pixcode@latest';
+    const updateCommandLabel = IS_PLATFORM
+        ? 'Pixcode platform update'
+        : installMode === 'git'
+            ? 'Pixcode source update'
+            : 'pixcode update';
 
     const updateCwd = IS_PLATFORM || installMode === 'git'
         ? projectRoot
@@ -770,7 +775,7 @@ app.post('/api/system/update', authenticateToken, async (req, res) => {
         }
     }
 
-    send('log', { stream: 'meta', chunk: `Running: ${updateCommand}\n` });
+    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