You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This PR adds a new feature, where in, if any command is found within either a triple or single code block, it's ignored and the bot does not trigger the message that is instead triggered if its not within codeblocks
Introduction of escapeRegex Utility Function
This update includes a new utility function called escapeRegex. Its role is to help manage special characters in a string that's used in search patterns, making the search operation more reliable and predictable.
Addition of shouldTriggerCommand Function
A new function named shouldTriggerCommand has been included. This function verifies if a command term is present in a message while overlooking the words situated inside the code sections written in Markdown language. This feature improves the function's precision by focusing only on relevant text sections.
Revision of Command Handling Logic
The logic responsible for managing commands has been enhanced in several ways:
It quickly disregards messages that don't start with a '!', making command processing more efficient.
It fetches a possible command and matches it to the corresponding command feature in a more efficient way.
Lastly, it performs a thorough validation to guarantee that the command is not found within a code section. This final check, though resource-intensive, ensures a clean execution of commands.
Integration with Existing Command Process Flow
The new logic has been well integrated into the existing command process flow. The updates prioritize certain checks to enhance performance, thereby reducing unnecessary computations and further optimizing the process flow.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📌 Description
This PR adds a new feature, where in, if any command is found within either a triple or single code block, it's ignored and the bot does not trigger the message that is instead triggered if its not within codeblocks
ℹ️ **Fixes **
Fixes: #412
🧱 Type of Change
🧪 How Has This Been Tested?
test/commands.test.ts)✅ Checklist
prettier(npm run prettier)🛠️ Affected Bot Areas
!su,!docs)📸 Screenshots / Demos (if applicable)
🧠 Additional Notes