Merge branch 'main' into fix/dynamic-negative-delta-color

Author: Pranav-IIITM

.github/scripts/issue-management/claim-handler.js

@@ -74,7 +74,7 @@ async function handleClaim({ github, context }) {
       owner,
       repo,
       issue_number: issueNumber,
-      body: `❌ You already have **${existingIssues.length}/${MAX_ASSIGNED_ISSUES}** active assigned issues (the maximum allowed).\nPlease complete or unassign one of your current issues before claiming another.\n\n${issueList}`,
+      body: `❌ You already have **${existingIssues.length}/${MAX_ASSIGNED_ISSUES}** active assigned issues (the maximum allowed).\nPlease complete or \`/unclaim\` one of your current issues before claiming another.\n\n${issueList}`,
     });
     return;
   }

.github/scripts/issue-management/main.js

@@ -4,6 +4,7 @@ const { handleAssign } = require('./assign-handler');
 const { handleUnassign } = require('./unassign-handler');
 const { handleAddLabel } = require('./addlabel-handler');
 const { handleClaim } = require('./claim-handler');
+const { handleUnclaim } = require('./unclaim-handler');
 
 module.exports = async ({ github, context, core }) => {
   const commentBody = context.payload.comment?.body;
@@ -48,6 +49,9 @@ module.exports = async ({ github, context, core }) => {
       case 'claim':
         await handleClaim({ github, context });
         break;
+      case 'unclaim':
+        await handleUnclaim({ github, context });
+        break;
       case 'ping':
         await github.rest.issues.createComment({
           owner,

.github/scripts/issue-management/parse-command.js

@@ -15,6 +15,9 @@ function parseCommand(commentBody) {
     const claimMatch = line.match(/^\/claim\s*$/i);
     if (claimMatch) return { command: 'claim' };
 
+    const unclaimMatch = line.match(/^\/unclaim\s*$/i);
+    if (unclaimMatch) return { command: 'unclaim' };
+
     const pingMatch = line.match(/^\/ping\s*$/i);
     if (pingMatch) return { command: 'ping' };
 

.github/scripts/issue-management/unclaim-handler.js

@@ -0,0 +1,44 @@
+async function handleUnclaim({ github, context }) {
+  const { owner, repo } = context.repo;
+  const issueNumber = context.payload.issue.number;
+  const commenter = context.payload.comment.user.login;
+  const issueState = context.payload.issue.state;
+
+  if (issueState === 'closed') {
+    await github.rest.issues.createComment({
+      owner,
+      repo,
+      issue_number: issueNumber,
+      body: `❌ Commands cannot be used on closed issues.`,
+    });
+    return;
+  }
+
+  const currentAssignees = context.payload.issue.assignees.map((a) => a.login.toLowerCase());
+
+  if (!currentAssignees.includes(commenter.toLowerCase())) {
+    await github.rest.issues.createComment({
+      owner,
+      repo,
+      issue_number: issueNumber,
+      body: `ℹ️ @${commenter}, you are not currently assigned to this issue, so there's nothing to unclaim.`,
+    });
+    return;
+  }
+
+  await github.rest.issues.removeAssignees({
+    owner,
+    repo,
+    issue_number: issueNumber,
+    assignees: [commenter],
+  });
+
+  await github.rest.issues.createComment({
+    owner,
+    repo,
+    issue_number: issueNumber,
+    body: `✅ Successfully unclaimed this issue for @${commenter}.\n\n> 🔓 The issue is now open for others to claim.`,
+  });
+}
+
+module.exports = { handleUnclaim };

.github/workflows/issue-management.yml

@@ -22,7 +22,8 @@ jobs:
         contains(github.event.comment.body, '/assign') ||
         contains(github.event.comment.body, '/unassign') ||
         contains(github.event.comment.body, '/addlabel') ||
-        contains(github.event.comment.body, '/claim')
+        contains(github.event.comment.body, '/claim') ||
+        contains(github.event.comment.body, '/unclaim')
       )
     steps:
       - name: Checkout Repository

CONTRIBUTING.md

@@ -338,6 +338,7 @@ Our automation runs entirely through issue comments. Here is how you interact wi
 | Command                       | Who Can Use It?                                         | What It Does                                              |
 | ----------------------------- | ------------------------------------------------------- | --------------------------------------------------------- |
 | `/claim`                      | **Issue Author (or Anyone if authored by jhasourav07)** | Self-assigns the issue to you.                            |
+| `/unclaim`                    | **Assigned Contributor**                                | Removes the assignment from yourself (opens it back up).  |
 | `/addlabel <label1> <label2>` | **Anyone**                                              | Adds labels to the issue (e.g. `/addlabel frontend bug`). |
 | `/unassign @username`         | **Maintainers Only**                                    | Removes the assignee from an issue.                       |
 | `/assign @username`           | **Maintainers Only**                                    | Manually assigns someone to an issue.                     |
@@ -363,7 +364,7 @@ To keep the project moving, assignments are not permanent.
 If the bot rejects your command, check these common scenarios:
 
 - **"Commands cannot be used on closed issues"**: You cannot claim, assign, or unassign on closed issues. Find an open one!
-- **"You already have X/3 active assigned issues"**: You have reached the maximum of 3 concurrent assignments. Finish one of your current tasks before claiming a new issue. If you're stuck, ask a maintainer to `/unassign` you from one.
+- **"You already have X/3 active assigned issues"**: You have reached the maximum of 3 concurrent assignments. Finish one of your current tasks before claiming a new issue. If you're stuck, use the `/unclaim` command to unassign yourself from an issue, or ask a maintainer to `/unassign` you.
 - **"This issue is already assigned to @username"**: Be faster next time! Look for issues without assignees.
 - **"Only the author of this issue can claim it"**: You tried to `/claim` an issue you did not create. You can only claim issues that you authored (unless the issue was authored by `jhasourav07`, which anyone can claim).
 - **"The following label(s) do not exist"**: You can only add existing repo labels. The bot will reply with a list of valid labels you can use.

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

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

app/api/streak/route.ts

@@ -96,6 +96,7 @@ export async function GET(request: Request) {
       versus,
       shading,
       gradient,
+      disable_particles,
     } = parseResult.data;
 
     const themeName = theme || 'dark';
@@ -169,6 +170,7 @@ export async function GET(request: Request) {
       versus,
       shading,
       gradient,
+      disable_particles,
     };
 
     let calendar;

app/api/track-user/route.test.ts

@@ -4,6 +4,11 @@ import { User } from '@/models/User';
 import dbConnect from '@/lib/mongodb';
 
 // Mock dependencies
+vi.mock('@/lib/rate-limit', () => ({
+  trackUserRateLimiter: {
+    check: vi.fn().mockResolvedValue(true),
+  },
+}));
 vi.mock('@/lib/mongodb', () => ({
   default: vi.fn(),
 }));
@@ -23,30 +28,13 @@ function makeRequest(body: Record<string, unknown>): Request {
 }
 
 describe('POST /api/track-user', () => {
-  let originalNodeEnv: string | undefined;
-  let originalMongoUri: string | undefined;
-
   beforeEach(() => {
-    originalNodeEnv = process.env.NODE_ENV;
-    originalMongoUri = process.env.MONGODB_URI;
     vi.clearAllMocks();
   });
 
   afterEach(() => {
-    vi.restoreAllMocks();
-
-    // Restore environment variables
-    if (originalNodeEnv === undefined) {
-      Reflect.deleteProperty(process.env, 'NODE_ENV');
-    } else {
-      Reflect.set(process.env, 'NODE_ENV', originalNodeEnv);
-    }
-
-    if (originalMongoUri === undefined) {
-      delete process.env.MONGODB_URI;
-    } else {
-      process.env.MONGODB_URI = originalMongoUri;
-    }
+    // Clean up environment variables
+    delete process.env.MONGODB_URI;
   });
 
   describe('Validation', () => {
@@ -65,17 +53,6 @@ describe('POST /api/track-user', () => {
       expect(data.success).toBe(false);
       expect(data.error).toBe('Malformed JSON request body');
     });
-    it('returns 400 when body is plain text (not JSON)', async () => {
-      const req = new Request('http://localhost/api/track-user', {
-        method: 'POST',
-        headers: { 'Content-Type': 'text/plain' },
-        body: 'not json',
-      });
-      const response = await POST(req);
-      expect(response.status).toBe(400);
-      const data = await response.json();
-      expect(data.success).toBe(false);
-    });
 
     it('returns 400 when username is missing', async () => {
       const response = await POST(makeRequest({}));
@@ -111,32 +88,8 @@ describe('POST /api/track-user', () => {
         'MONGODB_URI is not set. Bypassing user tracking for local development.'
       );
       expect(dbConnect).not.toHaveBeenCalled();
-    });
-  });
-
-  describe('Without MONGODB_URI (Production Environment)', () => {
-    it('returns 500 error when MONGODB_URI is missing in production', async () => {
-      (process.env as Record<string, string | undefined>).NODE_ENV = 'production';
-      delete process.env.MONGODB_URI;
-
-      // Spy on console.error
-      const consoleErrorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
 
-      const response = await POST(makeRequest({ username: 'octocat' }));
-
-      expect(response.status).toBe(500);
-      const data = await response.json();
-      expect(data.success).toBe(false);
-      expect(data.error).toBe('Database configuration error');
-      expect(data.bypassed).toBeUndefined();
-
-      // Verify critical error was logged for monitoring/alerting
-      expect(consoleErrorSpy).toHaveBeenCalledWith(
-        'CRITICAL: MONGODB_URI is not set in production environment. User tracking is disabled.'
-      );
-
-      // Verify database connection was never attempted
-      expect(dbConnect).not.toHaveBeenCalled();
+      consoleSpy.mockRestore();
     });
   });
 
@@ -153,7 +106,11 @@ describe('POST /api/track-user', () => {
       // Trims and lowercases
       expect(User.updateOne).toHaveBeenCalledWith(
         { username: 'octocat' },
-        { $setOnInsert: { username: 'octocat' } },
+        {
+          $setOnInsert: { username: 'octocat' },
+          $set: { lastSeen: expect.any(Date) },
+          $inc: { visitCount: 1 },
+        },
         { upsert: true }
       );
 
@@ -163,26 +120,8 @@ describe('POST /api/track-user', () => {
       expect(data.bypassed).toBeUndefined();
     });
 
-    it('normalizes purely uppercase usernames to lowercase', async () => {
-      const response = await POST(makeRequest({ username: 'GITHUB' }));
-
-      expect(dbConnect).toHaveBeenCalled();
-
-      expect(User.updateOne).toHaveBeenCalledWith(
-        { username: 'github' },
-        { $setOnInsert: { username: 'github' } },
-        { upsert: true }
-      );
-
-      expect(response.status).toBe(200);
-
-      const data = await response.json();
-
-      expect(data.success).toBe(true);
-    });
-
     it('returns 500 when database connection fails', async () => {
-      vi.spyOn(console, 'error').mockImplementation(() => {});
+      const consoleErrorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
       vi.mocked(dbConnect).mockRejectedValueOnce(new Error('DB Down'));
 
       const response = await POST(makeRequest({ username: 'octocat' }));
@@ -191,47 +130,8 @@ describe('POST /api/track-user', () => {
       const data = await response.json();
       expect(data.success).toBe(false);
       expect(data.error).toBe('Internal server error');
-    });
-
-    it('gracefully handles concurrent duplicate key (code 11000) race conditions', async () => {
-      const consoleErrorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
-
-      const mongoError = new Error('E11000 duplicate key error collection: username') as Error & {
-        code?: number;
-        keyPattern?: Record<string, number>;
-      };
-      mongoError.code = 11000;
-      mongoError.keyPattern = { username: 1 };
-      vi.mocked(User.updateOne).mockRejectedValueOnce(mongoError);
-
-      const response = await POST(makeRequest({ username: 'octocat' }));
-
-      expect(response.status).toBe(200);
-      const data = await response.json();
-      expect(data.success).toBe(true);
-      expect(consoleErrorSpy).not.toHaveBeenCalled();
-    });
-
-    it('rethrows duplicate key (code 11000) error if it is not related to username', async () => {
-      const consoleErrorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
 
-      const mongoError = new Error(
-        'E11000 duplicate key error collection: other_field'
-      ) as Error & {
-        code?: number;
-        keyPattern?: Record<string, number>;
-      };
-      mongoError.code = 11000;
-      mongoError.keyPattern = { other_field: 1 };
-      vi.mocked(User.updateOne).mockRejectedValueOnce(mongoError);
-
-      const response = await POST(makeRequest({ username: 'octocat' }));
-
-      expect(response.status).toBe(500);
-      const data = await response.json();
-      expect(data.success).toBe(false);
-      expect(data.error).toBe('Internal server error');
-      expect(consoleErrorSpy).toHaveBeenCalled();
+      consoleErrorSpy.mockRestore();
     });
   });
 });

app/api/track-user/route.ts

@@ -62,7 +62,11 @@ export async function POST(req: Request) {
       // Upsert the user: create if doesn't exist, do nothing if exists
       await User.updateOne(
         { username: trimmedUsername },
-        { $setOnInsert: { username: trimmedUsername } },
+        {
+          $setOnInsert: { username: trimmedUsername },
+          $set: { lastSeen: new Date() },
+          $inc: { visitCount: 1 },
+        },
         { upsert: true }
       );
     } catch (upsertError) {