formatting fixes

Signed-off-by: phernandez <paul@basicmachines.co>

Author: phernandez

benchmarks/src/basic_memory_benchmarks/cli.py

@@ -153,7 +153,9 @@ def convert_structure_corpus(
         max_workers=max_workers,
     )
 
-    console.print(f"Structured ([green]{mode}[/green]): [cyan]{output_dir}[/cyan] ({doc_count} docs)")
+    console.print(
+        f"Structured ([green]{mode}[/green]): [cyan]{output_dir}[/cyan] ({doc_count} docs)"
+    )
     console.print(f"Extractor: [green]{extractor}[/green]")
     if cats:
         console.print(f"Filtered to categories: {sorted(cats)}")

benchmarks/src/basic_memory_benchmarks/models.py

@@ -177,8 +177,12 @@ class ProviderDiagnosis(BaseModel):
     truly_missed: int = 0
     # Derived shares over answerable questions (0.0 when answerable == 0):
     qa_accuracy: float = 0.0  # correct / answerable
-    retrieval_ceiling: float = 0.0  # (correct + retrieved_but_unused) / answerable — max QA if answerer were perfect
-    answerer_gap: float = 0.0  # retrieved_but_unused / answerable — headroom left on the table by the answerer
+    retrieval_ceiling: float = (
+        0.0  # (correct + retrieved_but_unused) / answerable — max QA if answerer were perfect
+    )
+    answerer_gap: float = (
+        0.0  # retrieved_but_unused / answerable — headroom left on the table by the answerer
+    )
     retrieval_gap: float = 0.0  # truly_missed / answerable — headroom that needs better retrieval
     answerer_failure_share: float = 0.0  # retrieved_but_unused / (answerable failures) — of what we got wrong, how much was the answerer
     recall_field: str = "recall_at_10"

benchmarks/src/basic_memory_benchmarks/scoring/diagnose.py

@@ -128,8 +128,6 @@ def diagnose_run(
             seen.add(case.provider)
             providers.append(case.provider)
     return [
-        diagnose_provider(
-            qa_cases, retrieval_rows, provider=provider, recall_field=recall_field
-        )
+        diagnose_provider(qa_cases, retrieval_rows, provider=provider, recall_field=recall_field)
         for provider in providers
     ]

benchmarks/test-int/test_compare_regression.py

@@ -3,7 +3,10 @@
 
 import pytest
 
-from basic_memory_benchmarks.reporting.compare import compare_provider_metric, load_retrieval_summary
+from basic_memory_benchmarks.reporting.compare import (
+    compare_provider_metric,
+    load_retrieval_summary,
+)
 
 
 def test_compare_provider_metric(tmp_path: Path) -> None:

benchmarks/test-int/test_runner_orchestration.py

@@ -21,7 +21,11 @@ def search(self, query: str, limit: int, run_config: RunConfig) -> list[SearchHi
         doc_id = self.mapping.get(query)
         if not doc_id:
             return []
-        return [SearchHit(source_doc_id=doc_id, source_path=f"docs/{doc_id}.md", text="match", score=1.0)]
+        return [
+            SearchHit(
+                source_doc_id=doc_id, source_path=f"docs/{doc_id}.md", text="match", score=1.0
+            )
+        ]
 
     def cleanup(self, run_config: RunConfig) -> None:
         _ = run_config

tests/mcp/test_project_context.py

@@ -3199,9 +3199,7 @@ async def fake_index(context=None, force_refresh=False):
     monkeypatch.setattr(project_context, "_ensure_workspace_project_index", fake_index)
 
     # By external_id (the project_id routing path that failed in the field)
-    resolved = await resolve_workspace_project_identifier(
-        "22222222-2222-4222-8222-222222222222"
-    )
+    resolved = await resolve_workspace_project_identifier("22222222-2222-4222-8222-222222222222")
     assert resolved.project.permalink == "manual"
     assert calls == [False, True]
 

tests/mcp/test_tool_write_note.py

@@ -19,29 +19,38 @@
 
 def test_write_note_workspace_project_route_passthrough_without_workspace():
     """Without workspace, the project string passes through unchanged."""
-    assert _compose_workspace_project_route(
-        workspace=None,
-        project="my-project",
-        project_id=None,
-    ) == "my-project"
+    assert (
+        _compose_workspace_project_route(
+            workspace=None,
+            project="my-project",
+            project_id=None,
+        )
+        == "my-project"
+    )
 
 
 def test_write_note_workspace_project_route_combines_workspace_and_project():
     """workspace + project are joined as 'workspace/project'."""
-    assert _compose_workspace_project_route(
-        workspace="acme",
-        project="docs",
-        project_id=None,
-    ) == "acme/docs"
+    assert (
+        _compose_workspace_project_route(
+            workspace="acme",
+            project="docs",
+            project_id=None,
+        )
+        == "acme/docs"
+    )
 
 
 def test_write_note_workspace_project_route_passes_qualified_project_unchanged():
     """A pre-qualified 'workspace/project' string passes through when workspace is None."""
-    assert _compose_workspace_project_route(
-        workspace=None,
-        project="acme/docs",
-        project_id=None,
-    ) == "acme/docs"
+    assert (
+        _compose_workspace_project_route(
+            workspace=None,
+            project="acme/docs",
+            project_id=None,
+        )
+        == "acme/docs"
+    )
 
 
 @pytest.mark.parametrize(

tests/repository/test_fastembed_provider.py

@@ -595,6 +595,7 @@ async def test_fastembed_provider_fails_fast_without_cache_dir(monkeypatch):
 
     assert _SelfHealStubTextEmbedding.construct_count == 1
 
+
 @pytest.mark.asyncio
 async def test_factory_loads_native_model_once_across_repo_constructions(monkeypatch):
     """The native ONNX model must load exactly once per process despite reuse (#872).