🤖 feat: disable auto truncation by default for OpenAI models

Change the default value of disableAutoTruncation from false to true,
so the dev-only 'No Trunc' checkbox is checked by default.

Author: ibetitsmike

src/browser/contexts/ProviderOptionsContext.tsx

@@ -1,5 +1,5 @@
-import React, { createContext, useContext } from "react";
-import { usePersistedState } from "@/browser/hooks/usePersistedState";
+import React, { createContext, useContext, useLayoutEffect } from "react";
+import { updatePersistedState, usePersistedState } from "@/browser/hooks/usePersistedState";
 import type { MuxProviderOptions } from "@/common/types/providerOptions";
 
 interface ProviderOptionsContextType {
@@ -11,6 +11,10 @@ interface ProviderOptionsContextType {
 
 const ProviderOptionsContext = createContext<ProviderOptionsContextType | undefined>(undefined);
 
+// Force disableAutoTruncation to true on startup (overrides any stored value)
+const OPENAI_OPTIONS_KEY = "provider_options_openai";
+const FORCED_OPENAI_OPTIONS: MuxProviderOptions["openai"] = { disableAutoTruncation: true };
+
 export function ProviderOptionsProvider({ children }: { children: React.ReactNode }) {
   const [anthropicOptions, setAnthropicOptions] = usePersistedState<
     MuxProviderOptions["anthropic"]
@@ -19,12 +23,17 @@ export function ProviderOptionsProvider({ children }: { children: React.ReactNod
   });
 
   const [openaiOptions, setOpenAIOptions] = usePersistedState<MuxProviderOptions["openai"]>(
-    "provider_options_openai",
-    {
-      disableAutoTruncation: false,
-    }
+    OPENAI_OPTIONS_KEY,
+    FORCED_OPENAI_OPTIONS
   );
 
+  // Force disableAutoTruncation to true on every app start
+  useLayoutEffect(() => {
+    if (!openaiOptions?.disableAutoTruncation) {
+      updatePersistedState(OPENAI_OPTIONS_KEY, FORCED_OPENAI_OPTIONS);
+    }
+  }, []);
+
   const [googleOptions, setGoogleOptions] = usePersistedState<MuxProviderOptions["google"]>(
     "provider_options_google",
     {}

src/browser/utils/messages/sendOptions.ts

@@ -26,7 +26,7 @@ function getProviderOptions(): MuxProviderOptions {
     { use1MContext: false }
   );
   const openai = readPersistedState<MuxProviderOptions["openai"]>("provider_options_openai", {
-    disableAutoTruncation: false,
+    disableAutoTruncation: true,
   });
   const google = readPersistedState<MuxProviderOptions["google"]>("provider_options_google", {});
 

tests/ipc/sendMessage.heavy.test.ts

@@ -41,13 +41,21 @@ describeIntegration("sendMessage heavy/load tests", () => {
         await withSharedWorkspace(provider, async ({ env, workspaceId, collector }) => {
           // Build up large conversation history to exceed context limit
           // This approach is model-agnostic - it keeps sending until we've built up enough history
+          // Use auto-truncation enabled (disableAutoTruncation: false) so history builds up successfully
           const largeMessage = "x".repeat(50_000);
           for (let i = 0; i < 10; i++) {
             await sendMessageWithModel(
               env,
               workspaceId,
               `Message ${i}: ${largeMessage}`,
-              modelString(provider, model)
+              modelString(provider, model),
+              {
+                providerOptions: {
+                  openai: {
+                    disableAutoTruncation: false,
+                  },
+                },
+              }
             );
             await collector.waitForEvent("stream-end", 30000);
             collector.clear();
@@ -93,8 +101,14 @@ describeIntegration("sendMessage heavy/load tests", () => {
             env,
             workspaceId,
             "This should succeed with auto-truncation",
-            modelString(provider, model)
-            // disableAutoTruncation defaults to false (auto-truncation enabled)
+            modelString(provider, model),
+            {
+              providerOptions: {
+                openai: {
+                  disableAutoTruncation: false,
+                },
+              },
+            }
           );
 
           expect(successResult.success).toBe(true);