agentnode-sdk 0.11.2__tar.gz → 0.11.3__tar.gz

{agentnode_sdk-0.11.2 → agentnode_sdk-0.11.3}/CHANGELOG.md

@@ -1,5 +1,24 @@
 # Changelog
 
+## 0.11.3 — Test hygiene + multi-tool run guidance
+
+### Fixed
+
+- **Stale lock-integrity field-classification test.** `test_real_lockfile_fields_classified`
+  used the V1 `CANONICAL_FIELDS` set, so it flagged the V3 field `publisher_slug`
+  as "unclassified" on real lockfiles and failed. The integrity model itself
+  already seals `publisher_slug` (`CANONICAL_FIELDS_V3`); only the test was stale.
+  Updated to `CANONICAL_FIELDS_V3` — no change to the integrity model.
+
+### Changed
+
+- **Clearer `agentnode run <slug>` error for multi-tool packs.** When a package
+  exposes more than one tool and resolution fails without an explicit tool name,
+  the error now lists the available tools and points at
+  `agentnode run <slug>:<tool>` instead of a generic "no entrypoint" / "Function
+  'run' not found". Message-only — single-tool auto-select and multi-tool dispatch
+  behaviour are unchanged. Applied consistently to the host and container paths.
+
 ## 0.11.2 — CLI run resolution fixes
 
 ### Fixed

