fix(helpers.py): use keyword args for ensure_daemon/restart_daemon

[sduke1848]

Bug

The wrapper functions ensure_daemon() and restart_daemon() in helpers.py pass the BU_NAME value as a positional argument to admin.ensure_daemon(wait, name, env):

return _ensure(os.environ.get("BU_NAME") or "default")

Since wait is the first parameter (type float), passing a string name causes:

TypeError: unsupported operand type(s) for +: 'float' and 'str'
  File "admin.py", line 81, in ensure_daemon
    deadline = time.time() + wait

Fix

Pass name= as a keyword argument so it lands in the correct parameter:

return _ensure(name=os.environ.get("BU_NAME") or "default")

Also adds start_remote_daemon() and stop_remote_daemon() wrappers for completeness.

Testing

After fix:

$ BROWSER_USE_API_KEY=*** browser-harness -c "print(page_info())"
{'url': 'about:blank', 'title': ''}

---

Summary by cubic

Fixes a crash when starting or restarting the daemon by passing the name keyword instead of a positional arg. Also adds wrappers to start and stop a remote daemon.

• Bug Fixes

  • ensure_daemon() and restart_daemon() now pass name=... to admin.*, preventing a TypeError in admin.ensure_daemon(wait, name, env).

• New Features

  • Added start_remote_daemon(name=None, **kwargs) and stop_remote_daemon(name=None) helpers.

^{Written for commit 42a4d07. Summary will update on new commits.}

[cubic-dev-ai]

No issues found across 1 file

_{Re-trigger cubic}