style: fix import sorting with isort

- Fixed 36 files with incorrect import order
- Updated pyproject.toml to disable skip_gitignore for compatibility
- All imports now follow black profile
- Fixes GitHub Actions CI isort check failures

Author: zakariaf

app/main.py

@@ -19,7 +19,7 @@
     default_logging_config,
 )
 from app.api.routes import health, metrics, query
-from app.api.routes.docs import TAGS_METADATA, API_DESCRIPTION
+from app.api.routes.docs import API_DESCRIPTION, TAGS_METADATA
 from app.config import config
 from app.repositories.redis_repository import create_redis_pool
 from app.utils.logger import get_logger, setup_logging

app/monitoring/__init__.py

@@ -4,31 +4,31 @@
 Provides Prometheus-compatible metrics for observability.
 """
 
-from app.monitoring.prometheus import (
-    PrometheusMetrics,
-    get_metrics,
-    metrics_middleware,
+from app.monitoring.aggregator import (
+    AggregatedMetrics,
+    MetricsAggregator,
+)
+from app.monitoring.alerts import (
+    AlertLevel,
+    AlertManager,
+    AlertRule,
 )
 from app.monitoring.collectors import (
-    RequestMetrics,
     CacheMetrics,
     LLMMetrics,
+    RequestMetrics,
     SystemMetrics,
 )
 from app.monitoring.decorators import (
-    track_request,
     track_cache_operation,
-    track_llm_call,
     track_latency,
+    track_llm_call,
+    track_request,
 )
-from app.monitoring.aggregator import (
-    MetricsAggregator,
-    AggregatedMetrics,
-)
-from app.monitoring.alerts import (
-    AlertRule,
-    AlertManager,
-    AlertLevel,
+from app.monitoring.prometheus import (
+    PrometheusMetrics,
+    get_metrics,
+    metrics_middleware,
 )
 
 __all__ = [

app/optimization/__init__.py

@@ -5,33 +5,33 @@
 caching, and resource management.
 """
 
-from app.optimization.query_optimizer import (
-    QueryOptimizer,
-    OptimizationConfig,
+from app.optimization.async_optimizer import (
+    AsyncOptimizer,
+    ConcurrencyLimiter,
+)
+from app.optimization.batch_processor import (
+    BatchConfig,
+    BatchProcessor,
 )
 from app.optimization.cache_optimizer import (
     CacheOptimizer,
     CacheStrategy,
 )
-from app.optimization.embedding_optimizer import (
-    EmbeddingOptimizer,
-    BatchEmbeddingProcessor,
-)
 from app.optimization.connection_pool import (
     ConnectionPoolManager,
     PoolConfig,
 )
-from app.optimization.async_optimizer import (
-    AsyncOptimizer,
-    ConcurrencyLimiter,
+from app.optimization.embedding_optimizer import (
+    BatchEmbeddingProcessor,
+    EmbeddingOptimizer,
 )
 from app.optimization.memory_optimizer import (
-    MemoryOptimizer,
     MemoryConfig,
+    MemoryOptimizer,
 )
-from app.optimization.batch_processor import (
-    BatchProcessor,
-    BatchConfig,
+from app.optimization.query_optimizer import (
+    OptimizationConfig,
+    QueryOptimizer,
 )
 
 __all__ = [

app/pipeline/__init__.py

@@ -19,8 +19,8 @@
 from app.pipeline.async_processor import AsyncQueryProcessor, AsyncResult
 from app.pipeline.deduplication import (
     DeduplicatingProcessor,
-    QueryDeduplicator,
     DeduplicationStats,
+    QueryDeduplicator,
 )
 from app.pipeline.error_recovery import (
     PipelineErrorHandler,
@@ -42,8 +42,8 @@
     track_operation,
 )
 from app.pipeline.pipeline_builder import (
-    PipelineConfig,
     PipelineComponents,
+    PipelineConfig,
     QueryPipeline,
     QueryPipelineBuilder,
 )

app/pipeline/deduplication.py

@@ -14,7 +14,7 @@
 import time
 from collections import OrderedDict
 from dataclasses import dataclass
-from typing import Dict, Optional, Any
+from typing import Any, Dict, Optional
 
 from app.models.response import QueryResponse
 from app.utils.logger import get_logger

app/pipeline/pipeline_builder.py

@@ -10,7 +10,7 @@
 """
 
 from dataclasses import dataclass, field
-from typing import Callable, List, Optional, Any
+from typing import Any, Callable, List, Optional
 
 from app.cache.redis_cache import RedisCache
 from app.embeddings.embedding_generator import EmbeddingGenerator

app/pipeline/result_aggregator.py

@@ -13,7 +13,7 @@
 from enum import Enum
 from typing import Any, Callable, Dict, List, Optional
 
-from app.models.response import QueryResponse, CacheInfo, UsageMetrics
+from app.models.response import CacheInfo, QueryResponse, UsageMetrics
 from app.utils.logger import get_logger
 
 logger = get_logger(__name__)

pyproject.toml

@@ -23,7 +23,7 @@ extend-exclude = '''
 profile = "black"
 line_length = 88
 known_first_party = ["app"]
-skip_gitignore = true
+skip_gitignore = false
 
 [tool.pytest.ini_options]
 minversion = "7.0"

tests/integration/test_api_endpoints.py

@@ -1,8 +1,8 @@
 """Integration tests for API endpoints."""
 
-import pytest
 from unittest.mock import AsyncMock, MagicMock, patch
 
+import pytest
 from fastapi.testclient import TestClient
 
 from app.main import app

tests/integration/test_multi_provider.py

@@ -1,16 +1,17 @@
 """Integration tests for multi-provider LLM flow."""
 
-import pytest
 from unittest.mock import AsyncMock, MagicMock, patch
 
-from app.llm.factory import LLMProviderFactory
-from app.llm.openai_provider import OpenAIProvider
+import pytest
+
+from app.exceptions import LLMProviderError
 from app.llm.anthropic_provider import AnthropicProvider
+from app.llm.factory import LLMProviderFactory
 from app.llm.fallback_strategy import LLMFallbackStrategy
+from app.llm.openai_provider import OpenAIProvider
 from app.llm.provider_selector import LLMProviderSelector
-from app.models.query import QueryRequest
 from app.models.llm import LLMResponse
-from app.exceptions import LLMProviderError
+from app.models.query import QueryRequest
 
 
 class TestMultiProviderFlow: