From b2f53cb7977d5664d410c1022c2a8e5f8b2d45a0 Mon Sep 17 00:00:00 2001 From: Cal Corum Date: Tue, 3 Mar 2026 15:32:54 -0600 Subject: [PATCH] daily sync: 3 added, 4 modified, 0 deleted (3 edges) --- ...O--pr-review-paper-dynasty-databa-13f7c1.md | 13 +++++++++++++ ...O--pr-review-paper-dynasty-databa-933bc0.md | 13 +++++++++++++ ...O--pr-review-paper-dynasty-databa-ec6673.md | 13 +++++++++++++ ...er-always-truthy-column-reference-cd43d0.md | 18 +++++++++++++++++- ...paper-dynasty-database33-approved-04e57a.md | 7 ++++++- ...paper-dynasty-database33-approved-5ae1bf.md | 7 ++++++- ...paper-dynasty-database33-approved-d36a86.md | 7 ++++++- 7 files changed, 74 insertions(+), 4 deletions(-) create mode 100644 graph/edges/fix-peewee-boolean-filter-alwa--RELATED_TO--pr-review-paper-dynasty-databa-13f7c1.md create mode 100644 graph/edges/fix-peewee-boolean-filter-alwa--RELATED_TO--pr-review-paper-dynasty-databa-933bc0.md create mode 100644 graph/edges/fix-peewee-boolean-filter-alwa--RELATED_TO--pr-review-paper-dynasty-databa-ec6673.md diff --git a/graph/edges/fix-peewee-boolean-filter-alwa--RELATED_TO--pr-review-paper-dynasty-databa-13f7c1.md b/graph/edges/fix-peewee-boolean-filter-alwa--RELATED_TO--pr-review-paper-dynasty-databa-13f7c1.md new file mode 100644 index 00000000000..05110eff735 --- /dev/null +++ b/graph/edges/fix-peewee-boolean-filter-alwa--RELATED_TO--pr-review-paper-dynasty-databa-13f7c1.md @@ -0,0 +1,13 @@ +--- +id: 13f7c174-b2d6-4f57-846c-a2a66a2784f0 +type: RELATED_TO +from_id: cd43d051-d2e8-436e-b964-f9260739cc15 +from_title: "Fix: Peewee boolean filter always-truthy column reference" +to_id: 04e57a23-0a20-49d6-8c5a-2fa5fc4e55b5 +to_title: "PR review: paper-dynasty-database#33 (APPROVED)" +strength: 0.75 +created: "2026-03-03T21:32:53.925903+00:00" +updated: "2026-03-03T21:32:53.925903+00:00" +--- + +Auto-edge: Fix: Peewee boolean filter always-truthy column reference → PR review: paper-dynasty-database#33 (APPROVED) diff --git a/graph/edges/fix-peewee-boolean-filter-alwa--RELATED_TO--pr-review-paper-dynasty-databa-933bc0.md b/graph/edges/fix-peewee-boolean-filter-alwa--RELATED_TO--pr-review-paper-dynasty-databa-933bc0.md new file mode 100644 index 00000000000..15c81fbd59a --- /dev/null +++ b/graph/edges/fix-peewee-boolean-filter-alwa--RELATED_TO--pr-review-paper-dynasty-databa-933bc0.md @@ -0,0 +1,13 @@ +--- +id: 933bc0e0-297c-4d5b-9757-a46a5adee7ad +type: RELATED_TO +from_id: cd43d051-d2e8-436e-b964-f9260739cc15 +from_title: "Fix: Peewee boolean filter always-truthy column reference" +to_id: 5ae1bf6e-1e1c-4486-8a4d-10afd9e42189 +to_title: "PR review: paper-dynasty-database#33 (APPROVED)" +strength: 0.71 +created: "2026-03-03T21:32:53.992308+00:00" +updated: "2026-03-03T21:32:53.992308+00:00" +--- + +Auto-edge: Fix: Peewee boolean filter always-truthy column reference → PR review: paper-dynasty-database#33 (APPROVED) diff --git a/graph/edges/fix-peewee-boolean-filter-alwa--RELATED_TO--pr-review-paper-dynasty-databa-ec6673.md b/graph/edges/fix-peewee-boolean-filter-alwa--RELATED_TO--pr-review-paper-dynasty-databa-ec6673.md new file mode 100644 index 00000000000..dc90bb83981 --- /dev/null +++ b/graph/edges/fix-peewee-boolean-filter-alwa--RELATED_TO--pr-review-paper-dynasty-databa-ec6673.md @@ -0,0 +1,13 @@ +--- +id: ec667393-cda8-4047-88d3-875188af99b2 +type: RELATED_TO +from_id: cd43d051-d2e8-436e-b964-f9260739cc15 +from_title: "Fix: Peewee boolean filter always-truthy column reference" +to_id: d36a86f0-8183-4c94-8d63-0be65d3fd63a +to_title: "PR review: paper-dynasty-database#33 (APPROVED)" +strength: 0.71 +created: "2026-03-03T21:32:53.957935+00:00" +updated: "2026-03-03T21:32:53.957935+00:00" +--- + +Auto-edge: Fix: Peewee boolean filter always-truthy column reference → PR review: paper-dynasty-database#33 (APPROVED) diff --git a/graph/fixes/fix-peewee-boolean-filter-always-truthy-column-reference-cd43d0.md b/graph/fixes/fix-peewee-boolean-filter-always-truthy-column-reference-cd43d0.md index 30e2884763a..ba90d53aa2e 100644 --- a/graph/fixes/fix-peewee-boolean-filter-always-truthy-column-reference-cd43d0.md +++ b/graph/fixes/fix-peewee-boolean-filter-always-truthy-column-reference-cd43d0.md @@ -6,7 +6,23 @@ tags: [paper-dynasty-database, peewee, python, fastapi, boolean-filter, orm, bug importance: 0.6 confidence: 0.8 created: "2026-03-03T21:32:52.436667+00:00" -updated: "2026-03-03T21:32:52.436667+00:00" +updated: "2026-03-03T21:32:53.992308+00:00" +relations: + - target: 04e57a23-0a20-49d6-8c5a-2fa5fc4e55b5 + type: RELATED_TO + direction: outgoing + strength: 0.75 + edge_id: 13f7c174-b2d6-4f57-846c-a2a66a2784f0 + - target: d36a86f0-8183-4c94-8d63-0be65d3fd63a + type: RELATED_TO + direction: outgoing + strength: 0.71 + edge_id: ec667393-cda8-4047-88d3-875188af99b2 + - target: 5ae1bf6e-1e1c-4486-8a4d-10afd9e42189 + type: RELATED_TO + direction: outgoing + strength: 0.71 + edge_id: 933bc0e0-297c-4d5b-9757-a46a5adee7ad --- ## Problem 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 5709e6c9285..bba031bf382 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-03T21:22:09.723912+00:00" +updated: "2026-03-03T21:32:53.925903+00:00" relations: - target: d36a86f0-8183-4c94-8d63-0be65d3fd63a type: RELATED_TO @@ -38,6 +38,11 @@ relations: direction: incoming strength: 0.72 edge_id: d2bdb6b1-395e-4740-a402-b588331b72ba + - target: cd43d051-d2e8-436e-b964-f9260739cc15 + type: RELATED_TO + direction: incoming + strength: 0.75 + edge_id: 13f7c174-b2d6-4f57-846c-a2a66a2784f0 --- 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 7a510a605a4..198e3e2e336 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-03T20:33:06.391214+00:00" +updated: "2026-03-03T21:32:53.992308+00:00" relations: - target: 530bfb6f-841c-4381-b8a4-d0b6fbc02878 type: RELATED_TO @@ -18,6 +18,11 @@ relations: direction: incoming strength: 0.81 edge_id: 924f6a05-07de-4f96-af11-eea5211ef738 + - target: cd43d051-d2e8-436e-b964-f9260739cc15 + type: RELATED_TO + direction: incoming + strength: 0.71 + edge_id: 933bc0e0-297c-4d5b-9757-a46a5adee7ad --- 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 f7cf5ec5387..1208a0f5e54 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-03T21:22:09.788800+00:00" +updated: "2026-03-03T21:32:53.957935+00:00" relations: - target: 04e57a23-0a20-49d6-8c5a-2fa5fc4e55b5 type: RELATED_TO @@ -33,6 +33,11 @@ relations: direction: incoming strength: 0.71 edge_id: dd92b372-5e87-4cfe-9e5c-f97b09c0104b + - target: cd43d051-d2e8-436e-b964-f9260739cc15 + type: RELATED_TO + direction: incoming + strength: 0.71 + edge_id: ec667393-cda8-4047-88d3-875188af99b2 --- 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