{agentnode_sdk-0.11.2 → agentnode_sdk-0.11.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentnode-sdk
-Version: 0.11.2
+Version: 0.11.3
 Summary: Python SDK for AgentNode — the open upgrade and discovery infrastructure for AI agents.
 Project-URL: Homepage, https://agentnode.net
 Project-URL: Repository, https://github.com/agentnode-ai/agentnode

{agentnode_sdk-0.11.2 → agentnode_sdk-0.11.3}/agentnode_sdk/__init__.py

@@ -38,7 +38,7 @@ from agentnode_sdk.runtime import AgentNodeRuntime
 Client = AgentNodeClient
 ToolError = AgentNodeToolError
 
-__version__ = "0.11.2"
+__version__ = "0.11.3"
 __all__ = [
     "AgentNode",
     "AsyncAgentNode",

{agentnode_sdk-0.11.2 → agentnode_sdk-0.11.3}/agentnode_sdk/installer.py

@@ -1051,6 +1051,20 @@ def _default_tool_entrypoint(entry: dict) -> str | None:
     return entry.get("entrypoint")
 
 
+def _multi_tool_hint(entry: dict, slug: str = "<slug>") -> str:
+    """Trailing hint for resolution-failure errors: if the pack exposes MORE than
+    one tool, point the user at ``agentnode run <slug>:<tool>`` and list the tools.
+    Empty string for single-tool / no-tools packs. Message-only — does not change
+    which entrypoint is chosen."""
+    names = [t.get("name") for t in (entry.get("tools") or []) if t.get("name")]
+    if len(names) > 1:
+        return (
+            f" This package exposes multiple tools: {names}. "
+            f"Run a specific one with: agentnode run {slug}:<tool>."
+        )
+    return ""
+
+
 def load_tool(slug: str, tool_name: str | None = None, *, _internal: bool = False) -> Any:
     """Load an installed package's tool function.
 
@@ -1120,7 +1134,10 @@ def load_tool(slug: str, tool_name: str | None = None, *, _internal: bool = Fals
     # else fall back to the package-level entrypoint (multi-tool unchanged).
     entrypoint = _default_tool_entrypoint(pkg)
     if not entrypoint:
-        raise ImportError(f"Package '{slug}' has no entrypoint in lockfile.")
+        raise ImportError(
+            f"Package '{slug}' has no entrypoint in lockfile."
+            + _multi_tool_hint(pkg, slug)
+        )
 
     module_path, func_name = _resolve_entrypoint(entrypoint)
     mod = _import_module(module_path, slug)
@@ -1129,6 +1146,7 @@ def load_tool(slug: str, tool_name: str | None = None, *, _internal: bool = Fals
         raise ImportError(
             f"Function '{func_name}' not found in module '{module_path}' "
             f"for package '{slug}'."
+            + _multi_tool_hint(pkg, slug)
         )
     return func
 

{agentnode_sdk-0.11.2 → agentnode_sdk-0.11.3}/agentnode_sdk/runtimes/python_runner.py

@@ -445,7 +445,9 @@ def _resolve_container_target(entry: dict, tool_name: str | None) -> tuple[str,
         first then the default function (``run``), matching load_tool's getattr order;
       - no tool_name → package entrypoint module + its function.
     """
-    from agentnode_sdk.installer import _resolve_entrypoint, _default_tool_entrypoint
+    from agentnode_sdk.installer import (
+        _resolve_entrypoint, _default_tool_entrypoint, _multi_tool_hint,
+    )
 
     tools = entry.get("tools") or []
     if tool_name:
@@ -466,7 +468,8 @@ def _resolve_container_target(entry: dict, tool_name: str | None) -> tuple[str,
     ep = _default_tool_entrypoint(entry)
     if not ep:
         raise AgentNodeToolError(
-            "Package has no entrypoint in the lockfile.", tool_name=None,
+            "Package has no entrypoint in the lockfile." + _multi_tool_hint(entry),
+            tool_name=None,
         )
     module, func = _resolve_entrypoint(ep)
     return module, [func]

{agentnode_sdk-0.11.2 → agentnode_sdk-0.11.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "agentnode-sdk"
-version = "0.11.2"
+version = "0.11.3"
 description = "Python SDK for AgentNode — the open upgrade and discovery infrastructure for AI agents."
 readme = "README.md"
 requires-python = ">=3.10"

{agentnode_sdk-0.11.2 → agentnode_sdk-0.11.3}/tests/test_lock_integrity.py

@@ -595,7 +595,14 @@ class TestFieldClassification:
     """Every field in a real lockfile entry must be either canonical,
     mutable, or the _integrity seal itself."""
 
-    KNOWN_FIELDS = set(CANONICAL_FIELDS) | set(MUTABLE_FIELDS) | {"_integrity", "_signatures"}
+    # V3 = canonical V1 + _signatures + publisher_slug (all integrity-sealed).
+    KNOWN_FIELDS = set(CANONICAL_FIELDS_V3) | set(MUTABLE_FIELDS) | {"_integrity"}
+
+    def test_known_fields_cover_v3(self):
+        """Regression: V3 canonical fields (incl. _signatures, publisher_slug)
+        must be classified, else a real lockfile carrying them trips the check."""
+        assert "publisher_slug" in self.KNOWN_FIELDS
+        assert "_signatures" in self.KNOWN_FIELDS
 
     def test_toolpack_entry_fields_classified(self):
         e = seal_entry(_make_entry())

{agentnode_sdk-0.11.2 → agentnode_sdk-0.11.3}/tests/test_toolpack_sandbox.py

@@ -354,6 +354,16 @@ def test_resolve_target_multi_tool_keeps_package_default():
     assert _resolve_container_target(entry, None) == ("mod.tool", ["dispatch"])
 
 
+def test_resolve_target_multi_tool_no_entrypoint_hints():
+    # 0.11.3: multi-tool pack with NO package-level entrypoint → helpful hint
+    # listing tools + slug:tool (message-only; container path mirrors the host).
+    from agentnode_sdk.exceptions import AgentNodeToolError
+    entry = {"tools": [{"name": "a", "entrypoint": "m:a"},
+                       {"name": "b", "entrypoint": "m:b"}]}
+    with pytest.raises(AgentNodeToolError, match="multiple tools"):
+        _resolve_container_target(entry, None)
+
+
 def test_resolve_target_v02_colon_function():
     entry = {"entrypoint": "mod.tool:describe", "tools": []}
     assert _resolve_container_target(entry, None) == ("mod.tool", ["describe"])

{agentnode_sdk-0.11.2 → agentnode_sdk-0.11.3}/tests/test_v02.py

@@ -111,6 +111,31 @@ class TestLoadTool:
                 func = load_tool("multi-pack")
                 assert func == mock_module.dispatch
 
+    def test_load_multi_tool_no_toolname_lists_tools(self, tmp_path):
+        """0.11.3: multi-tool pack run without a tool name and an unresolvable
+        package entrypoint → helpful error listing tools + slug:tool hint.
+        Message-only; dispatch behaviour unchanged."""
+        self._write_lockfile(tmp_path, {
+            "multi-pack": {
+                "version": "1.0.0",
+                "entrypoint": "multi.tool",  # colon-less → 'run', which won't exist
+                "tools": [
+                    {"name": "a", "entrypoint": "multi.tool:a"},
+                    {"name": "b", "entrypoint": "multi.tool:b"},
+                ],
+            }
+        })
+        mock_module = MagicMock(spec=["a", "b"])  # no 'run' attribute
+
+        with patch("agentnode_sdk.installer._lockfile_path", return_value=tmp_path / "agentnode.lock"):
+            with patch("agentnode_sdk.installer.importlib.import_module", return_value=mock_module):
+                with pytest.raises(ImportError) as ei:
+                    load_tool("multi-pack")
+        msg = str(ei.value)
+        assert "multiple tools" in msg
+        assert "'a'" in msg and "'b'" in msg
+        assert "multi-pack:<tool>" in msg
+
     def test_load_v02_with_tool_name(self, tmp_path):
         """v0.2 pack — load_tool(slug, tool_name) returns specific function."""
         self._write_lockfile(tmp_path, {