fix: Make __del__ methods more robust in _api_client and client.

Guard `close()` in `_api_client.__del__` to only run if custom `httpx_client` is not set, and wrap `close()` in `client.__del__` with a try-except block to catch any exceptions during finalization. Fixes: #1567 Fixes: #1565 PiperOrigin-RevId: 825620765

Author: yinghsienwu

google/genai/_api_client.py

@@ -1824,11 +1824,17 @@ def __del__(self) -> None:
  """
 
  try:
- self.close()
+ # Let users close the custom client explicitly by themselves. Otherwise,
+ # close the client when the object is garbage collected.
+ if not self._http_options.httpx_client:
+ self.close()
  except Exception: # pylint: disable=broad-except
  pass
 
  try:
- asyncio.get_running_loop().create_task(self.aclose())
+ # Let users close the custom client explicitly by themselves. Otherwise,
+ # close the client when the object is garbage collected.
+ if not self._http_options.httpx_async_client:
+ asyncio.get_running_loop().create_task(self.aclose())
  except Exception: # pylint: disable=broad-except
  pass

google/genai/client.py

@@ -397,4 +397,7 @@ def __exit__(
  self.close()
 
  def __del__(self) -> None:
- self.close()
+ try:
+ self.close()
+ except Exception:
+ pass