fix(Task): catch updateClineMessage rejections on partial-message paths

The four fire-and-forget updateClineMessage calls in say()/ask()
partial-message branches were void-prefixed without .catch arms. The
callee's webview post is internally guarded, but its synchronous
RooCodeEventName.Message emit can throw via a consumer-attached
listener — so the void sites can produce unhandled rejections.

Replace each `void this.updateClineMessage(...)` with an explicit
.catch arm that logs the error, matching the other rejection-handling
sites in this file. Two new tests pin the say() and ask() catch arms.

Also pin the void on getCheckpointService with a rationale comment:
its top-level try/catch returns undefined on failure, so .catch is
not needed there.

Author: dennis-shorevapes

src/core/task/Task.ts

@@ -1188,8 +1188,13 @@ export class Task extends EventEmitter<TaskEvents> implements TaskLike {
 					// data or one whole message at a time so ignore partial for
 					// saves, and only post parts of partial message instead of
 					// whole array in new listener.
-					/* v8 ignore next -- fire-and-forget webview update; rejection is benign */
-					void this.updateClineMessage(lastMessage)
+					// Fire-and-forget: the webview post is internally guarded, but
+					// the `RooCodeEventName.Message` emit can synchronously throw
+					// if any consumer-attached listener does, which would surface
+					// here as an unhandled rejection. Log it instead.
+					this.updateClineMessage(lastMessage).catch((error) => {
+						console.error("[Task#ask] updateClineMessage failed:", error)
+					})
 					// console.log("Task#ask: current ask promise was ignored (#1)")
 					throw new AskIgnoredError("updating existing partial")
 				} else {
@@ -1227,8 +1232,11 @@ export class Task extends EventEmitter<TaskEvents> implements TaskLike {
 					lastMessage.progressStatus = progressStatus
 					lastMessage.isProtected = isProtected
 					await this.saveClineMessages()
-					/* v8 ignore next -- fire-and-forget webview update; rejection is benign */
-					void this.updateClineMessage(lastMessage)
+					// Fire-and-forget: see updateClineMessage call above for the
+					// rationale on the .catch arm.
+					this.updateClineMessage(lastMessage).catch((error) => {
+						console.error("[Task#ask] updateClineMessage failed:", error)
+					})
 				} else {
 					// This is a new and complete message, so add it like normal.
 					this.askResponse = undefined
@@ -1674,7 +1682,13 @@ export class Task extends EventEmitter<TaskEvents> implements TaskLike {
 					lastMessage.images = images
 					lastMessage.partial = partial
 					lastMessage.progressStatus = progressStatus
-					void this.updateClineMessage(lastMessage)
+					// Fire-and-forget: webview post is internally guarded, but the
+					// `RooCodeEventName.Message` emit can synchronously throw via a
+					// consumer-attached listener. Surface that as a log, not an
+					// unhandled rejection.
+					this.updateClineMessage(lastMessage).catch((error) => {
+						console.error("[Task#say] updateClineMessage failed:", error)
+					})
 				} else {
 					// This is a new partial message, so add it with partial state.
 					const sayTs = Date.now()
@@ -1713,7 +1727,11 @@ export class Task extends EventEmitter<TaskEvents> implements TaskLike {
 					await this.saveClineMessages()
 
 					// More performant than an entire `postStateToWebview`.
-					void this.updateClineMessage(lastMessage)
+					// Fire-and-forget: see updateClineMessage call above for the
+					// rationale on the .catch arm.
+					this.updateClineMessage(lastMessage).catch((error) => {
+						console.error("[Task#say] updateClineMessage failed:", error)
+					})
 				} else {
 					// This is a new and complete message, so add it like normal.
 					const sayTs = Date.now()
@@ -2365,6 +2383,10 @@ export class Task extends EventEmitter<TaskEvents> implements TaskLike {
 
 	private async initiateTaskLoop(userContent: Anthropic.Messages.ContentBlockParam[]): Promise<void> {
 		// Kicks off the checkpoints initialization process in the background.
+		// `getCheckpointService` wraps its full body in a try/catch and returns
+		// `undefined` on failure (see src/core/checkpoints/index.ts), so the
+		// returned promise cannot reject. `void` is sufficient — no `.catch`
+		// arm needed.
 		void getCheckpointService(this)
 
 		let nextUserContent = userContent

src/core/task/__tests__/Task.spec.ts

@@ -1976,6 +1976,79 @@ describe("Cline", () => {
 			)
 			expect(presentErrors).toHaveLength(0)
 		})
+
+		it("logs (instead of crashing) when updateClineMessage rejects from the say() partial-update path", async () => {
+			// Pins the symmetric .catch arm on the fire-and-forget
+			// updateClineMessage call in say(). The callee's webview post is
+			// internally guarded, but its synchronous emit can throw via a
+			// consumer-attached listener — that path must surface as a log,
+			// not an unhandled rejection.
+			const boom = new Error("updateClineMessage boom")
+			const updateSpy = vi.spyOn(Task.prototype as any, "updateClineMessage").mockImplementation(async () => {
+				throw boom
+			})
+
+			const task = new Task({
+				provider: mockProvider,
+				apiConfiguration: mockApiConfig,
+				task: "test task",
+				startTask: false,
+			})
+
+			// Seed a prior partial "say" so the partial-update branch fires.
+			task.clineMessages.push({
+				ts: Date.now() - 1,
+				type: "say",
+				say: "text",
+				text: "partial",
+				partial: true,
+			})
+
+			await task.say("text", "updated partial", undefined, true)
+			await flushMicrotasks()
+
+			expect(updateSpy).toHaveBeenCalled()
+			expect(consoleErrorSpy).toHaveBeenCalledWith("[Task#say] updateClineMessage failed:", boom)
+			updateSpy.mockRestore()
+		})
+
+		it("logs (instead of crashing) when updateClineMessage rejects from the ask() complete-partial path", async () => {
+			// Pins the symmetric .catch arm on the fire-and-forget
+			// updateClineMessage call in ask() when finalizing a partial.
+			const boom = new Error("updateClineMessage boom")
+			const updateSpy = vi.spyOn(Task.prototype as any, "updateClineMessage").mockImplementation(async () => {
+				throw boom
+			})
+			const saveSpy = vi.spyOn(Task.prototype as any, "saveClineMessages").mockResolvedValue(true)
+
+			const task = new Task({
+				provider: mockProvider,
+				apiConfiguration: mockApiConfig,
+				task: "test task",
+				startTask: false,
+			})
+
+			// Seed a prior partial "ask" of type "tool" so the complete-partial
+			// branch fires when ask("tool", ..., false) is called.
+			task.clineMessages.push({
+				ts: Date.now() - 1,
+				type: "ask",
+				ask: "tool",
+				text: "partial",
+				partial: true,
+			})
+
+			// ask() resolves only after a response — fire-and-forget so the
+			// promise the suite awaits stays bounded. The .catch on the
+			// pending ask handles the never-resolved promise.
+			void task.ask("tool", "complete", false).catch(() => {})
+			await flushMicrotasks()
+
+			expect(updateSpy).toHaveBeenCalled()
+			expect(consoleErrorSpy).toHaveBeenCalledWith("[Task#ask] updateClineMessage failed:", boom)
+			updateSpy.mockRestore()
+			saveSpy.mockRestore()
+		})
 	})
 })