diffgenome 0.1.0__tar.gz → 0.1.1__tar.gz

{diffgenome-0.1.0 → diffgenome-0.1.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: diffgenome
-Version: 0.1.0
+Version: 0.1.1
 Summary: Runtime evidence for code changes: run a repository's existing tests against a change and report what actually executed.
 Author: karims
 License-Expression: Apache-2.0
@@ -46,17 +46,20 @@ place an observed outcome on a changed line.
 pip install diffgenome
 ```
 
-Python 3.12+ (the Python tracer uses `sys.monitoring`). No runtime dependencies. Language
+Python 3.12+ (the Python tracer uses `sys.monitoring`; the target's own interpreter too). No runtime dependencies. Language
 support:
 
 | target | tests run with | needs |
 |---|---|---|
 | Python | pytest | the target's virtualenv (`--python`) |
-| Go | `go test` | a Go toolchain with the module's dependencies available offline |
+| Go | `go test` | a Go 1.22+ toolchain (it also builds the instrumenter; older modules are fine) with the module's dependencies downloaded (`go mod download`); tests run offline |
 | Node / TypeScript | Jest | the target's `node_modules` (TypeScript is taken from there, or from `DIFFGENOME_NODE_PATH`) |
 
 Tests run inside a disposable copy of the repository (never the checkout itself), under a
 sandbox with no network unless `--allow-loopback` is given for local test services.
+The sandbox in 0.1.0 is macOS `sandbox-exec`: on hosts without a supported sandbox (Linux
+today, including Linux CI runners) diffgenome refuses to run target tests and says so, so in
+CI use a macOS runner.
 
 ## Use
 

{diffgenome-0.1.0 → diffgenome-0.1.1}/README.md

@@ -25,17 +25,20 @@ place an observed outcome on a changed line.
 pip install diffgenome
 ```
 
-Python 3.12+ (the Python tracer uses `sys.monitoring`). No runtime dependencies. Language
+Python 3.12+ (the Python tracer uses `sys.monitoring`; the target's own interpreter too). No runtime dependencies. Language
 support:
 
 | target | tests run with | needs |
 |---|---|---|
 | Python | pytest | the target's virtualenv (`--python`) |
-| Go | `go test` | a Go toolchain with the module's dependencies available offline |
+| Go | `go test` | a Go 1.22+ toolchain (it also builds the instrumenter; older modules are fine) with the module's dependencies downloaded (`go mod download`); tests run offline |
 | Node / TypeScript | Jest | the target's `node_modules` (TypeScript is taken from there, or from `DIFFGENOME_NODE_PATH`) |
 
 Tests run inside a disposable copy of the repository (never the checkout itself), under a
 sandbox with no network unless `--allow-loopback` is given for local test services.
+The sandbox in 0.1.0 is macOS `sandbox-exec`: on hosts without a supported sandbox (Linux
+today, including Linux CI runners) diffgenome refuses to run target tests and says so, so in
+CI use a macOS runner.
 
 ## Use
 

{diffgenome-0.1.0 → diffgenome-0.1.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "diffgenome"
-version = "0.1.0"
+version = "0.1.1"
 description = "Runtime evidence for code changes: run a repository's existing tests against a change and report what actually executed."
 readme = "README.md"
 # 3.12+ is required: the Python tracer is built on sys.monitoring (PEP 669).

{diffgenome-0.1.0 → diffgenome-0.1.1}/src/diffgenome/__init__.py

@@ -4,4 +4,4 @@ Runs a repository's existing tests against a change and reports what actually ex
 `runtime` section (`diffgenome-runtime/1`) of a `diffgenome-change/1` artifact.
 """
 
-__version__ = "0.1.0"
+__version__ = "0.1.1"

{diffgenome-0.1.0 → diffgenome-0.1.1}/src/diffgenome/collect/py_runtime.py

@@ -3,6 +3,7 @@
 from __future__ import annotations
 
 import ast
+import shutil
 from pathlib import Path
 
 from diffgenome.collect.py_symbols import PythonSymbolIndex
@@ -12,7 +13,23 @@ from diffgenome.runtime import SymbolIndex
 from diffgenome.sandbox import Workspace
 from diffgenome.serialize import execution_from_json
 
-DIFFGENOME_SRC = Path(__file__).resolve().parents[2]
+DIFFGENOME_PACKAGE = Path(__file__).resolve().parents[1]
+
+
+def isolated_import_root(ws_root: Path) -> Path:
+    """A directory holding only the `diffgenome` package, for the target's PYTHONPATH.
+
+    The package's parent is `src/` in a checkout but the whole host `site-packages` when
+    installed; putting that on the target's PYTHONPATH would shadow the target's own
+    packages (pytest, pydantic, ...) with the host's, built for another interpreter.
+    """
+    root = ws_root / "diffgenome-import"
+    if not (root / "diffgenome").is_dir():
+        shutil.copytree(
+            DIFFGENOME_PACKAGE, root / "diffgenome",
+            ignore=shutil.ignore_patterns("__pycache__", "_collectors", "node_modules"),
+        )
+    return root
 
 
 class PytestRuntime:
@@ -69,7 +86,8 @@ class PytestRuntime:
             *(only if only else [self.tests]),
         ]  # fmt: skip
         roots = [str(ws.repo / p) for p in self.pythonpath]
-        env = {**self.test_env, "PYTHONPATH": ":".join([*roots, str(DIFFGENOME_SRC)])}
+        roots.append(str(isolated_import_root(ws.root)))
+        env = {**self.test_env, "PYTHONPATH": ":".join(roots)}
         result = ws.run(argv, env=env, timeout=1800, allow_loopback=self.allow_loopback)
         out_dir.mkdir(parents=True, exist_ok=True)
         copy_probe_traces(traces_ws, out_dir)

{diffgenome-0.1.0 → diffgenome-0.1.1}/tests/test_packaging.py

@@ -22,3 +22,15 @@ def test_collectors_ship_inside_the_package() -> None:
     root = Path(diffgenome.__file__).parent / "_collectors"
     assert (root / "go" / "instrument" / "main.go").is_file()
     assert (root / "node" / "instrument.js").is_file()
+
+
+def test_target_pythonpath_exposes_only_the_diffgenome_package(tmp_path: Path) -> None:
+    """Installed, the package's parent is the host site-packages: putting it on the target's
+    PYTHONPATH shadowed the target's pytest/pydantic with the host's (Baserow, 0.1.0)."""
+    from diffgenome.collect.py_runtime import isolated_import_root
+
+    root = isolated_import_root(tmp_path)
+    assert [p.name for p in root.iterdir()] == ["diffgenome"]
+    assert (root / "diffgenome" / "collect" / "pytest_plugin.py").is_file()
+    assert not (root / "diffgenome" / "_collectors").exists()
+    assert root.resolve() != Path(diffgenome.__file__).resolve().parents[1]