use-computer-cli 0.3.1__tar.gz → 0.3.2__tar.gz

{use_computer_cli-0.3.1 → use_computer_cli-0.3.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: use-computer-cli
-Version: 0.3.1
+Version: 0.3.2
 Summary: Execute input on a screen for computer-use agents: move, click, drag, scroll, type, key, screenshot.
 Project-URL: Homepage, https://github.com/applica-software-guru/use-computer
 Author: Bruno Fortunato

{use_computer_cli-0.3.1 → use_computer_cli-0.3.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "use-computer-cli"
-version = "0.3.1"
+version = "0.3.2"
 description = "Execute input on a screen for computer-use agents: move, click, drag, scroll, type, key, screenshot."
 readme = "README.md"
 requires-python = ">=3.10"

{use_computer_cli-0.3.1 → use_computer_cli-0.3.2}/src/use_computer/runner.py

@@ -681,7 +681,7 @@ class Session:
 
         # On the raw snapshot, before anything of ours removes a child: a region hidden by our
         # own pruning is not a region the platform failed to describe.
-        root = mark_unexposed(root)
+        root = mark_unexposed(root, depth)
         exposed_children = bool(root.children)
 
         if action.of is not None:

{use_computer_cli-0.3.1 → use_computer_cli-0.3.2}/src/use_computer/selectors.py

@@ -182,7 +182,7 @@ def active_window(entries: Sequence[WindowInfo]) -> WindowInfo:
     )
 
 
-def mark_unexposed(node: UINode) -> UINode:
+def mark_unexposed(node: UINode, depth_limit: int | None = None) -> UINode:
     """Say where a node's children do not account for the node's own area.
 
     A tree can be rich, correct, and silent about the only region that matters. Measured in GNOME
@@ -195,13 +195,15 @@ def mark_unexposed(node: UINode) -> UINode:
     served worst, because an agent could not tell it from a window that exposes everything.
 
     This runs on the **raw** snapshot, before pruning: a region hidden by our own pruning is not a
-    region the platform failed to describe.
+    region the platform failed to describe. For the same reason ``depth_limit`` is honoured -- at
+    the depth the snapshot stopped at, every node looks childless, and marking there would report
+    the caller's own limit as a property of the application.
     """
-    marked, _ = _mark(node)
+    marked, _ = _mark(node, depth_limit)
     return marked
 
 
-def _mark(node: UINode) -> tuple[UINode, bool]:
+def _mark(node: UINode, depth_limit: int | None, depth: int = 0) -> tuple[UINode, bool]:
     """Mark this subtree, and say whether anything in it carries a mark.
 
     Only the **innermost** node is marked. A canvas nested three panels deep would otherwise be
@@ -211,11 +213,14 @@ def _mark(node: UINode) -> tuple[UINode, bool]:
     children = []
     deeper = False
     for child in node.children:
-        marked, found = _mark(child)
+        marked, found = _mark(child, depth_limit, depth + 1)
         children.append(marked)
         deeper = deeper or found
     kids = tuple(children)
-    region = None if deeper else _blind_spot(node, kids)
+    # A node whose children were cut by the caller's own `--depth` has no evidence either way:
+    # saying the platform describes nothing there would be BUG-011 in a second place.
+    truncated_here = depth_limit is not None and depth >= depth_limit - 1
+    region = None if deeper or truncated_here else _blind_spot(node, kids)
     return (
         node.model_copy(update={"children": kids, "unexposed": region}),
         deeper or region is not None,

{use_computer_cli-0.3.1 → use_computer_cli-0.3.2}/tests/test_blind_spots.py

@@ -7,6 +7,8 @@ because the provider worked and returned plenty.
 
 from __future__ import annotations
 
+from collections.abc import Iterator
+
 from tests.fake_provider import node
 from use_computer.render import TREE_LEGEND
 from use_computer.render import tree as render_tree
@@ -155,3 +157,16 @@ def test_the_marker_is_rendered_and_the_legend_explains_it() -> None:
     text = render_tree(mark_unexposed(drawing_window()))
     assert "?unexposed" in TREE_LEGEND
     assert "?unexposed 163,106 1757x885" in text
+
+
+def test_a_depth_limit_does_not_manufacture_a_blind_spot() -> None:
+    # At the depth the snapshot stopped at every node looks childless. Marking there would report
+    # the caller's own limit as a property of the application -- the same mistake as BUG-011.
+    marked = mark_unexposed(drawing_window(), depth_limit=2)
+    assert all(found.unexposed is None for found in _every(marked))
+
+
+def _every(node: UINode) -> Iterator[UINode]:
+    yield node
+    for child in node.children:
+        yield from _every(child)