From e156de22c3ff70a5f7660f5a16dab90c78ec840e Mon Sep 17 00:00:00 2001 From: Justin Visser Date: Mon, 2 Mar 2026 20:11:10 +0100 Subject: [PATCH] ruff format --- backend/app/database.py | 4 +--- backend/app/routes/site.py | 4 +--- backend/app/services/segment_service.py | 32 ++++++++++++------------- backend/app/services/site_service.py | 4 +--- backend/tests/test_database.py | 18 +++++++------- 5 files changed, 28 insertions(+), 34 deletions(-) diff --git a/backend/app/database.py b/backend/app/database.py index 98f91ef..86ca965 100644 --- a/backend/app/database.py +++ b/backend/app/database.py @@ -26,9 +26,7 @@ def init_db(db_path: Path) -> None: updated_at TEXT NOT NULL ) """) - conn.execute( - "CREATE INDEX IF NOT EXISTS idx_segments_order ON segments(sort_order)" - ) + conn.execute("CREATE INDEX IF NOT EXISTS idx_segments_order ON segments(sort_order)") conn.commit() finally: conn.close() diff --git a/backend/app/routes/site.py b/backend/app/routes/site.py index 3a96e8f..17995fb 100644 --- a/backend/app/routes/site.py +++ b/backend/app/routes/site.py @@ -17,9 +17,7 @@ DbPath = Annotated[Path, Depends(get_db_path)] @router.get("/") def get_site(db_path: DbPath) -> SiteResponse: - title = site_service.get_site_title( - db_path, default=get_settings().site_title - ) + title = site_service.get_site_title(db_path, default=get_settings().site_title) return SiteResponse(title=title) diff --git a/backend/app/services/segment_service.py b/backend/app/services/segment_service.py index fc09e63..dae3cca 100644 --- a/backend/app/services/segment_service.py +++ b/backend/app/services/segment_service.py @@ -36,8 +36,16 @@ def create_segment(db_path: Path, request: SegmentCreateRequest) -> Segment: cols = "id, type, sort_order, title, content, metadata, created_at, updated_at" conn.execute( f"INSERT INTO segments ({cols}) VALUES (?, ?, ?, ?, ?, ?, ?, ?)", - (str(segment_id), request.type.value, sort_order, request.title, - request.content, metadata_json, now, now), + ( + str(segment_id), + request.type.value, + sort_order, + request.title, + request.content, + metadata_json, + now, + now, + ), ) conn.commit() @@ -58,9 +66,7 @@ def create_segment(db_path: Path, request: SegmentCreateRequest) -> Segment: def list_segments(db_path: Path) -> list[Segment]: conn = get_connection(db_path) try: - rows = conn.execute( - "SELECT * FROM segments ORDER BY sort_order ASC" - ).fetchall() + rows = conn.execute("SELECT * FROM segments ORDER BY sort_order ASC").fetchall() return [_row_to_segment(row) for row in rows] finally: conn.close() @@ -69,9 +75,7 @@ def list_segments(db_path: Path) -> list[Segment]: def get_segment(db_path: Path, segment_id: UUID) -> Segment | None: conn = get_connection(db_path) try: - row = conn.execute( - "SELECT * FROM segments WHERE id = ?", (str(segment_id),) - ).fetchone() + row = conn.execute("SELECT * FROM segments WHERE id = ?", (str(segment_id),)).fetchone() if row is None: return None return _row_to_segment(row) @@ -94,8 +98,7 @@ def update_segment( title = request.title if request.title is not None else existing["title"] content = request.content if request.content is not None else existing["content"] metadata_json = ( - json.dumps(request.metadata) if request.metadata is not None - else existing["metadata"] + json.dumps(request.metadata) if request.metadata is not None else existing["metadata"] ) conn.execute( @@ -113,9 +116,7 @@ def update_segment( def delete_segment(db_path: Path, segment_id: UUID) -> bool: conn = get_connection(db_path) try: - cursor = conn.execute( - "DELETE FROM segments WHERE id = ?", (str(segment_id),) - ) + cursor = conn.execute("DELETE FROM segments WHERE id = ?", (str(segment_id),)) conn.commit() return cursor.rowcount > 0 finally: @@ -125,10 +126,7 @@ def delete_segment(db_path: Path, segment_id: UUID) -> bool: def reorder_segments(db_path: Path, segment_ids: list[UUID]) -> list[Segment]: conn = get_connection(db_path) try: - existing_ids = { - row["id"] - for row in conn.execute("SELECT id FROM segments").fetchall() - } + existing_ids = {row["id"] for row in conn.execute("SELECT id FROM segments").fetchall()} requested_ids = {str(sid) for sid in segment_ids} missing = requested_ids - existing_ids if missing: diff --git a/backend/app/services/site_service.py b/backend/app/services/site_service.py index 2a650dd..15ea4fe 100644 --- a/backend/app/services/site_service.py +++ b/backend/app/services/site_service.py @@ -8,9 +8,7 @@ DEFAULT_SITE_TITLE = "Untitled Site" def get_site_title(db_path: Path, *, default: str = DEFAULT_SITE_TITLE) -> str: conn = get_connection(db_path) try: - row = conn.execute( - "SELECT value FROM site WHERE key = ?", ("title",) - ).fetchone() + row = conn.execute("SELECT value FROM site WHERE key = ?", ("title",)).fetchone() if row is None: return default return str(row["value"]) diff --git a/backend/tests/test_database.py b/backend/tests/test_database.py index f41becf..cc6003b 100644 --- a/backend/tests/test_database.py +++ b/backend/tests/test_database.py @@ -9,9 +9,7 @@ def test_init_db_creates_tables(tmp_data_dir: Path) -> None: init_db(db_path) conn = sqlite3.connect(db_path) - cursor = conn.execute( - "SELECT name FROM sqlite_master WHERE type='table' ORDER BY name" - ) + cursor = conn.execute("SELECT name FROM sqlite_master WHERE type='table' ORDER BY name") tables = {row[0] for row in cursor.fetchall()} conn.close() @@ -25,9 +23,7 @@ def test_init_db_idempotent(tmp_data_dir: Path) -> None: init_db(db_path) conn = sqlite3.connect(db_path) - cursor = conn.execute( - "SELECT name FROM sqlite_master WHERE type='table' ORDER BY name" - ) + cursor = conn.execute("SELECT name FROM sqlite_master WHERE type='table' ORDER BY name") tables = {row[0] for row in cursor.fetchall()} conn.close() @@ -63,8 +59,14 @@ def test_segments_table_schema(db: Path) -> None: conn.close() expected = { - "id", "type", "sort_order", "title", - "content", "metadata", "created_at", "updated_at", + "id", + "type", + "sort_order", + "title", + "content", + "metadata", + "created_at", + "updated_at", } assert columns == expected