Merge branch 'main' into fix/og-bypass-cache

Author: Kokila-chandrakar

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

@@ -104,6 +104,8 @@ describe('DashboardPage', () => {
     commitClock: [],
     graphData: { nodes: [], links: [] },
     lastSyncedAt: undefined,
+    popularRepos: [],
+    pinnedRepos: [],
   };
 
   beforeEach(() => {

app/api/compare/tests/validation.test.ts

@@ -0,0 +1,66 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { GET } from '../route';
+
+vi.mock('@/lib/github', () => ({
+  getFullDashboardData: vi.fn(),
+}));
+
+import { getFullDashboardData } from '@/lib/github';
+
+const makeRequest = (search: string) => new Request(`http://localhost:3000/api/compare?${search}`);
+
+describe('GET /api/compare - Validation', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+
+    vi.mocked(getFullDashboardData).mockResolvedValue({
+      calendar: {
+        totalContributions: 50,
+        weeks: [],
+      },
+    } as never);
+  });
+
+  it('returns 400 when both usernames are missing', async () => {
+    const res = await GET(makeRequest(''));
+
+    expect(res.status).toBe(400);
+
+    const data = await res.json();
+    expect(data.error).toBeDefined();
+  });
+
+  it('returns 400 when user1 exceeds the maximum length', async () => {
+    const res = await GET(makeRequest(`user1=${'a'.repeat(40)}&user2=octocat`));
+
+    expect(res.status).toBe(400);
+
+    const data = await res.json();
+    expect(data.error).toBeDefined();
+  });
+
+  it('returns 400 for invalid username format', async () => {
+    const res = await GET(makeRequest('user1=-invalid&user2=octocat'));
+
+    expect(res.status).toBe(400);
+
+    const data = await res.json();
+    expect(data.details.fieldErrors.user1).toBeDefined();
+  });
+
+  it('returns 400 when comparing identical usernames', async () => {
+    const res = await GET(makeRequest('user1=octocat&user2=octocat'));
+
+    expect(res.status).toBe(400);
+
+    const data = await res.json();
+
+    expect(data.details.fieldErrors.user2).toContain('Cannot compare a user with themselves.');
+  });
+
+  it('returns 400 for self-comparison regardless of case', async () => {
+    const res = await GET(makeRequest('user1=OctoCat&user2=octocat'));
+
+    expect(res.status).toBe(400);
+  });
+});

app/api/streak/route.test.ts

@@ -1146,6 +1146,46 @@ describe('GET /api/streak', () => {
       expect(body).toContain('COMMITS THIS MONTH');
     });
 
