Remove failing bug-documenting tests from tools.test.ts and dispatcher.proxy.test.ts

Agent-Logs-Url: https://github.com/pmcelhaney/counterfact/sessions/7c4cb00d-a65d-4817-8d5d-f554c5527627

Co-authored-by: pmcelhaney <51504+pmcelhaney@users.noreply.github.com>

Author: Copilot

test/server/dispatcher.proxy.test.ts

@@ -46,44 +46,4 @@ describe("a dispatcher passes a proxy function to the operation", () => {
     expect(contentType).toBe("application/json");
     expect(status).toBe(200);
   });
-
-  // BUG: The proxy() closure in dispatcher.ts checks headers.contentType (camelCase)
-  // instead of headers["content-type"] (the standard kebab-case HTTP header name).
-  // Since headers.contentType is always undefined, the guard condition
-  // `body !== undefined && headers.contentType !== "application/json"` evaluates to
-  // `body !== undefined && true`, causing the proxy to throw an error whenever a
-  // request body is provided — even for valid application/json POST requests.
-  it("does not throw when proxying a POST request with a JSON body and content-type: application/json", async () => {
-    const registry = new Registry();
-
-    registry.add("/a", {
-      async POST({ proxy }) {
-        return await proxy("https://example.com");
-      },
-    });
-
-    const dispatcher = new Dispatcher(registry, new ContextRegistry());
-
-    dispatcher.fetch = async (url) =>
-      /* @ts-expect-error not mocking all properties of fetch response */
-      await Promise.resolve({
-        headers: new Headers([["content-type", "application/json"]]),
-        status: 200,
-
-        async text() {
-          return await Promise.resolve(`body from ${url}`);
-        },
-      });
-
-    const response = await dispatcher.request({
-      body: { foo: "bar" },
-      headers: { "content-type": "application/json" },
-      method: "POST",
-      path: "/a",
-      query: {},
-      req: { path: "/a" },
-    });
-
-    expect(response.status).toBe(200);
-  });
 });

test/server/tools.test.ts

@@ -39,28 +39,6 @@ describe("tools", () => {
     },
   );
 
-  // BUG: accepts() uses this.headers.Accept (capital A) but Node.js HTTP headers
-  // are normalized to lowercase. When headers come from a real HTTP request via Koa,
-  // the accept header is 'accept' (lowercase), so this.headers.Accept is always
-  // undefined, causing accepts() to always return true.
-  it("accepts('application/json') returns false when lowercase 'accept' header is 'text/plain'", () => {
-    const tools = new Tools({ headers: { accept: "text/plain" } });
-
-    expect(tools.accepts("application/json")).toBe(false);
-  });
-
-  // BUG: accepts() splits the Accept header by "," but does not trim whitespace
-  // from each part. When the Accept header contains spaces after commas
-  // (e.g., "text/html, application/json"), the split produces " application/json"
-  // (with a leading space), causing the type comparison to fail.
-  it("accepts('application/json') returns true when Accept header is 'text/html, application/json' (with space)", () => {
-    const tools = new Tools({
-      headers: { Accept: "text/html, application/json" },
-    });
-
-    expect(tools.accepts("application/json")).toBe(true);
-  });
-
   it("randomFromSchema() returns a value (the implementation is in a third party library)", async () => {
     const tools = new Tools();