@copilotkit/react-textarea 1.78.0 → 1.79.1-canary.1792034358

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@copilotkit/react-textarea",
-  "version": "1.78.0",
+  "version": "1.79.1-canary.1792034358",
   "private": false,
   "keywords": [
     "ai",
@@ -41,6 +41,9 @@
     "access": "public"
   },
   "dependencies": {
+    "@copilotkit/react-core": "1.79.1-canary.1792034358",
+    "@copilotkit/runtime-client-gql": "1.79.1-canary.1792034358",
+    "@copilotkit/shared": "1.79.1-canary.1792034358",
     "@emotion/css": "^11.11.2",
     "@emotion/react": "^11.11.1",
     "@emotion/styled": "^11.11.0",
@@ -58,10 +61,7 @@
     "slate": "^0.94.1",
     "slate-history": "^0.93.0",
     "slate-react": "^0.98.1",
-    "tailwind-merge": "^1.13.2",
-    "@copilotkit/react-core": "1.78.0",
-    "@copilotkit/runtime-client-gql": "1.78.0",
-    "@copilotkit/shared": "1.78.0"
+    "tailwind-merge": "^1.13.2"
   },
   "devDependencies": {
     "@types/lodash.merge": "^4.6.7",
@@ -78,8 +78,8 @@
     "tsdown": "^0.20.3",
     "typescript": "^5.2.3",
     "vitest": "^3.2.4",
-    "tailwind-config": "1.4.12",
-    "tsconfig": "1.4.12"
+    "tsconfig": "1.4.12",
+    "tailwind-config": "1.4.12"
   },
   "peerDependencies": {
     "react": "^18 || ^19 || ^19.0.0-rc",

package/src/hooks/make-autosuggestions-function/__tests__/use-make-standard-autosuggestions-function.headers.test.tsx

@@ -0,0 +1,71 @@
+/**
+ * #1937: `useMakeStandardAutosuggestionFunction` used to read
+ * `copilotApiConfig.headers` directly in its render body (building a
+ * `headers` local that was itself dead — the actual GraphQL request is
+ * commented out, and `runtimeClient` is a stub; that local's only mention
+ * was building it). With an async `headers` builder on `<CopilotKit>`
+ * (`packages/react-core`'s v1 wrapper, which `CopilotTextarea` mounts under
+ * — see `copilot-textarea.tsx`), that render-time read called the builder
+ * on every render of `CopilotTextarea`. The dead `headers` local has been
+ * deleted; this test proves the hook's render body never touches
+ * `copilotApiConfig.headers` at all, so mounting (and re-rendering) it under
+ * an async builder never invokes that builder.
+ *
+ * react-textarea has no jsdom / `@testing-library/react` setup (see
+ * `vitest.config.mjs`: `environment: "node"`). Rather than add that
+ * infrastructure for one regression test, this renders through
+ * `react-dom/server`'s `renderToStaticMarkup` — a real React render pass
+ * (the same kind that ran the old, buggy read) that needs no DOM. Calling
+ * it repeatedly stands in for repeated re-renders: each call re-executes
+ * every component's render body, which is exactly the code path the bug
+ * lived in.
+ */
+import { CopilotKit } from "@copilotkit/react-core";
+import { renderToStaticMarkup } from "react-dom/server";
+import { describe, expect, it } from "vitest";
+import { defaultSuggestionsApiConfig } from "../../../types/autosuggestions-config/suggestions-api-config";
+import { useMakeStandardAutosuggestionFunction } from "../use-make-standard-autosuggestions-function";
+
+function Probe() {
+  // Same call shape as `CopilotTextarea` (see copilot-textarea.tsx:208-212).
+  useMakeStandardAutosuggestionFunction(
+    "a test textarea",
+    ["global"],
+    defaultSuggestionsApiConfig,
+  );
+  return null;
+}
+
+describe("useMakeStandardAutosuggestionFunction — async headers builder (#1937)", () => {
+  it("never calls an async headers builder during render, across repeated renders", () => {
+    let builderCalls = 0;
+
+    function App() {
+      return (
+        <CopilotKit
+          runtimeUrl="http://rt.test/api/copilotkit"
+          headers={async () => {
+            builderCalls++;
+            return { Authorization: "Bearer tok-1" };
+          }}
+        >
+          <Probe />
+        </CopilotKit>
+      );
+    }
+
+    // Discriminating assertion FIRST: mounting is itself a render, and the
+    // old code called the builder right there, in `useMakeStandard...`'s
+    // hook body, before any effect ever ran.
+    renderToStaticMarkup(<App />);
+    expect(builderCalls).toBe(0);
+
+    // Repeated renders (each call is a fresh, independent render pass
+    // through the same tree) must never invoke the builder either — the
+    // bug scaled with render COUNT, not just with mounting once.
+    for (let i = 0; i < 10; i++) {
+      renderToStaticMarkup(<App />);
+    }
+    expect(builderCalls).toBe(0);
+  });
+});

package/src/hooks/make-autosuggestions-function/use-make-standard-autosuggestions-function.tsx

@@ -1,12 +1,11 @@
-import { COPILOT_CLOUD_PUBLIC_API_KEY_HEADER } from "@copilotkit/shared";
 import { useCopilotContext } from "@copilotkit/react-core";
 import { useCallback } from "react";
-import { AutosuggestionsBareFunction } from "../../types";
+import type { AutosuggestionsBareFunction } from "../../types";
 import { retry } from "../../lib/retry";
-import { InsertionEditorState } from "../../types/base/autosuggestions-bare-function";
-import { SuggestionsApiConfig } from "../../types/autosuggestions-config/suggestions-api-config";
+import type { InsertionEditorState } from "../../types/base/autosuggestions-bare-function";
+import type { SuggestionsApiConfig } from "../../types/autosuggestions-config/suggestions-api-config";
+import type { Message } from "@copilotkit/runtime-client-gql";
 import {
-  Message,
   Role,
   TextMessage,
   convertGqlOutputToMessages,
@@ -35,18 +34,7 @@ export function useMakeStandardAutosuggestionFunction(
 ): AutosuggestionsBareFunction {
   const runtimeClient = { generateCopilotResponse: (...args: any[]) => {} };
   const { getContextString, copilotApiConfig } = useCopilotContext();
-  const {
-    chatApiEndpoint: url,
-    publicApiKey,
-    credentials,
-    properties,
-  } = copilotApiConfig;
-  const headers = {
-    ...copilotApiConfig.headers,
-    ...(publicApiKey
-      ? { [COPILOT_CLOUD_PUBLIC_API_KEY_HEADER]: publicApiKey }
-      : {}),
-  };
+  const { chatApiEndpoint: url, credentials, properties } = copilotApiConfig;
   const { maxTokens, stop, temperature = 0 } = apiConfig;
 
   return useCallback(

package/vitest.config.mjs

@@ -4,7 +4,7 @@ export default defineConfig({
   test: {
     environment: "node",
     globals: true,
-    include: ["src/**/*.{test,spec}.ts"],
+    include: ["src/**/*.{test,spec}.{ts,tsx}"],
     reporters: [["default", { summary: false }]],
     silent: true,
   },