fix: address CodeRabbit review on effort panel and OpenAI setup flow

- EffortPanelWrapper: invoke onDone from useEffect instead of during
  render when the current model does not support effort, avoiding
  cross-component render-phase updates
- ConsoleOAuthFlow: memoize openaiDisplayValues so doOpenAISave and
  handleOpenAIEnter callback references stay stable across renders

Co-Authored-By: Claude <noreply@anthropic.com>

Author: DavidShawa

src/commands/effort/effort.tsx

@@ -174,8 +174,13 @@ export async function call(onDone: LocalJSXCommandOnDone, _context: unknown, arg
 function EffortPanelWrapper({ onDone }: { onDone: (result: string) => void }): React.ReactNode {
   const effortValue = useAppState(s => s.effortValue);
   const model = useMainLoopModel();
-  if (!modelSupportsEffort(model)) {
-    onDone('Effort is disabled because thinking is disabled for the current model.');
+  const supported = modelSupportsEffort(model);
+  React.useEffect(() => {
+    if (!supported) {
+      onDone('Effort is disabled because thinking is disabled for the current model.');
+    }
+  }, [supported, onDone]);
+  if (!supported) {
     return null;
   }
   return <EffortPanel appStateEffort={effortValue} onDone={onDone} />;

src/components/ConsoleOAuthFlow.tsx

@@ -1,4 +1,4 @@
-import React, { useCallback, useEffect, useRef, useState } from 'react';
+import React, { useCallback, useEffect, useMemo, useRef, useState } from 'react';
 import {
   type AnalyticsMetadata_I_VERIFIED_THIS_IS_NOT_CODE_OR_FILEPATHS,
   logEvent,
@@ -825,13 +825,16 @@ function OAuthStatusMessage({
         thinkingEnabled: boolean;
       };
       const { activeField, baseUrl, apiKey, haikuModel, sonnetModel, opusModel, thinkingEnabled } = op;
-      const openaiDisplayValues: Record<Exclude<OpenAIField, 'thinking_enabled'>, string> = {
-        base_url: baseUrl,
-        api_key: apiKey,
-        haiku_model: haikuModel,
-        sonnet_model: sonnetModel,
-        opus_model: opusModel,
-      };
+      const openaiDisplayValues = useMemo<Record<Exclude<OpenAIField, 'thinking_enabled'>, string>>(
+        () => ({
+          base_url: baseUrl,
+          api_key: apiKey,
+          haiku_model: haikuModel,
+          sonnet_model: sonnetModel,
+          opus_model: opusModel,
+        }),
+        [baseUrl, apiKey, haikuModel, sonnetModel, opusModel],
+      );
       const getOpenAIFieldValue = (field: OpenAIField): string =>
         field === 'thinking_enabled' ? '' : openaiDisplayValues[field];