-
-
Notifications
You must be signed in to change notification settings - Fork 162
Description
Context
Upstream commit 2dcb2449 ("refactor(test): dedupe gateway and web scaffolding") renamed server.canvas-auth.e2e.test.ts to server.canvas-auth.test.ts. The vitest base config excludes **/*.e2e.test.ts, so this test was never running in the gateway suite before the rename.
The test asserts that a scoped A2UI fetch returns HTTP 200, but that requires a2ui.bundle.js to exist on disk. Our gateway-tests-build.sh only does pnpm install — it doesn't run canvas:a2ui:bundle, so the test gets 503 ("A2UI assets not found").
Current workaround
PR #63 patches the test expectation from 200 to 503 in gateway-postpatch.sh to unblock CI.
Proposed fix
Expand gateway-tests-build.sh to run the canvas:a2ui:bundle step (with the rolldown PATH shim, native dep rebuild, etc.) so the test environment has the assets it needs. This would bring the test build closer to production parity.
Considerations
- Increases test build time and complexity
- Requires the same rolldown shim from
gateway-build.sh - Need
chmod -R u+w node_modulesandpnpm rebuildfor native deps - Alternative: keep the 503 patch if upstream changes their test expectations
Related
- Build fails for openclaw v2026.2.21+ — canvas:a2ui:bundle step fails in Nix sandbox #62 — Build fails for openclaw v2026.2.21+ (production build fix)
- PR Bump to OpenClaw v2026.2.25, fix rolldown sandbox shim, exclude deleted bird tool #63 — Fix for Build fails for openclaw v2026.2.21+ — canvas:a2ui:bundle step fails in Nix sandbox #62, includes the 503 workaround