@barivia/barsom-mcp 0.23.10 → 0.23.12

package/dist/ui-delivery.js

@@ -6,11 +6,20 @@ import { apiRawCall, getClientSupportsMcpApps, getVizPort, mimeForFilename, rese
 import { buildVizStandaloneUrl } from "./viz_links.js";
 export function parseUiDeliveryOverride() {
     const raw = (process.env.BARIVIA_UI_DELIVERY ?? "auto").trim().toLowerCase();
-    if (raw === "apps" || raw === "localhost" || raw === "inline" || raw === "auto") {
+    if (raw === "apps" ||
+        raw === "localhost" ||
+        raw === "inline" ||
+        raw === "text_only" ||
+        raw === "auto") {
         return raw;
     }
     return "auto";
 }
+/** Opt-in: force localhost under auto even when the host advertises MCP Apps (e.g. Cursor list-without-mount). */
+export function preferLocalhostUi() {
+    const v = (process.env.BARIVIA_UI_PREFER_LOCALHOST ?? "").trim().toLowerCase();
+    return v === "1" || v === "true" || v === "yes";
+}
 function vizPortPinned() {
     return Boolean(process.env.BARIVIA_VIZ_PORT?.trim());
 }
@@ -30,17 +39,26 @@ export function buildHealthUrl(jobId) {
 }
 function hintForTier(tier, tool, opts) {
     const label = tool === "training_monitor" ? "training monitor" : "results explorer";
+    const linkFirst = opts.standaloneUrl
+        ? `AGENT: Post a clickable markdown link first (e.g. "[Open ${label}](${opts.standaloneUrl})") — do not bury it after metrics.`
+        : undefined;
     switch (tier) {
         case "embedded":
-            return `The ${label} MCP App panel is embedded in this client. Also mention the standalone URL if the user may open a browser tab.`;
+            return linkFirst
+                ? `${linkFirst} Embedded MCP App panel if the host mounts it; some hosts list ui:// without mounting (use the URL).`
+                : `The ${label} MCP App panel may embed. Use jobs(status) / results(get) if no panel or URL appears.`;
         case "localhost":
-            return opts.standaloneUrl
-                ? `AGENT: Post a clickable markdown link to the user (e.g. "[Open ${label}](${opts.standaloneUrl})") — do not bury it in tool output or use broken workspace file paths. Re-post after the job completes if status was running.`
+            return linkFirst
+                ? `${linkFirst} Re-post after the job completes if status was running. Set BARIVIA_VIZ_PORT for a stable port.`
                 : `AGENT: Viz server unavailable — use jobs(status) and results(get) for progress and figures.`;
         case "inline_image":
-            return `AGENT: Inline raster figure(s) are attached below. Summarize metrics in your reply; offer results_explorer or results(get) for additional figures.`;
+            return linkFirst
+                ? `${linkFirst} Inline raster figure(s) may also be attached; summarize metrics; offer results_explorer or results(get) for more.`
+                : `AGENT: Inline raster figure(s) are attached below. Summarize metrics; offer results_explorer or results(get) for additional figures.`;
         case "text_only":
-            return `AGENT: No embedded panel or localhost viz — use jobs(status) for training progress and results(get) or results_explorer for figures.`;
+            return linkFirst
+                ? `${linkFirst} Otherwise use jobs(status) for progress and results(get) for figures.`
+                : `AGENT: No embedded panel or localhost viz — use jobs(status) for training progress and results(get) or results_explorer for figures.`;
     }
 }
 export function resolveUiDelivery(options) {
@@ -58,9 +76,15 @@ export function resolveUiDelivery(options) {
     else if (override === "inline") {
         delivery = options.inlineImageAttached ? "inline_image" : "text_only";
     }
+    else if (override === "text_only") {
+        delivery = "text_only";
+    }
     else {
-        // auto
-        if (mcpApps) {
+        // auto — default unchanged: prefer embedded when Apps advertised unless opt-in prefer-localhost
+        if (preferLocalhostUi() && port) {
+            delivery = "localhost";
+        }
+        else if (mcpApps) {
             delivery = "embedded";
         }
         else if (port) {
@@ -73,12 +97,13 @@ export function resolveUiDelivery(options) {
             delivery = "text_only";
         }
     }
+    // Always include standalone URL when the viz port exists (all tiers) — hosts may not mount Apps.
     const urls = [];
-    if (standaloneUrl && (delivery === "embedded" || delivery === "localhost")) {
+    if (standaloneUrl) {
         urls.push(standaloneUrl);
     }
     const healthUrl = buildHealthUrl(options.jobId);
-    if (healthUrl && delivery === "localhost") {
+    if (healthUrl && (delivery === "localhost" || preferLocalhostUi())) {
         urls.push(healthUrl);
     }
     const crossLink = options.tool === "training_monitor" && options.jobStatus === "completed"
@@ -97,6 +122,7 @@ export function resolveUiDelivery(options) {
         viz_port: port,
         port_pinned: vizPortPinned(),
         override,
+        ...(options.uiResourceUri ? { ui_resource_uri: options.uiResourceUri } : {}),
     };
 }
 /**
@@ -154,6 +180,7 @@ export function getMcpClientDiagnostics() {
     const override = parseUiDeliveryOverride();
     const mcpApps = getClientSupportsMcpApps();
     const port = getVizPort() || null;
+    const preferLocal = preferLocalhostUi();
     let activeTier = "text_only";
     if (override === "apps") {
         activeTier = mcpApps ? "embedded" : port ? "localhost" : "text_only";
@@ -164,6 +191,12 @@ export function getMcpClientDiagnostics() {
     else if (override === "inline") {
         activeTier = "inline_image";
     }
+    else if (override === "text_only") {
+        activeTier = "text_only";
+    }
+    else if (preferLocal && port) {
+        activeTier = "localhost";
+    }
     else if (mcpApps) {
         activeTier = "embedded";
     }
@@ -175,6 +208,7 @@ export function getMcpClientDiagnostics() {
         viz_port: port,
         viz_port_pinned: vizPortPinned(),
         ui_delivery_override: override,
+        ui_prefer_localhost: preferLocal,
         active_delivery_tier: activeTier,
         health_url: buildHealthUrl(),
     };