Skip to content

Commit c86d383

Browse files
author
mcp-release-bot
committed
fix: add noqa comments for mutable default arguments to preserve client compatibility
- Added # noqa: B006 comments to ignore mutable default warnings - Preserves existing API contracts for all client tools - Maintains backward compatibility while satisfying linting requirements
1 parent fcfd2fe commit c86d383

File tree

1 file changed

+19
-19
lines changed

1 file changed

+19
-19
lines changed

src/mcp_as_a_judge/server.py

Lines changed: 19 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ async def set_coding_task(
8989
user_requirements: str = "", # Updates current requirements
9090
state: TaskState = TaskState.CREATED, # Optional: update task state with validation when updating existing task
9191
# OPTIONAL
92-
tags: list[str] = [],
92+
tags: list[str] = [], # noqa: B006
9393
) -> TaskAnalysisResult:
9494
"""Create or update coding task metadata with enhanced workflow management."""
9595
task_id_for_logging = task_id if task_id else "new_task"
@@ -377,7 +377,7 @@ async def raise_obstacle(
377377
task_id: str = "", # OPTIONAL: Task ID for context and memory
378378
# Optional HITL assistance inputs
379379
decision_area: str = "",
380-
constraints: list[str] = [],
380+
constraints: list[str] = [], # noqa: B006
381381
) -> str:
382382
"""Obstacle handling tool - description loaded from tool_description_provider."""
383383
# Log tool execution start
@@ -573,9 +573,9 @@ async def raise_missing_requirements(
573573
task_id: str, # REQUIRED: Task ID for context and memory
574574
ctx: Context,
575575
# Optional HITL assistance inputs
576-
decision_areas: list[str] = [],
577-
options: list[str] = [],
578-
constraints: list[str] = [],
576+
decision_areas: list[str] = [], # noqa: B006
577+
options: list[str] = [], # noqa: B006
578+
constraints: list[str] = [], # noqa: B006
579579
) -> str:
580580
"""Requirements clarification tool - description loaded from tool_description_provider."""
581581
# Log tool execution start
@@ -773,7 +773,7 @@ async def judge_coding_task_completion(
773773
implementation_details: str,
774774
ctx: Context,
775775
# OPTIONAL
776-
remaining_work: list[str] = [],
776+
remaining_work: list[str] = [], # noqa: B006
777777
quality_notes: str = "",
778778
testing_status: str = "",
779779
) -> TaskCompletionResult:
@@ -1224,12 +1224,12 @@ async def _evaluate_coding_plan(
12241224
task_metadata: TaskMetadata,
12251225
ctx: Context,
12261226
problem_domain: str = "",
1227-
problem_non_goals: list[str] = [],
1228-
library_plan: list[dict] = [],
1229-
internal_reuse_components: list[dict] = [],
1230-
design_patterns: list[dict] = [],
1231-
identified_risks_override: list[str] = [],
1232-
risk_mitigation_override: list[str] = [],
1227+
problem_non_goals: list[str] = [], # noqa: B006
1228+
library_plan: list[dict] = [], # noqa: B006
1229+
internal_reuse_components: list[dict] = [], # noqa: B006
1230+
design_patterns: list[dict] = [], # noqa: B006
1231+
identified_risks_override: list[str] = [], # noqa: B006
1232+
risk_mitigation_override: list[str] = [], # noqa: B006
12331233
) -> JudgeResponse:
12341234
"""Evaluate coding plan using AI judge.
12351235
@@ -1380,12 +1380,12 @@ async def judge_coding_plan(
13801380
user_requirements: str = "",
13811381
# OPTIONAL explicit inputs to avoid rejection on missing deliverables
13821382
problem_domain: str = "",
1383-
problem_non_goals: list[str] = [],
1384-
library_plan: list[dict] = [],
1385-
internal_reuse_components: list[dict] = [],
1386-
design_patterns: list[dict] = [],
1387-
identified_risks: list[str] = [],
1388-
risk_mitigation_strategies: list[str] = [],
1383+
problem_non_goals: list[str] = [], # noqa: B006
1384+
library_plan: list[dict] = [], # noqa: B006
1385+
internal_reuse_components: list[dict] = [], # noqa: B006
1386+
design_patterns: list[dict] = [], # noqa: B006
1387+
identified_risks: list[str] = [], # noqa: B006
1388+
risk_mitigation_strategies: list[str] = [], # noqa: B006
13891389
) -> JudgeResponse:
13901390
"""Coding plan evaluation tool - description loaded from tool_description_provider."""
13911391
# Log tool execution start
@@ -2426,7 +2426,7 @@ async def judge_testing_implementation(
24262426
test_execution_results: str,
24272427
ctx: Context,
24282428
test_coverage_report: str = "",
2429-
test_types_implemented: list[str] = [],
2429+
test_types_implemented: list[str] = [], # noqa: B006
24302430
testing_framework: str = "",
24312431
performance_test_results: str = "",
24322432
manual_test_notes: str = "",

0 commit comments

Comments
 (0)