ruff checks

Signed-off-by: Drew Cain <[email protected]>

Author: groksrc

src/basic_memory/api/routers/project_router.py

@@ -51,7 +51,7 @@ async def update_project(
         # Validate that path is absolute if provided
         if path and not os.path.isabs(path):
             raise HTTPException(status_code=400, detail="Path must be absolute")
-            
+
         # Get original project info for the response
         old_project_info = ProjectItem(
             name=name,

src/basic_memory/cli/commands/project.py

@@ -157,29 +157,33 @@ def move_project(
     """Move a project to a new location."""
     # Resolve to absolute path
     resolved_path = os.path.abspath(os.path.expanduser(new_path))
-    
+
     try:
         data = {"path": resolved_path}
         project_name = generate_permalink(name)
-        
+
         current_project = session.get_current_project()
-        response = asyncio.run(call_patch(client, f"/{current_project}/project/{project_name}", json=data))
+        response = asyncio.run(
+            call_patch(client, f"/{current_project}/project/{project_name}", json=data)
+        )
         result = ProjectStatusResponse.model_validate(response.json())
-        
+
         console.print(f"[green]{result.message}[/green]")
-        
+
         # Show important file movement reminder
         console.print()  # Empty line for spacing
-        console.print(Panel(
-            "[bold red]IMPORTANT:[/bold red] Project configuration updated successfully.\n\n"
-            "[yellow]You must manually move your project files from the old location to:[/yellow]\n"
-            f"[cyan]{resolved_path}[/cyan]\n\n"
-            "[dim]Basic Memory has only updated the configuration - your files remain in their original location.[/dim]",
-            title="⚠️  Manual File Movement Required",
-            border_style="yellow",
-            expand=False
-        ))
-        
+        console.print(
+            Panel(
+                "[bold red]IMPORTANT:[/bold red] Project configuration updated successfully.\n\n"
+                "[yellow]You must manually move your project files from the old location to:[/yellow]\n"
+                f"[cyan]{resolved_path}[/cyan]\n\n"
+                "[dim]Basic Memory has only updated the configuration - your files remain in their original location.[/dim]",
+                title="⚠️  Manual File Movement Required",
+                border_style="yellow",
+                expand=False,
+            )
+        )
+
     except Exception as e:
         console.print(f"[red]Error moving project: {str(e)}[/red]")
         raise typer.Exit(1)

src/basic_memory/importers/chatgpt_importer.py

@@ -209,24 +209,24 @@ def _traverse_messages(
 
         # Use iterative approach with stack to avoid recursion depth issues
         stack = [root_id]
-        
+
         while stack:
             node_id = stack.pop()
             if not node_id:
                 continue
-                
+
             node = mapping.get(node_id)
             if not node:
                 continue
-            
+
             # Process current node if it has a message and hasn't been seen
             if node["id"] not in seen and node.get("message"):
                 seen.add(node["id"])
                 messages.append(node["message"])
-            
+
             # Add children to stack in reverse order to maintain conversation flow
             children = node.get("children", [])
             for child_id in reversed(children):
                 stack.append(child_id)
-        
+
         return messages

src/basic_memory/repository/project_repository.py

@@ -86,11 +86,11 @@ async def set_as_default(self, project_id: int) -> Optional[Project]:
 
     async def update_path(self, project_id: int, new_path: str) -> Optional[Project]:
         """Update project path.
-        
+
         Args:
             project_id: ID of the project to update
             new_path: New filesystem path for the project
-            
+
         Returns:
             The updated project if found, None otherwise
         """

src/basic_memory/services/entity_service.py

@@ -47,35 +47,35 @@ def __init__(
 
     async def detect_file_path_conflicts(self, file_path: str) -> List[Entity]:
         """Detect potential file path conflicts for a given file path.
-        
+
         This checks for entities with similar file paths that might cause conflicts:
         - Case sensitivity differences (Finance/file.md vs finance/file.md)
         - Character encoding differences
         - Hyphen vs space differences
         - Unicode normalization differences
-        
+
         Args:
             file_path: The file path to check for conflicts
-            
+
         Returns:
             List of entities that might conflict with the given file path
         """
         from basic_memory.utils import detect_potential_file_conflicts
-        
+
         conflicts = []
-        
+
         # Get all existing file paths
         all_entities = await self.repository.find_all()
         existing_paths = [entity.file_path for entity in all_entities]
-        
+
         # Use the enhanced conflict detection utility
         conflicting_paths = detect_potential_file_conflicts(file_path, existing_paths)
-        
+
         # Find the entities corresponding to conflicting paths
         for entity in all_entities:
             if entity.file_path in conflicting_paths:
                 conflicts.append(entity)
-        
+
         return conflicts
 
     async def resolve_permalink(
@@ -88,19 +88,19 @@ async def resolve_permalink(
         2. If markdown has permalink but it's used by another file -> make unique
         3. For existing files, keep current permalink from db
         4. Generate new unique permalink from file path
-        
+
         Enhanced to detect and handle character-related conflicts.
         """
         file_path_str = str(file_path)
-        
+
         # Check for potential file path conflicts before resolving permalink
         conflicts = await self.detect_file_path_conflicts(file_path_str)
         if conflicts:
             logger.warning(
                 f"Detected potential file path conflicts for '{file_path_str}': "
                 f"{[entity.file_path for entity in conflicts]}"
             )
-        
+
         # If markdown has explicit permalink, try to validate it
         if markdown and markdown.frontmatter.permalink:
             desired_permalink = markdown.frontmatter.permalink

src/basic_memory/services/project_service.py

@@ -311,33 +311,33 @@ async def synchronize_projects(self) -> None:  # pragma: no cover
 
     async def move_project(self, name: str, new_path: str) -> None:
         """Move a project to a new location.
-        
+
         Args:
             name: The name of the project to move
             new_path: The new absolute path for the project
-            
+
         Raises:
             ValueError: If the project doesn't exist or repository isn't initialized
         """
         if not self.repository:
             raise ValueError("Repository is required for move_project")
-        
+
         # Resolve to absolute path
         resolved_path = os.path.abspath(os.path.expanduser(new_path))
-        
+
         # Validate project exists in config
         if name not in self.config_manager.projects:
             raise ValueError(f"Project '{name}' not found in configuration")
-        
+
         # Create the new directory if it doesn't exist
         Path(resolved_path).mkdir(parents=True, exist_ok=True)
-        
+
         # Update in configuration
         config = self.config_manager.load_config()
         old_path = config.projects[name]
         config.projects[name] = resolved_path
         self.config_manager.save_config(config)
-        
+
         # Update in database
         project = await self.repository.get_by_name(name)
         if project:

src/basic_memory/sync/sync_service.py

@@ -462,7 +462,7 @@ async def handle_move(self, old_path, new_path):
                     f"entity_id={entity.id} trying to move from '{old_path}' to '{new_path}', "
                     f"but entity_id={existing_at_destination.id} already occupies '{new_path}'"
                 )
-                
+
                 # Check if this is a file swap (the destination entity is being moved to our old path)
                 # This would indicate a simultaneous move operation
                 old_path_after_swap = await self.entity_repository.get_by_file_path(old_path)

src/basic_memory/utils.py

@@ -183,74 +183,74 @@ def setup_logging(
 
 def normalize_file_path_for_comparison(file_path: str) -> str:
     """Normalize a file path for conflict detection.
-    
+
     This function normalizes file paths to help detect potential conflicts:
     - Converts to lowercase for case-insensitive comparison
     - Normalizes Unicode characters
     - Handles path separators consistently
-    
+
     Args:
         file_path: The file path to normalize
-        
+
     Returns:
         Normalized file path for comparison purposes
     """
     import unicodedata
-    
+
     # Convert to lowercase for case-insensitive comparison
     normalized = file_path.lower()
-    
+
     # Normalize Unicode characters (NFD normalization)
-    normalized = unicodedata.normalize('NFD', normalized)
-    
+    normalized = unicodedata.normalize("NFD", normalized)
+
     # Replace path separators with forward slashes
-    normalized = normalized.replace('\\', '/')
-    
+    normalized = normalized.replace("\\", "/")
+
     # Remove multiple slashes
-    normalized = re.sub(r'/+', '/', normalized)
-    
+    normalized = re.sub(r"/+", "/", normalized)
+
     return normalized
 
 
 def detect_potential_file_conflicts(file_path: str, existing_paths: List[str]) -> List[str]:
     """Detect potential conflicts between a file path and existing paths.
-    
+
     This function checks for various types of conflicts:
     - Case sensitivity differences
     - Unicode normalization differences
     - Path separator differences
     - Permalink generation conflicts
-    
+
     Args:
         file_path: The file path to check
         existing_paths: List of existing file paths to check against
-        
+
     Returns:
         List of existing paths that might conflict with the given file path
     """
     conflicts = []
-    
+
     # Normalize the input file path
     normalized_input = normalize_file_path_for_comparison(file_path)
     input_permalink = generate_permalink(file_path)
-    
+
     for existing_path in existing_paths:
         # Skip identical paths
         if existing_path == file_path:
             continue
-            
+
         # Check for case-insensitive path conflicts
         normalized_existing = normalize_file_path_for_comparison(existing_path)
         if normalized_input == normalized_existing:
             conflicts.append(existing_path)
             continue
-            
+
         # Check for permalink conflicts
         existing_permalink = generate_permalink(existing_path)
         if input_permalink == existing_permalink:
             conflicts.append(existing_path)
             continue
-    
+
     return conflicts