Update deploy script to include SepoliaDepositAdapter contract upgrade

[vp4242]

Update deploy script to include SepoliaDepositAdapter contract upgrade

[TheDZhon]

🤔 Scarcely am I sure if we need to hardcode the addresses here instead of putting/reading them from the deployed-sepolia.json

wdyt, @arwer13?

[arwer13]

@TheDZhon @vp42ldo There is updateProxyImplementation from deploy.js for the case (if the adapter implementation not deployed yet). It will read the proxy address from the state file, deploy the implementation, bind it to the proxy and update the impl address in the state file.

[vp4242]

@TheDZhon @vp42ldo There is updateProxyImplementation from deploy.js for the case (if the adapter implementation not deployed yet). It will read the proxy address from the state file, deploy the implementation, bind it to the proxy and update the impl address in the state file.

There is an issue with updateProxyImplementation. DepositContractAdapter and it's proxy are not part of deployed-.json as it's specific for Sepolia-deploy only. Currently there is no reference to the proxy address except a parameter to stakingRouter

[arwer13]

Oh, we've forgotten to add it to the state file :/ I'd propose to add it now under /depositContractAdapter. It is part of the setup and thus must be presented in the state file as well.

[TheDZhon]

Having it would be prudent, I agree with @arwer13 🙏

[arwer13]

lgtm, taking into account the deploy script is most likely one-time-usage and the migration to the new core repo and typescript (won't move it there)