+    it('automatically overrides or widens the query bounds to encompass the start of the previous month when view=monthly is requested with custom from/to params', async () => {
+      vi.useFakeTimers();
+      vi.setSystemTime(new Date('2026-06-02T12:00:00Z'));
+
+      vi.mocked(fetchGitHubContributions).mockResolvedValueOnce({
+        calendar: {
+          totalContributions: 25,
+          weeks: [
+            {
+              contributionDays: [
+                { date: '2026-05-01', contributionCount: 0 },
+                { date: '2026-05-15', contributionCount: 10 },
+                { date: '2026-06-01', contributionCount: 15 },
+                { date: '2026-06-02', contributionCount: 0 },
+              ],
+            },
+          ],
+        } as ContributionCalendar,
+        repoContributions: [],
+      } as unknown as ExtendedContributionData);
+
+      try {
+        const response = await GET(
+          makeRequest({ user: 'octocat', view: 'monthly', from: '2026-06-01', to: '2026-06-02' })
+        );
+
+        expect(response.status).toBe(200);
+        // The expected prev month (May 2026) start is 2026-05-01.
+        // So 'from' should be widened to 2026-05-01T00:00:00Z.
+        // 'to' should be today's date in ISO: 2026-06-02T12:00:00.000Z.
+        expect(fetchGitHubContributions).toHaveBeenCalledWith('octocat', {
+          bypassCache: false,
+          from: '2026-05-01T00:00:00Z',
+          to: '2026-06-02T12:00:00.000Z',
+        });
+      } finally {
+        vi.useRealTimers();
+      }
+    });
+
     it('uses the selected year when generating archived monthly stats', async () => {
       vi.useFakeTimers();
       vi.setSystemTime(new Date('2026-05-20T12:00:00Z'));
@@ -1154,8 +1194,14 @@ describe('GET /api/streak', () => {
         calendar: {
           totalContributions: 25,
           weeks: [
-            { contributionDays: [{ date: '2024-11-15', contributionCount: 10 }] },
-            { contributionDays: [{ date: '2024-12-15', contributionCount: 15 }] },
+            {
+              contributionDays: [
+                { date: '2024-11-01', contributionCount: 0 },
+                { date: '2024-11-15', contributionCount: 10 },
+                { date: '2024-12-15', contributionCount: 15 },
+                { date: '2024-12-31', contributionCount: 0 },
+              ],
+            },
           ],
         } as ContributionCalendar,
         repoContributions: [],
@@ -1248,8 +1294,14 @@ describe('GET /api/streak', () => {
         calendar: {
           totalContributions: 150,
           weeks: [
-            { contributionDays: [{ date: '2026-04-15', contributionCount: 10 }] },
-            { contributionDays: [{ date: '2026-05-15', contributionCount: 15 }] },
+            {
+              contributionDays: [
+                { date: '2026-04-01', contributionCount: 0 },
+                { date: '2026-04-15', contributionCount: 10 },
+                { date: '2026-05-15', contributionCount: 15 },
+                { date: '2026-05-20', contributionCount: 0 },
+              ],
+            },
           ],
         } as ContributionCalendar,
         repoContributions: [],
@@ -1275,8 +1327,14 @@ describe('GET /api/streak', () => {
         calendar: {
           totalContributions: 150,
           weeks: [
-            { contributionDays: [{ date: '2026-04-15', contributionCount: 10 }] },
-            { contributionDays: [{ date: '2026-05-15', contributionCount: 15 }] },
+            {
+              contributionDays: [
+                { date: '2026-04-01', contributionCount: 0 },
+                { date: '2026-04-15', contributionCount: 10 },
+                { date: '2026-05-15', contributionCount: 15 },
+                { date: '2026-05-20', contributionCount: 0 },
+              ],
+            },
           ],
         } as ContributionCalendar,
         repoContributions: [],

app/api/streak/route.ts

@@ -103,25 +103,54 @@ export async function GET(request: Request) {
     } = parseResult.data;
     const normalizedView = view as 'default' | 'monthly' | 'heatmap' | 'pulse';
     const themeName = theme || 'dark';
-    const from = customFrom
+
+    let timezone = 'UTC';
+    if (tzParam) {
+      timezone = new Intl.DateTimeFormat(undefined, { timeZone: tzParam }).resolvedOptions()
+        .timeZone;
+    }
+
+    let from = customFrom
       ? new Date(customFrom).toISOString()
       : year
         ? `${year}-01-01T00:00:00Z`
         : undefined;
-    const to = customTo
+    let to = customTo
       ? new Date(customTo).toISOString()
       : year
         ? `${year}-12-31T23:59:59Z`
         : undefined;
-    const currentYear = new Date().getUTCFullYear();
-    const isHistoricalYear = !!year && Number(year) < currentYear;
 
-    let timezone = 'UTC';
-    if (tzParam) {
-      timezone = new Intl.DateTimeFormat(undefined, { timeZone: tzParam }).resolvedOptions()
-        .timeZone;
+    if (normalizedView === 'monthly') {
+      const referenceDate = getMonthlyReferenceDate(year, timezone) || new Date();
+      const localTodayStr = new Intl.DateTimeFormat('en-CA', { timeZone: timezone }).format(
+        referenceDate
+      );
+      const [currentYearStr, currentMonthStr] = localTodayStr.split('-');
+      const currentYearNum = parseInt(currentYearStr, 10);
+      const currentMonthNum = parseInt(currentMonthStr, 10);
+
+      let prevMonth = currentMonthNum - 1;
+      let prevYear = currentYearNum;
+      if (prevMonth === 0) {
+        prevMonth = 12;
+        prevYear -= 1;
+      }
+
+      const calculatedFromStr = `${prevYear}-${prevMonth.toString().padStart(2, '0')}-01T00:00:00Z`;
+      if (!from || new Date(from) > new Date(calculatedFromStr)) {
+        from = calculatedFromStr;
+      }
+
+      const referenceISO = referenceDate.toISOString();
+      if (!to || new Date(to) < new Date(referenceISO)) {
+        to = referenceISO;
+      }
     }
 
+    const currentYear = new Date().getUTCFullYear();
+    const isHistoricalYear = !!year && Number(year) < currentYear;
+
     const isAutoTheme = themeName === 'auto';
     const isRandomTheme = themeName === 'random';
     const selectedTheme = (() => {
@@ -256,7 +285,7 @@ export async function GET(request: Request) {
       }
     }
 
-    if (days) {
+    if (days && normalizedView !== 'monthly') {
       const allDays = calendar.weeks.flatMap((w) => w.contributionDays);
 
       const filteredDays = allDays.slice(-days);

app/compare/CompareClient.responsive-breakpoints.test.tsx

@@ -0,0 +1,75 @@
+import { render, screen } from '@testing-library/react';
+import '@testing-library/jest-dom/vitest';
+import React, { type ReactNode } from 'react';
+import { describe, expect, it, vi } from 'vitest';
+
+vi.mock('next/navigation', () => ({
+  useRouter: () => ({
+    replace: vi.fn(),
+  }),
+  useSearchParams: () => ({
+    get: vi.fn(() => null),
+  }),
+}));
+
+vi.mock('framer-motion', () => ({
+  motion: new Proxy(
+    {},
+    {
+      get: (_, tag) => {
+        return ({ children, ...props }: { children?: ReactNode; [key: string]: unknown }) =>
+          React.createElement(tag as string, props, children);
+      },
+    }
+  ),
+  AnimatePresence: ({ children }: { children?: ReactNode }) => <>{children}</>,
+}));
+
+vi.doMock('recharts', () => ({
+  ResponsiveContainer: ({ children }: { children?: ReactNode }) => <div>{children}</div>,
+  RadarChart: ({ children }: { children?: ReactNode }) => <div>{children}</div>,
+  Radar: () => <div />,
+  PolarGrid: () => <div />,
+  PolarAngleAxis: () => <div />,
+  PolarRadiusAxis: () => <div />,
+  Tooltip: () => <div />,
+}));
+
+async function renderCompareClient() {
+  const { default: CompareClient } = await import('./CompareClient');
+
+  return render(<CompareClient />);
+}
+
+describe('CompareClient responsive breakpoints', () => {
+  it('renders mobile friendly input controls', async () => {
+    await renderCompareClient();
+
+    expect(screen.getByPlaceholderText(/github username #1/i)).toBeInTheDocument();
+    expect(screen.getByPlaceholderText(/github username #2/i)).toBeInTheDocument();
+  });
+
+  it('renders compare button for small viewport interactions', async () => {
+    await renderCompareClient();
+
+    expect(screen.getByRole('button', { name: /compare/i })).toBeInTheDocument();
+  });
+
+  it('uses responsive container spacing classes', async () => {
+    const { container } = await renderCompareClient();
+
+    expect(container.querySelector('.px-4.sm\\:px-6')).toBeTruthy();
+  });
+
+  it('uses responsive layout grid classes', async () => {
+    const { container } = await renderCompareClient();
+
+    expect(container.querySelector('.flex-col.sm\\:flex-row')).toBeTruthy();
+  });
+
+  it('contains responsive typography classes for mobile scaling', async () => {
+    const { container } = await renderCompareClient();
+
+    expect(container.querySelector('.text-3xl.sm\\:text-4xl')).toBeTruthy();
+  });
+});

app/compare/page.empty-fallback.test.tsx

@@ -0,0 +1,38 @@
+import { render, screen } from '@testing-library/react';
+import { describe, expect, it, vi } from 'vitest';
+
+import ComparePage, { metadata } from './page';
+
+vi.mock('./CompareClient', () => ({
+  default: () => <div data-testid="compare-client">Compare Client</div>,
+}));
+
+vi.mock('../components/Footer', () => ({
+  Footer: () => <div data-testid="footer">Footer</div>,
+}));
+
+describe('ComparePage empty fallback behavior', () => {
+  it('renders safely without crashing', () => {
+    expect(() => render(<ComparePage />)).not.toThrow();
+  });
+
+  it('renders CompareClient content', () => {
+    render(<ComparePage />);
+
+    expect(screen.getByTestId('compare-client')).toBeDefined();
+  });
+
+  it('renders Footer component', () => {
+    render(<ComparePage />);
+
+    expect(screen.getByTestId('footer')).toBeDefined();
+  });
+
+  it('exports valid metadata', () => {
+    expect(metadata.title).toBe('Compare | CommitPulse');
+  });
+
+  it('contains openGraph fallback metadata', () => {
+    expect(metadata.openGraph?.title).toBe('Compare Developers | CommitPulse');
+  });
+});

app/components/CopyRepoButton.type-compiler.test.tsx

@@ -0,0 +1,26 @@
+import { describe, expectTypeOf, it } from 'vitest';
+import CopyRepoButton from './CopyRepoButton';
+
+describe('CopyRepoButton Type Compiler Validation', () => {
+  it('exports CopyRepoButton as a function component', () => {
+    expectTypeOf(CopyRepoButton).toBeFunction();
+  });
+
+  it('accepts no parameters', () => {
+    expectTypeOf(CopyRepoButton).parameters.toEqualTypeOf<[]>();
+  });
+
+  it('returns a valid component type', () => {
+    expectTypeOf<ReturnType<typeof CopyRepoButton>>().toBeObject();
+  });
+
+  it('maintains a stable callable signature', () => {
+    expectTypeOf(CopyRepoButton).toEqualTypeOf<typeof CopyRepoButton>();
+  });
+
+  it('preserves return type consistency', () => {
+    expectTypeOf<ReturnType<typeof CopyRepoButton>>().toEqualTypeOf<
+      ReturnType<typeof CopyRepoButton>
+    >();
+  });
+});