fix(installer): count verify progress stage

This commit is contained in:
Vincent Koc
2026-05-24 15:50:16 +02:00
parent 5bffd17e01
commit 102b2c18e9
3 changed files with 29 additions and 1 deletions
+1 -1
View File
@@ -7,7 +7,7 @@ Docs: https://docs.openclaw.ai
### Changes
### Fixes
- Installer: include the optional verify phase in the progress counter so `--verify` shows `[4/4] Verifying installation` instead of `[4/3]`.
- Tests: fail the kitchen-sink RPC Docker walk when gateway RSS sampling is unavailable instead of silently disabling the per-process memory guard.
- Tests: suppress the current Rolldown plugin timing warning format in the Vitest wrapper so tiny focused runs do not drown useful stderr in repeated build-timing noise.
- Models/OpenRouter: use endpoint-specific OpenRouter context limits from `top_provider` metadata so provider-routed models no longer overstate available context. (#85949) Thanks @TurboTheTurtle.
+9
View File
@@ -339,6 +339,14 @@ ui_error() {
INSTALL_STAGE_TOTAL=3
INSTALL_STAGE_CURRENT=0
configure_install_stage_total() {
INSTALL_STAGE_TOTAL=3
INSTALL_STAGE_CURRENT=0
if [[ "${VERIFY_INSTALL:-0}" == "1" ]]; then
INSTALL_STAGE_TOTAL=4
fi
}
ui_section() {
local title="$1"
if [[ -n "$GUM" ]]; then
@@ -3126,6 +3134,7 @@ main() {
if [[ "${OPENCLAW_INSTALL_SH_NO_RUN:-0}" != "1" ]]; then
parse_args "$@"
configure_install_stage_total
configure_verbose
main
fi
+19
View File
@@ -148,6 +148,25 @@ describe("install.sh", () => {
);
});
it("counts the verify stage when --verify is enabled", () => {
const result = runInstallShell(
[
`source ${JSON.stringify(SCRIPT_PATH)}`,
"parse_args --verify",
"configure_install_stage_total",
'ui_stage "Preparing environment"',
'ui_stage "Installing OpenClaw"',
'ui_stage "Finalizing setup"',
'ui_stage "Verifying installation"',
].join("\n"),
{ TERM: "dumb" },
);
expect(result.status).toBe(0);
expect(result.stdout).toContain("[4/4] Verifying installation");
expect(result.stdout).not.toContain("[4/3] Verifying installation");
});
it("loads nvm before checking Node.js so stale system Node does not win", () => {
expect(script).toMatch(
/# Step 2: Node\.js\s+load_nvm_for_node_detection\s+if ! check_node; then/,