feat: finish baseline architecture #106
Description
Wires the application's configuration system into the LLM and analysis pipeline, replacing the previously hardcoded API key and model values with user-managed settings persisted in localStorage.
Key changes:
- Settings UI – adds an API key input field to the Settings modal. Other settings were not added, a subsequent task to populate and style this modal to fit the application should be created in subsequent iteration.
-
Config-driven LLM –
LLMConnectornow accepts aConfigManagerinstead of a raw adapter, reconfigures itself reactively via the newsubscribe/notifymechanism whenever the config changes -
Config-driven analysis –
LLMAnalysisServicereadsanalysis.profileandllm.modelfrom config instead of using hardcoded values -
ConfigManagersubscriptions – adds subscribe/unsubscribe support so services can react to config changes without polling -
Error differentiation – distinguishes "LLM not configured" (missing API key) from other runtime errors, surfacing a dedicated
ErrorNotConfiguredstatus in the status bar
Related Issues
Edited by Michal Turek