test(ProfileOptimizerModal-timezone-boundaries): verify Timezone Norm…

[ShafinNigamana]

Description

Adds the timezone boundaries and calendar data alignment test suite for ProfileOptimizerModal.

The new test suite verifies:

• Stable initialization and rendering across standard timezone offsets (UTC, EST, IST, JST).
• Correct alignment of commit contributions on correct visual calendar dates independent of timezone.
• Parsing of leap-year boundaries correctly without gaps.
• Date format utility outputs matching target locales (en-US, en-IN, ja-JP).
• Components' stability around daylight-saving time transitions (Spring Forward & Fall Back) without shifting date boundaries.

Fixes #2690

Pillar

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

Visual Preview

Not applicable — test-only change.

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., test(ProfileOptimizerModal-timezone-boundaries): ...).
• 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.

[ShafinNigamana]

Hey @Aamod007, I've re-submitted the implementation for the timezone boundaries test suite. Since the previous closed PR for this issue (#3418) was tagged as level:advanced due to the complexity of mocking the timezone offsets, leap years, and DST transitions, I think this one fits best under the level:advanced tag as well. Thank you!

[Aamod-Dev]

I appreciate you taking the time to work on this PR. The direction is good, however, I noticed a couple of things that need tweaking before we can merge it. Could you address the following points?

• The current implementation does not handle null or undefined values strictly in some data manipulation methods, which may lead to runtime issues in edge cases. Adding optional chaining or explicit null checks would make it more robust.

Take your time with the updates, and just drop a comment when it's ready for another look.

[Aamod-Dev]

Thanks for the PR. I've reviewed the changes and they look good to me.

No concerns from my end. Approved.

[github-actions]

🎉 Congratulations @ShafinNigamana! 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! 💻✨

[Aamod-Dev]

Review Summary
Removed extra type labels ( ype:performance) to strictly meet the rule of exactly one type label ( ype:testing retained).