fix: Address memory leaks

- Fix a potential memory leak in secure_state_service.py by ensuring the deletion of frame objects created by inspect.currentframe().
- Correct a logic error in in_memory_session_repository.py to prevent memory leaks from empty user session lists.

Author: matdev83

data/test_suite_state.json

@@ -1,4 +1,4 @@
 {
-  "test_count": 5046,
-  "last_updated": "1762217835.0"
+  "test_count": 5119,
+  "last_updated": "1762168167.0802596"
 }

src/core/repositories/in_memory_session_repository.py

@@ -77,8 +77,8 @@ async def update(self, entity: Session) -> Session:
             tracked_sessions = self._user_sessions.get(previous_user_id)
             if tracked_sessions and entity.id in tracked_sessions:
                 tracked_sessions.remove(entity.id)
-                if not tracked_sessions:
-                    del self._user_sessions[previous_user_id]
+            if not self._user_sessions[previous_user_id]:
+                del self._user_sessions[previous_user_id]
 
         if new_user_id:
             tracked_sessions = self._user_sessions.setdefault(new_user_id, [])

src/core/services/secure_state_service.py

@@ -209,15 +209,18 @@ def __getattr__(self, name: str) -> Any:
         import inspect
 
         frame = inspect.currentframe()
-        if frame and frame.f_back:
-            caller_locals = frame.f_back.f_locals
-
-            # Check if 'self' in caller is an instance of allowed interfaces
-            caller_self = caller_locals.get("self")
-            if caller_self:
-                for interface in self._allowed_interfaces:
-                    if isinstance(caller_self, interface):
-                        return getattr(self._target, name)
+        try:
+            if frame and frame.f_back:
+                caller_locals = frame.f_back.f_locals
+
+                # Check if 'self' in caller is an instance of allowed interfaces
+                caller_self = caller_locals.get("self")
+                if caller_self:
+                    for interface in self._allowed_interfaces:
+                        if isinstance(caller_self, interface):
+                            return getattr(self._target, name)
+        finally:
+            del frame
 
         # If we get here, the access is not through an allowed interface
         raise StateAccessViolationError(
@@ -245,15 +248,18 @@ def __setattr__(self, name: str, value: Any) -> None:
         import inspect
 
         frame = inspect.currentframe()
-        if frame and frame.f_back:
-            caller_locals = frame.f_back.f_locals
-            caller_self = caller_locals.get("self")
-
-            if caller_self:
-                for interface in self._allowed_interfaces:
-                    if isinstance(caller_self, interface):
-                        setattr(self._target, name, value)
-                        return
+        try:
+            if frame and frame.f_back:
+                caller_locals = frame.f_back.f_locals
+                caller_self = caller_locals.get("self")
+
+                if caller_self:
+                    for interface in self._allowed_interfaces:
+                        if isinstance(caller_self, interface):
+                            setattr(self._target, name, value)
+                            return
+        finally:
+            del frame
 
         raise StateAccessViolationError(
             f"Direct setting of '{name}' is not allowed. "