crackerjack 0.37.9__py3-none-any.whl → 0.45.2__py3-none-any.whl

crackerjack/models/protocols.py

@@ -2,6 +2,63 @@ import subprocess
 import typing as t
 from pathlib import Path
 
+from crackerjack.config.hooks import HookDefinition
+from crackerjack.config.settings import CrackerjackSettings
+from crackerjack.models.results import ExecutionResult, ParallelExecutionResult
+
+
+@t.runtime_checkable
+class ServiceProtocol(t.Protocol):
+    """Base protocol for ACB services with standardized lifecycle methods."""
+
+    def initialize(self) -> None:
+        """Initialize service with proper lifecycle management."""
+        ...
+
+    def cleanup(self) -> None:
+        """Cleanup service resources."""
+        ...
+
+    def health_check(self) -> bool:
+        """Perform health check for service."""
+        ...
+
+    def shutdown(self) -> None:
+        """Shutdown service gracefully."""
+        ...
+
+    def metrics(self) -> dict[str, t.Any]:
+        """Get service metrics."""
+        ...
+
+    def is_healthy(self) -> bool:
+        """Check if service is healthy."""
+        ...
+
+    def register_resource(self, resource: t.Any) -> None:
+        """Register resource for cleanup."""
+        ...
+
+    def cleanup_resource(self, resource: t.Any) -> None:
+        """Cleanup specific resource."""
+        ...
+
+    def record_error(self, error: Exception) -> None:
+        """Record service error for monitoring."""
+        ...
+
+    def increment_requests(self) -> None:
+        """Increment request counter."""
+        ...
+
+    def get_custom_metric(self, name: str) -> t.Any:
+        """Get custom service metric."""
+        ...
+
+    def set_custom_metric(self, name: str, value: t.Any) -> None:
+        """Set custom service metric."""
+        ...
+
 
 @t.runtime_checkable
 class CommandRunner(t.Protocol):
@@ -44,7 +101,10 @@ class OptionsProtocol(t.Protocol):
     track_progress: bool = False
     fast: bool = False
     comp: bool = False
-    enterprise_batch: str | None = None
+    fast_iteration: bool = False
+    tool: str | None = None
+    changed_only: bool = False
+    advanced_batch: str | None = None
     monitor_dashboard: str | None = None
     skip_config_merge: bool = False
     disable_global_locks: bool = False
@@ -83,10 +143,19 @@ class GitInterface(t.Protocol):
 
     def get_staged_files(self) -> list[str]: ...
 
+    def get_changed_files_by_extension(
+        self,
+        extensions: list[str],
+        include_staged: bool = True,
+        include_unstaged: bool = True,
+    ) -> list[Path]: ...
+
     def commit(self, message: str) -> bool: ...
 
     def push(self) -> bool: ...
 
+    def push_with_tags(self) -> bool: ...
+
     def add_files(self, files: list[str]) -> bool: ...
 
     def add_all_files(self) -> bool: ...
@@ -95,6 +164,10 @@ class GitInterface(t.Protocol):
 
     def get_unpushed_commit_count(self) -> int: ...
 
+    def get_current_commit_hash(self) -> str | None: ...
+
+    def reset_hard(self, commit_hash: str) -> bool: ...
+
 
 @t.runtime_checkable
 class HookManager(t.Protocol):
@@ -121,7 +194,7 @@ class SecurityAwareHookManager(HookManager, t.Protocol):
 
 
 @t.runtime_checkable
-class CoverageRatchetProtocol(t.Protocol):
+class CoverageRatchetProtocol(ServiceProtocol, t.Protocol):
     def get_baseline_coverage(self) -> float: ...
 
     def update_baseline_coverage(self, new_coverage: float) -> bool: ...
@@ -138,16 +211,7 @@ class CoverageRatchetProtocol(t.Protocol):
 
 
 @t.runtime_checkable
-class ConfigurationServiceProtocol(t.Protocol):
-    def update_precommit_config(self, options: OptionsProtocol) -> bool: ...
-
-    def update_pyproject_config(self, options: OptionsProtocol) -> bool: ...
-
-    def get_temp_config_path(self) -> str | None: ...
-
-
-@t.runtime_checkable
-class SecurityServiceProtocol(t.Protocol):
+class SecurityServiceProtocol(ServiceProtocol, t.Protocol):
     def validate_file_safety(self, path: str | Path) -> bool: ...
 
     def check_hardcoded_secrets(self, content: str) -> list[dict[str, t.Any]]: ...
@@ -162,7 +226,7 @@ class SecurityServiceProtocol(t.Protocol):
 
 
 @t.runtime_checkable
-class InitializationServiceProtocol(t.Protocol):
+class InitializationServiceProtocol(ServiceProtocol, t.Protocol):
     def initialize_project(self, project_path: str | Path) -> bool: ...
 
     def validate_project_structure(self) -> bool: ...
