fix: validate API keys on AgenticManager init #29

New issue

Open

opened 2026-02-20 03:02:29 +00:00 by Clotho · 0 comments

Clotho commented

2026-02-20 03:02:29 +00:00

Member

Issue

Services/AgenticManager.php::registerProviders() doesn't validate API keys.

Current Behavior

Empty API key creates provider that fails on first use.

Expected Behavior

Log warning if provider configured without key
OR throw exception during initialization
Provide clear guidance on configuration

Acceptance Criteria

Add API key validation to registerProviders()
Log warning or throw on missing key
Document configuration requirements
Add tests for invalid configuration

References

TODO.md: DX-002
Affects Claude, Gemini, OpenAI providers

## Issue `Services/AgenticManager.php::registerProviders()` doesn't validate API keys. ## Current Behavior Empty API key creates provider that fails on first use. ## Expected Behavior - Log warning if provider configured without key - OR throw exception during initialization - Provide clear guidance on configuration ## Acceptance Criteria - [ ] Add API key validation to `registerProviders()` - [ ] Log warning or throw on missing key - [ ] Document configuration requirements - [ ] Add tests for invalid configuration ## References - TODO.md: DX-002 - Affects Claude, Gemini, OpenAI providers