feat(seer): Add dual-read helpers for Seer project preferences from Sentry DB#111591
1 issue
sentry-backend-bugs: Found 1 issue (1 medium)
Medium
Unhandled ValueError from invalid repository name format crashes bulk operations - `src/sentry/seer/autofix/utils.py:685`
build_repo_definition_from_project_repo raises ValueError at line 685 when a repository name doesn't contain "/", but neither read_preference_from_sentry_db (line 737) nor bulk_read_preferences_from_sentry_db (line 774) catch this exception. While repository names typically follow "owner/repo" format, there's no database constraint enforcing this. A single malformed repository will crash the entire operation instead of being skipped gracefully.
Duration: 2m 35s · Tokens: 775.1k in / 7.0k out · Cost: $0.94 (+extraction: $0.00)
Annotations
Check warning on line 685 in src/sentry/seer/autofix/utils.py
sentry-warden / warden: sentry-backend-bugs
Unhandled ValueError from invalid repository name format crashes bulk operations
`build_repo_definition_from_project_repo` raises `ValueError` at line 685 when a repository name doesn't contain "/", but neither `read_preference_from_sentry_db` (line 737) nor `bulk_read_preferences_from_sentry_db` (line 774) catch this exception. While repository names typically follow "owner/repo" format, there's no database constraint enforcing this. A single malformed repository will crash the entire operation instead of being skipped gracefully.