@omercnet/paseo-omp 1.3.3-next.241.1 → 1.3.3-next.243.1

package/README.md

@@ -95,7 +95,7 @@ Provider functionality outside the capability flags is tracked separately:
 | --- | ---: | --- |
 | Registration metadata and sanitized SVG icon | **100%** | Stable `omp-plugin` identity, label, description, and bundled icon. |
 | Strict `providerOptionsSchema` | **100%** | Command, literal environment, names-only profile inherited environment, output-redaction mode, session directory, RPC timeout, and role-model options are validated and normalized. Host-wide inherited names are separately persisted and merged at launch. |
-| Availability diagnostics | **50%** | The plugin implements a `checkAvailability` hook with bounded checks that distinguish missing, unrunnable, incompatible, and available OMP runtimes, but Paseo 0.10.2 and 0.11.1 do not call it. Hosts treat the provider as available when a connection opens, and launch failures surface at session start. |
+| Availability diagnostics | **50%** | The plugin implements a `checkAvailability` hook with bounded checks that distinguish missing, unrunnable, incompatible, and available OMP runtimes, but Paseo 0.11.1 does not call it. The host uses the `status()` hook below instead, which reuses this probe for the default launch. |
 | Catalog cache identity | **100%** | Hash includes effective options, merged host/profile inherited-environment names, settings, scope, cwd, and default command, but never resolves or fingerprints inherited values. |
 | Models, modes, and thinking catalog | **100%** | Native catalog is mapped to opaque public model IDs with committed defaults and permission-gated modes. |
 | Connection `send` / `onEvent` / `close` lifecycle | **100%** | Request correlation, multi-session ownership, process recovery, teardown, and provider reload/removal are covered. |
@@ -105,7 +105,8 @@ Provider functionality outside the capability flags is tracked separately:
 | Commands and committed session state events | **100%** | Publishes `session.commands`, `session.opened`, `session.config`, `session.ready`, and request completion in protocol order. |
 | Built-in timeline snapshots | **100%** | Assistant, reasoning, tools, todos, notifications, errors, compaction, and friendly MCP labels use stable IDs and complete snapshots. OMP emits client-safe PNG/JPEG when possible; validated legacy WebP images render on capable clients with an explicit per-image fallback elsewhere. |
 | Usage reporting | **100%** | Periodic, post-compaction, fallback, terminal, timeout, and recovered-runtime samples publish `session.usage`. |
-| 0.11 provider `status()` | **0%** | Not registered. Its request carries only an optional daemon-resolved `launch`; the daemon supplies one only when a registration declares `command`, which would move launch ownership to the daemon. Without that, status cannot see per-agent `providerOptions.command`, so probing a guessed default could wrongly mark a valid custom-command provider unavailable. |
+| 0.11 provider `status()` | **75%** | Registered for the plugin-owned default launch (`OMP_COMMAND` or `omp`, plus the fixed `--profile` wrapper for profile providers), using the bounded `--version`/`--help` probe. Only a missing executable reports `available: false`; slow, failing, or unrecognized probes stay available with a diagnostic. An unexpected internal probe error is not caught here, so the host reports it as unavailable. Available results are shared for 30 seconds; an unavailable result is never remembered, so installing `omp` takes effect on the next host call. The request carries only an optional daemon `launch` and never `providerOptions`, so a per-agent `providerOptions.command` or `env` is invisible. **Breaking on 0.11 hosts:** the host gates create, resume, and draft commands/features on this status, so a missing default executable blocks them all, including agents configured with an otherwise valid custom command. Any request `launch` is ignored. |
+| 0.11 provider `command` and daemon launch | **0%** | Deliberately not declared. Declaring `command` makes the daemon resolve it against its own `PATH` and reject every `connect` when the default binary is missing. The resulting `launch.env` is the daemon's entire process environment plus overrides, indistinguishable from configured values, so adopting it would defeat names-only `inheritEnv` and the fixed profile store environment. The host never supplies `launch` without a declared `command`; `connect` and `getCatalogCacheKey` do not read it, each pinned by `tests/provider-catalog.test.ts`. Revisit if the host stops gating on the default binary or exposes configured `env` separately from the daemon environment. |
 | 0.11 screens and sidebar items | **100%** | The global config surface migrates to `addScreen({ id: "config" })`, preserving legacy links and encoding profile selection in screen params. The Hub status row uses `addSidebarHeaderItem` + `SidebarRow`, lists processes and tails logs in a reusable popover. The legacy static config row and surface were removed with the 0.11 floor. |
 | 0.11 usage source | **0%** | Not registered. `server/quota.ts` reads OMP's recorded `usage_history`, not live quota, so there is no live fetcher to register. Upstream Claude and Codex sources fetch live quota for the Anthropic and Codex OAuth credentials in the daemon's default or `OMP_PROFILE` store; other providers and named profiles remain covered only by this plugin's historical quota pill. |
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@omercnet/paseo-omp",
-  "version": "1.3.3-next.241.1",
+  "version": "1.3.3-next.243.1",
   "type": "module",
   "description": "Paseo integration for OMP, including its direct provider and workspace tooling.",
   "license": "MIT",

