fix: Resolve ruff linting errors for CI

- Fix A002 (shadowing builtin 'format'): Rename format params to export_format/output_format
- Fix N806 (variable naming): Rename PascalCase class variables to snake_case
- Fix UP045: Use 'X | None' instead of Optional[X]
- Fix F401: Remove unused imports (parse_qs, os)
- Fix B009: Replace getattr with direct attribute access

All critical linting issues resolved for GitHub Actions CI to pass.

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

Co-Authored-By: Claude <[email protected]>

Author: Patrick Roebuck

memdocs/cli_modules/commands/export_cmd.py

@@ -15,7 +15,7 @@
 
 
 @click.command()
-@click.argument("format", type=click.Choice(["cursor", "claude", "continue"]))
+@click.argument("export_format", type=click.Choice(["cursor", "claude", "continue"]))
 @click.option(
     "--output",
     type=click.Path(path_type=Path),
@@ -32,7 +32,7 @@
     default=True,
     help="Include code symbols in export",
 )
-def export(format: str, output: Path | None, docs_dir: Path, include_symbols: bool) -> None:
+def export(export_format: str, output: Path | None, docs_dir: Path, include_symbols: bool) -> None:
     """Export docs for AI assistants.
 
     Examples:
@@ -42,7 +42,7 @@ def export(format: str, output: Path | None, docs_dir: Path, include_symbols: bo
         memdocs export continue
     """
     out.print_header("MemDocs Export")
-    out.step(f"Exporting to [cyan]{format}[/cyan] format")
+    out.step(f"Exporting to [cyan]{export_format}[/cyan] format")
 
     if not docs_dir.exists():
         out.error(f"Docs directory not found: {docs_dir}")
