Skip to content

ci: support native Windows Crabbox daemon hydration#87175

Merged
steipete merged 13 commits into
mainfrom
fix/windows-crabbox-hydrate
May 27, 2026
Merged

ci: support native Windows Crabbox daemon hydration#87175
steipete merged 13 commits into
mainfrom
fix/windows-crabbox-hydrate

Conversation

@steipete
Copy link
Copy Markdown
Contributor

Summary

  • Keep the default Crabbox hydrate job compatible with local Actions replay and custom marker jobs.
  • Add a separate hydrate-windows-daemon GitHub-runner job for native Windows daemon proof.
  • Persist Node/pnpm/PATH handoff files for native Windows and write GitHub command files as UTF-8.
  • Avoid forcing pnpm copy imports during Windows hydrate so the final native Windows hydrate completes in 8m instead of the prior 17m run.

Verification

  • pnpm test test/scripts/package-acceptance-workflow.test.ts
  • node -e "const fs=require('fs'); const yaml=require('yaml'); for (const file of ['.github/workflows/crabbox-hydrate.yml','.crabbox.yaml']) yaml.parse(fs.readFileSync(file,'utf8'));"
  • git diff --check
  • /Users/steipete/Projects/agent-scripts/skills/autoreview/scripts/autoreview --mode branch --base origin/main --prompt-file /tmp/openclaw-autoreview-context.md
  • Crabbox producer fix landed first: fix: stop native Windows hydrate runners crabbox#172, commit 68c60db, so native Windows crabbox stop writes the hydrate .stop marker.

Real behavior proof
Behavior addressed: native Windows Crabbox GitHub-runner hydration for focused OpenClaw daemon E2E proof.
Real environment tested: AWS Crabbox native Windows lease cbx_d5ac8185713b (windows-hydrate-tip), instance m7i.4xlarge, OpenClaw branch fix/windows-crabbox-hydrate at 7bfe3d4b.
Exact steps or command run after this patch: go run ./cmd/crabbox actions hydrate --provider aws --target windows --windows-mode normal --id windows-hydrate-tip --repo openclaw/openclaw --workflow .github/workflows/crabbox-hydrate.yml --ref fix/windows-crabbox-hydrate --job hydrate-windows-daemon --github-runner --keep-alive-minutes 30 --wait-timeout 60m -f ref=refs/heads/fix/windows-crabbox-hydrate --timing-json.
Evidence after fix: Actions run https://github.com/openclaw/openclaw/actions/runs/26492495359 completed hydration; Crabbox reported actions hydrate complete total=8m0.804s with actionsRunUrl set to that run.
Observed result after fix: follow-up crabbox run --no-sync reused workspace=actions-hydrated and printed pnpm 11.2.2, node v24.16.0, 7bfe3d4b, fix/windows-crabbox-hydrate; run id run_71f59c1fcbee.
What was not tested: broad OpenClaw CI on this branch; this PR only changes the hydrate workflow/test surface.

PR 85597 proof

  • Native Windows hydrated workspace ran pnpm test src/daemon/schtasks.startup-fallback.test.ts src/daemon/schtasks.stop.test.ts against PR 85597 at ccf3d0481c.
  • Result: 33 passed, 1 failed in src/daemon/schtasks.startup-fallback.test.ts:652, expected unknown, received stopped; Crabbox run run_360d3fd5200e.

@openclaw-barnacle openclaw-barnacle Bot added size: M maintainer Maintainer-authored PR labels May 27, 2026
@steipete steipete merged commit 95c8fc9 into main May 27, 2026
97 of 101 checks passed
@steipete steipete deleted the fix/windows-crabbox-hydrate branch May 27, 2026 05:32
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request May 27, 2026
* ci: support native Windows Crabbox hydration

* ci: use Windows PowerShell for Crabbox hydrate

* ci: reduce Windows pnpm install handle pressure

* ci: narrow native Windows hydrate install scope

* ci: keep Windows hydrate workspace-complete

* ci: persist Windows hydrate tool paths

* ci: split native Windows Crabbox hydration job

* ci: scope native Windows hydrate to daemon proof

* ci: use PowerShell for Windows hydrate fetch

* ci: write Windows hydrate command files as UTF-8

* ci: document Windows hydrate runner mode

* ci: preserve custom Crabbox hydrate markers

* ci: avoid forced copy imports in Windows hydrate
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer Maintainer-authored PR size: M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant