release: Version 2.0.9 - CI passing with all linting fixes

Patrick Roebuck · claude · Patrick Roebuck · commit c953d747d3b5 · 2025-11-11T14:30:52.000-05:00
Final release with complete CI/linting fixes: - Ruff linting: All checks pass (production + tests) - Black formatting: All files reformatted to standard - Test-specific exceptions: N806 variable naming allowed This version resolves all CI failures and maintains code quality without changing any functional behavior. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
diff --git a/memdocs/__init__.py b/memdocs/__init__.py
@@ -4,7 +4,7 @@
 Generate machine-friendly docs from code changes without bureaucratic overhead.
 """
 
-__version__ = "2.0.8"
+__version__ = "2.0.9"
 __author__ = "Patrick Roebuck"
 __license__ = "Apache-2.0"
 
diff --git a/memdocs/cli_modules/commands/init_cmd.py b/memdocs/cli_modules/commands/init_cmd.py
@@ -91,17 +91,15 @@ def _setup_mcp_infrastructure(cwd: Path) -> None:
                 "args": ["serve", "--mcp"],
                 "isBackground": True,
                 "problemMatcher": [],
-                "runOptions": {
-                    "runOn": "folderOpen"
-                },
+                "runOptions": {"runOn": "folderOpen"},
                 "presentation": {
                     "echo": False,
                     "reveal": "never",
                     "focus": False,
                     "panel": "shared",
                     "showReuseMessage": False,
-                    "clear": False
-                }
+                    "clear": False,
+                },
             },
             {
                 "label": "Update MemDocs Memory",
@@ -112,8 +110,8 @@ def _setup_mcp_infrastructure(cwd: Path) -> None:
                     "echo": True,
                     "reveal": "always",
                     "focus": False,
-                    "panel": "shared"
-                }
+                    "panel": "shared",
+                },
             },
             {
                 "label": "Export MemDocs for Cursor",
@@ -124,8 +122,8 @@ def _setup_mcp_infrastructure(cwd: Path) -> None:
                     "echo": True,
                     "reveal": "always",
                     "focus": False,
-                    "panel": "shared"
-                }
+                    "panel": "shared",
+                },
             },
             {
                 "label": "MemDocs Stats",
@@ -136,20 +134,17 @@ def _setup_mcp_infrastructure(cwd: Path) -> None:
                     "echo": True,
                     "reveal": "always",
                     "focus": True,
-                    "panel": "shared"
-                }
-            }
-        ]
+                    "panel": "shared",
+                },
+            },
+        ],
     }
 
     tasks_file.write_text(json.dumps(tasks_config, indent=2), encoding="utf-8")
     out.success(f"Created [green]{tasks_file}[/green]")
 
     # Create settings.json for auto-start
-    settings_config = {
-        "task.autoDetect": "on",
-        "task.allowAutomaticTasks": "on"
-    }
+    settings_config = {"task.autoDetect": "on", "task.allowAutomaticTasks": "on"}
 
     # Merge with existing settings if they exist
     if settings_file.exists():
diff --git a/memdocs/cli_modules/commands/query_cmd.py b/memdocs/cli_modules/commands/query_cmd.py
@@ -13,6 +13,7 @@
 def _get_memory_indexer():
     """Lazy import to avoid circular dependency."""
     from memdocs import cli
+
     return cli.MemoryIndexer
 
 
diff --git a/memdocs/cli_modules/commands/review_cmd.py b/memdocs/cli_modules/commands/review_cmd.py
@@ -76,6 +76,7 @@ def _is_git_repo() -> bool:
 def _get_cli_classes():
     """Lazy import to avoid circular dependency."""
     from memdocs import cli
+
     return cli.Extractor, cli.MemoryIndexer, cli.PolicyEngine, cli.Summarizer
 
 
@@ -191,9 +192,7 @@ def review(
 
         # Override config with CLI args (with validation)
         if max_files:
-            validated_max_files = ConfigValidator.validate_positive_int(
-                max_files, "max_files"
-            )
+            validated_max_files = ConfigValidator.validate_positive_int(max_files, "max_files")
             doc_config.policies.max_files_without_force = validated_max_files
         if escalate_on:
             # Validate and sanitize escalation rules
@@ -232,7 +231,9 @@ def review(
             sys.exit(1)
 
         # Get CLI classes (lazy import)
-        extractor_class, memory_indexer_class, policy_engine_class, summarizer_class = _get_cli_classes()
+        extractor_class, memory_indexer_class, policy_engine_class, summarizer_class = (
+            _get_cli_classes()
+        )
 
         out.print_header("MemDocs Review")
         out.step(f"Reviewing {len(paths)} path(s) for [cyan]{event}[/cyan] event")
@@ -266,7 +267,9 @@ def review(
 
         # Summarize with AI
         with out.spinner("Generating documentation with Claude Sonnet 4.5"):
-            summarizer = summarizer_class(model=doc_config.ai.model, max_tokens=doc_config.ai.max_tokens)
+            summarizer = summarizer_class(
+                model=doc_config.ai.model, max_tokens=doc_config.ai.max_tokens
+            )
             doc_index, markdown_summary = summarizer.summarize(context, scope)
 
         out.success("Documentation generated")
diff --git a/memdocs/cli_modules/commands/serve_cmd.py b/memdocs/cli_modules/commands/serve_cmd.py
@@ -210,9 +210,7 @@ def serve(
     """
     # Check if memory exists
     if not docs_dir.exists() and not memory_dir.exists():
