fix: address pre-commit issues (mypy, markdownlint, ruff formatting)

Signed-off-by: Sébastien Han <seb@redhat.com>

Author: leseb

docs/superpowers/specs/2026-05-18-alembic-migrations-design.md

@@ -96,7 +96,7 @@ Add `alembic>=1.16` to `pyproject.toml` dependencies.
 
 ### 2. Migration Directory Structure
 
-```
+```text
 src/ogx/core/storage/
   schema.py             # Canonical schema definitions (single source of truth)
   migrations/
@@ -186,7 +186,7 @@ New file: `src/ogx/cli/db.py`
 
 Implements the `Db` top-level subcommand group using the existing `argparse`-based `Subcommand` pattern:
 
-```
+```bash
 ogx db upgrade [config_path]   # Run pending migrations
 ogx db current [config_path]   # Show current schema revision
 ogx db history                 # Show migration history
@@ -217,7 +217,7 @@ In `lifespan()` in `src/ogx/core/server/server.py` (the async context manager, n
    d. If it doesn't exist: this is either a fresh DB (OK, `create_table()` will handle it) or a pre-migration DB (needs `ogx db upgrade`)
 2. If the revision is behind: log an error and raise `SystemExit(1)`
 
-```
+```text
 FATAL: Database schema on backend 'sql_default' is at revision 'abc123', expected 'def456'.
 Run 'ogx db upgrade <config.yaml>' before starting the server.
 ```
@@ -334,7 +334,7 @@ When you need to change the schema of a stable table:
 
 ### Architecture
 
-```
+```text
 ogx db upgrade <config.yaml>
        |
        v

src/ogx/core/storage/migrations/env.py

@@ -15,7 +15,7 @@
 
 
 def get_url() -> str:
-    url = context.config.get_main_option("sqlalchemy.url")
+    url: str | None = context.config.get_main_option("sqlalchemy.url")
     if not url:
         raise RuntimeError("sqlalchemy.url must be set in Alembic config")
     return url

src/ogx/core/storage/migrations/versions/__init__.py

@@ -3,4 +3,3 @@
 #
 # This source code is licensed under the terms described in the LICENSE file in
 # the root directory of this source tree.
-

tests/integration/providers/utils/sqlstore/test_migrations.py

@@ -119,22 +119,26 @@ def test_adds_backfill_columns(self, pg_engine, pg_url):
             conn.execute(sa.text("DROP TABLE IF EXISTS alembic_version CASCADE"))
             conn.execute(sa.text("DROP TABLE IF EXISTS conversation_items CASCADE"))
             conn.execute(sa.text("DROP TABLE IF EXISTS openai_responses CASCADE"))
-            conn.execute(sa.text("""
+            conn.execute(
+                sa.text("""
                 CREATE TABLE conversation_items (
                     id TEXT PRIMARY KEY,
                     conversation_id TEXT,
                     created_at INTEGER,
                     item_data TEXT
                 )
-            """))
-            conn.execute(sa.text("""
+            """)
+            )
+            conn.execute(
+                sa.text("""
                 CREATE TABLE openai_responses (
                     id TEXT PRIMARY KEY,
                     created_at INTEGER,
                     response_object TEXT,
                     model TEXT
                 )
-            """))
+            """)
+            )
 
         cfg = _alembic_cfg(pg_url)
         command.upgrade(cfg, "head")
@@ -181,14 +185,16 @@ def test_custom_responses_table_gets_backfills(self, pg_engine, pg_url):
         with pg_engine.begin() as conn:
             conn.execute(sa.text("DROP TABLE IF EXISTS alembic_version CASCADE"))
             conn.execute(sa.text(f'DROP TABLE IF EXISTS "{custom_table}" CASCADE'))
-            conn.execute(sa.text(f"""
+            conn.execute(
+                sa.text(f"""
                 CREATE TABLE "{custom_table}" (
                     id TEXT PRIMARY KEY,
                     created_at INTEGER,
                     response_object TEXT,
                     model TEXT
                 )
-            """))
+            """)
+            )
 
         cfg = _alembic_cfg(pg_url, x_responses_table=custom_table)
         command.upgrade(cfg, "head")
@@ -208,18 +214,22 @@ def test_default_table_untouched_when_custom_name_set(self, pg_engine, pg_url):
             conn.execute(sa.text("DROP TABLE IF EXISTS alembic_version CASCADE"))
             conn.execute(sa.text("DROP TABLE IF EXISTS openai_responses CASCADE"))
             conn.execute(sa.text(f'DROP TABLE IF EXISTS "{custom_table}" CASCADE'))
-            conn.execute(sa.text("""
+            conn.execute(
+                sa.text("""
                 CREATE TABLE openai_responses (
                     id TEXT PRIMARY KEY,
                     model TEXT
                 )
-            """))
-            conn.execute(sa.text(f"""
+            """)
+            )
+            conn.execute(
+                sa.text(f"""
                 CREATE TABLE "{custom_table}" (
                     id TEXT PRIMARY KEY,
                     model TEXT
                 )
-            """))
+            """)
+            )
 
         cfg = _alembic_cfg(pg_url, x_responses_table=custom_table)
         command.upgrade(cfg, "head")

tests/unit/server/test_server.py

@@ -340,7 +340,10 @@ async def fake_check(config: object) -> None:
             patch("ogx.core.server.server.Stack", FakeStack),
             patch("ogx.core.server.server._check_postgres_schema_versions", side_effect=fake_check),
             patch("ogx.core.server.server.concurrent.futures.ThreadPoolExecutor", _ImmediateExecutor),
-            patch("ogx.core.storage.sqlstore.sqlstore.reset_sqlstore_engines", side_effect=lambda: call_order.append("reset")),
+            patch(
+                "ogx.core.storage.sqlstore.sqlstore.reset_sqlstore_engines",
+                side_effect=lambda: call_order.append("reset"),
+            ),
         ):
             StackApp(config=Mock())