Skip to content

Commit 1f5eb21

Browse files
abrichrclaude
andcommitted
fix: add shell=True to launch handler for Windows app resolution
subprocess.Popen('calc') without shell=True fails on Windows because CreateProcessW doesn't search PATH the same way cmd.exe does. WAA's native /setup/launch endpoint hardcodes shell=True which is why it works. Match that behavior. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 1e7da2f commit 1f5eb21

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

  • openadapt_evals/adapters/waa

openadapt_evals/adapters/waa/live.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1618,7 +1618,7 @@ def _config_entry_to_command(entry: dict) -> str | None:
16181618
escaped = app.replace("\\", "\\\\").replace("'", "\\'")
16191619
return (
16201620
"import subprocess; "
1621-
f"subprocess.Popen('{escaped}')"
1621+
f"subprocess.Popen('{escaped}', shell=True)"
16221622
)
16231623

16241624
if entry_type == "open":

0 commit comments

Comments
 (0)