browser-use--browser-harness
Alezander9
51d62fbbe7
When BH_TMP_DIR is set, the caller owns the directory and is expected to give each daemon its own — so the bu-<NAME> filename prefix is redundant. Dropping it cuts ~30 chars off the AF_UNIX path on POSIX, which matters because sun_path is 104 bytes on macOS / 108 on Linux and a long BU_NAME (e.g. an opencode-style session id) can blow past it. Concretely the failing path on macOS for a 30-char session id under ~/.local/share/bcode/sessions/<sessionID>/bu-<sessionID>.sock was 117 bytes; with this change it becomes ~83 bytes. When BH_TMP_DIR is unset, _TMP is the shared default (/tmp on POSIX, gettempdir() on Windows) and the bu-<NAME> prefix is kept to disambiguate multiple daemons sharing the dir. Backward compatible by default. admin._daemon_endpoint_names: when BH_TMP_DIR is set, the dir is per-daemon by construction, so glob discovery doesn't apply — just check whether our local endpoint exists. Without BH_TMP_DIR, fall back to the existing shared-dir glob. Validation (_check on BU_NAME) still runs in both modes to catch garbage names early.