fix formatting and merge

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

Author: phernandez

src/basic_memory/api/routers/project_router.py

@@ -113,7 +113,7 @@ async def sync_project(
     force_full: bool = Query(
         False, description="Force full scan, bypassing watermark optimization"
     ),
-    run_in_background: bool = Query(True, description="Run in background")
+    run_in_background: bool = Query(True, description="Run in background"),
 ):
     """Force project filesystem sync to database.
 
@@ -142,7 +142,9 @@ async def sync_project(
             "message": f"Filesystem sync initiated for project '{project_config.name}'",
         }
     else:
-        report = await sync_service.sync(project_config.home, project_config.name, force_full=force_full)
+        report = await sync_service.sync(
+            project_config.home, project_config.name, force_full=force_full
+        )
         logger.info(
             f"Filesystem sync completed for project: {project_config.name} (force_full={force_full})"
         )

src/basic_memory/file_utils.py

@@ -213,7 +213,7 @@ def dump_frontmatter(post: frontmatter.Post) -> str:
         sort_keys=False,
         allow_unicode=True,
         default_flow_style=False,
-        Dumper=yaml.SafeDumper
+        Dumper=yaml.SafeDumper,
     )
 
     # Construct the final markdown with frontmatter

src/basic_memory/mcp/tools/write_note.py

@@ -16,9 +16,6 @@
 # Define TagType as a Union that can accept either a string or a list of strings or None
 TagType = Union[List[str], str, None]
 
-# Define TagType as a Union that can accept either a string or a list of strings or None
-TagType = Union[List[str], str, None]
-
 
 @mcp.tool(
     description="Create or update a markdown note. Returns a markdown formatted summary of the semantic content.",
@@ -29,7 +26,7 @@ async def write_note(
     folder: str,
     project: Optional[str] = None,
     tags: list[str] | str | None = None,
-    entity_type: str = "note",
+    note_type: str = "note",
     context: Context | None = None,
 ) -> str:
     """Write a markdown note to the knowledge base.

tests/api/test_project_router.py

@@ -588,9 +588,7 @@ async def test_sync_project_endpoint_foreground_with_changes(
         assert "deleted" in data
 
         # At least one of these should have changes
-        has_changes = (
-            len(data["new"]) > 0 or len(data["modified"]) > 0 or len(data["deleted"]) > 0
-        )
+        has_changes = len(data["new"]) > 0 or len(data["modified"]) > 0 or len(data["deleted"]) > 0
         assert has_changes or data["total"] >= 0  # Either changes detected or empty sync is valid
 
     finally:

tests/services/test_entity_service.py

@@ -1315,7 +1315,9 @@ async def test_edit_entity_replace_section_strips_duplicate_header(
 
     # Count occurrences of "## Testing" - should only be 1
     testing_header_count = file_content.count("## Testing")
-    assert testing_header_count == 1, f"Expected 1 '## Testing' header, found {testing_header_count}"
+    assert testing_header_count == 1, (
+        f"Expected 1 '## Testing' header, found {testing_header_count}"
+    )
 
     assert "New content for testing section" in file_content
     assert "Original content" not in file_content