crackerjack 0.31.10__py3-none-any.whl → 0.31.12__py3-none-any.whl

crackerjack/config/global_lock_config.py

@@ -0,0 +1,110 @@
+import os
+import socket
+from dataclasses import dataclass, field
+from pathlib import Path
+
+from ..models.protocols import OptionsProtocol
+
+
+@dataclass
+class GlobalLockConfig:
+    """Configuration for global hook locking system."""
+
+    enabled: bool = True
+    timeout_seconds: float = 600.0  # 10 minutes default
+    stale_lock_hours: float = 2.0  # Clean locks older than 2 hours
+    lock_directory: Path = field(
+        default_factory=lambda: Path.home() / ".crackerjack" / "locks"
+    )
+    session_heartbeat_interval: float = 30.0  # Heartbeat every 30s
+    max_retry_attempts: int = 3
+    retry_delay_seconds: float = 5.0
+    enable_lock_monitoring: bool = True
+
+    @classmethod
+    def from_options(cls, options: "OptionsProtocol") -> "GlobalLockConfig":
+        """Create GlobalLockConfig from CLI options.
+
+        Args:
+            options: Options object containing CLI arguments
+
+        Returns:
+            Configured GlobalLockConfig instance
+        """
+        # Import here to avoid circular import
+
+        # Create config with custom values from options
+        enabled = not options.disable_global_locks
+        timeout_seconds = float(options.global_lock_timeout)
+        # Get default lock directory from field definition
+        default_lock_dir = Path.home() / ".crackerjack" / "locks"
+        lock_directory = (
+            Path(options.global_lock_dir)
+            if options.global_lock_dir
+            else default_lock_dir
+        )
+
+        # Create instance with all parameters so __post_init__ is called
+        config = cls(
+            enabled=enabled,
+            timeout_seconds=timeout_seconds,
+            lock_directory=lock_directory,
+        )
+
+        return config
+
+    def __post_init__(self) -> None:
+        """Ensure lock directory exists and has proper permissions."""
+        self.lock_directory.mkdir(parents=True, exist_ok=True)
+        # Set restrictive permissions (owner read/write only)
+        self.lock_directory.chmod(0o700)
+
+    @property
+    def hostname(self) -> str:
+        """Get the current hostname for lock identification."""
+        return socket.gethostname()
+
+    @property
+    def session_id(self) -> str:
+        """Get a unique session identifier."""
+        return f"{self.hostname}_{os.getpid()}"
+
+    def get_lock_path(self, hook_name: str) -> Path:
+        """Get the lock file path for a specific hook.
+
+        Args:
+            hook_name: Name of the hook
+
+        Returns:
+            Path to the lock file
+        """
+        return self.lock_directory / f"{hook_name}.lock"
+
+    def get_heartbeat_path(self, hook_name: str) -> Path:
+        """Get the heartbeat file path for a specific hook.
+
+        Args:
+            hook_name: Name of the hook
+
+        Returns:
+            Path to the heartbeat file
+        """
+        return self.lock_directory / f"{hook_name}.heartbeat"
+
+    def is_valid_lock_file(self, lock_path: Path) -> bool:
+        """Check if a lock file is valid (not stale).
+
+        Args:
+            lock_path: Path to the lock file
+
+        Returns:
+            True if the lock file is valid and not stale
+        """
+        if not lock_path.exists():
+            return False
+
+        # Check if the lock file is older than stale_lock_hours
+        import time
+
+        file_age_hours = (time.time() - lock_path.stat().st_mtime) / 3600
+        return file_age_hours < self.stale_lock_hours

crackerjack/config/hooks.py

@@ -26,7 +26,21 @@ class HookDefinition:
     config_path: Path | None = None
 
     def get_command(self) -> list[str]:
-        cmd = ["pre-commit", "run"]
+        # Use direct pre-commit execution (pre-commit manages its own environments)
+        import shutil
+        from pathlib import Path
+
+        # Find pre-commit executable - prefer project venv, fallback to system
+        pre_commit_path = None
+        current_dir = Path.cwd()
+        project_pre_commit = current_dir / ".venv" / "bin" / "pre-commit"
+        if project_pre_commit.exists():
+            pre_commit_path = str(project_pre_commit)
+        else:
+            pre_commit_path = shutil.which("pre-commit") or "pre-commit"
+
+        # Build command for direct pre-commit execution
+        cmd = [pre_commit_path, "run"]
         if self.config_path:
             cmd.extend(["-c", str(self.config_path)])
         if self.manual_stage:
