vis-agent 0.2.54.dev7904__tar.gz → 0.3.1__tar.gz

{vis_agent-0.2.54.dev7904 → vis_agent-0.3.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: vis-agent
-Version: 0.2.54.dev7904
+Version: 0.3.1
 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.2.54.dev7904 → vis_agent-0.3.1}/contracts/schema/agents.json

@@ -481,6 +481,40 @@
         },
         "warning": {
           "type": "string"
+        },
+        "error": {
+          "description": "Why a failed child stopped. Code rate_limited means a 429 from its provider.",
+          "type": "object",
+          "properties": {
+            "code": {
+              "enum": [
+                "rate_limited"
+              ]
+            },
+            "message": {
+              "type": "string"
+            },
+            "provider": {
+              "type": "string",
+              "minLength": 1
+            },
+            "model": {
+              "type": "string",
+              "minLength": 1
+            }
+          },
+          "required": [
+            "code",
+            "message"
+          ],
+          "additionalProperties": false
+        },
+        "alternatives": {
+          "description": "Working provider and model routes for a new child, outside a rate-limit or authentication cooldown.",
+          "type": "array",
+          "items": {
+            "$ref": "#/$defs/model_pair"
+          }
         }
       },
       "required": [

{vis_agent-0.2.54.dev7904 → vis_agent-0.3.1}/contracts/schema/config.json

@@ -700,12 +700,6 @@
         "enabled": {
           "type": "boolean"
         },
-        "environment": {
-          "enum": [
-            "declared",
-            "inherit"
-          ]
-        },
         "filesystem": {
           "$ref": "#/$defs/jailFilesystem"
         },

{vis_agent-0.2.54.dev7904 → vis_agent-0.3.1}/pyproject.toml

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

{vis_agent-0.2.54.dev7904 → vis_agent-0.3.1}/src/blockether/vis/_outside.py

@@ -111,6 +111,25 @@ def workspace_root():
     return str(Path.cwd())
 
 
+def workspace_files():
+    """List the files that Git sees under the process directory, or None."""
+    root = Path.cwd()
+    try:
+        done = subprocess.run(
+            ["git", "ls-files", "-z", "--cached", "--others", "--exclude-standard"],
+            cwd=root,
+            capture_output=True,
+            timeout=60,
+            check=False,
+        )
+    except (OSError, subprocess.TimeoutExpired):
+        return None
+    if done.returncode != 0:
+        return None
+    names = done.stdout.decode("utf-8", "surrogateescape").split("\0")
+    return [str(root / name) for name in names if name]
+
+
 def python_environment():
     """Use the project environment when no Vis session owns this call."""
     return "project"
@@ -1443,6 +1462,7 @@ _IMPLEMENTATIONS = {
     "workspace_root": workspace_root,
     "project_root": workspace_root,
     "python_environment": python_environment,
+    "workspace_files": workspace_files,
     "setting_declaration": setting_declaration,
     "setting": setting,
     "state_get": state_get,

{vis_agent-0.2.54.dev7904 → vis_agent-0.3.1}/src/blockether/vis/engine/_agents.py

@@ -40,7 +40,9 @@ class SubagentRecord:
     ``status`` is ``asking`` when the child waits for its parent. Then ``question``,
     ``budget_request`` or ``files_request`` tells what it needs. ``held`` means that a
     human holds the queue of the child; only a person can release it. ``unknown`` names a
-    session that is not a subagent of the caller. ``warning`` reports a file overlap.
+    session that is not a subagent of the caller. ``warning`` reports a file overlap. A child
+    that a 429 stopped is ``failed`` with ``error`` (code ``rate_limited``) and ``alternatives``:
+    the working provider and model routes for a new child.
     """
 
     session_id: str
@@ -54,6 +56,8 @@ class SubagentRecord:
     iteration_budget: int | None = None
     iterations_used: int | None = None
     warning: str | None = None
+    error: dict[str, Any] | None = None
+    alternatives: list[dict[str, str]] | None = None
 
     @classmethod
     def from_wire(cls, value: dict[str, Any]) -> SubagentRecord:

{vis_agent-0.2.54.dev7904 → vis_agent-0.3.1}/src/blockether/vis/extension.py

@@ -65,6 +65,10 @@ class Host(Protocol):
         """Read the project's `python.environment`: `project` or `shared`."""
         ...
 
+    def workspace_files(self) -> list[str] | None:
+        """List the files that Git sees in the working copy, or None without Git."""
+        ...
+
     def setting_declaration(self, spec: Mapping[str, Any]) -> Mapping[str, Any]:
         """Validate a declaration and fill defaults from the contract schema."""
         ...
@@ -3497,6 +3501,19 @@ def project_root() -> Path:
     return Path(_host.project_root())
 
 
+def workspace_files() -> list[Path] | None:
+    """Return the files under `workspace_root()` that Git sees.
+
+    The list holds tracked files and untracked files that Git does not ignore.
+    The host lists the working copy once and gives the same list to each
+    extension that asks within a short time. Use it instead of a child process
+    for each extension. The result is None when the workspace is not in a Git
+    working tree. Outside Vis, it lists the process directory.
+    """
+    files = _host.workspace_files()
+    return None if files is None else [Path(name) for name in files]
+
+
 def python_environment() -> Literal["project", "shared"]:
     """Return the Python environment that the session's project selects.
 

{vis_agent-0.2.54.dev7904 → vis_agent-0.3.1}/src/blockether/vis/extension_package.py

@@ -1410,6 +1410,35 @@ def _sync_one(name, spec, directory, current, refresh, vis_version):
     return {**result, "status": "updated" if exists else "installed"}
 
 
+def _sync_read_only(specs, directory):
+    """Reconcile a scope that cannot be written, without the sync lock or network.
+
+    A container image or a read-only mount holds packages that a writable sync prepared.
+    Ready packages stay cached, also with `latest`. A needed change fails and names the step.
+    """
+    records = _sync_records(directory)
+    results = []
+    for name in sorted(set(specs) | set(records)):
+        spec, current = specs.get(name), records.get(name)
+        if spec is None:
+            results.append({"name": name, "status": "orphaned"})
+        elif (
+            current is not None
+            and current["spec"] == spec
+            and _sync_owned(_destination(directory, name), current)
+        ):
+            results.append({**current["result"], "name": name, "status": "cached"})
+        else:
+            results.append(
+                {
+                    "name": name,
+                    "status": "failed",
+                    "error": f"{directory} is read-only. Run the sync where it is writable",
+                }
+            )
+    return results
+
+
 def sync(
     configured,
     directory,
@@ -1435,6 +1464,8 @@ def sync(
     directory = Path(directory).expanduser().resolve()
     if not directory.exists() and not specs:
         return []
+    if not dry_run and directory.is_dir() and not os.access(directory, os.W_OK):
+        return _sync_read_only(specs, directory)
     if not dry_run:
         directory.mkdir(parents=True, exist_ok=True)
     lock = directory / ".sync-lock"

{vis_agent-0.2.54.dev7904 → vis_agent-0.3.1}/tests/test_agents.py

@@ -180,6 +180,11 @@ def test_child_controls_answer_records_and_wait_longer_than_the_gateway():
         {"status": "completed"},
         {"session_id": "child", "status": "invented"},
         {"session_id": "child", "status": "completed", "future_field": 1},
+        {
+            "session_id": "child",
+            "status": "failed",
+            "error": {"code": "other", "message": "x"},
+        },
     ],
 )
 def test_child_record_response_is_validated(record):
@@ -187,6 +192,24 @@ def test_child_record_response_is_validated(record):
         SubagentRecord.from_wire(record)
 
 
+def test_rate_limited_child_record_names_working_routes():
+    record = SubagentRecord.from_wire(
+        {
+            "session_id": "child",
+            "status": "failed",
+            "error": {
+                "code": "rate_limited",
+                "message": "Provider rate-limited",
+                "provider": "busy",
+                "model": "large",
+            },
+            "alternatives": [{"provider": "spare", "model": "small"}],
+        }
+    )
+    assert record.error["code"] == "rate_limited"
+    assert record.alternatives == [{"provider": "spare", "model": "small"}]
+
+
 @pytest.mark.parametrize("code", ["disabled", "group-not-found"])
 def test_unavailable_council_binding_does_not_block_team_controls(code):
     from blockether.vis.engine import GatewayError

{vis_agent-0.2.54.dev7904 → vis_agent-0.3.1}/tests/test_extension_package.py

@@ -1022,6 +1022,28 @@ def test_sync_local_install_cache_and_explicit_prune(tmp_path):
     assert not (target / "vis-greeter/current").exists()
 
 
+# A read-only image holds prepared packages; loading them must not need the sync lock.
+@pytest.mark.skipif(os.geteuid() == 0, reason="root ignores directory permissions")
+def test_sync_reads_a_read_only_scope_without_lock_or_changes(tmp_path):
+    source = project(tmp_path / "source")
+    target = tmp_path / "extensions"
+    configured = {"vis-greeter": {"source": str(source)}}
+    installed = package.sync(configured, target, trust=True)[0]
+    target.chmod(0o555)
+    try:
+        cached = package.sync(configured, target, trust=True)[0]
+        assert cached["status"] == "cached"
+        assert cached["path"] == installed["path"]
+        assert package.sync({}, target, trust=True)[0]["status"] == "orphaned"
+        changed = {"vis-other": {"source": str(source)}}
+        failed = {r["name"]: r for r in package.sync(changed, target, trust=True)}
+        assert failed["vis-other"]["status"] == "failed"
+        assert "read-only" in failed["vis-other"]["error"]
+        assert not (target / ".sync-lock").exists()
+    finally:
+        target.chmod(0o755)
+
+
 def test_sync_github_warm_path_does_not_fetch_or_check_catalog(releases, monkeypatch):
     _, target, commands = releases
     configured = {

vis_agent-0.2.54.dev7904/release-tag

@@ -1 +0,0 @@
-beta-317ff6130bb5a97928fab5a74225d812ace95bff