✨ Improve on the cache handler

Cussa · Cussa · commit 1bc5c5c27ee3 · 2025-11-23T10:20:57.000Z
diff --git a/main.mjs b/main.mjs
@@ -14,13 +14,16 @@ import {
     formatMarkdown,
     formatDate,
     renderIssueDetails,
-    updateCacheStatus
+    updateCacheStatus,
+    handleForceRefresh,
+    setupLoadButton
 } from './shared.mjs';
 
 // Initialize the application
-document.addEventListener('DOMContentLoaded', () => {
+document.addEventListener('DOMContentLoaded', async () => {
     setupCommonUI();
     setupAdBanner();
+    setupLoadButton((repos) => loadAllRepositories(repos, true));
     
     const initialRepos = getInitialRepos();
 
@@ -31,37 +34,11 @@ document.addEventListener('DOMContentLoaded', () => {
         url.searchParams.set('repos', initialRepos.join(','));
         window.history.replaceState({}, '', url);
         
-        loadAllRepositories(initialRepos, true);
-    }
-
-    // Setup load button handler
-    const loadBtn = document.getElementById('loadBtn');
-    loadBtn.addEventListener('click', async () => {
-        setGitHubToken(document.getElementById('token').value.trim());
-        const reposText = document.getElementById('repos').value.trim();
-
-        // Save to localStorage
-        localStorage.setItem('githubRepos', reposText);
-
-        if (!reposText) {
-            showError('Please enter at least one repository');
-            return;
-        }
-
-        const repos = reposText.split('\n')
-            .map(line => line.trim())
-            .filter(line => line && line.includes('/'));
-
-        if (repos.length === 0) {
-            showError('Please enter valid repositories in format: owner/repo');
-            return;
-        }
-
-        await loadAllRepositories(repos, true);
+        await loadAllRepositories(initialRepos, true);
         
         // Update cache status after loading
         updateCacheStatus();
-    });
+    }
 
     // Setup issue detail panel handlers
     const iframePanel = document.getElementById('iframePanel');
diff --git a/shared.mjs b/shared.mjs
@@ -1,8 +1,8 @@
 // GitHub API configuration
 const GITHUB_API_BASE = 'https://api.github.com';
 
-// Cache configuration (15 minutes default)
-const CACHE_DURATION_MS = 15 * 60 * 1000;
+// Cache configuration (1 hour default)
+const CACHE_DURATION_MS = 60 * 60 * 1000;
 const CACHE_KEY_PREFIX = 'github_cache_';
 
 // State
@@ -219,9 +219,11 @@ export async function fetchRepositoryData(repo, openOnly = false) {
     // Check cache first
     const cached = getCachedData(repo, openOnly);
     if (cached) {
+        console.log(`Using cache for ${repo}`);
         return cached;
     }
     
+    console.log(`Fetching fresh data for ${repo}`);
     try {
         const state = openOnly ? 'open' : 'all';
         const issuesAndPRsUrl = `${GITHUB_API_BASE}/repos/${owner}/${repoName}/issues?state=${state}&per_page=100`;
@@ -461,6 +463,21 @@ export function getCacheAgeText(repoData) {
     return `📦 Cached ${timeAgo} (expires in ${remainingMinutes}m)`;
 }
 
+/**
+ * Handle force refresh - clears cache if checkbox is checked
+ */
+export function handleForceRefresh(repos) {
+    const forceRefresh = document.getElementById('forceRefresh');
+    if (forceRefresh && forceRefresh.checked) {
+        const clearedCount = clearCache(repos);
+        console.log(`Force refresh: cleared ${clearedCount} cache entries for`, repos);
+        // Uncheck the box after clearing
+        forceRefresh.checked = false;
+        return true;
+    }
+    return false;
+}
+
 /**
  * Get initial repositories (from query string or textarea)
  */
@@ -522,15 +539,13 @@ export function setupLoadButton(onLoad) {
         url.searchParams.set('repos', repos.join(','));
         window.history.pushState({}, '', url);
 
-        // Clear cache for selected repos if force refresh is checked
-        if (forceRefresh && forceRefresh.checked) {
-            clearCache(repos);
-        }
+        // Handle force refresh
+        handleForceRefresh(repos);
 
         await onLoad(repos);
         
-        // Update cache status after loading
-        updateCacheStatus();
+        // Update cache status after loading (with small delay to ensure cache writes complete)
+        setTimeout(() => updateCacheStatus(), 100);
     });
 
     // Setup share button
