docs: update v0.19.0 release notes and apply formatting fixes

Add #634 (stale schema metadata) to bug fixes section.
Apply ruff formatting to schema.py, write_note.py, test files.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: phernandez <paul@basicmachines.co>

Author: phernandez

docs/releases/v0.19.0.md

@@ -255,6 +255,7 @@ making all major CLI commands scriptable for CI pipelines and automation.
 - **#31**: `search_notes` returns cluttered observation/relation-level results — now defaults to entity-level results
 - **#28**: `schema_infer` and `schema_diff` return raw Pydantic models as "undefined" in LLM output — added markdown formatters
 - Fix `schema_validate` identifier resolution (now uses LinkResolver) and text rendering (markdown formatter)
+- **#634**: `schema_validate` and `schema_diff` use stale database metadata instead of reading schema definitions from file — now reads frontmatter directly from the file with fallback to database metadata
 - Fix `Post(**metadata)` crash when frontmatter contains `content` or `handler` keys
 - Fix list-valued frontmatter fields (`title`, `type`) crashing on `.strip()` — now coerced to strings
 - Cap sqlite-vec knn `k` parameter at 4096 to prevent backend errors

src/basic_memory/mcp/tools/schema.py

@@ -84,8 +84,10 @@ def _format_inference_report(report: InferenceReport) -> str:
             req_marker = "required" if f.name in report.suggested_required else "optional"
             samples = ", ".join(f.sample_values[:3]) if f.sample_values else ""
             sample_str = f" (e.g. {samples})" if samples else ""
-            lines.append(f"- **{f.name}** ({f.source}) — {pct} ({f.count}/{f.total}) "
-                         f"[{req_marker}]{sample_str}")
+            lines.append(
+                f"- **{f.name}** ({f.source}) — {pct} ({f.count}/{f.total}) "
+                f"[{req_marker}]{sample_str}"
+            )
         lines.append("")
 
     # --- Excluded fields ---

src/basic_memory/mcp/tools/write_note.py