@@ -107,7 +107,7 @@ def export(format: str, output: Path | None, docs_dir: Path, include_symbols: bo
                 graph_section += f"- {feature.get('title', 'Untitled')}\n"
 
     # Format based on target
-    if format == "cursor":
+    if export_format == "cursor":
         output_path = output or Path(".cursorrules")
         content = f"""# Project Memory (Auto-generated by doc-intelligence)
 # Last updated: {datetime.now().strftime('%Y-%m-%d %H:%M:%S')}
@@ -133,7 +133,7 @@ def export(format: str, output: Path | None, docs_dir: Path, include_symbols: bo
 *Regenerate with: `memdocs export cursor`*
 """
 
-    elif format == "continue":
+    elif export_format == "continue":
         output_path = output or Path(".continue/context.md")
         content = f"""# Project Context
 

memdocs/cli_modules/commands/init_cmd.py

@@ -6,17 +6,17 @@
 import os
 import sys
 from pathlib import Path
-from typing import Optional
+from typing import Any
 
 import click
 
 from memdocs import cli_output as out
 from memdocs.security import PathValidator
 
 
-def _detect_project_type(cwd: Path) -> dict[str, any]:
+def _detect_project_type(cwd: Path) -> dict[str, Any]:
     """Detect project type and return metadata."""
-    project_info: dict[str, any] = {
+    project_info: dict[str, Any] = {
         "type": "unknown",
         "language": None,
         "source_dir": None,
@@ -182,7 +182,7 @@ def _setup_mcp_infrastructure(cwd: Path) -> None:
     default=None,
     help="Interactive mode (auto-detects TTY if not specified)",
 )
-def init(force: bool, no_mcp: bool, interactive: Optional[bool]) -> None:
+def init(force: bool, no_mcp: bool, interactive: bool | None) -> None:
     """Initialize MemDocs in the current project.
 
     MCP auto-start is enabled by default for VS Code/Cursor integration.

memdocs/cli_modules/commands/query_cmd.py

@@ -43,8 +43,8 @@ def query(query: str, k: int, memory_dir: Path) -> None:
         out.print_header("MemDocs Query")
         out.step(f'Searching for: [cyan]"{query}"[/cyan]')
 
-        MemoryIndexer = _get_memory_indexer()
-        indexer = MemoryIndexer(memory_dir=memory_dir, use_embeddings=True)
+        memory_indexer_class = _get_memory_indexer()
+        indexer = memory_indexer_class(memory_dir=memory_dir, use_embeddings=True)
 
         if not indexer.use_embeddings:
             out.error("Embeddings not available")

memdocs/cli_modules/commands/review_cmd.py

@@ -232,21 +232,21 @@ def review(
             sys.exit(1)
 
         # Get CLI classes (lazy import)
-        Extractor, MemoryIndexer, PolicyEngine, Summarizer = _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")
 
         # Extract context
         with out.spinner(f"Extracting context from {len(paths)} path(s)"):
-            extractor = Extractor(repo_path=Path("."))
+            extractor = extractor_class(repo_path=Path("."))
             context = extractor.extract_context(paths)
 
         out.success(f"Extracted context from {len(context.files)} files")
 
         # Apply policy
         out.step("Determining scope")
-        policy_engine = PolicyEngine(doc_config)
+        policy_engine = policy_engine_class(doc_config)
         scope = policy_engine.determine_scope(paths, context, force=force)
 
         # Show warnings
@@ -266,7 +266,7 @@ def review(
 
         # Summarize with AI
         with out.spinner("Generating documentation with Claude Sonnet 4.5"):
-            summarizer = Summarizer(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")
@@ -293,7 +293,7 @@ def review(
 
             # Generate embeddings (optional, v1.1)
             try:
-                indexer = MemoryIndexer(
+                indexer = memory_indexer_class(
                     memory_dir=doc_config.outputs.memory_dir, use_embeddings=True
                 )
 

memdocs/cli_modules/commands/serve_cmd.py

@@ -9,7 +9,7 @@
 from http.server import BaseHTTPRequestHandler, HTTPServer
 from pathlib import Path
 from typing import Any
-from urllib.parse import parse_qs, urlparse
+from urllib.parse import urlparse
 
 import click
 
@@ -19,7 +19,7 @@
 class MemDocsHTTPHandler(BaseHTTPRequestHandler):
     """HTTP handler for MemDocs MCP server."""
 
-    def log_message(self, format: str, *args: Any) -> None:
+    def log_message(self, format: str, *args: Any) -> None:  # noqa: A002
         """Override to use our logger."""
         if self.server.verbose:  # type: ignore
             out.info(f"{self.address_string()} - {format % args}")
@@ -110,8 +110,8 @@ def _send_error(self, code: int, message: str) -> None:
 
     def _get_stats(self) -> dict[str, Any]:
         """Get memory statistics."""
-        docs_dir = getattr(self.server, "docs_dir")  # type: ignore
-        memory_dir = getattr(self.server, "memory_dir")  # type: ignore
+        docs_dir = self.server.docs_dir  # type: ignore
+        memory_dir = self.server.memory_dir  # type: ignore
 
         stats: dict[str, Any] = {
             "docs_dir": str(docs_dir),

memdocs/cli_modules/commands/setup_hooks_cmd.py

@@ -2,7 +2,6 @@
 Setup-hooks command - Install git hooks for automatic memory updates.
 """
 
-import os
 import subprocess
 import sys
 from pathlib import Path

memdocs/cli_modules/commands/stats_cmd.py

@@ -32,11 +32,12 @@ def _get_memory_indexer():
 )
 @click.option(
     "--format",
+    "output_format",
     type=click.Choice(["table", "json"]),
     default="table",
     help="Output format",
 )
-def stats(docs_dir: Path, memory_dir: Path, format: str) -> None:
+def stats(docs_dir: Path, memory_dir: Path, output_format: str) -> None:
     """Show memory and documentation statistics.
 
     Examples:
@@ -77,14 +78,14 @@ def stats(docs_dir: Path, memory_dir: Path, format: str) -> None:
         # Embedding index stats
         index_stats = None
         try:
-            MemoryIndexer = _get_memory_indexer()
-            indexer = MemoryIndexer(memory_dir=memory_dir, use_embeddings=True)
+            memory_indexer_class = _get_memory_indexer()
+            indexer = memory_indexer_class(memory_dir=memory_dir, use_embeddings=True)
             if indexer.use_embeddings:
                 index_stats = indexer.get_stats()
         except Exception:
             pass
 
-        if format == "json":
+        if output_format == "json":
             # JSON output
             output = {
                 "docs": docs_stats,

memdocs/security.py

@@ -319,11 +319,11 @@ def validate_scope_level(scope: str) -> str:
         return scope
 
     @staticmethod
-    def validate_output_format(format: str) -> str:
+    def validate_output_format(output_format: str) -> str:
         """Validate output format.
 
         Args:
-            format: Output format to validate
+            output_format: Output format to validate
 
         Returns:
             Validated format
@@ -332,11 +332,11 @@ def validate_output_format(format: str) -> str:
             ValueError: If format is invalid
         """
         valid_formats = ["json", "yaml", "markdown"]
-        if format not in valid_formats:
+        if output_format not in valid_formats:
             raise ValueError(
-                f"Invalid format '{format}'. Valid formats: {', '.join(valid_formats)}"
+                f"Invalid format '{output_format}'. Valid formats: {', '.join(valid_formats)}"
             )
-        return format
+        return output_format
 
     @staticmethod
     def validate_positive_int(value: int, name: str, min_value: int = 1) -> int: