diff --git a/graph/workflows/pr-review-major-domo-database40-raise-httpexception-fix-stan-57e38d.md b/graph/workflows/pr-review-major-domo-database40-raise-httpexception-fix-stan-57e38d.md new file mode 100644 index 00000000000..02c36544ebf --- /dev/null +++ b/graph/workflows/pr-review-major-domo-database40-raise-httpexception-fix-stan-57e38d.md @@ -0,0 +1,28 @@ +--- +id: 57e38d6f-828e-496b-af23-da7f4806252c +type: workflow +title: "PR review: major-domo-database#40 — raise HTTPException fix (standings.py)" +tags: [pr-reviewer, major-domo-database, fastapi, python, standings, gitea] +importance: 0.6 +confidence: 0.8 +created: "2026-03-03T01:20:24.848534+00:00" +updated: "2026-03-03T01:20:24.848534+00:00" +--- + +## PR #40: fix: raise HTTPException in recalculate_standings on failure (#23) + +**Verdict**: REQUEST_CHANGES (review post blocked — Gitea rejects self-reviews) + +### Key Findings + +1. **Core fix is correct**: `recalculate_standings` in `standings.py` was missing a `raise` before `HTTPException(status_code=500, ...)`, causing the endpoint to silently return HTTP 200 on failure. Adding `raise` correctly propagates the error. + +2. **Undocumented `stratgame.py` changes**: PR body says only `standings.py` was changed, but diff includes full changes to `stratgame.py` (order_by return-value fix + reformatting). This is misleading. + +3. **Merge conflict risk with PR #39**: PR #39 ("fix: assign order_by() return value in GET /api/v3/games") modifies the exact same `stratgame.py` changes. Merging both creates conflict risk. + +4. **Gitea self-review block**: Both PRs (#39, #40) were created by `cal`, so `create_pull_request_review` fails with "reject your own pull is not allowed". Automated review posting is not possible for self-authored PRs in this repo. + +### Files Reviewed +- `app/routers_v3/standings.py` +- `app/routers_v3/stratgame.py`