fix: improve error handling and safe checking

[jlucaso1]

This PR is a continuation of #1891 ,but is more agressive too, so we need more tests to this changes

1. Implemented a defer-like withAck Wrapper:

   • Introduced a new withAck higher-order function in src/Socket/messages-recv.ts.
   • This function wraps message processing logic in a try...finally block, guaranteeing that an acknowledgment (ack) is always sent to the server, even if the handler returns early or throws an error.
   • Refactored handleReceipt, handleNotification, and handleMessage to use this new pattern, eliminating scattered ack calls and preventing bugs where an ack might be missed.

2. Hardened Core Functions Against Null/Undefined Values:

   • Message Sending (messages-send.ts): Added a guard clause to relayMessage to check for authState.creds.me.id before proceeding, preventing crashes if a message is sent before authentication is complete.
   • Message Decoding (decode-wa-message.ts): Replaced risky non-null assertions (!) on incoming message stanza attributes (id, from) with proper validation. The function now throws a descriptive Boom error if essential attributes are missing.
   • Chat ID Resolution (process-message.ts): Refactored getChatId to safely handle potentially null or undefined remoteJid and participant values by using nullish coalescing (??), preventing crashes on malformed message keys.

[whiskeysockets-bot]

Thanks for opening this pull request and contributing to the project!

The next step is for the maintainers to review your changes. If everything looks good, it will be approved and merged into the main branch.

In the meantime, anyone in the community is encouraged to test this pull request and provide feedback.

✅ How to confirm it works

If you’ve tested this PR, please comment below with:

Tested and working ✅

This helps us speed up the review and merge process.

📦 To test this PR locally:

# NPM
npm install @whiskeysockets/baileys@WhiskeySockets/Baileys#chore-improve-some-type-safety

# Yarn (v2+)
yarn add @whiskeysockets/baileys@WhiskeySockets/Baileys#chore-improve-some-type-safety

# PNPM
pnpm add @whiskeysockets/baileys@WhiskeySockets/Baileys#chore-improve-some-type-safety

If you encounter any issues or have feedback, feel free to comment as well.

[purpshell]

Code looks clean and makes sense, thanks.

Will need to test intensively before deployment.

[Salientekill]

Looking forward to testing

[Salientekill]

I've been testing this PR since it was released, it improved the issue of Messages waiting in groups in addition to stability improvements, I think it would be cool to merge it, it improves an annoying error that happens

[Salientekill]

Tested and working ✅

[jeffersonfelixdev]

Tested and working ✅

[Salientekill]

It was just a merge notification to make my night 😂

[github-actions]

This PR is stale because it has been open for 14 days with no activity. Remove the stale label or comment or this will be closed in 14 days

[purpshell]

Resolve conflicts and should merge

[github-actions]

This PR is stale because it has been open for 14 days with no activity. Remove the stale label or comment or this will be closed in 14 days

[vinikjkkj]

Tested and working ✅

[purpshell]

resolve conflicts!