fix(extensions-library): declare required API keys in weaviate and open-interpreter manifests#549
Closed
yasinBursali wants to merge 1 commit intoLight-Heart-Labs:resources/devfrom
Closed
Conversation
…en-interpreter manifests Both services use Docker's :? syntax to require API keys at startup, but neither manifest declared these in env_vars. Setup wizards reading manifests had no way to prompt users, causing cryptic Docker failures. Added WEAVIATE_API_KEY and OPEN_INTERPRETER_API_KEY with required: true and secret: true flags. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This was referenced Mar 22, 2026
Contributor
Author
|
Closing: CG review found that neither |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Adds missing
env_varsentries for required API keys in weaviate and open-interpreter manifests.Why
Both services use Docker's
:?syntax to require API keys at startup (WEAVIATE_API_KEYandOPEN_INTERPRETER_API_KEY), but neither manifest declared these inenv_vars. Setup wizards and CLIs reading manifests had no way to prompt users for these keys, causing cryptic Docker failures likeWEAVIATE_API_KEY must be set.How
env_vars: []withWEAVIATE_API_KEYentry (required: true,secret: true)env_varssection withOPEN_INTERPRETER_API_KEYentry (required: true,secret: true)Both entries include descriptive text explaining the key is required and compose will fail without it.
Scope
All changes within
resources/dev/extensions-library/services/weaviate/andresources/dev/extensions-library/services/open-interpreter/.Testing
:?variables are the two declared keys; other env vars use:-(optional defaults) and don't need manifest entriesMerge Order
No dependencies on other open PRs. Can merge independently.