package/server/package-version.ts

@@ -1,2 +1,2 @@
 /** Release Please keeps this bundle-safe constant aligned with package.json. */
-export const PASEO_OMP_PACKAGE_VERSION = "1.3.3-next.241.1"; // x-release-please-version
+export const PASEO_OMP_PACKAGE_VERSION = "1.3.3-next.243.1"; // x-release-please-version

package/server/provider/profile-providers.ts

@@ -7,7 +7,7 @@ import { ompAgentDir, ompCacheDir, ompDataDir, ompSessionDir, ompStateDir } from
 import { OmpRpcRuntime, type OmpRuntime } from "./omp-rpc";
 import type { OmpStartOptions } from "./omp-rpc-environment";
 import { parseOmpProviderOptions } from "./provider-options";
-import { createOmpProvider, type OmpProviderOptions } from "./registration";
+import { createOmpProvider, createOmpStatus, type OmpProviderOptions } from "./registration";
 import { OmpPublicError } from "./security";
 
 const MAX_PROFILES = 128;
@@ -229,6 +229,18 @@ export function createProfileOmpProvider(profile: string, options: OmpProviderOp
       sessionDir,
     );
   };
+  const checkAvailability = async (
+    input: Parameters<NonNullable<typeof provider.checkAvailability>>[0],
+    context?: Parameters<NonNullable<typeof provider.checkAvailability>>[1],
+  ) => {
+    const command = validateCatalog(input);
+    if (!provider.checkAvailability)
+      throw new OmpPublicError("OMP availability probe is unavailable");
+    return provider.checkAvailability(
+      { ...input, providerOptions: { ...input.providerOptions, command } },
+      context,
+    );
+  };
   return {
     ...provider,
     async getCatalogCacheKey(
@@ -237,18 +249,8 @@ export function createProfileOmpProvider(profile: string, options: OmpProviderOp
       validateCatalog(input);
       return provider.getCatalogCacheKey?.(input);
     },
-    async checkAvailability(
-      input: Parameters<NonNullable<typeof provider.checkAvailability>>[0],
-      context?: Parameters<NonNullable<typeof provider.checkAvailability>>[1],
-    ) {
-      const command = validateCatalog(input);
-      if (!provider.checkAvailability)
-        throw new OmpPublicError("OMP availability probe is unavailable");
-      return provider.checkAvailability(
-        { ...input, providerOptions: { ...input.providerOptions, command } },
-        context,
-      );
-    },
+    checkAvailability,
+    status: createOmpStatus(() => checkAvailability({ scope: "global" })),
     id,
     label: `OMP · ${profile}`,
   };

package/server/provider/registration.ts

@@ -1,6 +1,6 @@
 import { createHash } from "node:crypto";
 import { homedir } from "node:os";
-import type { ProviderRegistration } from "@getpaseo/plugin/server/provider";
+import type { ProviderRegistration, ProviderStatus } from "@getpaseo/plugin/server/provider";
 import { z } from "zod";
 import type { OmpBrowserAuthorizationRegistry } from "../mcp-browser";
 import type { OmpOperationalFailureReporter } from "../operational-failure-diagnostics";
@@ -41,6 +41,57 @@ type ProviderRegistrationCompat = Omit<
   ): Promise<ProviderAvailabilityCompat>;
 };
 
