fix: resolve linting errors for modern Python type annotations

- Replace typing.List with modern list[str] syntax
- Fix import organization and remove unused typing imports
- Use proper Optional syntax with | None union type
- All linting checks now pass
- All 41 tests still passing

Author: Zvi Fried

src/mcp_as_a_judge/models.py

@@ -5,8 +5,6 @@
 serialization, and API contracts.
 """
 
-from typing import List
-
 from pydantic import BaseModel, Field
 
 
@@ -139,7 +137,7 @@ class JudgeCodingPlanUserVars(BaseModel):
     research: str = Field(
         description="Research findings and analysis"
     )
-    research_urls: List[str] = Field(
+    research_urls: list[str] = Field(
         default_factory=list,
         description="URLs from MANDATORY online research - minimum 3 URLs required"
     )
@@ -193,7 +191,7 @@ class ResearchValidationUserVars(BaseModel):
     research: str = Field(
         description="Research findings to be validated"
     )
-    research_urls: List[str] = Field(
+    research_urls: list[str] = Field(
         default_factory=list,
         description="URLs from MANDATORY online research - minimum 3 URLs required"
     )

src/mcp_as_a_judge/server.py

@@ -9,8 +9,6 @@
 
 import json
 
-from typing import List
-
 from mcp.server.fastmcp import Context, FastMCP
 from mcp.types import (
     ClientCapabilities,
@@ -436,7 +434,7 @@ async def judge_coding_plan(
     user_requirements: str,
     ctx: Context,
     context: str = "",
-    research_urls: List[str] = None,
+    research_urls: list[str] | None = None,
 ) -> JudgeResponse:
     """🚨 MANDATORY: AI programming assistant MUST call this tool whenever you start to work on a coding task.