langbot-plugin 0.7.17__tar.gz → 0.7.18__tar.gz

{langbot_plugin-0.7.17 → langbot_plugin-0.7.18}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: langbot-plugin
-Version: 0.7.17
+Version: 0.7.18
 Summary: This package contains the SDK, CLI for building plugins for LangBot, plus the runtime for hosting LangBot plugins
 Project-URL: Homepage, https://langbot.app
 Project-URL: Repository, https://github.com/langbot-app/langbot-plugin-sdk

{langbot_plugin-0.7.17 → langbot_plugin-0.7.18}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "langbot-plugin"
-version = "0.7.17"
+version = "0.7.18"
 description = "This package contains the SDK, CLI for building plugins for LangBot, plus the runtime for hosting LangBot plugins"
 readme = "README.md"
 authors = [

{langbot_plugin-0.7.17 → langbot_plugin-0.7.18}/src/langbot_plugin/api/definition/components/manifest.py

@@ -258,8 +258,10 @@ class ComponentManifest(pydantic.BaseModel):
         if len(portable.split("/", 1)[0]) >= 2 and portable.split("/", 1)[0][1] == ":":
             raise ValueError(f"execution.python.path uses a drive letter: {declared}")
 
-        plugin_dir = os.path.abspath(self._rel_dir or ".")
-        target = os.path.abspath(os.path.join(plugin_dir, portable))
+        # Workers run at the plugin root; _rel_dir is the manifest's directory.
+        # Keep that directory in the module name so relative imports still work.
+        plugin_dir = os.path.abspath(".")
+        target = os.path.abspath(os.path.join(plugin_dir, self._rel_dir, portable))
         if os.path.commonpath([plugin_dir, target]) != plugin_dir:
             raise ValueError(
                 f"execution.python.path escapes the plugin directory: {declared}"

langbot_plugin-0.7.18/src/langbot_plugin/version.py

@@ -0,0 +1 @@
+__version__ = "0.7.18"

{langbot_plugin-0.7.17 → langbot_plugin-0.7.18}/tests/api/definition/test_manifest_component_path.py

@@ -14,8 +14,9 @@ from langbot_plugin.api.definition.components.manifest import ComponentManifest
 
 
 @pytest.fixture(autouse=True)
-def _clean_component_modules():
+def _clean_component_modules(tmp_path, monkeypatch):
     """Drop cached component modules so each test resolves its own plugin dir."""
+    monkeypatch.chdir(tmp_path)
     before = {
         name
         for name in sys.modules
@@ -140,3 +141,43 @@ def test_importing_a_missing_component_leaves_sys_path_clean(tmp_path, monkeypat
 
     assert str(plugin) not in sys.path
     assert len(sys.path) == len(before)
+
+
+@pytest.mark.parametrize("packages", [True, False])
+def test_nested_component_preserves_package_and_relative_imports(tmp_path, packages):
+    files = {
+        "components/knowledge_engine/parser.py": 'VALUE = "parsed"',
+        "components/knowledge_engine/engine.py": """
+            from .parser import VALUE
+
+            class Component:
+                marker = VALUE
+
+                def lazy_import(self):
+                    from .parser import VALUE
+                    return VALUE
+        """,
+    }
+    if packages:
+        files.update(
+            {
+                "components/__init__.py": "",
+                "components/knowledge_engine/__init__.py": "",
+            }
+        )
+    _plugin(tmp_path, files)
+    manifest = _manifest("components/knowledge_engine/engine.yaml", "engine.py")
+    before = list(sys.path)
+
+    component = manifest.get_python_component_class()
+
+    assert component.__module__ == "components.knowledge_engine.engine"
+    assert component.marker == "parsed"
+    assert component().lazy_import() == "parsed"
+    assert sys.path == before
+
+
+def test_manifest_outside_plugin_root_is_rejected(tmp_path):
+    manifest = _manifest(str(tmp_path.parent / "outside.yaml"), "outside.py")
+    with pytest.raises(ValueError, match="escapes the plugin directory"):
+        manifest.get_python_component_class()

langbot_plugin-0.7.17/src/langbot_plugin/version.py

@@ -1 +0,0 @@
-__version__ = "0.7.17"