From 08c1ceccdca142301ee825daef3c97f54e81d928 Mon Sep 17 00:00:00 2001 From: Cal Corum Date: Tue, 3 Mar 2026 21:18:20 -0600 Subject: [PATCH] daily sync: 3 added, 4 modified, 0 deleted (3 edges) --- ...O--pr-review-paper-dynasty-databa-74dd0a.md | 13 +++++++++++++ ...O--pr-review-paper-dynasty-databa-ce3834.md | 13 +++++++++++++ ...O--pr-review-paper-dynasty-databa-ec1d17.md | 13 +++++++++++++ ...paper-dynasty-database33-approved-04e57a.md | 7 ++++++- ...paper-dynasty-database33-approved-5ae1bf.md | 7 ++++++- ...paper-dynasty-database33-approved-d36a86.md | 7 ++++++- ...tabase52-consolidate-double-query-c9b2c0.md | 18 +++++++++++++++++- 7 files changed, 74 insertions(+), 4 deletions(-) create mode 100644 graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-74dd0a.md create mode 100644 graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-ce3834.md create mode 100644 graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-ec1d17.md diff --git a/graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-74dd0a.md b/graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-74dd0a.md new file mode 100644 index 00000000000..298cf77d9c4 --- /dev/null +++ b/graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-74dd0a.md @@ -0,0 +1,13 @@ +--- +id: 74dd0ac6-e39b-4fad-a669-8603c077c366 +type: RELATED_TO +from_id: c9b2c04d-258a-4028-849b-a47f6168a6ab +from_title: "PR review: paper-dynasty-database#52 — consolidate double-query in get_one_play" +to_id: 04e57a23-0a20-49d6-8c5a-2fa5fc4e55b5 +to_title: "PR review: paper-dynasty-database#33 (APPROVED)" +strength: 0.79 +created: "2026-03-04T03:18:20.267494+00:00" +updated: "2026-03-04T03:18:20.267494+00:00" +--- + +Auto-edge: PR review: paper-dynasty-database#52 — consolidate double-query in get_one_play → PR review: paper-dynasty-database#33 (APPROVED) diff --git a/graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-ce3834.md b/graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-ce3834.md new file mode 100644 index 00000000000..97ae0e7f6f0 --- /dev/null +++ b/graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-ce3834.md @@ -0,0 +1,13 @@ +--- +id: ce383453-a3ad-4713-99bb-64a5ba980679 +type: RELATED_TO +from_id: c9b2c04d-258a-4028-849b-a47f6168a6ab +from_title: "PR review: paper-dynasty-database#52 — consolidate double-query in get_one_play" +to_id: d36a86f0-8183-4c94-8d63-0be65d3fd63a +to_title: "PR review: paper-dynasty-database#33 (APPROVED)" +strength: 0.78 +created: "2026-03-04T03:18:20.303931+00:00" +updated: "2026-03-04T03:18:20.303931+00:00" +--- + +Auto-edge: PR review: paper-dynasty-database#52 — consolidate double-query in get_one_play → PR review: paper-dynasty-database#33 (APPROVED) diff --git a/graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-ec1d17.md b/graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-ec1d17.md new file mode 100644 index 00000000000..2c4a7dee8b9 --- /dev/null +++ b/graph/edges/pr-review-paper-dynasty-databa--RELATED_TO--pr-review-paper-dynasty-databa-ec1d17.md @@ -0,0 +1,13 @@ +--- +id: ec1d17f5-18bf-4be4-8547-ad124b4debd3 +type: RELATED_TO +from_id: c9b2c04d-258a-4028-849b-a47f6168a6ab +from_title: "PR review: paper-dynasty-database#52 — consolidate double-query in get_one_play" +to_id: 5ae1bf6e-1e1c-4486-8a4d-10afd9e42189 +to_title: "PR review: paper-dynasty-database#33 (APPROVED)" +strength: 0.78 +created: "2026-03-04T03:18:20.339054+00:00" +updated: "2026-03-04T03:18:20.339054+00:00" +--- + +Auto-edge: PR review: paper-dynasty-database#52 — consolidate double-query in get_one_play → 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 54389f7f67e..44be9e23cc7 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-04T03:03:31.325138+00:00" +updated: "2026-03-04T03:18:20.267494+00:00" relations: - target: d36a86f0-8183-4c94-8d63-0be65d3fd63a type: RELATED_TO @@ -73,6 +73,11 @@ relations: direction: incoming strength: 0.79 edge_id: d1610aeb-7f7f-4d7d-9911-b32a5e2fbd7c + - target: c9b2c04d-258a-4028-849b-a47f6168a6ab + type: RELATED_TO + direction: incoming + strength: 0.79 + edge_id: 74dd0ac6-e39b-4fad-a669-8603c077c366 --- 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 c331d8eff2d..5888b2bf812 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-04T03:03:31.389474+00:00" +updated: "2026-03-04T03:18:20.339054+00:00" relations: - target: 530bfb6f-841c-4381-b8a4-d0b6fbc02878 type: RELATED_TO @@ -53,6 +53,11 @@ relations: direction: incoming strength: 0.75 edge_id: 57e15307-bbd9-4314-b62e-5a042db329ce + - target: c9b2c04d-258a-4028-849b-a47f6168a6ab + type: RELATED_TO + direction: incoming + strength: 0.78 + edge_id: ec1d17f5-18bf-4be4-8547-ad124b4debd3 --- 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 0fdcc4d5b46..534ae75b83a 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-04T03:03:31.357547+00:00" +updated: "2026-03-04T03:18:20.303931+00:00" relations: - target: 04e57a23-0a20-49d6-8c5a-2fa5fc4e55b5 type: RELATED_TO @@ -78,6 +78,11 @@ relations: direction: incoming strength: 0.75 edge_id: 30ef8743-65a7-423d-9867-f9949c88bb8d + - target: c9b2c04d-258a-4028-849b-a47f6168a6ab + type: RELATED_TO + direction: incoming + strength: 0.78 + edge_id: ce383453-a3ad-4713-99bb-64a5ba980679 --- 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-database52-consolidate-double-query-c9b2c0.md b/graph/workflows/pr-review-paper-dynasty-database52-consolidate-double-query-c9b2c0.md index 539611b35e3..c83f017bd50 100644 --- a/graph/workflows/pr-review-paper-dynasty-database52-consolidate-double-query-c9b2c0.md +++ b/graph/workflows/pr-review-paper-dynasty-database52-consolidate-double-query-c9b2c0.md @@ -6,7 +6,23 @@ tags: [pr-reviewer, paper-dynasty-database, peewee, query-optimization, fastapi] importance: 0.4 confidence: 0.8 created: "2026-03-04T03:18:19.838537+00:00" -updated: "2026-03-04T03:18:19.838537+00:00" +updated: "2026-03-04T03:18:20.339054+00:00" +relations: + - target: 04e57a23-0a20-49d6-8c5a-2fa5fc4e55b5 + type: RELATED_TO + direction: outgoing + strength: 0.79 + edge_id: 74dd0ac6-e39b-4fad-a669-8603c077c366 + - target: d36a86f0-8183-4c94-8d63-0be65d3fd63a + type: RELATED_TO + direction: outgoing + strength: 0.78 + edge_id: ce383453-a3ad-4713-99bb-64a5ba980679 + - target: 5ae1bf6e-1e1c-4486-8a4d-10afd9e42189 + type: RELATED_TO + direction: outgoing + strength: 0.78 + edge_id: ec1d17f5-18bf-4be4-8547-ad124b4debd3 --- ## PR #52 — fix: consolidate redundant double-query in get_one_play