qatools-cli 1.1.10 → 1.1.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "qatools-cli",
-  "version": "1.1.10",
+  "version": "1.1.11",
   "description": "CLI for QA Tools — requirements, epics, sprints, sessions, App Context sync, project agenda. The companion to qa-tools.com.",
   "scripts": {
     "release": "npm version patch --no-git-tag-version && npm publish",

package/qatools.js

@@ -129,6 +129,50 @@ program
   .description("CLI for QA Tools")
   .version(require("./package.json").version);
 
+// Update nudge: before every command, print (from cache) if the installed CLI is behind npm,
+// and refresh the cache in the background (throttled to once/day). sync can't upgrade the tool itself.
+// Kept fully non-blocking and fail-open so it never slows or breaks a command (parse() is sync).
+const VERSION_CACHE_FILE = path.join(process.env.HOME || process.env.USERPROFILE || ".", ".qatools-version");
+function cmpVersion(a, b) {
+  const pa = String(a).split(".").map(Number), pb = String(b).split(".").map(Number);
+  for (let i = 0; i < 3; i++) { const d = (pa[i] || 0) - (pb[i] || 0); if (d) return d < 0 ? -1 : 1; }
+  return 0;
+}
+function printUpdateNoticeFromCache() {
+  try {
+    const current = require("./package.json").version;
+    const cache = JSON.parse(fs.readFileSync(VERSION_CACHE_FILE, "utf8"));
+    if (cache.latest && cmpVersion(current, cache.latest) < 0) {
+      // stderr so it never corrupts parseable stdout (e.g. -o json output)
+      console.error(`\n⚠ qatools-cli ${current} → ${cache.latest} available. Upgrade: npm i -g qatools-cli@latest\n`);
+    }
+  } catch { /* no cache yet or unreadable — skip */ }
+}
+function refreshVersionCacheInBackground() {
+  try {
+    let cache = {};
+    try { cache = JSON.parse(fs.readFileSync(VERSION_CACHE_FILE, "utf8")); } catch { /* none */ }
+    if (cache.checkedAt && Date.now() - cache.checkedAt < 24 * 3600 * 1000) return; // throttled: at most once/day
+    const req = require("https").get("https://registry.npmjs.org/qatools-cli/latest", { timeout: 3000 }, (res) => {
+      let body = "";
+      res.on("data", (c) => (body += c));
+      res.on("end", () => {
+        try {
+          const latest = JSON.parse(body).version;
+          if (latest) fs.writeFileSync(VERSION_CACHE_FILE, JSON.stringify({ latest, checkedAt: Date.now() }));
+        } catch { /* malformed — skip */ }
+      });
+    });
+    req.on("error", () => {});
+    req.on("timeout", () => req.destroy());
+    if (req.unref) req.unref(); // don't keep the process alive for this
+  } catch { /* never block the command */ }
+}
+program.hook("preAction", () => {
+  printUpdateNoticeFromCache();
+  refreshVersionCacheInBackground();
+});
+
 program
   .command("upload-analysis")
   .description("Upload a Claude analysis JSON to a test session")
@@ -740,36 +784,6 @@ program
     process.exit(0);
   });
 
-// Non-blocking: warn if a newer qatools-cli is published on npm (sync can't upgrade the tool itself)
-function cmpVersion(a, b) {
-  const pa = String(a).split(".").map(Number), pb = String(b).split(".").map(Number);
-  for (let i = 0; i < 3; i++) { const d = (pa[i] || 0) - (pb[i] || 0); if (d) return d < 0 ? -1 : 1; }
-  return 0;
-}
-function checkCliVersion() {
-  return new Promise((resolve) => {
-    try {
-      const current = require("./package.json").version;
-      const req = require("https").get("https://registry.npmjs.org/qatools-cli/latest", { timeout: 3000 }, (res) => {
-        let body = "";
-        res.on("data", (c) => (body += c));
-        res.on("end", () => {
-          try {
-            const latest = JSON.parse(body).version;
-            if (latest && cmpVersion(current, latest) < 0) {
-              console.log(`\n⚠ A newer qatools-cli is available: ${current} → ${latest}`);
-              console.log(`  Upgrade so commands, skills and rules are current: npm i -g qatools-cli@latest\n`);
-            }
-          } catch { /* malformed response — skip */ }
-          resolve();
-        });
-      });
-      req.on("error", () => resolve());
-      req.on("timeout", () => { req.destroy(); resolve(); });
-    } catch { resolve(); }
-  });
-}
-
 program
   .command("sync")
   .description("Sync App Context from qa-tools into a CLAUDE.md block")
@@ -780,7 +794,6 @@ program
   .option("-e, --email <email>", "Login email")
   .option("-p, --password <password>", "Login password")
   .action(async (options) => {
-    await checkCliVersion();
     await authenticate(options);
     const creds = getCredentials();
     const projectId = options.project || creds?.projectId;