fix(toolbox-core): Prevent ToolboxClient from closing externally managed client sessions

This change ensures that the `ToolboxClient` only closes client sessions that it creates and manages internally. If a client session is provided externally (i.e., injected into the `ToolboxClient`), the responsibility for managing its lifecycle, including closing it, remains with the entity that provided it. This prevents unintended closure of externally managed sessions.

Author: anubhav756

packages/toolbox-core/src/toolbox_core/client.py

@@ -33,6 +33,7 @@ class ToolboxClient:
 
     __base_url: str
     __session: ClientSession
+    __manage_session: bool
 
     def __init__(
         self,
@@ -56,7 +57,9 @@ def __init__(
         self.__base_url = url
 
         # If no aiohttp.ClientSession is provided, make our own
+        self.__manage_session = False
         if session is None:
+            self.__manage_session = True
             session = ClientSession()
         self.__session = session
 
@@ -145,7 +148,8 @@ async def close(self):
         is responsible for its lifecycle, but calling close here will still
         attempt to close it.
         """
-        await self.__session.close()
+        if self.__manage_session:
+            await self.__session.close()
 
     async def load_tool(
         self,