From c8f22c9dcb1e255bc47c765f743c900e7c97d02b Mon Sep 17 00:00:00 2001 From: OrcaWin Date: Wed, 28 Oct 2026 21:15:38 -0700 Subject: [PATCH] perf(mobile): skip model projection after discovery owner retires (#35540) * perf(mobile): skip model projection for retired discovery owners * test(mobile): drop spy-only model discovery projection test Existing hook suite already covers stale-result exclusion; the new file only asserted a namespace spy call. --------- Co-authored-by: OrcaWin <293788423+OrcaWin@users.noreply.github.com> --- mobile/src/session/use-mobile-native-chat-model-discovery.ts | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/mobile/src/session/use-mobile-native-chat-model-discovery.ts b/mobile/src/session/use-mobile-native-chat-model-discovery.ts index 2d65e37210e..0363bef7059 100644 --- a/mobile/src/session/use-mobile-native-chat-model-discovery.ts +++ b/mobile/src/session/use-mobile-native-chat-model-discovery.ts @@ -68,8 +68,11 @@ export function useMobileNativeChatModelDiscovery(args: { let cancelled = false void runRpcOperation(client, discoverModels, { worktree: `id:${worktreeId}`, agentId: agent }) .then((discovery) => { + if (cancelled) { + return + } const models = discovery ? catalogModelsFromModelDiscovery(agent, discovery) : null - if (!cancelled && models !== null) { + if (models !== null) { setResult({ client, scope, models: mergeDiscoveredCatalogModels(agent, catalog, models) }) } })