refactor: drop explanatory comments, tighten retry test assertion

Generated-By: PostHog Code
Task-Id: 88d22f31-54f1-4121-bd68-40c5958bb2cd

Author: robbie-c

packages/core/src/sessions/sessionService.ts

@@ -1299,9 +1299,6 @@ export class SessionService {
     session.channel = result.channel;
     session.status = "connected";
     session.adapter = adapter;
-    // Capture the run configuration so retry/reset flows (clearSessionError)
-    // can recreate the session with the user's original choices instead of
-    // falling back to DEFAULT_GATEWAY_MODEL and losing e.g. a Fable selection.
     session.model = model;
     session.executionMode = executionMode;
     session.reasoningLevel = reasoningLevel;
@@ -3413,8 +3410,6 @@ export class SessionService {
           "Unable to reach server. Please check your connection.",
         );
       }
-      // Preserve the original run configuration so the retried session keeps
-      // the user's model/adapter/mode instead of reverting to defaults.
       await this.createNewLocalSession(
         taskId,
         taskTitle,

packages/core/src/sessions/sessionServiceRetryConfig.test.ts

@@ -77,12 +77,16 @@ describe("SessionService.clearSessionError retry config", () => {
 
     await service.clearSessionError("task-1", "/repo");
 
-    // args: taskId, taskTitle, repoPath, auth, initialPrompt,
-    //       executionMode, adapter, model, reasoningLevel
-    const call = createNewLocalSession.mock.calls[0];
-    expect(call[5]).toBe("auto"); // executionMode
-    expect(call[6]).toBe("claude"); // adapter
-    expect(call[7]).toBe("claude-fable-5"); // model
-    expect(call[8]).toBe("high"); // reasoningLevel
+    expect(createNewLocalSession).toHaveBeenCalledWith(
+      "task-1",
+      "Test task",
+      "/repo",
+      { client: {} },
+      session.initialPrompt,
+      "auto", // executionMode
+      "claude", // adapter
+      "claude-fable-5", // model
+      "high", // reasoningLevel
+    );
   });
 });

packages/shared/src/sessions.ts

@@ -64,11 +64,6 @@ export interface AgentSession {
   processedLineCount?: number;
   framework?: "claude";
   adapter?: Adapter;
-  /**
-   * Run configuration captured at session creation so retry/reset flows can
-   * recreate the session with the user's original choices instead of silently
-   * falling back to defaults (e.g. reverting a Fable run to the default model).
-   */
   model?: string;
   executionMode?: ExecutionMode;
   reasoningLevel?: string;