fix(adapters): clamp reputation rates with coerceFraction (#91)

[rudra496]

Summary

Closes #91

This PR replaces the inline unclamped division (coerceDecimal(...) / 100) for completionRate and onTimeDeliveryRate in adaptReputation with coerceFraction, and extends coerceFraction to support an optional divisor parameter (defaulting to 1) and clamped fallback values.

Changes

1. src/lib/utils.ts:
   • Extended coerceFraction(value, fallback = 0, divisor = 1) to accept an optional divisor parameter, ensuring raw percentage values (e.g. "94", "150", "-20") are safely divided by 100 and strictly bounded to [0, 1].
   • Added JSDoc documentation detailing range guarantees and parameter conventions.
2. src/lib/adapters.ts:
   • Routed completionRate and onTimeDeliveryRate in adaptReputation through coerceFraction(snapshot.completionRate, 0, 100) and coerceFraction(snapshot.onTimeDeliveryPercentage, 0, 100).
   • Audited all adjacent adapter functions (adaptBounty, adaptMilestone, adaptMaintenancePool): confirmed that adaptBounty uses coercePercentage (clamped [0, 100]) and coerceNonNegative, and adaptMilestone/adaptMaintenancePool use coerceNonNegative. No other unclamped fractions/percentages exist.
3. Tests:
   • Added unit tests in src/lib/utils.test.ts for coerceFraction divisor support, upper/lower clamping, and non-numeric fallbacks.
   • Added unit tests in src/lib/adapters.test.ts verifying out-of-bounds inputs ("150" -> 1.0, "-20" -> 0.0), in-range inputs ("94" -> 0.94, "0" -> 0.0, "100" -> 1.0), corrupted strings (-> 0.0), and null snapshot behavior.

Verification

• npm test: 3/3 test suites passed (47/47 tests).
• npx tsc --noEmit: 0 type errors.
• npm run build: Turbopack production build succeeded cleanly.

[vercel]

@rudra496 is attempting to deploy a commit to the chonilius' projects Team on Vercel.

A member of the Team first needs to authorize it.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.