Skip to content

Feature/score bounds 1197#1275

Open
warisu wants to merge 3 commits into
LabsCrypt:mainfrom
warisu:feature/score-bounds-1197
Open

Feature/score bounds 1197#1275
warisu wants to merge 3 commits into
LabsCrypt:mainfrom
warisu:feature/score-bounds-1197

Conversation

@warisu

@warisu warisu commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

📝 Pull Request Summary [Wave 200pts]

Description

This PR addresses Issue #1197, hardening database integrity bounds by introducing a strict database CHECK constraint enforcing the standard 300–850 scoring index.

Key Modifications

  • Idempotent Data Backfill: Added a transactional preprocessing batch script in the migration path to clean and clamp legacy database entries ahead of time.
  • Database Contraband Guards: Attached the chk_score_range constraint via raw migration commands to reject stray manual modifications or query bugs trying to commit anomalous numbers.
  • Service Boundary Sync: Confirmed application query scripts map to the structural constraint cleanly without disrupting operational code execution.

Target Branch: main
Closes #1197

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Backend] scores table has no CHECK constraint enforcing the 300-850 range the app clamps in SQL

1 participant