@@ -171,16 +235,41 @@ class InitializationServiceProtocol(t.Protocol):
 
 
 @t.runtime_checkable
-class UnifiedConfigurationServiceProtocol(t.Protocol):
-    def merge_configurations(self) -> dict[str, t.Any]: ...
+class SmartSchedulingServiceProtocol(ServiceProtocol, t.Protocol):
+    """Protocol for smart scheduling service."""
 
-    def validate_configuration(self, config: dict[str, t.Any]) -> bool: ...
+    def should_scheduled_init(self) -> bool: ...
 
-    def get_merged_config(self) -> dict[str, t.Any]: ...
+    def record_init_timestamp(self) -> None: ...
 
 
 @t.runtime_checkable
-class TestManagerProtocol(t.Protocol):
+class UnifiedConfigurationServiceProtocol(ServiceProtocol, t.Protocol):
+    def get_config(self, reload: bool = False) -> CrackerjackSettings: ...
+
+    def get_precommit_config_mode(self) -> str: ...
+
+    def get_logging_config(self) -> dict[str, t.Any]: ...
+
+    def get_hook_execution_config(self) -> dict[str, t.Any]: ...
+
+    def get_testing_config(self) -> dict[str, t.Any]: ...
+
+    @staticmethod
+    def get_cache_config() -> dict[str, t.Any]: ...
+
+    def validate_current_config(self) -> bool: ...
+
+
+@t.runtime_checkable
+class ConfigIntegrityServiceProtocol(ServiceProtocol, t.Protocol):
+    """Protocol for config integrity service."""
+
+    def check_config_integrity(self) -> bool: ...
+
+
+@t.runtime_checkable
+class TestManagerProtocol(ServiceProtocol, t.Protocol):
     def run_tests(self, options: OptionsProtocol) -> bool: ...
 
     def get_test_failures(self) -> list[str]: ...
@@ -190,6 +279,24 @@ class TestManagerProtocol(t.Protocol):
     def get_coverage(self) -> dict[str, t.Any]: ...
 
 
+@t.runtime_checkable
+class BoundedStatusOperationsProtocol(ServiceProtocol, t.Protocol):
+    """Protocol for bounded status operations service."""
+
+    async def execute_bounded_operation(
+        self,
+        operation_type: str,
+        client_id: str,
+        operation_func: t.Callable[..., t.Awaitable[t.Any]],
+        *args: t.Any,
+        **kwargs: t.Any,
+    ) -> t.Any: ...
+
+    def get_operation_status(self) -> dict[str, t.Any]: ...
+
+    def reset_circuit_breaker(self, operation_type: str) -> bool: ...
+
+
 @t.runtime_checkable
 class PublishManager(t.Protocol):
     def bump_version(self, version_type: str) -> str: ...
@@ -200,11 +307,13 @@ class PublishManager(t.Protocol):
 
     def create_git_tag(self, version: str) -> bool: ...
 
+    def create_git_tag_local(self, version: str) -> bool: ...
+
     def cleanup_old_releases(self, keep_releases: int) -> None: ...
 
 
 @t.runtime_checkable
