fix(webapp): keep the other Anthropic options when the step breakpoint rolls off

Author: kathiekiwi

internal-packages/dashboard-agent/src/step-cache.test.ts

@@ -26,6 +26,17 @@ function turnHistory(): Message {
   };
 }
 
+function stepBreakpointWith(otherAnthropicOptions: Record<string, unknown>): Message {
+  return {
+    role: "tool",
+    content: "ok",
+    providerOptions: {
+      anthropic: { cacheControl: STEP_CACHE_CONTROL, ...otherAnthropicOptions },
+      openai: { store: false },
+    },
+  };
+}
+
 function ttlOf(message: Message | undefined): unknown {
   return (message?.providerOptions?.anthropic as { cacheControl?: { ttl?: unknown } } | undefined)
     ?.cacheControl?.ttl;
@@ -73,6 +84,28 @@ describe("the step cache breakpoint", () => {
     expect(compacted.filter((message) => ttlOf(message) === "5m")).toHaveLength(0);
   });
 
+  it("keeps the other anthropic options when it rolls the breakpoint off", () => {
+    const marked = markStepCacheBreakpoint([
+      stepBreakpointWith({ anotherOption: "keep" }),
+      toolResult(MIN_STEP_CACHE_CHARS),
+    ]);
+
+    expect(marked[0]!.providerOptions).toEqual({
+      anthropic: { anotherOption: "keep" },
+      openai: { store: false },
+    });
+    expect(ttlOf(marked.at(-1))).toBe("5m");
+  });
+
+  it("leaves no empty anthropic object behind", () => {
+    const marked = markStepCacheBreakpoint([
+      stepBreakpointWith({}),
+      toolResult(MIN_STEP_CACHE_CHARS),
+    ]);
+
+    expect(marked[0]!.providerOptions).toEqual({ openai: { store: false } });
+  });
+
   it("does nothing to an empty step", () => {
     const empty: Message[] = [];
     expect(markStepCacheBreakpoint(empty)).toBe(empty);

internal-packages/dashboard-agent/src/step-cache.ts

@@ -32,8 +32,12 @@ function cacheControlTtl(message: MaybeCached): string | undefined {
 
 function withoutStepBreakpoint<T extends MaybeCached>(message: T): T {
   if (cacheControlTtl(message) !== STEP_CACHE_CONTROL.ttl) return message;
-  const { anthropic: _dropped, ...rest } = message.providerOptions as Record<string, unknown>;
-  return { ...message, providerOptions: rest };
+  const { anthropic, ...rest } = message.providerOptions as Record<string, unknown>;
+  const { cacheControl: _dropped, ...anthropicRest } = anthropic as Record<string, unknown>;
+  // An empty `anthropic` is not the same as no Anthropic options, so drop the key.
+  const providerOptions =
+    Object.keys(anthropicRest).length > 0 ? { ...rest, anthropic: anthropicRest } : rest;
+  return { ...message, providerOptions };
 }
 
 // Only ever one step breakpoint at a time: Anthropic allows four in total, and the