@omercnet/paseo-context-mode 3.1.4-next.382.1 → 3.1.4-next.434.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@omercnet/paseo-context-mode",
-  "version": "3.1.4-next.382.1",
+  "version": "3.1.4-next.434.1",
   "type": "module",
   "description": "Provider-aware Context Mode health, savings, and MCP activation for Paseo.",
   "license": "MIT",
@@ -19,7 +19,7 @@
   },
   "packageManager": "bun@1.4.4",
   "engines": {
-    "node": ">=24"
+    "node": ">=24.15.0"
   },
   "keywords": [
     "paseo",
@@ -68,6 +68,6 @@
     "zod": "4.6.7"
   },
   "dependencies": {
-    "context-mode": "1.0.169"
+    "context-mode": "2.1.4"
   }
 }

package/server/actions.ts

@@ -1,78 +1,117 @@
+import { rm } from "node:fs/promises";
+import { basename, join } from "node:path";
 import type { PluginSettings } from "@getpaseo/plugin/server";
-import {
-  CONTEXT_MODE_TARGET_VERSION,
-  type ContextModeAction,
-  type ContextModeActionInput,
-  type DoctorReport,
-  type DoctorStatusRow,
-} from "../shared/actions";
+import type { ContextModeActionInput, DoctorReport, DoctorStatusRow } from "../shared/actions";
+import { contextModePlatformFor } from "../shared/providers";
 import type { contextModeSettings } from "../shared/settings";
-import { type BinaryResolution, type LaunchDescriptor, resolveContextModeBinary } from "./binary";
+import { type BinaryResolution, resolveContextModeBinary } from "./binary";
+import { contextModeProbeOptions } from "./handlers";
 import type { ProviderConfigSource } from "./integration";
-import { contextModeEnvironmentFor, contextModePlatformFor } from "./knowledge";
-import {
-  callContextModeTool,
-  inspectContextMode,
-  type McpInspection,
-  type McpProcessDependencies,
-  type McpToolArguments,
-} from "./mcp-process";
+import { callContextModeTool } from "./mcp-process";
 
-type SettingsHandle = PluginSettings<typeof contextModeSettings.schema>;
+type BinarySource = Extract<BinaryResolution, { state: "found" }>["source"];
 
-type ActionToolCaller = (
-  launch: LaunchDescriptor,
-  name: "ctx_doctor",
-  arguments_: McpToolArguments,
-  dependencies: McpProcessDependencies,
-) => Promise<string>;
+type SettingsHandle = PluginSettings<typeof contextModeSettings.schema>;
 
 export interface ActionHandlerDependencies {
-  osPlatform?: NodeJS.Platform;
   resolveBinary?: typeof resolveContextModeBinary;
-  inspect?: (
-    launch: LaunchDescriptor,
-    dependencies?: McpProcessDependencies,
-  ) => Promise<McpInspection>;
-  callTool?: ActionToolCaller;
+  callTool?: typeof callContextModeTool;
+  probeOptions?: typeof contextModeProbeOptions;
 }
 
 type ActionContext = { paseo?: ProviderConfigSource };
 
 export interface ContextModeActionHandlers {
   doctor(input: ContextModeActionInput, context?: ActionContext): Promise<DoctorReport>;
-  install(input: ContextModeActionInput, context?: ActionContext): Promise<ContextModeAction>;
-  upgrade(input: ContextModeActionInput, context?: ActionContext): Promise<ContextModeAction>;
 }
 
