refactor: Remove redundant EventTypeMessage emission in custom modules

[poorphd]

Description

EventTypeMessage events with sdk.AttributeKeyModule and sdk.AttributeKeySender are now emitted directly at message execution (in baseapp). This change removes the redundant boilerplate event emission from all custom modules.

The module name is assumed by baseapp to be the second element of the message route, e.g., "cosmos.bank.v1beta1.MsgSend" -> "bank". For modules that do not follow the standard message path (e.g., IBC), it is advised to continue emitting the module name event. Baseapp only emits the EventTypeMessage event if the module has not already done so.

This refactor ensures cleaner code and leverages the automatic event emission provided by baseapp.

Closes: L-4

---

Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

• targeted the correct branch (see PR Targeting)
• provided a link to the relevant issue or specification
• included the necessary unit and integration tests
• reviewed "Files changed" and left comments if necessary

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.

I have...

• confirmed all author checklist items have been addressed
• reviewed state machine logic
• reviewed API design and naming
• reviewed documentation is accurate
• reviewed tests and test coverage
• manually tested (if applicable)

[dudong2]

There are several event emit in x/<module name>/msg_server.go. We need to remove all event emit with sdk.AttributeKeyModule and sdk.AttributeKeySender(eg. RemoveLiquidity, ...)

[poorphd]

removed

[dongsam]

LGTM, adding the test code improved the review efficiency. Thanks 👍

[zsystm]

LGTM