fix(cli): rename libs to initializers, always use init as func

[dk1a]

For #361

[dk1a]

I like initializers, changed it up, and with the new name it seems self-descriptive too

[frolic]

👏 Looks great, thank you!

My only other curiosity now is in terms of mud dev and "hot reloading" of contracts, when do initializers run? Once only on initial world deploy? On every change to the initializer lib?

[dk1a]

My only other curiosity now is in terms of mud dev and "hot reloading" of contracts, when do initializers run? Once only on initial world deploy? On every change to the initializer lib?

Good point, I use hsr too rarely to have thought of this. I made initializers respect the init flag, but it's kinda useless, especially for hot reloading. I think I should move inits from deploySystems into deploy and use _reuseComponents, so initializers run only if components are redeployed

[frolic]

cc @alvrs for input

[dk1a]

Oh, and to be more specific, hsr always reuses components afaik, so deploy-contracts --watch will ignore any changes to components and initializers, and only update the system code

[dk1a]

Oh, and to be more specific, hsr always reuses components afaik, so deploy-contracts --watch will ignore any changes to components and initializers, and only update the system code

After some testing turns out deploy-contracts --watch always had reuseComponents: false, which means it just didn't work? Since components can never be redeployed (#286).
I fixed it and everything seems to work as I expect and think it should (as in, deploy-contracts --watch deploys everything and runs initializers once, and on any change only redeploys the systems).

@alvrs I may be missing something, check out the changes to see if they make sense and the bug really existed

[alvrs]

After some testing turns out deploy-contracts --watch always had reuseComponents: false, which means it just didn't work?

Good catch! Previously we set reuseComponents to true when specific systems were provided, but this was recently changed in #356 and the implications for HMR missed. Thanks for fixing!