-        out.warning(
-            "No MemDocs memory found. Run 'memdocs review' first to generate memory."
-        )
+        out.warning("No MemDocs memory found. Run 'memdocs review' first to generate memory.")
         out.info("Starting server anyway for testing...")
 
     # Daemon mode
diff --git a/memdocs/cli_modules/commands/stats_cmd.py b/memdocs/cli_modules/commands/stats_cmd.py
@@ -14,6 +14,7 @@
 def _get_memory_indexer():
     """Lazy import to avoid circular dependency."""
     from memdocs import cli
+
     return cli.MemoryIndexer
 
 
diff --git a/memdocs/embeddings.py b/memdocs/embeddings.py
@@ -129,9 +129,7 @@ def chunk_document(text: str, max_tokens: int = 512, overlap: int = 50) -> list[
         >>> # Each chunk has <= 512 tokens with 50 token overlap
     """
     if max_tokens <= overlap:
-        raise ValueError(
-            f"max_tokens ({max_tokens}) must be greater than overlap ({overlap})"
-        )
+        raise ValueError(f"max_tokens ({max_tokens}) must be greater than overlap ({overlap})")
 
     # Use cl100k_base encoding (used by OpenAI's text-embedding-ada-002 and similar models)
     # This encoding is also suitable for sentence-transformers models as an approximation
diff --git a/memdocs/mcp_server.py b/memdocs/mcp_server.py
@@ -257,7 +257,7 @@ async def serve_mcp() -> None:
 
     # Use secure temporary file for logging
     log_file = tempfile.NamedTemporaryFile(
-        mode='w', prefix='memdocs-mcp-', suffix='.log', delete=False
+        mode="w", prefix="memdocs-mcp-", suffix=".log", delete=False
     )
     logging.basicConfig(level=logging.INFO, filename=log_file.name)
 
diff --git a/memdocs/security.py b/memdocs/security.py
@@ -44,18 +44,15 @@ def validate_path(path: Path, base_dir: Path | None = None) -> Path:
                 except ValueError:
                     raise SecurityError(
                         f"Attempts to escape base directory '{base_dir}'. Path traversal attacks are not allowed.",
-                        path
+                        path,
                     )
 
             # Check for suspicious patterns
             path_str = str(path)
             if ".." in path_str:
                 # resolve() should handle this, but double-check
                 if ".." in str(abs_path.relative_to(abs_path.anchor)):
-                    raise SecurityError(
-                        "Contains suspicious '..' traversal sequences",
-                        path
-                    )
+                    raise SecurityError("Contains suspicious '..' traversal sequences", path)
 
             # Check for null bytes (path injection)
             if "\x00" in path_str:
@@ -95,7 +92,7 @@ def validate_write_path(path: Path, allowed_dirs: list[Path]) -> Path:
         allowed_str = ", ".join(str(d) for d in allowed_dirs)
         raise SecurityError(
             f"Not within allowed directories: {allowed_str}. Writing to arbitrary locations is not permitted.",
-            path
+            path,
         )
 
 
@@ -313,9 +310,7 @@ def validate_scope_level(scope: str) -> str:
         """
         valid_scopes = ["file", "module", "repo"]
         if scope not in valid_scopes:
-            raise ValueError(
-                f"Invalid scope '{scope}'. Valid scopes: {', '.join(valid_scopes)}"
-            )
+            raise ValueError(f"Invalid scope '{scope}'. Valid scopes: {', '.join(valid_scopes)}")
         return scope
 
     @staticmethod
diff --git a/memdocs/summarize.py b/memdocs/summarize.py
@@ -102,7 +102,12 @@ class Summarizer:
 
 Generate the YAML now:"""
 
-    def __init__(self, api_key: str | None = None, model: str = "claude-sonnet-4-5-20250929", max_tokens: int = DEFAULT_MAX_TOKENS):
+    def __init__(
+        self,
+        api_key: str | None = None,
+        model: str = "claude-sonnet-4-5-20250929",
+        max_tokens: int = DEFAULT_MAX_TOKENS,
+    ):
         """Initialize summarizer.
 
         Args:
diff --git a/memdocs/symbol_extractor.py b/memdocs/symbol_extractor.py
@@ -91,9 +91,7 @@ def __init__(self):
             # Additional languages can be added here
         }
 
