mulmoterminal 0.1.3 → 0.1.4

package/README.md

@@ -24,6 +24,10 @@ npm install -g mulmoterminal
 mulmoterminal
 ```
 
+A global install isn't auto-updated, so on startup MulmoTerminal checks npm and
+prints a one-line notice when a newer version is available (`npm i -g mulmoterminal`
+to update). Disable with `MULMOTERMINAL_NO_UPDATE_CHECK=1` (or `NO_UPDATE_NOTIFIER=1`).
+
 Options: `--cwd <dir>` (working directory — relative paths allowed; defaults to the
 directory you run the command from), `--port <n>` (default 3456), `--no-open`,
 `--version`, `--help`.

package/bin/mulmoterminal.js

@@ -12,6 +12,7 @@ import { createRequire } from "node:module";
 import { createServer } from "node:net";
 import { dirname, join, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
+import { fetchLatestVersion, isNewerVersion } from "./update-check.js";
 
 const __dirname = dirname(fileURLToPath(import.meta.url));
 const PKG_DIR = join(__dirname, "..");
@@ -30,6 +31,21 @@ const { version: VERSION } = createRequire(import.meta.url)("../package.json");
 const log = (msg) => console.log(`\x1b[36m[mulmoterminal]\x1b[0m ${msg}`);
 const error = (msg) => console.error(`\x1b[31m[mulmoterminal]\x1b[0m ${msg}`);
 
+// Non-blocking notice when a newer version is published — `npm i -g` never
+// auto-updates. Opt out via MULMOTERMINAL_NO_UPDATE_CHECK / NO_UPDATE_NOTIFIER.
+function checkForUpdate() {
+  if (process.env.MULMOTERMINAL_NO_UPDATE_CHECK || process.env.NO_UPDATE_NOTIFIER) return;
+  fetchLatestVersion()
+    .then((latest) => {
+      if (latest && isNewerVersion(latest, VERSION)) {
+        log(`\x1b[33mUpdate available: ${VERSION} → ${latest}  ·  run: npm i -g mulmoterminal\x1b[0m`);
+      }
+    })
+    .catch(() => {
+      // best-effort; never disrupt startup
+    });
+}
+
 function claudeInstalled() {
   try {
     // Intentionally resolves `claude` from the user's PATH — detecting their
@@ -230,6 +246,8 @@ Options:
     return;
   }
 
+  checkForUpdate();
+
   if (!claudeInstalled()) {
     error("Claude Code CLI not found.");
     error("Install it first:  npm install -g @anthropic-ai/claude-code  &&  claude auth login");

package/bin/update-check.js

@@ -0,0 +1,37 @@
+// Update-check helpers for the launcher, split out so the version comparison is
+// unit-testable. Network calls are best-effort and never throw.
+
+const REGISTRY = (process.env.npm_config_registry || "https://registry.npmjs.org").replace(/\/$/, "");
+
+// Best-effort latest-version lookup. Resolves null on any failure (offline,
+// timeout, non-OK, bad payload) so callers never block or break startup.
+export async function fetchLatestVersion(pkg = "mulmoterminal") {
+  try {
+    const res = await fetch(`${REGISTRY}/${pkg}/latest`, {
+      signal: AbortSignal.timeout(1500),
+      headers: { accept: "application/json" },
+    });
+    if (!res.ok) return null;
+    const body = await res.json();
+    return typeof body.version === "string" ? body.version : null;
+  } catch {
+    return null;
+  }
+}
+
+// True if `latest` is a strictly newer major.minor.patch than `current`.
+// Pre-release suffixes are ignored and parts are compared numerically (so
+// 0.1.10 > 0.1.9, which a lexical compare would get wrong).
+export function isNewerVersion(latest, current) {
+  const parts = (v) =>
+    String(v)
+      .split("-")[0]
+      .split(".")
+      .map((n) => Number.parseInt(n, 10) || 0);
+  const a = parts(latest);
+  const b = parts(current);
+  for (let i = 0; i < 3; i++) {
+    if ((a[i] ?? 0) !== (b[i] ?? 0)) return (a[i] ?? 0) > (b[i] ?? 0);
+  }
+  return false;
+}

package/bin/update-check.spec.ts

@@ -0,0 +1,46 @@
+import { describe, it, expect, vi, afterEach } from "vitest";
+import { isNewerVersion, fetchLatestVersion } from "./update-check.js";
+
+describe("isNewerVersion", () => {
+  const cases: [string, string, boolean][] = [
+    ["0.1.3", "0.1.0", true],
+    ["0.2.0", "0.1.9", true],
+    ["1.0.0", "0.9.9", true],
+    ["0.1.10", "0.1.9", true], // numeric, not lexical (the bug a string compare hits)
+    ["0.1.3", "0.1.3", false],
+    ["0.1.0", "0.1.3", false],
+    ["0.9.9", "1.0.0", false],
+    ["0.1.4-beta.1", "0.1.3", true], // pre-release suffix ignored on the core
+    ["0.1.3", "0.1.3-beta.1", false], // equal core → not newer
+  ];
+  it.each(cases)("isNewerVersion(%s, %s) === %s", (latest, current, expected) => {
+    expect(isNewerVersion(latest, current)).toBe(expected);
+  });
+});
+
+describe("fetchLatestVersion", () => {
+  const stubFetch = (impl: () => Promise<unknown>) => vi.stubGlobal("fetch", vi.fn(impl));
+  afterEach(() => vi.unstubAllGlobals());
+
+  it("returns the version from a 200 response", async () => {
+    stubFetch(async () => ({ ok: true, json: async () => ({ version: "1.2.3" }) }));
+    expect(await fetchLatestVersion()).toBe("1.2.3");
+  });
+
+  it("returns null on a non-OK response", async () => {
+    stubFetch(async () => ({ ok: false, json: async () => ({}) }));
+    expect(await fetchLatestVersion()).toBeNull();
+  });
+
+  it("returns null when fetch rejects (offline / timeout)", async () => {
+    stubFetch(async () => {
+      throw new Error("offline");
+    });
+    expect(await fetchLatestVersion()).toBeNull();
+  });
+
+  it("returns null when the payload has no version string", async () => {
+    stubFetch(async () => ({ ok: true, json: async () => ({ name: "mulmoterminal" }) }));
+    expect(await fetchLatestVersion()).toBeNull();
+  });
+});