Skip to content

fix(update.sh): match reinstall.sh's OKX-first install flow#5

Merged
KillerQueen-Z merged 1 commit into
mainfrom
fix/update-sh-okx-flow
May 13, 2026
Merged

fix(update.sh): match reinstall.sh's OKX-first install flow#5
KillerQueen-Z merged 1 commit into
mainfrom
fix/update-sh-okx-flow

Conversation

@KillerQueen-Z
Copy link
Copy Markdown
Collaborator

Why

scripts/update.sh — the script users actually hit when they run the live curl https://blockrun.ai/XClawRouter-update | bash one-liner — was missing the OKX-first bootstrap that scripts/reinstall.sh already had.

End result: users running the official one-liner installed the new plugin (v0.12.182) but saw:

  • no Bootstrapping OKX Agentic Wallet section (no onchainos auto-install)
  • no Next step — log in to your OKX Agentic Wallet block (so they didn't know to run xclawrouter setup)
  • a misleading New wallet will be generated on next gateway start message — the plugin no longer auto-generates one, so users refreshed the gateway and silently ended up with no wallet at all

What

Bring update.sh in sync with reinstall.sh:

  • Add the same → Bootstrapping OKX Agentic Wallet (onchainos)... preflight that auto-installs the binary via OKX's install.sh, with the existing escape hatches (XCLAWROUTER_SKIP_OKX_INSTALL=1, existing wallet.key, XCLAWROUTER_USE_LOCAL_WALLET=1).
  • Replace the misleading post-install fallthrough with one that explains the user will use the OKX wallet (and to look at the Next step block).
  • Add the same Next step — log in to your OKX Agentic Wallet: npx @blockrun/xclawrouter setup block at the end, gated on the same "not already set up" condition.

Not in scope

  • scripts/update.ps1 (PowerShell sibling) needs the same treatment — tracked separately.

Test plan

  • bash -n scripts/update.sh — syntax clean
  • Reproduced the missing-bootstrap bug by running the live curl … | bash URL: install completed but no OKX guidance shown, gateway restarted with no wallet, plugin silently failed
  • Ran the local fixed scripts/update.sh end-to-end: OKX bootstrap section appears, no misleading "wallet will be generated" line, Next step block prints at the end

scripts/update.sh — which is what users actually hit when they run
the live `curl https://blockrun.ai/XClawRouter-update | bash` command
— was missing the OKX-first bootstrap that scripts/reinstall.sh
already had. Net effect: users running the official one-liner saw:

  - no `Bootstrapping OKX Agentic Wallet` section
  - no `Next step — log in to your OKX Agentic Wallet` block
  - a misleading "New wallet will be generated on next gateway start"
    message (the plugin no longer auto-generates one, so users would
    refresh the gateway and silently end up with no wallet at all)

Bring the two scripts in sync:

- Add the same `→ Bootstrapping OKX Agentic Wallet (onchainos)...`
  preflight that auto-installs the binary via OKX's install.sh, or
  skips when XCLAWROUTER_SKIP_OKX_INSTALL=1 / an existing local
  wallet / opt-in env var is present.
- Replace the misleading post-install fallthrough message with one
  that explains the user will use the OKX wallet via the upcoming
  Next step.
- Add the same `Next step — log in to your OKX Agentic Wallet` block
  at the end of the script, gated on the same "wallet not already
  set up" condition.

scripts/update.ps1 (PowerShell sibling) still needs the same
treatment; tracked separately.
@KillerQueen-Z KillerQueen-Z merged commit b20458e into main May 13, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant