website/templates/projects/repo_detail.html (1)

1310-1339: Well-implemented pagination with accessibility considerations

The pagination controls are thoughtfully implemented with:

• Clear visibility of current page
• Previous/Next buttons that appear only when relevant
• Smart handling of large page numbers with ellipses
• Maintenance of the current filter when navigating pages

Consider adding ARIA attributes to improve accessibility for screen readers.

-<div class="flex flex-col items-center space-y-4 mt-8">
+<div class="flex flex-col items-center space-y-4 mt-8" role="navigation" aria-label="Stargazers pagination">

website/static/js/repo_detail.js (2)

429-455: Consider adding more robust error handling for user feedback

While you have console error logging, the user doesn't receive any visual feedback when an error occurs during stargazer fetching. Consider adding a user-facing error message similar to other sections of the site.

 function fetchStargazers(url) {
     const stargazersSection = document.getElementById('stargazers-section');
     if (!stargazersSection) return;
     // Show loading state
     stargazersSection.classList.add('opacity-50');
     fetch(url, {
         headers: { 'X-Requested-With': 'XMLHttpRequest' }
     })
     .then(response => {
         if (!response.ok) throw new Error('Network response was not ok');
         return response.text();
     })
     .then(html => {
         // Parse the returned HTML and extract the stargazers section
         const tempDiv = document.createElement('div');
         tempDiv.innerHTML = html;
         const newSection = tempDiv.querySelector('#stargazers-section');
         if (newSection) {
             stargazersSection.innerHTML = newSection.innerHTML;
             // Update URL
             window.history.pushState({}, '', url);
             // Re-attach listeners
             attachStargazersListeners();
         }
     })
     .catch(error => {
         console.error('Error fetching stargazers:', error);
+        // Add user-facing error message
+        const errorHtml = `<div class="bg-red-100 border border-red-400 text-red-700 px-4 py-3 rounded mb-4">
+            Failed to load stargazers. Please try again later.
+        </div>`;
+        // Insert at the top of the section, preserving existing content
+        const contentWrapper = stargazersSection.querySelector('.bg-white');
+        if (contentWrapper) {
+            const errorDiv = document.createElement('div');
+            errorDiv.innerHTML = errorHtml;
+            contentWrapper.insertBefore(errorDiv.firstChild, contentWrapper.firstChild.nextSibling);
+        }
     })
     .finally(() => {
         stargazersSection.classList.remove('opacity-50');
     });
 }

---

425-428: Add error checking for mandatory URL parameter

