feature/CONFIG-2: Added toggleable caching/metrics

[MVPandey]

No description provided.

[Copilot]

Pull Request Overview

This PR adds toggleable caching and metrics functionality to the Conversational Analysis Engine, making the system more flexible by allowing core functionality to work with minimal configuration while providing optional features when additional configuration is present.

• Configuration system refactored to support minimal setup (only LLM_API_KEY required) with smart defaults
• Features like semantic caching and Prometheus metrics are now optional and conditionally enabled
• Test files updated to remove redundant inline comments for cleaner code

Reviewed Changes

Copilot reviewed 50 out of 50 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
app/utils/config.py	Major refactor to support feature toggles and minimal configuration requirements
app/main.py	Updated to conditionally initialize features based on configuration flags
tests/unit/utils/test_config.py	Added comprehensive tests for new feature toggle functionality
docker-compose.yml	Enhanced with monitoring profile and updated defaults
monitoring/prometheus.yml	New Prometheus configuration for optional monitoring
README.md	Updated documentation to reflect simplified setup requirements
Multiple test files	Cleaned up inline comments throughout test suite

Comments suppressed due to low confidence (2)

tests/unit/test_main.py:54

• This test assertion is fragile as it only checks for a log message starting with 'Feature configuration:' but doesn't verify the actual feature configuration behavior. Consider testing the specific feature flags or configuration values that should be logged.

        assert any(call.args[0].startswith("Feature configuration:") for call in mock_logger.info.call_args_list)

tests/unit/utils/test_config.py:121

• The test reduces the expected required fields from a comprehensive set to only LLM_API_KEY, but the test name suggests it should test 'missing_required_fields' (plural). This could miss validation errors for other fields that should be required. Consider updating the test name to reflect it only tests the single required field or expand the assertion.

        expected_fields = {"LLM_API_KEY"}