fix: address reviewer feedback on LockMap implementation

- Fix AttributeError: Change _session.py reset() to use .drop() instead of .pop()
- Fix LRU eviction logic: Continue past locked entries instead of breaking
- Update docstring: Clarify asyncio-safe vs thread-safe guarantees

Co-authored-by: Mervin Praison <MervinPraison@users.noreply.github.com>

Author: praisonai-triage-agent[bot]

src/praisonai/praisonai/_lockmap.py

@@ -11,10 +11,14 @@
 
 
 class LockMap:
-    """Thread-safe per-key asyncio.Lock map with automatic cleanup.
+    """Asyncio-safe per-key asyncio.Lock map with automatic cleanup.
     
     Provides a lock per key with LRU eviction and TTL-based cleanup to prevent
     unbounded memory growth in long-running applications.
+    
+    Note: Safe for use within a single asyncio event loop (cooperative
+    multitasking means no interleaving between non-awaited calls), but NOT
+    safe for concurrent access from multiple OS threads.
     """
 
     def __init__(self, *, max_entries: int = 10_000, ttl_seconds: float = 3600.0):
@@ -68,8 +72,10 @@ def _evict_stale(self, now: float) -> None:
             if lock.locked():
                 # Don't evict locks currently held; bump them to the end
                 self._locks.move_to_end(k)
-                break
+                # Continue trying to evict other unlocked entries
+                continue
             self._locks.popitem(last=False)
+            break
 
     def drop(self, key: Hashable) -> None:
         """Manually remove a lock for the given key."""

src/praisonai/praisonai/bots/_session.py

@@ -522,7 +522,7 @@ def reset(self, user_id: str) -> bool:
         existed = storage_key in self._histories
         self._histories.pop(storage_key, None)
         self._last_active.pop(storage_key, None)
-        self._locks.pop(storage_key, None)
+        self._locks.drop(storage_key)
 
         if self._store is not None:
             persist_key = self._session_key(user_id)