thejayman77/upbeatBytes
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Make the API read-only (healthz no longer runs init_db)

Browse Source 
Lets the API run as a read-only replica against a shared DB owned by the
ingestion CLI — needed for the production split (Caddy-proxied API container
reading the host-written database).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
jay
2026-06-01 02:19:14 +00:00
parent f57b63edef
commit 92fafa8785
1 changed files with 8 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
goodnews/api.py
+8 -4
View File
@@ -28,7 +28,7 @@ from fastapi.staticfiles import StaticFiles
from pydantic import BaseModel from pydantic import BaseModel
from . import feeds, queries from . import feeds, queries
from .db import connect, init_db from .db import connect
from .filters import filter_articles, prefs_from_json from .filters import filter_articles, prefs_from_json
from .hero import safe_to_lead from .hero import safe_to_lead
from .llm import LocalModelClient from .llm import LocalModelClient
@@ -218,9 +218,13 @@ def create_app() -> FastAPI:
@app.get("/healthz") @app.get("/healthz")
def healthz() -> dict: def healthz() -> dict:
with get_conn() as conn: # Read-only: the schema is owned by the ingestion CLI, so the API never
init_db(conn) # writes (it can run as a read-only replica against a shared DB).
scored = conn.execute("SELECT COUNT(*) FROM article_scores").fetchone()[0] try:
with get_conn() as conn:
scored = conn.execute("SELECT COUNT(*) FROM article_scores").fetchone()[0]
except sqlite3.Error:
scored = 0
return {"status": "ok", "scored_articles": scored} return {"status": "ok", "scored_articles": scored}
@app.get("/api/categories", response_model=CategoriesResponse) @app.get("/api/categories", response_model=CategoriesResponse)