Skip to content

Fix process wait timeout and bind mode; add tests#151

Closed
nicolaspiet wants to merge 3 commits into
cloudflare:mainfrom
nicolaspiet:fix/waitforprocess-bind-mode-tests
Closed

Fix process wait timeout and bind mode; add tests#151
nicolaspiet wants to merge 3 commits into
cloudflare:mainfrom
nicolaspiet:fix/waitforprocess-bind-mode-tests

Conversation

@nicolaspiet

Copy link
Copy Markdown

Problem

  • waitForProcess could return while a process was still starting or running, causing flaky admin CLI behavior.
  • CLAWDBOT_BIND_MODE was ignored because the startup script hard-coded lan.

Fix

  • waitForProcess now waits through starting and throws on timeout.
  • start-moltbot.sh uses CLAWDBOT_BIND_MODE with default lan.

Tests

  • npm test

@elithrar

elithrar commented Feb 3, 2026

Copy link
Copy Markdown
Contributor

Good catch! Waiting on CI.

@andreasjansson

Copy link
Copy Markdown
Member

Superseded by subsequent changes. The waitForProcess fix was addressed in PR #235 (merged Feb 11), the bind mode is now hardcoded to lan (PR #186), and the entire process management approach has been overhauled in PRs #336/#337 (merged Mar 27-28). This PR was based on the old clawdbot architecture. Thank you for the contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants