-
Notifications
You must be signed in to change notification settings - Fork 16
[UX]: Add Configuration Validation for Environment Variables #52
Copy link
Copy link
Closed
Labels
coreRelated to core engineRelated to core engineenhancementNew feature or requestNew feature or requesthelp wantedExtra attention is neededExtra attention is neededv0.2.0Target for v0.2.0Target for v0.2.0
Milestone
Description
Problem Statement
The configuration loader uses os.ExpandEnv, which silently replaces missing environment variables with empty strings. This leads to cryptic errors from providers (e.g., "API key is empty") rather than a clear diagnostic message about which environment variable is missing.
Proposed Solution
- Add a
Validate() errormethod to theConfigstruct ininternal/core/config/config.go. - Call validation after loading the configuration.
- Check for required fields like
Qdrant.URL,Qdrant.APIKey, andEmbedding.APIKey. - Provide clear error messages indicating which environment variable or config field is missing.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
coreRelated to core engineRelated to core engineenhancementNew feature or requestNew feature or requesthelp wantedExtra attention is neededExtra attention is neededv0.2.0Target for v0.2.0Target for v0.2.0
Type
Projects
Status
Done