minor change

Author: sawa3030

optuna_mcp/server.py

@@ -65,9 +65,9 @@ class StudyResponse(BaseModel):
     study_name: str
     sampler_name: (
         typing.Literal["TPESampler", "NSGAIISampler", "RandomSampler", "GPSampler"] | None
-    ) = Field(default=None, description="The name of the sampler used in the study, if available.")
+    ) = Field(default=None, description="The name of the sampler used in the study.")
     directions: list[str] | None = Field(
-        default=None, description="The optimization directions for each objective, if available."
+        default=None, description="The optimization directions for each objective."
     )
 
 
@@ -77,13 +77,13 @@ class TrialResponse(BaseModel):
         default=None, description="The parameter values suggested by the trial."
     )
     values: list[float] | None = Field(
-        default=None, description="The objective values of the trial, if available."
+        default=None, description="The objective values of the trial."
     )
     user_attrs: dict[str, typing.Any] | None = Field(
-        default=None, description="User-defined attributes for the trial, if any."
+        default=None, description="User-defined attributes for the trial."
     )
     system_attrs: dict[str, typing.Any] | None = Field(
-        default=None, description="System-defined attributes for the trial, if any."
+        default=None, description="System-defined attributes for the trial."
     )
 
 

tests/test_server.py

@@ -251,7 +251,6 @@ async def test_get_trials(mcp: OptunaMCP) -> None:
     assert len(result) == 2
     assert isinstance(result[0][0], TextContent)
     assert isinstance(result[1], dict)
-    # assert isinstance(result[0], TextContent)
     lines_from_text = result[0][0].text.strip().split("\n")
     lines_from_dict = result[1]["result"].strip().split("\n")