vis-agent 0.3.5.dev8266__tar.gz → 0.4.3__tar.gz

{vis_agent-0.3.5.dev8266 → vis_agent-0.4.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: vis-agent
-Version: 0.3.5.dev8266
+Version: 0.4.3
 Summary: The Vis Python SDK — author extensions and control local or remote engines.
 Project-URL: Homepage, https://vis.blockether.com
 Project-URL: Documentation, https://vis.blockether.com

{vis_agent-0.3.5.dev8266 → vis_agent-0.4.3}/contracts/schema/agents.json

@@ -638,16 +638,12 @@
                 "type": "integer",
                 "minimum": 1
               },
-              "kind": {
-                "type": "string"
-              },
               "content": {
                 "type": "string"
               }
             },
             "required": [
               "entry_id",
-              "kind",
               "content"
             ],
             "additionalProperties": false

{vis_agent-0.3.5.dev8266 → vis_agent-0.4.3}/contracts/schema/gateway.json

@@ -967,7 +967,8 @@
       "operations": {
         "get": {
           "request": "none",
-          "response": "json"
+          "response": "json",
+          "response_schema": "file_preview"
         }
       },
       "path": "/v1/sessions/:sid/fs/file"
@@ -3735,6 +3736,128 @@
           "pattern": "^[0-9a-f]{64}$"
         }
       }
+    },
+    "preview_image_bytes": {
+      "type": "integer",
+      "minimum": 0,
+      "maximum": 8388608,
+      "description": "Bytes of one picture in a file preview. The picture goes as base64 inside JSON, which is 4/3 larger, and a phone decodes it. A larger picture gets a 413 answer."
+    },
+    "file_preview": {
+      "description": "The answer of GET /v1/sessions/:sid/fs/file. The kind tells what the file holds and which keys come back.",
+      "oneOf": [
+        {
+          "type": "object",
+          "title": "Text",
+          "description": "One window of clipped lines around the line that the client named.",
+          "additionalProperties": false,
+          "required": [
+            "path",
+            "kind",
+            "line",
+            "first_line",
+            "lines",
+            "is_truncated",
+            "size_bytes"
+          ],
+          "properties": {
+            "path": {
+              "type": "string",
+              "minLength": 1
+            },
+            "kind": {
+              "const": "text"
+            },
+            "line": {
+              "type": "integer",
+              "minimum": 1,
+              "description": "The line that the client named, or 1."
+            },
+            "first_line": {
+              "type": "integer",
+              "minimum": 1,
+              "description": "The number of the first line in lines."
+            },
+            "lines": {
+              "type": "array",
+              "items": {
+                "type": "string"
+              }
+            },
+            "is_truncated": {
+              "type": "boolean",
+              "description": "True when the byte cap stopped the read before the window ended."
+            },
+            "size_bytes": {
+              "type": "integer",
+              "minimum": 0
+            }
+          }
+        },
+        {
+          "type": "object",
+          "title": "Image",
+          "description": "A picture that a browser draws, typed by its first bytes, not by its file name.",
+          "additionalProperties": false,
+          "required": [
+            "path",
+            "kind",
+            "media_type",
+            "size_bytes",
+            "data"
+          ],
+          "properties": {
+            "path": {
+              "type": "string",
+              "minLength": 1
+            },
+            "kind": {
+              "const": "image"
+            },
+            "media_type": {
+              "enum": [
+                "image/png",
+                "image/jpeg",
+                "image/gif",
+                "image/webp",
+                "image/avif"
+              ]
+            },
+            "size_bytes": {
+              "$ref": "#/$defs/preview_image_bytes"
+            },
+            "data": {
+              "type": "string",
+              "pattern": "^[A-Za-z0-9+/]*={0,2}$",
+              "description": "The bytes of the picture as standard base64."
+            }
+          }
+        },
+        {
+          "type": "object",
+          "title": "Binary",
+          "description": "Another file with a NUL byte. The gateway does not read it.",
+          "additionalProperties": false,
+          "required": [
+            "path",
+            "kind",
+            "size_bytes"
+          ],
+          "properties": {
+            "path": {
+              "type": "string",
+              "minLength": 1
+            },
+            "kind": {
+              "const": "binary"
+            },
+            "size_bytes": {
+              "type": "integer",
+              "minimum": 0
+            }
+          }
+        }
+      ]
     }
   }
 }

