test: clarify delegated Testbox proof status · openclaw/openclaw@461e551
shakkernerd
·
2026-06-28
·
via Recent Commits to openclaw:main
| Original file line number | Diff line number | Diff line change |
|---|
@@ -651,7 +651,16 @@ pnpm crabbox:run -- --provider blacksmith-testbox \
|
651 | 651 | "corepack pnpm test" |
652 | 652 | ``` |
653 | 653 | |
654 | | -Read the final JSON summary. The useful fields are `provider`, `leaseId`, `syncDelegated`, `exitCode`, `commandMs`, and `totalMs`. One-shot Blacksmith-backed Crabbox runs should stop the Testbox automatically; if a run is interrupted or cleanup is unclear, inspect live boxes and stop only the boxes you created: |
| 654 | +Read the final JSON summary. The useful fields are `provider`, `leaseId`, |
| 655 | +`syncDelegated`, `exitCode`, `commandMs`, and `totalMs`. For delegated |
| 656 | +Blacksmith Testbox runs, the Crabbox wrapper exit code and JSON summary are the |
| 657 | +command result. The linked GitHub Actions run owns hydration and keepalive; it |
| 658 | +can finish as `cancelled` when the Testbox is stopped externally after the SSH |
| 659 | +command has already returned. Treat that as a cleanup/status artifact unless |
| 660 | +the wrapper `exitCode` is non-zero or the command output shows a failed test. |
| 661 | +One-shot Blacksmith-backed Crabbox runs should stop the Testbox automatically; |
| 662 | +if a run is interrupted or cleanup is unclear, inspect live boxes and stop only |
| 663 | +the boxes you created: |
655 | 664 | |
656 | 665 | ```bash |
657 | 666 | blacksmith testbox list --all |
|
| Original file line number | Diff line number | Diff line change |
|---|
@@ -20,6 +20,7 @@ title: "Tests"
|
20 | 20 | - `pnpm changed:lanes`: shows the architectural lanes triggered by the diff against `origin/main`. |
21 | 21 | - `pnpm check:changed`: delegates to Crabbox/Testbox by default outside CI, then runs the smart changed check gate for the diff against `origin/main` inside the remote child. It runs typecheck, lint, and guard commands for the affected architectural lanes, but does not run Vitest tests. Use `pnpm test:changed` or explicit `pnpm test <target>` for test proof. |
22 | 22 | - Codex worktrees and linked/sparse checkouts: avoid direct local `pnpm test*`, `pnpm check*`, and `pnpm crabbox:run` unless you have verified pnpm will not reconcile dependencies. For tiny explicit-file proof use `node scripts/run-vitest.mjs <path-or-filter>`; for changed gates or broad proof use `node scripts/crabbox-wrapper.mjs run --provider blacksmith-testbox ... -- env OPENCLAW_CHECK_CHANGED_REMOTE_CHILD=1 OPENCLAW_CHANGED_LANES_RAW_SYNC=1 corepack pnpm check:changed` so pnpm runs inside Testbox. |
| 23 | +- Testbox-through-Crabbox proof: use the wrapper's final `exitCode` and timing JSON as the command result. The delegated Blacksmith GitHub Actions run may show `cancelled` after a successful SSH command because the Testbox is stopped from outside the keepalive action; verify the wrapper summary and command output before treating that as a test failure. |
23 | 24 | - `OPENCLAW_HEAVY_CHECK_LOCK_SCOPE=worktree <local-heavy-check command>`: keeps heavy-check serialization inside the current worktree instead of the Git common dir for commands such as `pnpm check:changed` and targeted `pnpm test ...`. Use it only on high-capacity local hosts when you intentionally run independent checks across linked worktrees. |
24 | 25 | - `pnpm test`: routes explicit file/directory targets through scoped Vitest lanes. Untargeted runs are full-suite proof: they use fixed shard groups, expand to leaf configs for local parallel execution, and print the expected local shard fanout before starting. The extension group always expands to the per-extension shard configs instead of one giant root-project process. |
25 | 26 | - Test wrapper runs end with a short `[test] passed|failed|skipped ... in ...` summary. Vitest's own duration line stays the per-shard detail. |
|
此内容由惯性聚合(RSS阅读器)自动聚合整理,仅供阅读参考。 原文来自 — 版权归原作者所有。