755 Feature request add senders id to the chat reaction information and update test builders

[CapoMK25]

Brief description

This PR implements the requirement to include the player's unique ID (sender_id) within chat reactions. This ensures the frontend can reliably identify the author of a reaction, consistent with our ChatMessage ownership model.

Change list

• Modified Reaction class in reaction.schema.ts to include sender_id as a required String property.
• Updated ReactionDto and CreateReactionDto to enforce the inclusion of sender_id.
• Updated ChatService.addReaction() to accept sender_id as a mandatory 4th argument.
• Refactored BaseChatService and gateways to extract the playerId from the authenticated socket client and pass it through to the service layer.
• Updated ReactionDtoBuilder and ChatBuilderFactory with default mock IDs to satisfy TypeScript's strict requirements across all existing tests.
• Updated addReaction.test.ts to verify that sender_id is correctly persisted and returned.
• Full Pass: All 249 test suites (including global and clan chat) are passing with the new mandatory argument.

[CapoMK25]

We have to go through the transaction support PRs first since they touch the same code, and then we can move on to this.

[hoan301298]

Looks good to me. But if you can change a bit more, that could be nice.

[hoan301298]

I suggest to add

setSenderId(id: string) {
    this.base.sender_id = id;
    return this;
}

[hoan301298]

This comment is up to you if you want it simple like using the senderId below to pass into addReaction(). This action means the sender also reacting with his message. Or another sender react to it like you did.