fix: resolve linting errors for release preparation

- Replace bare except clauses with Exception in legal_file_inventory.py
- Remove unused variables in test files
- Prepare codebase for v0.14.4 release

Author: jope-bm

legal_file_inventory.py

@@ -95,14 +95,13 @@ def get_file_contributors(self, file_path: str) -> Dict[str, int]:
             blame_output = self.run_git_command(['blame', '--line-porcelain', file_path])
             contributors = {}
 
-            current_commit = None
             for line in blame_output.split('\n'):
                 if line.startswith('author '):
                     author = line[7:]  # Remove 'author ' prefix
                     contributors[author] = contributors.get(author, 0) + 1
 
             return contributors
-        except:
+        except Exception:
             return {}
 
     def get_file_history(self, file_path: str) -> Dict[str, any]:
@@ -134,7 +133,7 @@ def get_file_history(self, file_path: str) -> Dict[str, any]:
                 'last_modified': last_commit or 'Unknown',
                 'commit_count': commit_count
             }
-        except:
+        except Exception:
             return {
                 'created': 'Unknown',
                 'last_modified': 'Unknown', 
@@ -165,7 +164,7 @@ def calculate_file_hash(self, file_path: Path) -> str:
         try:
             with open(file_path, 'rb') as f:
                 return hashlib.sha256(f.read()).hexdigest()
-        except:
+        except Exception:
             return ""
 
     def categorize_file(self, file_path: Path) -> str:
@@ -229,7 +228,7 @@ def scan_repository(self):
                     stat_info = file_path.stat()
                     file_size = stat_info.st_size
                     modified_time = datetime.fromtimestamp(stat_info.st_mtime)
-                except:
+                except Exception:
                     file_size = 0
                     modified_time = datetime.now()
 

src/basic_memory/markdown/markdown_processor.py

@@ -2,7 +2,6 @@
 from typing import Optional
 from collections import OrderedDict
 
-import frontmatter
 from frontmatter import Post
 from loguru import logger
 

tests/db/test_issue_254_foreign_key_constraints.py

@@ -139,7 +139,7 @@ async def test_issue_254_reproduction(project_service: ProjectService, tmp_path)
         "created_at": datetime.now(timezone.utc),
         "updated_at": datetime.now(timezone.utc),
     }
-    entity = await entity_repo.create(entity_data)
+    await entity_repo.create(entity_data)
 
     # This should eventually work without errors once issue #254 is fixed
     #with pytest.raises(Exception) as exc_info:

tests/mcp/test_obsidian_yaml_formatting.py

@@ -1,7 +1,6 @@
 """Integration tests for Obsidian-compatible YAML formatting in write_note tool."""
 
 import pytest
-from pathlib import Path
 
 from basic_memory.mcp.tools import write_note
 
@@ -70,7 +69,7 @@ async def test_write_note_stringified_json_tags(app, project_config):
 @pytest.mark.asyncio
 async def test_write_note_single_tag_yaml_format(app, project_config):
     """Test that single tags are still formatted as YAML lists."""
-    result = await write_note.fn(
+    await write_note.fn(
         title="Single Tag Test",
         folder="test",
         content="Testing single tag formatting",
@@ -88,7 +87,7 @@ async def test_write_note_single_tag_yaml_format(app, project_config):
 @pytest.mark.asyncio
 async def test_write_note_no_tags(app, project_config):
     """Test that notes without tags work normally."""
-    result = await write_note.fn(
+    await write_note.fn(
         title="No Tags Test",
         folder="test", 
         content="Testing note without tags",
@@ -106,7 +105,7 @@ async def test_write_note_no_tags(app, project_config):
 @pytest.mark.asyncio
 async def test_write_note_empty_tags_list(app, project_config):
     """Test that empty tag lists are handled properly."""
-    result = await write_note.fn(
+    await write_note.fn(
         title="Empty Tags Test",
         folder="test",
         content="Testing empty tag list",
@@ -164,7 +163,7 @@ async def test_write_note_update_preserves_yaml_format(app, project_config):
 @pytest.mark.asyncio
 async def test_complex_tags_yaml_format(app, project_config):
     """Test that complex tags with special characters format correctly."""
-    result = await write_note.fn(
+    await write_note.fn(
         title="Complex Tags Test",
         folder="test",
         content="Testing complex tag formats",

tests/utils/test_frontmatter_obsidian_compatible.py

@@ -1,6 +1,5 @@
 """Tests for Obsidian-compatible YAML frontmatter formatting."""
 
-import pytest
 import frontmatter
 
 from basic_memory.file_utils import dump_frontmatter