Merge branch 'main' into support-cache-busting-querys

# Conflicts:
#	middleware.ts

Author: Vartika-l7

.github/scripts/issue-management/stale-assignment.js

@@ -29,6 +29,7 @@ async function handleStaleAssignments({ github, context, core }) {
 
       if (timeSinceUpdate > TWO_DAYS_MS) {
         const currentAssignees = issue.assignees.map((a) => a.login);
+        if (currentAssignees.length === 0) continue;
 
         // Check if any open PRs reference this issue before unassigning
         const { data: searchResult } = await github.rest.search.issuesAndPullRequests({
@@ -46,24 +47,22 @@ async function handleStaleAssignments({ github, context, core }) {
           `Issue #${issue.number} has been inactive since ${issue.updated_at}. Removing assignees.`
         );
 
-        if (currentAssignees.length > 0) {
-          await github.rest.issues.removeAssignees({
-            owner,
-            repo,
-            issue_number: issue.number,
-            assignees: currentAssignees,
-          });
-
-          // 2. Post a comment
-          await github.rest.issues.createComment({
-            owner,
-            repo,
-            issue_number: issue.number,
-            body: `⚠️ Assignment automatically removed due to inactivity.\nFeel free to reclaim the issue if you want to continue working on it.`,
-          });
-
-          staleCount++;
-        }
+        await github.rest.issues.removeAssignees({
+          owner,
+          repo,
+          issue_number: issue.number,
+          assignees: currentAssignees,
+        });
+
+        // 2. Post a comment
+        await github.rest.issues.createComment({
+          owner,
+          repo,
+          issue_number: issue.number,
+          body: `⚠️ Assignment automatically removed due to inactivity.\nFeel free to reclaim the issue if you want to continue working on it.`,
+        });
+
+        staleCount++;
       }
     }
 

.prettierignore

@@ -0,0 +1 @@
+package-lock.json

README.md

@@ -111,6 +111,12 @@ Transform your GitHub contribution history into a cinematic 3D monolith.
 ![CommitPulse](https://commitpulse.vercel.app/api/streak?user=jhasourav07&bg=0a0a0a&accent=ff6b35&text=ffffff)
 ```
 
+#### 📅 Monthly Summary
+
+```md
+![CommitPulse Monthly](https://commitpulse.vercel.app/api/streak?user=octocat&view=monthly)
+```
+
 ---
 
 ## 📚 Documentation Index

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

@@ -0,0 +1,9 @@
+import DashboardSkeleton from '@/components/dashboard/DashboardSkeleton';
+
+export default function DashboardUserLoading() {
+  return (
+    <div className="p-4 md:p-6 lg:p-8 min-h-screen bg-black text-white">
+      <DashboardSkeleton />
+    </div>
+  );
+}

app/(root)/dashboard/error.test.tsx

@@ -3,31 +3,34 @@ import { render, screen, fireEvent } from '@testing-library/react';
 import { describe, it, expect, vi } from 'vitest';
 import ErrorPage from './error';
 
-import type { ReactNode } from 'react';
-
 vi.mock('next/link', () => ({
-  default: ({ children, href }: { children: ReactNode; href: string }) => (
-    <a href={href}>{children}</a>
+  default: ({
+    children,
+    ...props
+  }: React.AnchorHTMLAttributes<HTMLAnchorElement> & { children: React.ReactNode }) => (
+    <a {...props}>{children}</a>
   ),
 }));
 
 describe('Dashboard Error Page', () => {
-  it('renders API limit UI', () => {
+  it('renders the API limit emoji for API limit reached errors', () => {
     render(<ErrorPage error={new Error('API limit reached')} reset={vi.fn()} />);
 
     expect(screen.getByRole('heading', { name: 'API Limit Reached' })).toBeInTheDocument();
+    expect(screen.getByText('⏳')).toBeInTheDocument();
   });
 
-  it('renders not found UI', () => {
-    render(<ErrorPage error={new Error('not found')} reset={vi.fn()} />);
+  it('renders the not found emoji for User not found errors', () => {
+    render(<ErrorPage error={new Error('User not found')} reset={vi.fn()} />);
 
     expect(screen.getByText(/not found/i)).toBeInTheDocument();
+    expect(screen.getByText('🕵️‍♂️')).toBeInTheDocument();
   });
 
-  it('renders generic error UI', () => {
-    render(<ErrorPage error={new Error('something went wrong')} reset={vi.fn()} />);
+  it('renders the generic error emoji for other errors', () => {
+    render(<ErrorPage error={new Error('Something went wrong')} reset={vi.fn()} />);
 
-    expect(screen.getByRole('button', { name: /try again/i })).toBeInTheDocument();
+    expect(screen.getByText('⚠️')).toBeInTheDocument();
   });
 
   it('shows Try again button', () => {

app/(root)/dashboard/loading.test.tsx

@@ -20,10 +20,10 @@ describe('DashboardLoading', () => {
     expect(container).toBeTruthy();
   });
 
-  it('renders 2 StatsCardSkeleton components', () => {
+  it('renders 3 StatsCardSkeleton components', () => {
     render(<DashboardLoading />);
     const skeletons = screen.getAllByTestId('stats-card-skeleton');
-    expect(skeletons).toHaveLength(2);
+    expect(skeletons).toHaveLength(3);
   });
 
   it('renders shimmer skeleton elements in the left sidebar', () => {

app/(root)/dashboard/loading.tsx

@@ -1,49 +1,9 @@
-import StatsCardSkeleton from '@/components/dashboard/StatsCardSkeleton';
-import AchievementsSkeleton from '@/components/dashboard/AchievementsSkeleton';
-import AIInsightsSkeleton from '@/components/dashboard/AIInsightsSkeleton';
+import DashboardSkeleton from '@/components/dashboard/DashboardSkeleton';
 
 export default function DashboardLoading() {
   return (
     <div className="p-4 md:p-6 lg:p-8 min-h-screen bg-black text-white">
-      <div className="grid grid-cols-1 lg:grid-cols-[300px_1fr_320px] gap-6 lg:gap-8">
-        {/* Left Sidebar Skeleton */}
-        <div className="flex flex-col gap-6">
-          {/* Profile Card Skeleton Placeholder */}
-          <div className="h-100 rounded-2xl shimmer border border-white/10" />
-
-          {/* Achievements Skeleton */}
-          <div className="p-6 rounded-xl bg-[#0a0a0a] border border-[rgba(255,255,255,0.08)]">
-            <div className="flex items-center gap-2.5 mb-5">
-              <div className="w-4 h-4 shimmer rounded" />
-              <div className="w-24 h-4 shimmer rounded" />
-            </div>
-            <AchievementsSkeleton />
-          </div>
-        </div>
-
-        {/* Main Content Skeleton */}
-        <div className="flex flex-col gap-6 lg:gap-8">
-          {/* Hero/Profile section */}
-          <div className="h-64 rounded-2xl shimmer border border-white/10" />
-
-          {/* Stats Cards Grid - 2 cards */}
-          <div className="grid grid-cols-1 md:grid-cols-2 gap-6">
-            <StatsCardSkeleton />
-            <StatsCardSkeleton />
-          </div>
-
-          {/* Bottom section */}
-          <div className="h-48 rounded-2xl shimmer border border-white/10" />
-        </div>
-
-        {/* Right Sidebar Skeleton */}
-        <div className="flex flex-col gap-6">
-          <div className="h-24 rounded-2xl shimmer border border-white/10" />
-          <div className="h-24 rounded-2xl shimmer border border-white/10" />
-          <div className="h-24 rounded-2xl shimmer border border-white/10" />
-          <AIInsightsSkeleton />
-        </div>
-      </div>
+      <DashboardSkeleton />
     </div>
   );
 }

app/api/notify/route.test.ts

@@ -12,8 +12,19 @@ vi.mock('@/models/Notification', () => ({
   },
 }));
 vi.mock('@/lib/rate-limit', () => ({
+  getRateLimitHeaders: vi.fn((result) => ({
+    'X-RateLimit-Limit': result.limit.toString(),
+    'X-RateLimit-Remaining': result.remaining.toString(),
+    'X-RateLimit-Reset': result.reset.toString(),
+  })),
   notifyRateLimiter: {
     check: vi.fn().mockResolvedValue(true),
+    checkWithResult: vi.fn().mockResolvedValue({
+      success: true,
+      limit: 5,
+      remaining: 4,
+      reset: Date.now() + 60000,
+    }),
   },
 }));
 vi.mock('@/services/github/validate-user', () => ({
@@ -102,9 +113,18 @@ describe('POST /api/notify', () => {
   // ── Rate limiting ────────────────────────────────────────────────────────
 
   it('returns 429 when rate limited', async () => {
-    vi.mocked(notifyRateLimiter.check).mockResolvedValue(false);
+    const reset = Date.now() + 60000;
+    vi.mocked(notifyRateLimiter.checkWithResult).mockResolvedValueOnce({
+      success: false,
+      limit: 5,
+      remaining: 0,
+      reset,
+    });
     const res = await POST(makeRequest('POST', { username: 'testuser', email: 'a@b.com' }));
     expect(res.status).toBe(429);
+    expect(res.headers.get('x-ratelimit-limit')).toBe('5');
+    expect(res.headers.get('x-ratelimit-remaining')).toBe('0');
+    expect(res.headers.get('x-ratelimit-reset')).toBe(reset.toString());
   });
 
   // ── Per-username write cooldown ───────────────────────────────────────────
@@ -231,9 +251,18 @@ describe('GET /api/notify', () => {
   // ── Rate limiting ────────────────────────────────────────────────────────
 
   it('returns 429 when rate limited', async () => {
-    vi.mocked(notifyRateLimiter.check).mockResolvedValue(false);
+    const reset = Date.now() + 60000;
+    vi.mocked(notifyRateLimiter.checkWithResult).mockResolvedValueOnce({
+      success: false,
+      limit: 5,
+      remaining: 0,
+      reset,
+    });
     const res = await GET(makeRequest('GET', undefined, 'user=testuser'));
     expect(res.status).toBe(429);
+    expect(res.headers.get('x-ratelimit-limit')).toBe('5');
+    expect(res.headers.get('x-ratelimit-remaining')).toBe('0');
+    expect(res.headers.get('x-ratelimit-reset')).toBe(reset.toString());
   });
 
   // ── MONGODB_URI handling ──────────────────────────────────────────────────

app/api/notify/route.ts

@@ -1,11 +1,11 @@
-import { NextResponse } from 'next/server';
+import { NextRequest, NextResponse } from 'next/server';
 import dbConnect from '@/lib/mongodb';
 import { Notification } from '@/models/Notification';
 import { notifyPostSchema, notifyGetSchema } from '@/lib/validations';
-import { notifyRateLimiter } from '@/lib/rate-limit';
 import { getClientIp } from '@/utils/getClientIp';
 import { DistributedCache } from '@/lib/cache';
 import { gitHubUserValidator } from '@/services/github/validate-user';
+import { getRateLimitHeaders, notifyRateLimiter } from '@/lib/rate-limit';
 
 const notifyWriteCache = new DistributedCache<number>(5000, 60000);
 const NOTIFY_WRITE_COOLDOWN_MS = 5 * 60 * 1000;
@@ -44,11 +44,12 @@ export async function POST(req: Request) {
   // fallback ensures rate limit is ALWAYS applied
   const rateLimitKey =
     ip && ip !== 'unknown' ? ip : `unknown:${req.headers.get('user-agent') ?? 'no-agent'}`;
+  const rateLimitResult = await notifyRateLimiter.checkWithResult(rateLimitKey);
 
-  if (!(await notifyRateLimiter.check(rateLimitKey))) {
+  if (!rateLimitResult.success) {
     return NextResponse.json(
       { success: false, message: 'Too many requests, please try again later.' },
-      { status: 429 }
+      { status: 429, headers: getRateLimitHeaders(rateLimitResult) }
     );
   }
 
@@ -172,19 +173,102 @@ export async function POST(req: Request) {
   }
 }
 
-// ─── GET /api/notify ─────────────────────────────────────────────────────────
-// Fetch notification preferences for a user
-export async function GET(req: Request) {
+// ─── DELETE /api/notify ──────────────────────────────────────────────────────
+// Remove notification preferences for a user (unsubscribe / right to erasure)
+export async function DELETE(req: NextRequest) {
   // Rate limiting
   const ip = getClientIp(req);
 
-  if (ip !== 'unknown' && !(await notifyRateLimiter.check(ip))) {
+  const rateLimitKey =
+    ip && ip !== 'unknown' ? ip : `unknown:${req.headers.get('user-agent') ?? 'no-agent'}`;
+
+  if (!(await notifyRateLimiter.check(rateLimitKey))) {
     return NextResponse.json(
       { success: false, message: 'Too many requests, please try again later.' },
       { status: 429 }
     );
   }
 
+  // Validate query params with Zod (reuse notifyGetSchema — expects ?user=)
+  const { searchParams } = new URL(req.url);
+  const parsed = notifyGetSchema.safeParse({
+    user: searchParams.get('user') ?? undefined,
+  });
+
+  if (!parsed.success) {
+    const fieldErrors = parsed.error.flatten();
+    const firstError =
+      Object.values(fieldErrors.fieldErrors).flat()[0] ??
+      fieldErrors.formErrors[0] ??
+      'Invalid request parameters.';
+    return NextResponse.json({ success: false, message: firstError }, { status: 400 });
+  }
+
+  const { user: username } = parsed.data;
+
+  try {
+    // Graceful MONGODB_URI handling
+    if (!process.env.MONGODB_URI) {
+      if (process.env.NODE_ENV === 'production') {
+        console.error(
+          'CRITICAL: MONGODB_URI is not set in production environment. Notification deletion is disabled.'
+        );
+        return NextResponse.json(
+          { success: false, message: 'Database configuration error.' },
+          { status: 500 }
+        );
+      }
+
+      console.warn(
+        'MONGODB_URI is not set. Bypassing notification deletion for local development.'
+      );
+      return NextResponse.json({
+        success: true,
+        message: 'Notification deletion bypassed (no database configured).',
+      });
+    }
+
+    await dbConnect();
+
+    const result = await Notification.deleteOne({ username: username.toLowerCase() });
+
+    if (result.deletedCount === 0) {
+      return NextResponse.json(
+        { success: false, message: 'No notification preferences found for this user.' },
+        { status: 404 }
+      );
+    }
+
+    return NextResponse.json(
+      { success: true, message: 'Notification preferences deleted successfully.' },
+      { status: 200 }
+    );
+  } catch (error) {
+    console.error('[/api/notify] Error deleting notification preferences:', error);
+    return NextResponse.json(
+      { success: false, message: 'Internal server error.' },
+      { status: 500 }
+    );
+  }
+}
+
+// ─── GET /api/notify ─────────────────────────────────────────────────────────
+// Fetch notification preferences for a user
+export async function GET(req: Request) {
+  // Rate limiting
+  const ip = getClientIp(req);
+
+  if (ip !== 'unknown') {
+    const rateLimitResult = await notifyRateLimiter.checkWithResult(ip);
+
+    if (!rateLimitResult.success) {
+      return NextResponse.json(
+        { success: false, message: 'Too many requests, please try again later.' },
+        { status: 429, headers: getRateLimitHeaders(rateLimitResult) }
+      );
+    }
+  }
+
   // Validate query params with Zod
   const { searchParams } = new URL(req.url);
   const parsed = notifyGetSchema.safeParse({