Merge branch 'main' into test-validation-timezone-boundary

Author: JhaSourav07

.env.local.example

@@ -20,8 +20,7 @@
 #   Generate new token (classic)
 #   Required scope: read:user
 # ------------------------------------------------------------
-GITHUB_TOKEN=ghp_your_personal_access_token_here
-
+GITHUB_TOKEN=
 
 # ------------------------------------------------------------
 # REQUIRED — Public site URL

app/(root)/dashboard/[username]/page.test.tsx

@@ -100,6 +100,7 @@ describe('DashboardPage', () => {
     achievements: [],
     commitClock: [],
     graphData: { nodes: [], links: [] },
+    lastSyncedAt: undefined,
   };
 
   beforeEach(() => {

app/(root)/dashboard/[username]/page.tsx

@@ -63,6 +63,7 @@ export default async function DashboardPage({
   params: Promise<{ username: string }>;
   searchParams: Promise<{
     refresh?: string;
+    compare?: string;
     year?: string;
     month?: string;
     from?: string;
@@ -72,6 +73,7 @@ export default async function DashboardPage({
   const { username } = await params;
   const resolvedSearchParams = await searchParams;
   const bypassCache = resolvedSearchParams?.refresh === 'true';
+  const compareUsername = resolvedSearchParams?.compare;
   const period = resolveDashboardPeriod({
     year: resolvedSearchParams?.year,
     month: resolvedSearchParams?.month,
@@ -90,10 +92,11 @@ export default async function DashboardPage({
     });
   } catch (error) {
     if (error instanceof Error && error.message.includes('not found')) {
-      // Smart Redirect: If the GraphQL "user" query fails, check if it's actually an Organization
       let fallbackProfile;
       try {
-        fallbackProfile = await fetchUserProfile(username, { bypassCache });
+        fallbackProfile = await fetchUserProfile(username, {
+          bypassCache,
+        });
       } catch {
         return notFound();
       }
@@ -105,5 +108,24 @@ export default async function DashboardPage({
     throw error;
   }
 
-  return <DashboardClient initialData={data} username={username} period={period} />;
+  let compareData = null;
+
+  if (compareUsername && compareUsername.toLowerCase() !== username.toLowerCase()) {
+    try {
+      compareData = await getFullDashboardData(compareUsername, {
+        bypassCache,
+      });
+    } catch {
+      compareData = null;
+    }
+  }
+
+  return (
+    <DashboardClient
+      initialData={data}
+      username={username}
+      compareData={compareData}
+      period={period}
+    />
+  );
 }

app/api/compare/route.test.ts

@@ -90,4 +90,38 @@ describe('GET /api/compare', () => {
     const res = await GET(makeRequest('user1=octocat&user2=ghost123'));
     expect(res.status).toBe(404);
   });
+
+  it('returns 403 when the user1 fetch hits a GitHub rate limit', async () => {
+    vi.mocked(getFullDashboardData).mockRejectedValueOnce(new Error('API Rate Limit Exceeded'));
+
+    const res = await GET(makeRequest('user1=octocat&user2=torvalds'));
+
+    expect(res.status).toBe(403);
+    const data = await res.json();
+    expect(data.error).toBe('GitHub API rate limit reached. Please configure GITHUB_TOKEN.');
+  });
+
+  it('returns 403 when the user2 fetch hits a GitHub rate limit', async () => {
+    vi.mocked(getFullDashboardData)
+      .mockResolvedValueOnce({ calendar: { totalContributions: 0, weeks: [] } } as never)
+      .mockRejectedValueOnce(new Error('GitHub GraphQL API returned status 403'));
+
+    const res = await GET(makeRequest('user1=octocat&user2=torvalds'));
+
+    expect(res.status).toBe(403);
+    const data = await res.json();
+    expect(data.error).toBe('GitHub API rate limit reached. Please configure GITHUB_TOKEN.');
+  });
+
+  it('returns 500 for non-rate-limit upstream failures instead of reporting not found', async () => {
+    vi.mocked(getFullDashboardData).mockRejectedValueOnce(
+      new Error('GitHub GraphQL API returned status 500')
+    );
+
+    const res = await GET(makeRequest('user1=octocat&user2=torvalds'));
+
+    expect(res.status).toBe(500);
+    const data = await res.json();
+    expect(data.error).toContain('GitHub GraphQL API returned status 500');
+  });
 });

app/api/compare/route.ts

@@ -4,6 +4,38 @@ import { compareParamsSchema } from '@/lib/validations';
 
 export const revalidate = 3600;
 
+function buildCompareFetchErrorResponse(user: string, reason: unknown): NextResponse {
+  const message = reason instanceof Error ? reason.message : 'Unknown error';
+  const lowerMessage = message.toLowerCase();
+
+  if (lowerMessage.includes('not found') || lowerMessage.includes('could not resolve')) {
+    return NextResponse.json(
+      {
+        error: `Failed to fetch data for "${user}": ${message}`,
+      },
+      { status: 404 }
+    );
+  }
+
+  if (
+    lowerMessage.includes('rate limit') ||
+    message.includes('API limit reached') ||
+    message.includes('status 403')
+  ) {
+    return NextResponse.json(
+      { error: 'GitHub API rate limit reached. Please configure GITHUB_TOKEN.' },
+      { status: 403 }
+    );
+  }
+
+  return NextResponse.json(
+    {
+      error: `Failed to fetch data for "${user}": ${message}`,
+    },
+    { status: 500 }
+  );
+}
+
 export async function GET(request: Request) {
   const { searchParams } = new URL(request.url);
 
@@ -26,21 +58,11 @@ export async function GET(request: Request) {
     ]);
 
     if (result1.status === 'rejected') {
-      return NextResponse.json(
-        {
-          error: `Failed to fetch data for "${user1}": ${result1.reason?.message || 'Unknown error'}`,
-        },
-        { status: 404 }
-      );
+      return buildCompareFetchErrorResponse(user1, result1.reason);
     }
 
     if (result2.status === 'rejected') {
-      return NextResponse.json(
-        {
-          error: `Failed to fetch data for "${user2}": ${result2.reason?.message || 'Unknown error'}`,
-        },
-        { status: 404 }
-      );
+      return buildCompareFetchErrorResponse(user2, result2.reason);
     }
 
     return NextResponse.json({