fix: use shared sanitizeFont in generateMonthlySVG and generateAutoThemeMonthlySVG

[KaparthyReddy]

Fixes #905

Description

Both generateMonthlySVG and generateAutoThemeMonthlySVG in lib/svg/generator.ts defined their own local sanitizeFont function:

const sanitizeFont = (name: string) => name.replace(/[^a-zA-Z0-9\s-]/g, '').trim();

This shadowed the imported sanitizeFont from lib/svg/sanitizer.ts and stripped single quotes from font names. The real shared sanitizer explicitly allows single quotes (/[^a-zA-Z0-9\s\-']/g), meaning font names behaved differently in monthly view vs the default view — a silent inconsistency with no error thrown.

The fix removes both local definitions and replaces them with the already-imported shared sanitizeFont, making font sanitization consistent across all badge views.

Files changed: lib/svg/generator.ts only — 2 lines removed, 2 lines simplified.

Pillar

• 🛠️ Other (Bug fix, refactoring, docs)

Visual Preview

N/A — font sanitization logic fix, no visual change for valid font names.

Checklist before requesting a review:

• I have read the CONTRIBUTING.md file.
• I have tested these changes locally.
• I have run npm run format and npm run lint locally and resolved all errors.
• My commits follow the Conventional Commits format.
• I have updated README.md if I added a new theme or URL parameter.
• I have starred 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.
• (Recommended) I joined the CommitPulse Discord community.

[vercel]

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

A member of the Team first needs to authorize it.

[github-actions]

👋 Hey @KaparthyReddy, welcome to CommitPulse! 🎉

Thanks for opening your first pull request — this is a big deal and we appreciate the effort!

While you wait for a review, please double-check:

• ✅ You've read the CONTRIBUTING.md checklist
• ✅ npm run lint, npm run format, and npm run test all pass locally
• ✅ Your PR has a visual preview if it touches any SVG output
• 💬 You've joined our Discord for faster PR feedback

A maintainer will review your PR shortly. Hang tight! 🚀

[github-actions]

⚠️ Hey @KaparthyReddy, this PR has merge conflicts with the main branch.

Please pull the latest changes and resolve the conflicts so we can review it!

git fetch origin
git rebase origin/main
# resolve any conflicts, then:
git push --force-with-lease

Once resolved, the needs-rebase label will be removed automatically on the next check. 🙌

[github-actions]

🚨 Hey @KaparthyReddy, 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. 💪

[github-actions]

🎉 Congratulations @KaparthyReddy! Your PR has been successfully merged. 🚀

Thank you for contributing to CommitPulse. Your work helps us build a better tool for the community.

⚠️ Important for GSSoC Contributors:
You are strictly advised to join our Discord Server as it is mandatory for all GSSoC participants. All important announcements, point claims, and community discussions happen there.

Keep building! 💻✨