@@ -46,55 +60,64 @@ class HookStrategy:
 
 
 FAST_HOOKS = [
+    HookDefinition(
+        name="validate-regex-patterns",
+        command=[],  # Dynamically built by get_command()
+        is_formatting=True,  # Treat as formatting for autofix purposes
+        timeout=30,
+        retry_on_failure=True,  # Enable retries for autofix
+    ),
     HookDefinition(
         name="trailing-whitespace",
-        command=["pre-commit", "run", "trailing-whitespace", "--all-files"],
+        command=[],  # Dynamically built by get_command()
         is_formatting=True,
         retry_on_failure=True,
     ),
     HookDefinition(
         name="end-of-file-fixer",
-        command=["pre-commit", "run", "end-of-file-fixer", "--all-files"],
+        command=[],  # Dynamically built by get_command()
         is_formatting=True,
         retry_on_failure=True,
     ),
     HookDefinition(
         name="check-yaml",
-        command=["pre-commit", "run", "check-yaml", "--all-files"],
+        command=[],  # Dynamically built by get_command()
     ),
     HookDefinition(
         name="check-toml",
-        command=["pre-commit", "run", "check-toml", "--all-files"],
+        command=[],  # Dynamically built by get_command()
     ),
     HookDefinition(
         name="check-added-large-files",
-        command=["pre-commit", "run", "check-added-large-files", "--all-files"],
+        command=[],  # Dynamically built by get_command()
     ),
     HookDefinition(
         name="uv-lock",
-        command=["pre-commit", "run", "uv-lock", "--all-files"],
+        command=[],  # Dynamically built by get_command()
     ),
     HookDefinition(
         name="gitleaks",
-        command=["pre-commit", "run", "gitleaks", "--all-files"],
+        command=[],  # Dynamically built by get_command()
     ),
     HookDefinition(
         name="codespell",
-        command=["pre-commit", "run", "codespell", "--all-files"],
+        command=[],  # Dynamically built by get_command()
     ),
     HookDefinition(
         name="ruff-check",
-        command=["pre-commit", "run", "ruff-check", "--all-files"],
+        command=[],  # Dynamically built by get_command()
+        is_formatting=True,  # Treat as formatting for autofix purposes
+        retry_on_failure=True,  # Enable retries for autofix
     ),
     HookDefinition(
         name="ruff-format",
-        command=["pre-commit", "run", "ruff-format", "--all-files"],
+        command=[],  # Dynamically built by get_command()
         is_formatting=True,
         retry_on_failure=True,
     ),
     HookDefinition(
         name="mdformat",
-        command=["pre-commit", "run", "mdformat", "--all-files"],
+        command=[],  # Dynamically built by get_command()
         is_formatting=True,
         retry_on_failure=True,
     ),
@@ -103,85 +126,43 @@ FAST_HOOKS = [
 COMPREHENSIVE_HOOKS = [
     HookDefinition(
         name="pyright",
-        command=[
-            "pre-commit",
-            "run",
-            "--hook-stage",
-            "manual",
-            "pyright",
-            "--all-files",
-        ],
+        command=[],  # Dynamically built by get_command()
         timeout=120,
         stage=HookStage.COMPREHENSIVE,
         manual_stage=True,
     ),
     HookDefinition(
         name="bandit",
-        command=[
-            "pre-commit",
-            "run",
-            "--hook-stage",
-            "manual",
-            "bandit",
-            "--all-files",
-        ],
+        command=[],  # Dynamically built by get_command()
         timeout=120,
         stage=HookStage.COMPREHENSIVE,
         manual_stage=True,
     ),
     HookDefinition(
         name="vulture",
-        command=[
-            "pre-commit",
-            "run",
-            "--hook-stage",
-            "manual",
-            "vulture",
-            "--all-files",
-        ],
+        command=[],  # Dynamically built by get_command()
         timeout=120,
         stage=HookStage.COMPREHENSIVE,
         manual_stage=True,
     ),
     HookDefinition(
         name="refurb",
-        command=[
-            "pre-commit",
-            "run",
-            "--hook-stage",
-            "manual",
-            "refurb",
-            "--all-files",
-        ],
+        command=[],  # Dynamically built by get_command()
         timeout=120,
         stage=HookStage.COMPREHENSIVE,
         manual_stage=True,
     ),
     HookDefinition(
         name="creosote",
-        command=[
-            "pre-commit",
-            "run",
-            "--hook-stage",
-            "manual",
-            "creosote",
-            "--all-files",
-        ],
+        command=[],  # Dynamically built by get_command()
         timeout=120,
         stage=HookStage.COMPREHENSIVE,
         manual_stage=True,
     ),
     HookDefinition(
         name="complexipy",
-        command=[
-            "pre-commit",
-            "run",
-            "--hook-stage",
-            "manual",
-            "complexipy",
-            "--all-files",
-        ],
-        timeout=120,
+        command=[],  # Dynamically built by get_command()
+        timeout=60,
         stage=HookStage.COMPREHENSIVE,
         manual_stage=True,
     ),
@@ -213,6 +194,4 @@ class HookConfigLoader:
         msg = f"Unknown hook strategy: {name}"
         raise ValueError(msg)
 
-    @staticmethod
-    def get_all_strategies() -> dict[str, HookStrategy]:
-        return {"fast": FAST_STRATEGY, "comprehensive": COMPREHENSIVE_STRATEGY}
+    # Removed unused method: get_all_strategies