statepatch 0.1.0__tar.gz → 0.1.1__tar.gz

{statepatch-0.1.0 → statepatch-0.1.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: statepatch
-Version: 0.1.0
+Version: 0.1.1
 Summary: Immer-style ops over a plain JSON tree: observables that emit every mutation as an op
 Project-URL: Repository, https://github.com/assistant-ui/harness-sdk
 License-Expression: MIT

{statepatch-0.1.0 → statepatch-0.1.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "statepatch"
-version = "0.1.0"
+version = "0.1.1"
 description = "Immer-style ops over a plain JSON tree: observables that emit every mutation as an op"
 readme = "README.md"
 license = "MIT"

{statepatch-0.1.0 → statepatch-0.1.1}/src/statepatch/__init__.py

@@ -320,11 +320,7 @@ class Proxy:
             else container[key]
         )
         path = rel + (key,)
-        if isinstance(old, str) and isinstance(value, str) and value.startswith(old):
-            if value == old:
-                return
-            container[key] = value
-            owner._emit(Op("add", path + (len(old),), value[len(old) :]))
+        if isinstance(old, str) and value == old:
             return
         stored, mounts = _adopt(value, path)
         owner._admit(mounts, path)
@@ -522,3 +518,22 @@ class Proxy:
     def __repr__(self) -> str:
         _owner, _rel, node = self._target()
         return repr(_snapshot(node))
+
+
+def append(parent: Any, key: Segment, suffix: str) -> None:
+    """Append ``suffix`` to the string at ``parent[key]``, recording an end-offset text-insert ``add`` op."""
+    if isinstance(parent, Observable):
+        parent = Proxy(parent, ())
+    if not isinstance(parent, Proxy):
+        raise TypeError(f"append expects a proxy or observable, got {type(parent).__name__}")
+    if not isinstance(suffix, str):
+        raise TypeError(f"append suffix must be a string, got {type(suffix).__name__}")
+    owner, rel, container = parent._target()
+    key = _index(container, key)
+    old = container[key]
+    if not isinstance(old, str):
+        raise TypeError(f"cannot append to a non-string at path {list(rel + (key,))!r}")
+    if suffix == "":
+        return
+    container[key] = old + suffix
+    owner._emit(Op("add", rel + (key, len(old)), suffix))

{statepatch-0.1.0 → statepatch-0.1.1}/tests/test_observable.py

@@ -4,7 +4,7 @@ mounted observables forward their op streams at the mount path."""
 from datetime import datetime, timezone
 
 import pytest
-from statepatch import Op, observable
+from statepatch import Op, append, observable
 
 
 def collect(obs):
@@ -27,18 +27,46 @@ def test_dict_and_list_mutations_emit_wire_shaped_ops():
     state["status"] = "streaming"
     state["messages"].append({"id": "m1", "content": ""})
     state["messages"][0]["content"] = "hel"
-    state["messages"][0]["content"] = "hello"  # suffix diff -> string add
+    state["messages"][0]["content"] = "hello"  # assign is always a replace
     state["messages"].pop(0)
     assert ops == [
         Op("replace", ("status",), "streaming"),
         Op("add", ("messages", 0), {"id": "m1", "content": ""}),
-        Op("add", ("messages", 0, "content", 0), "hel"),
-        Op("add", ("messages", 0, "content", 3), "lo"),
+        Op("replace", ("messages", 0, "content"), "hel"),
+        Op("replace", ("messages", 0, "content"), "hello"),
         Op("remove", ("messages", 0)),
     ]
     assert state.value == {"status": "streaming", "messages": []}
 
 
+def test_append_emits_an_end_offset_string_add():
+    state = observable({"text": "he", "parts": ["ab"]})
+    ops = collect(state)
+    append(state, "text", "llo")
+    append(state["parts"], 0, "c")
+    append(state, "text", "")  # empty suffix is a no-op
+    assert state.value == {"text": "hello", "parts": ["abc"]}
+    assert ops == [
+        Op("add", ("text", 2), "llo"),
+        Op("add", ("parts", 0, 2), "c"),
+    ]
+
+
+def test_append_fails_fast_on_bad_targets():
+    state = observable({"n": 1, "text": "hi"})
+    ops = collect(state)
+    with pytest.raises(TypeError, match="non-string"):
+        append(state, "n", "x")
+    with pytest.raises(TypeError, match="suffix must be a string"):
+        append(state, "text", 1)
+    with pytest.raises(KeyError):
+        append(state, "missing", "x")
+    with pytest.raises(TypeError, match="proxy or observable"):
+        append({"text": "hi"}, "text", "x")
+    assert state.value == {"n": 1, "text": "hi"}
+    assert ops == []
+
+
 def test_values_are_dealised_on_write():
     state = observable({})
     held = {"a": [1]}