crawlmark-mcp 0.1.2 → 0.1.3

package/README.md

@@ -309,7 +309,7 @@ Configure your MCP client
 **Claude Code** (from npm):
 
 ```bash
-claude mcp add crawlmark -s user -- npx -y crawlmark-mcp
+claude mcp add crawlmark -s user -- npx -y crawlmark-mcp@latest
 ```
 
 **Claude Code** (from source):
@@ -325,18 +325,18 @@ claude mcp add crawlmark -s user -- node --disable-warning=ExperimentalWarning /
   "mcpServers": {
     "crawlmark": {
       "command": "npx",
-      "args": ["-y", "crawlmark-mcp"]
+      "args": ["-y", "crawlmark-mcp@latest"]
     }
   }
 }
 ```
 
-**Codex** (`~/.codex/config.toml`, or run `codex mcp add crawlmark -- npx -y crawlmark-mcp`):
+**Codex** (`~/.codex/config.toml`, or run `codex mcp add crawlmark -- npx -y crawlmark-mcp@latest`):
 
 ```toml
 [mcp_servers.crawlmark]
 command = "npx"
-args = ["-y", "crawlmark-mcp"]
+args = ["-y", "crawlmark-mcp@latest"]
 ```
 
 For a source checkout, use `"command": "node"` and `"args": ["--disable-warning=ExperimentalWarning", "/absolute/path/to/crawlmark-mcp/dist/index.js"]`. Claude Desktop, Cursor and Codex may not load your shell's `PATH`, so if one can't find `node` or `npx`, use the absolute path (`which node`). Credentials come from `.env` in the config home, so no `env` block is needed unless your settings live somewhere other than the default; then add `"env": { "CRAWLMARK_HOME": "/path/to/home" }`.

package/dist/setup/apps.js

@@ -29,9 +29,33 @@ export function cmdLine(cmd, args) {
 export function isNpxCache(projectRoot) {
     return projectRoot.split(/[\\/]/).includes("_npx");
 }
+/** What an npx install registers. `@latest` makes npx check for a newer crawlmark each time an app starts it. */
+export const NPX_SPEC = "crawlmark-mcp@latest";
+/**
+ * True when `current` is the entry an earlier crawlmark wrote from an npx install (`npx -y crawlmark-mcp`, no
+ * version) and `launch` is the same entry with `@latest`. That's an update of crawlmark's own entry, not a
+ * different server, so the wizard offers it with yes as the default.
+ */
+export function isNpxLatestUpgrade(current, launch) {
+    if (!current || typeof current.command !== "string")
+        return false;
+    // `claude mcp get` prints the args as one string; app configs keep an array.
+    const args = Array.isArray(current.args) ? current.args : typeof current.args === "string" ? current.args.trim().split(/\s+/) : undefined;
+    if (!args)
+        return false;
+    const npx = /(^|[\\/])npx(\.cmd)?$/i;
+    if (!npx.test(current.command) || !npx.test(launch.command))
+        return false;
+    if (JSON.stringify(args) !== JSON.stringify(["-y", "crawlmark-mcp"]) || JSON.stringify(launch.args) !== JSON.stringify(["-y", NPX_SPEC]))
+        return false;
+    const env = (e) => JSON.stringify(Object.entries((e && typeof e === "object" ? e : {})).sort());
+    return env(current.env) === env(launch.env);
+}
+const NPX_UPGRADE_NOTE = "It runs `npx -y crawlmark-mcp`, which keeps reusing the version npx downloaded first.";
+const NPX_UPGRADE_QUESTION = "Update it to `npx -y crawlmark-mcp@latest`, so it picks up new versions?";
 /**
  * How an app should start the server. CRAWLMARK_HOME is passed when the home isn't
- * the project root. From an npx cache, `npx -y crawlmark-mcp` is registered instead
+ * the project root. From an npx cache, `npx -y crawlmark-mcp@latest` is registered instead
  * of the cache path. `nodeCommand` is `node` (Claude Code, which inherits the shell
  * PATH) or an absolute path (Claude Desktop, Cursor and Codex, which may not).
  */
@@ -39,7 +63,7 @@ export function serverLaunch(ctx, nodeCommand) {
     const env = ctx.paths.homeSource === "project" ? undefined : { CRAWLMARK_HOME: ctx.paths.home };
     if (isNpxCache(ctx.paths.projectRoot)) {
         const npx = nodeCommand === "node" ? "npx" : join(dirname(nodeCommand), process.platform === "win32" ? "npx.cmd" : "npx");
-        return { command: npx, args: ["-y", "crawlmark-mcp"], ...(env ? { env } : {}) };
+        return { command: npx, args: ["-y", NPX_SPEC], ...(env ? { env } : {}) };
     }
     return { command: nodeCommand, args: ["--disable-warning=ExperimentalWarning", join(ctx.paths.projectRoot, "dist", "index.js")], ...(env ? { env } : {}) };
 }
@@ -153,11 +177,15 @@ async function registerClaudeCode(ctx, probe, notes) {
         return "manual";
     }
     const scope = current.scope ?? "user";
-    ctx.out("Claude Code already has a crawlmark server, with a different command:");
+    const upgrade = isNpxLatestUpgrade(current, launch);
+    if (upgrade)
+        ctx.out(`Claude Code has crawlmark registered. ${NPX_UPGRADE_NOTE}`);
+    else
+        ctx.out("Claude Code already has a crawlmark server, with a different command:");
     ctx.out(`  current: ${describeRegistration(current)}`);
     ctx.out(`  new:     ${describeRegistration({ ...launch, scope: "user" })}`);
-    // Default no, so --yes never removes an existing registration.
-    if (!(await ctx.prompt.confirm(`Replace it (claude mcp remove ${SERVER_NAME} -s ${scope}, then add)?`, false))) {
+    // Default no for a different server, so --yes never removes one; yes for crawlmark's own older npx entry.
+    if (!(await ctx.prompt.confirm(upgrade ? NPX_UPGRADE_QUESTION : `Replace it (claude mcp remove ${SERVER_NAME} -s ${scope}, then add)?`, upgrade))) {
         notes.push({ kind: "info", text: "Kept the existing Claude Code registration." });
         return "skipped";
     }
@@ -195,8 +223,15 @@ async function registerJsonApp(ctx, id, notes) {
         }
         uiFor(ctx).preview(`${app} config: ${path}`, describeDesktopChange(merge, entry));
         // Replacing a different crawlmark entry defaults to no, so --yes never replaces one.
-        const question = merge.replaced ? "Replace the existing crawlmark entry? (the current file is kept as .bak)" : "Write this change? (the current file is kept as .bak)";
-        if (!(await ctx.prompt.confirm(question, !merge.replaced))) {
+        const upgrade = merge.replaced && isNpxLatestUpgrade(merge.previous, entry);
+        if (upgrade)
+            ctx.out(NPX_UPGRADE_NOTE);
+        const question = upgrade
+            ? `${NPX_UPGRADE_QUESTION} (the current file is kept as .bak)`
+            : merge.replaced
+                ? "Replace the existing crawlmark entry? (the current file is kept as .bak)"
+                : "Write this change? (the current file is kept as .bak)";
+        if (!(await ctx.prompt.confirm(question, upgrade || !merge.replaced))) {
             notes.push({ kind: "info", text: `Skipped ${app}.` });
             return "skipped";
         }
@@ -287,10 +322,14 @@ async function registerCodexCli(ctx, probe, notes) {
         notes.push({ kind: "warn", text: `To replace it yourself: codex mcp remove ${SERVER_NAME}, then ${display}` });
         return "manual";
     }
-    ctx.out("Codex already has a crawlmark server, with a different command:");
+    const upgrade = isNpxLatestUpgrade(state.entry, launch);
+    if (upgrade)
+        ctx.out(`Codex has crawlmark registered. ${NPX_UPGRADE_NOTE}`);
+    else
+        ctx.out("Codex already has a crawlmark server, with a different command:");
     ctx.out(`  current: ${describeRegistration(state.entry)}`);
     ctx.out(`  new:     ${describeRegistration(want)}`);
-    if (!(await ctx.prompt.confirm(`Replace it (codex mcp remove ${SERVER_NAME}, then add)?`, false))) {
+    if (!(await ctx.prompt.confirm(upgrade ? NPX_UPGRADE_QUESTION : `Replace it (codex mcp remove ${SERVER_NAME}, then add)?`, upgrade))) {
         notes.push({ kind: "info", text: "Kept the existing Codex registration." });
         return "skipped";
     }
@@ -330,8 +369,15 @@ async function registerCodexToml(ctx, notes) {
         return "already";
     }
     uiFor(ctx).preview(`Codex config: ${path}`, describeCodexChange(plan, entry));
-    const question = plan.replaced ? "Replace the existing crawlmark entry? (the current file is kept as .bak)" : "Write this change? (the current file is kept as .bak)";
-    if (!(await ctx.prompt.confirm(question, !plan.replaced))) {
+    const upgrade = plan.replaced && isNpxLatestUpgrade(plan.previous, serverLaunch(ctx, ctx.node.execPath));
+    if (upgrade)
+        ctx.out(NPX_UPGRADE_NOTE);
+    const question = upgrade
+        ? `${NPX_UPGRADE_QUESTION} (the current file is kept as .bak)`
+        : plan.replaced
+            ? "Replace the existing crawlmark entry? (the current file is kept as .bak)"
+            : "Write this change? (the current file is kept as .bak)";
+    if (!(await ctx.prompt.confirm(question, upgrade || !plan.replaced))) {
         notes.push({ kind: "info", text: "Skipped Codex." });
         return "skipped";
     }
@@ -370,7 +416,7 @@ export async function registerApps(ctx) {
     const ui = uiFor(ctx);
     const notes = [];
     if (isNpxCache(ctx.paths.projectRoot)) {
-        ctx.out("crawlmark is running from a temporary npx cache, so Claude will start it with `npx -y crawlmark-mcp` instead of this path.");
+        ctx.out("crawlmark is running from a temporary npx cache, so Claude will start it with `npx -y crawlmark-mcp@latest` instead of this path.");
     }
     const probes = await ui.spin("Looking for AI apps...", () => detectApps(ctx), { plain: false });
     const defaults = probes.filter((p) => p.detected).map((p) => p.id);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "crawlmark-mcp",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "MCP server for automated SEO audits and reports: Google Search Console, GA4, Tag Manager, DataForSEO, Lighthouse and axe-core",
   "keywords": [
     "mcp",