59 lines
2.4 KiB
Markdown
59 lines
2.4 KiB
Markdown
---
|
|
id: 759c8f57-2519-4e21-afd6-bf89386aedb8
|
|
type: workflow
|
|
title: "PR review: paper-dynasty-database#47 — HTTPException 200 fix with undisclosed reformatting"
|
|
tags: [pr-reviewer, paper-dynasty-database, fastapi, python, scope-creep, ai-branch]
|
|
importance: 0.6
|
|
confidence: 0.8
|
|
created: "2026-03-04T00:49:28.716137+00:00"
|
|
updated: "2026-03-04T01:18:04.616819+00:00"
|
|
relations:
|
|
- target: 1e6628fa-dc5d-4047-a9ae-ecb884e04ba2
|
|
type: RELATED_TO
|
|
direction: outgoing
|
|
strength: 0.81
|
|
edge_id: 722ced20-f470-4abc-b77f-66348a21d318
|
|
- target: d36a86f0-8183-4c94-8d63-0be65d3fd63a
|
|
type: RELATED_TO
|
|
direction: outgoing
|
|
strength: 0.81
|
|
edge_id: ad1d9792-9f19-4d2d-b143-05f27208a318
|
|
- target: 5ae1bf6e-1e1c-4486-8a4d-10afd9e42189
|
|
type: RELATED_TO
|
|
direction: outgoing
|
|
strength: 0.81
|
|
edge_id: 2fd15bb5-01a0-4ad7-bcac-0fd3d4da87ea
|
|
- target: 37cca1c8-7fbd-43c3-a289-129b1c530f9f
|
|
type: RELATED_TO
|
|
direction: incoming
|
|
strength: 0.76
|
|
edge_id: 8668a1bd-c71f-44f4-b15a-34ad97917e66
|
|
---
|
|
|
|
## Review Summary
|
|
|
|
**PR**: paper-dynasty-database#47 — "fix: replace raise HTTPException(status_code=200) with return statements (#16)"
|
|
**Branch**: ai/paper-dynasty-database#16 → next-release
|
|
**Verdict**: REQUEST_CHANGES (review could not be posted — Gitea rejects self-review)
|
|
|
|
## Key Findings
|
|
|
|
### Core fix: Correct
|
|
- 22 `raise HTTPException(status_code=200, detail=...)` → `return {"message": ...}` replacements across 16 router files
|
|
- Semantically correct; HTTPException is for errors, not 200 success
|
|
- None of the affected endpoints have response_model annotations that conflict with the new return shape
|
|
|
|
### Problem: Undisclosed scope creep
|
|
The diff is ~10x larger than described (1374 added vs 776 removed lines):
|
|
- **Quote style**: Single → double quotes throughout all 16 files
|
|
- **Function reformatting**: Params to one-per-line with trailing commas
|
|
- **APIRouter/logging.basicConfig** reformatting
|
|
|
|
These changes are individually safe but make confident review impossible for an AI-generated PR.
|
|
|
|
## Pattern to Watch
|
|
AI issue-worker (ai/ branches) tends to apply auto-formatter on touched files, producing massive undisclosed style diffs. This is a recurring concern. PRs should be regenerated with only targeted changes when this occurs.
|
|
|
|
## Note on Review Posting
|
|
Gitea error: "reject your own pull is not allowed" — PR author is `cal`, same as reviewer identity. Cannot post formal review on self-authored PRs.
|