set param required tag to False by default

PiperOrigin-RevId: 754382785

Author: seanzhougoogle

CHANGELOG.md

@@ -27,7 +27,7 @@
 ### ⚠ BREAKING CHANGES
 
 * Auth: expose `access_token` and `refresh_token` at top level of auth
-  credentials, instead of a `dict`
+  credentails, instead of a `dict`
   ([commit](https://github.com/google/adk-python/commit/956fb912e8851b139668b1ccb8db10fd252a6990)).
 
 ### Features
@@ -50,7 +50,7 @@
 
 ### Miscellaneous Chores
 
-* README.md improvements.
+* README.md impprovements.
 * Various code improvements.
 * Various typo fixes.
 * Bump min version of google-genai to 1.11.0.
@@ -108,4 +108,4 @@
 * Built-in evaluation support
 * Development UI that makes local development easy
 * Deploy to Google Cloud Run, Agent Engine
-* (Experimental) Live(Bidi) audio/video agent support and Compositional Function Calling(CFC) support
+* (Experimental) Live(Bidi) auido/video agent support and Compositional Function Calling(CFC) support

src/google/adk/sessions/database_session_service.py

@@ -484,11 +484,9 @@ def append_event(self, session: Session, event: Event) -> Event:
 
       if storage_session.update_time.timestamp() > session.last_update_time:
         raise ValueError(
-          f"Session last_update_time "
-          f"{datetime.fromtimestamp(session.last_update_time):%Y-%m-%d %H:%M:%S} "
-          f"is later than the update_time in storage "
-          f"{storage_session.update_time:%Y-%m-%d %H:%M:%S}"
-      )
+            f"Session last_update_time {session.last_update_time} is later than"
+            f" the upate_time in storage {storage_session.update_time}"
+        )
 
       # Fetch states from storage
       storage_app_state = sessionFactory.get(

src/google/adk/tools/openapi_tool/common/common.py

@@ -14,11 +14,7 @@
 
 import keyword
 import re
-from typing import Any
-from typing import Dict
-from typing import List
-from typing import Optional
-from typing import Union
+from typing import Any, Dict, List, Optional, Union
 
 from fastapi.openapi.models import Response
 from fastapi.openapi.models import Schema
@@ -100,7 +96,7 @@ class ApiParameter(BaseModel):
   py_name: Optional[str] = ''
   type_value: type[Any] = Field(default=None, init_var=False)
   type_hint: str = Field(default=None, init_var=False)
-  required: Optional[bool] = None
+  required: bool = False
 
   def model_post_init(self, _: Any):
     self.py_name = (

src/google/adk/tools/openapi_tool/openapi_spec_parser/operation_parser.py

@@ -17,9 +17,13 @@
 from typing import Any, Dict, List, Optional, Union
 
 from fastapi.encoders import jsonable_encoder
-from fastapi.openapi.models import Operation, Parameter, Schema
+from fastapi.openapi.models import Operation
+from fastapi.openapi.models import Parameter
+from fastapi.openapi.models import Schema
 
-from ..common.common import ApiParameter, PydocHelper, to_snake_case
+from ..common.common import ApiParameter
+from ..common.common import PydocHelper
+from ..common.common import to_snake_case
 
 
 class OperationParser:
@@ -76,7 +80,9 @@ def _process_operation_parameters(self):
         description = param.description or ''
         location = param.in_ or ''
         schema = param.schema_ or {}  # Use schema_ instead of .schema
-        schema.description = description if schema.description is None and description != '' else schema.description
+        schema.description = (
+            description if not schema.description else schema.description
+        )
         required = param.required
 
         self.params.append(
@@ -85,7 +91,7 @@ def _process_operation_parameters(self):
                 param_location=location,
                 param_schema=schema,
                 description=description,
-                required=required
+                required=required,
             )
         )
 
@@ -233,7 +239,7 @@ def get_json_schema(self) -> Dict[str, Any]:
     }
     return {
         'properties': properties,
-        'required': [p.py_name for p in self.params if p.required is not False],
+        'required': [p.py_name for p in self.params if p.required],
         'title': f"{self.operation.operationId or 'unnamed'}_Arguments",
         'type': 'object',
     }