fix: resolve Ruff lint errors (F821, B008, I001, UP045)

Author: Coding-Dev-Tools

pyproject.toml

@@ -55,7 +55,7 @@ line-length = 120
 
 [tool.ruff.lint]
 select = ["E", "F", "W", "I", "UP", "B", "SIM"]
-ignore = ["E501"]
+ignore = ["E501", "B008"]
 
 [tool.ruff.lint.isort]
 known-first-party = ["*"]

src/api_contract_guardian/cli.py

@@ -3,9 +3,8 @@
 from __future__ import annotations
 
 import json
-from pathlib import Path
-
 import typer
+from pathlib import Path
 from rich.console import Console
 from rich.table import Table
 
@@ -18,7 +17,7 @@ def require_license(product: str) -> None:  # type: ignore[misc]
         pass
 
 from .diff import DiffResult, diff_specs
-from .gate import GateResult, check_gate
+from .gate import check_gate
 from .loader import SpecLoadError, load_spec, validate_openapi_version
 from .migration import generate_migration_guide, generate_migration_guide_json
 
@@ -85,7 +84,7 @@ def _print_result(result: DiffResult) -> None:
 def diff(
     old: str = typer.Argument(..., help="Path to old (baseline) OpenAPI spec"),
     new: str = typer.Argument(..., help="Path to new (proposed) OpenAPI spec"),
-    output: Optional[str] = typer.Option(None, "--output", "-o", help="Output file path"),
+    output: str | None = typer.Option(None, "--output", "-o", help="Output file path"),
     format: str = typer.Option("rich", "--format", "-f", help="Output format: rich, json, markdown"),
 ) -> None:
     """Compare two OpenAPI specs and show all detected changes."""
@@ -125,7 +124,7 @@ def check(
     fail_on_dangerous: bool = typer.Option(False, "--fail-on-dangerous/--allow-dangerous", help="Fail on dangerous changes"),
     max_breaking: int = typer.Option(0, "--max-breaking", help="Max allowed breaking changes (default 0)"),
     max_dangerous: int = typer.Option(-1, "--max-dangerous", help="Max allowed dangerous changes (-1=unlimited)"),
-    output: Optional[str] = typer.Option(None, "--output", "-o", help="Output file path"),
+    output: str | None = typer.Option(None, "--output", "-o", help="Output file path"),
 ) -> None:
     """Gate CI pipeline on breaking changes. Returns exit code 1 if gate fails."""
     require_license("api-contract-guardian")
@@ -164,7 +163,7 @@ def check(
 def migrate(
     old: str = typer.Argument(..., help="Path to old (baseline) OpenAPI spec"),
     new: str = typer.Argument(..., help="Path to new (proposed) OpenAPI spec"),
-    output: Optional[str] = typer.Option(None, "--output", "-o", help="Output file path"),
+    output: str | None = typer.Option(None, "--output", "-o", help="Output file path"),
     format: str = typer.Option("markdown", "--format", "-f", help="Output format: markdown, json"),
 ) -> None:
     """Generate a migration guide between two OpenAPI spec versions."""