chore(docs): add PR ↔ bug report mapping #196
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 10 commits
January 3, 2026 10:59
- Fix Prisma connection pool exhaustion (BUG-001) * Implement singleton pattern in newsletter and user-stats APIs * Prevents memory leaks and connection failures under load - Fix XSS vulnerability in error modal (BUG-002) * Replace innerHTML with textContent to prevent script injection * Use DOM createElement for secure rendering - Fix JSON parse crashes (BUG-003) * Add try-catch blocks for localStorage parsing in job and resume pages * Graceful error handling with fallback values - Add API rate limiting (BUG-018) * Implement in-memory rate limiter * Contact API: 5 req/hour per IP * Newsletter API: 3 req/hour per IP * User Stats API: 60 req/min per user * Includes rate limit headers in responses
…nd consistent identifiers (BUG-021 + improvements)
…idance, and enhance CI workflow example
…et calc, time format); harden production failure behavior; add test cleanup
|
Someone is attempting to deploy a commit to the Darshan Rajput's projects Team on Vercel. A member of the Team first needs to authorize it. |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR is titled as a documentation-only change to add PR-to-bug-report mapping, but actually contains substantial code changes including rate limiting implementation, XSS vulnerability fixes, localStorage error handling improvements, Jest test setup, and multiple dependency additions. This is a significant mismatch between the PR description and the actual changes.
Key changes:
- Rate limiting infrastructure with in-memory and Upstash adapters
- XSS vulnerability fix in error handling modal
- localStorage error handling with try-catch blocks
- Jest testing framework setup with initial tests
Reviewed changes
Copilot reviewed 15 out of 16 changed files in this pull request and generated 11 comments.
Show a summary per file
| File | Description |
|---|---|
| bug-reports/PR_MAPPING.md | New documentation mapping bug reports to PR numbers (references multiple PRs: #180, #190, #193-195) |
| bug-reports/FEATURE-020-test-suite.md | Comprehensive test suite documentation and implementation guide |
| bug-reports/BUG-018-no-rate-limiting.md | Documentation of rate limiting vulnerability and proposed solutions |
| docs/RATE_LIMITING.md | Rate limiting implementation guide for in-memory and Upstash modes |
| package.json | Added Jest testing dependencies and rate limiting packages; updated test script |
| jest.config.js | New Jest configuration for TypeScript tests |
| lib/rate-limit.ts | New in-memory rate limiter with concurrency handling and IP detection |
| lib/rate-limit-upstash.ts | Upstash Redis adapter for production rate limiting |
| lib/error-handling.tsx | XSS fix replacing innerHTML with DOM element creation and textContent |
| app/api/contact/route.ts | Added rate limiting (5 requests/hour per IP) and Prisma singleton pattern |
| app/api/newsletter/route.ts | Added rate limiting (3 requests/hour per IP) and Prisma singleton pattern |
| app/api/user-stats/route.ts | Added rate limiting (60 requests/minute per user) and Prisma singleton pattern |
| app/job/page.tsx | Added try-catch for localStorage.getItem with corruption recovery |
| app/dashboard/resume/page.tsx | Added try-catch for localStorage.getItem with corruption recovery |
| tests/lib/rate-limit.test.ts | Basic tests for in-memory rate limiter |
| tests/lib/rate-limit-upstash.mock.test.ts | Mocked tests for Upstash adapter |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Owner
|
resolve the conflicts |
Contributor
Author
Owner
|
resolve the conflicts |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add to summarize which bug reports map to the PRs created in this audit and their current status. This makes it easier for maintainers to review and merge per-issue PRs.
This is a documentation-only PR.