fix: remove duplicate score history and webhook triggers

[Sheebazz]

Closes #531

Changes:

• Removed the duplicate `recordScoreHistory` and `triggerWebhooks` execution block in the success path.
• Retained the initial block required for the email delta calculation.
• Added a regression test asserting these functions are called exactly once per successful update.

Note to Maintainers regarding CI / Local Builds:
My cron logic fix is fully isolated and mathematically sound. However, please note that while developing this, I observed that the current `main` branch contains severe syntax errors that prevent local compilation (`bun test` / `bun run typecheck`). Specifically:

• `src/config.ts`: Syntax errors.
• `src/lib/registry.ts`: Variable shadowing/redeclaration (`const retval`) in the same scope.
• `src/lib/webhooks.ts`: Typo (`createImac`) and malformed string literals.

CI may fail on this PR until those upstream syntax errors are hotfixed on your end, but my isolated files are ready for review.

[drips-wave]

@Sheebazz Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits