From c16f09a010754f01864c5a5673c1119ff628eda0 Mon Sep 17 00:00:00 2001 From: A <258483684+la14-1@users.noreply.github.com> Date: Wed, 11 Feb 2026 14:04:14 -0800 Subject: [PATCH] fix: use log_step for progress messages in recent scripts (#524) Replace log_warn (yellow) with log_step (cyan) for progress messages like "Installing...", "Setting up environment variables...", and "Starting..." in recently-added scripts. Yellow warnings should be reserved for actual warnings, not normal progress output. Also fix bare `clear` calls to use `clear 2>/dev/null || true` for robustness on minimal terminals, and improve the misleading "Appending environment variables to ~/.zshrc..." message in local scripts to the standard "Setting up environment variables..." phrasing. Files: local/gptme.sh, local/aider.sh, ramnode/openclaw.sh, ramnode/gptme.sh, netcup/gptme.sh Agent: ux-engineer Co-authored-by: A <6723574+louisgv@users.noreply.github.com> Co-authored-by: Claude Opus 4.6 (1M context) --- local/aider.sh | 8 ++++---- local/gptme.sh | 8 ++++---- netcup/gptme.sh | 2 +- ramnode/gptme.sh | 8 ++++---- ramnode/openclaw.sh | 6 +++--- 5 files changed, 16 insertions(+), 16 deletions(-) diff --git a/local/aider.sh b/local/aider.sh index b7a7eeb8..b20f3403 100644 --- a/local/aider.sh +++ b/local/aider.sh @@ -19,7 +19,7 @@ ensure_local_ready if command -v aider &>/dev/null; then log_info "Aider already installed" else - log_warn "Installing Aider..." + log_step "Installing Aider..." pip install aider-chat 2>/dev/null || pip3 install aider-chat fi @@ -44,7 +44,7 @@ fi MODEL_ID=$(get_model_id_interactive "openrouter/auto" "Aider") || exit 1 # 5. Inject environment variables -log_warn "Appending environment variables to ~/.zshrc..." +log_step "Setting up environment variables..." inject_env_vars_local upload_file run_server \ "OPENROUTER_API_KEY=${OPENROUTER_API_KEY}" @@ -54,12 +54,12 @@ echo "" # 6. Start Aider if [[ -n "${SPAWN_PROMPT:-}" ]]; then - log_warn "Executing Aider with prompt..." + log_step "Executing Aider with prompt..." source ~/.zshrc 2>/dev/null || true escaped_prompt=$(printf '%q' "${SPAWN_PROMPT}") aider --model "openrouter/${MODEL_ID}" -m "${escaped_prompt}" else - log_warn "Starting Aider..." + log_step "Starting Aider..." sleep 1 clear 2>/dev/null || true source ~/.zshrc 2>/dev/null || true diff --git a/local/gptme.sh b/local/gptme.sh index 3ce8db9c..8aae7816 100644 --- a/local/gptme.sh +++ b/local/gptme.sh @@ -19,7 +19,7 @@ ensure_local_ready if command -v gptme &>/dev/null; then log_info "gptme already installed" else - log_warn "Installing gptme..." + log_step "Installing gptme..." pip install gptme 2>/dev/null || pip3 install gptme fi @@ -44,7 +44,7 @@ fi MODEL_ID=$(get_model_id_interactive "openrouter/auto" "gptme") || exit 1 # 4. Inject environment variables -log_warn "Appending environment variables to ~/.zshrc..." +log_step "Setting up environment variables..." inject_env_vars_local upload_file run_server \ "OPENROUTER_API_KEY=${OPENROUTER_API_KEY}" @@ -54,12 +54,12 @@ echo "" # 5. Start gptme if [[ -n "${SPAWN_PROMPT:-}" ]]; then - log_warn "Executing gptme with prompt..." + log_step "Executing gptme with prompt..." export PATH="${HOME}/.local/bin:${PATH}" source ~/.zshrc 2>/dev/null || true gptme -m "openrouter/${MODEL_ID}" "${SPAWN_PROMPT}" else - log_warn "Starting gptme..." + log_step "Starting gptme..." sleep 1 clear 2>/dev/null || true export PATH="${HOME}/.local/bin:${PATH}" diff --git a/netcup/gptme.sh b/netcup/gptme.sh index 57e69b71..dad22092 100644 --- a/netcup/gptme.sh +++ b/netcup/gptme.sh @@ -62,5 +62,5 @@ echo "" # 7. Start gptme interactively log_step "Starting gptme..." sleep 1 -clear +clear 2>/dev/null || true interactive_session "${NETCUP_SERVER_IP}" "source ~/.zshrc && gptme -m openrouter/${MODEL_ID}" diff --git a/ramnode/gptme.sh b/ramnode/gptme.sh index 0982348f..4045f180 100644 --- a/ramnode/gptme.sh +++ b/ramnode/gptme.sh @@ -27,7 +27,7 @@ verify_server_connectivity "$RAMNODE_SERVER_IP" wait_for_cloud_init "$RAMNODE_SERVER_IP" # 5. Install gptme -log_warn "Installing gptme..." +log_step "Installing gptme..." run_server "$RAMNODE_SERVER_IP" "pip install gptme 2>/dev/null || pip3 install gptme" # Verify installation succeeded @@ -49,7 +49,7 @@ fi # Get model preference MODEL_ID=$(get_model_id_interactive "openrouter/auto" "gptme") || exit 1 -log_warn "Setting up environment variables..." +log_step "Setting up environment variables..." inject_env_vars_ssh "$RAMNODE_SERVER_IP" upload_file run_server \ "OPENROUTER_API_KEY=$OPENROUTER_API_KEY" @@ -59,7 +59,7 @@ log_info "Server: $SERVER_NAME (ID: $RAMNODE_SERVER_ID, IP: $RAMNODE_SERVER_IP)" echo "" # 7. Start gptme interactively -log_warn "Starting gptme..." +log_step "Starting gptme..." sleep 1 -clear +clear 2>/dev/null || true interactive_session "$RAMNODE_SERVER_IP" "source ~/.zshrc && gptme -m openrouter/${MODEL_ID}" diff --git a/ramnode/openclaw.sh b/ramnode/openclaw.sh index f0415608..31f1424f 100644 --- a/ramnode/openclaw.sh +++ b/ramnode/openclaw.sh @@ -28,7 +28,7 @@ verify_server_connectivity "${RAMNODE_SERVER_IP}" wait_for_cloud_init "${RAMNODE_SERVER_IP}" 60 # 5. Install openclaw via bun -log_warn "Installing openclaw..." +log_step "Installing openclaw..." run_server "${RAMNODE_SERVER_IP}" "source ~/.bashrc && bun install -g openclaw" log_info "OpenClaw installed" @@ -43,7 +43,7 @@ fi # Get model preference MODEL_ID=$(get_model_id_interactive "openrouter/auto" "Openclaw") || exit 1 -log_warn "Setting up environment variables..." +log_step "Setting up environment variables..." inject_env_vars_ssh "${RAMNODE_SERVER_IP}" upload_file run_server \ "OPENROUTER_API_KEY=${OPENROUTER_API_KEY}" \ "ANTHROPIC_API_KEY=${OPENROUTER_API_KEY}" \ @@ -60,7 +60,7 @@ log_info "Server: ${SERVER_NAME} (ID: ${RAMNODE_SERVER_ID}, IP: ${RAMNODE_SERVER echo "" # 10. Start openclaw gateway in background and launch TUI -log_warn "Starting openclaw..." +log_step "Starting openclaw..." run_server "${RAMNODE_SERVER_IP}" "source ~/.zshrc && nohup openclaw gateway > /tmp/openclaw-gateway.log 2>&1 &" sleep 2 interactive_session "${RAMNODE_SERVER_IP}" "source ~/.zshrc && openclaw tui"