@tambo-ai/react 0.69.1 → 0.71.0

package/dist/providers/tambo-thread-provider.test.js

@@ -1,9 +1,42 @@
 "use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-const typescript_sdk_1 = require("@tambo-ai/typescript-sdk");
+const typescript_sdk_1 = __importStar(require("@tambo-ai/typescript-sdk"));
 const react_1 = require("@testing-library/react");
 const react_2 = __importDefault(require("react"));
 const v4_1 = require("zod/v4");
@@ -28,9 +61,14 @@ jest.mock("./tambo-client-provider", () => {
         TamboClientContext: react_2.default.createContext(undefined),
     };
 });
-jest.mock("@tambo-ai/typescript-sdk", () => ({
-    advanceStream: jest.fn(),
-}));
+jest.mock("@tambo-ai/typescript-sdk", () => {
+    const actual = jest.requireActual("@tambo-ai/typescript-sdk");
+    return {
+        __esModule: true,
+        ...actual,
+        advanceStream: jest.fn(),
+    };
+});
 // Mock the getCustomContext
 jest.mock("../util/registry", () => ({
     ...jest.requireActual("../util/registry"),
@@ -73,26 +111,9 @@ const createMockAdvanceResponse = (overrides = {}) => ({
 });
 describe("TamboThreadProvider", () => {
     const mockThread = createMockThread();
-    const mockThreadsApi = {
-        messages: {
-            create: jest.fn(),
-        },
-        retrieve: jest.fn(),
-        advance: jest.fn(),
-        advanceByID: jest.fn(),
-        generateName: jest.fn(),
-    };
-    const mockProjectsApi = {
-        getCurrent: jest.fn(),
-    };
-    const mockBeta = {
-        threads: mockThreadsApi,
-        projects: mockProjectsApi,
-    };
-    const mockTamboAI = {
-        apiKey: "",
-        beta: mockBeta,
-    };
+    let mockTamboAI;
+    let mockThreadsApi;
+    let mockProjectsApi;
     let mockQueryClient;
     const mockRegistry = [
         {
@@ -144,8 +165,19 @@ describe("TamboThreadProvider", () => {
     };
     // Default wrapper for most tests
     const Wrapper = createWrapper();
+    afterEach(() => {
+        jest.restoreAllMocks();
+    });
     beforeEach(() => {
         jest.clearAllMocks();
+        mockTamboAI = new typescript_sdk_1.default({
+            apiKey: "",
+            fetch: () => {
+                throw new Error("Unexpected network call in test");
+            },
+        });
+        mockThreadsApi = mockTamboAI.beta.threads;
+        mockProjectsApi = mockTamboAI.beta.projects;
         // Setup mock query client
         mockQueryClient = {
             invalidateQueries: jest.fn().mockResolvedValue(undefined),
@@ -154,21 +186,22 @@ describe("TamboThreadProvider", () => {
         jest
             .mocked(tambo_client_provider_1.useTamboQueryClient)
             .mockReturnValue(mockQueryClient);
-        jest.mocked(mockThreadsApi.retrieve).mockResolvedValue(mockThread);
+        jest.spyOn(mockThreadsApi, "retrieve").mockResolvedValue(mockThread);
         jest
-            .mocked(mockThreadsApi.messages.create)
+            .spyOn(mockThreadsApi.messages, "create")
             .mockResolvedValue(createMockMessage());
         jest
-            .mocked(mockThreadsApi.advance)
+            .spyOn(mockThreadsApi, "advance")
             .mockResolvedValue(createMockAdvanceResponse());
         jest
-            .mocked(mockThreadsApi.advanceByID)
+            .spyOn(mockThreadsApi, "advanceByID")
             .mockResolvedValue(createMockAdvanceResponse());
-        jest.mocked(mockThreadsApi.generateName).mockResolvedValue({
+        jest.spyOn(mockThreadsApi, "generateName").mockResolvedValue({
             ...mockThread,
             name: "Generated Thread Name",
         });
-        jest.mocked(mockProjectsApi.getCurrent).mockResolvedValue({
+        jest.spyOn(mockThreadsApi, "update").mockResolvedValue({});
+        jest.spyOn(mockProjectsApi, "getCurrent").mockResolvedValue({
             id: "test-project-id",
             name: "Test Project",
             isTokenRequired: false,
@@ -236,7 +269,7 @@ describe("TamboThreadProvider", () => {
         });
     });
     it("should send a message and update thread state", async () => {
-        const mockAdvanceResponse = {
+        const mockStreamResponse = {
             responseMessageDto: {
                 id: "response-1",
                 content: [{ type: "text", text: "Response" }],
@@ -249,14 +282,17 @@ describe("TamboThreadProvider", () => {
             generationStage: generate_component_response_1.GenerationStage.COMPLETE,
             mcpAccessToken: "test-mcp-access-token",
         };
-        jest
-            .mocked(mockThreadsApi.advanceByID)
-            .mockResolvedValue(mockAdvanceResponse);
+        const mockAsyncIterator = {
+            [Symbol.asyncIterator]: async function* () {
+                yield mockStreamResponse;
+            },
+        };
+        jest.mocked(typescript_sdk_1.advanceStream).mockResolvedValue(mockAsyncIterator);
         const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), { wrapper: Wrapper });
         await (0, react_1.act)(async () => {
             await result.current.sendThreadMessage("Hello", {
                 threadId: "test-thread-1",
-                streamResponse: false,
+                streamResponse: true,
                 additionalContext: {
                     custom: {
                         message: "additional instructions",
@@ -264,7 +300,7 @@ describe("TamboThreadProvider", () => {
                 },
             });
         });
-        expect(mockThreadsApi.advanceByID).toHaveBeenCalledWith("test-thread-1", {
+        expect(typescript_sdk_1.advanceStream).toHaveBeenCalledWith(expect.anything(), {
             messageToAppend: {
                 content: [{ type: "text", text: "Hello" }],
                 role: "user",
@@ -278,7 +314,7 @@ describe("TamboThreadProvider", () => {
             contextKey: undefined,
             clientTools: [],
             toolCallCounts: {},
-        });
+        }, "test-thread-1");
         expect(result.current.generationStage).toBe(generate_component_response_1.GenerationStage.COMPLETE);
     });
     it("should handle streaming responses", async () => {
@@ -313,7 +349,7 @@ describe("TamboThreadProvider", () => {
         expect(result.current.generationStage).toBe(generate_component_response_1.GenerationStage.COMPLETE);
     });
     it("should handle tool calls during message processing.", async () => {
-        const mockToolCallResponse = {
+        const mockToolCallChunk = {
             responseMessageDto: {
                 id: "tool-call-1",
                 content: [{ type: "text", text: "Tool response" }],
@@ -329,10 +365,7 @@ describe("TamboThreadProvider", () => {
             generationStage: generate_component_response_1.GenerationStage.COMPLETE,
             mcpAccessToken: "test-mcp-access-token",
         };
-        jest
-            .mocked(mockThreadsApi.advanceByID)
-            .mockResolvedValueOnce(mockToolCallResponse)
-            .mockResolvedValueOnce({
+        const mockFinalChunk = {
             responseMessageDto: {
                 id: "advance-response2",
                 content: [{ type: "text", text: "response 2" }],
@@ -343,12 +376,26 @@ describe("TamboThreadProvider", () => {
             },
             generationStage: generate_component_response_1.GenerationStage.COMPLETE,
             mcpAccessToken: "test-mcp-access-token",
-        });
+        };
+        const mockAsyncIterator = {
+            [Symbol.asyncIterator]: async function* () {
+                yield mockToolCallChunk;
+            },
+        };
+        const mockAsyncIterator2 = {
+            [Symbol.asyncIterator]: async function* () {
+                yield mockFinalChunk;
+            },
+        };
+        jest
+            .mocked(typescript_sdk_1.advanceStream)
+            .mockResolvedValueOnce(mockAsyncIterator)
+            .mockResolvedValueOnce(mockAsyncIterator2);
         const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), { wrapper: Wrapper });
         await (0, react_1.act)(async () => {
             await result.current.sendThreadMessage("Use tool", {
                 threadId: "test-thread-1",
-                streamResponse: false,
+                streamResponse: true,
             });
         });
         expect(result.current.generationStage).toBe(generate_component_response_1.GenerationStage.COMPLETE);
@@ -361,7 +408,7 @@ describe("TamboThreadProvider", () => {
         const mockOnCallUnregisteredTool = jest
             .fn()
             .mockResolvedValue("unregistered-tool-result");
-        const mockUnregisteredToolCallResponse = {
+        const mockUnregisteredToolCallChunk = {
             responseMessageDto: {
                 id: "unregistered-tool-call-1",
                 content: [{ type: "text", text: "Unregistered tool response" }],
@@ -379,10 +426,7 @@ describe("TamboThreadProvider", () => {
             generationStage: generate_component_response_1.GenerationStage.COMPLETE,
             mcpAccessToken: "test-mcp-access-token",
         };
-        jest
-            .mocked(mockThreadsApi.advanceByID)
-            .mockResolvedValueOnce(mockUnregisteredToolCallResponse)
-            .mockResolvedValueOnce({
+        const mockFinalChunk = {
             responseMessageDto: {
                 id: "advance-response2",
                 content: [{ type: "text", text: "response 2" }],
@@ -393,7 +437,21 @@ describe("TamboThreadProvider", () => {
             },
             generationStage: generate_component_response_1.GenerationStage.COMPLETE,
             mcpAccessToken: "test-mcp-access-token",
-        });
+        };
+        const mockAsyncIterator = {
+            [Symbol.asyncIterator]: async function* () {
+                yield mockUnregisteredToolCallChunk;
+            },
+        };
+        const mockAsyncIterator2 = {
+            [Symbol.asyncIterator]: async function* () {
+                yield mockFinalChunk;
+            },
+        };
+        jest
+            .mocked(typescript_sdk_1.advanceStream)
+            .mockResolvedValueOnce(mockAsyncIterator)
+            .mockResolvedValueOnce(mockAsyncIterator2);
         const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), {
             wrapper: createWrapper({
                 onCallUnregisteredTool: mockOnCallUnregisteredTool,
@@ -402,14 +460,14 @@ describe("TamboThreadProvider", () => {
         await (0, react_1.act)(async () => {
             await result.current.sendThreadMessage("Use unregistered tool", {
                 threadId: "test-thread-1",
-                streamResponse: false,
+                streamResponse: true,
             });
         });
         expect(result.current.generationStage).toBe(generate_component_response_1.GenerationStage.COMPLETE);
         expect(mockOnCallUnregisteredTool).toHaveBeenCalledWith("unregistered-tool", [{ parameterName: "input", parameterValue: "test-input" }]);
     });
     it("should handle unregistered tool calls without onCallUnregisteredTool", async () => {
-        const mockUnregisteredToolCallResponse = {
+        const mockUnregisteredToolCallChunk = {
             responseMessageDto: {
                 id: "unregistered-tool-call-1",
                 content: [{ type: "text", text: "Unregistered tool response" }],
@@ -427,10 +485,7 @@ describe("TamboThreadProvider", () => {
             generationStage: generate_component_response_1.GenerationStage.COMPLETE,
             mcpAccessToken: "test-mcp-access-token",
         };
-        jest
-            .mocked(mockThreadsApi.advanceByID)
-            .mockResolvedValueOnce(mockUnregisteredToolCallResponse)
-            .mockResolvedValueOnce({
+        const mockFinalChunk = {
             responseMessageDto: {
                 id: "advance-response2",
                 content: [{ type: "text", text: "response 2" }],
@@ -441,12 +496,26 @@ describe("TamboThreadProvider", () => {
             },
             generationStage: generate_component_response_1.GenerationStage.COMPLETE,
             mcpAccessToken: "test-mcp-access-token",
-        });
+        };
+        const mockAsyncIterator = {
+            [Symbol.asyncIterator]: async function* () {
+                yield mockUnregisteredToolCallChunk;
+            },
+        };
+        const mockAsyncIterator2 = {
+            [Symbol.asyncIterator]: async function* () {
+                yield mockFinalChunk;
+            },
+        };
+        jest
+            .mocked(typescript_sdk_1.advanceStream)
+            .mockResolvedValueOnce(mockAsyncIterator)
+            .mockResolvedValueOnce(mockAsyncIterator2);
         const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), { wrapper: Wrapper });
         await (0, react_1.act)(async () => {
             await result.current.sendThreadMessage("Use unregistered tool", {
                 threadId: "test-thread-1",
-                streamResponse: false,
+                streamResponse: true,
             });
         });
         expect(result.current.generationStage).toBe(generate_component_response_1.GenerationStage.COMPLETE);
@@ -507,77 +576,16 @@ describe("TamboThreadProvider", () => {
             expect(mockThreadsApi.advance).not.toHaveBeenCalled();
             expect(mockThreadsApi.advanceByID).not.toHaveBeenCalled();
         });
-        it("should call advanceById when streamResponse=false for existing thread", async () => {
-            // Use wrapper with streaming=true to show that explicit streamResponse=false overrides provider setting
+        it("should throw error when streamResponse=false (non-streaming not supported)", async () => {
             const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), {
                 wrapper: createWrapper({ streaming: true }),
             });
             await (0, react_1.act)(async () => {
-                await result.current.sendThreadMessage("Hello non-streaming", {
+                await expect(result.current.sendThreadMessage("Hello non-streaming", {
                     threadId: "test-thread-1",
                     streamResponse: false,
-                    additionalContext: {
-                        custom: {
-                            message: "additional instructions",
-                        },
-                    },
-                });
-            });
-            expect(mockThreadsApi.advanceByID).toHaveBeenCalledWith("test-thread-1", {
-                messageToAppend: {
-                    content: [{ type: "text", text: "Hello non-streaming" }],
-                    role: "user",
-                    additionalContext: {
-                        custom: {
-                            message: "additional instructions",
-                        },
-                    },
-                },
-                availableComponents: (0, tools_1.serializeRegistry)(mockRegistry),
-                contextKey: undefined,
-                clientTools: [],
-                forceToolChoice: undefined,
-                toolCallCounts: {},
-            });
-            // Should not call advance or advanceStream
-            expect(mockThreadsApi.advance).not.toHaveBeenCalled();
-            expect(typescript_sdk_1.advanceStream).not.toHaveBeenCalled();
-        });
-        it("should call advanceById when streamResponse is undefined and provider streaming=false", async () => {
-            // Use wrapper with streaming=false to test that undefined streamResponse respects provider setting
-            const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), {
-                wrapper: createWrapper({ streaming: false }),
-            });
-            await (0, react_1.act)(async () => {
-                await result.current.sendThreadMessage("Hello default", {
-                    threadId: "test-thread-1",
-                    // streamResponse is undefined, should use provider's streaming=false
-                    additionalContext: {
-                        custom: {
-                            message: "additional instructions",
-                        },
-                    },
-                });
-            });
-            expect(mockThreadsApi.advanceByID).toHaveBeenCalledWith("test-thread-1", {
-                messageToAppend: {
-                    content: [{ type: "text", text: "Hello default" }],
-                    role: "user",
-                    additionalContext: {
-                        custom: {
-                            message: "additional instructions",
-                        },
-                    },
-                },
-                availableComponents: (0, tools_1.serializeRegistry)(mockRegistry),
-                contextKey: undefined,
-                clientTools: [],
-                forceToolChoice: undefined,
-                toolCallCounts: {},
+                })).rejects.toThrow();
             });
-            // Should not call advance or advanceStream
-            expect(mockThreadsApi.advance).not.toHaveBeenCalled();
-            expect(typescript_sdk_1.advanceStream).not.toHaveBeenCalled();
         });
         it("should call advanceStream when streamResponse is undefined and provider streaming=true (default)", async () => {
             const mockStreamResponse = {
@@ -633,44 +641,6 @@ describe("TamboThreadProvider", () => {
             expect(mockThreadsApi.advance).not.toHaveBeenCalled();
             expect(mockThreadsApi.advanceByID).not.toHaveBeenCalled();
         });
-        it("should call advance when streamResponse=false for placeholder thread", async () => {
-            // Use wrapper with streaming=true to show that explicit streamResponse=false overrides provider setting
-            const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), {
-                wrapper: createWrapper({ streaming: true }),
-            });
-            // Start with placeholder thread (which is the default state)
-            expect(result.current.thread.id).toBe("placeholder");
-            await (0, react_1.act)(async () => {
-                await result.current.sendThreadMessage("Hello new thread", {
-                    threadId: "placeholder",
-                    streamResponse: false,
-                    additionalContext: {
-                        custom: {
-                            message: "additional instructions",
-                        },
-                    },
-                });
-            });
-            expect(mockThreadsApi.advance).toHaveBeenCalledWith({
-                messageToAppend: {
-                    content: [{ type: "text", text: "Hello new thread" }],
-                    role: "user",
-                    additionalContext: {
-                        custom: {
-                            message: "additional instructions",
-                        },
-                    },
-                },
-                availableComponents: (0, tools_1.serializeRegistry)(mockRegistry),
-                contextKey: undefined,
-                clientTools: [],
-                forceToolChoice: undefined,
-                toolCallCounts: {},
-            });
-            // Should not call advanceById or advanceStream
-            expect(mockThreadsApi.advanceByID).not.toHaveBeenCalled();
-            expect(typescript_sdk_1.advanceStream).not.toHaveBeenCalled();
-        });
         it("should call advanceStream when streamResponse=true for placeholder thread", async () => {
             const mockStreamResponse = {
                 responseMessageDto: {
@@ -817,10 +787,10 @@ describe("TamboThreadProvider", () => {
         });
     });
     describe("error handling", () => {
-        it("should set generation stage to ERROR when non-streaming sendThreadMessage fails", async () => {
-            const testError = new Error("API call failed");
-            // Mock advanceById to throw an error
-            jest.mocked(mockThreadsApi.advanceByID).mockRejectedValue(testError);
+        it("should set generation stage to ERROR when streaming sendThreadMessage fails", async () => {
+            const testError = new Error("Streaming API call failed");
+            // Mock advanceStream to throw an error
+            jest.mocked(typescript_sdk_1.advanceStream).mockRejectedValue(testError);
             const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), {
                 wrapper: Wrapper,
             });
@@ -829,48 +799,91 @@ describe("TamboThreadProvider", () => {
                 await result.current.switchCurrentThread("test-thread-1");
                 await expect(result.current.sendThreadMessage("Hello", {
                     threadId: "test-thread-1",
-                    streamResponse: false,
-                })).rejects.toThrow("API call failed");
+                    streamResponse: true,
+                })).rejects.toThrow("Streaming API call failed");
             });
             // Verify generation stage is set to ERROR
             expect(result.current.generationStage).toBe(generate_component_response_1.GenerationStage.ERROR);
         });
-        it("should set generation stage to ERROR when streaming sendThreadMessage fails", async () => {
-            const testError = new Error("Streaming API call failed");
-            // Mock advanceStream to throw an error
+        it("should rollback optimistic user message when sendThreadMessage fails", async () => {
+            const testError = new Error("API call failed");
             jest.mocked(typescript_sdk_1.advanceStream).mockRejectedValue(testError);
             const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), {
                 wrapper: Wrapper,
             });
-            // Expect the error to be thrown
             await (0, react_1.act)(async () => {
                 await result.current.switchCurrentThread("test-thread-1");
+            });
+            const initialMessageCount = result.current.thread.messages.length;
+            await (0, react_1.act)(async () => {
                 await expect(result.current.sendThreadMessage("Hello", {
                     threadId: "test-thread-1",
                     streamResponse: true,
-                })).rejects.toThrow("Streaming API call failed");
+                })).rejects.toThrow("API call failed");
             });
-            // Verify generation stage is set to ERROR
-            expect(result.current.generationStage).toBe(generate_component_response_1.GenerationStage.ERROR);
+            // Verify user message was rolled back
+            expect(result.current.thread.messages.length).toBe(initialMessageCount);
         });
-        it("should set generation stage to ERROR when advance API call fails for placeholder thread", async () => {
-            const testError = new Error("Advance API call failed");
-            // Mock advance to throw an error
-            jest.mocked(mockThreadsApi.advance).mockRejectedValue(testError);
+        it("should rollback optimistic message when addThreadMessage fails", async () => {
+            const testError = new Error("Create message failed");
+            jest.mocked(mockThreadsApi.messages.create).mockRejectedValue(testError);
             const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), {
                 wrapper: Wrapper,
             });
-            // Start with placeholder thread (which is the default state)
-            expect(result.current.thread.id).toBe("placeholder");
-            // Expect the error to be thrown
             await (0, react_1.act)(async () => {
-                await expect(result.current.sendThreadMessage("Hello", {
-                    threadId: "placeholder",
-                    streamResponse: false,
-                })).rejects.toThrow("Advance API call failed");
+                await result.current.switchCurrentThread("test-thread-1");
             });
-            // Verify generation stage is set to ERROR
-            expect(result.current.generationStage).toBe(generate_component_response_1.GenerationStage.ERROR);
+            const initialMessageCount = result.current.thread.messages.length;
+            const newMessage = createMockMessage({ threadId: "test-thread-1" });
+            await (0, react_1.act)(async () => {
+                await expect(result.current.addThreadMessage(newMessage, true)).rejects.toThrow("Create message failed");
+            });
+            // Verify message was rolled back
+            expect(result.current.thread.messages.length).toBe(initialMessageCount);
+        });
+        it("should rollback optimistic update when updateThreadMessage fails", async () => {
+            const testError = new Error("Update message failed");
+            jest.mocked(mockThreadsApi.messages.create).mockRejectedValue(testError);
+            const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), {
+                wrapper: Wrapper,
+            });
+            await (0, react_1.act)(async () => {
+                await result.current.switchCurrentThread("test-thread-1");
+            });
+            const existingMessage = createMockMessage({
+                id: "existing-msg",
+                threadId: "test-thread-1",
+                content: [{ type: "text", text: "Old content" }],
+            });
+            await (0, react_1.act)(async () => {
+                await result.current.addThreadMessage(existingMessage, false);
+            });
+            const initialMessageCount = result.current.thread.messages.length;
+            await (0, react_1.act)(async () => {
+                await expect(result.current.updateThreadMessage("existing-msg", {
+                    threadId: "test-thread-1",
+                    content: [{ type: "text", text: "New content" }],
+                    role: "assistant",
+                }, true)).rejects.toThrow("Update message failed");
+            });
+            // Verify message was rolled back
+            expect(result.current.thread.messages.length).toBe(initialMessageCount - 1);
+        });
+        it("should rollback optimistic name update when updateThreadName fails", async () => {
+            const testError = new Error("Update name failed");
+            jest.mocked(mockThreadsApi.update).mockRejectedValue(testError);
+            const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), {
+                wrapper: Wrapper,
+            });
+            await (0, react_1.act)(async () => {
+                await result.current.switchCurrentThread("test-thread-1");
+            });
+            const initialName = result.current.thread.name;
+            await (0, react_1.act)(async () => {
+                await expect(result.current.updateThreadName("New Name", "test-thread-1")).rejects.toThrow("Update name failed");
+            });
+            // Verify name was rolled back
+            expect(result.current.thread.name).toBe(initialName);
         });
     });
     describe("refetch threads list behavior", () => {
@@ -878,8 +891,8 @@ describe("TamboThreadProvider", () => {
             const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), {
                 wrapper: Wrapper,
             });
-            // Mock the advance response to return a new thread ID
-            const mockAdvanceResponse = {
+            // Mock the stream response to return a new thread ID
+            const mockStreamResponse = {
                 responseMessageDto: {
                     id: "response-1",
                     content: [{ type: "text", text: "Response" }],
@@ -892,16 +905,19 @@ describe("TamboThreadProvider", () => {
                 generationStage: generate_component_response_1.GenerationStage.COMPLETE,
                 mcpAccessToken: "test-mcp-access-token",
             };
-            jest
-                .mocked(mockThreadsApi.advance)
-                .mockResolvedValue(mockAdvanceResponse);
+            const mockAsyncIterator = {
+                [Symbol.asyncIterator]: async function* () {
+                    yield mockStreamResponse;
+                },
+            };
+            jest.mocked(typescript_sdk_1.advanceStream).mockResolvedValue(mockAsyncIterator);
             // Start with placeholder thread
             expect(result.current.thread.id).toBe("placeholder");
             // Send a message which will create a new thread with contextKey
             await (0, react_1.act)(async () => {
                 await result.current.sendThreadMessage("Hello", {
                     threadId: "placeholder",
-                    streamResponse: false,
+                    streamResponse: true,
                     contextKey: "test-context-key",
                 });
             });
@@ -939,93 +955,6 @@ describe("TamboThreadProvider", () => {
         });
     });
     describe("transformToContent", () => {
-        it("should use custom transformToContent when provided (non-streaming)", async () => {
-            const mockTransformToContent = jest.fn().mockReturnValue([
-                { type: "text", text: "Custom transformed content" },
-                {
-                    type: "image_url",
-                    image_url: { url: "https://example.com/image.png" },
-                },
-            ]);
-            const customToolRegistry = [
-                {
-                    name: "TestComponent",
-                    component: () => react_2.default.createElement("div", null, "Test"),
-                    description: "Test",
-                    propsSchema: v4_1.z.object({ test: v4_1.z.string() }),
-                    associatedTools: [
-                        {
-                            name: "custom-tool",
-                            tool: jest.fn().mockResolvedValue({ data: "tool result" }),
-                            description: "Tool with custom transform",
-                            inputSchema: v4_1.z.object({ input: v4_1.z.string() }),
-                            outputSchema: v4_1.z.object({ data: v4_1.z.string() }),
-                            transformToContent: mockTransformToContent,
-                        },
-                    ],
-                },
-            ];
-            const mockToolCallResponse = {
-                responseMessageDto: {
-                    id: "tool-call-1",
-                    content: [{ type: "text", text: "Tool response" }],
-                    role: "tool",
-                    threadId: "test-thread-1",
-                    toolCallRequest: {
-                        toolName: "custom-tool",
-                        parameters: [{ parameterName: "input", parameterValue: "test" }],
-                    },
-                    componentState: {},
-                    createdAt: new Date().toISOString(),
-                },
-                generationStage: generate_component_response_1.GenerationStage.COMPLETE,
-                mcpAccessToken: "test-mcp-access-token",
-            };
-            jest
-                .mocked(mockThreadsApi.advanceByID)
-                .mockResolvedValueOnce(mockToolCallResponse)
-                .mockResolvedValueOnce({
-                responseMessageDto: {
-                    id: "final-response",
-                    content: [{ type: "text", text: "Final response" }],
-                    role: "assistant",
-                    threadId: "test-thread-1",
-                    componentState: {},
-                    createdAt: new Date().toISOString(),
-                },
-                generationStage: generate_component_response_1.GenerationStage.COMPLETE,
-                mcpAccessToken: "test-mcp-access-token",
-            });
-            const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), {
-                wrapper: createWrapper({ components: customToolRegistry }),
-            });
-            await (0, react_1.act)(async () => {
-                await result.current.sendThreadMessage("Use custom tool", {
-                    threadId: "test-thread-1",
-                    streamResponse: false,
-                });
-            });
-            // Verify the tool was called with single object arg (new inputSchema interface)
-            expect(customToolRegistry[0]?.associatedTools?.[0]?.tool).toHaveBeenCalledWith({ input: "test" });
-            // Verify transformToContent was called with the tool result
-            expect(mockTransformToContent).toHaveBeenCalledWith({
-                data: "tool result",
-            });
-            // Verify the second advance call included the transformed content
-            expect(mockThreadsApi.advanceByID).toHaveBeenCalledTimes(2);
-            expect(mockThreadsApi.advanceByID).toHaveBeenLastCalledWith("test-thread-1", expect.objectContaining({
-                messageToAppend: expect.objectContaining({
-                    content: [
-                        { type: "text", text: "Custom transformed content" },
-                        {
-                            type: "image_url",
-                            image_url: { url: "https://example.com/image.png" },
-                        },
-                    ],
-                    role: "tool",
-                }),
-            }));
-        });
         it("should use custom async transformToContent when provided (streaming)", async () => {
             const mockTransformToContent = jest
                 .fn()
@@ -1122,169 +1051,6 @@ describe("TamboThreadProvider", () => {
                 }),
             }), "test-thread-1");
         });
-        it("should fallback to stringified text when transformToContent is not provided", async () => {
-            const toolWithoutTransform = [
-                {
-                    name: "TestComponent",
-                    component: () => react_2.default.createElement("div", null, "Test"),
-                    description: "Test",
-                    propsSchema: v4_1.z.object({ test: v4_1.z.string() }),
-                    associatedTools: [
-                        {
-                            name: "no-transform-tool",
-                            tool: jest
-                                .fn()
-                                .mockResolvedValue({ complex: "data", nested: { value: 42 } }),
-                            description: "Tool without custom transform",
-                            inputSchema: v4_1.z.object({ input: v4_1.z.string() }),
-                            outputSchema: v4_1.z.object({
-                                complex: v4_1.z.string(),
-                                nested: v4_1.z.object({ value: v4_1.z.number() }),
-                            }),
-                            // No transformToContent provided
-                        },
-                    ],
-                },
-            ];
-            const mockToolCallResponse = {
-                responseMessageDto: {
-                    id: "tool-call-1",
-                    content: [{ type: "text", text: "Tool call" }],
-                    role: "tool",
-                    threadId: "test-thread-1",
-                    toolCallRequest: {
-                        toolName: "no-transform-tool",
-                        parameters: [{ parameterName: "input", parameterValue: "test" }],
-                    },
-                    componentState: {},
-                    createdAt: new Date().toISOString(),
-                },
-                generationStage: generate_component_response_1.GenerationStage.COMPLETE,
-                mcpAccessToken: "test-mcp-access-token",
-            };
-            jest
-                .mocked(mockThreadsApi.advanceByID)
-                .mockResolvedValueOnce(mockToolCallResponse)
-                .mockResolvedValueOnce({
-                responseMessageDto: {
-                    id: "final-response",
-                    content: [{ type: "text", text: "Final response" }],
-                    role: "assistant",
-                    threadId: "test-thread-1",
-                    componentState: {},
-                    createdAt: new Date().toISOString(),
-                },
-                generationStage: generate_component_response_1.GenerationStage.COMPLETE,
-                mcpAccessToken: "test-mcp-access-token",
-            });
-            const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), {
-                wrapper: createWrapper({ components: toolWithoutTransform }),
-            });
-            await (0, react_1.act)(async () => {
-                await result.current.sendThreadMessage("Use tool without transform", {
-                    threadId: "test-thread-1",
-                    streamResponse: false,
-                });
-            });
-            // Verify the tool was called with single object arg (new inputSchema interface)
-            expect(toolWithoutTransform[0]?.associatedTools?.[0]?.tool).toHaveBeenCalledWith({ input: "test" });
-            // Verify the second advance call used stringified content
-            expect(mockThreadsApi.advanceByID).toHaveBeenLastCalledWith("test-thread-1", expect.objectContaining({
-                messageToAppend: expect.objectContaining({
-                    content: [
-                        {
-                            type: "text",
-                            text: '{"complex":"data","nested":{"value":42}}',
-                        },
-                    ],
-                    role: "tool",
-                }),
-            }));
-        });
-        it("should always return text for error responses even with transformToContent", async () => {
-            const mockTransformToContent = jest.fn().mockReturnValue([
-                {
-                    type: "image_url",
-                    image_url: { url: "https://example.com/error.png" },
-                },
-            ]);
-            const toolWithTransform = [
-                {
-                    name: "TestComponent",
-                    component: () => react_2.default.createElement("div", null, "Test"),
-                    description: "Test",
-                    propsSchema: v4_1.z.object({ test: v4_1.z.string() }),
-                    associatedTools: [
-                        {
-                            name: "error-tool",
-                            tool: jest
-                                .fn()
-                                .mockRejectedValue(new Error("Tool execution failed")),
-                            description: "Tool that errors",
-                            inputSchema: v4_1.z.object({ input: v4_1.z.string() }),
-                            outputSchema: v4_1.z.string(),
-                            transformToContent: mockTransformToContent,
-                        },
-                    ],
-                },
-            ];
-            const mockToolCallResponse = {
-                responseMessageDto: {
-                    id: "tool-call-1",
-                    content: [{ type: "text", text: "Tool call" }],
-                    role: "tool",
-                    threadId: "test-thread-1",
-                    toolCallRequest: {
-                        toolName: "error-tool",
-                        parameters: [{ parameterName: "input", parameterValue: "test" }],
-                    },
-                    componentState: {},
-                    createdAt: new Date().toISOString(),
-                },
-                generationStage: generate_component_response_1.GenerationStage.COMPLETE,
-                mcpAccessToken: "test-mcp-access-token",
-            };
-            jest
-                .mocked(mockThreadsApi.advanceByID)
-                .mockResolvedValueOnce(mockToolCallResponse)
-                .mockResolvedValueOnce({
-                responseMessageDto: {
-                    id: "final-response",
-                    content: [{ type: "text", text: "Final response" }],
-                    role: "assistant",
-                    threadId: "test-thread-1",
-                    componentState: {},
-                    createdAt: new Date().toISOString(),
-                },
-                generationStage: generate_component_response_1.GenerationStage.COMPLETE,
-                mcpAccessToken: "test-mcp-access-token",
-            });
-            const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), {
-                wrapper: createWrapper({ components: toolWithTransform }),
-            });
-            await (0, react_1.act)(async () => {
-                await result.current.sendThreadMessage("Use error tool", {
-                    threadId: "test-thread-1",
-                    streamResponse: false,
-                });
-            });
-            // Verify the tool was called with single object arg (new inputSchema interface)
-            expect(toolWithTransform[0]?.associatedTools?.[0]?.tool).toHaveBeenCalledWith({ input: "test" });
-            // Verify transformToContent was NOT called for error responses
-            expect(mockTransformToContent).not.toHaveBeenCalled();
-            // Verify the second advance call used text content with the error message
-            expect(mockThreadsApi.advanceByID).toHaveBeenLastCalledWith("test-thread-1", expect.objectContaining({
-                messageToAppend: expect.objectContaining({
-                    content: [
-                        expect.objectContaining({
-                            type: "text",
-                            // Error message should be in text format
-                        }),
-                    ],
-                    role: "tool",
-                }),
-            }));
-        });
     });
     describe("tamboStreamableHint streaming behavior", () => {
         it("should call streamable tool during streaming when tamboStreamableHint is true", async () => {
@@ -1617,6 +1383,25 @@ describe("TamboThreadProvider", () => {
     });
     describe("auto-generate thread name", () => {
         it("should auto-generate thread name after reaching threshold", async () => {
+            const mockStreamResponse = {
+                responseMessageDto: {
+                    id: "response-1",
+                    content: [{ type: "text", text: "Response" }],
+                    role: "assistant",
+                    threadId: "test-thread-1",
+                    component: undefined,
+                    componentState: {},
+                    createdAt: new Date().toISOString(),
+                },
+                generationStage: generate_component_response_1.GenerationStage.COMPLETE,
+                mcpAccessToken: "test-mcp-access-token",
+            };
+            const mockAsyncIterator = {
+                [Symbol.asyncIterator]: async function* () {
+                    yield mockStreamResponse;
+                },
+            };
+            jest.mocked(typescript_sdk_1.advanceStream).mockResolvedValue(mockAsyncIterator);
             const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), {
                 wrapper: createWrapper({ autoGenerateNameThreshold: 2 }),
             });
@@ -1648,13 +1433,34 @@ describe("TamboThreadProvider", () => {
                 }), false);
             });
             await (0, react_1.act)(async () => {
-                await result.current.sendThreadMessage("Test message");
+                await result.current.sendThreadMessage("Test message", {
+                    streamResponse: true,
+                });
             });
             expect(mockThreadsApi.generateName).toHaveBeenCalledWith("test-thread-1");
             expect(result.current.thread.name).toBe("Generated Thread Name");
             expect(mockQueryClient.setQueryData).toHaveBeenCalledWith(["threads", "test-project-id", undefined], expect.any(Function));
         });
         it("should NOT auto-generate when autoGenerateThreadName is false", async () => {
+            const mockStreamResponse = {
+                responseMessageDto: {
+                    id: "response-1",
+                    content: [{ type: "text", text: "Response" }],
+                    role: "assistant",
+                    threadId: "test-thread-1",
+                    component: undefined,
+                    componentState: {},
+                    createdAt: new Date().toISOString(),
+                },
+                generationStage: generate_component_response_1.GenerationStage.COMPLETE,
+                mcpAccessToken: "test-mcp-access-token",
+            };
+            const mockAsyncIterator = {
+                [Symbol.asyncIterator]: async function* () {
+                    yield mockStreamResponse;
+                },
+            };
+            jest.mocked(typescript_sdk_1.advanceStream).mockResolvedValue(mockAsyncIterator);
             const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), {
                 wrapper: createWrapper({
                     autoGenerateThreadName: false,
@@ -1686,12 +1492,33 @@ describe("TamboThreadProvider", () => {
                 }), false);
             });
             await (0, react_1.act)(async () => {
-                await result.current.sendThreadMessage("Test message");
+                await result.current.sendThreadMessage("Test message", {
+                    streamResponse: true,
+                });
             });
             // Should NOT generate name because feature is disabled
             expect(mockThreadsApi.generateName).not.toHaveBeenCalled();
         });
         it("should NOT auto-generate when thread already has a name", async () => {
+            const mockStreamResponse = {
+                responseMessageDto: {
+                    id: "response-1",
+                    content: [{ type: "text", text: "Response" }],
+                    role: "assistant",
+                    threadId: "test-thread-1",
+                    component: undefined,
+                    componentState: {},
+                    createdAt: new Date().toISOString(),
+                },
+                generationStage: generate_component_response_1.GenerationStage.COMPLETE,
+                mcpAccessToken: "test-mcp-access-token",
+            };
+            const mockAsyncIterator = {
+                [Symbol.asyncIterator]: async function* () {
+                    yield mockStreamResponse;
+                },
+            };
+            jest.mocked(typescript_sdk_1.advanceStream).mockResolvedValue(mockAsyncIterator);
             const { result } = (0, react_1.renderHook)(() => (0, tambo_thread_provider_1.useTamboThread)(), {
                 wrapper: createWrapper({ autoGenerateNameThreshold: 2 }),
             });
@@ -1725,7 +1552,9 @@ describe("TamboThreadProvider", () => {
             expect(result.current.thread.messages).toHaveLength(2);
             // Send another message to reach threshold (3 messages total)
             await (0, react_1.act)(async () => {
-                await result.current.sendThreadMessage("Test message");
+                await result.current.sendThreadMessage("Test message", {
+                    streamResponse: true,
+                });
             });
             // Should NOT generate name because thread already has one
             expect(mockThreadsApi.generateName).not.toHaveBeenCalled();