summer-engine 3.8.0 → 4.2.0

package/dist/installer/skills-refresh.js

@@ -0,0 +1,261 @@
+import { closeSync, cpSync, existsSync, lstatSync, openSync, readdirSync, readFileSync, renameSync, rmSync, statSync, unlinkSync, writeFileSync } from "node:fs";
+import { join } from "node:path";
+import { getSkillRegistry, resolveSkillDir } from "../core/skills-registry.js";
+import { allAgentSpecs, defaultPathContext, resolveSkillPath } from "./agent-table.js";
+import { compareSemver, parseSemver, readKeptSkills, readSkillMarker, SKILL_VERSION_MARKER_FILENAME } from "./version-check.js";
+/**
+ * After an upgrade, bring the Summer skills already installed on this
+ * computer up to this CLI's version. Skills written for 3.x name 3.x tools,
+ * and while their marker says 3.x the MCP server keeps the legacy toolset on
+ * (legacy-toolset.ts), which gives back most of 4.0's smaller tool list.
+ *
+ * Only what Summer installed and nobody changed is replaced, in a skills
+ * folder that carries Summer's `.summer-version` marker older than this CLI:
+ * a real folder (not a link) whose `resource.yaml` says `id: skill/<name>`
+ * (every Summer release ships it; other packs reuse names such as
+ * `brainstorming`), with no file changed after the marker was written, and
+ * not one an earlier refresh already left for that reason (the marker lists
+ * them, so the protection lasts until `summer setup --force`). A copy that
+ * already matches the library counts as current, not as changed. Each
+ * replacement is a copy beside it and two renames, so a kill or a copy error
+ * leaves the old skill in place. One process per folder (a lock file).
+ * Nothing is removed. Missing library skills are added only to a folder that
+ * holds nearly the whole library, so an assets-only install stays one.
+ * Rule-file installs (Cline, Roo Code, old Cursor rules) keep the refresh
+ * command `summer doctor` prints.
+ */
+const LOCK_FILE = ".summer-refresh.lock";
+const STALE_LOCK_MS = 60_000;
+/** Setup writes the marker right after its copy; a file changed later than this was changed by someone. */
+const EDIT_GRACE_MS = 60_000;
+/** A folder holding this share of the bundled library is a full install: new library skills join it. */
+const FULL_INSTALL_SHARE = 0.9;
+const TEMP_PREFIX = ".summer-refresh-";
+const OLD_PREFIX = ".summer-old-";
+/** User-scope skill folders with one folder per skill, each listed once. */
+export function skillFolderLocations() {
+    const ctx = defaultPathContext();
+    const dirs = new Set([join(ctx.home, ".summer", "skills")]);
+    for (const spec of allAgentSpecs()) {
+        const home = resolveSkillPath(spec, "user", ctx);
+        if (home?.kind === "skill-dir")
+            dirs.add(home.path);
+    }
+    return [...dirs];
+}
+const isPrerelease = (version) => /^\d+\.\d+\.\d+-/.test(version.trim());
+/** Refresh when the marker is older, or a pre-release of this release (4.2.0-next.1 -> 4.2.0, never back). */
+function isBehind(marker, cliVersion) {
+    const installed = parseSemver(marker.version);
+    const current = parseSemver(cliVersion);
+    if (!installed || !current)
+        return false;
+    const order = compareSemver(installed, current);
+    return order < 0 || (order === 0 && isPrerelease(marker.version) && !isPrerelease(cliVersion));
+}
+const errorCode = (error) => error?.code ?? (error instanceof Error ? error.message : String(error));
+/** Summer's own copy: a real folder (not a link) whose resource.yaml names this skill. */
+function isSummersSkill(dest, name) {
+    try {
+        if (!lstatSync(dest).isDirectory())
+            return false;
+        const id = readFileSync(join(dest, "resource.yaml"), "utf-8").match(/^id:\s*["']?([^"'\s]+)["']?\s*$/m)?.[1];
+        return id === `skill/${name}`;
+    }
+    catch {
+        return false;
+    }
+}
+/** The folder, or any file in it, changed after `since` (ms); the folder's own time covers a deleted file. Links are not followed. */
+function changedSince(dir, since) {
+    if (lstatSync(dir).mtimeMs > since)
+        return true;
+    for (const entry of readdirSync(dir, { withFileTypes: true })) {
+        const path = join(dir, entry.name);
+        if (lstatSync(path).mtimeMs > since)
+            return true;
+        if (entry.isDirectory() && changedSince(path, since))
+            return true;
+    }
+    return false;
+}
+/** Relative paths of every file under `dir`. */
+function filesUnder(dir, prefix = "") {
+    return readdirSync(dir, { withFileTypes: true }).flatMap((entry) => entry.isDirectory() ? filesUnder(join(dir, entry.name), `${prefix}${entry.name}/`) : [`${prefix}${entry.name}`]);
+}
+/** The installed folder holds exactly the library's files with the library's bytes (a refresh a kill interrupted already wrote it). */
+function sameAsLibrary(dest, src) {
+    try {
+        const installed = filesUnder(dest).sort();
+        const library = filesUnder(src).sort();
+        return (installed.length === library.length &&
+            installed.every((file, index) => file === library[index] && readFileSync(join(dest, file)).equals(readFileSync(join(src, file)))));
+    }
+    catch {
+        return false;
+    }
+}
+/** Copy beside, rename the old one aside, rename the new one in; the old one comes back on any error. */
+function replaceSkill(dir, name, src) {
+    const dest = join(dir, name);
+    const temp = join(dir, `${TEMP_PREFIX}${name}-${process.pid}`);
+    const old = join(dir, `${OLD_PREFIX}${name}-${process.pid}`);
+    rmSync(temp, { recursive: true, force: true });
+    try {
+        cpSync(src, temp, { recursive: true });
+    }
+    catch (error) {
+        rmSync(temp, { recursive: true, force: true });
+        throw error;
+    }
+    const hadOld = existsSync(dest);
+    if (hadOld)
+        renameSync(dest, old);
+    try {
+        renameSync(temp, dest);
+    }
+    catch (error) {
+        if (hadOld)
+            renameSync(old, dest);
+        rmSync(temp, { recursive: true, force: true });
+        throw error;
+    }
+    if (hadOld)
+        rmSync(old, { recursive: true, force: true });
+}
+/** What a killed refresh left: half copies go, a skill renamed aside comes back. */
+function recoverInterrupted(dir) {
+    for (const entry of readdirSync(dir)) {
+        const path = join(dir, entry);
+        if (entry.startsWith(TEMP_PREFIX))
+            rmSync(path, { recursive: true, force: true });
+        if (!entry.startsWith(OLD_PREFIX))
+            continue;
+        const name = entry.slice(OLD_PREFIX.length).replace(/-\d+$/, "");
+        if (existsSync(join(dir, name)))
+            rmSync(path, { recursive: true, force: true });
+        else
+            renameSync(path, join(dir, name));
+    }
+}
+/** The folder's lock, or null while another process holds a fresh one. */
+function takeLock(dir) {
+    const path = join(dir, LOCK_FILE);
+    for (let attempt = 0; attempt < 2; attempt++) {
+        try {
+            closeSync(openSync(path, "wx"));
+            return () => rmSync(path, { force: true });
+        }
+        catch (error) {
+            if (errorCode(error) !== "EEXIST")
+                throw error;
+            try {
+                if (Date.now() - statSync(path).mtimeMs < STALE_LOCK_MS)
+                    return null;
+                unlinkSync(path);
+            }
+            catch {
+                return null;
+            }
+        }
+    }
+    return null;
+}
+function refreshFolder(dir, marker, cliVersion, registry, failures) {
+    const release = takeLock(dir);
+    if (!release)
+        return null;
+    try {
+        recoverInterrupted(dir);
+        // The marker must be writable before anything is copied, or every start would copy again.
+        const markerTemp = join(dir, `${SKILL_VERSION_MARKER_FILENAME}.tmp`);
+        writeFileSync(markerTemp, "", "utf-8");
+        const installedAt = Date.parse(marker.installedAt);
+        const editedAfter = (Number.isFinite(installedAt) ? installedAt : 0) + EDIT_GRACE_MS;
+        const folder = { dir, from: marker.version, to: cliVersion, updated: [], added: [], kept: [] };
+        const keptBefore = new Set(readKeptSkills(dir));
+        const owned = registry.filter((entry) => isSummersSkill(join(dir, entry.name), entry.name));
+        for (const entry of owned) {
+            const dest = join(dir, entry.name);
+            if (sameAsLibrary(dest, resolveSkillDir(entry))) {
+                folder.updated.push(entry.name);
+                continue;
+            }
+            if (keptBefore.has(entry.name) || changedSince(dest, editedAfter)) {
+                folder.kept.push(entry.name);
+                continue;
+            }
+            try {
+                replaceSkill(dir, entry.name, resolveSkillDir(entry));
+                folder.updated.push(entry.name);
+            }
+            catch (error) {
+                failures.push({ dir, skill: entry.name, code: errorCode(error) });
+            }
+        }
+        const bundled = registry.filter((entry) => entry.bundled !== false && entry.status !== "deprecated");
+        if (bundled.length > 0 && owned.length >= FULL_INSTALL_SHARE * bundled.length) {
+            for (const entry of bundled) {
+                if (existsSync(join(dir, entry.name)))
+                    continue;
+                try {
+                    replaceSkill(dir, entry.name, resolveSkillDir(entry));
+                    folder.added.push(entry.name);
+                }
+                catch (error) {
+                    failures.push({ dir, skill: entry.name, code: errorCode(error) });
+                }
+            }
+        }
+        // Written last, through a rename: a kill before this leaves the old marker, and the next start finishes.
+        const next = { version: cliVersion, installedAt: new Date().toISOString(), ...(folder.kept.length > 0 ? { kept: folder.kept } : {}) };
+        writeFileSync(markerTemp, JSON.stringify(next, null, 2) + "\n", "utf-8");
+        renameSync(markerTemp, join(dir, SKILL_VERSION_MARKER_FILENAME));
+        return folder;
+    }
+    finally {
+        release();
+    }
+}
+export function refreshInstalledSkills(cliVersion, options = {}) {
+    const result = { refreshed: [], failures: [] };
+    if (!parseSemver(cliVersion))
+        return result;
+    for (const dir of options.dirs ?? skillFolderLocations()) {
+        const marker = readSkillMarker(dir);
+        if (!marker || !isBehind(marker, cliVersion))
+            continue;
+        try {
+            const folder = refreshFolder(dir, marker, cliVersion, options.registry ?? getSkillRegistry(), result.failures);
+            if (folder)
+                result.refreshed.push(folder);
+        }
+        catch (error) {
+            // A folder we cannot lock or write keeps its skills and its marker; doctor still names the refresh.
+            result.failures.push({ dir, code: errorCode(error) });
+        }
+    }
+    return result;
+}
+const listed = (names) => names.length > 5 ? `${names.slice(0, 5).join(", ")} and ${names.length - 5} more` : names.join(", ");
+/** The one message a person and their agent see after a refresh, or null when nothing changed. */
+export function refreshNotice(refreshed, failures = []) {
+    const skills = new Set(refreshed.flatMap((folder) => [...folder.updated, ...folder.added])).size;
+    const kept = [...new Set(refreshed.flatMap((folder) => folder.kept))];
+    const failed = [...new Set(failures.flatMap((failure) => (failure.skill ? [failure.skill] : [])))];
+    if (skills === 0 && kept.length === 0)
+        return null;
+    const changedFolders = refreshed.filter((folder) => folder.updated.length + folder.added.length > 0).length;
+    const from = [...new Set(refreshed.map((folder) => folder.from))].join(", ");
+    const to = refreshed[0].to;
+    return [
+        skills > 0 ? `Summer updated its ${skills} installed skills (${changedFolders === 1 ? "1 folder" : `${changedFolders} folders`}) from ${from} to ${to}.` : "",
+        kept.length > 0
+            ? `It left ${kept.length} Summer skills you changed as they are (${listed(kept)}); summer setup <agent> --force replaces them.`
+            : "",
+        failed.length > 0 ? `${failed.length} could not be updated (${listed(failed)}); summer setup <agent> --force retries them.` : "",
+        "3.x tool names still run through summer_call; summer mcp --toolsets legacy lists them again.",
+    ]
+        .filter(Boolean)
+        .join(" ");
+}

