kryptheon 0.1.40 → 0.1.41

package/bin/kryptheon.js

@@ -2632,9 +2632,12 @@ function dashboardAlreadyRunning(port) {
       res.on('data', (chunk) => { if (body.length < 5000000) body += chunk; });
       res.on('end', () => {
         try {
-          const root = JSON.parse(body).project.root;
+          const state = JSON.parse(body);
+          const root = state.project.root;
           const same = (a) => (process.platform === 'win32' ? path.resolve(a).toLowerCase() : path.resolve(a));
-          resolve(res.statusCode === 200 && typeof root === 'string' && same(root) === same(USER_DIR));
+          if (res.statusCode !== 200 || typeof root !== 'string' || same(root) !== same(USER_DIR)) return resolve(false);
+          // Before 0.1.41 a dashboard did not say which it was: an old one.
+          resolve({ version: typeof state.kryptheon === 'string' ? state.kryptheon : null });
         } catch (err) {
           resolve(false);
         }
@@ -2645,6 +2648,49 @@ function dashboardAlreadyRunning(port) {
   });
 }
 
+/** The process listening on 127.0.0.1:port, or null. */
+function listenerOn(port) {
+  try {
+    if (process.platform === 'win32') {
+      const out = spawnSync('netstat', ['-ano', '-p', 'TCP'], { encoding: 'utf8', windowsHide: true, timeout: 10000 }).stdout || '';
+      for (const line of out.split(/\r?\n/)) {
+        const m = line.trim().split(/\s+/);
+        if (m.length >= 5 && m[1] === '127.0.0.1:' + port && m[3] === 'LISTENING' && /^\d+$/.test(m[4])) return Number(m[4]);
+      }
+      return null;
+    }
+    const out = spawnSync('lsof', ['-nP', '-iTCP@127.0.0.1:' + port, '-sTCP:LISTEN', '-t'], { encoding: 'utf8', timeout: 10000 }).stdout || '';
+    const pid = Number(out.trim().split(/\s+/)[0]);
+    return pid > 0 ? pid : null;
+  } catch (err) {
+    return null;
+  }
+}
+
+/**
+ * An older dashboard for this same folder, still running - from yesterday's
+ * Win+R, or started at sign-in - is stopped so this version takes its place.
+ * Found when the first paying user ran the new version and kept getting the
+ * old page: npx fetched the new one, which then only opened the old one.
+ * Only the process that answered as this folder's Kryptheon dashboard is
+ * stopped. Whether the port came free is the answer.
+ */
+async function replaceOldDashboard(port) {
+  const pid = listenerOn(port);
+  if (!pid || pid === process.pid) return false;
+  try {
+    if (process.platform === 'win32') spawnSync('taskkill', ['/PID', String(pid), '/F'], { windowsHide: true, timeout: 10000 });
+    else process.kill(pid, 'SIGTERM');
+  } catch (err) {
+    return false;
+  }
+  for (let i = 0; i < 50; i++) {
+    if (listenerOn(port) === null) return true;
+    await new Promise((r) => setTimeout(r, 100));
+  }
+  return false;
+}
+
 /**
  * The dashboard, started from the home folder: kept in a kryptheon-tests
  * folder there, the one record already uses - set up the first time, asked
@@ -2732,7 +2778,16 @@ async function dashboardCommand() {
   }
   // Running already: open that one rather than a second, which would watch
   // the same project twice. Exit code 0 is how the launcher knows.
-  if (await dashboardAlreadyRunning(DASHBOARD_PORT)) {
+  const running = await dashboardAlreadyRunning(DASHBOARD_PORT);
+  const mine = require(path.join(PACKAGE_DIR, 'package.json')).version;
+  if (running && running.version !== mine) {
+    const was = running.version || 'from before 0.1.41';
+    console.log('');
+    console.log(await replaceOldDashboard(DASHBOARD_PORT)
+      ? '  An older Kryptheon dashboard (' + was + ') was running for this folder.\n  It was closed, and ' + mine + ' starts in its place.'
+      : '  An older Kryptheon dashboard (' + was + ') is still running and could not be\n  closed. This one starts at another address; close the old window when you can.');
+  }
+  if (running && running.version === mine) {
     const address = 'http://127.0.0.1:' + DASHBOARD_PORT + '/';
     const shown = openInBrowser(address);
     console.log('');

package/kryptheon-dashboard.js

@@ -25,6 +25,7 @@ const { spawnSync } = require('child_process');
 const store = require('./kryptheon-store.js');
 const changes = require('./kryptheon-changes.js');
 const autostart = require('./kryptheon-autostart.js');
+const VERSION = require('./package.json').version;
 
 /**
  * Every check the dashboard knows the name of, in the order the person asked
@@ -543,6 +544,9 @@ function buildState(root, env) {
 
   return {
     project: { name: path.basename(root), root: root, storeDir: paths.dir },
+    // Which kryptheon this dashboard is: a newer `dashboard` asks, and takes
+    // the place of an older one still running, instead of opening it.
+    kryptheon: VERSION,
     looks: looks,
     proven: proven,
     checks: checks,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kryptheon",
-  "version": "0.1.40",
+  "version": "0.1.41",
   "description": "Record what you do in your app and get plain-language reports when it breaks.",
   "license": "SEE LICENSE IN LICENSE",
   "type": "commonjs",