Repository navigation
Read the CLI-written CPK_INTELLIGENCE_API_KEY as the Intelligence key fallback #802
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
841f1d2
0341f6e
41773cc
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -949,7 +949,11 @@ function runtimeCapabilities(environment: Environment): RuntimeCapabilities { | |
| const settings = { | ||
| apiUrl: url(environment, "INTELLIGENCE_API_URL"), | ||
| gatewayWsUrl: url(environment, "INTELLIGENCE_GATEWAY_WS_URL"), | ||
| apiKey: optional(environment, "INTELLIGENCE_API_KEY"), | ||
| // The CopilotKit CLI writes the runtime key as CPK_INTELLIGENCE_API_KEY; setup-learning.ts | ||
| // copies it across, but a deployment that ran only the CLI would otherwise refuse to boot. | ||
| apiKey: | ||
| optional(environment, "INTELLIGENCE_API_KEY") ?? | ||
| optional(environment, "CPK_INTELLIGENCE_API_KEY"), | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
When users follow the README's local Intelligence flow, Useful? React with 👍 / 👎. |
||
| licenseToken: optional(environment, "COPILOTKIT_LICENSE_TOKEN"), | ||
| }; | ||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When users follow the checked-in setup guidance,
.env.example:134still says that the server cannot seeCPK_INTELLIGENCE_API_KEY, whileprompt.txt:106-108still requires copying that value toINTELLIGENCE_API_KEYbefore startup. This fallback makes both instructions false, so the documented CLI/local-connect workflow continues to require the manual step that this change is intended to remove; update those guidance surfaces alongside the new alias.Useful? React with 👍 / 👎.