package/dist/installer/version-check.d.ts

@@ -52,6 +52,15 @@ export interface RegistryFetchFailure {
     message?: string;
 }
 export declare function fetchLatestRegistryVersion(options?: RegistryFetchOptions): Promise<RegistryFetchResult | RegistryFetchFailure>;
+export interface RegistryDistTagsResult {
+    ok: true;
+    /** dist-tag -> version, e.g. { latest: "3.8.0", next: "3.1.0" }. */
+    tags: Record<string, string>;
+}
+export declare function fetchRegistryDistTags(options?: RegistryFetchOptions): Promise<RegistryDistTagsResult | RegistryFetchFailure>;
+/** Whether npm has this exact version: true, false (registry says 404), or
+ *  null when the registry could not be asked. */
+export declare function isRegistryVersionPublished(version: string, options?: RegistryFetchOptions): Promise<boolean | null>;
 /**
  * Build the `cli-version-current` doctor check. Pure helper so tests can drive
  * registry responses through `fetchImpl` rather than touching the network.
@@ -70,9 +79,18 @@ export declare const SKILL_VERSION_MARKER_FILENAME = ".summer-version";
 export interface SkillVersionMarker {
     version: string;
     installedAt: string;
+    /** Summer skills the upgrade refresh left because someone changed them (installer/skills-refresh.ts). */
+    kept?: string[];
 }
 export declare function readSkillMarker(dir: string): SkillVersionMarker | null;
