[Regression]: Reaction remove logs causes consistent crashes

[jonbarrow]

Checked Existing

• I have checked the repository for duplicate issues.

What happened?

In reactionRemoveHandler if messageAuditRelationship does not exist then it tries to create a thread and send a log message there. However the thread is not checked for being null or undefined and thus causes crashes if it is.

Why was this functionality added? What purpose does logging the removal of reactions have?

What was the previous functionality?

There was no reaction remove logs

When did this occur?

The reaction logs system was added in #29

Other relevant information. (OPTIONAL)

The stacktrace when the crash happens is:

node:events:497
      throw er; // Unhandled 'error' event
      ^
TypeError: Cannot read properties of null (reading 'send')
    at Client.reactionRemoveHandler (/home/jonbarrow/Chubby/dist/events/reactionRemove.js:39:22)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Emitted 'error' event on Client instance at:
    at emitUnhandledRejectionOrErr (node:events:402:10)
    at process.processTicksAndRejections (node:internal/process/task_queues:84:21)