Skip to content

Consolidate staking pool landing into install.sh - #448

Open
camembera wants to merge 5 commits into
mainfrom
chore/staking-pool-installer-rewrite
Open

Consolidate staking pool landing into install.sh#448
camembera wants to merge 5 commits into
mainfrom
chore/staking-pool-installer-rewrite

Conversation

@camembera

@camembera camembera commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Replace the staking-pool operator Node CLI and pruned bash helper sprawl with a single remote install.sh flow in install-helpers/
  • Self-funded and delegated landing share one script; network is read from eth_chainId, deposits are validated on the validator host, and confirmed txs append to staking-pool-receipts.jsonl
  • Keep delegator-delegate.sh for internal Foundation handler formation; drop standalone activate/deploy/status shell entrypoints

Companion docs PR: berachain/docs#98

Test plan

  • Local herdr run on Bepolia: deploy + activate through ./install.sh (pool 0xA9acE3Cf13D9609561eC3D8D472e3d9b70e64395)
  • ./install.sh --help on a clean host with Foundry cast, jq, and curl
  • Self-funded path: validator deposit paste-back + Ledger cast send for deploy/activate
  • Delegated path: existing DelegationHandler auto-detected; createStakingPoolWithDelegatedFunds landing

Single bastion entry point for self-funded and delegated landing; validator
ops stay in install-helpers. Drop redundant register/activate scripts,
env.sh generation, and plan-era naming. Harden lib-common logging and
deposit validation for bash 3.2 and Ledger paste flows.
… defaults.

Merge installer into install-helpers, drop legacy post-install scripts, add
interactive defaults via env.sh, expand README and template, and fix shared
address validation for delegator-delegate.sh.
Ask operators to confirm the beacond validate success line instead of exit
0, and exit explicitly from main after install or --help.
Comment thread apps/staking-pools/install-helpers/lib-common.sh Outdated
Comment thread apps/staking-pools/install-helpers/lib-common.sh Outdated
The workflow only runs Prettier; foundry-toolchain was failing on a bad
cached foundryup binary before that step ran. Format install-helpers README.
Semgrep flagged eval in run_cast_or_paste; build cast argv arrays and invoke cast directly for hot-key broadcasts.
@camembera
camembera requested a review from chuck-bear September 2, 2026 16:15

@chuck-bear chuck-bear left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

2 participants