-export declare function writeSkillMarker(dir: string, version: string, now?: Date): string;
+/** The skills the upgrade refresh left in this folder (the raw marker; readSkillMarker keeps setup's fields). */
+export declare function readKeptSkills(dir: string): string[];
+/**
+ * Write the folder's marker. The skills an upgrade refresh left stay listed
+ * (so later refreshes keep leaving them), except the ones this install just
+ * wrote: `installed` names them; leave it out to keep every one.
+ */
+export declare function writeSkillMarker(dir: string, version: string, now?: Date, installed?: readonly string[]): string;
 /**
  * Candidate locations for skill-version markers. Listed roughly in order of
  * how likely a Summer user is to have them populated. We probe every candidate,
@@ -99,17 +117,30 @@ export interface RecordedInstall {
     /** Where the answer came from: the agent's config file, or — when it has
      *  no readable Summer entry — the nature of the CLI running this check. */
     source: "agent-config" | "running-cli";
+    /** The agent's config lists no summer-engine server in any shape (`setup
+     *  --cli`, or skills installed alone) and `setup --cli` can set the agent
+     *  up: the refresh keeps it without an MCP server. */
+    cliOnly?: boolean;
+}
+/** An npm dist-tag (`latest`, `next`) or an exact version (`3.9.1`). */
+export type RefreshChannel = string;
+/** The npm spec a refresh command or a new MCP entry should name, so the agent
+ *  never runs an older release than this CLI (release review P1-3). `latest`
+ *  unless this CLI is AHEAD of npm latest; then `next` only when the next
+ *  dist-tag is this exact version, else this exact version. A CLI that is ahead
+ *  is not always on `next`: `next` stayed on 3.1.0 while latest moved to 3.8.0,
+ *  and pinning it dropped every option newer than 3.1.0 (SUMMER_PROFILE). */
+export declare function refreshChannel(installedVersion: string, latestVersion: string | undefined, nextVersion?: string): RefreshChannel;
+export interface DefaultChannelDeps {
+    fetchDistTags?: typeof fetchRegistryDistTags;
+    isPublished?: typeof isRegistryVersionPublished;
 }