{vis_agent-0.3.5.dev8266 → vis_agent-0.4.3}/contracts/schema/view.json

@@ -1493,6 +1493,14 @@
               ],
               "default": "activate"
             },
+            "variant": {
+              "enum": [
+                "primary",
+                "secondary",
+                "destructive"
+              ],
+              "default": "secondary"
+            },
             "is_disabled": {
               "type": "boolean"
             },

{vis_agent-0.3.5.dev8266 → vis_agent-0.4.3}/pyproject.toml

@@ -8,7 +8,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "vis-agent"
-version = "0.3.5.dev8266"
+version = "0.4.3"
 description = "The Vis Python SDK — author extensions and control local or remote engines."
 readme = "README.md"
 requires-python = ">=3.12"

{vis_agent-0.3.5.dev8266 → vis_agent-0.4.3}/src/blockether/vis/_contracts.py

@@ -61,6 +61,22 @@ def live_items(key: str, items: list[dict[str, Any]]) -> list[dict[str, Any]]:
     return result
 
 
+def _button_branch() -> dict[str, Any]:
+    return next(
+        branch
+        for branch in definition("view", "component")["oneOf"]
+        if branch["properties"]["type"].get("const") == "button"
+    )
+
+
+def button_variant_error(variant: Any) -> str | None:
+    """Answer why a declared button variant is invalid, or None when it is valid."""
+    allowed = _button_branch()["properties"]["variant"]["enum"]
+    if variant is None or variant in allowed:
+        return None
+    return f"button variant must be one of {', '.join(allowed)}, not {variant!r}"
+
+
 def _defaults(key: str) -> dict[str, Any]:
     properties = definition("view", key)["properties"]
     return {

{vis_agent-0.3.5.dev8266 → vis_agent-0.4.3}/src/blockether/vis/_outside.py

@@ -1076,6 +1076,10 @@ def _live_check_node(node, seen, *, is_declaration=True):
             return f"a {kind} input has no value: only the host holds its handle"
     node_id = node.get("id")
     if not isinstance(node_id, str) or not node_id.strip():
+        if is_group:
+            from .extension import _LIVE_GROUP_NEEDS_ID
+
+            return _LIVE_GROUP_NEEDS_ID
         return f"a {kind} node needs an id"
     if node_id in seen:
         return f"two nodes answer to {node_id!r}"
@@ -1129,6 +1133,9 @@ def _live_check_node(node, seen, *, is_declaration=True):
             return "a button needs a label"
         if is_declaration and "clicks" in node:
             return "button clicks are stamped by the engine"
+        complaint = _contracts.button_variant_error(node.get("variant"))
+        if complaint:
+            return complaint
         node.setdefault("clicks", 0)
         node.setdefault("is_disabled", False)
         if type(node["clicks"]) is not int or node["clicks"] < 0:

{vis_agent-0.3.5.dev8266 → vis_agent-0.4.3}/src/blockether/vis/engine/_client.py

@@ -1578,7 +1578,7 @@ class ExecutionLayer(ABC):
     def get_session_fs_file(
         self, sid: str, *, query: Query | None = None, timeout: float | None = None
     ) -> JSONValue:
-        """GET /v1/sessions/:sid/fs/file — read one workspace file as text."""
+        """GET /v1/sessions/:sid/fs/file — preview one workspace file by its kind."""
         response = self._request(
             "GET",
             "/v1/sessions/:sid/fs/file",

{vis_agent-0.3.5.dev8266 → vis_agent-0.4.3}/src/blockether/vis/evals/grade.py

@@ -52,6 +52,7 @@ CHECK_KEYS = frozenset(
         "id",
         "max_form_output_chars",
         "max_patch_calls_per_file",
+        "max_patch_refusals",
         "max_total_output_chars",
         "measurement",
         "verify",
@@ -306,51 +307,27 @@ def council_recovery_evidence(form_outputs, activities, expected):
             block.get("text") for block in blocks if block.get("type") == "markdown"
         ]
 
-    publications = [
-        row
-        for row in activities
-        if row.get("operation") == "council.publish" and row.get("state") == "succeeded"
-    ]
-    roots = [
-        row
-        for row in publications
-        if references(row, "council-entry") == {thread_id}
-        and references(row, "council-thread") == {thread_id}
-        and bodies(row) == [expected["root_content"]]
-    ]
-    replies = [
-        row
-        for row in publications
-        if references(row, "council-entry") == {entry_id}
-        and references(row, "council-thread") == {thread_id}
-        and bodies(row) == [expected["reply_content"]]
-    ]
+    # A publication has no Activity row. The repaired read proves both messages.
     reads = [
         row
         for row in activities
         if row.get("operation") == "council.read" and row.get("state") == "succeeded"
     ]
-    if (
-        len(publications) != 2
-        or len(roots) != 1
-        or len(replies) != 1
-        or len(reads) != 1
-    ):
+    if len(reads) != 1:
         return (
-            ["Council recovery lacks the two real publications and one repaired read"],
+            ["Council recovery lacks one repaired read"],
             set(),
             [],
         )
-    root, reply, repaired = roots[0], replies[0], reads[0]
+    repaired = reads[0]
     failures = []
     if failed.get("state") != "failed" or failed.get("scope") != event.get("scope"):
         failures.append("Council recovery error does not match its failed Activity")
     if any(
-        not row.get("scope") or row["scope"] not in order
-        for row in (root, reply, failed, repaired)
+        not row.get("scope") or row["scope"] not in order for row in (failed, repaired)
     ):
         failures.append("Council recovery is missing a form scope")
-    if not position(root) < position(reply) < position(failed) < position(repaired):
+    if not position(failed) < position(repaired):
         failures.append(
             "Council recovery did not read the thread after the reply-ID error"
         )
@@ -436,6 +413,18 @@ def structured_failures(sc, work, answer, activities):
                 failures.append(
                     f"{path} was patched {count} times; collect its edits into at most {limit} patch call(s)"
                 )
+    # User report 2026-10-31: each refused patch cost the agent one more step.
+    limit = sc.get("max_patch_refusals")
+    if type(limit) is int:
+        refused = sum(
+            1
+            for row in activities
+            if row["operation"] == "patch"
+            for call in row.get("children") or [row]
+            if call.get("state", row.get("state")) == "failed"
+        )
+        if refused > limit:
+            failures.append(f"patch was refused {refused} times; the limit is {limit}")
     forbidden = set(sc.get("forbid_tools") or [])
     for operation in sorted({row["operation"] for row in activities} & forbidden):
         failures.append(f"forbidden tool {operation!r} was used")

{vis_agent-0.3.5.dev8266 → vis_agent-0.4.3}/src/blockether/vis/extension.py

@@ -3886,6 +3886,30 @@ def notify(text, level="info"):
     _host.notify(str(text), str(level))
 
 
+def write_private_file(path: str | PathLike[str], text: str) -> None:
+    """Replace `path` with `text` in one atomic step, readable only by its owner.
+
+    A failure keeps the previous file. A symbolic link writes the file it names.
+    Missing parent directories are created owner-only.
+    """
+    target = Path(path)
+    if target.is_symlink():
+        target = target.resolve(strict=True)
+    for directory in reversed([d for d in target.parents if not d.exists()]):
+        directory.mkdir(mode=0o700)
+    tmp = target.with_name(f".{target.name}.tmp-{os.urandom(8).hex()}")
+    fd = os.open(tmp, os.O_WRONLY | os.O_CREAT | os.O_EXCL, 0o600)
+    try:
+        with os.fdopen(fd, "w", encoding="utf-8") as handle:
+            handle.write(text)
+            handle.flush()
+            os.fsync(handle.fileno())
+        os.replace(tmp, target)
+    except BaseException:
+        tmp.unlink(missing_ok=True)
+        raise
+
+
 def _shell_options(name, opts):
     if not isinstance(opts, dict):
         raise TypeError(
@@ -6277,6 +6301,12 @@ def live(title, nodes, placement=None, key=None, on_event=None, **options):
 
 # -- Extension live-view test harness -----------------------------------------
 
+# The refusal of a live group without an id, in the recorder and the outside host.
+_LIVE_GROUP_NEEDS_ID = (
+    "a live view group needs an id, because patches and after= name it: "
+    "write vis.ui.group('actions', ...) or vis.ui.wrap('actions', ...)"
+)
+
 
 class _LiveRecorder:
     """An isolated in-memory live host for testing any extension live view.
@@ -6366,6 +6396,12 @@ class _LiveRecorder:
                 node.setdefault("variant", "braille")
                 node.setdefault("is_active", True)
             elif kind == "button":
+                from blockether.vis import _contracts
+
+                complaint = _contracts.button_variant_error(node.get("variant"))
+                if complaint:
+                    # Vis raises VisToolError, a RuntimeError, so the recorder does too.
+                    raise RuntimeError(f"button {node.get('id')}: {complaint}")
                 node["clicks"] = 0
                 node.setdefault("is_disabled", False)
             elif kind == "card" and node.get("is_selectable"):
@@ -6568,12 +6604,9 @@ class _LiveRecorder:
         if action == "open":
             for node in self._nodes(envelope["view"].get("nodes")):
                 # The engine refuses a live group without an id, so a test must too.
+                # Vis raises VisToolError, a RuntimeError, so the recorder does too.
                 if node.get("type") == "group" and not node.get("id"):
-                    raise AssertionError(
-                        "a live view group needs an id, because patches and "
-                        "after= name it: write vis.ui.group('actions', ...) "
-                        "or vis.ui.wrap('actions', ...)"
-                    )
+                    raise RuntimeError(_LIVE_GROUP_NEEDS_ID)
             self._view = self._materialize(envelope["view"])
             self._declared = set(envelope.get("validators") or [])
             self._verdicts = {}
@@ -7077,14 +7110,22 @@ def _ui_spinner(node_id, text="Working", *, variant="braille", is_active=True, *
     )
 
 
-def _ui_button(node_id, label, *, is_disabled=False, **spec):
+def _ui_button(node_id, label, *, is_disabled=False, variant=None, **spec):
     """An operator action. Accepted presses increment ``clicks`` in ``view.state()``.
 
-    No callback or code crosses the wire. The producer decides how to respond.
+    ``variant`` is ``"primary"``, ``"secondary"`` (the default) or ``"destructive"``.
+    Give one primary action per group. No callback or code crosses the wire.
     Disabled buttons and completed receipts cannot be activated.
     """
+    from blockether.vis import _contracts
+
+    complaint = _contracts.button_variant_error(variant)
+    if complaint:
+        raise ValueError(f"button {node_id}: {complaint}")
     return _live_node(
-        "button", node_id, dict(spec, label=str(label), is_disabled=is_disabled)
+        "button",
+        node_id,
+        dict(spec, label=str(label), is_disabled=is_disabled, variant=variant),
     )
 
 

{vis_agent-0.3.5.dev8266 → vis_agent-0.4.3}/src/blockether/vis/extension_package.py

@@ -43,6 +43,8 @@ LATEST = "latest"
 LOCK_WAIT_S = 300
 MAX_METADATA = 128 * 1024
 _EXCLUDED = {".git", ".venv", "venv", "__pycache__", "node_modules", ".DS_Store"}
+# `.env*` holds secrets. `.venv-*` is an environment that Vis prepares for one interpreter.
+_EXCLUDED_PREFIXES = (".env", ".venv-")
 
 
 def _relative(value):
@@ -51,7 +53,7 @@ def _relative(value):
     if len(value) > 512 or any(ord(c) < 32 or ord(c) == 127 for c in value):
         raise ValueError("Project paths are too long or contain control characters")
     if any(
-        p in {"", ".", ".."} or p in _EXCLUDED or p.startswith(".env")
+        p in {"", ".", ".."} or p in _EXCLUDED or p.startswith(_EXCLUDED_PREFIXES)
         for p in value.split("/")
     ):
         raise ValueError("Project contains an unsafe or excluded path")
@@ -346,7 +348,11 @@ def _checkout(repository, directory, revision):
 
 def _copy_project(source, destination):
     def excluded(_directory, names):
-        return {name for name in names if name in _EXCLUDED or name.startswith(".env")}
+        return {
+            name
+            for name in names
+            if name in _EXCLUDED or name.startswith(_EXCLUDED_PREFIXES)
+        }
 
     count, size = 0, 0
     for base, directories, files in os.walk(source):

{vis_agent-0.3.5.dev8266 → vis_agent-0.4.3}/tests/test_engine.py

@@ -48,16 +48,22 @@ def sdk_flow(mode):
 
 def sdk_native_wait(marker: str):
     "Wait inside C until cancellation reclaims this trusted extension worker."
+    import _thread
     import os
     import threading
     from pathlib import Path
-    condition = threading.Condition()
+    # A raw `_thread` lock acquire stays in C. The runtime slices threading's
+    # own waits, so a Condition wait would unwind on the interrupt.
+    gate = threading.Lock()
+    waiter = _thread.allocate_lock()
+    waiter.acquire()
     def notify_waiting():
-        with condition:
+        with gate:
             Path(marker).write_text(str(os.getpid()))
-    with condition:
-        threading.Thread(target=notify_waiting, daemon=True).start()
-        condition.wait(120)
+    gate.acquire()
+    threading.Thread(target=notify_waiting, daemon=True).start()
+    gate.release()
+    waiter.acquire(timeout=120)
 
 vis.register_extension(vis.Extension(name="sdk-fixture", description="SDK integration fixture", alias="sdk",
     symbols=[vis.Symbol(sdk_flow, activity=vis.Activity(presenter="observation", label="SDK flow", render=sdk_activity)),
@@ -602,9 +608,9 @@ def test_real_council_roundtrip(tmp_path, monkeypatch, transport):
                         "turn.cancelled",
                     }:
                         break
+            # User request: the team notes show each message, so a publication has no row.
             assert sorted(operations.values()) == [
                 "council.get",
-                "council.publish",
                 "council.threads",
             ]
             assert turn.wait(timeout=30)["status"] == "completed"

{vis_agent-0.3.5.dev8266 → vis_agent-0.4.3}/tests/test_evals_guards.py

@@ -1444,6 +1444,22 @@ class EvaluationIntegrationEdgeTest(unittest.TestCase):
         group = atif.row("group-call-1", "patch", children=split)
         self.assertEqual(1, len(run(group)))
 
+    def test_patch_refusals_stay_within_the_limit(self):
+        # User report 2026-10-31: each refused patch cost the agent one more step.
+        def run(*rows):
+            return grade.structured_failures(
+                {"max_patch_refusals": 0}, ".", "", list(rows)
+            )
+
+        written = atif.row("call-1", "patch", summary="app.py")
+        refused = atif.row("call-2", "patch", "failed", summary="app.py", sequence=2)
+        self.assertEqual([], run(written))
+        self.assertEqual(
+            ["patch was refused 1 times; the limit is 0"], run(written, refused)
+        )
+        group = atif.row("group-call-1", "patch", children=[written, refused])
+        self.assertEqual(1, len(run(group)))
+
     def test_malformed_run_metrics_are_a_failure_not_a_crash(self):
         helper = DiscoveryEvaluationTest()
         malformed = {
@@ -1509,17 +1525,6 @@ class CouncilRecoveryTest(unittest.TestCase):
             )
 
         return {
-            "root": row(
-                "root", "council.publish", "succeeded", [42], expected["root_content"]
-            ),
-            "reply": row(
-                "reply",
-                "council.publish",
-                "succeeded",
-                [43],
-                expected["reply_content"],
-                sequence=2,
-            ),
             "mistake": row("mistake", "council.read", "failed", []),
             "error": {
                 "type": "error",
@@ -1531,11 +1536,12 @@ class CouncilRecoveryTest(unittest.TestCase):
         }
 
     def blocks(self, parts):
-        """Return the publish, mistake and repair blocks of the run, in that order."""
+        """Return the publish, mistake and repair blocks of the run, in that order.
+
+        A Council publication leaves no Activity row, so the publish block has none.
+        """
         return [
-            atif.block(
-                "t1/i1/f1", activity=atif.receipt(parts["root"], parts["reply"])
-            ),
+            atif.block("t1/i1/f1"),
             atif.block(
                 "t1/i2/f1",
                 error=parts["error"],

{vis_agent-0.3.5.dev8266 → vis_agent-0.4.3}/tests/test_live_primitives.py

@@ -241,16 +241,44 @@ def test_table_builder_refuses_text_columns():
         vis.ui.table("chats", columns=[vis.ui.table_column("chat", "Chat"), "Category"])
 
 
+def test_button_variant_is_kept_and_a_typo_names_the_variants():
+    # #430: a button had no hierarchy, and a misspelled variant reached the host.
+    assert vis.ui.button("save", "Save", variant="primary")["variant"] == "primary"
+    assert "variant" not in vis.ui.button("save", "Save")
+    with pytest.raises(ValueError, match=r"primary, secondary, destructive.*'danger'"):
+        vis.ui.button("drop", "Drop", variant="danger")
+
+
+def test_outside_refuses_an_unknown_button_variant():
+    # #430: outside Vis a raw button with a typo variant was accepted.
+    button = {"id": "drop", "type": "button", "label": "Drop", "variant": "danger"}
+    view = {"title": "Rows", "nodes": [button]}
+    with pytest.raises(RuntimeError, match=r"primary, secondary, destructive"):
+        _outside.live(json.dumps({"op": "open", "view": view}))
+
+
 def test_recorder_refuses_a_live_group_without_an_id(monkeypatch):
     # #428: the recorder hid the buttons of an unnamed group; the engine refuses it.
+    # Vis raises VisToolError, a RuntimeError, so the recorder must raise one too.
     monkeypatch.setattr(vis, "_host", vis.testing.LiveRecorder(vis._host))
-    with pytest.raises(AssertionError, match="group needs an id"):
+    with pytest.raises(
+        RuntimeError, match=r"group needs an id.*vis\.ui\.group\('actions'"
+    ):
         with vis.live(
             "Chats", [vis.ui.wrap(vis.ui.button("save", "Save"))], flush_ms=0
         ):
             pass
 
 
+def test_outside_refuses_a_live_group_without_an_id_with_the_fix():
+    # #428: outside Vis the refusal named no fix for an unnamed group.
+    view = {"title": "Chats", "nodes": [vis.ui.wrap(vis.ui.button("save", "Save"))]}
+    with pytest.raises(
+        RuntimeError, match=r"group needs an id.*vis\.ui\.group\('actions'"
+    ):
+        _outside.live(json.dumps({"op": "open", "view": view}))
+
+
 def test_outside_refused_patch_leaves_state_unchanged():
     opened = json.loads(
         _outside.live(