Merge pull request #1 from fedenanni/claude/update-news-cache-interval-z9wVq

Change news cache from daily to 4-hour TTL

Author: fedenanni

api/index.py

@@ -5,7 +5,7 @@
 """
 
 import os
-from datetime import date
+from datetime import date, datetime
 from flask import Flask, request, jsonify
 
 from api.date_calculator import (
@@ -107,14 +107,14 @@ def get_news():
                 'error': 'Missing required parameter: option'
             }), 400
 
-        # Get today's date for cache key
-        today = date.today()
+        # Get current time for cache
+        now = datetime.now()
 
         # Get cache instance
         cache = get_cache()
 
-        # Check if we have cached data for this option and today
-        cached_data = cache.get(option, today)
+        # Check if we have cached data for this option
+        cached_data = cache.get(option, now)
         if cached_data is not None:
             # Return cached response
             return jsonify(cached_data)
@@ -170,7 +170,7 @@ def get_news():
             }
 
         # Cache the response for non-random options
-        cache.set(option, response_data, today)
+        cache.set(option, response_data, now)
 
         return jsonify(response_data)
 

api/news_cache.py

@@ -1,119 +1,124 @@
 """News caching module for Past News application.
 
-This module provides a simple cache that stores fetched news for a specific day.
-When a new day starts, old caches are automatically cleared.
+This module provides a simple cache that stores fetched news with a 4-hour TTL.
+After 4 hours, the cache is automatically cleared to fetch fresh news.
 """
 
-from datetime import date
+from datetime import datetime, timedelta
 from typing import Optional
 
+# Cache TTL in hours
+CACHE_TTL_HOURS = 4
+
 
 class NewsCache:
     """
-    Simple in-memory cache for storing news articles by date and option.
+    Simple in-memory cache for storing news articles with time-based expiration.
 
-    The cache stores articles for each day, with keys for 'one_week', 'two_weeks',
-    and 'one_month'. When the date changes, old caches are automatically cleared.
+    The cache stores articles with a 4-hour TTL. After 4 hours, the cache
+    is automatically cleared when accessed.
 
     Note: 'random' option is not cached since it should return different results.
     """
 
     def __init__(self):
         """Initialize empty cache."""
         self._cache = {}
-        self._cache_date = None
+        self._cache_time = None
 
-    def _check_and_clear_old_cache(self, current_date: date) -> None:
+    def _check_and_clear_expired_cache(self, current_time: datetime) -> None:
         """
-        Check if the date has changed and clear old cache if needed.
+        Check if the cache has expired and clear it if needed.
 
         Args:
-            current_date: The current date to check against
+            current_time: The current datetime to check against
         """
-        if self._cache_date is None:
-            self._cache_date = current_date
-        elif self._cache_date != current_date:
-            # Date has changed, clear old cache
-            self._cache.clear()
-            self._cache_date = current_date
-
-    def get(self, option: str, current_date: date = None) -> Optional[dict]:
+        if self._cache_time is None:
+            self._cache_time = current_time
+        else:
+            elapsed = current_time - self._cache_time
+            if elapsed >= timedelta(hours=CACHE_TTL_HOURS):
+                # Cache has expired, clear it
+                self._cache.clear()
+                self._cache_time = current_time
+
+    def get(self, option: str, current_time: datetime = None) -> Optional[dict]:
         """
         Get cached article data for a specific option.
 
         Args:
             option: Time period option ('one_week', 'two_weeks', 'one_month')
-            current_date: The current date (default: today)
+            current_time: The current datetime (default: now)
 
         Returns:
             Cached article data or None if not found
 
         Note:
             Returns None for 'random' option since it should not be cached.
         """
-        if current_date is None:
-            current_date = date.today()
+        if current_time is None:
+            current_time = datetime.now()
 
         # Don't cache random options
         if option == 'random':
             return None
 
-        # Check and clear old cache
-        self._check_and_clear_old_cache(current_date)
+        # Check and clear expired cache
+        self._check_and_clear_expired_cache(current_time)
 
         # Return cached data if available
         return self._cache.get(option)
 
-    def set(self, option: str, data: dict, current_date: date = None) -> None:
+    def set(self, option: str, data: dict, current_time: datetime = None) -> None:
         """
         Store article data in cache for a specific option.
 
         Args:
             option: Time period option ('one_week', 'two_weeks', 'one_month')
             data: Article data to cache
-            current_date: The current date (default: today)
+            current_time: The current datetime (default: now)
 
         Note:
             Does not cache 'random' option since it should not be cached.
         """
-        if current_date is None:
-            current_date = date.today()
+        if current_time is None:
+            current_time = datetime.now()
 
         # Don't cache random options
         if option == 'random':
             return
 
-        # Check and clear old cache
-        self._check_and_clear_old_cache(current_date)
+        # Check and clear expired cache
+        self._check_and_clear_expired_cache(current_time)
 
         # Store data
         self._cache[option] = data
 
     def clear(self) -> None:
         """Clear all cached data."""
         self._cache.clear()
-        self._cache_date = None
+        self._cache_time = None
 
-    def has(self, option: str, current_date: date = None) -> bool:
+    def has(self, option: str, current_time: datetime = None) -> bool:
         """
         Check if cache has data for a specific option.
 
         Args:
             option: Time period option to check
