fix(webview): clear durable view state on reset broadcast

Author: easonliang28

src/core/webview/ClineProvider.ts

@@ -3057,15 +3057,17 @@ export class ClineProvider
 	}
 
 	/**
-	 * Broadcast a reset event to all other live ClineProvider instances, clearing their
-	 * view-local state caches and posting updated state so parallel tabs stay in sync.
-	 * Also exposed for use by importSettingsWithFeedback (via broadcastResetToAllInstances callback).
+	 * Broadcast a reset/import invalidation to all live ClineProvider instances, clearing
+	 * both in-memory view-local caches and durable per-view selections so stale view state
+	 * cannot mask imported/reset shared state after reload.
 	 */
 	async broadcastResetToAllInstances(): Promise<void> {
 		const allInstances = ClineProvider.getAllInstances()
 		for (const instance of allInstances) {
+			instance._clearViewLocalState()
+			await instance.contextProxy.setValue("viewStates", undefined)
+
 			if (instance !== this) {
-				instance._clearViewLocalState()
 				await instance.postStateToWebview()
 			}
 		}
@@ -3102,8 +3104,7 @@ export class ClineProvider
 		await this.customModesManager.resetCustomModes()
 		await this.removeClineFromStack()
 
-		// Clear this instance and all other live instances so parallel tabs don't keep stale state.
-		this._clearViewLocalState()
+		// Clear durable and in-memory per-view state across live instances so parallel tabs don't keep stale state.
 		await this.broadcastResetToAllInstances()
 
 		await this.postStateToWebview()

src/core/webview/__tests__/ClineProvider.parallelMode.spec.ts

@@ -1095,8 +1095,10 @@ describe("ClineProvider - Parallel Mode Support", () => {
 
 			await provider1.broadcastResetToAllInstances()
 
-			expect((provider1 as any).viewLocalState.mode).toBe("architect")
+			expect((provider1 as any).viewLocalState).toEqual({})
 			expect((provider2 as any).viewLocalState).toEqual({})
+			expect(provider1.contextProxy.getValue("viewStates" as any)).toBeUndefined()
+			expect(provider2.contextProxy.getValue("viewStates" as any)).toBeUndefined()
 			expect(postStateToWebview2).toHaveBeenCalledTimes(1)
 
 			await provider1.dispose()
@@ -1111,7 +1113,8 @@ describe("ClineProvider - Parallel Mode Support", () => {
 
 			await provider.broadcastResetToAllInstances()
 
-			expect((provider as any).viewLocalState.mode).toBe("architect")
+			expect((provider as any).viewLocalState).toEqual({})
+			expect(provider.contextProxy.getValue("viewStates" as any)).toBeUndefined()
 			expect(postStateToWebview).not.toHaveBeenCalled()
 
 			await provider.dispose()
@@ -1136,9 +1139,12 @@ describe("ClineProvider - Parallel Mode Support", () => {
 
 			await provider1.broadcastResetToAllInstances()
 
-			expect((provider1 as any).viewLocalState.mode).toBe("code")
+			expect((provider1 as any).viewLocalState).toEqual({})
 			expect((provider2 as any).viewLocalState).toEqual({})
 			expect((provider3 as any).viewLocalState).toEqual({})
+			expect(provider1.contextProxy.getValue("viewStates" as any)).toBeUndefined()
+			expect(provider2.contextProxy.getValue("viewStates" as any)).toBeUndefined()
+			expect(provider3.contextProxy.getValue("viewStates" as any)).toBeUndefined()
 			expect(postStateToWebview1).not.toHaveBeenCalled()
 			expect(postStateToWebview2).toHaveBeenCalledTimes(1)
 			expect(postStateToWebview3).toHaveBeenCalledTimes(1)
@@ -1171,6 +1177,8 @@ describe("ClineProvider - Parallel Mode Support", () => {
 
 			expect((provider1 as any).viewLocalState).toEqual({})
 			expect((provider2 as any).viewLocalState).toEqual({})
+			expect(provider1.contextProxy.getValue("viewStates" as any)).toBeUndefined()
+			expect(provider2.contextProxy.getValue("viewStates" as any)).toBeUndefined()
 			expect((provider1.contextProxy as any).resetAllState).toHaveBeenCalledTimes(1)
 			expect((provider1.providerSettingsManager as any).resetAllConfigs).toHaveBeenCalledTimes(1)
 			expect((provider1.customModesManager as any).resetCustomModes).toHaveBeenCalledTimes(1)