@corva/ui 3.78.0-1 → 3.78.0-2

package/mcp-server/server.mjs

@@ -224,7 +224,7 @@ const MCP_PROTOCOL_VERSION = '2025-03-26';
 const MCP_NETWORK_TRANSPORT = 'pipe';
 const UNKNOWN_CLIENT_ATTR = 'unknown';
 const DEFAULT_SAMPLING_RATE = 1.0;
-const TELEMETRY_CONFIG_TIMEOUT_MS = 10_000;
+const TELEMETRY_CONFIG_TIMEOUT_MS = 5000;
 const TELEMETRY_INIT_TIMEOUT_MS = 10_000;
 const TELEMETRY_SHUTDOWN_TIMEOUT_MS = 5000;
 const OTLP_TRACES_ENDPOINT = 'https://otlp.uptrace.dev/v1/traces';
@@ -246,9 +246,9 @@ const validateDsn = dsn => {
   return url.href;
 };
 
-const MCP_SERVER_VERSION = '1.5.1';
+const MCP_SERVER_VERSION = '1.5.2';
 
-var version = "3.78.0-1";
+var version = "3.78.0-2";
 
 const CORVA_UI_VERSION = version;
 
@@ -38053,6 +38053,7 @@ const formatUptime = ms => {
   }
   return `${seconds}s`;
 };
+const formatSeconds = ms => `${(ms / 1000).toFixed(2)}s`;
 const formatBytes = bytes => {
   const mb = bytes / (1024 * 1024);
   return `${mb.toFixed(1)} MB`;
@@ -38069,7 +38070,15 @@ const handleGetDiagnostics = (stats, telemetry, identity, registeredPromptNames)
   text += h2('Server');
   text += `${bold('Version')}: ${MCP_SERVER_VERSION}\n\n`;
   text += h2('Uptime');
-  text += `${bold('Server running for')}: ${formatUptime(uptimeMs)}\n\n`;
+  text += `${bold('Server running for')}: ${formatUptime(uptimeMs)}\n`;
+  if (stats.startupMs !== undefined) {
+    text += `${bold('Startup')}: ${formatSeconds(stats.startupMs)} from process start to transport connected`;
+    if (stats.telemetryInitMs !== undefined) {
+      text += `, of which telemetry init ${formatSeconds(stats.telemetryInitMs)}`;
+    }
+    text += '\n';
+  }
+  text += '\n';
   text += h2('Memory Usage');
   text += table(['Metric', 'Value'], [['Heap Used', formatBytes(heapUsed)], ['Heap Total', formatBytes(heapTotal)], ['Resident Set Size', formatBytes(rss)]]);
   text += '\n\n';
@@ -38888,6 +38897,9 @@ class CorvaUiMcpServer {
   isConnected = false;
   startTime = Date.now();
   requestCount = 0;
+
+  /** Process start → transport connected, in ms. Set once run() has connected. */
+
   telemetry = createNoopTelemetryClient();
   telemetryInfo = null;
   telemetryInitPromise = null;
@@ -39123,29 +39135,39 @@ class CorvaUiMcpServer {
     });
   }
   async run() {
-    const connectTimer = this.mcpLogger.time('transport-connect');
-    this.transport = new StdioServerTransport();
-    await this.server.connect(this.transport);
-    this.isConnected = true;
-    this.mcpLogger.timeEnd('transport-connect', connectTimer);
-    this.mcpLogger.info('Corva UI MCP Server running on stdio');
-    // Non-blocking: tool calls can begin; first calls within the 10s window may miss telemetry
+    // Telemetry settles before the transport connects, so no request can reach a handler while
+    // telemetry initialization is pending. Agents call tools right after initialize, and the
+    // calls that landed during the config fetch used to get no span and no metric (AI-369). The
+    // cost is an initialize response delayed by the fetch: ~2 s, at most the config timeout.
+    // stdin stays paused until connect(), so nothing the client sends meanwhile is lost.
+    const telemetryTimer = this.mcpLogger.time('telemetry-init');
     this.telemetryInitPromise = initTelemetry(this.mcpLogger).then(({
       client,
       info
     }) => {
-      if (this.shutdownRequested) {
-        client.shutdown().catch(() => {
-          /* shutting down anyway — a failed shutdown of a discarded client changes nothing */
-        });
-      } else {
-        this.telemetry = client;
-        this.telemetryInfo = info;
-        this.flushPendingInitData();
-      }
+      this.telemetry = client;
+      this.telemetryInfo = info;
     }).catch(() => {
       /* telemetry is best-effort — it must never take the server down */
     });
+    await this.telemetryInitPromise;
+    this.telemetryInitMs = telemetryTimer.stop();
+    this.mcpLogger.timeEnd('telemetry-init', telemetryTimer);
+
+    // If shutdown was requested during the wait (a signal; a closed stdin is usually noticed only
+    // once reading starts), shutdown() already runs and owns the client.
+    if (this.shutdownRequested) {
+      return;
+    }
+    const connectTimer = this.mcpLogger.time('transport-connect');
+    this.transport = new StdioServerTransport();
+    await this.server.connect(this.transport);
+    this.isConnected = true;
+    // performance.now() counts from process start, so this is the time to ready as the client saw
+    // it, minus whatever ran before the process (the data regeneration in `yarn mcp:dev`).
+    this.startupMs = performance.now();
+    this.mcpLogger.timeEnd('transport-connect', connectTimer);
+    this.mcpLogger.info('Corva UI MCP Server running on stdio');
   }
   async shutdown() {
     this.mcpLogger.info('Shutting down MCP server...');
@@ -39157,7 +39179,10 @@ class CorvaUiMcpServer {
     if (this.identityPromise) {
       await withTimeout(this.identityPromise, TELEMETRY_INIT_TIMEOUT_MS);
     }
-    if (this.telemetry.isEnabled()) {
+
+    // No transport means shutdown came during the telemetry wait: the client never connected, so
+    // there is no session to end.
+    if (this.telemetry.isEnabled() && this.transport) {
       const sessionDurationMs = Date.now() - this.startTime;
       const clientInfo = this.server.server.getClientVersion();
       this.telemetry.metrics.recordSessionEnd(sessionDurationMs, this.requestCount, clientInfo?.name, clientInfo?.version, this.projectIdentity);
@@ -39179,7 +39204,9 @@ class CorvaUiMcpServer {
   getStats() {
     return {
       startTime: this.startTime,
-      requestCount: this.requestCount
+      requestCount: this.requestCount,
+      startupMs: this.startupMs,
+      telemetryInitMs: this.telemetryInitMs
     };
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@corva/ui",
-  "version": "3.78.0-1",
+  "version": "3.78.0-2",
   "license": "UNLICENSED",
   "description": "Shared components/utils for Corva ui projects",
   "repository": {