diff --git a/graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-4b6932.md b/graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-4b6932.md new file mode 100644 index 00000000000..e45b5b09b9a --- /dev/null +++ b/graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-4b6932.md @@ -0,0 +1,13 @@ +--- +id: 4b69329c-2881-4d73-b690-3567df580e5f +type: RELATED_TO +from_id: 07eb63c2-1823-4f26-b976-a725ea5ab324 +from_title: "PR review: paper-dynasty-database#43 — FastAPI int type annotations on path params" +to_id: 04e57a23-0a20-49d6-8c5a-2fa5fc4e55b5 +to_title: "PR review: paper-dynasty-database#33 (APPROVED)" +strength: 0.74 +created: "2026-03-03T22:48:00.696003+00:00" +updated: "2026-03-03T22:48:00.696003+00:00" +--- + +Auto-edge: PR review: paper-dynasty-database#43 — FastAPI int type annotations on path params → PR review: paper-dynasty-database#33 (APPROVED) diff --git a/graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-a422a7.md b/graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-a422a7.md new file mode 100644 index 00000000000..5301a8fdbf1 --- /dev/null +++ b/graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-a422a7.md @@ -0,0 +1,13 @@ +--- +id: a422a779-6644-4af6-be68-7caa845547a9 +type: RELATED_TO +from_id: 07eb63c2-1823-4f26-b976-a725ea5ab324 +from_title: "PR review: paper-dynasty-database#43 — FastAPI int type annotations on path params" +to_id: 5ae1bf6e-1e1c-4486-8a4d-10afd9e42189 +to_title: "PR review: paper-dynasty-database#33 (APPROVED)" +strength: 0.71 +created: "2026-03-03T22:48:00.764331+00:00" +updated: "2026-03-03T22:48:00.764331+00:00" +--- + +Auto-edge: PR review: paper-dynasty-database#43 — FastAPI int type annotations on path params → PR review: paper-dynasty-database#33 (APPROVED) diff --git a/graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-f73ec2.md b/graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-f73ec2.md new file mode 100644 index 00000000000..f8a2012d0b2 --- /dev/null +++ b/graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-f73ec2.md @@ -0,0 +1,13 @@ +--- +id: f73ec266-dfcd-4a88-a0ad-10dfdf3eade7 +type: RELATED_TO +from_id: 07eb63c2-1823-4f26-b976-a725ea5ab324 +from_title: "PR review: paper-dynasty-database#43 — FastAPI int type annotations on path params" +to_id: d36a86f0-8183-4c94-8d63-0be65d3fd63a +to_title: "PR review: paper-dynasty-database#33 (APPROVED)" +strength: 0.71 +created: "2026-03-03T22:48:00.734269+00:00" +updated: "2026-03-03T22:48:00.734269+00:00" +--- + +Auto-edge: PR review: paper-dynasty-database#43 — FastAPI int type annotations on path params → PR review: paper-dynasty-database#33 (APPROVED) diff --git a/graph/workflows/pr-review-paper-dynasty-database33-approved-04e57a.md b/graph/workflows/pr-review-paper-dynasty-database33-approved-04e57a.md index 08a0aa932e9..1e14fa942a1 100644 --- a/graph/workflows/pr-review-paper-dynasty-database33-approved-04e57a.md +++ b/graph/workflows/pr-review-paper-dynasty-database33-approved-04e57a.md @@ -6,7 +6,7 @@ tags: [pr-reviewer, paper-dynasty-database, fastapi, peewee, middleware, python] importance: 0.5 confidence: 0.8 created: "2026-03-03T18:08:58.224854+00:00" -updated: "2026-03-03T22:17:25.527091+00:00" +updated: "2026-03-03T22:48:00.696003+00:00" relations: - target: d36a86f0-8183-4c94-8d63-0be65d3fd63a type: RELATED_TO @@ -48,6 +48,11 @@ relations: direction: incoming strength: 0.79 edge_id: 3561b670-7afa-4a12-8131-5b099f890db5 + - target: 07eb63c2-1823-4f26-b976-a725ea5ab324 + type: RELATED_TO + direction: incoming + strength: 0.74 + edge_id: 4b69329c-2881-4d73-b690-3567df580e5f --- Reviewed paper-dynasty-database PR #33 — fix: replace 467 manual db.close() calls with middleware (#30). Verdict: APPROVED. diff --git a/graph/workflows/pr-review-paper-dynasty-database33-approved-5ae1bf.md b/graph/workflows/pr-review-paper-dynasty-database33-approved-5ae1bf.md index 8ffb8b30409..36ea1ba78b4 100644 --- a/graph/workflows/pr-review-paper-dynasty-database33-approved-5ae1bf.md +++ b/graph/workflows/pr-review-paper-dynasty-database33-approved-5ae1bf.md @@ -6,7 +6,7 @@ tags: [pr-reviewer, paper-dynasty-database, automation] importance: 0.5 confidence: 0.8 created: "2026-03-03T18:09:25.812471+00:00" -updated: "2026-03-03T22:17:25.595134+00:00" +updated: "2026-03-03T22:48:00.764331+00:00" relations: - target: 530bfb6f-841c-4381-b8a4-d0b6fbc02878 type: RELATED_TO @@ -28,6 +28,11 @@ relations: direction: incoming strength: 0.77 edge_id: 4b330e3c-087d-41e8-8ffc-3b53f004f551 + - target: 07eb63c2-1823-4f26-b976-a725ea5ab324 + type: RELATED_TO + direction: incoming + strength: 0.71 + edge_id: a422a779-6644-4af6-be68-7caa845547a9 --- Reviewed paper-dynasty-database PR #33 — fix: replace 467 manual db.close() calls with middleware (#30). Verdict: APPROVED. Summary: Correct implementation of the standard Peewee + FastAPI middleware pattern. 467 db.close() calls removed from 30 router files; middleware correctly uses reuse_if_open=True with try/finally and is_clos. Cost: $1.062883 diff --git a/graph/workflows/pr-review-paper-dynasty-database33-approved-d36a86.md b/graph/workflows/pr-review-paper-dynasty-database33-approved-d36a86.md index 3197cc92d17..ebfc7ff9f11 100644 --- a/graph/workflows/pr-review-paper-dynasty-database33-approved-d36a86.md +++ b/graph/workflows/pr-review-paper-dynasty-database33-approved-d36a86.md @@ -6,7 +6,7 @@ tags: [pr-reviewer, paper-dynasty-database, automation] importance: 0.5 confidence: 0.8 created: "2026-03-03T06:06:26.108954+00:00" -updated: "2026-03-03T22:17:25.564811+00:00" +updated: "2026-03-03T22:48:00.734269+00:00" relations: - target: 04e57a23-0a20-49d6-8c5a-2fa5fc4e55b5 type: RELATED_TO @@ -43,6 +43,11 @@ relations: direction: incoming strength: 0.77 edge_id: 2276edb6-002f-4656-8ff2-897a026d813c + - target: 07eb63c2-1823-4f26-b976-a725ea5ab324 + type: RELATED_TO + direction: incoming + strength: 0.71 + edge_id: f73ec266-dfcd-4a88-a0ad-10dfdf3eade7 --- Reviewed paper-dynasty-database PR #33 — fix: replace 467 manual db.close() calls with middleware (#30). Verdict: APPROVED. Summary: Correct implementation of the standard Peewee + FastAPI middleware pattern. Eliminates connection-leak bug across 467 call sites. Safe for both SQLite (dev) and PooledPostgresqlDatabase (prod). Minor:. Cost: $0.98641425 diff --git a/graph/workflows/pr-review-paper-dynasty-database43-fastapi-int-type-annotati-07eb63.md b/graph/workflows/pr-review-paper-dynasty-database43-fastapi-int-type-annotati-07eb63.md index 7982d933a24..5fd0e7c9bc7 100644 --- a/graph/workflows/pr-review-paper-dynasty-database43-fastapi-int-type-annotati-07eb63.md +++ b/graph/workflows/pr-review-paper-dynasty-database43-fastapi-int-type-annotati-07eb63.md @@ -6,7 +6,23 @@ tags: [pr-reviewer, paper-dynasty-database, fastapi, type-annotations, python, p importance: 0.4 confidence: 0.8 created: "2026-03-03T22:48:00.357115+00:00" -updated: "2026-03-03T22:48:00.357115+00:00" +updated: "2026-03-03T22:48:00.764331+00:00" +relations: + - target: 04e57a23-0a20-49d6-8c5a-2fa5fc4e55b5 + type: RELATED_TO + direction: outgoing + strength: 0.74 + edge_id: 4b69329c-2881-4d73-b690-3567df580e5f + - target: d36a86f0-8183-4c94-8d63-0be65d3fd63a + type: RELATED_TO + direction: outgoing + strength: 0.71 + edge_id: f73ec266-dfcd-4a88-a0ad-10dfdf3eade7 + - target: 5ae1bf6e-1e1c-4486-8a4d-10afd9e42189 + type: RELATED_TO + direction: outgoing + strength: 0.71 + edge_id: a422a779-6644-4af6-be68-7caa845547a9 --- ## PR #43 Review — APPROVED