From a46c8d7655a52d680930dc2f1481c63a6032b7a7 Mon Sep 17 00:00:00 2001 From: Cal Corum Date: Tue, 3 Mar 2026 14:06:06 -0600 Subject: [PATCH] daily sync: 3 added, 4 modified, 0 deleted (3 edges) --- ...O--pr-review-paper-dynasty-databa-62213d.md | 13 +++++++++++++ ...O--pr-review-paper-dynasty-databa-b210b1.md | 13 +++++++++++++ ...O--pr-review-paper-dynasty-databa-b3e61c.md | 13 +++++++++++++ ...valuated-at-class-definition-time-530bfb.md | 18 +++++++++++++++++- ...paper-dynasty-database33-approved-04e57a.md | 7 ++++++- ...paper-dynasty-database33-approved-5ae1bf.md | 8 +++++++- ...paper-dynasty-database33-approved-d36a86.md | 7 ++++++- 7 files changed, 75 insertions(+), 4 deletions(-) create mode 100644 graph/edges/fix-pydantic-bare-default-eval--RELATED_TO--pr-review-paper-dynasty-databa-62213d.md create mode 100644 graph/edges/fix-pydantic-bare-default-eval--RELATED_TO--pr-review-paper-dynasty-databa-b210b1.md create mode 100644 graph/edges/fix-pydantic-bare-default-eval--RELATED_TO--pr-review-paper-dynasty-databa-b3e61c.md diff --git a/graph/edges/fix-pydantic-bare-default-eval--RELATED_TO--pr-review-paper-dynasty-databa-62213d.md b/graph/edges/fix-pydantic-bare-default-eval--RELATED_TO--pr-review-paper-dynasty-databa-62213d.md new file mode 100644 index 00000000000..1e0e07467af --- /dev/null +++ b/graph/edges/fix-pydantic-bare-default-eval--RELATED_TO--pr-review-paper-dynasty-databa-62213d.md @@ -0,0 +1,13 @@ +--- +id: 62213d38-b5b9-4166-8d1b-4b48f216772a +type: RELATED_TO +from_id: 530bfb6f-841c-4381-b8a4-d0b6fbc02878 +from_title: "Fix: Pydantic bare default evaluated at class definition time (paper-dynasty-database)" +to_id: d36a86f0-8183-4c94-8d63-0be65d3fd63a +to_title: "PR review: paper-dynasty-database#33 (APPROVED)" +strength: 0.68 +created: "2026-03-03T20:06:06.729601+00:00" +updated: "2026-03-03T20:06:06.729601+00:00" +--- + +Auto-edge: Fix: Pydantic bare default evaluated at class definition time (paper-dynasty-database) → PR review: paper-dynasty-database#33 (APPROVED) diff --git a/graph/edges/fix-pydantic-bare-default-eval--RELATED_TO--pr-review-paper-dynasty-databa-b210b1.md b/graph/edges/fix-pydantic-bare-default-eval--RELATED_TO--pr-review-paper-dynasty-databa-b210b1.md new file mode 100644 index 00000000000..9ead9edf331 --- /dev/null +++ b/graph/edges/fix-pydantic-bare-default-eval--RELATED_TO--pr-review-paper-dynasty-databa-b210b1.md @@ -0,0 +1,13 @@ +--- +id: b210b16d-b168-4798-b73f-98ef27f46dd8 +type: RELATED_TO +from_id: 530bfb6f-841c-4381-b8a4-d0b6fbc02878 +from_title: "Fix: Pydantic bare default evaluated at class definition time (paper-dynasty-database)" +to_id: 04e57a23-0a20-49d6-8c5a-2fa5fc4e55b5 +to_title: "PR review: paper-dynasty-database#33 (APPROVED)" +strength: 0.7 +created: "2026-03-03T20:06:06.697825+00:00" +updated: "2026-03-03T20:06:06.697825+00:00" +--- + +Auto-edge: Fix: Pydantic bare default evaluated at class definition time (paper-dynasty-database) → PR review: paper-dynasty-database#33 (APPROVED) diff --git a/graph/edges/fix-pydantic-bare-default-eval--RELATED_TO--pr-review-paper-dynasty-databa-b3e61c.md b/graph/edges/fix-pydantic-bare-default-eval--RELATED_TO--pr-review-paper-dynasty-databa-b3e61c.md new file mode 100644 index 00000000000..91a3982c35c --- /dev/null +++ b/graph/edges/fix-pydantic-bare-default-eval--RELATED_TO--pr-review-paper-dynasty-databa-b3e61c.md @@ -0,0 +1,13 @@ +--- +id: b3e61cf3-d61d-4421-8e81-8818a7601791 +type: RELATED_TO +from_id: 530bfb6f-841c-4381-b8a4-d0b6fbc02878 +from_title: "Fix: Pydantic bare default evaluated at class definition time (paper-dynasty-database)" +to_id: 5ae1bf6e-1e1c-4486-8a4d-10afd9e42189 +to_title: "PR review: paper-dynasty-database#33 (APPROVED)" +strength: 0.68 +created: "2026-03-03T20:06:06.763300+00:00" +updated: "2026-03-03T20:06:06.763300+00:00" +--- + +Auto-edge: Fix: Pydantic bare default evaluated at class definition time (paper-dynasty-database) → PR review: paper-dynasty-database#33 (APPROVED) diff --git a/graph/fixes/fix-pydantic-bare-default-evaluated-at-class-definition-time-530bfb.md b/graph/fixes/fix-pydantic-bare-default-evaluated-at-class-definition-time-530bfb.md index 55fefc41f55..6484b8dfb6d 100644 --- a/graph/fixes/fix-pydantic-bare-default-evaluated-at-class-definition-time-530bfb.md +++ b/graph/fixes/fix-pydantic-bare-default-evaluated-at-class-definition-time-530bfb.md @@ -6,7 +6,23 @@ tags: [paper-dynasty, python, pydantic, fix, bug, fastapi] importance: 0.65 confidence: 0.8 created: "2026-03-03T20:06:06.122074+00:00" -updated: "2026-03-03T20:06:06.122074+00:00" +updated: "2026-03-03T20:06:06.763300+00:00" +relations: + - target: 04e57a23-0a20-49d6-8c5a-2fa5fc4e55b5 + type: RELATED_TO + direction: outgoing + strength: 0.7 + edge_id: b210b16d-b168-4798-b73f-98ef27f46dd8 + - target: d36a86f0-8183-4c94-8d63-0be65d3fd63a + type: RELATED_TO + direction: outgoing + strength: 0.68 + edge_id: 62213d38-b5b9-4166-8d1b-4b48f216772a + - target: 5ae1bf6e-1e1c-4486-8a4d-10afd9e42189 + type: RELATED_TO + direction: outgoing + strength: 0.68 + edge_id: b3e61cf3-d61d-4421-8e81-8818a7601791 --- ## 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 df5ef4bba9f..8dba1f9d773 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-03T18:08:58.604732+00:00" +updated: "2026-03-03T20:06:06.697825+00:00" relations: - target: d36a86f0-8183-4c94-8d63-0be65d3fd63a type: RELATED_TO @@ -23,6 +23,11 @@ relations: direction: outgoing strength: 0.74 edge_id: 92a1af4d-2dc8-4672-bce1-33f09bde6a19 + - target: 530bfb6f-841c-4381-b8a4-d0b6fbc02878 + type: RELATED_TO + direction: incoming + strength: 0.7 + edge_id: b210b16d-b168-4798-b73f-98ef27f46dd8 --- 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 6ab887cd4ff..9aac7db7193 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,13 @@ 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-03T18:09:25.812471+00:00" +updated: "2026-03-03T20:06:06.763300+00:00" +relations: + - target: 530bfb6f-841c-4381-b8a4-d0b6fbc02878 + type: RELATED_TO + direction: incoming + strength: 0.68 + edge_id: b3e61cf3-d61d-4421-8e81-8818a7601791 --- 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 7f328e2f0b2..fabf0b46f0d 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-03T18:35:30.163861+00:00" +updated: "2026-03-03T20:06:06.729601+00:00" relations: - target: 04e57a23-0a20-49d6-8c5a-2fa5fc4e55b5 type: RELATED_TO @@ -18,6 +18,11 @@ relations: direction: incoming strength: 0.7 edge_id: 58e767ee-ef21-42a5-afd8-1d96917849bb + - target: 530bfb6f-841c-4381-b8a4-d0b6fbc02878 + type: RELATED_TO + direction: incoming + strength: 0.68 + edge_id: 62213d38-b5b9-4166-8d1b-4b48f216772a --- 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