Eddiebot: [BUG]: The bot word-filter doesn't flag ", <word>,"

[Vyvy-vi]

This bug seems to be a big issue as one may just list comma-separated bad-words and non-inclusive language, and not get flagged by Alexjs 😱

This isn't recognised or flagged by the bot: , <bad-word>,

Context:

At present this seems to be the only case in which the bot isn't flagging.(I tried other combinations)

[github-actions[bot]]

It's great having you contribute to this project

Feel free to raise an Issue! Welcome to the community :nerd_face:

If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord chat and our GitHub Organisation - we help and encourage each other to contribute to open source little and often 🤓 . Any questions let us know.

[Vyvy-vi]

The consequences:
(It can be a lot more bad 🙁 )

[eddiejaoude]

I think this is currently sent to the library, we might have to see what options we have available

[Vyvy-vi]

Another bug was found where "<bad-word>" was not flagged by the bot.
Could we perhaps do string processing, on the text, before sending it to Alex, in order to remove punctuation and ensure a space-separated sentence is sent??

[eddiejaoude]

ah yes that is a great idea 👍

[adityaraute]

At present this seems to be the only case in which the bot isn't flagging.(I tried other combinations)

Found one more:

I wonder if it is even reparable. Cuz when we split based on spaces, such characters will be associated with the word. If we consider flagging substrings even words like they will be flagged because the substring he is present in it.
Another way is to completely ignore all characters that are not alphabets. But again, is this ideal?

[Vyvy-vi]

I guess it is, for now, we could just strip off punctuation from the sentence and then join all the words with spaces in between

[mstanciu552]

Also -word- and 'word' and (word) and <word> and {word} aren't taken into consideration.

Still testing, but i think we have to check for non-alphabetical characters in the words and if a word is in between we send that to alexjs.

[Vyvy-vi]

Can I work on this ?
or perhaps we could team up on this? 🙃

[mstanciu552]

You can work on this and if you need any help i am here.
Super loaded with school next week and this weekend.

I'm gonna do more open source in the upcoming vacation. 😄 😋 👍
Is this ok?

[BOLT04]

@Vyvy-vi I've assigned it to you, but you can both work together 😃.
Thank you all for wanting to work on this and improve our word filter feature 👍 👍