fix(setup): remove legacy key copy and clarify configuration errors

Author: samjulien

scripts/setup-learning.test.ts

@@ -132,7 +132,8 @@ test("fresh managed setup creates and verifies the selected project's container
   const content = await readFile(f.envPath, "utf8");
   expect(content).toContain("# Keep my other configuration");
   expect(parseEnv(content)).toMatchObject({
-    INTELLIGENCE_API_KEY: key,
+    CPK_INTELLIGENCE_API_KEY: key,
+    INTELLIGENCE_API_KEY: "",
     CPK_INTELLIGENCE_LEARNING_CONTAINER_ID: "openbot",
     OPENAI_API_KEY: "synthetic-model",
   });

scripts/setup-learning.ts

@@ -228,8 +228,7 @@ export async function setupLearning(
       "The selected project is invalid. Run npx --yes copilotkit@latest project select again.",
     );
   }
-  // A saved record alone does not prove a stale CPK key belongs to it. Re-provision here.
-  // CLI project select removes INTELLIGENCE_API_KEY, hence the precondition above.
+  // No runtime key is available. Provision one for the saved project before assigning Learning.
   const provision = await run(
     ["project", "select", "--project", projectId, "--json"],
     directory,
@@ -286,11 +285,7 @@ export async function setupLearning(
       "The CLI did not write a runtime key. Run project select and this helper again.",
     );
   }
-  const content = upsert(
-    upsert(current, "INTELLIGENCE_API_KEY", key),
-    CONTAINER,
-    "openbot",
-  );
+  const content = upsert(current, CONTAINER, "openbot");
   await saveEnvironment(envPath, content);
   return "configured";
 }

server/src/config.ts

@@ -958,7 +958,7 @@ function runtimeCapabilities(environment: Environment): RuntimeCapabilities {
   const missing = Object.entries({
     INTELLIGENCE_API_URL: settings.apiUrl,
     INTELLIGENCE_GATEWAY_WS_URL: settings.gatewayWsUrl,
-    INTELLIGENCE_API_KEY: settings.apiKey,
+    "CPK_INTELLIGENCE_API_KEY (or INTELLIGENCE_API_KEY)": settings.apiKey,
   })
     .filter(([, value]) => !value)
     .map(([name]) => name);

server/tests/config.test.ts

@@ -132,7 +132,7 @@ describe("deployment configuration", () => {
     delete environment[name];
 
     expect(() => loadConfig(environment)).toThrow(
-      `CopilotKit Intelligence is required and is not configured. Missing: ${name}`,
+      `CopilotKit Intelligence is required and is not configured. Missing: ${name === "INTELLIGENCE_API_KEY" ? "CPK_INTELLIGENCE_API_KEY (or INTELLIGENCE_API_KEY)" : name}`,
     );
   });