fix(e2e): find start-openclaw.sh process correctly in log dump#190
Merged
andreasjansson merged 7 commits intomainfrom Feb 7, 2026
Merged
fix(e2e): find start-openclaw.sh process correctly in log dump#190andreasjansson merged 7 commits intomainfrom
andreasjansson merged 7 commits intomainfrom
Conversation
added 5 commits
February 6, 2026 22:03
Use jq to find the process with 'start-openclaw' in its command, instead of grabbing the first process (which was a mount check).
The cron sync was failing with 'no config file found' because getLogs() sometimes returned empty stdout due to a race between process completion and log flushing. Using exitCode from 'test -f' is more reliable than parsing stdout for 'ok'.
The cron job was racing with the startup script — it would check for the config file before openclaw onboard had created it. Now it checks if the gateway process is running first and skips the sync if not.
In CI the browser takes longer to start, causing setExtraHTTPHeaders to fail silently. The browser wasn't ready at 2s so the Access service token headers were never set, causing all requests to be redirected to the Cloudflare Access login page.
The playwright-cli 'open' command creates a new browser process, which loses the CF-Access-Client-Id/Secret headers set via setExtraHTTPHeaders in start-browser. Replaced all 'open' calls with 'run-code page.goto()' to navigate within the existing browser context. Also added IMPORTANT comment in start-browser explaining this constraint for future maintainers.
andreasjansson
force-pushed
the
fix-ci
branch
from
079d645 to
527d35e
Compare
…ertion cctr captures both stdout and stderr. Redirect all playwright output to stderr so it doesn't pollute the test output. Add a 1s sleep before echoing 'ready' to let stderr flush, and use 'endswith ready' assertion.
andreasjansson
force-pushed
the
fix-ci
branch
from
afe9be3 to
62fb7c6
Compare
E2E Test Recording (workers-ai)✅ Tests passed
|
E2E Test Recording (discord)✅ Tests passed
|
E2E Test Recording (base)✅ Tests passed
|
E2E Test Recording (telegram)✅ Tests passed
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Use jq to find the process with 'start-openclaw' in its command, instead of grabbing the first process (which was a mount check).