mirror of
https://github.com/NousResearch/hermes-agent.git
synced 2026-07-26 17:38:36 +00:00
fix(cron): respect the platform-conditional decode design in _run_job_script + taskkill kwarg snapshot
cron/scheduler.py deliberately applies utf-8/replace only on Windows via popen_kwargs (non-Windows keeps locale default per its test contract) — drop the sweep's unconditional inline kwargs there. Update the gateway force-kill kwarg snapshot for the new guard.
This commit is contained in:
parent
804ce793de
commit
0f732cb3d6
2 changed files with 3 additions and 1 deletions
|
|
@ -2301,7 +2301,7 @@ def _run_job_script(script_path: str) -> tuple[bool, str]:
|
|||
result = subprocess.run(
|
||||
argv,
|
||||
capture_output=True,
|
||||
text=True, encoding='utf-8', errors='replace',
|
||||
text=True,
|
||||
timeout=script_timeout,
|
||||
cwd=str(path.parent),
|
||||
env=env,
|
||||
|
|
|
|||
|
|
@ -508,6 +508,8 @@ def test_gateway_force_kill_hides_taskkill_window(monkeypatch):
|
|||
{
|
||||
"capture_output": True,
|
||||
"text": True,
|
||||
"encoding": "utf-8",
|
||||
"errors": "replace",
|
||||
"timeout": 10,
|
||||
"creationflags": _CREATE_NO_WINDOW,
|
||||
},
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue