fix(acp): remove duplicate exe init and rename stale-kill config test

- Drop redundant cur_exe assignment in stale_kill_still_same_os_process.
- Rename test to match behavior (env applies when CLI not passed).

Made-with: Cursor

Author: Mateusz

src/connectors/acp_core/acp_subprocess_identity.py

@@ -114,7 +114,6 @@ def stale_kill_still_same_os_process(
         return False
 
     if identity.exe_key:
-        cur_exe = ""
         cur_exe = ""
         with contextlib.suppress(_psutil.AccessDenied, _psutil.Error):
             cur_exe = _normalize_exe_key(current.exe())

tests/unit/core/config/test_stale_acp_agent_kills_config.py

@@ -77,8 +77,8 @@ def test_load_config_merges_disable_stale_acp_agent_kills_from_env() -> None:
     assert cfg.disable_stale_acp_agent_kills is True
 
 
-def test_cli_overrides_env_false_stale_kills_enabled_via_cli() -> None:
-    """Env requests disable; CLI does not pass --disable -> env wins."""
+def test_env_disables_stale_kills_when_cli_not_passed() -> None:
+    """Env requests disable; CLI not used -> env applies."""
     merged = {"DISABLE_STALE_ACP_AGENT_KILLS": "true"}
     with patch.dict("os.environ", merged, clear=True):
         args = parse_cli_args([])