fix: resolve multiple bugs in commitpulse

[saurabhhhcodes]

Description

This PR fixes real bugs found in the codebase:

• Added rejection handler to Promise.all: an unhandled rejection in any input promise previously crashed silently.
• Removed return undefined: bare return conveys the same intent without the redundant literal.
• Replaced global isNaN with Number.isNaN: the global version coerces its argument, so isNaN('1') returns false while Number.isNaN is strict.
• Added Number.EPSILON to Math.round: prevents floating-point drift (e.g. 1.005 * 100 rounding to 100 instead of 101).

Type of Change

• Bug fix (non-breaking change fixing an issue)

How Has This Been Tested?

• Local manual testing

Checklist

• My code follows the style guidelines
• I have performed a self-review

Related Issue

Ref: #8924

[vercel]

Someone 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 effectively resolves multiple bugs in the commitpulse codebase, including the addition of a rejection handler to Promise.all, replacing global isNaN with Number.isNaN, and adding Number.EPSILON to Math.round to prevent floating-point drift. These changes align with previous decisions made to enhance code reliability and prevent silent crashes. Thank you for your contributions!

[github-actions]

👋 Hey @saurabhhhcodes! Thanks for your contribution! 🎉

Unfortunately, this PR has been automatically closed because it is not linked to any open issue.

To resolve this, please do the following:

1. Link a valid open issue by editing your PR description to include a closing keyword (e.g., Fixes #<issue-number>).
2. Reopen this PR once the link is added.

💡 You can link multiple issues if needed (e.g. Fixes #12, Closes #34).
If you're working on something that doesn't have an issue yet, please open one first and then link it here.

We look forward to reviewing your PR once an issue is linked! 🚀