Reformat

Author: barisgit

kicad_lib_manager/commands/add_hook/command.py

@@ -77,7 +77,9 @@ def add_hook(directory, force):
             else:
                 # Merge with existing content to preserve user logic
                 click.echo("Merging KiLM content with existing hook...")
-                new_content = merge_hook_content(existing_content, create_kilm_hook_content())
+                new_content = merge_hook_content(
+                    existing_content, create_kilm_hook_content()
+                )
 
         except (OSError, UnicodeDecodeError):
             click.echo("Warning: Could not read existing hook content, overwriting...")

kicad_lib_manager/commands/config/command.py

@@ -292,7 +292,9 @@ def set_default(library_name, library_type):
 
         # Set as current library
         if library_path is None:
-            click.echo(f"Error: Could not find path for library '{library_name}'", err=True)
+            click.echo(
+                f"Error: Could not find path for library '{library_name}'", err=True
+            )
             sys.exit(1)
         config.set_current_library(library_path)
         click.echo(f"Set {library_type} library '{library_name}' as default.")
@@ -350,7 +352,9 @@ def remove(library_name, library_type, force):
         # Find libraries matching the name and type
         matching_libraries = []
         for lib in all_libraries:
-            if lib.get("name") == library_name and (library_type == "all" or lib.get("type") == library_type):
+            if lib.get("name") == library_name and (
+                library_type == "all" or lib.get("type") == library_type
+            ):
                 matching_libraries.append(lib)
 
         if not matching_libraries:

kicad_lib_manager/commands/setup/command.py