-export type RefreshChannel = "latest" | "next";
-/** The npm dist-tag a refresh command should name. A CLI that is AHEAD of
- *  npm `latest` was installed from `next` (a soaking release); telling that
- *  user to run `summer-engine@latest` would silently downgrade the MCP server
- *  while leaving the newer skills in place (release review P1-3). */
-export declare function refreshChannel(installedVersion: string, latestVersion: string | undefined): RefreshChannel;
 /** Default `summer setup` channel when neither --channel nor SUMMER_CHANNEL is
- *  given: `next` while this CLI is ahead of npm latest, else `latest`. Network
- *  failure means `latest` (never blocks setup). */
-export declare function resolveDefaultChannel(installedVersion: string): Promise<{
+ *  given: `latest`, unless this CLI is ahead of npm latest. Then `next` when
+ *  next is this exact version, this exact version when npm has it, and
+ *  otherwise (a checkout that was never published) `latest` with a note that
+ *  points at --local-dev. Network failure means `latest` (never blocks setup). */
+export declare function resolveDefaultChannel(installedVersion: string, deps?: DefaultChannelDeps): Promise<{
     channel: RefreshChannel;
     note?: string;
 }>;
@@ -118,6 +149,8 @@ export interface SkillsVersionCheckInput {
     candidates: SkillMarkerCandidate[];
     /** npm latest, when known, so the refresh command names the right dist-tag. */
     latestRegistryVersion?: string;
+    /** npm next, when known: the refresh names `next` only when it is this CLI. */
+    nextRegistryVersion?: string;
     /** Resolves how the stale agent's MCP entry was recorded; defaults to
      *  detectRecordedInstall. Seam for tests. */
     recordedInstall?: (agent: string) => Promise<RecordedInstall | null> | RecordedInstall | null;
@@ -133,7 +166,8 @@ export declare function buildSkillsVersionCheck(input: SkillsVersionCheckInput):
  * The command that refreshes an agent's skills without changing how its MCP
  * entry was recorded: the local-dev form re-links the same checkout
  * (`setup --local-dev --force` re-copies the checkout's skills); the npx form
- * is for installs that already run the published package.
+ * is for installs that already run the published package. An agent with no
+ * summer-engine entry refreshes with `--cli`, so the refresh adds no MCP server.
  */
 export declare function skillsRefreshCommand(agent: string, install: RecordedInstall | null, channel?: RefreshChannel): string;
 /** The shape `summer setup --local-dev` writes: `node <…>/dist/bin/summer.js mcp`
@@ -153,6 +187,12 @@ export declare function isLocalDevServerConfig(server: StdioMcpServerConfig): bo
  * Returns null when the file is missing, unparsable, or has no Summer entry.
  */
 export declare function readRecordedMcpServer(text: string, format: McpFormat): StdioMcpServerConfig | null;
+/**
+ * Does the agent's config list a summer-engine server in any shape: the stdio
+ * entry setup writes, a URL entry under that name, or (Claude Code) a
+ * per-project entry under projects.<dir>.mcpServers?
+ */
+export declare function listsSummerEngine(text: string, format: McpFormat): boolean;
 /** Is the CLI running this check a source checkout rather than an installed
  *  package? npm/npx/global installs all live under a node_modules directory. */
 export declare function runningCliIsCheckout(packageRoot?: string): boolean;

package/dist/installer/version-check.js

@@ -13,11 +13,13 @@
  */
 import { existsSync, readFileSync, writeFileSync } from "node:fs";
 import { parse as parseYaml } from "yaml";
-import { allAgentSpecs, defaultPathContext, resolveSkillPath } from "./agent-table.js";
+import { agentSpec, allAgentSpecs, cliSetupRefusal, defaultPathContext, resolveSkillPath } from "./agent-table.js";
 import { join, sep } from "node:path";
 import { PACKAGE_ROOT } from "../core/package-root.js";
 import { configureAgentMcp, parseAgent, SUMMER_MCP_SERVER_NAME, topLevelKey, } from "./agent-config.js";
 const NPM_REGISTRY_LATEST_URL = "https://registry.npmjs.org/summer-engine/latest";
+const NPM_REGISTRY_DIST_TAGS_URL = "https://registry.npmjs.org/-/package/summer-engine/dist-tags";
+const NPM_REGISTRY_PACKAGE_URL = "https://registry.npmjs.org/summer-engine";
 const REGISTRY_FETCH_TIMEOUT_MS = 3000;
 export function parseSemver(version) {
     if (!version)
@@ -69,8 +71,8 @@ export function classifyDrift(installed, target) {
     }
     return { severity: "warning", reason: "minor-drift" };
 }
-export async function fetchLatestRegistryVersion(options = {}) {
-    const url = options.url ?? NPM_REGISTRY_LATEST_URL;
+async function fetchRegistryJson(defaultUrl, options) {
+    const url = options.url ?? defaultUrl;
     const timeoutMs = options.timeoutMs ?? REGISTRY_FETCH_TIMEOUT_MS;
     const fetchImpl = options.fetchImpl ?? globalThis.fetch;
     if (!fetchImpl) {
@@ -88,15 +90,7 @@ export async function fetchLatestRegistryVersion(options = {}) {
                 message: `registry responded ${response.status}`,
             };
         }
-        const body = (await response.json());
-        if (typeof body.version !== "string") {
-            return {
-                ok: false,
-                reason: "registry-unreachable",
-                message: "registry payload missing version",
-            };
-        }
-        return { ok: true, version: body.version };
+        return { ok: true, body: await response.json() };
     }
     catch (error) {
         return {
@@ -106,6 +100,43 @@ export async function fetchLatestRegistryVersion(options = {}) {
         };
     }
 }
+export async function fetchLatestRegistryVersion(options = {}) {
+    const result = await fetchRegistryJson(NPM_REGISTRY_LATEST_URL, options);
+    if (!result.ok)
+        return result;
+    const body = result.body;
+    if (typeof body?.version !== "string") {
+        return {
+            ok: false,
+            reason: "registry-unreachable",
+            message: "registry payload missing version",
+        };
+    }
+    return { ok: true, version: body.version };
+}
+export async function fetchRegistryDistTags(options = {}) {
+    const result = await fetchRegistryJson(NPM_REGISTRY_DIST_TAGS_URL, options);
+    if (!result.ok)
+        return result;
+    const body = result.body;
+    const tags = {};
+    for (const [tag, value] of Object.entries(body ?? {})) {
+        if (typeof value === "string")
+            tags[tag] = value;
+    }
+    if (!tags.latest) {
+        return { ok: false, reason: "registry-unreachable", message: "registry payload missing latest" };
+    }
+    return { ok: true, tags };
+}
+/** Whether npm has this exact version: true, false (registry says 404), or
+ *  null when the registry could not be asked. */
+export async function isRegistryVersionPublished(version, options = {}) {
+    const result = await fetchRegistryJson(`${NPM_REGISTRY_PACKAGE_URL}/${encodeURIComponent(version)}`, options);
+    if (result.ok)
+        return true;
+    return result.message === "registry responded 404" ? false : null;
+}
 const RECOMMENDED_REFRESH_COMMAND = "npx clear-npx-cache && npx -y summer-engine@latest setup <agent> --yes --force";
 export function buildCliVersionCheck(input) {
     if (!input.registry.ok) {
@@ -188,11 +219,28 @@ export function readSkillMarker(dir) {
         return null;
     }
 }
-export function writeSkillMarker(dir, version, now = new Date()) {
+/** The skills the upgrade refresh left in this folder (the raw marker; readSkillMarker keeps setup's fields). */
+export function readKeptSkills(dir) {
+    try {
+        const kept = JSON.parse(readFileSync(join(dir, SKILL_VERSION_MARKER_FILENAME), "utf-8")).kept;
+        return Array.isArray(kept) ? kept.filter((name) => typeof name === "string") : [];
+    }
+    catch {
+        return [];
+    }
+}
+/**
+ * Write the folder's marker. The skills an upgrade refresh left stay listed
+ * (so later refreshes keep leaving them), except the ones this install just
+ * wrote: `installed` names them; leave it out to keep every one.
+ */
+export function writeSkillMarker(dir, version, now = new Date(), installed) {
     const path = join(dir, SKILL_VERSION_MARKER_FILENAME);
+    const kept = readKeptSkills(dir).filter((name) => !installed?.includes(name));
     const payload = {
         version,
         installedAt: now.toISOString(),
+        ...(kept.length > 0 ? { kept } : {}),
     };
     writeFileSync(path, JSON.stringify(payload, null, 2) + "\n", "utf-8");
     return path;
@@ -212,28 +260,51 @@ export function defaultSkillMarkerCandidates() {
     }
     return candidates;
 }
-/** The npm dist-tag a refresh command should name. A CLI that is AHEAD of
- *  npm `latest` was installed from `next` (a soaking release); telling that
- *  user to run `summer-engine@latest` would silently downgrade the MCP server
- *  while leaving the newer skills in place (release review P1-3). */
-export function refreshChannel(installedVersion, latestVersion) {
+/** The npm spec a refresh command or a new MCP entry should name, so the agent
+ *  never runs an older release than this CLI (release review P1-3). `latest`
+ *  unless this CLI is AHEAD of npm latest; then `next` only when the next
+ *  dist-tag is this exact version, else this exact version. A CLI that is ahead
+ *  is not always on `next`: `next` stayed on 3.1.0 while latest moved to 3.8.0,
+ *  and pinning it dropped every option newer than 3.1.0 (SUMMER_PROFILE). */
+export function refreshChannel(installedVersion, latestVersion, nextVersion) {
     const installed = parseSemver(installedVersion);
     const latest = latestVersion ? parseSemver(latestVersion) : null;
     if (!installed || !latest)
         return "latest";
-    return classifyDrift(installed, latest).reason === "ahead" ? "next" : "latest";
+    if (classifyDrift(installed, latest).reason !== "ahead")
+        return "latest";
+    return nextVersion?.trim() === installedVersion.trim() ? "next" : installedVersion.trim();
 }
 /** Default `summer setup` channel when neither --channel nor SUMMER_CHANNEL is
- *  given: `next` while this CLI is ahead of npm latest, else `latest`. Network
- *  failure means `latest` (never blocks setup). */
-export async function resolveDefaultChannel(installedVersion) {
-    const registry = await fetchLatestRegistryVersion();
+ *  given: `latest`, unless this CLI is ahead of npm latest. Then `next` when
+ *  next is this exact version, this exact version when npm has it, and
+ *  otherwise (a checkout that was never published) `latest` with a note that
+ *  points at --local-dev. Network failure means `latest` (never blocks setup). */
+export async function resolveDefaultChannel(installedVersion, deps = {}) {
+    const registry = await (deps.fetchDistTags ?? fetchRegistryDistTags)();
     if (!registry.ok)
         return { channel: "latest" };
-    const channel = refreshChannel(installedVersion, registry.version);
-    return channel === "next"
-        ? { channel, note: `This CLI (${installedVersion}) is ahead of npm latest (${registry.version}); the MCP entry is pinned to summer-engine@next so the agent keeps running this release. Pass --channel latest to override.` }
-        : { channel };
+    const latest = registry.tags.latest;
+    const channel = refreshChannel(installedVersion, latest, registry.tags.next);
+    if (channel === "latest")
+        return { channel };
+    if (channel === "next") {
+        return {
+            channel,
+            note: `This CLI (${installedVersion}) is the npm next release, ahead of latest (${latest}); the MCP entry runs summer-engine@next. Pass --channel latest to override.`,
+        };
+    }
+    const published = await (deps.isPublished ?? isRegistryVersionPublished)(installedVersion);
+    if (published) {
+        return {
+            channel,
+            note: `This CLI (${installedVersion}) is ahead of npm latest (${latest}); the MCP entry is pinned to summer-engine@${installedVersion} so the agent runs this release. Pass --channel latest to override.`,
+        };
+    }
+    return {
+        channel: "latest",
+        note: `This CLI (${installedVersion}) is not on npm, so the MCP entry runs summer-engine@latest (${latest}), which can lack options this CLI has. To run this build, pass --local-dev.`,
+    };
 }
 export async function buildSkillsVersionCheck(input) {
     const cliParsed = parseSemver(input.installedCliVersion);
@@ -306,7 +377,7 @@ export async function buildSkillsVersionCheck(input) {
             ...baseDetails,
             drift: worst.drift.reason,
             ...(install ? { install } : {}),
-            recommendedAction: skillsRefreshCommand(worst.agent, install, refreshChannel(input.installedCliVersion, input.latestRegistryVersion)),
+            recommendedAction: skillsRefreshCommand(worst.agent, install, refreshChannel(input.installedCliVersion, input.latestRegistryVersion, input.nextRegistryVersion)),
         },
     };
 }
@@ -324,14 +395,20 @@ async function resolveRecordedInstall(resolver, agent) {
  * The command that refreshes an agent's skills without changing how its MCP
  * entry was recorded: the local-dev form re-links the same checkout
  * (`setup --local-dev --force` re-copies the checkout's skills); the npx form
- * is for installs that already run the published package.
+ * is for installs that already run the published package. An agent with no
+ * summer-engine entry refreshes with `--cli`, so the refresh adds no MCP server.
  */
 export function skillsRefreshCommand(agent, install, channel = "latest") {
+    if (install?.cliOnly) {
+        // No MCP entry to keep: --local-dev and --channel shape only that entry.
+        return install.localDev && install.cliPath
+            ? `node ${install.cliPath} setup ${agent} --cli --yes --force`
+            : `npx clear-npx-cache && npx -y summer-engine@${channel} setup ${agent} --cli --yes --force`;
+    }
     if (install?.localDev && install.cliPath) {
         return `node ${install.cliPath} setup ${agent} --local-dev --yes --force`;
     }
-    const tag = channel === "next" ? "next" : "latest";
-    return `npx clear-npx-cache && npx -y summer-engine@${tag} setup ${agent} --yes --force${channel === "next" ? " --channel next" : ""}`;
+    return `npx clear-npx-cache && npx -y summer-engine@${channel} setup ${agent} --yes --force${channel === "latest" ? "" : ` --channel ${channel}`}`;
 }
 /** The shape `summer setup --local-dev` writes: `node <…>/dist/bin/summer.js mcp`
  *  (createSummerMcpServerConfig(true)). */
@@ -420,6 +497,37 @@ export function readRecordedMcpServer(text, format) {
     }
     return null;
 }
+/**
+ * Does the agent's config list a summer-engine server in any shape: the stdio
+ * entry setup writes, a URL entry under that name, or (Claude Code) a
+ * per-project entry under projects.<dir>.mcpServers?
+ */
+export function listsSummerEngine(text, format) {
+    if (readRecordedMcpServer(text, format))
+        return true;
+    if (!text.trim())
+        return false;
+    if (format === "toml")
+        return new RegExp(`^\\s*\\[mcp_servers\\.(?:"${SUMMER_MCP_SERVER_NAME}"|${SUMMER_MCP_SERVER_NAME})[\\].]`, "m").test(text);
+    if (format === "toml-array")
+        return new RegExp(`^\\s*name\\s*=\\s*"${SUMMER_MCP_SERVER_NAME}"`, "m").test(text);
+    let parsed;
+    try {
+        parsed = format === "yaml-goose" || format === "yaml-hermes" ? parseYaml(text) : JSON.parse(text);
+    }
+    catch {
+        return false;
+    }
+    if (!parsed || typeof parsed !== "object")
+        return false;
+    const root = parsed;
+    const projects = root.projects && typeof root.projects === "object" ? Object.values(root.projects) : [];
+    const maps = [
+        ...[topLevelKey(format), "mcpServers", "servers", "mcp", "context_servers", "extensions", "mcp_servers", "amp.mcpServers"].map((key) => root[key]),
+        ...projects.map((project) => (project && typeof project === "object" ? project.mcpServers : undefined)),
+    ];
+    return maps.some((map) => Boolean(map) && typeof map === "object" && SUMMER_MCP_SERVER_NAME in map);
+}
 /** Is the CLI running this check a source checkout rather than an installed
  *  package? npm/npx/global installs all live under a node_modules directory. */
 export function runningCliIsCheckout(packageRoot = PACKAGE_ROOT) {
@@ -435,16 +543,17 @@ export function runningCliIsCheckout(packageRoot = PACKAGE_ROOT) {
  */
 export async function detectRecordedInstall(agent) {
     const supported = parseAgent(agent);
+    let cliOnly = false;
     if (supported) {
         try {
             const target = await configureAgentMcp({ agent: supported, scope: "user", print: true });
-            if (existsSync(target.path)) {
-                const recorded = readRecordedMcpServer(readFileSync(target.path, "utf-8"), target.format);
-                if (recorded) {
-                    const localDev = isLocalDevServerConfig(recorded);
-                    return { localDev, cliPath: localDev ? recorded.args[0] ?? null : null, source: "agent-config" };
-                }
+            const text = existsSync(target.path) ? readFileSync(target.path, "utf-8") : "";
+            const recorded = readRecordedMcpServer(text, target.format);
+            if (recorded) {
+                const localDev = isLocalDevServerConfig(recorded);
+                return { localDev, cliPath: localDev ? recorded.args[0] ?? null : null, source: "agent-config" };
             }
+            cliOnly = !listsSummerEngine(text, target.format) && cliSetupRefusal(agentSpec(supported)) === null;
         }
         catch {
             // Unreadable config: fall through to the running-CLI answer.
@@ -455,6 +564,7 @@ export async function detectRecordedInstall(agent) {
         localDev: checkout,
         cliPath: checkout ? [PACKAGE_ROOT, "dist", "bin", "summer.js"].join(sep) : null,
         source: "running-cli",
+        ...(cliOnly ? { cliOnly } : {}),
     };
 }
 export function buildBootDriftNotice(installedVersion, registry, agent) {