qiksy 1.2.1 → 1.2.2

package/bin/qiksy.mjs

@@ -13,7 +13,7 @@
  * Everything runs on the user's machine, on their own Claude. Nothing is uploaded.
  */
 import { spawn, spawnSync } from 'node:child_process';
-import { existsSync, mkdirSync, rmSync, writeFileSync } from 'node:fs';
+import { existsSync, mkdirSync, readFileSync, rmSync, writeFileSync } from 'node:fs';
 import { homedir } from 'node:os';
 import { dirname, join, resolve } from 'node:path';
 import { platform } from 'node:process';
@@ -40,6 +40,15 @@ try {
   /* unwritable home: each backend falls back to its own folder */
 }
 
+/* The supervisor's own pid. `stop` used to kill the three tools by port — and the
+   supervisor, doing exactly what it was built for, brought them back two seconds later,
+   so stopping never actually stopped (found 2026-07-26 while upgrading a live install).
+   It records its pid here and `stop` ends the supervisor first. */
+const PID_FILE = join(DATA_HOME, 'engine.pid');
+const writePid = () => { try { writeFileSync(PID_FILE, String(process.pid)); } catch { /* unwritable home */ } };
+const readPid = () => { try { return Number(readFileSync(PID_FILE, 'utf8').trim()) || 0; } catch { return 0; } };
+const clearPid = () => { try { if (existsSync(PID_FILE)) rmSync(PID_FILE); } catch { /* ignore */ } };
+
 const TOOLS = [
   {
     id: 'work-finder',
@@ -180,14 +189,20 @@ async function cmdStatus() {
 function cmdStop() {
   console.log(`\n${c.blue(c.b('  ◆ Qiksy'))}${c.dim('  — stopping')}\n`);
   let killed = 0;
+  // Who was actually up BEFORE we touch anything — killing the supervisor takes its
+  // children with it, and reporting "was not running" about something we just stopped
+  // is the kind of small lie that makes people distrust the rest of the output.
+  const wasUp = new Map(TOOLS.map((t) => [t.id, pidsOn(t.port).length > 0]));
+  // The supervisor first, or it restarts every tool we are about to stop.
+  const sup = readPid();
+  if (sup && sup !== process.pid) { if (killPid(sup)) killed++; clearPid(); }
   for (const t of TOOLS) {
-    const pids = pidsOn(t.port);
-    if (!pids.length) {
-      console.log(c.dim(`  · ${t.name.padEnd(18)} was not running`));
-      continue;
-    }
-    for (const p of pids) if (killPid(p)) killed++;
-    console.log(c.green(`  ✓ ${t.name.padEnd(18)} stopped`));
+    for (const p of pidsOn(t.port)) if (killPid(p)) killed++;
+    console.log(
+      wasUp.get(t.id)
+        ? c.green(`  ✓ ${t.name.padEnd(18)} stopped`)
+        : c.dim(`  · ${t.name.padEnd(18)} was not running`)
+    );
   }
   console.log(c.dim(`\n  ${killed ? 'Everything is off.' : 'Nothing was running.'}  Start again: npx qiksy\n`));
 }
@@ -448,10 +463,12 @@ function stopAll() {
 process.on('SIGINT', () => {
   console.log(c.dim('\n  Stopping…'));
   stopAll();
+  clearPid();
   process.exit(0);
 });
 process.on('SIGTERM', () => {
   stopAll();
+  clearPid();
   process.exit(0);
 });
 
@@ -546,6 +563,7 @@ process.on('SIGTERM', () => {
     if (ok) up.push(tool);
   }
 
+  writePid();
   startWatchdog();
 
   console.log('');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "qiksy",
-  "version": "1.2.1",
+  "version": "1.2.2",
   "description": "Qiksy \u2014 one local engine for every Qiksy tool (Work Finder, Travel Assistant, Client Finder). Runs on your own Claude; nothing leaves your machine.",
   "type": "module",
   "bin": {