-export function parseContextModeDoctorOutput(rawOutput: string): DoctorStatusRow[] {
+const DOCTOR_STATUS: Readonly<Record<string, DoctorStatusRow["status"]>> = {
+  PASS: "ok",
+  WARN: "warning",
+  FAIL: "error",
+  UNKNOWN: "error",
+};
+
+/**
+ * D3 compares the agent's own host config with Context Mode's setup and D5 exercises its hooks.
+ * Paseo injects only the MCP server, so both fail for every injected agent: advice, not a fault,
+ * and their `context-mode setup` fix would replace Paseo's injection with a native install.
+ */
+const INFORMATIONAL_CHECK = /^D(3|5b?)\b/;
+
+/** Upstream's fix for an outdated runtime; the plugin offers no in-place upgrade. */
+const UPGRADE_FIX = "run ctx_upgrade";
+
+const UPGRADE_ADVICE: Readonly<Record<BinarySource, string>> = {
+  bundled: "Update the Context Mode plugin in Paseo",
+  settings: "Upgrade the Context Mode install configured in settings",
+};
+
+function doctorRow(
+  check: string,
+  status: DoctorStatusRow["status"],
+  detail: string,
+  upstreamFix: string,
+  runtime: BinarySource,
+): DoctorStatusRow {
+  const row = { check: check.slice(0, 256), detail: detail.slice(0, 4_096) };
+  if (INFORMATIONAL_CHECK.test(check)) {
+    return { ...row, status: status === "ok" ? "ok" : "warning" };
+  }
+  if (upstreamFix === UPGRADE_FIX) {
+    // A bundled runtime that trails npm is expected until the next plugin release.
+    const upgradeStatus = runtime === "bundled" ? "warning" : status;
+    return { ...row, status: upgradeStatus, fix: UPGRADE_ADVICE[runtime] };
+  }
+  return upstreamFix ? { ...row, status, fix: upstreamFix.slice(0, 4_096) } : { ...row, status };
+}
+
+/**
+ * Parses the `| Check | Status | Detail | Fix |` table that `ctx_doctor` prints. Upstream does not
+ * escape pipes, so the first two cells are check and status, the last is the fix, and whatever is
+ * between them is the detail. The fix cell leads with a restatement; only the command after the
+ * last `Fix:` is kept, and advice the plugin cannot act on is rewritten for `runtime`.
+ */
+export function parseContextModeDoctorOutput(
+  rawOutput: string,
+  runtime: BinarySource,
+): DoctorStatusRow[] {
   const rows: DoctorStatusRow[] = [];
   for (const line of rawOutput.split(/\r?\n/)) {
-    const match = /^\[(OK|WARN|FAIL)\]\s+(.+)$/.exec(line.trim());
-    if (!match) continue;
-    const body = match[2] ?? "";
-    const separator = body.indexOf(":");
-    const check = (separator < 0 ? body : body.slice(0, separator)).trim().slice(0, 256);
-    if (!check) continue;
-    const detail = (separator < 0 ? "" : body.slice(separator + 1).trim()).slice(0, 4_096);
-    rows.push({
-      status: match[1] === "OK" ? "ok" : match[1] === "WARN" ? "warning" : "error",
-      check,
-      detail,
-    });
+    const trimmed = line.trim();
+    if (!trimmed.startsWith("|") || !trimmed.endsWith("|")) continue;
+    const cells = trimmed
+      .slice(1, -1)
+      .split("|")
+      .map((cell) => cell.trim());
+    const [check = "", statusCell = ""] = cells;
+    const status = Object.hasOwn(DOCTOR_STATUS, statusCell) ? DOCTOR_STATUS[statusCell] : null;
+    if (cells.length < 4 || !check || !status) continue;
+    const detail = cells.slice(2, -1).join(" | ");
+    const fix = (cells.at(-1) ?? "").split("Fix:").at(-1)?.trim() ?? "";
+    rows.push(doctorRow(check, status, detail, fix, runtime));
     if (rows.length === 64) break;
   }
   return rows;
 }
 
+/** Upstream's report file name; the shown path may be `~`-abbreviated, so only the name is trusted. */
+const DOCTOR_REPORT_NAME = /^doctor-[\w-]+\.md$/;
+
+/** ctx_doctor writes a report file per failing check and nothing prunes them; the panel keeps none. */
+async function discardDoctorReport(output: string, dataDir: string): Promise<void> {
+  const reported = /^Report file: (.+)$/m.exec(output)?.[1]?.trim();
+  const name = reported ? basename(reported) : "";
+  if (DOCTOR_REPORT_NAME.test(name)) await rm(join(dataDir, "reports", name), { force: true });
+}
+
 export function createContextModeActionHandlers(
   settingsHandle: SettingsHandle,
   dependencies: ActionHandlerDependencies = {},
 ): ContextModeActionHandlers {
   const resolveBinary = dependencies.resolveBinary ?? resolveContextModeBinary;
-  const inspect = dependencies.inspect ?? inspectContextMode;
   const callTool = dependencies.callTool ?? callContextModeTool;
-  const osPlatform = dependencies.osPlatform ?? process.platform;
+  const probeOptions = dependencies.probeOptions ?? contextModeProbeOptions;
 
   async function resolvedBinary(
     paseo: ProviderConfigSource | undefined,
@@ -86,68 +125,18 @@ export function createContextModeActionHandlers(
     return binary;
   }
 
-  async function currentVersion(
-    input: ContextModeActionInput,
-    paseo: ProviderConfigSource | undefined,
-  ): Promise<string | null> {
-    try {
-      const binary = await resolvedBinary(paseo);
-      const inspection = await inspect(binary.launch, {
-        env: contextModeEnvironmentFor(input.provider),
-      });
-      return inspection.version?.slice(0, 128) ?? null;
-    } catch {
-      return null;
-    }
-  }
-
   return {
     async doctor(input, context) {
-      const binary = await resolvedBinary(context?.paseo);
-      const rawOutput = await callTool(
-        binary.launch,
-        "ctx_doctor",
-        {},
-        {
-          env: contextModeEnvironmentFor(input.provider),
-        },
-      );
-      return {
-        platform: contextModePlatformFor(input.provider),
-        rows: parseContextModeDoctorOutput(rawOutput),
-        rawOutput,
-      };
-    },
-    async install(input, context) {
-      const platform = contextModePlatformFor(input.provider);
-      return {
-        program: osPlatform === "win32" ? "npm.cmd" : "npm",
-        args: [
-          "install",
-          "-g",
-          `context-mode@${CONTEXT_MODE_TARGET_VERSION}`,
-          "--no-audit",
-          "--no-fund",
-        ],
-        currentVersion: await currentVersion(input, context?.paseo),
-        targetVersion: CONTEXT_MODE_TARGET_VERSION,
-        platform,
-        requiresRestart: true,
-      };
-    },
-    async upgrade(input, context) {
       const platform = contextModePlatformFor(input.provider);
       const binary = await resolvedBinary(context?.paseo);
-      const inspection = await inspect(binary.launch, {
-        env: contextModeEnvironmentFor(input.provider),
-      });
+      // ctx_doctor writes a report and index rows, so it runs against the plugin-owned probe store.
+      const options = await probeOptions(platform);
+      const rawOutput = await callTool(binary.launch, "ctx_doctor", {}, {}, options);
+      await discardDoctorReport(rawOutput, options.dataDir);
       return {
-        program: binary.launch.program,
-        args: [...binary.launch.args, "upgrade", "--platform", platform],
-        currentVersion: inspection.version?.slice(0, 128) ?? null,
-        targetVersion: "latest",
         platform,
-        requiresRestart: true,
+        rows: parseContextModeDoctorOutput(rawOutput, binary.source),
+        rawOutput,
       };
     },
   };