Add stake on behalf of account functionality

[daveroga]

• Add stake on behalf function stakeAndLockOnBehalf
• Add setStakerOnBehalf function to only allow stakerOnBehalfaddress to execute the previous function.
• Add stakeOnBehalf to only stake on behalf without lock period.
• AccessControlUpgradeable for different roles: REWARDS_DISTRIBUTOR_ROLE, STAKER_ON_BEHALF_ROLE.
• Initial lock period to lock withdraw and get rewards during this period.

[Copilot]

Pull request overview

This PR adds "stake on behalf" functionality to the StakingRewards contract, replacing the single rewardsDistribution address with OpenZeppelin's AccessControlUpgradeable for role-based access, and introduces an initial lock period that restricts withdrawals and reward claims.

Changes:

• Added stakeOnBehalf and stakeAndLockOnBehalf functions gated by STAKER_ON_BEHALF_ROLE, refactoring stake and lockStake into internal _stake and _lockStake helpers
• Replaced rewardsDistribution address with AccessControlUpgradeable roles (REWARDS_DISTRIBUTOR_ROLE, STAKER_ON_BEHALF_ROLE)
• Added setInitialLockPeriod owner function to block withdrawals and reward claims during a configurable initial lock period

Reviewed changes

Copilot reviewed 3 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
contracts/staking/StakingRewards.sol	Core changes: AccessControl integration, stake-on-behalf functions, initial lock period
contracts/staking/interfaces/IStakingRewards.sol	Interface updated to reflect new functions and removed rewardsDistribution/setRewardsDistribution
test/StakingRewards.test.ts	Tests updated for role-based access and new functionality
package-lock.json	Dependency peer/direct flag changes
README.md	Minor trailing whitespace change
.gitignore	Added testnet deploy script JSON exclusion

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

This PR adds "stake on behalf" functionality and role-based access control to the StakingRewards upgradeable contract, replacing the single rewardsDistribution address with OpenZeppelin's AccessControlUpgradeable roles, and introducing an initial lock period feature.

Changes:

• Replaced rewardsDistribution address and onlyRewardsDistribution modifier with AccessControlUpgradeable roles (REWARDS_DISTRIBUTOR_ROLE, STAKER_ON_BEHALF_ROLE)
• Added stakeOnBehalf, stakeAndLockOnBehalf functions and refactored stake/lockStake into internal _stake/_lockStake helpers
• Added setInitialLockPeriod to block withdrawals and reward claims during a configurable initial lock period

Reviewed changes

Copilot reviewed 3 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
contracts/staking/StakingRewards.sol	Core changes: ACL roles, stake-on-behalf functions, initial lock period, internal refactors
contracts/staking/interfaces/IStakingRewards.sol	Interface updated to reflect new functions and removed rewardsDistribution/setRewardsDistribution
test/StakingRewards.test.ts	Tests updated for role-based access, new stakeOnBehalf/stakeAndLockOnBehalf, and initial lock period
package-lock.json	Dependency peer/direct classification changes
README.md	Whitespace-only change
.gitignore	Added testnet deployment JSON exclusion

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.