-class ConfigMergeServiceProtocol(t.Protocol):
+class ConfigMergeServiceProtocol(ServiceProtocol, t.Protocol):
     def smart_merge_pyproject(
         self,
         source_content: dict[str, t.Any],
@@ -251,9 +360,7 @@ class ConfigMergeServiceProtocol(t.Protocol):
 class HookLockManagerProtocol(t.Protocol):
     def requires_lock(self, hook_name: str) -> bool: ...
 
-    async def acquire_hook_lock(
-        self, hook_name: str
-    ) -> t.AsyncContextManager[None]: ...
+    def acquire_hook_lock(self, hook_name: str) -> t.AsyncContextManager[None]: ...
 
     def get_lock_stats(self) -> dict[str, t.Any]: ...
 
@@ -275,7 +382,7 @@ class HookLockManagerProtocol(t.Protocol):
 
 
 @t.runtime_checkable
-class DocumentationServiceProtocol(t.Protocol):
+class DocumentationServiceProtocol(ServiceProtocol, t.Protocol):
     """Service for automated documentation generation and maintenance."""
 
     def extract_api_documentation(
@@ -389,3 +496,1040 @@ class FileSystemServiceProtocol(t.Protocol):
     def mkdir(self, path: str | Path, parents: bool = False) -> None: ...
 
     def ensure_directory(self, path: str | Path) -> None: ...
+
+
+@t.runtime_checkable
+class EnhancedFileSystemServiceProtocol(ServiceProtocol, t.Protocol):
+    """Protocol for enhanced file system service."""
+
+    def read_file(self, path: str | Path) -> str: ...
+
+    def write_file(self, path: str | Path, content: str) -> None: ...
+
+    async def read_file_async(self, path: Path) -> str: ...
+
+    async def write_file_async(self, path: Path, content: str) -> None: ...
+
+    async def read_multiple_files(self, paths: list[Path]) -> dict[Path, str]: ...
+
+    async def write_multiple_files(self, file_data: dict[Path, str]) -> None: ...
+
+    def file_exists(self, path: str | Path) -> bool: ...
+
+    def create_directory(self, path: str | Path) -> None: ...
+
+    def delete_file(self, path: str | Path) -> None: ...
+
+    def list_files(self, path: str | Path, pattern: str = "*") -> t.Iterator[Path]: ...
+
+    async def flush_operations(self) -> None: ...
+
+    def get_cache_stats(self) -> dict[str, t.Any]: ...
+
+    def clear_cache(self) -> None: ...
+
+    def exists(self, path: str | Path) -> bool: ...
+
+    def mkdir(self, path: str | Path, parents: bool = False) -> None: ...
+
+
+@t.runtime_checkable
+class QAAdapterProtocol(t.Protocol):
+    """Protocol for quality assurance adapters (ACB-based).
+
+    All QA adapters must implement this protocol to ensure compatibility
+    with the QA orchestration system.
+    """
+
+    settings: t.Any | None  # QABaseSettings
+
+    async def init(self) -> None:
+        """Initialize adapter (ACB standard method)."""
+        ...
+
+    async def check(
+        self,
+        files: list[Path] | None = None,
+        config: t.Any | None = None,
+    ) -> t.Any:
+        """Execute the quality assurance check.
+
+        Args:
+            files: List of files to check (None = all matching files)
+            config: Optional configuration override for this check
+
+        Returns:
+            QAResult containing the check execution results
+        """
+        ...
+
+    async def validate_config(self, config: t.Any) -> bool:
+        """Validate that the provided configuration is valid.
+
+        Args:
+            config: Configuration to validate
+
+        Returns:
+            True if configuration is valid, False otherwise
+        """
+        ...
+
+    def get_default_config(self) -> t.Any:
+        """Get the default configuration for this adapter.
+
+        Returns:
+            QACheckConfig with sensible defaults for this check
+        """
+        ...
+
+    async def health_check(self) -> dict[str, t.Any]:
+        """Check adapter health (ACB standard method).
+
+        Returns:
+            Dictionary with health status and metadata
+        """
+        ...
+
+    @property
+    def adapter_name(self) -> str:
+        """Human-readable adapter name."""
+        ...
+
+    @property
+    def module_id(self) -> t.Any:
+        """Reference to module-level MODULE_ID (UUID)."""
+        ...
+
+
+@t.runtime_checkable
+class QAOrchestratorProtocol(t.Protocol):
+    """Protocol for QA orchestration service.
+
+    Coordinates multiple QA adapters, handles parallel execution,
+    caching, and result aggregation.
+    """
+
+    async def run_checks(
+        self,
+        stage: str = "fast",
+        files: list[Path] | None = None,
+    ) -> list[t.Any]:
+        """Run QA checks for specified stage.
+
+        Args:
+            stage: Execution stage ('fast' or 'comprehensive')
+            files: Optional list of files to check
+
+        Returns:
+            List of QAResult objects
+        """
+        ...
+
+    async def run_all_checks(
+        self,
+        files: list[Path] | None = None,
+    ) -> dict[str, t.Any]:
+        """Run all registered QA checks.
+
+        Args:
+            files: Optional list of files to check
+
+        Returns:
+            Dictionary mapping adapter names to results
+        """
+        ...
+
+    def register_adapter(self, adapter: QAAdapterProtocol) -> None:
+        """Register a QA adapter.
+
+        Args:
+            adapter: QA adapter to register
+        """
+        ...
+
+    def get_adapter(self, name: str) -> QAAdapterProtocol | None:
+        """Get registered adapter by name.
+
+        Args:
+            name: Adapter name
+
+        Returns:
+            Adapter if found, None otherwise
+        """
+        ...
+
+
+# ==================== Hook Orchestration Protocols (Phase 3) ====================
+
+
+@t.runtime_checkable
+class ExecutionStrategyProtocol(t.Protocol):
+    """Protocol for hook execution strategies.
+
+    Implementations:
+    - ParallelExecutionStrategy: Concurrent execution with resource limits
+    - SequentialExecutionStrategy: One-at-a-time execution for dependencies
+    """
+
+    async def execute(
+        self,
+        hooks: list[t.Any],  # HookDefinition
+        max_parallel: int = 3,
+        timeout: int = 300,
+    ) -> list[t.Any]:  # list[HookResult]
+        """Execute hooks according to strategy.
+
+        Args:
+            hooks: List of hook definitions to execute
+            max_parallel: Maximum concurrent executions (ignored for sequential)
+            timeout: Default timeout per hook in seconds
+
+        Returns:
+            List of HookResult objects
+        """
+        ...
+
+    def get_execution_order(
+        self,
+        hooks: list[t.Any],  # HookDefinition
+    ) -> list[list[t.Any]]:  # list[list[HookDefinition]]
+        """Return batches of hooks for execution.
+
+        Sequential strategy returns one hook per batch.
+        Parallel strategy groups independent hooks into batches.
+
+        Args:
+            hooks: List of hook definitions
+
+        Returns:
+            List of hook batches for execution
+        """
+        ...
+
+
+@t.runtime_checkable
+class CacheStrategyProtocol(t.Protocol):
+    """Protocol for result caching strategies.
+
+    Implementations:
+    - ToolProxyCacheAdapter: Bridges to existing tool_proxy cache
+    - RedisCacheAdapter: Redis-backed caching (Phase 4+)
+    - MemoryCacheAdapter: In-memory LRU cache for testing
+    """
+
+    async def get(self, key: str) -> t.Any | None:  # HookResult | None
+        """Retrieve cached result.
+
+        Args:
+            key: Cache key (computed from hook + file content)
+
+        Returns:
+            Cached HookResult if found, None otherwise
+        """
+        ...
+
+    async def set(self, key: str, result: t.Any, ttl: int = 3600) -> None:
+        """Cache result with TTL.
+
+        Args:
+            key: Cache key
+            result: HookResult to cache
+            ttl: Time-to-live in seconds
+        """
+        ...
+
+    def compute_key(self, hook: t.Any, files: list[Path]) -> str:
+        """Compute cache key from hook and file content.
+
+        Key format: {hook_name}:{config_hash}:{content_hash}
+
+        Args:
+            hook: HookDefinition
+            files: List of files being checked
+
+        Returns:
+            Cache key string
+        """
+        ...
+
+
+@t.runtime_checkable
+class HookOrchestratorProtocol(t.Protocol):
+    """Protocol for hook orchestration.
+
+    The orchestrator manages hook lifecycle, dependency resolution,
+    and execution strategies. Supports dual execution mode for migration.
+    """
+
+    async def init(self) -> None:
+        """Initialize orchestrator and build dependency graph."""
+        ...
+
+    async def execute_strategy(
+        self,
+        strategy: t.Any,  # HookStrategy
+        execution_mode: str | None = None,
+        execution_context: t.Any | None = None,
+    ) -> list[t.Any]:  # list[HookResult]
+        """Execute hook strategy with specified mode.
+
+        Args:
+            strategy: HookStrategy (fast or comprehensive)
+            execution_mode: "legacy" (pre-commit CLI) or "acb" (direct adapters)
+            execution_context: Context containing options and execution environment
+
+        Returns:
+            List of HookResult objects
+        """
+        ...
+
+    @property
+    def module_id(self) -> t.Any:  # UUID
+        """Reference to module-level MODULE_ID."""
+        ...
+
+    @property
+    def adapter_name(self) -> str:
+        """Human-readable adapter name."""
+        ...
+
+
+# ==================== Performance & Quality Protocols ====================
+
+
+@t.runtime_checkable
+class PerformanceMonitorProtocol(t.Protocol):
+    """Protocol for performance monitoring."""
+
+    def start_workflow(self, workflow_id: str) -> None:
+        """Start monitoring a workflow."""
+        ...
+
+    def end_workflow(
+        self, workflow_id: str, success: bool = True
+    ) -> t.Any:  # WorkflowPerformance
+        """End workflow monitoring and return performance data."""
+        ...
+
+    def start_phase(self, workflow_id: str, phase_name: str) -> None:
+        """Start monitoring a phase."""
+        ...
+
+    def end_phase(
+        self, workflow_id: str, phase_name: str, success: bool = True
+    ) -> t.Any:  # PhasePerformance
+        """End phase monitoring and return performance data."""
+        ...
+
+    def get_performance_summary(self, last_n_workflows: int = 10) -> dict[str, t.Any]:
+        """Get performance summary for recent workflows."""
+        ...
+
+    def get_benchmark_trends(self) -> dict[str, dict[str, t.Any]]:
+        """Get benchmark trend analysis."""
+        ...
+
+
+@t.runtime_checkable
+class MemoryOptimizerProtocol(t.Protocol):
+    """Protocol for memory optimization."""
+
+    def record_checkpoint(self, name: str = "") -> float:
+        """Record a memory checkpoint and return current usage."""
+        ...
+
+    def get_stats(self) -> dict[str, t.Any]:
+        """Get memory optimization statistics."""
+        ...
+
+
+@t.runtime_checkable
+class PerformanceCacheProtocol(t.Protocol):
+    """Protocol for performance caching."""
+
+    def get(self, key: str) -> ExecutionResult | None:
+        """Get cached value."""
+        ...
+
+    def set(self, key: str, value: ExecutionResult, ttl: int = 3600) -> None:
+        """Set cached value with TTL."""
+        ...
+
+    def invalidate(self, key: str) -> bool:
+        """Invalidate cache entry."""
+        ...
+
+    def clear_all(self) -> None:
+        """Clear all cache entries."""
+        ...
+
+
+@t.runtime_checkable
+class QualityBaselineProtocol(t.Protocol):
+    """Protocol for quality baseline tracking."""
+
+    def get_current_baseline(self) -> dict[str, t.Any]:
+        """Get current baseline metrics."""
+        ...
+
+    def update_baseline(self, metrics: dict[str, t.Any]) -> bool:
+        """Update baseline with new metrics."""
+        ...
+
+    def compare(self, current: dict[str, t.Any]) -> dict[str, t.Any]:
+        """Compare current metrics against baseline."""
+        ...
+
+
+@t.runtime_checkable
+class ParallelExecutorProtocol(t.Protocol):
+    """Protocol for parallel task execution."""
+
+    async def execute_parallel(
+        self,
+        tasks: list[t.Any],
+        max_workers: int = 3,
+    ) -> list[t.Any]:
+        """Execute tasks in parallel."""
+        ...
+
+    def get_results(self) -> list[t.Any]:
+        """Get execution results."""
+        ...
+
+
+@t.runtime_checkable
+class ParallelHookExecutorProtocol(ServiceProtocol, t.Protocol):
+    """Protocol for parallel hook executor service."""
+
+    async def execute_hooks_parallel(
+        self,
+        hooks: list[HookDefinition],
+        hook_runner: t.Callable[[HookDefinition], t.Awaitable[ExecutionResult]],
+    ) -> ParallelExecutionResult: ...
+
+
+@t.runtime_checkable
+class AsyncCommandExecutorProtocol(ServiceProtocol, t.Protocol):
+    """Protocol for async command executor service."""
+
+    async def execute_command(
+        self,
+        command: list[str],
+        cwd: Path | None = None,
+        timeout: int = 60,
+        cache_ttl: int = 120,
+    ) -> ExecutionResult: ...
+
+    async def execute_commands_batch(
+        self,
+        commands: list[tuple[list[str], Path | None]],
+        timeout: int = 60,
+    ) -> list[ExecutionResult]: ...
+
+
+@t.runtime_checkable
+class PerformanceBenchmarkProtocol(t.Protocol):
+    """Protocol for performance benchmarking."""
+
+    def run_benchmark(self, operation: str) -> dict[str, t.Any]: ...
+
+    def get_report(self) -> dict[str, t.Any]: ...
+
+    def compare_benchmarks(
+        self,
+        baseline: dict[str, t.Any],
+        current: dict[str, t.Any],
+    ) -> dict[str, t.Any]: ...
+
+
+@t.runtime_checkable
+class PerformanceBenchmarkServiceProtocol(ServiceProtocol, t.Protocol):
+    """Protocol for performance benchmark service."""
+
+    async def run_benchmark_suite(self) -> t.Any | None: ...  # BenchmarkSuite
+
+    def export_results(
+        self, suite: t.Any, output_path: Path
+    ) -> None: ...  # BenchmarkSuite
+
+
+@t.runtime_checkable
+class DebugServiceProtocol(ServiceProtocol, t.Protocol):
+    """Protocol for AI agent debugging services."""
+
+    def start_debug_session(self, session_id: str) -> None: ...
+
+    @property
+    def enabled(self) -> bool: ...
+
+    def log_workflow_phase(
+        self,
+        phase: str,
+        status: str,
+        details: dict[str, t.Any] | None = None,
+        duration: float | None = None,
+    ) -> None: ...
+
+    def set_workflow_success(self, success: bool) -> None: ...
+
+    def print_debug_summary(self) -> None: ...
+
+    def log_iteration_start(self, iteration_number: int) -> None: ...
+
+    def log_iteration_end(self, iteration_number: int, success: bool) -> None: ...
+
+    def log_test_failures(self, count: int) -> None: ...
+
+    def log_test_fixes(self, count: int) -> None: ...
+
+    def log_hook_failures(self, count: int) -> None: ...
+
+    def log_hook_fixes(self, count: int) -> None: ...
+
+
+@t.runtime_checkable
+class QualityIntelligenceProtocol(ServiceProtocol, t.Protocol):
+    """Protocol for quality intelligence services."""
+
+    def analyze_quality_trends(self) -> dict[str, t.Any]:
+        """Analyze quality trends."""
+        ...
+
+    def predict_quality_issues(self) -> list[dict[str, t.Any]]:
+        """Predict potential quality issues."""
+        ...
+
+    def recommend_improvements(self) -> list[dict[str, t.Any]]:
+        """Recommend quality improvements."""
+        ...
+
+    def get_intelligence_report(self) -> dict[str, t.Any]:
+        """Get quality intelligence report."""
+        ...
+
+
+@t.runtime_checkable
+class CoverageRatchetServiceProtocol(ServiceProtocol, t.Protocol):
+    """Protocol for coverage ratchet services."""
+
+    def get_current_coverage(self) -> float:
+        """Get current test coverage percentage."""
+        ...
+
+    def get_coverage_history(self) -> list[dict[str, t.Any]]:
+        """Get coverage history."""
+        ...
+
+    def check_coverage_increase(self) -> bool:
+        """Check if coverage has increased."""
+        ...
+
+    def get_next_milestone(self) -> float | None:
+        """Get the next coverage milestone."""
+        ...
+
+    def update_coverage_baseline(self, new_baseline: float) -> bool:
+        """Update the coverage baseline."""
+        ...
+
+
+@t.runtime_checkable
+class ServerManagerProtocol(ServiceProtocol, t.Protocol):
+    """Protocol for server management services."""
+
+    def find_processes(self, process_name: str) -> list[dict[str, t.Any]]:
+        """Find running processes matching name."""
+        ...
+
+    def start_server(self, command: list[str]) -> bool:
+        """Start a server."""
+        ...
+
+    def stop_server(self, process_id: int) -> bool:
+        """Stop a server."""
+        ...
+
+    def restart_server(self, command: list[str]) -> bool:
+        """Restart a server."""
+        ...
+
+    def get_server_status(self) -> dict[str, t.Any]:
+        """Get server status information."""
+        ...
+
+
+@t.runtime_checkable
+class LogManagementProtocol(ServiceProtocol, t.Protocol):
+    """Protocol for log management services."""
+
+    def get_log_manager(self) -> t.Any:
+        """Get log management instance."""
+        ...
+
+    def setup_structured_logging(
+        self,
+        *,
+        level: str = "INFO",
+        json_output: bool = False,
+        log_file: Path | None = None,
+    ) -> None:
+        """Setup structured logging."""
+        ...
+
+    def write_log(self, message: str, level: str = "INFO") -> None:
+        """Write a log message."""
+        ...
+
+    def get_logs(
+        self, filter_criteria: dict[str, t.Any] | None = None
+    ) -> list[dict[str, t.Any]]:
+        """Get logs based on filter criteria."""
+        ...
+
+
+# Protocol definitions for services imported directly in managers
+
+
+@t.runtime_checkable
+class RegexPatternsProtocol(t.Protocol):
+    """Protocol for regex patterns service."""
+
+    def update_pyproject_version(self, content: str, new_version: str) -> str: ...
+
+    def remove_coverage_fail_under(self, content: str) -> str: ...
+
+    def update_version_in_changelog(self, content: str, new_version: str) -> str: ...
+
+    def mask_tokens_in_text(self, text: str) -> str: ...
+
+
+@t.runtime_checkable
+class SecureStatusFormatterProtocol(t.Protocol):
+    """Protocol for secure status formatter service."""
+
+    def format_status(
+        self,
+        status_data: dict[str, t.Any],
+        verbosity: t.Any,  # StatusVerbosity
+        user_context: str | None = None,
+    ) -> dict[str, t.Any]: ...
+
+    def format_error_response(
+        self,
+        error_message: str,
+        verbosity: t.Any,  # StatusVerbosity
+        include_details: bool = False,
+    ) -> dict[str, t.Any]: ...
+
+
+@t.runtime_checkable
+class GitServiceProtocol(t.Protocol):
+    """Protocol for Git service."""
+
+    def get_current_branch(self) -> str: ...
+
+    def get_commit_history(self, since_commit: str | None = None) -> list[str]: ...
+
+    def create_new_branch(self, branch_name: str) -> bool: ...
+
+    def commit_changes(self, message: str) -> bool: ...
+
+    def push_changes(self) -> bool: ...
+
+    def create_pull_request(self, title: str, body: str) -> bool: ...
+
+    def get_changed_files_since(self, since: str, project_root: Path) -> list[Path]: ...
+
+    def get_staged_files(self, project_root: Path) -> list[Path]: ...
+
+    def get_unstaged_files(self, project_root: Path) -> list[Path]: ...
+
+
+@t.runtime_checkable
+class SmartFileFilterProtocol(ServiceProtocol, t.Protocol):
+    """Protocol for smart file filter service."""
+
+    def get_changed_files(self, since: str = "HEAD") -> list[Path]: ...
+
+    def get_staged_files(self) -> list[Path]: ...
+
+    def get_unstaged_files(self) -> list[Path]: ...
+
+    def filter_by_pattern(self, files: list[Path], pattern: str) -> list[Path]: ...
+
+    def filter_by_tool(self, files: list[Path], tool: str) -> list[Path]: ...
+
+    def get_all_modified_files(self) -> list[Path]: ...
+
+    def filter_by_extensions(
+        self, files: list[Path], extensions: list[str]
+    ) -> list[Path]: ...
+
+    def get_python_files(self, files: list[Path]) -> list[Path]: ...
+
+    def get_markdown_files(self, files: list[Path]) -> list[Path]: ...
+
+
+@t.runtime_checkable
+class SafeFileModifierProtocol(ServiceProtocol, t.Protocol):
+    """Protocol for safe file modification service."""
+
+    async def apply_fix(
+        self,
+        file_path: str,
+        fixed_content: str,
+        dry_run: bool = False,
+        create_backup: bool = True,
+    ) -> dict[str, t.Any]: ...
+
+
+@t.runtime_checkable
+class VersionAnalyzerProtocol(t.Protocol):
+    """Protocol for version analysis service."""
+
+    def analyze_changes(self, commit_messages: list[str]) -> dict[str, t.Any]: ...
+
+    def recommend_next_version(self) -> str: ...
+
+    def get_version_bump_type(self, changes: dict[str, t.Any]) -> str: ...
+
+
+@t.runtime_checkable
+class HealthMetricsServiceProtocol(ServiceProtocol, t.Protocol):
+    """Protocol for health metrics service."""
+
+    def collect_current_metrics(self) -> t.Any: ...  # ProjectHealth
+
+    def analyze_project_health(
+        self, save_metrics: bool = True
+    ) -> t.Any: ...  # ProjectHealth
+
+    def report_health_status(self, health: t.Any) -> None: ...  # ProjectHealth
+
+    def get_health_trend_summary(self, days: int = 30) -> dict[str, t.Any]: ...
+
+
+@t.runtime_checkable
+class ChangelogGeneratorProtocol(t.Protocol):
+    """Protocol for changelog generation service."""
+
+    def generate_changelog_entries(self, changes: dict[str, t.Any]) -> list[str]: ...
+
+    def write_changelog(
+        self, entries: list[str], changelog_file: str | Path
+    ) -> bool: ...
+
+    def update_changelog_with_version(
+        self, changelog_file: str | Path, version: str
+    ) -> bool: ...
+
+
+@t.runtime_checkable
+class CoverageBadgeServiceProtocol(ServiceProtocol, t.Protocol):
+    """Protocol for coverage badge service."""
+
+    def update_readme_coverage_badge(self, coverage_percent: float) -> bool: ...
+
+    def should_update_badge(self, coverage_percent: float) -> bool: ...
+
+
+# ============================================================================
+# Agent System Protocols (Phase 4)
+# ============================================================================
+
+
+@t.runtime_checkable
+class AgentCoordinatorProtocol(ServiceProtocol, t.Protocol):
+    """Protocol for agent coordination and issue handling.
+
+    The AgentCoordinator manages a pool of specialized AI agents that can
+    diagnose and fix various code quality issues. It routes issues to
+    appropriate agents, handles agent execution, and aggregates results.
+    """
+
+    def initialize_agents(self) -> None:
+        """Initialize all registered agents."""
+        ...
+
+    async def handle_issues(
+        self, issues: list[t.Any]
+    ) -> t.Any:  # list[Issue] -> FixResult
+        """Handle a batch of issues using appropriate specialist agents.
+
+        Args:
+            issues: List of Issue objects to be processed
+
+        Returns:
+            FixResult containing success status, confidence, and applied fixes
+        """
+        ...
+
+    async def handle_issues_proactively(
+        self, issues: list[t.Any]
+    ) -> t.Any:  # list[Issue] -> FixResult
+        """Handle issues with proactive architectural planning.
+
+        Uses ArchitectAgent to create a strategic plan before applying fixes.
+
+        Args:
+            issues: List of Issue objects to be processed
+
+        Returns:
+            FixResult containing success status, confidence, and applied fixes
+        """
+        ...
+
+    def get_agent_capabilities(self) -> dict[str, dict[str, t.Any]]:
+        """Get capabilities of all registered agents.
+
+        Returns:
+            Dict mapping agent names to their supported issue types and metadata
+        """
+        ...
+
+    def set_proactive_mode(self, enabled: bool) -> None:
+        """Enable or disable proactive architectural planning mode.
+
+        Args:
+            enabled: Whether to use proactive planning
+        """
+        ...
+
+
+@t.runtime_checkable
+class AgentTrackerProtocol(t.Protocol):
+    """Protocol for tracking agent execution and metrics.
+
+    The AgentTracker monitors agent activity, collects performance metrics,
+    and provides insights into agent effectiveness and success rates.
+    """
+
+    def register_agents(self, agent_types: list[str]) -> None:
+        """Register agent types for tracking.
+
+        Args:
+            agent_types: List of agent class names
+        """
+        ...
+
+    def set_coordinator_status(self, status: str) -> None:
+        """Set the overall coordinator status.
+
+        Args:
+            status: Status string (e.g., 'active', 'idle', 'processing')
+        """
+        ...
+
+    def track_agent_processing(
+        self, agent_name: str, issue: t.Any, confidence: float
+    ) -> None:  # issue: Issue
+        """Track when an agent begins processing an issue.
+
+        Args:
+            agent_name: Name of the agent
+            issue: Issue being processed
+            confidence: Agent's confidence in handling this issue (0.0-1.0)
+        """
+        ...
+
+    def track_agent_complete(
+        self, agent_name: str, result: t.Any
+    ) -> None:  # result: FixResult
+        """Track agent completion and results.
+
+        Args:
+            agent_name: Name of the agent
+            result: FixResult from agent execution
+        """
+        ...
+
+    def get_agent_stats(self) -> dict[str, t.Any]:
+        """Get aggregate statistics for all agents.
+
+        Returns:
+            Dict containing success rates, average confidence, etc.
+        """
+        ...
+
+
+@t.runtime_checkable
+class AgentDebuggerProtocol(t.Protocol):
+    """Protocol for agent debugging and activity logging.
+
+    The AgentDebugger provides detailed logging for agent activities,
+    enabling troubleshooting and performance analysis.
+    """
+
+    def log_agent_activity(
+        self,
+        agent_name: str,
+        activity: str,
+        **metadata: t.Any,
+    ) -> None:
+        """Log an agent activity with optional metadata.
+
+        Args:
+            agent_name: Name of the agent
+            activity: Activity type (e.g., 'processing_started', 'processing_completed')
+            **metadata: Additional context (issue_id, confidence, result, etc.)
+        """
+        ...
+
+    def get_activity_log(
+        self, agent_name: str | None = None, limit: int = 100
+    ) -> list[dict[str, t.Any]]:
+        """Get recent activity log entries.
+
+        Args:
+            agent_name: Optional filter by agent name
+            limit: Maximum number of entries to return
+
+        Returns:
+            List of activity log entries
+        """
+        ...
+
+    def enable_verbose_mode(self, enabled: bool = True) -> None:
+        """Enable or disable verbose debugging mode.
+
+        Args:
+            enabled: Whether to enable verbose output
+        """
+        ...
+
+
+# ============================================================================
+# Orchestration Protocols (Phase 4)
+# ============================================================================
+
+
+@t.runtime_checkable
+class ServiceWatchdogProtocol(ServiceProtocol, t.Protocol):
+    """Protocol for service health monitoring and restart coordination.
+
+    The ServiceWatchdog monitors long-running services (MCP server, WebSocket
+    server, LSP servers) and automatically restarts them on failure.
+    """
+
+    def register_service(self, config: t.Any) -> None:  # config: ServiceConfig
+        """Register a service for monitoring.
+
+        Args:
+            config: ServiceConfig with command, health checks, and restart policy
+        """
+        ...
+
+    async def start(self) -> None:
+        """Start the watchdog monitoring loop."""
+        ...
+
+    async def stop(self) -> None:
+        """Stop the watchdog and shutdown monitored services."""
+        ...
+
+    async def restart_service(self, service_name: str) -> bool:
+        """Manually restart a specific service.
+
+        Args:
+            service_name: Name of service to restart
+
+        Returns:
+            True if restart successful
+        """
+        ...
+
+    def get_service_status(
+        self, service_name: str
+    ) -> t.Any | None:  # ServiceStatus | None
+        """Get current status of a specific service.
+
+        Args:
+            service_name: Name of service
+
+        Returns:
+            ServiceStatus object or None if not found
+        """
+        ...
+
+    def get_all_services_status(self) -> dict[str, t.Any]:  # dict[str, ServiceStatus]
+        """Get status of all monitored services.
+
+        Returns:
+            Dict mapping service names to ServiceStatus objects
+        """
+        ...
+
+    async def check_service_health(self, service_name: str) -> bool:
+        """Perform health check on a specific service.
+
+        Args:
+            service_name: Name of service to check
+
+        Returns:
+            True if service is healthy
+        """
+        ...
+
+
+@t.runtime_checkable
+class TimeoutManagerProtocol(t.Protocol):
+    """Protocol for timeout management and strategies.
+
+    The TimeoutManager provides centralized timeout configuration for
+    various operations (hooks, tests, service startups, etc).
+    """
+
+    def get_timeout(self, operation: str) -> float:
+        """Get timeout for a specific operation type.
+
+        Args:
+            operation: Operation type (e.g., 'hook_execution', 'test_run')
+
+        Returns:
+            Timeout in seconds
+        """
+        ...
+
+    def set_timeout(self, operation: str, timeout: float) -> None:
+        """Set timeout for a specific operation type.
+
+        Args:
+            operation: Operation type
+            timeout: Timeout in seconds
+        """
+        ...
+
+    def get_strategy(self, operation: str) -> t.Any:  # TimeoutStrategy
+        """Get timeout strategy for an operation.
+
+        Args:
+            operation: Operation type
+
+        Returns:
+            TimeoutStrategy enum value
+        """
+        ...
+
+    def apply_timeout(
+        self,
+        operation: str,
+        func: t.Callable[..., t.Any],
+        *args: t.Any,
+        **kwargs: t.Any,
+    ) -> t.Any:
+        """Apply timeout to a function execution.
+
+        Args:
+            operation: Operation type (determines timeout value)
+            func: Function to execute with timeout
+            *args: Positional arguments for function
+            **kwargs: Keyword arguments for function
+
+        Returns:
+            Function result
+
+        Raises:
+            TimeoutError: If operation exceeds timeout
+        """
+        ...