fix: respect frontmatter entity_type in write_note operations

- Add frontmatter type extraction in EntityService.create_entity() and update_entity()
- Override schema.entity_type when content frontmatter contains 'type' field
- Fix lint error in alembic/env.py with noqa comment
- Update test formatting for consistency
- Ensures write_note respects both parameter and frontmatter entity_type values

Fixes #144

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: phernandez

src/basic_memory/services/entity_service.py

@@ -117,10 +117,15 @@ async def create_entity(self, schema: EntitySchema) -> EntityModel:
                 f"file for entity {schema.folder}/{schema.title} already exists: {file_path}"
             )
 
-        # Parse content frontmatter to check for user-specified permalink
+        # Parse content frontmatter to check for user-specified permalink and entity_type
         content_markdown = None
         if schema.content and has_frontmatter(schema.content):
             content_frontmatter = parse_frontmatter(schema.content)
+
+            # If content has entity_type/type, use it to override the schema entity_type
+            if "type" in content_frontmatter:
+                schema.entity_type = content_frontmatter["type"]
+
             if "permalink" in content_frontmatter:
                 # Create a minimal EntityMarkdown object for permalink resolution
                 from basic_memory.markdown.schemas import EntityFrontmatter
@@ -172,10 +177,15 @@ async def update_entity(self, entity: EntityModel, schema: EntitySchema) -> Enti
         # Read existing frontmatter from the file if it exists
         existing_markdown = await self.entity_parser.parse_file(file_path)
 
-        # Parse content frontmatter to check for user-specified permalink
+        # Parse content frontmatter to check for user-specified permalink and entity_type
         content_markdown = None
         if schema.content and has_frontmatter(schema.content):
             content_frontmatter = parse_frontmatter(schema.content)
+
+            # If content has entity_type/type, use it to override the schema entity_type
+            if "type" in content_frontmatter:
+                schema.entity_type = content_frontmatter["type"]
+
             if "permalink" in content_frontmatter:
                 # Create a minimal EntityMarkdown object for permalink resolution
                 from basic_memory.markdown.schemas import EntityFrontmatter

tests/mcp/test_tool_write_note.py

@@ -622,10 +622,10 @@ async def test_write_note_update_existing_with_different_entity_type(app):
     assert "- guide" in content
 
 
-@pytest.mark.asyncio  
+@pytest.mark.asyncio
 async def test_write_note_respects_frontmatter_entity_type(app):
     """Test that entity_type in frontmatter is respected when parameter is not provided.
-    
+
     This verifies that when write_note is called without entity_type parameter,
     but the content includes frontmatter with a 'type' field, that type is respected
     instead of defaulting to 'note'.
@@ -645,11 +645,7 @@ async def test_write_note_respects_frontmatter_entity_type(app):
         """).strip()
 
     # Call write_note without entity_type parameter - it should respect frontmatter type
-    result = await write_note.fn(
-        title="Test Guide",
-        folder="guides", 
-        content=note
-    )
+    result = await write_note.fn(title="Test Guide", folder="guides", content=note)
 
     assert result
     assert "# Created note" in result