@@ -478,7 +478,6 @@ def setup(
 
             # Also list existing libraries to pin them all
             try:
-
                 existing_symbols, existing_footprints = list_libraries(kicad_lib_dir)
                 symbol_libs = existing_symbols
                 footprint_libs = existing_footprints

kicad_lib_manager/commands/status/command.py

@@ -26,7 +26,11 @@ def status():
                         config_data = yaml.safe_load(f)
 
                     # Show libraries
-                    if config_data and "libraries" in config_data and config_data["libraries"]:
+                    if (
+                        config_data
+                        and "libraries" in config_data
+                        and config_data["libraries"]
+                    ):
                         click.echo("  Configured Libraries:")
 
                         # Group by type
@@ -48,7 +52,8 @@ def status():
                                 path = lib.get("path", "unknown")
                                 current = (
                                     " (current)"
-                                    if config_data and config_data.get("current_library") == path
+                                    if config_data
+                                    and config_data.get("current_library") == path
                                     else ""
                                 )
                                 click.echo(f"      - {name}: {path}{current}")
@@ -67,7 +72,8 @@ def status():
                                 path = lib.get("path", "unknown")
                                 current = (
                                     " (current)"
-                                    if config_data and config_data.get("current_library") == path
+                                    if config_data
+                                    and config_data.get("current_library") == path
                                     else ""
                                 )
                                 click.echo(f"      - {name}: {path}{current}")
@@ -83,7 +89,8 @@ def status():
 
                     # Show current library
                     if (
-                        config_data and "current_library" in config_data
+                        config_data
+                        and "current_library" in config_data
                         and config_data["current_library"]
                     ):
                         click.echo(

kicad_lib_manager/commands/template/command.py

@@ -817,7 +817,12 @@ def make(
                 if not git_path.endswith("/"):
                     git_path += "/"
 
-                if gitignore_spec and gitignore_spec.match_file(git_path) or additional_spec and additional_spec.match_file(git_path):
+                if (
+                    gitignore_spec
+                    and gitignore_spec.match_file(git_path)
+                    or additional_spec
+                    and additional_spec.match_file(git_path)
+                ):
                     dirs_to_remove.append(d)
                     excluded_files.append(f"{rel_path}/")
 

kicad_lib_manager/commands/unpin/command.py

@@ -50,7 +50,9 @@ def unpin(symbols, footprints, all, dry_run, max_backups, verbose):
     """Unpin libraries in KiCad"""
     # Enforce mutual exclusivity of --all with --symbols/--footprints
     if all and (symbols or footprints):
-        raise click.UsageError("'--all' cannot be used with '--symbols' or '--footprints'")
+        raise click.UsageError(
+            "'--all' cannot be used with '--symbols' or '--footprints'"
+        )
 
     # Find KiCad configuration
     try:

kicad_lib_manager/commands/update/command.py

@@ -175,6 +175,7 @@ def update(dry_run, verbose, auto_setup):
         )
         click.echo("Use 'kilm status' to check your current configuration.")
 
+
 # TODO: Should be in services or utils
 def check_for_library_changes(git_output, lib_path):
     """
@@ -202,23 +203,35 @@ def check_for_library_changes(git_output, lib_path):
     templates_path = lib_path / "templates"
 
     # Look for symbol libraries (.kicad_sym files)
-    if symbols_path.exists() and symbols_path.is_dir() and any(
-        f.name.endswith(".kicad_sym") for f in symbols_path.glob("**/*.kicad_sym")
+    if (
+        symbols_path.exists()
+        and symbols_path.is_dir()
+        and any(
+            f.name.endswith(".kicad_sym") for f in symbols_path.glob("**/*.kicad_sym")
+        )
     ):
         changes.append("symbols")
 
     # Look for footprint libraries (.pretty directories)
-    if footprints_path.exists() and footprints_path.is_dir() and any(
-        f.is_dir() and f.name.endswith(".pretty")
-        for f in footprints_path.glob("**/*.pretty")
+    if (
+        footprints_path.exists()
+        and footprints_path.is_dir()
+        and any(
+            f.is_dir() and f.name.endswith(".pretty")
+            for f in footprints_path.glob("**/*.pretty")
+        )
     ):
         changes.append("footprints")
 
     # Look for project templates (directories with metadata.yaml)
-    if templates_path.exists() and templates_path.is_dir() and any(
-        (f / "metadata.yaml").exists()
-        for f in templates_path.glob("*")
-        if f.is_dir()
+    if (
+        templates_path.exists()
+        and templates_path.is_dir()
+        and any(
+            (f / "metadata.yaml").exists()
+            for f in templates_path.glob("*")
+            if f.is_dir()
+        )
     ):
         changes.append("templates")
 

kicad_lib_manager/config.py

@@ -18,6 +18,7 @@
 
 class LibraryDict(TypedDict):
     """Type definition for library configuration."""
+
     name: str
     path: str
     type: str
@@ -62,7 +63,9 @@ def _get_config_file(self) -> Path:
         config_dir.mkdir(parents=True, exist_ok=True)
         return config_dir / CONFIG_FILE_NAME
 
-    def get(self, key: str, default: Optional[ConfigValue] = None) -> Optional[ConfigValue]:
+    def get(
+        self, key: str, default: Optional[ConfigValue] = None
+    ) -> Optional[ConfigValue]:
         """Get a configuration value"""
         return self._config.get(key, default)
 
@@ -122,9 +125,7 @@ def remove_library(self, name: str, library_type: Optional[str] = None) -> bool:
                 if not (lib["name"] == name and lib["type"] == library_type)
             ]
         else:
-            filtered_libraries = [
-                lib for lib in libraries if lib["name"] != name
-            ]
+            filtered_libraries = [lib for lib in libraries if lib["name"] != name]
 
         self._config["libraries"] = filtered_libraries
         removed = len(filtered_libraries) < original_count
@@ -263,14 +264,19 @@ def _normalize_libraries_field(self) -> None:
 
         # If libraries is not a list, reset to empty list
         if not isinstance(libraries, list):
-            click.echo(f"Warning: libraries field in config was {type(libraries).__name__}, resetting to empty list", err=True)
+            click.echo(
+                f"Warning: libraries field in config was {type(libraries).__name__}, resetting to empty list",
+                err=True,
+            )
             self._config["libraries"] = []
             return
 
         # Validate and clean up each library entry
         normalized_libraries: List[LibraryDict] = []
         for lib in libraries:
-            if isinstance(lib, dict) and all(key in lib for key in ["name", "path", "type"]):
+            if isinstance(lib, dict) and all(
+                key in lib for key in ["name", "path", "type"]
+            ):
                 # Ensure all values are strings
                 normalized_lib = _make_library_dict(
                     name=str(lib["name"]),
@@ -294,7 +300,10 @@ def _get_normalized_libraries(self) -> List[LibraryDict]:
 
         # If libraries is not a list, reset to empty list
         if not isinstance(libraries_raw, list):
-            click.echo(f"Warning: libraries field was {type(libraries_raw).__name__}, resetting to empty list", err=True)
+            click.echo(
+                f"Warning: libraries field was {type(libraries_raw).__name__}, resetting to empty list",
+                err=True,
+            )
             self._config["libraries"] = []
             return []
 
@@ -303,7 +312,9 @@ def _get_normalized_libraries(self) -> List[LibraryDict]:
         needs_save = False
 
         for lib in libraries_raw:
-            if isinstance(lib, dict) and all(key in lib for key in ["name", "path", "type"]):
+            if isinstance(lib, dict) and all(
+                key in lib for key in ["name", "path", "type"]
+            ):
                 # Ensure all values are strings
                 normalized_lib = _make_library_dict(
                     name=str(lib["name"]),

kicad_lib_manager/utils/file_ops.py

@@ -7,7 +7,9 @@
 from typing import List, Optional, Tuple
 
 
-def read_file_with_encoding(file_path: Path, encodings: Optional[List[str]] = None) -> str:
+def read_file_with_encoding(
+    file_path: Path, encodings: Optional[List[str]] = None
+) -> str:
     """
     Read a file trying multiple encodings until successful
 

kicad_lib_manager/utils/git_utils.py

@@ -32,7 +32,9 @@ def get_git_hooks_directory(repo_path: Path) -> Path:
     try:
         rp = subprocess.run(
             ["git", "-C", str(repo_path), "rev-parse", "--git-path", "hooks"],
-            capture_output=True, text=True, check=True
+            capture_output=True,
+            text=True,
+            check=True,
         )
         hooks_dir = Path(rp.stdout.strip())
         if not hooks_dir.is_absolute():
@@ -44,7 +46,9 @@ def get_git_hooks_directory(repo_path: Path) -> Path:
         try:
             cd = subprocess.run(
                 ["git", "-C", str(repo_path), "rev-parse", "--git-common-dir"],
-                capture_output=True, text=True, check=True
+                capture_output=True,
+                text=True,
+                check=True,
             )
             common_dir = Path(cd.stdout.strip())
             if not common_dir.is_absolute():
@@ -93,7 +97,7 @@ def merge_hook_content(existing_content: str, kilm_content: str) -> str:
     # Check if KiLM content is already present
     if "KiLM-managed section" in existing_content:
         # Already has KiLM content, replace the section
-        lines = existing_content.split('\n')
+        lines = existing_content.split("\n")
         start_marker = "# BEGIN KiLM-managed section"
         end_marker = "# END KiLM-managed section"
 
@@ -109,8 +113,8 @@ def merge_hook_content(existing_content: str, kilm_content: str) -> str:
 
         if start_idx is not None and end_idx is not None:
             # Replace existing KiLM section
-            new_lines = lines[:start_idx] + [kilm_content] + lines[end_idx + 1:]
-            return '\n'.join(new_lines)
+            new_lines = lines[:start_idx] + [kilm_content] + lines[end_idx + 1 :]
+            return "\n".join(new_lines)
 
     # Add KiLM content at the end with clear markers
     return f"{existing_content.rstrip()}\n\n{kilm_content}"