fix(registerCommands): defensive error handling on toggleAutoApprove + log context

Addresses two CodeRabbit review items:

- toggleAutoApprove now wraps the await in try/catch logging to
  outputChannel, matching the defensive posture used on the other
  handlers (settingsButtonClicked, historyButtonClicked, etc.).
  An unhandled rejection from postMessageToWebview would otherwise
  bubble to VS Code's command dispatcher rather than being logged
  consistently with the rest of the file.

- All postMessageToWebview failure log messages now carry a
  [<command-name>] prefix so multi-failure logs are unambiguous.

Author: dennis-shorevapes

src/activate/__tests__/registerCommands.spec.ts

@@ -292,23 +292,48 @@ describe("registerCommands handlers", () => {
 	// postMessageToWebview sites in registerCommands.ts (settingsButtonClicked
 	// posts twice, plus historyButtonClicked, marketplaceButtonClicked, and
 	// acceptInput). Each handler is synchronous, so the .catch arm runs on a
-	// microtask; awaiting Promise.resolve() flushes it before we assert.
+	// microtask; awaiting Promise.resolve() flushes it before we assert. The
+	// log messages carry a `[<handlerName>]` prefix so multi-failure logs
+	// remain unambiguous; the prefix is per-handler, not per-call (both of
+	// settingsButtonClicked's posts share the same prefix).
 	it.each([
-		{ command: "zoo-code.settingsButtonClicked", expectedCalls: 2 },
-		{ command: "zoo-code.historyButtonClicked", expectedCalls: 1 },
-		{ command: "zoo-code.marketplaceButtonClicked", expectedCalls: 1 },
-		{ command: "zoo-code.acceptInput", expectedCalls: 1 },
-	])("$command logs to outputChannel when postMessageToWebview rejects", async ({ command, expectedCalls }) => {
+		{ command: "zoo-code.settingsButtonClicked", prefix: "settingsButtonClicked", expectedCalls: 2 },
+		{ command: "zoo-code.historyButtonClicked", prefix: "historyButtonClicked", expectedCalls: 1 },
+		{ command: "zoo-code.marketplaceButtonClicked", prefix: "marketplaceButtonClicked", expectedCalls: 1 },
+		{ command: "zoo-code.acceptInput", prefix: "acceptInput", expectedCalls: 1 },
+	])(
+		"$command logs to outputChannel when postMessageToWebview rejects",
+		async ({ command, prefix, expectedCalls }) => {
+			const boom = new Error("boom")
+			mockVisibleProvider.postMessageToWebview.mockReset()
+			mockVisibleProvider.postMessageToWebview.mockRejectedValue(boom)
+
+			handlers[command]()
+
+			// Flush microtasks so the chained .catch arm runs.
+			await new Promise((resolve) => setImmediate(resolve))
+
+			expect(mockOutputChannel.appendLine).toHaveBeenCalledTimes(expectedCalls)
+			expect(mockOutputChannel.appendLine).toHaveBeenCalledWith(
+				`[${prefix}] postMessageToWebview failed: ${boom}`,
+			)
+		},
+	)
+
+	it("toggleAutoApprove logs to outputChannel when postMessageToWebview rejects", async () => {
+		// toggleAutoApprove is `async` and awaits postMessageToWebview inside a
+		// try/catch (rather than relying on a `.catch` microtask like the
+		// void-prefixed sites), so awaiting the handler itself is sufficient to
+		// observe the appendLine call.
 		const boom = new Error("boom")
 		mockVisibleProvider.postMessageToWebview.mockReset()
 		mockVisibleProvider.postMessageToWebview.mockRejectedValue(boom)
 
-		handlers[command]()
+		await handlers["zoo-code.toggleAutoApprove"]()
 
-		// Flush microtasks so the chained .catch arm runs.
-		await new Promise((resolve) => setImmediate(resolve))
-
-		expect(mockOutputChannel.appendLine).toHaveBeenCalledTimes(expectedCalls)
-		expect(mockOutputChannel.appendLine).toHaveBeenCalledWith(`postMessageToWebview failed: ${boom}`)
+		expect(mockOutputChannel.appendLine).toHaveBeenCalledTimes(1)
+		expect(mockOutputChannel.appendLine).toHaveBeenCalledWith(
+			`[toggleAutoApprove] postMessageToWebview failed: ${boom}`,
+		)
 	})
 })

src/activate/registerCommands.ts

@@ -105,11 +105,11 @@ const getCommandsMap = ({ context, outputChannel, provider }: RegisterCommandOpt
 
 		void visibleProvider
 			.postMessageToWebview({ type: "action", action: "settingsButtonClicked" })
-			.catch((error) => outputChannel.appendLine(`postMessageToWebview failed: ${error}`))
+			.catch((error) => outputChannel.appendLine(`[settingsButtonClicked] postMessageToWebview failed: ${error}`))
 		// Also explicitly post the visibility message to trigger scroll reliably
 		void visibleProvider
 			.postMessageToWebview({ type: "action", action: "didBecomeVisible" })
-			.catch((error) => outputChannel.appendLine(`postMessageToWebview failed: ${error}`))
+			.catch((error) => outputChannel.appendLine(`[settingsButtonClicked] postMessageToWebview failed: ${error}`))
 	},
 	historyButtonClicked: () => {
 		const visibleProvider = getVisibleProviderOrLog(outputChannel)
@@ -122,14 +122,16 @@ const getCommandsMap = ({ context, outputChannel, provider }: RegisterCommandOpt
 
 		void visibleProvider
 			.postMessageToWebview({ type: "action", action: "historyButtonClicked" })
-			.catch((error) => outputChannel.appendLine(`postMessageToWebview failed: ${error}`))
+			.catch((error) => outputChannel.appendLine(`[historyButtonClicked] postMessageToWebview failed: ${error}`))
 	},
 	marketplaceButtonClicked: () => {
 		const visibleProvider = getVisibleProviderOrLog(outputChannel)
 		if (!visibleProvider) return
 		void visibleProvider
 			.postMessageToWebview({ type: "action", action: "marketplaceButtonClicked" })
-			.catch((error) => outputChannel.appendLine(`postMessageToWebview failed: ${error}`))
+			.catch((error) =>
+				outputChannel.appendLine(`[marketplaceButtonClicked] postMessageToWebview failed: ${error}`),
+			)
 	},
 	newTask: handleNewTask,
 	setCustomStoragePath: async () => {
@@ -180,7 +182,7 @@ const getCommandsMap = ({ context, outputChannel, provider }: RegisterCommandOpt
 
 		void visibleProvider
 			.postMessageToWebview({ type: "acceptInput" })
-			.catch((error) => outputChannel.appendLine(`postMessageToWebview failed: ${error}`))
+			.catch((error) => outputChannel.appendLine(`[acceptInput] postMessageToWebview failed: ${error}`))
 	},
 	toggleAutoApprove: async () => {
 		const visibleProvider = getVisibleProviderOrLog(outputChannel)
@@ -189,10 +191,14 @@ const getCommandsMap = ({ context, outputChannel, provider }: RegisterCommandOpt
 			return
 		}
 
-		await visibleProvider.postMessageToWebview({
-			type: "action",
-			action: "toggleAutoApprove",
-		})
+		try {
+			await visibleProvider.postMessageToWebview({
+				type: "action",
+				action: "toggleAutoApprove",
+			})
+		} catch (error) {
+			outputChannel.appendLine(`[toggleAutoApprove] postMessageToWebview failed: ${error}`)
+		}
 	},
 })