fix(contracts): InterchainAccountRouter minor audit remediation

[aroralanuk]

Description

• disabled the ICARouter's ability to change hook given that the user doesn't expect the hook to change after they deploy their ICA account. Hook is not part of the derivation like ism on the destination chain and hence, cannot be configured custom by the user.

Drive-by changes

• MailboxClient events for hook and ism setting
• ProtocolFee events for setting beneficiary and owner

Related issues

• partly fixes https://github.com/chainlight-io/2024-08-hyperlane/issues/14

Backward compatibility

No

Testing

Unit tests

[changeset-bot]

🦋 Changeset detected

Latest commit: 495f7eb

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 9 packages

Name	Type
@hyperlane-xyz/core	Minor
@hyperlane-xyz/helloworld	Patch
@hyperlane-xyz/sdk	Patch
@hyperlane-xyz/infra	Patch
@hyperlane-xyz/cli	Patch
@hyperlane-xyz/widgets	Patch
@hyperlane-xyz/ccip-server	Patch
@hyperlane-xyz/github-proxy	Patch
@hyperlane-xyz/utils	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.78%. Comparing base (7f3e066) to head (495f7eb).
Report is 32 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4581      +/-   ##
==========================================
+ Coverage   73.74%   73.78%   +0.03%     
==========================================
  Files         100      100              
  Lines        1436     1442       +6     
  Branches      187      187              
==========================================
+ Hits         1059     1064       +5     
- Misses        356      357       +1     
  Partials       21       21              

Components	Coverage Δ
core	84.61% <ø> (ø)
hooks	76.41% <100.00%> (+0.70%)	⬆️
isms	77.58% <ø> (ø)
token	88.75% <ø> (ø)
middlewares	77.58% <100.00%> (+0.19%)	⬆️

[yorhodes]

my mental model for events is to surface data that is cumulative/not a constant time view call
dont feel strongly but this seems superfluous
I dont think we will ever build an indexer for these?

[aroralanuk]

my mental model for events is to surface data that is cumulative/not a constant time view call dont feel strongly but this seems superfluous I dont think we will ever build an indexer for these?

is hookset not something we can use in the validator if say the merkle hook gets updated in the future?

[yorhodes]

all lgtm except the only initializing change, please revert