@delorenj/pjangler 1.5.27 → 1.5.29

package/dist/crank/tui.js

@@ -435,6 +435,7 @@ function CrankDashboard({ controller, refreshMs = 5e3, clock = Date.now, onQuit
 }
 async function runCrankDashboard(controller, run = {}) {
   const { refreshMs, ...ink } = run;
+  controller.read();
   await runInk((finish) => /* @__PURE__ */ jsx3(CrankDashboard, { controller, refreshMs, onQuit: () => finish() }), { alternateScreen: true, ...ink });
 }
 export {

package/dist/index.js

@@ -6064,7 +6064,7 @@ var init_capture = __esm({
 
 // src/index.ts
 import { spawnSync as spawnSync17 } from "node:child_process";
-import { existsSync as existsSync25, readdirSync as readdirSync12, readFileSync as readFileSync29, statSync as statSync8 } from "node:fs";
+import { existsSync as existsSync26, readdirSync as readdirSync12, readFileSync as readFileSync29, statSync as statSync8 } from "node:fs";
 import { basename as basename11, join as join34, resolve as resolve25 } from "node:path";
 import { Command as Command3, CommanderError, Option } from "commander";
 
@@ -20377,6 +20377,8 @@ init_project();
 init_style();
 
 // src/tui/load.ts
+import { existsSync as existsSync24 } from "node:fs";
+import { fileURLToPath as fileURLToPath6 } from "node:url";
 var TuiUnavailableError = class extends Error {
 };
 function isInteractive(streams = {}) {
@@ -20384,6 +20386,9 @@ function isInteractive(streams = {}) {
   const stdout = streams.stdout ?? process.stdout;
   return Boolean(stdin.isTTY && stdout.isTTY);
 }
+var RESOLVER_FRAME = /node:internal\/modules\/(?:esm\/resolve|package_json_reader)/;
+var CANNOT_FIND = /^Cannot find (package|module) '([^']+)' imported from (.+)$/;
+var INSTALL_DEPS = "run npm ci in this checkout, or reinstall the global package (npm i -g @delorenj/pjangler)";
 async function loadScreen(relativeToDist, base) {
   const url = new URL(relativeToDist, base);
   try {
@@ -20392,10 +20397,15 @@ async function loadScreen(relativeToDist, base) {
       url.href
     );
   } catch (error) {
-    if (error?.code === "ERR_MODULE_NOT_FOUND") {
+    const err = error;
+    if (err?.code !== "ERR_MODULE_NOT_FOUND") throw error;
+    if (!existsSync24(fileURLToPath6(url))) {
       throw new TuiUnavailableError(`TUI screen ${relativeToDist} is not built (run: npm run build); looked for ${url.pathname}`);
     }
-    throw error;
+    if (typeof err.stack !== "string" || !RESOLVER_FRAME.test(err.stack)) throw error;
+    const missing = CANNOT_FIND.exec(err.message);
+    const detail = missing ? `cannot find ${missing[1]} '${missing[2]}' imported by ${missing[3]}` : "an import of it cannot be resolved";
+    throw new TuiUnavailableError(`TUI screen ${relativeToDist} is built but its dependencies are not installed: ${detail}; ${INSTALL_DEPS}`);
   }
 }
 
@@ -20420,7 +20430,7 @@ var TAB_STATUS_SUFFIX = /[\s🟢🟡🔴🔔⚫✳✻●·─]+$/u;
 var defaultShell = (command, args) => {
   const result2 = spawnSync16(command, args, { encoding: "utf8", timeout: 1e4 });
   if (result2.error) return { stdout: "", status: 127 };
-  return { stdout: result2.stdout ?? "", status: result2.status ?? 1 };
+  return { stdout: result2.stdout ?? "", status: result2.status ?? 1, stderr: result2.stderr ?? "" };
 };
 function tabDisplay(name) {
   const trimmed2 = (name ?? "").trim();
@@ -20503,10 +20513,28 @@ function redis(deps = {}) {
   const endpoint = redisEndpoint();
   const host = deps.host ?? endpoint.host;
   const port = deps.port ?? endpoint.port;
-  const call = (args) => shell("redis-cli", ["--raw", "-h", host, "-p", String(port), ...args]);
+  const call = (args) => shell("redis-cli", ["--raw", "-e", "-h", host, "-p", String(port), ...args]);
+  let errorExitSupport = "unprobed";
+  const requireErrorExit = () => {
+    if (errorExitSupport === "unprobed") {
+      const help = shell("redis-cli", ["--help"]);
+      errorExitSupport = help.status === 0 && /^usage: redis-cli/im.test(help.stdout) ? /(^|\n)\s+-e\s/.test(help.stdout) ? "yes" : "no" : "unknown";
+    }
+    if (errorExitSupport === "no") {
+      throw new Error(
+        `redis-cli is too old for the crank store at ${host}:${port}: no -e flag (needs redis-tools >= 6.2) to fail on Redis error replies \u2014 upgrade redis-tools`
+      );
+    }
+  };
+  const notAnswering = (stderr) => !stderr || stderr.startsWith("Could not connect to Redis at") || stderr.startsWith("Error: ");
   const run = (...args) => {
+    requireErrorExit();
     const result2 = call(args);
-    if (result2.status !== 0) throw new Error(`Redis is not answering at ${host}:${port} (${args[0]} failed)`);
+    if (result2.status !== 0) {
+      const stderr = (result2.stderr ?? "").trim();
+      if (notAnswering(stderr)) throw new Error(`Redis is not answering at ${host}:${port} (${args[0]} failed)`);
+      throw new Error(`Redis error reply at ${host}:${port}: ${stderr.split("\n")[0]} (${args.join(" ")})`);
+    }
     return result2.stdout.replace(/\r?\n$/, "");
   };
   const lines = (out) => out.split("\n").map((line) => line.replace(/\r$/, "")).filter(Boolean);
@@ -20515,6 +20543,7 @@ function redis(deps = {}) {
     endpoint: `${host}:${port}`,
     /** PING -> true only when Redis answers PONG. */
     ping: () => {
+      requireErrorExit();
       const result2 = call(["PING"]);
       return result2.status === 0 && result2.stdout.replace(/\r?\n$/, "") === "PONG";
     },
@@ -20970,24 +20999,23 @@ function registerCrankCli(program2) {
       console.error(`${xmark} --tail needs a positive integer, got "${options.tail}"`);
       process.exit(1);
     }
-    let current;
     try {
-      current = resolveCurrentTab();
+      const current = resolveCurrentTab();
+      const store = redis();
+      const rows = store.logRows(current.tab);
+      if (!rows.length) {
+        console.log(`  ${dim(`no crank log rows for ${current.tab}`)}`);
+        return;
+      }
+      const shown = rows.slice(-tail);
+      console.log(`  ${bold("crank log")} ${dim(glyph.dot)} ${cyan(current.tab)} ${dim(`(${shown.length}/${rows.length} rows, newest last)`)}`);
+      for (const raw of shown) {
+        console.log(renderLogRow(raw) ?? raw);
+      }
     } catch (error) {
       console.error(`${xmark} ${error instanceof Error ? error.message : String(error)}`);
       process.exit(1);
     }
-    const store = redis();
-    const rows = store.logRows(current.tab);
-    if (!rows.length) {
-      console.log(`  ${dim(`no crank log rows for ${current.tab}`)}`);
-      return;
-    }
-    const shown = rows.slice(-tail);
-    console.log(`  ${bold("crank log")} ${dim(glyph.dot)} ${cyan(current.tab)} ${dim(`(${shown.length}/${rows.length} rows, newest last)`)}`);
-    for (const raw of shown) {
-      console.log(renderLogRow(raw) ?? raw);
-    }
   });
 }
 function fail(error, what) {
@@ -21100,7 +21128,7 @@ async function promptForRuleIds(rules) {
   return selected;
 }
 function readJson2(path) {
-  if (!existsSync25(path)) return void 0;
+  if (!existsSync26(path)) return void 0;
   try {
     const parsed = JSON.parse(readFileSync29(path, "utf8"));
     return parsed && typeof parsed === "object" && !Array.isArray(parsed) ? parsed : void 0;
@@ -21195,7 +21223,7 @@ function actionNeedsRun(plan, kind, syncMode) {
     if (!action || action.kind !== "project.write-manifest") return false;
     const next = `${JSON.stringify(action.manifest, null, 2)}
 `;
-    return !existsSync25(action.path) || readFileSync29(action.path, "utf8") !== next;
+    return !existsSync26(action.path) || readFileSync29(action.path, "utf8") !== next;
   }
   if (kind === "copier.copy.commonproject") return true;
   if (kind === "ticket-provider.create-or-link") return plan.actions.some((action) => action.kind === kind && action.enabled);
@@ -21252,7 +21280,7 @@ async function resolveProjectInitTarget(name, options) {
   if (!targetDir && cwdGitRoot) {
     targetDir = cwdGitRoot;
   }
-  if (!targetDir && !name && existsSync25(join34(cwd, ".project.json"))) {
+  if (!targetDir && !name && existsSync26(join34(cwd, ".project.json"))) {
     targetDir = cwd;
   }
   if (!targetDir && interactive) {
@@ -21263,10 +21291,10 @@ async function resolveProjectInitTarget(name, options) {
   if (!targetDir) {
     throw new Error("Project name or --target-dir is required when project init is not run inside a git repo or a directory carrying .project.json");
   }
-  const targetExists = existsSync25(targetDir);
+  const targetExists = existsSync26(targetDir);
   if (targetExists && !statSync8(targetDir).isDirectory()) throw new Error(`Target path is not a directory: ${targetDir}`);
   const targetGitRoot = targetExists ? findGitRoot(targetDir) : void 0;
-  const alreadyScaffolded = targetExists && (existsSync25(join34(targetDir, ".project.json")) || existsSync25(join34(targetDir, ".copier-answers.yml")));
+  const alreadyScaffolded = targetExists && (existsSync26(join34(targetDir, ".project.json")) || existsSync26(join34(targetDir, ".copier-answers.yml")));
   const syncMode = Boolean(
     targetGitRoot && resolve25(targetGitRoot) === resolve25(targetDir) || alreadyScaffolded
   );

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@delorenj/pjangler",
-  "version": "1.5.27",
+  "version": "1.5.29",
   "description": "Project subsystem bootstrapper CLI",
   "type": "module",
   "main": "dist/index.js",