codeprobe 0.3.3__tar.gz → 0.3.5__tar.gz

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: codeprobe
-Version: 0.3.3
+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.3 → codeprobe-0.3.5}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "codeprobe"
-version = "0.3.3"
+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.3 → 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.3 → codeprobe-0.3.5}/src/codeprobe/adapters/claude.py

@@ -78,22 +78,32 @@ class ClaudeAdapter(BaseAdapter):
         """Return a per-slot CLAUDE_CONFIG_DIR for session isolation.
 
         Copies authentication credentials from the real ``~/.claude/``
-        directory so the agent subprocess can authenticate.
+        directory so the agent subprocess can authenticate.  If no credential
+        files are found (e.g. auth lives in the system keychain), skips
+        isolation and lets the agent use the real config dir.
         """
+        real_config = Path.home() / ".claude"
+        _CRED_NAMES = ("credentials.json", ".credentials.json")
+
+        # Check whether any copyable credential files exist.
+        has_creds = real_config.is_dir() and any(
+            (real_config / name).is_file() for name in _CRED_NAMES
+        )
+        if not has_creds:
+            # No file-based credentials found — don't override config dir
+            # so the CLI can use keychain / default auth.
+            return {}
+
         config_dir = (
             Path(tempfile.gettempdir()) / "codeprobe-claude" / f"slot-{slot_id}"
         )
         config_dir.mkdir(parents=True, exist_ok=True)
 
-        # Copy auth credentials from the user's real config dir.
-        # Without these the subprocess gets "Not logged in".
-        real_config = Path.home() / ".claude"
-        if real_config.is_dir():
-            for name in ("credentials.json", ".credentials.json"):
-                src = real_config / name
-                dst = config_dir / name
-                if src.is_file():
-                    shutil.copy2(src, dst)
+        for name in _CRED_NAMES:
+            src = real_config / name
+            dst = config_dir / name
+            if src.is_file():
+                shutil.copy2(src, dst)
 
         return {"CLAUDE_CONFIG_DIR": str(config_dir)}
 

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: codeprobe
-Version: 0.3.3
+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.3 → 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:
@@ -1146,18 +1162,35 @@ class TestClaudeMcpConfig:
 
 
 class TestIsolateSession:
-    def test_claude_isolate_session_returns_config_dir(self) -> None:
+    def test_claude_isolate_session_returns_config_dir(self, tmp_path: Path) -> None:
         adapter = ClaudeAdapter()
-        env = adapter.isolate_session(0)
+        fake_home = tmp_path / "home"
+        (fake_home / ".claude").mkdir(parents=True)
+        (fake_home / ".claude" / ".credentials.json").write_text("{}")
+        with patch.object(Path, "home", return_value=fake_home):
+            env = adapter.isolate_session(0)
         assert "CLAUDE_CONFIG_DIR" in env
         assert "slot-0" in env["CLAUDE_CONFIG_DIR"]
 
-    def test_claude_isolate_session_different_slots(self) -> None:
+    def test_claude_isolate_session_different_slots(self, tmp_path: Path) -> None:
         adapter = ClaudeAdapter()
-        env0 = adapter.isolate_session(0)
-        env1 = adapter.isolate_session(1)
+        fake_home = tmp_path / "home"
+        (fake_home / ".claude").mkdir(parents=True)
+        (fake_home / ".claude" / ".credentials.json").write_text("{}")
+        with patch.object(Path, "home", return_value=fake_home):
+            env0 = adapter.isolate_session(0)
+            env1 = adapter.isolate_session(1)
         assert env0["CLAUDE_CONFIG_DIR"] != env1["CLAUDE_CONFIG_DIR"]
 
+    def test_claude_isolate_session_skips_when_no_creds(self, tmp_path: Path) -> None:
+        """When no credential files exist, returns empty dict."""
+        adapter = ClaudeAdapter()
+        fake_home = tmp_path / "home"
+        (fake_home / ".claude").mkdir(parents=True)
+        with patch.object(Path, "home", return_value=fake_home):
+            env = adapter.isolate_session(0)
+        assert env == {}
+
     def test_base_adapter_isolate_session_returns_empty(self) -> None:
         adapter = _StubAdapter()
         env = adapter.isolate_session(42)