From 7b5056168800f4f159bfbcae85d2653cc46ca5bd Mon Sep 17 00:00:00 2001
From: Cursor Agent <cursoragent@cursor.com>
Date: Tue, 17 Mar 2026 00:42:01 +0000
Subject: [PATCH] Remove redundant preferredProvider variable that always
 equals threadProvider

Co-authored-by: Julius Marminge <juliusmarminge@users.noreply.github.com>
---
 .../src/orchestration/Layers/ProviderCommandReactor.ts       | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/apps/server/src/orchestration/Layers/ProviderCommandReactor.ts b/apps/server/src/orchestration/Layers/ProviderCommandReactor.ts
index 38d0b7bbd..3c8fb79bc 100644
--- a/apps/server/src/orchestration/Layers/ProviderCommandReactor.ts
+++ b/apps/server/src/orchestration/Layers/ProviderCommandReactor.ts
@@ -229,7 +229,6 @@ const make = Effect.gen(function* () {
         detail: `Model '${options.model}' does not belong to provider '${threadProvider}' for thread '${threadId}'.`,
       });
     }
-    const preferredProvider: ProviderKind = currentProvider ?? threadProvider;
     const desiredModel = options?.model ?? thread.model;
     const effectiveCwd = resolveThreadWorkspaceCwd({
       thread,
@@ -247,8 +246,8 @@ const make = Effect.gen(function* () {
     }) =>
       providerService.startSession(threadId, {
         threadId,
-        ...((input?.provider ?? preferredProvider)
-          ? { provider: input?.provider ?? preferredProvider }
+        ...((input?.provider ?? threadProvider)
+          ? { provider: input?.provider ?? threadProvider }
           : {}),
         ...(effectiveCwd ? { cwd: effectiveCwd } : {}),
         ...(desiredModel ? { model: desiredModel } : {}),