ccproxy-api 0.1.7__py3-none-any.whl → 0.2.0a4__py3-none-any.whl

ccproxy/api/app.py

@@ -1,61 +1,85 @@
-"""FastAPI application factory for CCProxy API Server."""
+"""FastAPI application factory for CCProxy API Server with plugin system."""
 
 from collections.abc import AsyncGenerator, Awaitable, Callable
 from contextlib import asynccontextmanager
-from typing import Any, TypedDict
+from enum import Enum
+from typing import Any
 
-from fastapi import APIRouter, FastAPI
-from fastapi.staticfiles import StaticFiles
-from structlog import get_logger
+import structlog
+from fastapi import FastAPI
+from fastapi.routing import APIRouter
+from typing_extensions import TypedDict
 
-from ccproxy import __version__
+from ccproxy.api.bootstrap import create_service_container
+from ccproxy.api.format_validation import validate_route_format_chains
 from ccproxy.api.middleware.cors import setup_cors_middleware
 from ccproxy.api.middleware.errors import setup_error_handlers
-from ccproxy.api.middleware.logging import AccessLogMiddleware
-from ccproxy.api.middleware.request_content_logging import (
-    RequestContentLoggingMiddleware,
-)
-from ccproxy.api.middleware.request_id import RequestIDMiddleware
-from ccproxy.api.middleware.server_header import ServerHeaderMiddleware
-from ccproxy.api.routes.claude import router as claude_router
-from ccproxy.api.routes.codex import router as codex_router
 from ccproxy.api.routes.health import router as health_router