+const STATUS_CACHE_TTL_MS = 30_000;
+
+/**
+ * The 0.11 host calls `status` before every create, resume, and draft request and passes only
+ * the daemon-resolved launch, never per-agent `providerOptions`. Only a missing executable is
+ * definitive for the plugin-owned default launch; slow, failing, or unrecognized probes stay
+ * available with a diagnostic so load or a differently launched agent is never refused.
+ */
+function toProviderStatus(availability: ProviderAvailabilityCompat): ProviderStatus {
+  if (availability.status === "available") return { available: true };
+  if (availability.status === "missing") {
+    return {
+      available: false,
+      diagnostic: availability.diagnostic ?? "OMP executable was not found",
+    };
+  }
+  return {
+    available: true,
+    diagnostic: availability.diagnostic ?? `OMP launch probe reported ${availability.status}`,
+  };
+}
+
+/**
+ * Share one bounded probe across the host's frequent status calls. Only an available result is
+ * kept for the interval; an unavailable or failed result is shared by in-flight callers and then
+ * dropped, so installing the executable is picked up by the very next host call.
+ */
+export function createOmpStatus(
+  probe: () => Promise<ProviderAvailabilityCompat>,
+): () => Promise<ProviderStatus> {
+  let cached: { expiresAt: number; result: Promise<ProviderStatus> } | undefined;
+  return () => {
+    const now = Date.now();
+    if (cached && cached.expiresAt > now) return cached.result;
+    const entry = {
+      expiresAt: now + STATUS_CACHE_TTL_MS,
+      result: probe().then(toProviderStatus),
+    };
+    cached = entry;
+    entry.result.then(
+      (status) => {
+        if (!status.available && cached === entry) cached = undefined;
+      },
+      () => {
+        if (cached === entry) cached = undefined;
+      },
+    );
+    return entry.result;
+  };
+}
+
 const CAPABILITIES = [
   "prompt.message",
   "prompt.command",
@@ -98,6 +149,26 @@ export function createOmpProvider(options: OmpProviderOptions = {}): ProviderReg
       reportProtocolViolation: options.reportProtocolViolation,
     });
   const nativeReservations = new OmpNativeSessionReservations();
+  const checkAvailability: NonNullable<ProviderRegistrationCompat["checkAvailability"]> = async (
+    catalogOptions,
+    context,
+  ) => {
+    if (options.availabilityProbe) {
+      return await options.availabilityProbe(catalogOptions, context?.timeoutMs);
+    }
+    const providerOptions = parseOmpProviderOptions(catalogOptions.providerOptions);
+    const environment = { ...(options.environment ?? process.env), ...providerOptions.env };
+    const configuredCommand: readonly [string, ...string[]] = providerOptions.command?.[0]
+      ? [providerOptions.command[0], ...providerOptions.command.slice(1)]
+      : [environment.OMP_COMMAND || "omp"];
+    return await probeOmpAvailability({
+      command: configuredCommand,
+      cwd:
+        catalogOptions.scope === "workspace" && catalogOptions.cwd ? catalogOptions.cwd : homedir(),
+      environment,
+      timeoutMs: context?.timeoutMs,
+    });
+  };
   return {
     id: "omp-plugin",
     label: "OMP Plugin",
@@ -130,25 +201,8 @@ export function createOmpProvider(options: OmpProviderOptions = {}): ProviderReg
       }
       return createHash("sha256").update(stableJson(identity)).digest("base64url");
     },
-    async checkAvailability(catalogOptions, context) {
-      if (options.availabilityProbe) {
-        return await options.availabilityProbe(catalogOptions, context?.timeoutMs);
-      }
-      const providerOptions = parseOmpProviderOptions(catalogOptions.providerOptions);
-      const environment = { ...(options.environment ?? process.env), ...providerOptions.env };
-      const configuredCommand: readonly [string, ...string[]] = providerOptions.command?.[0]
-        ? [providerOptions.command[0], ...providerOptions.command.slice(1)]
-        : [environment.OMP_COMMAND || "omp"];
-      return await probeOmpAvailability({
-        command: configuredCommand,
-        cwd:
-          catalogOptions.scope === "workspace" && catalogOptions.cwd
-            ? catalogOptions.cwd
-            : homedir(),
-        environment,
-        timeoutMs: context?.timeoutMs,
-      });
-    },
+    checkAvailability,
+    status: createOmpStatus(() => checkAvailability({ scope: "global" })),
     async connect(request) {
       if (boundedJsonBytes(request, 8 * 1024, 32, 256, 64) === Number.POSITIVE_INFINITY) {
         throw new Error("OMP provider received an oversized connection request");