Feature/score bounds 1197#1275
Conversation
|
conflicting with main (it touches the initial-schema migration + scoresService, which have both moved), please rebase and i'll review the score-bounds logic and the requestLogger changes: one thing to reconsider while you're in there: this edits backend/migrations/1771691269865_initial-schema.js, the initial migration that's already been applied in every environment. schema changes should almost always be a NEW migration rather than an edit to an applied one (editing it won't re-run anywhere it's already applied). worth splitting the score-bounds schema change into its own new migration. if you want to keep contributing, join us on Telegram: https://t.me/+DOylgFv1jyJlNzM0 |
📝 Pull Request Summary [Wave 200pts]
Description
This PR addresses Issue #1197, hardening database integrity bounds by introducing a strict database
CHECKconstraint enforcing the standard 300–850 scoring index.Key Modifications
chk_score_rangeconstraint via raw migration commands to reject stray manual modifications or query bugs trying to commit anomalous numbers.Target Branch:
mainCloses #1197