@@ -144,8 +144,7 @@ async def write_note(
     # Trigger: caller omitted the parameter (None)
     # Why: lets users set a global default without breaking per-call overrides
     effective_overwrite = (
-        overwrite if overwrite is not None
-        else ConfigManager().config.write_note_overwrite_default
+        overwrite if overwrite is not None else ConfigManager().config.write_note_overwrite_default
     )
 
     async with get_project_client(project, workspace, context) as (client, active_project):
@@ -230,9 +229,7 @@ async def write_note(
                             "action": "conflict",
                             "error": "NOTE_ALREADY_EXISTS",
                         }
-                    return _format_overwrite_error(
-                        title, entity.permalink, active_project.name
-                    )
+                    return _format_overwrite_error(title, entity.permalink, active_project.name)
 
                 logger.debug(f"Entity exists, updating instead permalink={entity.permalink}")
                 try:

tests/markdown/test_date_frontmatter_parsing.py

@@ -16,7 +16,8 @@
 def test_file_with_date(tmp_path):
     """Create a test file with date fields in frontmatter."""
     test_file = tmp_path / "test_note.md"
-    test_file.write_text(dedent("""\
+    test_file.write_text(
+        dedent("""\
         ---
         title: Test Note
         date: 2025-10-24
@@ -29,15 +30,17 @@ def test_file_with_date(tmp_path):
         # Test Content
 
         This file has date fields in frontmatter that PyYAML will parse as datetime.date objects.
-    """))
+    """)
+    )
     return test_file
 
 
 @pytest.fixture
 def test_file_with_date_in_tags(tmp_path):
     """Create a test file with a date value in tags (edge case)."""
     test_file = tmp_path / "test_note_date_tags.md"
-    test_file.write_text(dedent("""\
+    test_file.write_text(
+        dedent("""\
         ---
         title: Test Note with Date Tags
         tags: 2025-10-24
@@ -46,15 +49,17 @@ def test_file_with_date_in_tags(tmp_path):
         # Test Content
 
         This file has a date value as tags, which will be parsed as datetime.date.
-    """))
+    """)
+    )
     return test_file
 
 
 @pytest.fixture
 def test_file_with_dates_in_tag_list(tmp_path):
     """Create a test file with dates in a tag list (edge case)."""
     test_file = tmp_path / "test_note_dates_in_list.md"
-    test_file.write_text(dedent("""\
+    test_file.write_text(
+        dedent("""\
         ---
         title: Test Note with Dates in Tags List
         tags:
@@ -66,7 +71,8 @@ def test_file_with_dates_in_tag_list(tmp_path):
         # Test Content
 
         This file has date values mixed into tags list.
-    """))
+    """)
+    )
     return test_file
 
 
@@ -141,7 +147,8 @@ async def test_parse_file_with_list_frontmatter_fields(tmp_path):
     to be strings, causing 'list' object has no attribute 'strip'.
     """
     test_file = tmp_path / "test_list_fields.md"
-    test_file.write_text(dedent("""\
+    test_file.write_text(
+        dedent("""\
         ---
         title:
           - Week 2 Discussion Post
@@ -160,7 +167,8 @@ async def test_parse_file_with_list_frontmatter_fields(tmp_path):
         # Content
 
         Some body text.
-    """))
+    """)
+    )
 
     parser = EntityParser(tmp_path)
     entity_markdown = await parser.parse_file(test_file)
@@ -192,14 +200,16 @@ async def test_parse_file_with_list_frontmatter_fields(tmp_path):
 async def test_parse_file_with_list_title_single_item(tmp_path):
     """Test that a single-item list title is coerced to a plain string."""
     test_file = tmp_path / "test_single_list_title.md"
-    test_file.write_text(dedent("""\
+    test_file.write_text(
+        dedent("""\
         ---
         title:
           - My Single Title
         ---
 
         # Content
-    """))
+    """)
+    )
 
     parser = EntityParser(tmp_path)
     entity_markdown = await parser.parse_file(test_file)
@@ -218,7 +228,8 @@ async def test_parse_file_with_various_yaml_types(tmp_path):
     when code expects strings and calls .strip().
     """
     test_file = tmp_path / "test_yaml_types.md"
-    test_file.write_text(dedent("""\
+    test_file.write_text(
+        dedent("""\
         ---
         title: Test YAML Types
         date: 2025-10-24
@@ -235,7 +246,8 @@ async def test_parse_file_with_various_yaml_types(tmp_path):
         # Test Content
 
         This file has various YAML types that need to be normalized.
-    """))
+    """)
+    )
 
     parser = EntityParser(tmp_path)
     entity_markdown = await parser.parse_file(test_file)
@@ -282,7 +294,8 @@ async def test_parse_file_with_datetime_objects(tmp_path):
     with time components (as parsed by PyYAML), ensuring they're converted to ISO format strings.
     """
     test_file = tmp_path / "test_datetime.md"
-    test_file.write_text(dedent("""\
+    test_file.write_text(
+        dedent("""\
         ---
         title: Test Datetime
         created_at: 2025-10-24 14:30:00
@@ -292,7 +305,8 @@ async def test_parse_file_with_datetime_objects(tmp_path):
         # Test Content
 
         This file has datetime values in frontmatter that PyYAML will parse as datetime objects.
-    """))
+    """)
+    )
 
     parser = EntityParser(tmp_path)
     entity_markdown = await parser.parse_file(test_file)
@@ -322,7 +336,8 @@ async def test_parse_file_with_reserved_frontmatter_field_content(tmp_path):
     the 'content' positional argument.
     """
     test_file = tmp_path / "topic-note-template.md"
-    test_file.write_text(dedent("""\
+    test_file.write_text(
+        dedent("""\
         ---
         title: Topic Note Template
         content: Template for topic notes
@@ -332,7 +347,8 @@ async def test_parse_file_with_reserved_frontmatter_field_content(tmp_path):
         # Template Body
 
         Actual body content here.
-    """))
+    """)
+    )
 
     parser = EntityParser(tmp_path)
     entity_markdown = await parser.parse_file(test_file)

tests/mcp/test_project_context.py

@@ -515,9 +515,7 @@ async def fake_factory():
 
         # Patch workspace resolution to fail if called — factory mode should skip it
         async def fail_if_called(**kwargs):  # pragma: no cover
-            raise AssertionError(
-                "resolve_workspace_parameter must not be called in factory mode"
-            )
+            raise AssertionError("resolve_workspace_parameter must not be called in factory mode")
 
         monkeypatch.setattr(
             "basic_memory.mcp.project_context.resolve_workspace_parameter",