From 95d7e42b184b1bc3e46583bed893c2c95bde16b8 Mon Sep 17 00:00:00 2001
From: Mark IJbema <mark@kilocode.ai>
Date: Tue, 10 Feb 2026 12:49:59 +0100
Subject: [PATCH] chore: remove debug logging added in model-selector PR

Remove noisy console.log statements that were useful during development
but too verbose for production:
- resolveWebviewView/Webview HTML lifecycle logs
- Providers loaded diagnostic log with counts
- postMessage result log (fired on every single message)
- sendMessage request body diagnostic log
- messageCounter field (only supported debug logging)

Keep error/warn logging that aids production troubleshooting:
- postMessage dropped (no webview) warning
- postMessage delivery failure error
- HTTP request/response errors (failed, empty body, invalid JSON)
- Provider fetch failure error
---
 packages/kilo-vscode/src/KiloProvider.ts      | 40 +++++--------------
 .../src/services/cli-backend/http-client.ts   | 12 ------
 2 files changed, 11 insertions(+), 41 deletions(-)

diff --git a/packages/kilo-vscode/src/KiloProvider.ts b/packages/kilo-vscode/src/KiloProvider.ts
index 21ffee1a4de..c2fc1a724db 100644
--- a/packages/kilo-vscode/src/KiloProvider.ts
+++ b/packages/kilo-vscode/src/KiloProvider.ts
@@ -9,7 +9,6 @@ export class KiloProvider implements vscode.WebviewViewProvider {
   private connectionState: "connecting" | "connected" | "disconnected" | "error" = "connecting"
   private loginAttempt = 0
   private isWebviewReady = false
-  private messageCounter = 0
   /** Cached providersLoaded payload so requestProviders can be served before httpClient is ready */
   private cachedProvidersMessage: unknown = null
 
@@ -92,7 +91,6 @@ export class KiloProvider implements vscode.WebviewViewProvider {
   ) {
     // Store the webview references
     this.isWebviewReady = false
-    console.log("[Kilo New] KiloProvider: 🧩 resolveWebviewView called")
     this.webview = webviewView.webview
 
     // Set up webview options
@@ -103,7 +101,6 @@ export class KiloProvider implements vscode.WebviewViewProvider {
 
     // Set HTML content
     webviewView.webview.html = this._getHtmlForWebview(webviewView.webview)
-    console.log("[Kilo New] KiloProvider: 🧩 Webview HTML set")
 
     // Handle messages from webview (shared handler)
     this.setupWebviewMessageHandler(webviewView.webview)
@@ -422,13 +419,6 @@ export class KiloProvider implements vscode.WebviewViewProvider {
       const providerID = config.get<string>("providerID", "kilo")
       const modelID = config.get<string>("modelID", "kilo/auto")
 
-      console.log("[Kilo New] KiloProvider: 📦 Providers loaded, sending to webview", {
-        providerCount: Object.keys(normalized).length,
-        connectedCount: response.connected.length,
-        defaultProvider: providerID,
-        defaultModel: modelID,
-      })
-
       const message = {
         type: "providersLoaded",
         providers: normalized,
@@ -743,29 +733,21 @@ export class KiloProvider implements vscode.WebviewViewProvider {
    * Public so toolbar button commands can send messages.
    */
   public postMessage(message: unknown): void {
-    const id = ++this.messageCounter
-
-    const type =
-      typeof message === "object" &&
-      message !== null &&
-      "type" in message &&
-      typeof (message as { type?: unknown }).type === "string"
-        ? ((message as { type: string }).type as string)
-        : "<unknown>"
-
     if (!this.webview) {
-      console.warn("[Kilo New] KiloProvider: ⚠️ postMessage dropped (no webview)", { id, type })
+      const type =
+        typeof message === "object" &&
+        message !== null &&
+        "type" in message &&
+        typeof (message as { type?: unknown }).type === "string"
+          ? (message as { type: string }).type
+          : "<unknown>"
+      console.warn("[Kilo New] KiloProvider: ⚠️ postMessage dropped (no webview)", { type })
       return
     }
 
-    void this.webview.postMessage(message).then(
-      (delivered) => {
-        console.log("[Kilo New] KiloProvider: 📬 postMessage result", { id, type, delivered })
-      },
-      (error) => {
-        console.error("[Kilo New] KiloProvider: ❌ postMessage failed", { id, type, error })
-      },
-    )
+    void this.webview.postMessage(message).then(undefined, (error) => {
+      console.error("[Kilo New] KiloProvider: ❌ postMessage failed", error)
+    })
   }
 
   /**
diff --git a/packages/kilo-vscode/src/services/cli-backend/http-client.ts b/packages/kilo-vscode/src/services/cli-backend/http-client.ts
index b0622d2ef00..59cc0096acf 100644
--- a/packages/kilo-vscode/src/services/cli-backend/http-client.ts
+++ b/packages/kilo-vscode/src/services/cli-backend/http-client.ts
@@ -165,18 +165,6 @@ export class HttpClient {
       body.model = { providerID: options.providerID, modelID: options.modelID }
     }
 
-    // Diagnostic logging (redacts message contents).
-    const debugBody = {
-      ...body,
-      parts: parts.map((p) => {
-        if (p.type === "text") {
-          return { type: "text", textLength: p.text.length }
-        }
-        return { type: "file", mime: p.mime, urlLength: p.url.length }
-      }),
-    }
-    console.log("[Kilo New] HTTP: 📨 sendMessage request body", JSON.stringify(debugBody))
-
     await this.request<void>("POST", `/session/${sessionId}/message`, body, { directory, allowEmpty: true })
   }