style: format + autofix drift; re-apply test_w805 orphaned-return fix (lost to add-glob)

Author: stoa-automerge

tests/test_cmd_stats.py

@@ -19,11 +19,14 @@ def test_count_git_commits_missing_timestamp_returns_zero() -> None:
     conn = sqlite3.connect(":memory:")
     conn.execute("CREATE TABLE git_commits (id INTEGER PRIMARY KEY)")
 
-    assert _count_git_commits(
-        conn,
-        "SELECT COUNT(*) FROM git_commits WHERE timestamp >= ?",
-        (0,),
-    ) == 0
+    assert (
+        _count_git_commits(
+            conn,
+            "SELECT COUNT(*) FROM git_commits WHERE timestamp >= ?",
+            (0,),
+        )
+        == 0
+    )
 
 
 def test_count_git_commits_unexpected_operational_error_raises() -> None:

tests/test_cmd_understand_tour_graph_errors.py

@@ -17,7 +17,9 @@ def boom(_conn):
     monkeypatch.setattr(builder, "build_symbol_graph", boom)
     monkeypatch.setattr(cmd_tour, "_entry_points", lambda _conn: entry_points)
     monkeypatch.setattr(cmd_tour, "_reading_order", lambda _conn, _graph: pytest.fail("graph fallback not used"))
-    monkeypatch.setattr(cmd_tour, "_top_symbols", lambda _conn, _graph, limit=10: pytest.fail("graph fallback not used"))
+    monkeypatch.setattr(
+        cmd_tour, "_top_symbols", lambda _conn, _graph, limit=10: pytest.fail("graph fallback not used")
+    )
 
     assert cmd_understand._gather_tour_data(object(), None) == {
         "reading_order": [],

tests/test_cmd_why_fail_exception_scope.py

@@ -3,7 +3,6 @@
 import ast
 from pathlib import Path
 
-
 _MODULE = Path("src/roam/commands/cmd_why_fail.py")
 
 
@@ -23,12 +22,7 @@ def test_why_fail_bfs_catches_only_missing_networkx_nodes() -> None:
     for node in ast.walk(tree):
         if not isinstance(node, ast.Try):
             continue
-        call_names = [
-            _name(call.func)
-            for stmt in node.body
-            for call in ast.walk(stmt)
-            if isinstance(call, ast.Call)
-        ]
+        call_names = [_name(call.func) for stmt in node.body for call in ast.walk(stmt) if isinstance(call, ast.Call)]
         if "nx.single_source_shortest_path_length" in call_names:
             bfs_handlers.extend(node.handlers)
 

tests/test_constitution.py

@@ -31,14 +31,14 @@
 
 from roam.constitution.loader import (  # noqa: E402
     Constitution,
+    _deprecated_commands,
+    _known_commands,
     _project_name,
     apply_constitution,
     check_constitution,
     constitution_path,
     init_constitution,
     load_constitution,
-    _deprecated_commands,
-    _known_commands,
 )
 
 # ---------------------------------------------------------------------------

tests/test_flag_dead.py

@@ -90,8 +90,7 @@ def test_index_read_database_error_falls_back_to_filesystem(self, tmp_path, monk
         project = tmp_path / "project"
         project.mkdir()
         (project / "app.py").write_text(
-            "def enabled(client):\n"
-            "    return client.variation('stale-flag', False)\n",
+            "def enabled(client):\n    return client.variation('stale-flag', False)\n",
             encoding="utf-8",
         )
 

tests/test_guard_pr_and_github_check.py

@@ -14,8 +14,8 @@
 import pytest
 from click.testing import CliRunner
 
-from roam.commands import cmd_guard_pr
 from roam.cli import cli
+from roam.commands import cmd_guard_pr
 from roam.github_check import (
     SUMMARY_BYTE_CAP,
     VERDICT_TO_CONCLUSION,

tests/test_health_baseline.py

@@ -266,14 +266,12 @@ def test_health_baseline_last_resolves_most_recent(cli_runner, indexed_project,
 
 def test_health_baseline_auto_filesystem_root_error_falls_back(monkeypatch):
     """--baseline auto falls back to cwd only for filesystem root failures."""
-    from roam.commands import cmd_health
     import roam.db.connection as db_connection
+    from roam.commands import cmd_health
 
     conn = sqlite3.connect(":memory:")
     conn.row_factory = sqlite3.Row
-    conn.execute(
-        "CREATE TABLE snapshots (timestamp INTEGER, git_branch TEXT, git_commit TEXT, health_score INTEGER)"
-    )
+    conn.execute("CREATE TABLE snapshots (timestamp INTEGER, git_branch TEXT, git_commit TEXT, health_score INTEGER)")
     conn.execute(
         "INSERT INTO snapshots (timestamp, git_branch, git_commit, health_score) VALUES (?, ?, ?, ?)",
         (123, "main", "abc1234", 90),
@@ -293,14 +291,12 @@ def _raise_oserror():
 
 def test_health_baseline_auto_project_root_programmer_error_propagates(monkeypatch):
     """--baseline auto must not swallow bug-class project-root failures."""
-    from roam.commands import cmd_health
     import roam.db.connection as db_connection
+    from roam.commands import cmd_health
 
     conn = sqlite3.connect(":memory:")
     conn.row_factory = sqlite3.Row
-    conn.execute(
-        "CREATE TABLE snapshots (timestamp INTEGER, git_branch TEXT, git_commit TEXT, health_score INTEGER)"
-    )
+    conn.execute("CREATE TABLE snapshots (timestamp INTEGER, git_branch TEXT, git_commit TEXT, health_score INTEGER)")
 
     def _raise_typeerror():
         raise TypeError("bad find_project_root refactor")

tests/test_runtime.py

@@ -416,9 +416,7 @@ def test_cli_ingest_trace_callback_name_is_distinct_from_mcp_wrapper(self):
         from roam.cli import _COMMANDS
         from roam.commands.cmd_ingest_trace import ingest_trace_cmd
 
-        mcp_function_names = {
-            name for name, obj in inspect.getmembers(mcp_server, inspect.isfunction)
-        }
+        mcp_function_names = {name for name, obj in inspect.getmembers(mcp_server, inspect.isfunction)}
         assert _COMMANDS["ingest-trace"] == ("roam.commands.cmd_ingest_trace", "ingest_trace_cmd")
         assert ingest_trace_cmd.callback.__name__ == "ingest_trace_cmd"
         assert ingest_trace_cmd.callback.__name__ not in mcp_function_names

tests/test_stale_refs.py

@@ -2488,7 +2488,9 @@ def _raise_type_error(*_args, **_kwargs):
 
         monkeypatch.setattr(stale_refs, "_check_one_external_url", _raise_type_error)
         with pytest.raises(TypeError, match="bad worker"):
-            stale_refs._check_external_urls_parallel([("https://example.com", "README.md", 1)], timeout=0.5, concurrency=1)
+            stale_refs._check_external_urls_parallel(
+                [("https://example.com", "README.md", 1)], timeout=0.5, concurrency=1
+            )
 
 
 class TestStaleRefsWatchHelpersComposition:

tests/test_w607_e_cmd_search_warnings_out_envelope.py

@@ -644,8 +644,7 @@ def test_caller_unmodified():
         (
             n
             for n in tree.body
-            if isinstance(n, ast.Assign)
-            and any(isinstance(t, ast.Name) and t.id == "search" for t in n.targets)
+            if isinstance(n, ast.Assign) and any(isinstance(t, ast.Name) and t.id == "search" for t in n.targets)
         ),
         None,
     )