ninepm 0.51.0 → 0.52.0

package/dist/index.d.ts

@@ -114,7 +114,7 @@ export interface CreatedServiceToken {
     clientSecret: string;
 }
 export type FetchLike = (input: string | URL | Request, init?: RequestInit) => Promise<Response>;
-export declare const CLI_VERSION = "0.51.0";
+export declare const CLI_VERSION = "0.52.0";
 type CredentialBackend = "macos-keychain" | "windows-dpapi" | "linux-secret-service" | "file-plaintext" | "none";
 export interface StoredCredentialEntry {
     token: string;
@@ -291,6 +291,17 @@ export interface UpdateNudgeOptions {
     env?: NodeJS.ProcessEnv;
 }
 export declare function deployUpdateNudge(options?: UpdateNudgeOptions): Promise<string | null>;
+export interface SkillFreshness {
+    status: "up_to_date" | "stale" | "not_installed" | "unknown";
+    installedStamp?: string;
+    publishedStamp?: string;
+    detail?: string;
+}
+export declare function checkSkillFreshness(options?: {
+    fetchFn?: FetchLike;
+    skillPath?: string;
+    timeoutMs?: number;
+}): Promise<SkillFreshness>;
 export interface ListSitesResult {
     projects: Project[];
 }
@@ -360,6 +371,8 @@ export interface DeviceAuthProgressReporter {
     stop(): void;
 }
 export declare function createDeviceAuthProgressReporter(stream?: NodeJS.WriteStream): DeviceAuthProgressReporter;
+export declare function noteCliOutdatedAdvisory(response: Response, log?: (line: string) => void): void;
+export declare function resetCliOutdatedAdvisoryForTests(): void;
 type CliCommand = {
     kind: "deploy";
     options: DeployOptions;

package/dist/index.js

@@ -14,7 +14,7 @@ import { MAX_DEPLOY_DESCRIPTION_LENGTH, normalizeDeployDescription } from "./dep
 import { MAX_STATIC_ASSET_FILE_BYTES, MAX_WORKER_ENTRY_COMPRESSED_BYTES, MAX_WORKER_SCRIPT_COMPRESSED_BYTES, WORKER_ENTRY_COMPRESSED_WARN_BYTES, assetFileTooLargeMessage, formatMebibytes, formatMegabytes, oversizedAssets, workerBundleNearLimitMessage, workerBundleTooLargeMessage, workerEntryNearLimitMessage, workerEntryTooLargeMessage, } from "./deploy-limits.js";
 import { buildFileManifest, readManifestFile } from "./manifest.js";
 import { API_KEY_HELP, API_URL_HELP, CLI_CLIENT_ID, DEFAULT_API_URL, reportCliError } from "./cli-config.js";
-export const CLI_VERSION = "0.51.0";
+export const CLI_VERSION = "0.52.0";
 const PUBLIC_DASHBOARD_URL = "https://9pm.ai/dashboard";
 const TOKEN_SERVICE = "9pm";
 const TOKEN_ACCOUNT = "default";
@@ -499,6 +499,7 @@ async function runDeploy(initialOptions, slugHolder) {
             method: upload.method,
             headers: {
                 authorization: `Bearer ${options.token}`,
+                ...cliVersionHeader(),
                 ...upload.headers,
             },
             body: arrayBufferFromBytes(body),
@@ -856,8 +857,9 @@ async function doctor(options) {
     catch (error) {
         versionError = error instanceof Error ? error.message : String(error);
     }
+    const skill = await checkSkillFreshness();
     if (options.json) {
-        console.log(JSON.stringify({ apiUrl: normalizeApiUrl(options.apiUrl), credentials, account, accountError, version: versionCheck, versionError }, null, 2));
+        console.log(JSON.stringify({ apiUrl: normalizeApiUrl(options.apiUrl), credentials, account, accountError, version: versionCheck, versionError, skill }, null, 2));
         return;
     }
     console.log("9pm.ai doctor");
@@ -898,6 +900,18 @@ async function doctor(options) {
     else if (versionError) {
         console.log(`CLI: ${CLI_VERSION} (latest unavailable: ${versionError})`);
     }
+    if (skill.status === "up_to_date") {
+        console.log(`Skill: up to date (stamp ${skill.installedStamp})`);
+    }
+    else if (skill.status === "stale") {
+        console.log(`Skill: stale (installed ${skill.installedStamp ?? "no stamp"}, published ${skill.publishedStamp}). Refresh: ${SKILL_REFRESH_COMMAND}`);
+    }
+    else if (skill.status === "not_installed") {
+        console.log("Skill: not installed (expected at ~/.agents/skills/9pm/SKILL.md)");
+    }
+    else {
+        console.log(`Skill: installed; freshness unknown (${skill.detail ?? "no detail"})`);
+    }
 }
 export async function getSiteAccess(options) {
     return apiRequest(options, `/v1/sites/${encodeURIComponent(options.site)}/access`, {
@@ -1071,6 +1085,56 @@ export async function deployUpdateNudge(options = {}) {
     }
     return `Update available: 9pm CLI ${CLI_VERSION} -> ${latest}. Run: npm i -g ninepm@latest`;
 }
+// Skill-freshness half of `9pm doctor` (#996): compare the installed skill's
+// stamp line against the public source copy — the same reference SKILL.md's
+// own "Keeping this skill fresh" section names, and the source the install
+// command actually pulls from (the platform-hosted copy tracks releases and
+// can lag it). Never throws: doctor is diagnostics, not a gate.
+const SKILL_INSTALL_PATH_SEGMENTS = [".agents", "skills", "9pm", "SKILL.md"];
+const SKILL_SOURCE_URL = "https://raw.githubusercontent.com/tmrw-realityos/9pm-skill/main/skills/9pm/SKILL.md";
+const SKILL_STAMP_PATTERN = /^Skill guide stamp:\s*(\S+)/m;
+const SKILL_REFRESH_COMMAND = "npx skills add tmrw-realityos/9pm-skill --skill 9pm -g";
+export async function checkSkillFreshness(options = {}) {
+    const skillPath = options.skillPath ?? path.join(os.homedir(), ...SKILL_INSTALL_PATH_SEGMENTS);
+    let installed;
+    try {
+        installed = readFileSync(skillPath, "utf8");
+    }
+    catch {
+        return { status: "not_installed" };
+    }
+    const installedStamp = SKILL_STAMP_PATTERN.exec(installed)?.[1] ?? null;
+    const fetchFn = options.fetchFn ?? fetch;
+    const controller = new AbortController();
+    const timer = setTimeout(() => controller.abort(), options.timeoutMs ?? 3000);
+    let publishedStamp = null;
+    try {
+        const response = await fetchFn(SKILL_SOURCE_URL, { signal: controller.signal });
+        if (!response.ok) {
+            throw new Error(`skill source returned ${response.status}`);
+        }
+        publishedStamp = SKILL_STAMP_PATTERN.exec(await response.text())?.[1] ?? null;
+    }
+    catch (error) {
+        return {
+            status: "unknown",
+            ...(installedStamp ? { installedStamp } : {}),
+            detail: error instanceof Error ? error.message : String(error),
+        };
+    }
+    finally {
+        clearTimeout(timer);
+    }
+    if (!publishedStamp) {
+        return { status: "unknown", ...(installedStamp ? { installedStamp } : {}), detail: "published copy has no stamp line" };
+    }
+    // An installed copy with no stamp predates the stamp convention, which is
+    // itself proof it is behind.
+    if (!installedStamp || installedStamp !== publishedStamp) {
+        return { status: "stale", ...(installedStamp ? { installedStamp } : {}), publishedStamp };
+    }
+    return { status: "up_to_date", installedStamp, publishedStamp };
+}
 // Best-effort read of the update-check cache; the caller judges freshness.
 // checkedAtMs is null when the file is missing or malformed (so the next probe
 // runs now), and latest is null when the last attempt failed to get an answer
@@ -1114,7 +1178,7 @@ async function deviceAuthFlow(options) {
     // made here; an unpredictable store failure is caught after the poll instead.
     const startResponse = await fetchFn(`${apiUrl}/v1/oauth/device/start`, {
         method: "POST",
-        headers: { "content-type": "application/json" },
+        headers: { "content-type": "application/json", ...cliVersionHeader() },
         body: JSON.stringify({ machineName: options.machineName ?? os.hostname(), purpose: options.purpose }),
     });
     // The control plane refuses a signup it knows can never be approved (#507) with
@@ -1180,7 +1244,7 @@ async function deviceAuthFlow(options) {
             await sleep(intervalMs);
             const pollResponse = await fetchFn(`${apiUrl}/v1/oauth/device/poll`, {
                 method: "POST",
-                headers: { "content-type": "application/json" },
+                headers: { "content-type": "application/json", ...cliVersionHeader() },
                 body: JSON.stringify({ code: deviceCode }),
             });
             if (pollResponse.status === 400) {
@@ -1349,6 +1413,7 @@ export async function putManagedFile(options) {
         headers: {
             authorization: `Bearer ${options.token}`,
             "content-type": "application/octet-stream",
+            ...cliVersionHeader(),
         },
         body: bytes,
     });
@@ -1359,7 +1424,7 @@ export async function getManagedFile(options) {
     const apiUrl = normalizeApiUrl(options.apiUrl);
     const response = await fetchFn(`${apiUrl}${filesRoute(options)}/${encodeURIComponent(options.fileId)}`, {
         method: "GET",
-        headers: { authorization: `Bearer ${options.token}` },
+        headers: { authorization: `Bearer ${options.token}`, ...cliVersionHeader() },
     });
     await expectOk(response, "download file");
     const bytes = new Uint8Array(await response.arrayBuffer());
@@ -1432,7 +1497,7 @@ async function finalizeDeployment(fetchFn, apiUrl, token, deploymentId) {
 async function getDeployment(fetchFn, apiUrl, token, deploymentId) {
     const response = await fetchFn(`${apiUrl}/v1/deployments/${encodeURIComponent(deploymentId)}`, {
         method: "GET",
-        headers: { authorization: `Bearer ${token}` },
+        headers: { authorization: `Bearer ${token}`, ...cliVersionHeader() },
     });
     const payload = await expectJson(response, "get deployment");
     return payload.deployment;
@@ -1869,20 +1934,56 @@ function jsonHeaders(token) {
     return {
         authorization: `Bearer ${token}`,
         "content-type": "application/json",
-        "x-9pm-cli-version": CLI_VERSION,
+        ...cliVersionHeader(),
     };
 }
+// Every control-plane request must carry the CLI's version (#996): the server's
+// version floor only judges requests that have it, so a hand-built header object
+// that omits it silently exempts that call from both the advisory and an
+// enforced floor. Spread this into any headers literal that bypasses
+// apiRequest/jsonHeaders.
+function cliVersionHeader() {
+    return { "x-9pm-cli-version": CLI_VERSION };
+}
 async function expectJson(response, action) {
     await expectOk(response, action);
     return (await response.json());
 }
 async function expectOk(response, action) {
+    noteCliOutdatedAdvisory(response);
     if (response.ok) {
         return;
     }
+    // 426 is the control plane's enforced CLI floor (#996): the server-written
+    // message IS the response — surface it alone rather than burying it in the
+    // raw envelope the generic path prints.
+    if (response.status === 426) {
+        throw new Error(await apiErrorMessage(response, action));
+    }
     const text = await response.text();
     throw new Error(`Failed to ${action}: ${response.status} ${text}`);
 }
+// Advisory half of the control plane's CLI floor (#996): when a response says
+// this version is below the platform's recommended minimum, say so once per
+// invocation, on stderr so command output (including --json) stays parseable.
+const CLI_OUTDATED_HEADER = "x-9pm-cli-outdated";
+let cliOutdatedAdvisoryPrinted = false;
+export function noteCliOutdatedAdvisory(response, log = (line) => console.error(line)) {
+    if (cliOutdatedAdvisoryPrinted) {
+        return;
+    }
+    const min = response.headers.get(CLI_OUTDATED_HEADER);
+    if (!min) {
+        return;
+    }
+    cliOutdatedAdvisoryPrinted = true;
+    log(`Note: this CLI (${CLI_VERSION}) is older than the platform's recommended minimum (${min}). Update: npm i -g ninepm@latest`);
+}
+// Test seam: the once-per-invocation latch above is module state, which one
+// test process shares across cases.
+export function resetCliOutdatedAdvisoryForTests() {
+    cliOutdatedAdvisoryPrinted = false;
+}
 // Pull the control plane's own message out of its {error:{code,message}} envelope,
 // for the few refusals whose message is written for the tester and is the whole
 // point of the response (e.g. device signup being unavailable, #507). Dumping the

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ninepm",
-  "version": "0.51.0",
+  "version": "0.52.0",
   "description": "CLI for deploying built apps to 9pm.ai.",
   "homepage": "https://9pm.ai",
   "keywords": [