-    def extract_from_file(
-        self, file_path: Path, language: Language | None = None
-    ) -> list[Symbol]:
+    def extract_from_file(self, file_path: Path, language: Language | None = None) -> list[Symbol]:
         """
         Extract symbols from a source file.
 
@@ -404,9 +402,7 @@ def _extract_javascript_symbols(self, code: str) -> list[ExtractedSymbol]:
         return []
 
 
-def extract_symbols_for_memdocs(
-    file_path: Path, language: Language | None = None
-) -> SymbolsOutput:
+def extract_symbols_for_memdocs(file_path: Path, language: Language | None = None) -> SymbolsOutput:
     """
     Convenience function to extract symbols in DocInt format.
 
diff --git a/memdocs/wizards/empathy_sync.py b/memdocs/wizards/empathy_sync.py
@@ -161,6 +161,7 @@ def _is_local_empathy(self) -> bool:
         try:
             # Check if EmpathyService module is available
             import importlib.util
+
             spec = importlib.util.find_spec("app.backend.services.empathy_service")
             return spec is not None
         except (ImportError, ModuleNotFoundError):
diff --git a/pyproject.toml b/pyproject.toml
@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "memdocs"
-version = "2.0.8"
+version = "2.0.9"
 description = "Persistent memory management for AI projects - Git-native documentation intelligence"
 authors = [{name = "Patrick Roebuck", email = "patrick.roebuck@smartaimemory.com"}]
 license = {text = "Apache-2.0"}
diff --git a/tests/integration/test_cli.py b/tests/integration/test_cli.py
@@ -336,7 +336,7 @@ def test_review_command_with_path_mock(
         mock_extractor_class,
         mock_summarizer_class,
         cli_runner: CliRunner,
-        temp_repo: Path
+        temp_repo: Path,
     ):
         """Test review command with mocked dependencies."""
         from datetime import datetime, timezone
diff --git a/tests/unit/test_embeddings.py b/tests/unit/test_embeddings.py
@@ -74,7 +74,8 @@ def test_chunk_with_special_characters(self):
 
     def test_chunk_code_text(self):
         """Test chunking programming code."""
-        code = '''
+        code = (
+            '''
 def calculate_fibonacci(n):
     """Calculate the nth Fibonacci number."""
     if n <= 1:
@@ -92,7 +93,9 @@ def filter(self, condition):
 
     def map(self, transform):
         return [transform(item) for item in self.data]
-''' * 10  # Repeat to ensure multiple chunks
+'''
+            * 10
+        )  # Repeat to ensure multiple chunks
 
         chunks = chunk_document(code, max_tokens=200, overlap=30)
 
@@ -108,11 +111,14 @@ def map(self, transform):
     def test_chunk_accurate_vs_old_approximation(self):
         """Test that tiktoken gives more accurate results than word approximation."""
         # Text with varying token-to-word ratios
-        text = """
+        text = (
+            """
         This is a test. Some words are single tokens, but compound-words and
         special-characters create different token counts than simple word counts.
         The old approximation of 1 token ≈ 0.75 words was inaccurate.
-        """ * 50
+        """
+            * 50
+        )
 
         max_tokens = 100
         chunks = chunk_document(text, max_tokens=max_tokens, overlap=20)
@@ -173,7 +179,8 @@ def test_chunk_very_large_document(self):
 
     def test_chunk_with_markdown(self):
         """Test chunking markdown documentation."""
-        markdown = """
+        markdown = (
+            """
 # Main Title
 
 ## Section 1
@@ -196,7 +203,9 @@ def example():
 | Column 1 | Column 2 |
 |----------|----------|
 | Data 1   | Data 2   |
-""" * 20  # Repeat for multiple chunks
+"""
+            * 20
+        )  # Repeat for multiple chunks
 
         chunks = chunk_document(markdown, max_tokens=256, overlap=30)
 
diff --git a/tests/unit/test_extract.py b/tests/unit/test_extract.py
@@ -292,7 +292,9 @@ def test_parse_pyproject_toml_poetry(self, temp_repo: Path):
         assert any("anthropic" in d for d in context.dependencies)
         assert any("click" in d for d in context.dependencies)
         assert any("pydantic" in d for d in context.dependencies)
-        assert not any("python" in d.lower() and d.startswith("python") for d in context.dependencies)
+        assert not any(
+            "python" in d.lower() and d.startswith("python") for d in context.dependencies
+        )
 
     def test_parse_package_json(self, temp_repo: Path):
         """Test parsing package.json."""
diff --git a/tests/unit/test_mcp_server.py b/tests/unit/test_mcp_server.py
diff --git a/tests/unit/test_security.py b/tests/unit/test_security.py

Original file line number	Diff line number	Diff line change
`@@ -257,7 +257,7 @@ async def serve_mcp() -> None:`
`257`	`257`
`258`	`258`	`# Use secure temporary file for logging`
`259`	`259`	`log_file = tempfile.NamedTemporaryFile(`
`260`		`- mode='w', prefix='memdocs-mcp-', suffix='.log', delete=False`
	`260`	`+ mode="w", prefix="memdocs-mcp-", suffix=".log", delete=False`
`261`	`261`	`)`
`262`	`262`	`logging.basicConfig(level=logging.INFO, filename=log_file.name)`
`263`	`263`