Merge branch 'main' into test/full-year-zero-contributions-edge-case

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/api/stats/route.ts

@@ -24,10 +24,22 @@ export async function GET(request: Request) {
   const parseResult = statsParamsSchema.safeParse(Object.fromEntries(searchParams.entries()));
 
   if (!parseResult.success) {
+    const details = parseResult.error.flatten();
+
+    if (details.fieldErrors.tz?.length) {
+      return NextResponse.json(
+        {
+          error: 'Invalid "tz" parameter',
+          details,
+        },
+        { status: 400 }
+      );
+    }
+
     return NextResponse.json(
       {
         error: 'Invalid parameters',
-        details: parseResult.error.flatten(),
+        details,
       },
       { status: 400 }
     );

app/api/streak/route.test.ts

@@ -837,6 +837,22 @@ describe('GET /api/streak', () => {
       expect(fieldError).toContain('light');
       expect(fieldError).toContain('neon');
     });
+
+    it('accepts capitalized or mixed-case theme parameter like "NEON" and maps it correctly', async () => {
+      const response = await GET(makeRequest({ user: 'octocat', theme: 'NEON' }));
+      const body = await response.text();
+
+      expect(response.status).toBe(200);
+      expect(body).toContain('ff00ff'); // Neon theme accent is #ff00ff — confirms the neon theme is applied
+    });
+
+    it('accepts mixed-case "random" or "auto" and resolves correctly', async () => {
+      const response = await GET(makeRequest({ user: 'octocat', theme: 'aUtO' }));
+      const body = await response.text();
+
+      expect(response.status).toBe(200);
+      expect(body).toContain('prefers-color-scheme: dark');
+    });
   });
 
   describe('custom colour overrides', () => {

app/api/streak/route.ts

@@ -99,7 +99,7 @@ export async function GET(request: Request) {
       glow,
       format,
     } = parseResult.data;
-
+    const normalizedView = view as 'default' | 'monthly' | 'heatmap' | 'pulse';
     const themeName = theme || 'dark';
     const from = customFrom
       ? new Date(customFrom).toISOString()
@@ -161,7 +161,7 @@ export async function GET(request: Request) {
       hideBackground: hide_background,
       hide_stats,
       lang,
-      view,
+      view: normalizedView,
       delta_format,
       width,
       height,
@@ -278,17 +278,17 @@ export async function GET(request: Request) {
 
     // ─── SVG output mode (default) ──────────────────────────────────────────
     let svg = '';
-    if (view === 'monthly') {
+    if (normalizedView === 'monthly') {
       const stats = calculateMonthlyStats(
         calendar,
         timezone,
         getMonthlyReferenceDate(year, timezone)
       );
       svg = generateMonthlySVG(stats, params);
-    } else if (view === 'heatmap') {
+    } else if (normalizedView === 'heatmap') {
       const stats = calculateStreak(calendar, timezone, undefined, grace);
       svg = generateHeatmapSVG(stats, params, calendar);
-    } else if (view === 'pulse') {
+    } else if (normalizedView === 'pulse') {
       // We still use calculateStreak here to efficiently parse totalContributions for the stat display,
       // even though the sparkline generator will extract its own daily 30-day timeline below.
       const stats = calculateStreak(calendar, timezone, undefined, grace);

app/components/Footer.test.tsx

@@ -1,91 +1,82 @@
-import { describe, it, expect } from 'vitest';
+import '@testing-library/jest-dom/vitest';
 import { render, screen } from '@testing-library/react';
+import { describe, expect, it } from 'vitest';
 import { Footer } from './Footer';
 
 describe('Footer Component', () => {
   it('renders community text', () => {
     render(<Footer />);
 
-    const text = screen.getByText(/Designed for the elite builder community/i);
-
-    expect(text).toBeTruthy();
+    expect(screen.getByText(/Designed for the elite builder community/i)).toBeInTheDocument();
   });
 
-  it('renders Documentation link', () => {
+  it('renders Documentation link with the correct destination', () => {
     render(<Footer />);
 
-    const docLink = screen.getByText(/Documentation/i);
-
-    expect(docLink).toBeTruthy();
+    const documentationLink = screen.getByRole('link', {
+      name: /Documentation/i,
+    });
 
-    expect(docLink.closest('a')?.getAttribute('href')).toBe(
+    expect(documentationLink).toHaveAttribute(
+      'href',
       'https://github.com/JhaSourav07/commitpulse/blob/main/README.md'
     );
   });
 
-  it('opens documentation in new tab', () => {
+  it('opens Documentation link in a new tab securely', () => {
     render(<Footer />);
 
-    const docLink = screen.getByText(/Documentation/i);
+    const documentationLink = screen.getByRole('link', {
+      name: /Documentation/i,
+    });
 
-    expect(docLink.closest('a')?.getAttribute('target')).toBe('_blank');
+    expect(documentationLink).toHaveAttribute('target', '_blank');
+    expect(documentationLink).toHaveAttribute('rel', expect.stringContaining('noopener'));
+    expect(documentationLink).toHaveAttribute('rel', expect.stringContaining('noreferrer'));
   });
 
   it('renders Contributors link', () => {
     render(<Footer />);
 
-    const contributorsLink = screen.getByText(/Contributors/i);
-
-    expect(contributorsLink).toBeTruthy();
+    expect(
+      screen.getByRole('link', {
+        name: /Contributors/i,
+      })
+    ).toBeInTheDocument();
   });
 
-  describe('responsive links and footer tag', () => {
-    it.each([
-      ['mobile', 375],
-      ['desktop', 1280],
-    ])('renders documented links and the footer tag at the %s breakpoint', (_breakpoint, width) => {
-      window.innerWidth = width;
-
-      const { container } = render(<Footer />);
-
-      const layout = container.querySelector('.flex.flex-col.md\\:flex-row');
-      const contributorsLink = screen.getByRole('link', { name: 'Contributors' });
-      const documentationLink = screen.getByRole('link', { name: 'Documentation' });
-      const creatorLink = screen.getByRole('link', { name: 'Creator' });
-
-      expect(layout).toBeTruthy();
-      expect(contributorsLink.getAttribute('href')).toBe('/contributors');
-      expect(documentationLink.getAttribute('href')).toBe(
-        'https://github.com/JhaSourav07/commitpulse/blob/main/README.md'
-      );
-      expect(creatorLink.getAttribute('href')).toBe('https://github.com/jhasourav07');
-      expect(screen.getByText(/2026 CommitPulse\. All rights reserved\./i)).toBeTruthy();
-    });
-  });
-});
-it('renders CommitPulse heading', () => {
-  render(<Footer />);
-
-  const heading = screen.getByText('CommitPulse');
-
-  expect(heading).toBeTruthy();
-});
-it('renders Creator link', () => {
-  render(<Footer />);
+  it('exposes the footer as a semantic contentinfo landmark for screen readers', () => {
+    render(<Footer />);
 
-  const creatorLink = screen.getByText(/Creator/i);
+    // A semantic <footer> is exposed to assistive technology as the contentinfo landmark.
+    const footer = screen.getByRole('contentinfo');
 
-  expect(creatorLink).toBeTruthy();
-});
-it('creator link points to GitHub profile', () => {
-  render(<Footer />);
-
-  const creatorLink = screen.getByText(/Creator/i);
+    expect(footer).toBeInTheDocument();
+    expect(footer.tagName).toBe('FOOTER');
+  });
 
-  expect(creatorLink.closest('a')?.getAttribute('href')).toBe('https://github.com/jhasourav07');
-});
-it('renders copyright text', () => {
-  render(<Footer />);
+  it('includes responsive layout classes for mobile and desktop breakpoints', () => {
+    render(<Footer />);
 
-  expect(screen.getByText(/© 2026 CommitPulse. All rights reserved./i)).toBeTruthy();
+    const footer = screen.getByRole('contentinfo');
+
+    // JSDOM cannot calculate Tailwind media queries, so this verifies the actual
+    // responsive utility classes that switch the footer layout at the md breakpoint.
+    const responsiveLayout = footer.querySelector('.flex.flex-col.md\\:flex-row.md\\:items-start');
+
+    expect(responsiveLayout).toBeInTheDocument();
+    expect(responsiveLayout).toHaveClass(
+      'mx-auto',
+      'flex',
+      'max-w-6xl',
+      'flex-col',
+      'items-center',
+      'justify-between',
+      'gap-6',
+      'text-sm',
+      'md:flex-row',
+      'md:items-start',
+      'md:gap-0'
+    );
+  });
 });

app/page.tsx

@@ -15,6 +15,7 @@ import { Footer } from '@/app/components/Footer';
 
 import { FeatureCard, FeatureCardsSection } from '@/components/FeatureCards';
 import { DiscordButton } from '@/components/DiscordButton';
+
 import { WallOfLove } from '@/components/WallOfLove';
 
 const Icons = {

components/WallOfLove.tsx

@@ -1,6 +1,6 @@
 'use client';
 
-import { useRef, useState, useEffect, type ReactNode } from 'react';
+import { useRef, useState, useEffect } from 'react';
 import gsap from 'gsap';
 import { ScrollTrigger } from 'gsap/ScrollTrigger';
 import { motion } from 'framer-motion';

components/dashboard/Heatmap.tsx

@@ -128,15 +128,16 @@ export default function Heatmap({
                 height: (7 * (CELL + GAP) - GAP) * scale,
               }}
             >
-              <div className="flex" style={{ gap: GAP }}>
+              <div className="flex" role="grid" style={{ gap: GAP }}>
                 {weeks.map((week, wIndex) => (
-                  <div key={wIndex} className="flex flex-col" style={{ gap: GAP }}>
+                  <div key={wIndex} className="flex flex-col" role="row" style={{ gap: GAP }}>
                     {week.map((day, dIndex) => {
                       const originalIndex = wIndex * 7 + dIndex;
 
                       return (
                         <div
                           key={day.date}
+                          role="gridcell"
                           aria-label={`${getContributionLabel(
                             day.count
                           )} on ${formatTooltipDate(day.date)}`}

components/dashboard/StatsCard.tsx

@@ -65,7 +65,7 @@ export default function StatsCard({
           {showUTCDisclaimer && (
             <div className="mt-3 space-y-1">
               <p className="text-[11px] text-[#71717A] leading-relaxed">
-                ℹ Streaks are calculated in UTC and may differ from your local timezone.
+                ℹ Streaks are calculated in UTC and may differ from your local timezone.
               </p>
 
               {utcDate && <p className="text-[10px] text-[#52525B]">UTC Date: {utcDate}</p>}

components/dashboard/StatsCard.utc-disclaimer.test.tsx

@@ -0,0 +1,41 @@
+/* eslint-disable @typescript-eslint/no-explicit-any */
+import { describe, expect, it, vi } from 'vitest';
+import { render } from '@testing-library/react';
+import StatsCard from './StatsCard';
+
+vi.mock('framer-motion', () => ({
+  motion: {
+    div: ({ children, className, ...props }: any) => {
+      delete props.initial;
+      delete props.whileInView;
+      delete props.viewport;
+      delete props.whileHover;
+      delete props.transition;
+
+      return (
+        <div className={className} {...props}>
+          {children}
+        </div>
+      );
+    },
+  },
+}));
+
+describe('StatsCard UTC disclaimer', () => {
+  it('renders a clean UTC disclaimer prefix and preserves the UTC date text', () => {
+    const { container } = render(
+      <StatsCard
+        title="Current Streak"
+        value="12"
+        description="Consecutive contribution days"
+        icon="Flame"
+        showUTCDisclaimer
+        utcDate="2026-06-01"
+      />
+    );
+
+    expect(container.textContent).toContain('ℹ');
+    expect(container.textContent).not.toContain('\u00E2\u20AC\u2039');
+    expect(container.textContent).toContain('UTC Date: 2026-06-01');
+  });
+});