fix: optimize hook module read in warp apply process by using provide…

[mshojaei-txfusion]

Description

This PR simplifies the hook module read functionality by removing the EvmHookReader class and its associated complexity. The changes include:

1. Removal of the EvmHookReader import and instance in EvmHookModule
2. Simplification of the read() method to directly return the config
3. Reordering of config update in the update() method to ensure proper comparison

These changes make the code more maintainable and reduce unnecessary complexity while maintaining the same functionality.

Drive-by changes

• Reordered the config update in the update() method to happen after normalization comparison

Related issues

• Fixes HookModule.read() always derives when given a Hook Config #5449

Backward compatibility

Yes - These changes are backward compatible. The functionality remains the same, we're just simplifying the implementation. No infrastructure implications as this is a refactoring of the internal SDK code.

Testing

Manual - The changes have been tested manually to ensure the hook module read and update functionality works as expected. The changes are minimal and focused on internal implementation details.

[changeset-bot]

🦋 Changeset detected

Latest commit: 8100621

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/sdk	Minor
@hyperlane-xyz/cli	Minor
@hyperlane-xyz/helloworld	Minor
@hyperlane-xyz/infra	Minor
@hyperlane-xyz/widgets	Minor
@hyperlane-xyz/ccip-server	Minor
@hyperlane-xyz/github-proxy	Minor
@hyperlane-xyz/utils	Minor
@hyperlane-xyz/core	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