Harden provider debug option reads against traps

Co-authored-by: Shri Sukhani <shrisukhani@users.noreply.github.com>

Author: cursoragent

currentState.md

@@ -153,6 +153,7 @@ HyperAgent exposes a TypeScript SDK for browser automation with three primary pa
 - Expanded settle trace regressions with cleaner log-capture coverage to ensure recording-video trace diagnostics remain validated without noisy test output.
 - Hardened settle debug-option lookup against trap-prone `getDebugOptions()` reads, with deterministic fallback trace defaults and sanitized warning diagnostics.
 - Hardened A11y DOM option ingestion (`useCache`, `onFrameChunk`, `filterAdTrackingFrames`) with trap-safe reads, so malformed option objects no longer break extraction setup.
+- Hardened OpenAI/Anthropic structured-schema debug-option reads so trap-prone debug-option access no longer interrupts structured invocation paths.
 - Hardened constructor LLM validation to reject malformed non-provider/non-client `llm` payloads instead of accepting invalid runtime objects, while preserving trap-safe config reads.
 - Added explicit constructor regression coverage for malformed partial `llm` objects to lock in fail-fast configuration behavior.
 - Hardened prompt final-goal rendering against malformed/trap-prone task inputs by normalizing non-string goals into bounded readable diagnostics instead of throwing.

src/llm/providers/anthropic.test.ts

@@ -39,6 +39,10 @@ jest.mock("@/debug/options", () => ({
   getDebugOptions: jest.fn(() => debugOptions),
 }));
 
+const { getDebugOptions } = jest.requireMock("@/debug/options") as {
+  getDebugOptions: jest.Mock;
+};
+
 describe("AnthropicClient", () => {
   beforeEach(() => {
     createMessageMock.mockReset();
@@ -346,6 +350,36 @@ describe("AnthropicClient", () => {
     }
   });
 
+  it("continues structured invocation when debug option getter traps", async () => {
+    createMessageMock.mockResolvedValue({
+      content: [
+        {
+          type: "tool_use",
+          input: {
+            result: {
+              value: "ok",
+            },
+          },
+        },
+      ],
+    });
+    getDebugOptions.mockImplementationOnce(() => {
+      throw new Error("debug options trap");
+    });
+
+    const client = new AnthropicClient({ model: "claude-test" });
+    const result = await client.invokeStructured(
+      {
+        schema: z.object({
+          value: z.string(),
+        }),
+      },
+      [{ role: "user", content: "extract value" }]
+    );
+
+    expect(result.parsed).toEqual({ value: "ok" });
+  });
+
   it("ignores reserved provider option overrides while preserving custom options", async () => {
     createMessageMock.mockResolvedValue({
       content: [{ type: "text", text: "ok" }],

src/llm/providers/anthropic.ts

@@ -35,7 +35,12 @@ const RESERVED_ANTHROPIC_PROVIDER_OPTION_KEYS = new Set([
 const MAX_ANTHROPIC_DIAGNOSTIC_CHARS = 300;
 
 function shouldDebugStructuredSchema(): boolean {
-  const opts = getDebugOptions();
+  let opts: ReturnType<typeof getDebugOptions>;
+  try {
+    opts = getDebugOptions();
+  } catch {
+    return ENV_STRUCTURED_SCHEMA_DEBUG;
+  }
   if (opts.enabled && typeof opts.structuredSchema === "boolean") {
     return opts.structuredSchema;
   }

src/llm/providers/openai.test.ts

@@ -33,6 +33,10 @@ jest.mock("@/debug/options", () => ({
   getDebugOptions: jest.fn(() => debugOptions),
 }));
 
+const { getDebugOptions } = jest.requireMock("@/debug/options") as {
+  getDebugOptions: jest.Mock;
+};
+
 describe("OpenAIClient", () => {
   beforeEach(() => {
     createCompletionMock.mockReset();
@@ -297,6 +301,33 @@ describe("OpenAIClient", () => {
     }
   });
 
+  it("continues structured invocation when debug option getter traps", async () => {
+    createCompletionMock.mockResolvedValue({
+      choices: [
+        {
+          message: {
+            content: '{"ok":"yes"}',
+          },
+        },
+      ],
+    });
+    getDebugOptions.mockImplementationOnce(() => {
+      throw new Error("debug options trap");
+    });
+
+    const client = new OpenAIClient({ model: "gpt-test" });
+    const result = await client.invokeStructured(
+      {
+        schema: z.object({
+          ok: z.string(),
+        }),
+      },
+      [{ role: "user", content: "hello" }]
+    );
+
+    expect(result.parsed).toEqual({ ok: "yes" });
+  });
+
   it("throws readable error when completion choices are unreadable", async () => {
     const response = new Proxy(
       {},

src/llm/providers/openai.ts

@@ -122,7 +122,12 @@ function safeReadUsageTokens(
 }
 
 function shouldDebugStructuredSchema(): boolean {
-  const opts = getDebugOptions();
+  let opts: ReturnType<typeof getDebugOptions>;
+  try {
+    opts = getDebugOptions();
+  } catch {
+    return ENV_STRUCTURED_SCHEMA_DEBUG;
+  }
   if (opts.enabled && typeof opts.structuredSchema === "boolean") {
     return opts.structuredSchema;
   }