-            current_date: The current date (default: today)
+            current_time: The current datetime (default: now)
 
         Returns:
             True if data is cached, False otherwise
         """
-        if current_date is None:
-            current_date = date.today()
+        if current_time is None:
+            current_time = datetime.now()
 
         # Random is never cached
         if option == 'random':
             return False
 
-        # Check and clear old cache
-        self._check_and_clear_old_cache(current_date)
+        # Check and clear expired cache
+        self._check_and_clear_expired_cache(current_time)
 
         return option in self._cache
 

src/news_cache.py

@@ -1,119 +1,124 @@
 """News caching module for Past News application.
 
-This module provides a simple cache that stores fetched news for a specific day.
-When a new day starts, old caches are automatically cleared.
+This module provides a simple cache that stores fetched news with a 4-hour TTL.
+After 4 hours, the cache is automatically cleared to fetch fresh news.
 """
 
-from datetime import date
+from datetime import datetime, timedelta
 from typing import Optional
 
+# Cache TTL in hours
+CACHE_TTL_HOURS = 4
+
 
 class NewsCache:
     """
-    Simple in-memory cache for storing news articles by date and option.
+    Simple in-memory cache for storing news articles with time-based expiration.
 
-    The cache stores articles for each day, with keys for 'one_week', 'two_weeks',
-    and 'one_month'. When the date changes, old caches are automatically cleared.
+    The cache stores articles with a 4-hour TTL. After 4 hours, the cache
+    is automatically cleared when accessed.
 
     Note: 'random' option is not cached since it should return different results.
     """
 
     def __init__(self):
         """Initialize empty cache."""
         self._cache = {}
-        self._cache_date = None
+        self._cache_time = None
 
-    def _check_and_clear_old_cache(self, current_date: date) -> None:
+    def _check_and_clear_expired_cache(self, current_time: datetime) -> None:
         """
-        Check if the date has changed and clear old cache if needed.
+        Check if the cache has expired and clear it if needed.
 
         Args:
-            current_date: The current date to check against
+            current_time: The current datetime to check against
         """
-        if self._cache_date is None:
-            self._cache_date = current_date
-        elif self._cache_date != current_date:
-            # Date has changed, clear old cache
-            self._cache.clear()
-            self._cache_date = current_date
-
-    def get(self, option: str, current_date: date = None) -> Optional[dict]:
+        if self._cache_time is None:
+            self._cache_time = current_time
+        else:
+            elapsed = current_time - self._cache_time
+            if elapsed >= timedelta(hours=CACHE_TTL_HOURS):
+                # Cache has expired, clear it
+                self._cache.clear()
+                self._cache_time = current_time
+
+    def get(self, option: str, current_time: datetime = None) -> Optional[dict]:
         """
         Get cached article data for a specific option.
 
         Args:
             option: Time period option ('one_week', 'two_weeks', 'one_month')
-            current_date: The current date (default: today)
+            current_time: The current datetime (default: now)
 
         Returns:
             Cached article data or None if not found
 
         Note:
             Returns None for 'random' option since it should not be cached.
         """
-        if current_date is None:
-            current_date = date.today()
+        if current_time is None:
+            current_time = datetime.now()
 
         # Don't cache random options
         if option == 'random':
             return None
 
-        # Check and clear old cache
-        self._check_and_clear_old_cache(current_date)
+        # Check and clear expired cache
+        self._check_and_clear_expired_cache(current_time)
 
         # Return cached data if available
         return self._cache.get(option)
 
-    def set(self, option: str, data: dict, current_date: date = None) -> None:
+    def set(self, option: str, data: dict, current_time: datetime = None) -> None:
         """
         Store article data in cache for a specific option.
 
         Args:
             option: Time period option ('one_week', 'two_weeks', 'one_month')
             data: Article data to cache
-            current_date: The current date (default: today)
+            current_time: The current datetime (default: now)
 
         Note:
             Does not cache 'random' option since it should not be cached.
         """
-        if current_date is None:
-            current_date = date.today()
+        if current_time is None:
+            current_time = datetime.now()
 
         # Don't cache random options
         if option == 'random':
             return
 
-        # Check and clear old cache
-        self._check_and_clear_old_cache(current_date)
+        # Check and clear expired cache
+        self._check_and_clear_expired_cache(current_time)
 
         # Store data
         self._cache[option] = data
 
     def clear(self) -> None:
         """Clear all cached data."""
         self._cache.clear()
-        self._cache_date = None
+        self._cache_time = None
 
-    def has(self, option: str, current_date: date = None) -> bool:
+    def has(self, option: str, current_time: datetime = None) -> bool:
         """
         Check if cache has data for a specific option.
 
         Args:
             option: Time period option to check
-            current_date: The current date (default: today)
+            current_time: The current datetime (default: now)
 
         Returns:
             True if data is cached, False otherwise
         """
-        if current_date is None:
-            current_date = date.today()
+        if current_time is None:
+            current_time = datetime.now()
 
         # Random is never cached
         if option == 'random':
             return False
 
-        # Check and clear old cache
-        self._check_and_clear_old_cache(current_date)
+        # Check and clear expired cache
+        self._check_and_clear_expired_cache(current_time)
 
         return option in self._cache