The fetchStargazers function doesn't verify if the URL parameter is valid. Adding a simple check would prevent potential errors.

 function fetchStargazers(url) {
+    if (!url) {
+        console.error('fetchStargazers called without a URL');
+        return;
+    }
     const stargazersSection = document.getElementById('stargazers-section');
     if (!stargazersSection) return;
     // Show loading state
     stargazersSection.classList.add('opacity-50');

website/views/project.py (3)

1267-1295: Remove debug print statements before deploying to production.

The code contains multiple debug print statements that should be removed or replaced with proper logging before deploying to production. These statements could expose sensitive information or clutter server logs.

-            print(f"[DEBUG] repo.repo_url: {repo.repo_url}")
-            repo_path = repo.repo_url.split("github.com/")[-1]
-            print(f"[DEBUG] repo_path: {repo_path}")
-            owner, repo_name = repo_path.split("/")
-            print(f"[DEBUG] Owner: {owner}, Repo Name: {repo_name}")
+            repo_path = repo.repo_url.split("github.com/")[-1]
+            owner, repo_name = repo_path.split("/")

-            api_url = f"https://api.github.com/repos/{owner}/{repo_name}/stargazers"
-            print(f"[DEBUG] Stargazers API URL: {api_url}")
+            api_url = f"https://api.github.com/repos/{owner}/{repo_name}/stargazers"

-                print(f"[DEBUG] Stargazers API page {current_page} status: {response.status_code}")
-                try:
-                    print(f"[DEBUG] Stargazers API response body (first 500 chars): {response.text[:500]}")
-                except Exception as debug_e:
-                    print(f"[DEBUG] Could not print response body: {debug_e}")

Consider replacing debug prints with proper logging:

import logging
logger = logging.getLogger(__name__)

# Then replace prints with:
logger.debug(f"Stargazers API URL: {api_url}")

---

1322-1336: Improve filtering implementation and pagination efficiency.

The current filtering implementation for "recent" stargazers simply reverses the list, which might not accurately represent recent stargazers depending on how GitHub returns the data. Consider using API parameters if GitHub supports sorting or implementing a more sophisticated filtering approach.

             if "stargazers" not in context:
                 # Apply filtering based on filter_type
                 if filter_type == "recent":
-                    all_stargazers.reverse()
+                    # Option 1: If GitHub API supports sorting by starred_at date
+                    # This would require using a different API endpoint that includes dates
+                    # api_url = f"https://api.github.com/repos/{owner}/{repo_name}/stargazers"
+                    # headers["Accept"] = "application/vnd.github.v3.star+json"
+                    # Then parse the "starred_at" field from responses
+                    
+                    # For now, reversing is reasonable if the API returns oldest first
+                    all_stargazers.reverse()
                 total_stargazers = len(all_stargazers)
                 total_pages = (total_stargazers + per_page - 1) // per_page
                 page = max(1, min(page, total_pages)) if total_pages > 0 else 1
                 start_idx = (page - 1) * per_page
                 end_idx = start_idx + per_page
                 context["stargazers"] = all_stargazers[start_idx:end_idx]

Consider using GitHub's StarredAt API for more accurate timestamp-based filtering:

# For more accurate "recent" filtering:
if filter_type == "recent":
    # Use GitHub's StarredAt API which provides timestamps
    headers["Accept"] = "application/vnd.github.v3.star+json"
    # Then sort by the "starred_at" field in the response

---

1267-1345: Extract stargazers fetching into a separate method for better maintainability.

The stargazers fetching logic is quite complex and would benefit from being extracted into a separate method. This would improve readability, allow easier testing, and make the get_context_data method less cluttered.

def fetch_repo_stargazers(self, repo, filter_type="all", page=1, per_page=10):
    """
    Fetch stargazers for a repository with pagination and filtering.
    
    Args:
        repo: The repository object
        filter_type: Type of filter to apply ('all' or 'recent')
        page: Page number to fetch
        per_page: Number of items per page
        
    Returns:
        dict: Dictionary containing:
            - stargazers: List of stargazers for the requested page
            - error: Error message if any
            - total_stargazers: Total number of stargazers
            - total_pages: Total number of pages
            - current_page: Current page number
    """
    try:
        # Extract repo info
        repo_path = repo.repo_url.split("github.com/")[-1]
        owner, repo_name = repo_path.split("/")
        
        # Set up API request
        api_url = f"https://api.github.com/repos/{owner}/{repo_name}/stargazers"
        headers = {"Accept": "application/vnd.github.v3+json"}
        if hasattr(settings, "GITHUB_TOKEN") and settings.GITHUB_TOKEN and settings.GITHUB_TOKEN != "blank":
            headers["Authorization"] = f"token {settings.GITHUB_TOKEN}"
        
        # Fetch all stargazers
        all_stargazers = []
        current_page = 1
        api_per_page = 100
        max_retries = 3
        
        # Rest of the implementation...
        
        return {
            "stargazers": paginated_stargazers,
            "error": None,
            "total_stargazers": total_stargazers,
            "total_pages": total_pages,
            "current_page": page
        }
    except Exception as e:
        logger.error(f"Error fetching stargazers: {e}", exc_info=True)
        return {
            "stargazers": [],
            "error": "Error fetching stargazers. Please try again later.",
            "total_stargazers": 0,
            "total_pages": 0,
            "current_page": 1
        }

Then in get_context_data:

# Fetch stargazers
filter_type = self.request.GET.get("filter", "all")
page = int(self.request.GET.get("page", 1))
stargazers_data = self.fetch_repo_stargazers(repo, filter_type, page)

# Add to context
context.update(stargazers_data)
context["filter_type"] = filter_type

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Knowledge Base: Disabled due to Reviews > Disable Knowledge Base setting

website/views/project.py:1267

• This new logic should have corresponding unit or integration tests (e.g., mocking GitHub API responses) to verify pagination, filtering, and error handling behaviors.

        # Fetch stargazers for this repo (with pagination and filter)