@jam-mcp/server 1.4.1 → 1.4.2

package/README.md

@@ -74,10 +74,10 @@ auth login   Store Jira credentials in this user's OS secret store
 runtime      Show or change which JAM build this machine runs
 ```
 
-Written out, that is `npx --yes @jam-mcp/launcher@1.4.1 doctor`, or just `jam
+Written out, that is `npx --yes @jam-mcp/launcher@1.4.2 doctor`, or just `jam
 doctor` if you took the launcher's optional global install. Starting from
 nothing — no install, no runtime chosen yet — use
-`npx --yes @jam-mcp/bootstrap@1.4.1 init` instead.
+`npx --yes @jam-mcp/bootstrap@1.4.2 init` instead.
 
 Credentials come from the process environment or this user's OS secret store —
 never from a repository file — and never appear in logs, telemetry, or tool

package/dist/bootstrap/host-mcp.d.ts

@@ -30,6 +30,8 @@ export type HostState = {
     entryVersion?: string;
     /** The entry is present but does not run the launcher this release registers. */
     entryStale?: boolean;
+    /** The entry runs the global `jam` executable rather than an npx pin. */
+    entryBare?: boolean;
 };
 export type HostRunResult = {
     status: number | null;
@@ -40,7 +42,9 @@ export type HostRunResult = {
 /** Injected by tests. Nothing in this module may reach a real CLI unasked. */
 export type HostRunner = (command: HostCommand) => HostRunResult;
 export declare const defaultHostRunner: HostRunner;
-export declare function hostRegistration(id: HostId): HostCommand | undefined;
+export declare function hostRegistration(id: HostId, options?: {
+    bare?: boolean;
+}): HostCommand | undefined;
 /** The removal that has to precede re-registering an entry this host already has. */
 export declare function hostUnregistration(id: HostId): HostCommand | undefined;
 /**
@@ -60,15 +64,38 @@ export declare function listsJamEntry(stdout: string): boolean;
 export declare function jamEntryLine(stdout: string): string | null;
 /** The launcher version a listing line runs, when the line names one. */
 export declare function entryLauncherVersion(line: string): string | undefined;
+/**
+ * Does this entry run the persistent `jam` executable rather than an npx pin?
+ *
+ * `jam: jam serve`, `jam: /usr/local/bin/jam serve`, `jam: C:\...\jam.cmd serve`
+ * all count; an npx line never does - its command token is `npx`. A bare entry
+ * carries no version in the listing, so its staleness has to be measured from
+ * the executable it would actually run (bareJamVersion), not from the line.
+ */
+export declare function isBareJamEntry(line: string): boolean;
+/**
+ * The version a bare `jam` registration actually runs, measured by asking the
+ * executable itself. `runtime status --json` answers from ~/.jam/config.yaml
+ * and the resolved build - the same resolution the registered entry performs.
+ *
+ * undefined when `jam` is not on PATH or does not answer: a registration that
+ * cannot be measured counts as stale, same as an unreadable pin.
+ */
+export declare function bareJamVersion(run?: HostRunner): string | undefined;
+/** Is this measured launcher version the one this release registers? */
+export declare function preferBareRegistration(version: string | undefined): boolean;
 /**
  * Does this entry run the launcher this release registers?
  *
- * Anything else - an older pin, an unpinned spec, a line whose command JAM
- * cannot read - counts as stale. That direction is deliberate: `mcp add`
- * rewrites the same entry, so a needless repair costs one command, while a
- * missed one leaves the agent talking to a server nobody tested it against.
+ * An npx pin answers from the line itself. A bare `jam` line names no version,
+ * so the caller passes what the executable measured (`bareVersion`) - without
+ * it, bare stays stale. Anything else - an older pin, an unpinned spec, a line
+ * whose command JAM cannot read - counts as stale. That direction is
+ * deliberate: `mcp add` rewrites the same entry, so a needless repair costs
+ * one command, while a missed one leaves the agent talking to a server nobody
+ * tested it against.
  */
-export declare function isEntryStale(line: string): boolean;
+export declare function isEntryStale(line: string, bareVersion?: string): boolean;
 /**
  * Ask each host what it has, and whether it is there at all.
  *

package/dist/bootstrap/host-mcp.js

@@ -23,6 +23,13 @@ export const defaultHostRunner = ({ command, args }) => {
  * Windows shell, and every argument here is a bare token instead.
  */
 const LAUNCH = ["--", JAM_MCP_ENTRY.command, ...JAM_MCP_ENTRY.args];
+/**
+ * What a persistent install registers: the global `jam` executable, no
+ * package runner, no cache. Only offered when the measured global launcher
+ * is exactly this release (preferBareRegistration) - registering bare over
+ * an older global would silently downgrade the served toolset.
+ */
+const LAUNCH_BARE = ["--", "jam", "serve"];
 const ADAPTERS = [
     {
         id: "claude-code",
@@ -39,8 +46,14 @@ const ADAPTERS = [
         unregister: { command: "codex", args: ["mcp", "remove", "jam"] },
     },
 ];
-export function hostRegistration(id) {
-    return ADAPTERS.find((a) => a.id === id)?.register;
+export function hostRegistration(id, options = {}) {
+    const adapter = ADAPTERS.find((a) => a.id === id);
+    if (!adapter)
+        return undefined;
+    if (!options.bare)
+        return adapter.register;
+    const at = adapter.register.args.indexOf("--");
+    return { command: adapter.register.command, args: [...adapter.register.args.slice(0, at), ...LAUNCH_BARE] };
 }
 /** The removal that has to precede re-registering an entry this host already has. */
 export function hostUnregistration(id) {
@@ -74,16 +87,60 @@ export function jamEntryLine(stdout) {
 export function entryLauncherVersion(line) {
     return /@jam-mcp\/launcher@([^\s"']+)/.exec(line)?.[1];
 }
+/**
+ * Does this entry run the persistent `jam` executable rather than an npx pin?
+ *
+ * `jam: jam serve`, `jam: /usr/local/bin/jam serve`, `jam: C:\...\jam.cmd serve`
+ * all count; an npx line never does - its command token is `npx`. A bare entry
+ * carries no version in the listing, so its staleness has to be measured from
+ * the executable it would actually run (bareJamVersion), not from the line.
+ */
+export function isBareJamEntry(line) {
+    const command = line.replace(/^\s*jam\s*:?\s*/, "");
+    return /^(?:\S*[\\/])?jam(?:\.cmd|\.exe)?["']?\s+serve\b/i.test(command);
+}
+/**
+ * The version a bare `jam` registration actually runs, measured by asking the
+ * executable itself. `runtime status --json` answers from ~/.jam/config.yaml
+ * and the resolved build - the same resolution the registered entry performs.
+ *
+ * undefined when `jam` is not on PATH or does not answer: a registration that
+ * cannot be measured counts as stale, same as an unreadable pin.
+ */
+export function bareJamVersion(run = defaultHostRunner) {
+    const result = run({ command: "jam", args: ["runtime", "status", "--json"] });
+    if (result.failed || result.status !== 0)
+        return undefined;
+    try {
+        const version = JSON.parse(result.stdout).version;
+        return typeof version === "string" ? version : undefined;
+    }
+    catch {
+        return undefined;
+    }
+}
+/** Is this measured launcher version the one this release registers? */
+export function preferBareRegistration(version) {
+    return version !== undefined && version === EXPECTED_LAUNCHER_VERSION;
+}
 /**
  * Does this entry run the launcher this release registers?
  *
- * Anything else - an older pin, an unpinned spec, a line whose command JAM
- * cannot read - counts as stale. That direction is deliberate: `mcp add`
- * rewrites the same entry, so a needless repair costs one command, while a
- * missed one leaves the agent talking to a server nobody tested it against.
+ * An npx pin answers from the line itself. A bare `jam` line names no version,
+ * so the caller passes what the executable measured (`bareVersion`) - without
+ * it, bare stays stale. Anything else - an older pin, an unpinned spec, a line
+ * whose command JAM cannot read - counts as stale. That direction is
+ * deliberate: `mcp add` rewrites the same entry, so a needless repair costs
+ * one command, while a missed one leaves the agent talking to a server nobody
+ * tested it against.
  */
-export function isEntryStale(line) {
-    return entryLauncherVersion(line) !== EXPECTED_LAUNCHER_VERSION;
+export function isEntryStale(line, bareVersion) {
+    const pinned = entryLauncherVersion(line);
+    if (pinned !== undefined)
+        return pinned !== EXPECTED_LAUNCHER_VERSION;
+    if (isBareJamEntry(line))
+        return bareVersion !== EXPECTED_LAUNCHER_VERSION;
+    return true;
 }
 const EXPECTED_LAUNCHER_VERSION = entryLauncherVersion(JAM_MCP_ENTRY.args.join(" "));
 /**
@@ -107,6 +164,7 @@ export function detectHosts(run = defaultHostRunner) {
 }
 let cachedHosts;
 function probeHosts(run) {
+    let bareCache;
     return ADAPTERS.map((adapter) => {
         const result = run(adapter.probe);
         if (result.failed || result.status !== 0) {
@@ -115,15 +173,23 @@ function probeHosts(run) {
         const line = jamEntryLine(result.stdout);
         if (!line)
             return { id: adapter.id, cliAvailable: true, hasJamEntry: false };
-        const version = entryLauncherVersion(line);
+        // A bare entry's version lives in the executable, not the line - measure
+        // it once, only when a bare entry actually shows up.
+        const bare = isBareJamEntry(line);
+        const version = entryLauncherVersion(line) ?? (bare ? measuredBare() : undefined);
         return {
             id: adapter.id,
             cliAvailable: true,
             hasJamEntry: true,
             ...(version ? { entryVersion: version } : {}),
-            entryStale: isEntryStale(line),
+            ...(bare ? { entryBare: true } : {}),
+            entryStale: isEntryStale(line, version),
         };
     });
+    function measuredBare() {
+        bareCache ??= { version: bareJamVersion(run) };
+        return bareCache.version;
+    }
 }
 /** How a person would do it by hand, for the hosts JAM could not reach. */
 export function describeHostCommand({ command, args }) {

package/dist/bootstrap/mcp-config-merger.d.ts

@@ -13,7 +13,7 @@ import { LAUNCHER_PACKAGE_SPEC } from "@jam-mcp/launcher";
 export { LAUNCHER_PACKAGE_SPEC };
 export declare const JAM_MCP_ENTRY: {
     readonly command: "npx";
-    readonly args: readonly ["--yes", "@jam-mcp/launcher@1.4.1", "serve"];
+    readonly args: readonly ["--yes", "@jam-mcp/launcher@1.4.2", "serve"];
 };
 /**
  * Recognise wiring from before the launcher existed: a hard-coded `node` path

package/dist/bootstrap/setup-plan.js

@@ -1,6 +1,6 @@
 import { join } from "node:path";
 import { CONFIG_RELATIVE_PATH } from "../config/load-config.js";
-import { hostRegistration, hostUnregistration } from "./host-mcp.js";
+import { hostRegistration, preferBareRegistration, hostUnregistration } from "./host-mcp.js";
 import { projectBindingsPath } from "./project-bindings.js";
 import { decideProjectKey } from "./project-config-bootstrapper.js";
 import { portableBootstrapCommand } from "@jam-mcp/launcher";
@@ -161,7 +161,11 @@ function planHostChanges(state) {
         // that pin decides which server, and so which tools, the agent actually gets.
         if (host.hasJamEntry && host.entryStale !== true)
             continue;
-        const registration = hostRegistration(host.id);
+        // A machine whose global `jam` already runs this release gets the
+        // persistent registration; anything else keeps the npx pin fallback.
+        const registration = hostRegistration(host.id, {
+            bare: preferBareRegistration(state.bareLauncher),
+        });
         if (!registration)
             continue;
         const removal = hostUnregistration(host.id);

package/dist/bootstrap/setup-state.d.ts

@@ -50,6 +50,12 @@ export type SetupState = {
      * host, which `jam doctor` has no reason to pay.
      */
     hosts: HostState[];
+    /**
+     * The version the global `jam` executable actually runs, when one answers.
+     * Measured with the hosts (same probe budget); undefined otherwise. This is
+     * what decides whether a repair registers bare `jam` or an npx pin.
+     */
+    bareLauncher?: string;
 };
 export type DetectOptions = {
     cwd?: string;

package/dist/bootstrap/setup-state.js

@@ -1,7 +1,7 @@
 import { readRuntimeConfig, resolveRuntime, } from "@jam-mcp/launcher";
 import { CompositeCredentialProvider } from "../adapters/credentials/composite.js";
 import { loadConfig } from "../config/load-config.js";
-import { detectHosts } from "./host-mcp.js";
+import { bareJamVersion, detectHosts } from "./host-mcp.js";
 import { inspectMcpConfig, isLegacyJamEntry } from "./mcp-config-merger.js";
 import { inspectProjectBindings } from "./project-bindings.js";
 import { resolveProjectRoot } from "./project-root-resolver.js";
@@ -31,6 +31,12 @@ export function detectSetupState(options = {}) {
         project: { ...detectProject(located), ...(binding ? { binding } : {}) },
         mcp: detectMcp(located.root),
         hosts: options.probeHosts ? detectHosts(options.runHost) : [],
+        ...(options.probeHosts
+            ? (() => {
+                const measured = bareJamVersion(options.runHost);
+                return measured !== undefined ? { bareLauncher: measured } : {};
+            })()
+            : {}),
     };
 }
 function detectRuntime(home) {

package/dist/cli/agent-api.js

@@ -161,7 +161,10 @@ async function inspectAxes(state, options) {
     // and asking it means launching that older release. Repair first.
     if (registered.entryStale)
         return axes;
-    const registration = hostRegistration(registered.id);
+    // Launch what is actually registered: a bare entry runs the global `jam`,
+    // an npx pin runs the pinned launcher. Testing the other one would prove
+    // nothing about the entry the agent uses.
+    const registration = hostRegistration(registered.id, { bare: registered.entryBare === true });
     const launch = registration ? launcherArgv(registration.args) : null;
     if (!launch)
         return { ...axes, live: "UNCHECKED", detail: "could not read the registered command" };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jam-mcp/server",
-  "version": "1.4.1",
+  "version": "1.4.2",
   "description": "JAM (Jira Agent MCP) - agent-facing Jira access layer: MCP server, setup core, and CLI",
   "keywords": [
     "jira",
@@ -41,7 +41,7 @@
     "test:watch": "vitest"
   },
   "dependencies": {
-    "@jam-mcp/launcher": "1.4.1",
+    "@jam-mcp/launcher": "1.4.2",
     "@modelcontextprotocol/sdk": "^1.30.0",
     "yaml": "^2.9.0",
     "zod": "^4.4.3"