fix(#134): Add generic get/set methods to CacheService

[DevanshuNEU]

Bug Fix: #134 - CacheService missing get/set methods

Problem

POST /api/v1/playground/validate-repo returns 500 Internal Server Error:

AttributeError: 'CacheService' object has no attribute 'get'

The endpoint uses generic cache.get() and cache.set() but CacheService only had specialized methods like get_search_results().

Solution

Added generic methods to CacheService:

def get(self, key: str) -> Optional[Dict]:
    """Get value by key (generic cache access)."""
    
def set(self, key: str, value: Dict, ttl: int = 300) -> bool:
    """Set value by key with TTL (default 5 min)."""

Features

• Graceful handling when Redis unavailable (returns None/False)
• Proper error handling with logging
• Metrics tracking (cache_hits, cache_misses, cache_errors)
• Consistent with existing method patterns

Tests Added

8 new unit tests in test_cache_service.py:

• test_get_returns_cached_value
• test_get_returns_none_when_key_missing
• test_get_returns_none_when_redis_unavailable
• test_get_handles_redis_error
• test_set_stores_value_with_ttl
• test_set_uses_default_ttl
• test_set_returns_false_when_redis_unavailable
• test_set_handles_redis_error

Test Results

52 passed (44 anonymous indexing + 8 cache service)

Next Step

After merge, ping Arjun to flip USE_MOCK = false in frontend.

Fixes #134

[vercel]

@DevanshuNEU is attempting to deploy a commit to the Dev's projects Team on Vercel.

A member of the Team first needs to authorize it.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Review	Updated (UTC)
opencodeintel	Ignored	Preview	Dec 26, 2025 8:55pm