daily sync: 3 added, 4 modified, 0 deleted (3 edges)
This commit is contained in:
parent
30785e4393
commit
f605f399ce
@ -0,0 +1,13 @@
|
||||
---
|
||||
id: 82c2dfef-28b1-4e09-a5b4-94a83a4dcbd5
|
||||
type: RELATED_TO
|
||||
from_id: 29f1641c-5539-4814-8efa-c03692025c15
|
||||
from_title: "PR review: paper-dynasty-database#38 — offense_col default_factory fix (APPROVED)"
|
||||
to_id: 04e57a23-0a20-49d6-8c5a-2fa5fc4e55b5
|
||||
to_title: "PR review: paper-dynasty-database#33 (APPROVED)"
|
||||
strength: 0.8
|
||||
created: "2026-03-03T20:33:06.424317+00:00"
|
||||
updated: "2026-03-03T20:33:06.424317+00:00"
|
||||
---
|
||||
|
||||
Auto-edge: PR review: paper-dynasty-database#38 — offense_col default_factory fix (APPROVED) → PR review: paper-dynasty-database#33 (APPROVED)
|
||||
@ -0,0 +1,13 @@
|
||||
---
|
||||
id: 924f6a05-07de-4f96-af11-eea5211ef738
|
||||
type: RELATED_TO
|
||||
from_id: 29f1641c-5539-4814-8efa-c03692025c15
|
||||
from_title: "PR review: paper-dynasty-database#38 — offense_col default_factory fix (APPROVED)"
|
||||
to_id: 5ae1bf6e-1e1c-4486-8a4d-10afd9e42189
|
||||
to_title: "PR review: paper-dynasty-database#33 (APPROVED)"
|
||||
strength: 0.81
|
||||
created: "2026-03-03T20:33:06.391214+00:00"
|
||||
updated: "2026-03-03T20:33:06.391214+00:00"
|
||||
---
|
||||
|
||||
Auto-edge: PR review: paper-dynasty-database#38 — offense_col default_factory fix (APPROVED) → PR review: paper-dynasty-database#33 (APPROVED)
|
||||
@ -0,0 +1,13 @@
|
||||
---
|
||||
id: 92c4541e-21a0-4898-a56d-9d6bc785aced
|
||||
type: RELATED_TO
|
||||
from_id: 29f1641c-5539-4814-8efa-c03692025c15
|
||||
from_title: "PR review: paper-dynasty-database#38 — offense_col default_factory fix (APPROVED)"
|
||||
to_id: d36a86f0-8183-4c94-8d63-0be65d3fd63a
|
||||
to_title: "PR review: paper-dynasty-database#33 (APPROVED)"
|
||||
strength: 0.81
|
||||
created: "2026-03-03T20:33:06.359804+00:00"
|
||||
updated: "2026-03-03T20:33:06.359804+00:00"
|
||||
---
|
||||
|
||||
Auto-edge: PR review: paper-dynasty-database#38 — offense_col default_factory fix (APPROVED) → PR review: paper-dynasty-database#33 (APPROVED)
|
||||
@ -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-03T20:06:06.697825+00:00"
|
||||
updated: "2026-03-03T20:33:06.424317+00:00"
|
||||
relations:
|
||||
- target: d36a86f0-8183-4c94-8d63-0be65d3fd63a
|
||||
type: RELATED_TO
|
||||
@ -28,6 +28,11 @@ relations:
|
||||
direction: incoming
|
||||
strength: 0.7
|
||||
edge_id: b210b16d-b168-4798-b73f-98ef27f46dd8
|
||||
- target: 29f1641c-5539-4814-8efa-c03692025c15
|
||||
type: RELATED_TO
|
||||
direction: incoming
|
||||
strength: 0.8
|
||||
edge_id: 82c2dfef-28b1-4e09-a5b4-94a83a4dcbd5
|
||||
---
|
||||
|
||||
Reviewed paper-dynasty-database PR #33 — fix: replace 467 manual db.close() calls with middleware (#30). Verdict: APPROVED.
|
||||
|
||||
@ -6,13 +6,18 @@ 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:06:06.763300+00:00"
|
||||
updated: "2026-03-03T20:33:06.391214+00:00"
|
||||
relations:
|
||||
- target: 530bfb6f-841c-4381-b8a4-d0b6fbc02878
|
||||
type: RELATED_TO
|
||||
direction: incoming
|
||||
strength: 0.68
|
||||
edge_id: b3e61cf3-d61d-4421-8e81-8818a7601791
|
||||
- target: 29f1641c-5539-4814-8efa-c03692025c15
|
||||
type: RELATED_TO
|
||||
direction: incoming
|
||||
strength: 0.81
|
||||
edge_id: 924f6a05-07de-4f96-af11-eea5211ef738
|
||||
---
|
||||
|
||||
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
|
||||
|
||||
@ -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-03T20:06:06.729601+00:00"
|
||||
updated: "2026-03-03T20:33:06.359804+00:00"
|
||||
relations:
|
||||
- target: 04e57a23-0a20-49d6-8c5a-2fa5fc4e55b5
|
||||
type: RELATED_TO
|
||||
@ -23,6 +23,11 @@ relations:
|
||||
direction: incoming
|
||||
strength: 0.68
|
||||
edge_id: 62213d38-b5b9-4166-8d1b-4b48f216772a
|
||||
- target: 29f1641c-5539-4814-8efa-c03692025c15
|
||||
type: RELATED_TO
|
||||
direction: incoming
|
||||
strength: 0.81
|
||||
edge_id: 92c4541e-21a0-4898-a56d-9d6bc785aced
|
||||
---
|
||||
|
||||
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
|
||||
|
||||
@ -6,7 +6,23 @@ tags: [pr-reviewer, paper-dynasty-database, pydantic, python, fastapi, bug-fix]
|
||||
importance: 0.4
|
||||
confidence: 0.8
|
||||
created: "2026-03-03T20:33:06.020351+00:00"
|
||||
updated: "2026-03-03T20:33:06.020351+00:00"
|
||||
updated: "2026-03-03T20:33:06.424317+00:00"
|
||||
relations:
|
||||
- target: d36a86f0-8183-4c94-8d63-0be65d3fd63a
|
||||
type: RELATED_TO
|
||||
direction: outgoing
|
||||
strength: 0.81
|
||||
edge_id: 92c4541e-21a0-4898-a56d-9d6bc785aced
|
||||
- target: 5ae1bf6e-1e1c-4486-8a4d-10afd9e42189
|
||||
type: RELATED_TO
|
||||
direction: outgoing
|
||||
strength: 0.81
|
||||
edge_id: 924f6a05-07de-4f96-af11-eea5211ef738
|
||||
- target: 04e57a23-0a20-49d6-8c5a-2fa5fc4e55b5
|
||||
type: RELATED_TO
|
||||
direction: outgoing
|
||||
strength: 0.8
|
||||
edge_id: 82c2dfef-28b1-4e09-a5b4-94a83a4dcbd5
|
||||
---
|
||||
|
||||
Reviewed PR #38 in cal/paper-dynasty-database.
|
||||
|
||||
Loading…
Reference in New Issue
Block a user