path in change events

src/hooked_containers/common.py

@@ -1,6 +1,6 @@
 from abc import ABC, abstractmethod
-from collections.abc import MutableMapping, MutableSequence
-from typing import Generic, Protocol, TypeVar
+from collections.abc import Container, Iterable, MutableMapping, MutableSequence, Sized
+from typing import Protocol, TypeVar
 
 from . import events as e
 
@@ -13,7 +13,7 @@ class HookFunction(Protocol):
     def __call__(self, event: e.ChangeEvent[T]) -> None: ...
 
 
-class HookedContainer(Generic[T], ABC):
+class HookedContainer(ABC, Sized, Iterable[T], Container[T]):
     _path: MutableSequence[int]
     hook: HookFunction
 
@@ -35,12 +35,12 @@ class HookedContainer(Generic[T], ABC):
     def __setitem__(self, s, value):
         self._data[s] = value
         if self.hook:
-            self.hook(e.SetItemEvent(index=s, item=value))
+            self.hook(e.SetItemEvent(index=s, item=value, path=self._path))
 
     def __delitem__(self, s):
-        del self._data[s]
+        item = self._data.pop(s)
         if self.hook:
-            self.hook(e.RemoveItemEvent(index=s, item=self._data[s]))
+            self.hook(e.RemoveItemEvent(index=s, item=item, path=self._path))
 
     @abstractmethod
     def insert(self, index, value): ...

src/hooked_containers/events.py

@@ -1,3 +1,4 @@
+from collections.abc import MutableSequence
 from dataclasses import dataclass
 from typing import Generic, TypeVar
 
@@ -8,6 +9,7 @@ T = TypeVar("T")
 class ChangeEvent(Generic[T]):
     index: int
     item: T
+    path: MutableSequence[int] | None = None
 
 
 @dataclass(frozen=True)

src/hooked_containers/sequence.py

@@ -24,12 +24,14 @@ class HookedList(HookedContainer[T], MutableSequence[T]):
         self._path = list(path) if path is not None else []
 
     def __getitem__(self, s):
-        # print("Getting item:", s)
         match self._data[s]:
+            case MutableSequence() as seq:
+                new_path = copy(self._path)
+                new_path.append(s)
+                return type(self)(self.hook, existing=seq, path=new_path)
             case HookedContainer(_data=seq):
                 new_path = copy(self._path)
                 new_path.append(s)
-                # print(new_path)
                 return type(self)(self.hook, existing=seq, path=new_path)
             case _ as item:
                 return item
@@ -37,4 +39,4 @@ class HookedList(HookedContainer[T], MutableSequence[T]):
     def insert(self, index, value):
         self._data.insert(index, value)
         if self.hook:
-            self.hook(e.AddItemEvent(index=index, item=value))
+            self.hook(e.AddItemEvent(index=index, item=value, path=self._path))