[AI Improvement] [Task] Clear src/mcp no-floating-promises by typing the MCP logger as void (lint: no-floating-promises, slice 1)

src/mcp/apps/init/mcp-app.ts

@@ -28,13 +28,13 @@
 let filteredProjects: Project[] = [];
 let selectedProjectId: string | null = null;
 
 function setStatus(message: string, type: "info" | "success" | "error" = "info") {
   statusBox.className = `status ${type}`;
   statusBox.textContent = message;
   statusBox.style.display = "block";
 }
 
 function renderProjects() {
   projectListContainer.innerHTML = "";
 
   if (filteredProjects.length === 0) {
@@ -196,7 +196,7 @@
     if (res.isError) {
       setStatus(`Failed to initialize: ${JSON.stringify(res.content)}`, "error");
     } else {
       setStatus(`Successfully initialized ${selectedProduct}!`, "success");
     }
   } catch (err: unknown) {
     const message = err instanceof Error ? err.message : String(err);
@@ -222,7 +222,8 @@
   }
 };
 
-(async () => {
+// Initial connection and project load catch all errors and report status via setStatus
+void (async () => {
   const envDirEl = document.getElementById("env-dir") as HTMLSpanElement;
   try {
     await app.connect();

src/mcp/apps/update_environment/mcp-app.ts

@@ -25,13 +25,13 @@
 const envProjectIdEl = document.getElementById("env-project-id") as HTMLSpanElement;
 const envUserEl = document.getElementById("env-user") as HTMLSpanElement;
 
 function showStatus(message: string, type: "success" | "error" | "info") {
   statusBox.textContent = message;
   statusBox.className = `status ${type}`;
   statusBox.style.display = "block";
 }
 
 function renderProjects() {
   projectListContainer.innerHTML = "";
 
   if (filteredProjects.length === 0) {
@@ -119,7 +119,8 @@
   }
 };
 
-(async () => {
+// Initial connection and project load catch all errors and report status via showStatus
+void (async () => {
   try {
     await app.connect();
     showStatus("Connecting to server...", "info");

src/mcp/index.ts

@@ -163,7 +163,7 @@
   }
 
   /** Wait until initialization has finished. */
   ready() {
     if (this._ready) return Promise.resolve();
     return new Promise((resolve, reject) => {
       this._readyPromises.push({ resolve: resolve as () => void, reject });
@@ -174,16 +174,16 @@
     return this.clientInfo?.name ?? (isFirebaseStudio() ? "Firebase Studio" : "<unknown-client>");
   }
 
   private get clientConfigKey() {
     return `mcp.clientConfigs.${this.clientName}:${this.startupRoot || process.cwd()}`;
   }
 
   getStoredClientConfig(): ClientConfig {
     return configstore.get(this.clientConfigKey) || {};
   }
 
   updateStoredClientConfig(update: Partial<ClientConfig>) {
     const config = configstore.get(this.clientConfigKey) || {};
     const newConfig = { ...config, ...update };
     configstore.set(this.clientConfigKey, newConfig);
     return newConfig;
@@ -647,13 +647,19 @@
         while (this._pendingMessages.length) {
           const message = this._pendingMessages.shift();
           if (!message) continue;
-          this.server.sendLoggingMessage({
-            level: message.level,
-            data: message.data,
-          });
+          this.server
+            .sendLoggingMessage({
+              level: message.level,
+              data: message.data,
+            })
+            .catch(() => {
+              // Logging must never throw; the transport may already be closed.
+            });
         }
 
-        void this.server.sendLoggingMessage({ level, data });
+        this.server.sendLoggingMessage({ level, data }).catch(() => {
+          // Logging must never throw; the transport may already be closed.
+        });
       } else {
         this._pendingMessages.push({ level, data });
       }
@@ -664,6 +670,6 @@
         logLevel,
         (message: unknown) => logAtLevel(logLevel, message),
       ]),
-    ) as Record<LoggingLevel, (message: unknown) => Promise<void>>;
+    ) as Record<LoggingLevel, (message: unknown) => void>;
   }
 }