-from ccproxy.api.routes.mcp import setup_mcp
-from ccproxy.api.routes.metrics import (
-    dashboard_router,
-    logs_router,
-    prometheus_router,
+from ccproxy.api.routes.plugins import router as plugins_router
+from ccproxy.auth.oauth.router import oauth_router
+from ccproxy.config.settings import Settings
+from ccproxy.core import __version__
+from ccproxy.core.async_task_manager import start_task_manager, stop_task_manager
+from ccproxy.core.logging import TraceBoundLogger, get_logger, setup_logging
+from ccproxy.core.plugins import (
+    MiddlewareManager,
+    PluginRegistry,
+    load_plugin_system,
+    setup_default_middleware,
 )
-from ccproxy.api.routes.permissions import router as permissions_router
-from ccproxy.api.routes.proxy import router as proxy_router
-from ccproxy.auth.oauth.routes import router as oauth_router
-from ccproxy.config.settings import Settings, get_settings
-from ccproxy.core.logging import setup_logging
-from ccproxy.utils.models_provider import get_models_list
+from ccproxy.core.plugins.hooks import HookManager
+from ccproxy.core.plugins.hooks.events import HookEvent
+from ccproxy.services.container import ServiceContainer
 from ccproxy.utils.startup_helpers import (
     check_claude_cli_startup,
-    check_codex_cli_startup,
     check_version_updates_startup,
-    flush_streaming_batches_shutdown,
-    initialize_claude_detection_startup,
-    initialize_claude_sdk_startup,
-    initialize_codex_detection_startup,
-    initialize_log_storage_shutdown,
-    initialize_log_storage_startup,
-    initialize_permission_service_startup,
-    setup_permission_service_shutdown,
     setup_scheduler_shutdown,
     setup_scheduler_startup,
-    setup_session_manager_shutdown,
-    validate_claude_authentication_startup,
-    validate_codex_authentication_startup,
 )
 
 
-logger = get_logger(__name__)
+logger: TraceBoundLogger = get_logger()
+
+
+def merge_router_tags(
+    router: APIRouter,
+    spec_tags: list[str] | None = None,
+    default_tags: list[str] | None = None,
+) -> list[str | Enum] | None:
+    """Merge router tags with spec tags, removing duplicates while preserving order.
+
+    Args:
+        router: FastAPI router instance
+        spec_tags: Tags from route specification
+        default_tags: Fallback tags if no other tags exist
+
+    Returns:
+        Deduplicated list of tags, or None if no tags
+    """
+    router_tags: list[str | Enum] = list(router.tags) if router.tags else []
+    spec_tags_list: list[str | Enum] = list(spec_tags) if spec_tags else []
+    default_tags_list: list[str | Enum] = list(default_tags) if default_tags else []
+
+    # Only use defaults if no other tags exist
+    if not router_tags and not spec_tags_list and default_tags_list:
+        return default_tags_list
+
+    # Merge all non-default tags and deduplicate
+    all_tags: list[str | Enum] = router_tags + spec_tags_list
+    if not all_tags:
+        return None
+
+    # Deduplicate by string value while preserving order
+    unique: list[str | Enum] = []
+    seen: set[str] = set()
+    for t in all_tags:
+        s = str(t)
+        if s not in seen:
+            seen.add(s)
+            unique.append(t)
+    return unique
 
 
-# Type definitions for lifecycle components
 class LifecycleComponent(TypedDict):
     name: str
     startup: Callable[[FastAPI, Any], Awaitable[None]] | None
@@ -71,47 +95,159 @@ class ShutdownComponent(TypedDict):
     shutdown: Callable[[FastAPI], Awaitable[None]] | None
 
 
-# Define lifecycle components for startup/shutdown organization
+async def setup_task_manager_startup(app: FastAPI, settings: Settings) -> None:
+    """Start the async task manager."""
+    container: ServiceContainer = app.state.service_container
+    await start_task_manager(container=container)
+    logger.debug("task_manager_startup_completed", category="lifecycle")
+
+
+async def setup_task_manager_shutdown(app: FastAPI) -> None:
+    """Stop the async task manager."""
+    container: ServiceContainer = app.state.service_container
+    await stop_task_manager(container=container)
+    logger.debug("task_manager_shutdown_completed", category="lifecycle")
+
+
+async def setup_service_container_shutdown(app: FastAPI) -> None:
+    """Close the service container and its resources."""
+    if hasattr(app.state, "service_container"):
+        service_container = app.state.service_container
+        await service_container.shutdown()
+
+
+async def initialize_plugins_startup(app: FastAPI, settings: Settings) -> None:
+    """Initialize plugins during startup (runtime phase)."""
+    if not settings.enable_plugins:
+        logger.info("plugin_system_disabled", category="lifecycle")
+        return
+
+    if not hasattr(app.state, "plugin_registry"):
+        logger.warning("plugin_registry_not_found", category="lifecycle")
+        return
+
+    plugin_registry: PluginRegistry = app.state.plugin_registry
+    service_container: ServiceContainer = app.state.service_container
+
+    hook_registry = service_container.get_hook_registry()
+    background_thread_manager = service_container.get_background_hook_thread_manager()
+    hook_manager = HookManager(hook_registry, background_thread_manager)
+    app.state.hook_registry = hook_registry
+    app.state.hook_manager = hook_manager
+    service_container.register_service(HookManager, instance=hook_manager)
+
+    # StreamingHandler now requires HookManager at construction via DI factory,
+    # so no post-hoc patching is needed here.
+
+    # Perform manifest population with access to http_pool_manager
+    # This allows plugins to modify their manifests during context creation
+    for plugin_name, factory in plugin_registry.factories.items():
+        try:
+            factory.create_context(service_container)
+        except Exception as e:
+            logger.warning(
+                "plugin_context_creation_failed",
+                plugin=plugin_name,
+                error=str(e),
+                exc_info=e,
+                category="plugin",
+            )
+            # Continue with other plugins
+
+    await plugin_registry.initialize_all(service_container)
+    # A consolidated summary is already emitted by PluginRegistry.initialize_all()
+
+
+async def shutdown_plugins(app: FastAPI) -> None:
+    """Shutdown plugins."""
+    if hasattr(app.state, "plugin_registry"):
+        plugin_registry: PluginRegistry = app.state.plugin_registry
+        await plugin_registry.shutdown_all()
+        logger.debug("plugins_shutdown_completed", category="lifecycle")
+
+
+async def shutdown_hook_system(app: FastAPI) -> None:
+    """Shutdown the hook system and background thread."""
+    try:
+        # Get hook manager from app state - it will shutdown its own background manager
+        hook_manager = getattr(app.state, "hook_manager", None)
+        if hook_manager:
+            hook_manager.shutdown()
+
+        logger.debug("hook_system_shutdown_completed", category="lifecycle")
+    except Exception as e:
+        logger.error(
+            "hook_system_shutdown_failed",
+            error=str(e),
+            category="lifecycle",
+        )
+
+
+async def initialize_hooks_startup(app: FastAPI, settings: Settings) -> None:
+    """Initialize hook system with plugins."""
+    if hasattr(app.state, "hook_registry") and hasattr(app.state, "hook_manager"):
+        hook_registry = app.state.hook_registry
+        hook_manager = app.state.hook_manager
+        logger.debug("hook_system_already_created", category="lifecycle")
+    else:
+        service_container: ServiceContainer = app.state.service_container
+        hook_registry = service_container.get_hook_registry()
+        background_thread_manager = (
+            service_container.get_background_hook_thread_manager()
+        )
+        hook_manager = HookManager(hook_registry, background_thread_manager)
+        app.state.hook_registry = hook_registry
+        app.state.hook_manager = hook_manager
+
+    # Register plugin hooks
+    if hasattr(app.state, "plugin_registry"):
+        plugin_registry: PluginRegistry = app.state.plugin_registry
+
+        for name, factory in plugin_registry.factories.items():
+            manifest = factory.get_manifest()
+            for hook_spec in manifest.hooks:
+                try:
+                    hook_instance = hook_spec.hook_class(**hook_spec.kwargs)
+                    hook_registry.register(hook_instance)
+                    logger.debug(
+                        "plugin_hook_registered",
+                        plugin_name=name,
+                        hook_class=hook_spec.hook_class.__name__,
+                        category="lifecycle",
+                    )
+                except Exception as e:
+                    logger.error(
+                        "plugin_hook_registration_failed",
+                        plugin_name=name,
+                        hook_class=hook_spec.hook_class.__name__,
+                        error=str(e),
+                        exc_info=e,
+                        category="lifecycle",
+                    )
+
+    try:
+        await hook_manager.emit(HookEvent.APP_STARTUP, {"phase": "startup"})
+    except Exception as e:
+        logger.error(
+            "startup_hook_failed", error=str(e), exc_info=e, category="lifecycle"
+        )
+
+
 LIFECYCLE_COMPONENTS: list[LifecycleComponent] = [
     {
-        "name": "Claude Authentication",
-        "startup": validate_claude_authentication_startup,
-        "shutdown": None,  # One-time validation, no cleanup needed
-    },
-    {
-        "name": "Codex Authentication",
-        "startup": validate_codex_authentication_startup,
-        "shutdown": None,  # One-time validation, no cleanup needed
+        "name": "Task Manager",
+        "startup": setup_task_manager_startup,
+        "shutdown": setup_task_manager_shutdown,
     },
     {
         "name": "Version Check",
         "startup": check_version_updates_startup,
-        "shutdown": None,  # One-time check, no cleanup needed
+        "shutdown": None,
     },
     {
         "name": "Claude CLI",
         "startup": check_claude_cli_startup,
-        "shutdown": None,  # Detection only, no cleanup needed
-    },
-    {
-        "name": "Codex CLI",
-        "startup": check_codex_cli_startup,
-        "shutdown": None,  # Detection only, no cleanup needed
-    },
-    {
-        "name": "Claude Detection",
-        "startup": initialize_claude_detection_startup,
-        "shutdown": None,  # No cleanup needed
-    },
-    {
-        "name": "Codex Detection",
-        "startup": initialize_codex_detection_startup,
-        "shutdown": None,  # No cleanup needed
-    },
-    {
-        "name": "Claude SDK",
-        "startup": initialize_claude_sdk_startup,
-        "shutdown": setup_session_manager_shutdown,
+        "shutdown": None,
     },
     {
         "name": "Scheduler",
@@ -119,154 +255,190 @@ LIFECYCLE_COMPONENTS: list[LifecycleComponent] = [
         "shutdown": setup_scheduler_shutdown,
     },
     {
-        "name": "Log Storage",
-        "startup": initialize_log_storage_startup,
-        "shutdown": initialize_log_storage_shutdown,
+        "name": "Service Container",
+        "startup": None,
+        "shutdown": setup_service_container_shutdown,
     },
     {
-        "name": "Permission Service",
-        "startup": initialize_permission_service_startup,
-        "shutdown": setup_permission_service_shutdown,
+        "name": "Plugin System",
+        "startup": initialize_plugins_startup,
+        "shutdown": shutdown_plugins,
     },
-]
-
-# Additional shutdown-only components that need special handling
-SHUTDOWN_ONLY_COMPONENTS: list[ShutdownComponent] = [
     {
-        "name": "Streaming Batches",
-        "shutdown": flush_streaming_batches_shutdown,
+        "name": "Hook System",
+        "startup": initialize_hooks_startup,
+        "shutdown": shutdown_hook_system,
     },
 ]
 
-
-# Create shared models router
-models_router = APIRouter(tags=["models"])
-
-
-@models_router.get("/v1/models", response_model=None)
-async def list_models() -> dict[str, Any]:
-    """List available models.
-
-    Returns a combined list of Anthropic models and recent OpenAI models.
-    This endpoint is shared between both SDK and proxy APIs.
-    """
-    return get_models_list()
+SHUTDOWN_ONLY_COMPONENTS: list[ShutdownComponent] = []
 
 
 @asynccontextmanager
 async def lifespan(app: FastAPI) -> AsyncGenerator[None, None]:
     """Application lifespan manager using component-based approach."""
-    settings = get_settings()
-
-    # Store settings in app state for reuse in dependencies
-    app.state.settings = settings
-
-    # Startup
+    service_container: ServiceContainer = app.state.service_container
+    settings = service_container.get_service(Settings)
     logger.info(
-        "server_start",
+        "server_starting",
         host=settings.server.host,
         port=settings.server.port,
         url=f"http://{settings.server.host}:{settings.server.port}",
+        category="lifecycle",
     )
+    # Demote granular config detail to DEBUG
     logger.debug(
-        "server_configured", host=settings.server.host, port=settings.server.port
+        "server_configured",
+        host=settings.server.host,
+        port=settings.server.port,
+        category="config",
     )
 
-    # Log Claude CLI configuration
-    if settings.claude.cli_path:
-        logger.debug("claude_cli_configured", cli_path=settings.claude.cli_path)
-    else:
-        logger.debug("claude_cli_auto_detect")
-        logger.debug(
-            "claude_cli_search_paths", paths=settings.claude.get_searched_paths()
-        )
-
-    # Execute startup components in order
     for component in LIFECYCLE_COMPONENTS:
         if component["startup"]:
             component_name = component["name"]
             try:
-                logger.debug(f"starting_{component_name.lower().replace(' ', '_')}")
+                logger.debug(
+                    f"starting_{component_name.lower().replace(' ', '_')}",
+                    category="lifecycle",
+                )
                 await component["startup"](app, settings)
+            except (OSError, PermissionError) as e:
+                logger.error(
+                    f"{component_name.lower().replace(' ', '_')}_startup_io_failed",
+                    error=str(e),
+                    component=component_name,
+                    exc_info=e,
+                    category="lifecycle",
+                )
             except Exception as e:
                 logger.error(
                     f"{component_name.lower().replace(' ', '_')}_startup_failed",
                     error=str(e),
                     component=component_name,
+                    exc_info=e,
+                    category="lifecycle",
                 )
-                # Continue with graceful degradation
+
+    # After startup completes (post-yield happens on shutdown); emit ready before yielding
+    # Safely derive feature flags from settings which may be models or dicts
+    def _get_plugin_enabled(name: str) -> bool:
+        plugins_cfg = getattr(settings, "plugins", None)
+        if plugins_cfg is None:
+            return False
+        # dict-like
+        if isinstance(plugins_cfg, dict):
+            cfg = plugins_cfg.get(name)
+            if isinstance(cfg, dict):
+                return bool(cfg.get("enabled", False))
+            try:
+                return bool(getattr(cfg, "enabled", False))
+            except Exception:
+                return False
+        # object-like
+        try:
+            sub = getattr(plugins_cfg, name, None)
+            return bool(getattr(sub, "enabled", False))
+        except Exception:
+            return False
+
+    def _get_auth_enabled() -> bool:
+        auth_cfg = getattr(settings, "auth", None)
+        if auth_cfg is None:
+            return False
+        if isinstance(auth_cfg, dict):
+            return bool(auth_cfg.get("enabled", False))
+        return bool(getattr(auth_cfg, "enabled", False))
+
+    logger.info(
+        "server_ready",
+        url=f"http://{settings.server.host}:{settings.server.port}",
+        version=__version__,
+        workers=settings.server.workers,
+        reload=settings.server.reload,
+        features_enabled={
+            "plugins": bool(getattr(settings, "enable_plugins", False)),
+            "metrics": _get_plugin_enabled("metrics"),
+            "access": _get_plugin_enabled("access_log"),
+            "auth": _get_auth_enabled(),
+        },
+        category="lifecycle",
+    )
 
     yield
 
-    # Shutdown
-    logger.debug("server_stop")
+    logger.debug("server_stop", category="lifecycle")
 
-    # Execute shutdown-only components first
     for shutdown_component in SHUTDOWN_ONLY_COMPONENTS:
         if shutdown_component["shutdown"]:
             component_name = shutdown_component["name"]
             try:
-                logger.debug(f"stopping_{component_name.lower().replace(' ', '_')}")
+                logger.debug(
+                    f"stopping_{component_name.lower().replace(' ', '_')}",
+                    category="lifecycle",
+                )
                 await shutdown_component["shutdown"](app)
+            except (OSError, PermissionError) as e:
+                logger.error(
+                    f"{component_name.lower().replace(' ', '_')}_shutdown_io_failed",
+                    error=str(e),
+                    component=component_name,
+                    exc_info=e,
+                    category="lifecycle",
+                )
             except Exception as e:
                 logger.error(
                     f"{component_name.lower().replace(' ', '_')}_shutdown_failed",
                     error=str(e),
                     component=component_name,
+                    exc_info=e,
+                    category="lifecycle",
                 )
 
-    # Execute shutdown components in reverse order
     for component in reversed(LIFECYCLE_COMPONENTS):
         if component["shutdown"]:
             component_name = component["name"]
             try:
-                logger.debug(f"stopping_{component_name.lower().replace(' ', '_')}")
-                # Some shutdown functions need settings, others don't
+                logger.debug(
+                    f"stopping_{component_name.lower().replace(' ', '_')}",
+                    category="lifecycle",
+                )
                 if component_name == "Permission Service":
                     await component["shutdown"](app, settings)  # type: ignore
                 else:
                     await component["shutdown"](app)  # type: ignore
+            except (OSError, PermissionError) as e:
+                logger.error(
+                    f"{component_name.lower().replace(' ', '_')}_shutdown_io_failed",
+                    error=str(e),
+                    component=component_name,
+                    exc_info=e,
+                    category="lifecycle",
+                )
             except Exception as e:
                 logger.error(
                     f"{component_name.lower().replace(' ', '_')}_shutdown_failed",
                     error=str(e),
                     component=component_name,
+                    exc_info=e,
+                    category="lifecycle",
                 )
 
 
-def create_app(settings: Settings | None = None) -> FastAPI:
-    """Create and configure the FastAPI application.
-
-    Args:
-        settings: Optional settings override. If None, uses get_settings().
-
-    Returns:
-        Configured FastAPI application instance.
-    """
-    if settings is None:
-        settings = get_settings()
-    # Configure logging based on settings BEFORE any module uses logger
-    # This is needed for reload mode where the app is re-imported
-
-    import structlog
-
-    # Only configure if not already configured or if no file handler exists
-    # okay we have the first debug line but after uvicorn start they are not show root_logger = logging.getLogger()
-    # for h in root_logger.handlers:
-    #     print(h)
-    # has_file_handler = any(
-    #     isinstance(h, logging.FileHandler) for h in root_logger.handlers
-    # )
-
+def create_app(service_container: ServiceContainer | None = None) -> FastAPI:
+    if service_container is None:
+        service_container = create_service_container()
+    """Create and configure the FastAPI application with plugin system."""
+    settings = service_container.get_service(Settings)
     if not structlog.is_configured():
-        # Only setup logging if structlog is not configured at all
-        # Always use console output, but respect file logging from settings
-        json_logs = False
+        json_logs = settings.logging.format == "json"
+
         setup_logging(
             json_logs=json_logs,
-            log_level_name=settings.server.log_level,
-            log_file=settings.server.log_file,
+            log_level_name=settings.logging.level,
+            log_file=settings.logging.file,
         )
+    logger.trace("settings", category="lifecycle", settings=settings)
 
     app = FastAPI(
         title="CCProxy API Server",
@@ -275,92 +447,135 @@ def create_app(settings: Settings | None = None) -> FastAPI:
         lifespan=lifespan,
     )
 
-    # Setup middleware
-    setup_cors_middleware(app, settings)
-    setup_error_handlers(app)
-
-    # Add request content logging middleware first (will run fourth due to middleware order)
-    app.add_middleware(RequestContentLoggingMiddleware)
+    app.state.service_container = service_container
+
+    # Make the FastAPI instance available via the service container for plugin contexts
+    service_container.register_service(FastAPI, instance=app)
+
+    app.state.oauth_registry = service_container.get_oauth_registry()
+
+    plugin_registry = PluginRegistry()
+    middleware_manager = MiddlewareManager()
+
+    if settings.enable_plugins:
+        plugin_registry, middleware_manager = load_plugin_system(settings)
+
+        # Consolidated plugin init summary at INFO
+        logger.info(
+            "plugins_initialized",
+            plugin_count=len(plugin_registry.factories),
+            providers=sum(
+                1
+                for f in plugin_registry.factories.values()
+                if f.get_manifest().is_provider
+            ),
+            system_plugins=len(plugin_registry.factories)
+            - sum(
+                1
+                for f in plugin_registry.factories.values()
+                if f.get_manifest().is_provider
+            ),
+            names=list(plugin_registry.factories.keys()),
+            category="plugin",
+        )
 
-    # Add custom access log middleware second (will run third due to middleware order)
-    app.add_middleware(AccessLogMiddleware)
+        # Manifest population will be done during startup when core services are available
+
+        plugin_middleware_count = 0
+        for name, factory in plugin_registry.factories.items():
+            manifest = factory.get_manifest()
+            if manifest.middleware:
+                middleware_manager.add_plugin_middleware(name, manifest.middleware)
+                plugin_middleware_count += len(manifest.middleware)
+                logger.trace(
+                    "plugin_middleware_collected",
+                    plugin=name,
+                    count=len(manifest.middleware),
+                    category="lifecycle",
+                )
 
-    # Add request ID middleware fourth (will run first to initialize context)
-    app.add_middleware(RequestIDMiddleware)
+        if plugin_middleware_count > 0:
+            plugins_with_middleware = [
+                n
+                for n, f in plugin_registry.factories.items()
+                if f.get_manifest().middleware
+            ]
+            logger.debug(
+                "plugin_middleware_collection_completed",
+                total_middleware=plugin_middleware_count,
+                plugins_with_middleware=len(plugins_with_middleware),
+                plugin_names=plugins_with_middleware,
+                category="lifecycle",
+            )
 
-    # Add server header middleware (for non-proxy routes)
-    # You can customize the server name here
-    app.add_middleware(ServerHeaderMiddleware, server_name="uvicorn")
+        for name, factory in plugin_registry.factories.items():
+            manifest = factory.get_manifest()
+            for route_spec in manifest.routes:
+                default_tag = name.replace("_", "-")
+                # Merge router tags with spec tags, removing duplicates
+                merged_tags = merge_router_tags(
+                    route_spec.router,
+                    spec_tags=route_spec.tags,
+                    default_tags=[default_tag],
+                )
 
-    # Include health router (always enabled)
-    app.include_router(health_router, tags=["health"])
+                app.include_router(
+                    route_spec.router,
+                    prefix=route_spec.prefix,
+                    tags=merged_tags,
+                    dependencies=route_spec.dependencies,
+                )
+                logger.debug(
+                    "plugin_routes_registered",
+                    plugin=name,
+                    prefix=route_spec.prefix,
+                    category="lifecycle",
+                )
 
-    # Include observability routers with granular controls
-    if settings.observability.metrics_endpoint_enabled:
-        app.include_router(prometheus_router, tags=["metrics"])
+    app.state.plugin_registry = plugin_registry
+    app.state.middleware_manager = middleware_manager
 
-    if settings.observability.logs_endpoints_enabled:
-        app.include_router(logs_router, prefix="/logs", tags=["logs"])
+    app.state.settings = settings
 
-    if settings.observability.dashboard_enabled:
-        app.include_router(dashboard_router, tags=["dashboard"])
+    setup_cors_middleware(app, settings)
+    setup_error_handlers(app)
 
-    app.include_router(oauth_router, prefix="/oauth", tags=["oauth"])
+    # Validate format adapters once routes are registered
+    try:
+        registry = service_container.get_format_registry()
+        validate_route_format_chains(app=app, registry=registry, logger=logger)
+    except Exception as exc:
+        # Best-effort registration/validation; do not block app startup
+        logger.warning("format_registry_setup_skipped", error=str(exc))
 
-    # Codex routes for OpenAI integration
-    app.include_router(codex_router, tags=["codex"])
+    setup_default_middleware(middleware_manager)
 
-    # New /sdk/ routes for Claude SDK endpoints
-    app.include_router(claude_router, prefix="/sdk", tags=["claude-sdk"])
+    middleware_manager.apply_to_app(app)
 
-    # New /api/ routes for proxy endpoints (includes OpenAI-compatible /v1/chat/completions)
-    app.include_router(proxy_router, prefix="/api", tags=["proxy-api"])
+    # Core router registrations with tag merging
+    app.include_router(
+        health_router, tags=merge_router_tags(health_router, default_tags=["health"])
+    )
 
-    # Shared models endpoints for both SDK and proxy APIs
-    app.include_router(models_router, prefix="/sdk", tags=["claude-sdk", "models"])
-    app.include_router(models_router, prefix="/api", tags=["proxy-api", "models"])
+    app.include_router(
+        oauth_router,
+        prefix="/oauth",
+        tags=merge_router_tags(oauth_router, default_tags=["oauth"]),
+    )
 
-    # Confirmation endpoints for SSE streaming and responses (conditional on builtin_permissions)
-    if settings.claude.builtin_permissions:
+    if settings.enable_plugins:
         app.include_router(
-            permissions_router, prefix="/permissions", tags=["permissions"]
+            plugins_router,
+            tags=merge_router_tags(plugins_router, default_tags=["plugins"]),
         )
-        setup_mcp(app)
-
-    # Mount static files for dashboard SPA
-    from pathlib import Path
-
-    # Get the path to the dashboard static files
-    current_file = Path(__file__)
-    project_root = (
-        current_file.parent.parent.parent
-    )  # ccproxy/api/app.py -> project root
-    dashboard_static_path = project_root / "ccproxy" / "static" / "dashboard"
-
-    # Mount dashboard static files if they exist
-    if dashboard_static_path.exists():
-        # Mount the _app directory for SvelteKit assets at the correct base path
-        app_path = dashboard_static_path / "_app"
-        if app_path.exists():
-            app.mount(
-                "/dashboard/_app",
-                StaticFiles(directory=str(app_path)),
-                name="dashboard-assets",
-            )
-
-        # Mount favicon.svg at root level
-        favicon_path = dashboard_static_path / "favicon.svg"
-        if favicon_path.exists():
-            # For single files, we'll handle this in the dashboard route or add a specific route
-            pass
 
     return app
 
 
 def get_app() -> FastAPI:
-    """Get the FastAPI application instance.
+    """Get the FastAPI app instance."""
+    container = create_service_container()
+    return create_app(container)
 
-    Returns:
-        FastAPI application instance.
-    """
-    return create_app()
+
+__all__ = ["create_app", "get_app"]