From 939afa9afb2f4f61a725824b430716e4d20df2f7 Mon Sep 17 00:00:00 2001 From: Cal Corum Date: Wed, 4 Mar 2026 21:47:43 -0600 Subject: [PATCH] store: PR review: paper-dynasty-database#59 (REQUEST_CHANGES) --- ...aper-dynasty-database59-request-changes-8236b4.md | 12 ++++++++++++ 1 file changed, 12 insertions(+) create mode 100644 graph/workflows/pr-review-paper-dynasty-database59-request-changes-8236b4.md diff --git a/graph/workflows/pr-review-paper-dynasty-database59-request-changes-8236b4.md b/graph/workflows/pr-review-paper-dynasty-database59-request-changes-8236b4.md new file mode 100644 index 00000000000..2ba4f88bdf0 --- /dev/null +++ b/graph/workflows/pr-review-paper-dynasty-database59-request-changes-8236b4.md @@ -0,0 +1,12 @@ +--- +id: 8236b452-26b3-4287-a126-29fb046c4b0b +type: workflow +title: "PR review: paper-dynasty-database#59 (REQUEST_CHANGES)" +tags: [pr-reviewer, paper-dynasty-database, automation] +importance: 0.5 +confidence: 0.8 +created: "2026-03-05T03:47:43.310852+00:00" +updated: "2026-03-05T03:47:43.310852+00:00" +--- + +Reviewed paper-dynasty-database PR #59 — feat: add scout opportunities and claims API (#44). Verdict: REQUEST_CHANGES. Summary: Three regressions from recently merged PRs: (1) plaintext bearer token logging (#7/#8 regression, security), (2) raise HTTPException(status_code=200) in delete handlers (#47 regression), (3) except Exception instead of peewee.DoesNotExist (#48 regression). Also: unused import of ScoutOpportunity in scout_claims.py, and Peewee FK on_delete missing for opener_team vs SQL migration. Domain model and API design are otherwise solid.. Cost: $0.5874949999999999