fix(task): scope profile updates to the focused task

Author: edelauna

src/core/task/Task.ts

@@ -703,6 +703,10 @@ export class Task extends EventEmitter<TaskEvents> implements TaskLike {
 		}
 
 		this.providerProfileChangeListener = async () => {
+			if (provider.getCurrentTask()?.taskId !== this.taskId) {
+				return
+			}
+
 			try {
 				const newState = await provider.getState()
 				if (newState?.apiConfiguration) {

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

@@ -8,6 +8,7 @@ import { Anthropic } from "@anthropic-ai/sdk"
 
 import {
 	providerIdentifiers,
+	RooCodeEventName,
 	type GlobalState,
 	type ProviderSettings,
 	type ModelInfo,
@@ -526,6 +527,48 @@ describe("Cline", () => {
 			const metadata = requireDefined(createMessage.mock.calls[0])[2]
 			expect(metadata?.mode).toBe("ask")
 		})
+
+		it("only applies profile changes to the focused task", async () => {
+			const parentConfiguration: ProviderSettings = {
+				...mockApiConfig,
+				apiModelId: "parent-model",
+				rateLimitSeconds: 4,
+			}
+			const childConfiguration: ProviderSettings = {
+				...mockApiConfig,
+				apiModelId: "child-model",
+				rateLimitSeconds: 8,
+			}
+			const activeConfiguration: ProviderSettings = {
+				...mockApiConfig,
+				apiModelId: "active-model",
+				rateLimitSeconds: 12,
+			}
+			const parent = new Task({
+				provider: mockProvider,
+				apiConfiguration: parentConfiguration,
+				taskId: "parent-task",
+				task: "parent task",
+				startTask: false,
+			})
+			const child = new Task({
+				provider: mockProvider,
+				apiConfiguration: childConfiguration,
+				taskId: "child-task",
+				task: "child task",
+				startTask: false,
+			})
+			vi.spyOn(mockProvider, "getCurrentTask").mockReturnValue(child)
+			vi.spyOn(mockProvider, "getState").mockResolvedValue({ apiConfiguration: activeConfiguration })
+
+			mockProvider.emit(RooCodeEventName.ProviderProfileChanged, {
+				name: "active-profile",
+				provider: activeConfiguration.apiProvider,
+			})
+
+			await vi.waitFor(() => expect(child.apiConfiguration).toEqual(activeConfiguration))
+			expect(parent.apiConfiguration).toEqual(parentConfiguration)
+		})
 	})
 
 	describe("sayAndCreateMissingParamError", () => {