vis-agent 0.2.37.dev7443__tar.gz → 0.2.38.dev7473__tar.gz

{vis_agent-0.2.37.dev7443 → vis_agent-0.2.38.dev7473}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: vis-agent
-Version: 0.2.37.dev7443
+Version: 0.2.38.dev7473
 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.37.dev7443 → vis_agent-0.2.38.dev7473}/contracts/schema/gateway.json

@@ -1779,7 +1779,7 @@
         "objective": {
           "type": "string",
           "minLength": 1,
-          "maxLength": 8192,
+          "maxLength": 44000,
           "pattern": "\\S"
         },
         "status": {

{vis_agent-0.2.37.dev7443 → vis_agent-0.2.38.dev7473}/pyproject.toml

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

{vis_agent-0.2.37.dev7443 → vis_agent-0.2.38.dev7473}/src/blockether/vis/engine/_client.py

@@ -3251,8 +3251,8 @@ class Session:
         replies. The last iteration can run its tools, and then the limit blocks
         the next model request. Token usage is recorded only as a statistic.
         """
-        if not isinstance(objective, str) or not 1 <= len(objective.strip()) <= 8192:
-            raise ValueError("objective must contain 1–8192 characters")
+        if not isinstance(objective, str) or not 1 <= len(objective.strip()) <= 44000:
+            raise ValueError("objective must contain 1–44000 characters")
         if iteration_budget is not None and (
             type(iteration_budget) is not int
             or not 1 <= iteration_budget <= 9007199254740991

{vis_agent-0.2.37.dev7443 → vis_agent-0.2.38.dev7473}/src/blockether/vis/extension_package.py

@@ -954,6 +954,59 @@ def install(
     return _admit(source, directory, subdirectory, revision, vis_version, release)
 
 
+def _replace_managed(directory, active, current, release, vis_version):
+    """Activate a managed version and keep its sync ownership in the same operation."""
+    name = current["name"]
+    lock = directory / ".sync-lock"
+    fd = _acquire(lock)
+    try:
+        records = _sync_records(directory)
+        owned = records.get(name)
+        destination = _destination(directory, name)
+        if owned is not None and not _sync_owned(destination, owned):
+            raise ValueError(
+                "Existing extension was changed externally; no files replaced"
+            )
+        spec = _sync_spec(owned["spec"]) if owned is not None else None
+        previous_target = str(active)
+        snapshot = destination.parent / release["version"]
+        receipt_path = snapshot / "receipt.json"
+        previous_receipt = receipt_path.read_bytes() if receipt_path.is_file() else None
+        result = _admit(
+            current["repository_url"],
+            directory,
+            current["subdirectory"],
+            release["revision"],
+            vis_version,
+            release,
+            name,
+            active.name,
+        )
+        if spec is None:
+            return result
+        # Configuration pins remain authoritative on the next reconciliation.
+        if spec["revision"] is not None:
+            spec["revision"] = result["revision"]
+        elif spec["version"] not in (None, LATEST):
+            spec["version"] = result["version"]
+        try:
+            _save_sync_records(
+                directory,
+                {
+                    **records,
+                    name: {"spec": spec, "target": result["path"], "result": result},
+                },
+            )
+        except BaseException:
+            _restore_saved_link(directory, name, result["path"], previous_target)
+            if previous_receipt is not None:
+                receipt_path.write_bytes(previous_receipt)
+            raise
+        return result
+    finally:
+        _release(lock, fd)
+
+
 def update(
     source, directory, trust=False, version=None, vis_version=None, subdirectory=None
 ):
@@ -988,16 +1041,7 @@ def update(
         }
     if Version(release["version"]) == Version(current["version"]):
         raise ValueError("A published version cannot change its approved commit")
-    return _admit(
-        current["repository_url"],
-        directory,
-        current["subdirectory"],
-        release["revision"],
-        vis_version,
-        release,
-        name,
-        active.name,
-    )
+    return _replace_managed(directory, active, current, release, vis_version)
 
 
 def rollback(
@@ -1016,7 +1060,6 @@ def rollback(
         folder = None
     active, current = _installed_repository(directory, repository, folder)
     assert active is not None and current is not None  # A required lookup raises.
-    name = current["name"]
     if version is not None:
         release = _select(
             _releases(current["repository_url"], current["subdirectory"]), version
@@ -1031,24 +1074,15 @@ def rollback(
             raise ValueError(
                 "No previous installation; choose an approved older --version"
             )
-        release = _receipt(active.parent / _version(previous), name)
+        release = _receipt(active.parent / _version(previous), current["name"])
         if (
             release.get("repository_url") != current["repository_url"]
             or release.get("subdirectory") != current["subdirectory"]
-            or release.get("name") != name
+            or release.get("name") != current["name"]
             or not re.fullmatch(r"[0-9a-f]{40}", release.get("revision", ""))
         ):
             raise ValueError("Invalid previous package receipt")
-    return _admit(
-        current["repository_url"],
-        directory,
-        current["subdirectory"],
-        release["revision"],
-        vis_version,
-        release,
-        name,
-        active.name,
-    )
+    return _replace_managed(directory, active, current, release, vis_version)
 
 
 def _sync_spec(spec):

{vis_agent-0.2.37.dev7443 → vis_agent-0.2.38.dev7473}/tests/test_extension_package.py

@@ -1075,6 +1075,107 @@ def test_sync_pins_updates_refresh_and_rollback(releases):
     )
 
 
+def test_explicit_update_and_rollback_keep_sync_ownership(releases):
+    metadata, target, _ = releases
+    newer = metadata.pop(1)
+    configured = {
+        "vis-greeter": {"source": REPOSITORY, "subdirectory": "plugins/greeting"}
+    }
+    package.sync(configured, target, trust=True)
+    metadata.append(newer)
+    package.update(REPOSITORY, target, trust=True)
+    updated = package.sync(configured, target, trust=True)[0]
+    assert (updated["status"], updated.get("version")) == ("cached", "1.1.0")
+    package.rollback(REPOSITORY, target, trust=True)
+    restored = package.sync(configured, target, trust=True)[0]
+    assert (restored["status"], restored.get("version")) == ("cached", "1.0.0")
+
+
+@pytest.mark.parametrize("operation", ["update", "rollback"])
+def test_managed_activation_restores_ownership_when_sync_receipt_write_fails(
+    releases, monkeypatch, operation
+):
+    metadata, target, _ = releases
+    newer = metadata.pop(1)
+    configured = {
+        "vis-greeter": {"source": REPOSITORY, "subdirectory": "plugins/greeting"}
+    }
+    package.sync(configured, target, trust=True)
+    metadata.append(newer)
+    if operation == "rollback":
+        package.update(REPOSITORY, target, trust=True)
+    pointer = target / "vis-greeter/current"
+    previous_target = pointer.resolve()
+    previous_records = (target / ".sync.json").read_bytes()
+    receipt_path = target / "vis-greeter/1.0.0/receipt.json"
+    previous_receipt = receipt_path.read_bytes()
+
+    def fail(*_):
+        raise OSError("Fixture receipt write failure")
+
+    monkeypatch.setattr(package, "_save_sync_records", fail)
+    with pytest.raises(OSError, match="Fixture receipt write failure"):
+        getattr(package, operation)(REPOSITORY, target, trust=True)
+    assert pointer.resolve() == previous_target
+    assert (target / ".sync.json").read_bytes() == previous_records
+    assert receipt_path.read_bytes() == previous_receipt
+    assert package._sync_owned(pointer, package._sync_records(target)["vis-greeter"])
+    assert package.sync(configured, target, trust=True)[0]["status"] == "cached"
+
+
+@pytest.mark.parametrize("operation", ["update", "rollback"])
+def test_managed_activation_refuses_an_externally_changed_sync_pointer(
+    releases, operation
+):
+    metadata, target, _ = releases
+    newer = metadata.pop(1)
+    configured = {
+        "vis-greeter": {"source": REPOSITORY, "subdirectory": "plugins/greeting"}
+    }
+    package.sync(configured, target, trust=True)
+    metadata.append(newer)
+    package.update(REPOSITORY, target, trust=True)
+    pointer = target / "vis-greeter/current"
+    pointer.unlink()
+    pointer.symlink_to("1.0.0")
+    before = (target / ".sync.json").read_bytes()
+    kwargs = {"version": "1.1.0"} if operation == "update" else {}
+    with pytest.raises(ValueError):
+        getattr(package, operation)(REPOSITORY, target, trust=True, **kwargs)
+    assert pointer.resolve().name == "1.0.0"
+    assert (target / ".sync.json").read_bytes() == before
+
+
+@pytest.mark.parametrize("pin", ["version", "revision"])
+def test_explicit_update_does_not_override_a_configuration_pin(releases, pin):
+    metadata, target, _ = releases
+    spec = {"source": REPOSITORY, "subdirectory": "plugins/greeting"}
+    spec[pin] = metadata[0][pin]
+    configured = {"vis-greeter": spec}
+    package.sync(configured, target, trust=True)
+    package.update(REPOSITORY, target, trust=True)
+    restored = package.sync(configured, target, trust=True)[0]
+    assert (restored["status"], restored.get("version")) == ("updated", "1.0.0")
+
+
+def test_update_validates_sync_spec_before_changing_the_active_version(releases):
+    metadata, target, _ = releases
+    newer = metadata.pop(1)
+    configured = {
+        "vis-greeter": {"source": REPOSITORY, "subdirectory": "plugins/greeting"}
+    }
+    package.sync(configured, target, trust=True)
+    metadata.append(newer)
+    records = package._sync_records(target)
+    records["vis-greeter"]["spec"] = {}
+    package._save_sync_records(target, records)
+    before = (target / ".sync.json").read_bytes()
+    with pytest.raises(ValueError, match="source"):
+        package.update(REPOSITORY, target, trust=True)
+    assert (target / "vis-greeter/current").resolve().name == "1.0.0"
+    assert (target / ".sync.json").read_bytes() == before
+
+
 def test_sync_latest_tracks_approved_stable_releases(releases):
     metadata, target, commands = releases
     newer = metadata.pop(1)

{vis_agent-0.2.37.dev7443 → vis_agent-0.2.38.dev7473}/tests/test_goals.py

@@ -40,7 +40,7 @@ def test_goal_uses_slash_turn(budget):
     [
         ("", None),
         (" ", None),
-        ("x" * 8193, None),
+        ("x" * 44001, None),
         (None, None),
         ("work", 0),
         ("work", -1),