fix: pass hookParams to onAfterRequest instead of empty object (CopilotKit#2124)

The onAfterRequest middleware callback was being called with an empty
object `{}` instead of the available hook parameters (threadId, runId,
messages, path). Extract and forward these from the hookParams that the
v2 runtime provides.

Author: jpr5

packages/runtime/src/lib/runtime/__tests__/on-after-request.test.ts

@@ -0,0 +1,41 @@
+import { describe, it, expect, vi } from "vitest";
+import { CopilotRuntime } from "../copilot-runtime";
+
+describe("onAfterRequest middleware (#2124)", () => {
+  it("should pass hookParams to onAfterRequest, not an empty object", async () => {
+    const onAfterRequest = vi.fn();
+
+    const runtime = new CopilotRuntime({
+      middleware: {
+        onAfterRequest,
+      },
+    });
+
+    // Access the internal afterRequestMiddleware function
+    const afterRequestMw = runtime.instance.afterRequestMiddleware;
+    expect(afterRequestMw).toBeDefined();
+
+    // Simulate calling the middleware with hookParams (as the v2 runtime would)
+    const fakeHookParams = {
+      runtime: {} as any,
+      response: new Response("test"),
+      path: "/api/copilotkit",
+      messages: [{ id: "msg-1", role: "assistant", content: "Hello" }],
+      threadId: "thread-123",
+      runId: "run-456",
+    };
+
+    await (afterRequestMw as Function)(fakeHookParams);
+
+    // The onAfterRequest callback should have been called
+    expect(onAfterRequest).toHaveBeenCalledTimes(1);
+
+    // CRITICAL: It should NOT be called with an empty object
+    const callArg = onAfterRequest.mock.calls[0][0];
+    expect(callArg).not.toEqual({});
+
+    // It should receive the hookParams (or at least threadId/messages)
+    expect(callArg).toHaveProperty("threadId", "thread-123");
+    expect(callArg).toHaveProperty("runId", "run-456");
+  });
+});

packages/runtime/src/lib/runtime/copilot-runtime.ts

@@ -590,9 +590,18 @@ export class CopilotRuntime<const T extends Parameter[] | [] = []> {
       params?.afterRequestMiddleware?.(hookParams);
 
       if (params?.middleware?.onAfterRequest) {
-        // TODO: provide old expected params here when available
-        // @ts-expect-error -- missing arguments.
-        params.middleware.onAfterRequest({});
+        params.middleware.onAfterRequest({
+          threadId: hookParams.threadId ?? "",
+          runId: hookParams.runId,
+          inputMessages: (hookParams.messages ?? []).filter(
+            (m: any) => "role" in m && m.role === "user",
+          ),
+          outputMessages: (hookParams.messages ?? []).filter(
+            (m: any) => "role" in m && m.role !== "user",
+          ),
+          properties: {},
+          url: hookParams.path,
+        });
       }
     };
   }