fix: return 422 for invalid historyLength in REST get_task

Replace unhandled int() conversion with Pydantic validation so
non-numeric historyLength query params (e.g. ?historyLength=abc)
return 422 InvalidParamsError instead of 500.

Author: azilbersteinSFDC

src/a2a/server/request_handlers/rest_handler.py

@@ -3,6 +3,7 @@
 from collections.abc import AsyncIterable, AsyncIterator
 from typing import TYPE_CHECKING, Any
 
+from pydantic import ValidationError
 from google.protobuf.json_format import MessageToDict, MessageToJson, Parse
 
 
@@ -21,6 +22,7 @@
 from a2a.types import (
     AgentCard,
     GetTaskPushNotificationConfigParams,
+    InvalidParamsError,
     TaskIdParams,
     TaskNotFoundError,
     TaskQueryParams,
@@ -257,8 +259,15 @@ async def on_get_task(
         """
         task_id = request.path_params['id']
         history_length_str = request.query_params.get('historyLength')
-        history_length = int(history_length_str) if history_length_str else None
-        params = TaskQueryParams(id=task_id, history_length=history_length)
+        try:
+            params = TaskQueryParams(
+                id=task_id,
+                history_length=history_length_str if history_length_str else None,
+            )
+        except ValidationError:
+            raise ServerError(
+                error=InvalidParamsError(message='historyLength must be a valid integer')
+            )
         task = await self.request_handler.on_get_task(params, context)
         if task:
             return MessageToDict(proto_utils.ToProto.task(task))

tests/server/apps/rest/test_rest_fastapi_app.py

@@ -399,5 +399,17 @@ async def test_send_message_rejected_task(
     assert expected_response == actual_response
 
 
+@pytest.mark.anyio
+async def test_get_task_invalid_history_length_returns_422(
+    client: AsyncClient,
+) -> None:
+    """Non-numeric historyLength query param returns 422 InvalidParamsError."""
+    response = await client.get('/v1/tasks/some-task-id?historyLength=abc')
+    assert response.status_code == 422
+    data = response.json()
+    assert 'message' in data
+    assert 'historylength' in data['message'].lower()
+
+
 if __name__ == '__main__':
     pytest.main([__file__])