Optimize Bot Filter: Cache Ignored Authors to Enhance Performance

[tamilr0727-ux]

Description

Optimizes getIgnoredAuthors() in lib/bot-filter.ts by eliminating repeated synchronous filesystem operations when checking bot authors across large commit datasets.

Changes

1. Prevented .commitpulse.json from being read from disk on every isBotAuthor() invocation.
2. Added caching/memoization for ignored-author configuration.
3. Ensured configuration is loaded once and reused across repeated bot-author checks.
4. Avoided repeated fs.existsSync() and fs.readFileSync() calls inside array operations such as .filter().
5. Preserved existing bot detection and ignored-author behavior.
6. Added regression and performance-focused tests for repeated author lookups.

Verification

✅ vitest run passes successfully.
✅ Repeated isBotAuthor() calls no longer trigger unnecessary synchronous disk reads.
✅ Existing bot-filtering behavior remains unchanged.
✅ Large commit datasets can be processed more efficiently.

Fixes #8982

Pillar

• 🎨 Pillar 1 — New Theme Design
• 📐 Pillar 2 — Geometric SVG Improvement
• 🕐 Pillar 3 — Timezone Logic Optimization
• 🛠️ Other (Bug fix, refactoring, docs)

Visual Preview

Can Check in the Files Changed Section.

Checklist before requesting a review:

• I have read the CONTRIBUTING.md file.
• I have tested these changes locally (localhost:3000/api/streak?user=YOUR_USERNAME).
• I have run npm run format and npm run lint locally and resolved all errors (CI will fail otherwise).
• My commits follow the Conventional Commits format (e.g., feat(themes): ..., fix(calculate): ...).
• I have updated README.md if I added a new theme or URL parameter.
• I have started the repo.
• I have made sure that i have only one commit to merge in this PR.
• The SVG output matches the CommitPulse "premium quality" aesthetic standard (no raw elements, smooth animations, correct fonts).
• (Recommended) I joined the CommitPulse Discord community for contributor discussions, mentorship, and faster PR support.

[vercel]

@tamilr0727-ux is attempting to deploy a commit to the jhasourav07's projects Team on Vercel.

A member of the Team first needs to authorize it.

[retenta-bot]

This pull request introduces significant optimizations to the bot author detection mechanism by caching ignored authors, which should enhance performance, especially with large datasets. This aligns with our previous decisions to optimize performance in similar contexts, such as the CI analytics service for large repositories. It's great to see these improvements being implemented! Looking forward to your review. 😊

[github-actions]

🚨 Hey @tamilr0727-ux, the CI Pipeline is failing on this PR and it has been marked as status:blocked.

Please fix the issues before this can be reviewed. Here's how:

1. Run checks locally before pushing:

npm run format:check   # Check Prettier formatting
npm run lint           # Run ESLint
npm run typecheck      # TypeScript type check
npm run test           # Run unit tests (Vitest)
npm run build          # Verify production build passes

2. Auto-fix common issues:

npm run format         # Auto-fix formatting with Prettier
npm run lint -- --fix  # Auto-fix lint errors where possible

3. Check the full failure log here:
👉 View CI Run

Once you push a fix and the CI passes, the status:blocked label will be removed automatically. 💪