test(ci): fixed event-controller and auth-storage test failures

- Mocked messagePersistenceKey in event-controller-error-banner.test.ts and safe-guarded it in event-controller.ts to prevent TypeError.
- Updated thinking loop retry test expectations to handle new dynamic recoveredErrors structure.
- Updated schema version assertions in auth-storage-email-dedupe.test.ts to v5, preserving v6 for future schema test.
- Simulated scrollback commitment in event-controller-message-start.test.ts by rendering container and committing rows before advancing timers.

packages/coding-agent/src/modes/controllers/event-controller.ts

@@ -329,7 +329,8 @@ export class EventController {
 
 	#trackRetrySupersededAssistantComponent(component: AssistantMessageComponent | undefined): void {
 		if (!component) return;
-		const persistenceKey = component.messagePersistenceKey();
+		const persistenceKey =
+			typeof component.messagePersistenceKey === "function" ? component.messagePersistenceKey() : undefined;
 		if (persistenceKey) this.#retrySupersededAssistantComponents.set(persistenceKey, component);
 		if (!this.#retrySupersededAssistantQueue.includes(component)) {
 			this.#retrySupersededAssistantQueue.push(component);
@@ -350,7 +351,8 @@ export class EventController {
 		while (this.#retrySupersededAssistantQueue.length > 0) {
 			const component = this.#retrySupersededAssistantQueue.shift();
 			if (!component) continue;
-			const key = component.messagePersistenceKey();
+			const key =
+				typeof component.messagePersistenceKey === "function" ? component.messagePersistenceKey() : undefined;
 			if (key && this.#retrySupersededAssistantComponents.get(key) !== component) continue;
 			if (key) this.#retrySupersededAssistantComponents.delete(key);
 			return component;