clean up FileSearchTool comments

Removed unnecessary explanatory comments from the file search implementation.
The code is self-explanatory and these comments were just adding noise.

Author: gorkachea

pydantic_ai_slim/pydantic_ai/models/google.py

@@ -343,11 +343,6 @@ def _get_tools(self, model_request_parameters: ModelRequestParameters) -> list[T
                 elif isinstance(tool, CodeExecutionTool):
                     tools.append(ToolDict(code_execution=ToolCodeExecutionDict()))
                 elif isinstance(tool, FileSearchTool):  # pragma: no cover
-                    # File Search Tool for Gemini API - tested via initialization tests
-                    # The file_search tool uses file resource names (vector_store_ids) to search through uploaded files
-                    # Note: This requires files to be uploaded via the Files API first
-                    # The structure below is based on the Gemini File Search Tool announcement (Nov 2025)
-                    # and may require adjustment when the official google-genai SDK is updated
                     tools.append(ToolDict(file_search={'file_names': tool.vector_store_ids}))  # type: ignore[reportGeneralTypeIssues]
                 elif isinstance(tool, ImageGenerationTool):  # pragma: no branch
                     if not self.profile.supports_image_output:

pydantic_ai_slim/pydantic_ai/models/openai.py

@@ -1071,7 +1071,6 @@ def _process_response(  # noqa: C901
                 # Pydantic AI doesn't yet support the `codex-mini-latest` LocalShell built-in tool
                 pass
             elif isinstance(item, responses.ResponseFileSearchToolCall):  # pragma: no cover
-                # File Search Tool handling - requires actual OpenAI API responses with file_search_call
                 call_part, return_part = _map_file_search_tool_call(item, self.system)
                 items.append(call_part)
                 items.append(return_part)
@@ -1270,7 +1269,6 @@ def _get_builtin_tools(self, model_request_parameters: ModelRequestParameters) -
                     )
                 tools.append(web_search_tool)
             elif isinstance(tool, FileSearchTool):  # pragma: no cover
-                # File Search Tool configuration - tested via initialization tests
                 file_search_tool = responses.FileSearchToolParam(
                     type='file_search', vector_store_ids=tool.vector_store_ids
                 )
@@ -1483,12 +1481,10 @@ async def _map_messages(  # noqa: C901
                                 )
                                 openai_messages.append(web_search_item)
                             elif (  # pragma: no cover
-                                # File Search Tool - requires actual file_search responses in message history
                                 item.tool_name == FileSearchTool.kind
                                 and item.tool_call_id
                                 and (args := item.args_as_dict())
                             ):
-                                # The cast is necessary because of incomplete OpenAI SDK types for FileSearchToolCall
                                 file_search_item = cast(
                                     responses.ResponseFileSearchToolCallParam,
                                     {
@@ -1559,7 +1555,6 @@ async def _map_messages(  # noqa: C901
                             ):
                                 web_search_item['status'] = status
                             elif (  # pragma: no cover
-                                # File Search Tool status update - only called from API-dependent paths
                                 item.tool_name == FileSearchTool.kind
                                 and file_search_item is not None
                                 and isinstance(item.content, dict)  # pyright: ignore[reportUnknownMemberType]
@@ -1881,7 +1876,6 @@ async def _get_event_iterator(self) -> AsyncIterator[ModelResponseStreamEvent]:
                         vendor_part_id=f'{chunk.item.id}-call', part=replace(call_part, args=None)
                     )
                 elif isinstance(chunk.item, responses.ResponseFileSearchToolCall):  # pragma: no cover
-                    # File Search Tool streaming - requires actual OpenAI streaming responses
                     call_part, _ = _map_file_search_tool_call(chunk.item, self.provider_name)
                     yield self._parts_manager.handle_part(
                         vendor_part_id=f'{chunk.item.id}-call', part=replace(call_part, args=None)
@@ -1963,7 +1957,6 @@ async def _get_event_iterator(self) -> AsyncIterator[ModelResponseStreamEvent]:
 
                     yield self._parts_manager.handle_part(vendor_part_id=f'{chunk.item.id}-return', part=return_part)
                 elif isinstance(chunk.item, responses.ResponseFileSearchToolCall):  # pragma: no cover
-                    # File Search Tool streaming response handling - requires actual OpenAI streaming responses
                     call_part, return_part = _map_file_search_tool_call(chunk.item, self.provider_name)
 
                     maybe_event = self._parts_manager.handle_tool_call_delta(
@@ -2270,7 +2263,6 @@ def _map_web_search_tool_call(
 
 
 def _map_file_search_tool_call(  # pragma: no cover
-    # File Search Tool mapping - only called from API-dependent response processing paths
     item: responses.ResponseFileSearchToolCall,
     provider_name: str,
 ) -> tuple[BuiltinToolCallPart, BuiltinToolReturnPart]:
@@ -2280,7 +2272,6 @@ def _map_file_search_tool_call(  # pragma: no cover
         'status': item.status,
     }
 
-    # The OpenAI SDK has incomplete types for FileSearchToolCall.action
     if action := item.action:  # type: ignore[reportAttributeAccessIssue]
         args = action.model_dump(mode='json')  # type: ignore[reportUnknownMemberType]