Remove octokit dependency (#681)

.github/workflows/playwright.yml

@@ -5,7 +5,7 @@ on:
     pull_request:
         branches: [main, master]
 env:
-  GITHUB_ACCESS_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+  GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 jobs:
     test:
         timeout-minutes: 60

package.json

@@ -20,7 +20,6 @@
     "@ai-sdk/azure": "^1.0.7",
     "@googlemaps/react-wrapper": "^1.1.35",
     "@googlemaps/typescript-guards": "^2.0.1",
-    "@octokit/rest": "^21.0.2",
     "@playwright/test": "^1.49.0",
     "@radix-ui/react-dialog": "^1.1.4",
     "@radix-ui/react-icons": "^1.3.0",

src/lib/github.ts

@@ -1,8 +1,13 @@
-import { Octokit } from "@octokit/rest";
-import { VWCContributor, GithubRepo, GithubContributor } from "@utils/types";
+import { VWCContributor, GithubRepo, GithubContributor, GithubUser } from "@utils/types";
+import axios from "axios";
 
-const token = process.env.GITHUB_ACCESS_TOKEN || "";
-const octokit = new Octokit({ auth: token });
+const token = process.env.GITHUB_TOKEN || "";
+const git_api = axios.create({
+    baseURL: "https://api.github.com",
+    headers: {
+        Authorization: `Bearer ${token}`,
+    },
+});
 
 export const getProjectContributors = async (
     owner: string,
@@ -12,48 +17,60 @@ export const getProjectContributors = async (
     const topContributors = await getGithubRepoContributors(owner, repo, top);
     const projectContributors = Promise.all(
         topContributors.map(async (contributor) => {
-            const user = await octokit.rest.users.getByUsername({
-                username: contributor.login,
-            });
-            if (user.data.name) {
+            const response = await git_api.get(`/users/${contributor.login}`);
+            if (response.status == 200) {
+                const user = response.data as GithubUser;
                 return {
                     ...contributor,
-                    ...user.data,
-                    name: user.data.name!,
+                    ...user,
                 };
+            } else {
+                if ("error" in response) {
+                    throw new Error(
+                        `Error fetching user data for ${contributor.login}\nStatus code: ${response.status}\nError: ${response.error}`
+                    );
+                }
+                throw new Error(
+                    `Error fetching user data for ${contributor.login}\nStatus code: ${response.status}`
+                );
             }
-            return;
         })
     );
-    return (await projectContributors).filter((contributor) => contributor !== undefined);
+    return await projectContributors;
 };
 
 export const getGithubRepo = async (owner: string, repo: string): Promise<GithubRepo> => {
-    const response = await octokit.rest.repos.get({
-        owner: owner,
-        repo: repo,
-    });
-    return response.data;
+    const response = await git_api.get(`/repos/${owner}/${repo}`);
+    if (response.status == 200) {
+        return response.data as GithubRepo;
+    } else {
+        if ("error" in response) {
+            throw new Error(
+                `Error fetching repo data for ${owner}/${repo}\nStatus code: ${response.status}\nError: ${response.error}`
+            );
+        }
+        throw new Error(
+            `Error fetching repo data for ${owner}/${repo}\nStatus code: ${response.status}`
+        );
+    }
 };
 
 export const getGithubRepoContributors = async (
     owner: string,
     repo: string,
     top: number = 4
 ): Promise<GithubContributor[]> => {
-    const response = await octokit.rest.repos.listContributors({
-        owner: owner,
-        repo: repo,
-        per_page: top,
-    });
-    const contributors = response.data.map((contributor) => {
-        if (contributor.login) {
-            return {
-                ...contributor,
-                login: contributor.login!,
-            };
+    const response = await git_api.get(`/repos/${owner}/${repo}/contributors`);
+    if (response.status == 200) {
+        return (response.data as GithubContributor[]).slice(0, top);
+    } else {
+        if ("error" in response) {
+            throw new Error(
+                `Error fetching contributor data for ${owner}/${repo}\nStatus code: ${response.status}\nError: ${response.error}`
+            );
         }
-        return;
-    });
-    return contributors.filter((contributor) => contributor !== undefined);
+        throw new Error(
+            `Error fetching contributor data for ${owner}/${repo}\nStatus code: ${response.status}`
+        );
+    }
 };