fix(api): fix topThreeUsers endpoint 404 issue

Author: nikohoffren

pages/api/topThreeUsers.js

@@ -4,83 +4,36 @@ import {
 } from '../../utils/fetchTopUsersFromDb'
 import { fetchTopThreeUsersByPullRequests } from '../../utils/fetchTopThreeUsersByPullRequests'
 
-export default async (req, res) => {
+export default async function handler(req, res) {
   res.setHeader('Access-Control-Allow-Origin', '*')
   res.setHeader('Access-Control-Allow-Methods', 'GET')
   res.setHeader('Access-Control-Allow-Headers', 'Content-Type')
-  res.setHeader('Cache-Control', 'public, max-age=300') //* Cache for 5 minutes
+  res.setHeader('Cache-Control', 'public, max-age=300')
 
   try {
     const message = 'API: Checking DB for cached top three users...'
     console.log(message)
     res.setHeader('X-Debug-Message', message)
 
-    const timeoutPromise = new Promise((_, reject) => {
-      setTimeout(
-        () => reject(new Error('DB operation timed out after 15 seconds')),
-        15000
-      )
-    })
+    let data = await getTopThreeUsersFromDb()
 
-    let data
-    try {
-      data = await Promise.race([getTopThreeUsersFromDb(), timeoutPromise])
-    } catch (dbError) {
-      console.log('DB fetch timed out or failed:', dbError.message)
-      res.setHeader('X-Debug-Message-1', `DB Error: ${dbError.message}`)
-      data = null
-    }
-
-    if (!data || data.length === 0) {
-      const message2 = 'API: No cached data found, fetching from GitHub...'
-      console.log(message2)
-      res.setHeader('X-Debug-Message-2', message2)
-
-      try {
-        data = await fetchTopThreeUsersByPullRequests(
-          'fork-commit-merge/fork-commit-merge'
-        )
+    if (!data) {
+      console.log('No cached data found, fetching from GitHub...')
+      data = await fetchTopThreeUsersByPullRequests('fork-commit-merge/fork-commit-merge')
 
-        if (data && data.length > 0) {
-          const message3 = 'API: Storing new data in DB...'
-          console.log(message3)
-          res.setHeader('X-Debug-Message-3', message3)
-
-          storeTopThreeUsersInDb(data).catch(storeError => {
-            console.error('Failed to store data in DB:', storeError)
-          })
-        }
-      } catch (githubError) {
-        console.error('GitHub fetch error:', githubError.message)
-        res.setHeader(
-          'X-Debug-Message-5',
-          `GitHub Error: ${githubError.message}`
-        )
-        return res
-          .status(500)
-          .json({ error: 'Failed to fetch contributors data' })
+      if (data && data.length > 0) {
+        await storeTopThreeUsersInDb(data)
       }
-    } else {
-      const message4 = 'API: Using cached data from DB'
-      console.log(message4)
-      res.setHeader('X-Debug-Message-4', message4)
     }
 
     if (!data || data.length === 0) {
-      const message5 = 'API: No data available'
-      console.log(message5)
-      res.setHeader('X-Debug-Message-5', message5)
       return res.status(404).json({ error: 'No data available' })
     }
 
-    res.status(200).json(data)
+    return res.status(200).json(data)
   } catch (error) {
     console.error('API route error:', error)
-    console.error('Error details:', error.response?.data || error.message)
-
-    res.status(500).json({
-      error: 'Internal Server Error',
-      message: error.message || 'Unknown error'
-    })
+    return res.status(500).json({ error: 'Internal Server Error' })
   }
 }
+

utils/fetchTopThreeUsersByPullRequests.ts

@@ -1,13 +1,26 @@
 export async function fetchTopThreeUsersByPullRequests(repoPath: string) {
   try {
-    console.log('Starting fetchTopThreeUsersByPullRequests...')
     const headers: Record<string, string> = {
       Accept: 'application/vnd.github.v3+json'
     }
 
     if (process.env.GITHUB_TOKEN) {
       headers.Authorization = `token ${process.env.GITHUB_TOKEN}`
-      console.log('Using GitHub token for authentication')
+      console.log('Token present and being used') // Debug log
+    } else {
+      console.log('No GitHub token found') // Debug log
+    }
+
+    const testUrl = `https://api.github.com/repos/${repoPath}/pulls?state=closed&per_page=100`
+
+    // Test the token
+    const testResponse = await fetch(testUrl, { headers })
+    if (!testResponse.ok) {
+      console.error('GitHub API Error:', {
+        status: testResponse.status,
+        statusText: testResponse.statusText,
+        headers: Object.fromEntries(testResponse.headers)
+      })
     }
 
     let url: string | null = `https://api.github.com/repos/fork-commit-merge/fork-commit-merge/pulls?state=closed&per_page=100`
@@ -84,7 +97,8 @@ export async function fetchTopThreeUsersByPullRequests(repoPath: string) {
     console.log('Final sorted users:', sortedUsers)
     return sortedUsers
   } catch (error) {
-    console.error('API Error in fetchTopThreeUsersByPullRequests:', error)
-    return []
+    console.error('Detailed error in fetchTopThreeUsersByPullRequests:', error)
+    throw error
   }
 }
+