codeprobe 0.3.4__tar.gz → 0.3.5__tar.gz

{codeprobe-0.3.4 → codeprobe-0.3.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: codeprobe
-Version: 0.3.4
+Version: 0.3.5
 Summary: Benchmark AI coding agents against your own codebase. Mine real tasks from repo history, run agents, interpret results.
 Author: codeprobe contributors
 License-Expression: Apache-2.0

{codeprobe-0.3.4 → codeprobe-0.3.5}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "codeprobe"
-version = "0.3.4"
+version = "0.3.5"
 description = "Benchmark AI coding agents against your own codebase. Mine real tasks from repo history, run agents, interpret results."
 readme = "README.md"
 license = "Apache-2.0"

{codeprobe-0.3.4 → codeprobe-0.3.5}/src/codeprobe/adapters/_base.py

@@ -158,7 +158,7 @@ class BaseAdapter:
                 text=True,
                 timeout=config.timeout_seconds,
                 cwd=config.cwd,
-                env=_adapter_safe_env(session_env),
+                env=_adapter_safe_env(session_env) if session_env else None,
             )
         except subprocess.TimeoutExpired as exc:
             duration = time.monotonic() - start

{codeprobe-0.3.4 → codeprobe-0.3.5}/src/codeprobe.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: codeprobe
-Version: 0.3.4
+Version: 0.3.5
 Summary: Benchmark AI coding agents against your own codebase. Mine real tasks from repo history, run agents, interpret results.
 Author: codeprobe contributors
 License-Expression: Apache-2.0

{codeprobe-0.3.4 → codeprobe-0.3.5}/tests/test_adapters.py

@@ -274,9 +274,10 @@ class _StubAdapter(BaseAdapter):
 
 
 class TestBaseAdapterEnvWhitelist:
-    """Verify subprocess.run() uses a filtered environment, not full parent env."""
+    """Verify subprocess.run() env handling: inherit when no isolation, filter when isolated."""
 
-    def test_subprocess_receives_explicit_env(self) -> None:
+    def test_inherits_full_env_without_session_env(self) -> None:
+        """Without session isolation, subprocess inherits the full parent env."""
         adapter = _StubAdapter()
         config = AgentConfig(timeout_seconds=5)
         fake_result = subprocess.CompletedProcess(
@@ -286,22 +287,36 @@ class TestBaseAdapterEnvWhitelist:
             adapter.run("test", config)
 
         _, kwargs = mock_run.call_args
-        assert "env" in kwargs, "subprocess.run must receive explicit env"
-        env = kwargs["env"]
+        assert kwargs.get("env") is None, "env=None inherits parent process env"
+
+    def test_filters_env_with_session_env(self) -> None:
+        """With session isolation, subprocess gets a filtered env."""
+        adapter = _StubAdapter()
+        config = AgentConfig(timeout_seconds=5)
+        fake_result = subprocess.CompletedProcess(
+            args=["fake-agent"], returncode=0, stdout="ok", stderr=""
+        )
+        session_env = {"CLAUDE_CONFIG_DIR": "/tmp/test"}
+        with patch("subprocess.run", return_value=fake_result) as mock_run:
+            adapter.run("test", config, session_env=session_env)
+
+        env = mock_run.call_args[1]["env"]
         assert isinstance(env, dict)
+        assert env.get("CLAUDE_CONFIG_DIR") == "/tmp/test"
 
-    def test_env_includes_path_and_home(self) -> None:
+    def test_filtered_env_includes_path_and_home(self) -> None:
         adapter = _StubAdapter()
         config = AgentConfig(timeout_seconds=5)
         fake_result = subprocess.CompletedProcess(
             args=["fake-agent"], returncode=0, stdout="ok", stderr=""
         )
+        session_env = {"CLAUDE_CONFIG_DIR": "/tmp/test"}
         with patch("subprocess.run", return_value=fake_result) as mock_run:
             import os
 
             old_path = os.environ.get("PATH", "")
             old_home = os.environ.get("HOME", "")
-            adapter.run("test", config)
+            adapter.run("test", config, session_env=session_env)
 
         env = mock_run.call_args[1]["env"]
         if old_path:
@@ -309,18 +324,19 @@ class TestBaseAdapterEnvWhitelist:
         if old_home:
             assert env.get("HOME") == old_home
 
-    def test_env_excludes_random_secrets(self) -> None:
+    def test_filtered_env_excludes_random_secrets(self) -> None:
         adapter = _StubAdapter()
         config = AgentConfig(timeout_seconds=5)
         fake_result = subprocess.CompletedProcess(
             args=["fake-agent"], returncode=0, stdout="ok", stderr=""
         )
+        session_env = {"CLAUDE_CONFIG_DIR": "/tmp/test"}
         import os
 
         os.environ["MY_SUPER_SECRET_DB_PASSWORD"] = "hunter2"
         try:
             with patch("subprocess.run", return_value=fake_result) as mock_run:
-                adapter.run("test", config)
+                adapter.run("test", config, session_env=session_env)
             env = mock_run.call_args[1]["env"]
             assert "MY_SUPER_SECRET_DB_PASSWORD" not in env
         finally: