fix(api): add request limit to topThreeContributors to prevent infinite loops

Author: nikohoffren

components/index/TopThreeContributors.tsx

@@ -136,12 +136,3 @@ const TopThreeContributors: FC = () => {
 }
 
 export { TopThreeContributors }
-
-
-
-
-
-
-
-
-

pages/api/topThreeUsers.js

@@ -50,5 +50,3 @@ export default async (req, res) => {
     res.status(500).json({ error: 'Internal Server Error' });
   }
 };
-
-

utils/fetchTopThreeUsersByPullRequests.ts

@@ -10,16 +10,21 @@ export async function fetchTopThreeUsersByPullRequests(repoPath: string) {
 
     let url = `https://api.github.com/repos/fork-commit-merge/fork-commit-merge/pulls?state=closed&per_page=100`;
     const userContributions = new Map();
+    let requestCount = 0;
+    const MAX_REQUESTS = 10;
 
-    while (url) {
-      const response = await axios.get(url, { headers });
-      console.log(`Fetched ${response.data.length} pull requests`);
+    while (url && requestCount < MAX_REQUESTS) {
+      const response = await axios.get(url, {
+        headers,
+        timeout: 10000
+      });
+      requestCount++;
+      console.log(`Fetched ${response.data.length} pull requests (Request ${requestCount}/${MAX_REQUESTS})`);
 
       const pullRequests = response.data;
       pullRequests.forEach((pr: any) => {
         const username = pr.user.login;
         if (username === "dependabot" || username === "dependabot[bot]" || username === "nikohoffren") {
-          console.log(`Skipping contribution from ${username}`);
           return;
         }
 
@@ -41,7 +46,7 @@ export async function fetchTopThreeUsersByPullRequests(repoPath: string) {
       .map(([username, count]) => ({
         username,
         contributions: count,
-        avatarUrl: `https://avatars.githubusercontent.com/${username}`,
+        avatarUrl: `https://avatars.githubusercontent.com//${username}`,
       }));
 
     console.log('Final sorted users:', sortedUsers);
@@ -56,12 +61,3 @@ export async function fetchTopThreeUsersByPullRequests(repoPath: string) {
     return [];
   }
 }
-
-
-
-
-
-
-
-
-

utils/fetchTopUsersFromDb.ts

@@ -53,6 +53,3 @@ export async function storeTopThreeUsersInDb(data: any[]) {
     throw error;
   }
 }
-
-
-