fix(api): fix topThreeUsers endpoint 404 issue

Author: nikohoffren

pages/api/topThreeUsers.js

@@ -5,35 +5,45 @@ import {
 import { fetchTopThreeUsersByPullRequests } from '../../utils/fetchTopThreeUsersByPullRequests'
 
 export default async function handler(req, res) {
+  res.setTimeout(30000);
+
   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')
 
   try {
-    const message = 'API: Checking DB for cached top three users...'
-    console.log(message)
-    res.setHeader('X-Debug-Message', message)
+    console.log('API: Starting request processing')
+    console.log('GitHub Token present:', !!process.env.GITHUB_TOKEN)
 
     let data = await getTopThreeUsersFromDb()
+    console.log('DB Data found:', !!data)
 
     if (!data) {
-      console.log('No cached data found, fetching from GitHub...')
+      console.log('Fetching from GitHub...')
       data = await fetchTopThreeUsersByPullRequests('fork-commit-merge/fork-commit-merge')
+      console.log('GitHub data fetched:', !!data)
 
       if (data && data.length > 0) {
         await storeTopThreeUsersInDb(data)
+        console.log('Data stored in DB')
       }
     }
 
     if (!data || data.length === 0) {
+      console.log('No data available')
       return res.status(404).json({ error: 'No data available' })
     }
 
+    console.log('Returning data:', data.length, 'items')
     return res.status(200).json(data)
   } catch (error) {
     console.error('API route error:', error)
-    return res.status(500).json({ error: 'Internal Server Error' })
+    console.error('Stack trace:', error.stack)
+    return res.status(500).json({
+      error: 'Internal Server Error',
+      message: error.message,
+      stack: process.env.NODE_ENV === 'development' ? error.stack : undefined
+    })
   }
 }
-

utils/fetchTopThreeUsersByPullRequests.ts

@@ -5,22 +5,26 @@ export async function fetchTopThreeUsersByPullRequests(repoPath: string) {
     }
 
     if (process.env.GITHUB_TOKEN) {
-      headers.Authorization = `token ${process.env.GITHUB_TOKEN}`
-      console.log('Token present and being used') // Debug log
+      headers.Authorization = `Bearer ${process.env.GITHUB_TOKEN}`
+      console.log('Token present and being used')
     } else {
-      console.log('No GitHub token found') // Debug log
+      console.log('No GitHub token found')
     }
 
     const testUrl = `https://api.github.com/repos/${repoPath}/pulls?state=closed&per_page=100`
 
-    // Test the token
     const testResponse = await fetch(testUrl, { headers })
+    console.log('GitHub API Response Status:', testResponse.status)
+
     if (!testResponse.ok) {
+      const responseText = await testResponse.text()
       console.error('GitHub API Error:', {
         status: testResponse.status,
         statusText: testResponse.statusText,
-        headers: Object.fromEntries(testResponse.headers)
+        headers: Object.fromEntries(testResponse.headers),
+        body: responseText
       })
+      throw new Error(`GitHub API Error: ${testResponse.status} ${testResponse.statusText}`)
     }
 
     let url: string | null = `https://api.github.com/repos/fork-commit-merge/fork-commit-merge/pulls?state=closed&per_page=100`
@@ -101,4 +105,3 @@ export async function fetchTopThreeUsersByPullRequests(repoPath: string) {
     throw error
   }
 }
-