feat(experimental): added a job/introduction bot detection

[Simplyalex99]

description:

• Added a bot scanner to remove messages of job postings in channels: help-react, help-js, help-react-native, help-styling, help-backend, general-react,general-tech. After 5 minutes, the reply message from the bot disappears.
• Added missing channels help-react-native, help-styling, help-backend, general-react,general-tech to channels.ts.

I have attached a video as a preview of this new feature. For some context, the first two messages in the video are about a job posting, which gets removed by the bot, and the last message is legit and does not get removed.

demo_LndJtupw.mp4

update: The bot now solely removes posts related to job postings rather than both intro and job posts to simplify the process and prevent false positives/negatives. Added an edge case for code blocks with dollar signs inside for languages like jQuery and other libraries. Any attempt to bypass the bot by including code blocks with dollar a sign and other job keywords gets marked as a job posting ensuring consistency and accuracy.

[what-the-diff]

PR Summary

• Enhanced Channel Information in channels.ts:
  The channel identifiers were expanded in our chat platform. Channels for introduction, generalTech, jobsAdvice, generalReact, helpReactNative, helpStyling, and helpBackend were added. This helps in categorizing topics and conversations better and assists in smooth communication transitions.

• Creation of message-scanner.ts File:

  • A new feature called message-scanner has been introduced. This works akin to a watch guard to monitor messages related to job postings.
  • Whenever a job post is detected outside of the designated job channels, it warns the user to relocate the post to the relevant channels and then proceeds to automatically delete the original message after a given amount of time. This helps maintain the order and relevance of content across various channels.

• Incorporation of messageScanner in index.ts:

  • The new messageScanner module has been integrated into the main functioning of the app. It is now set up to keep track of selected channels. This enhancement will help maintain the balance and integrity of content in various channels.

[Simplyalex99]

update: After careful inspection, the .includes function is returning false positives for legitimate post like "help me". Currently working on using regex instead of .includes.

[Simplyalex99]

update: Fixed incorrect messages being marked as a job posting with partial job keywords rather than the full job keywords by using regex to test for full keywords in job keywords. Removed currencies from job keywords into a separate array to test if a message contains any substring of currencies.

Also added two more keywords that 100% should be marked as a job posting such as "we are hiring". The code is ready to be merged.

[Copilot]

Pull Request Overview

This PR introduces an experimental bot detection feature to remove job posting messages in specific channels and updates the channels configuration accordingly.

• Implements a new message scanner that detects job postings via keyword matching and bypasses messages containing code blocks with dollar signs.
• Updates the handler registration in index.ts to include the new scanner on additional channels.
• Adds missing channel entries in channels.ts for the new feature.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
src/index.ts	Registers the message scanner for job posting detection on added channels.
src/features/message-scanner.ts	Implements bot detection logic using regex and keyword matching.
src/constants/channels.ts	Extends the channel constants with new channel mappings.

Comments suppressed due to low confidence (2)

src/features/message-scanner.ts:62

• CHANNELS.jobsLog and CHANNELS.lookingForGroup are referenced in the warning message but are not defined in the channels constants. Please update the message to reference only the valid, defined channels.

const warningMsg = `Oops <@${msg.author.id}>! This message looks more like a job/collaboration/advice post. Mind sharing that in <#${CHANNELS.jobsLog}> or <#${CHANNELS.lookingForGroup}> or <#${CHANNELS.jobsAdvice}> instead? If this was a mistake, please try again and ask your question. Appreciate you helping us keep channels on-topic 🙌`;

src/constants/channels.ts:24

• Local channel IDs for generalTech, jobsAdvice, generalReact, helpReactNative, helpStyling, and helpBackend share the same value. Please verify if each channel should have a unique identifier.

  generalTech: "1166062952084942940",

[Simplyalex99]

Pull Request Overview

This PR introduces an experimental bot detection feature to remove job posting messages in specific channels and updates the channels configuration accordingly.

• Implements a new message scanner that detects job postings via keyword matching and bypasses messages containing code blocks with dollar signs.
• Updates the handler registration in index.ts to include the new scanner on additional channels.
• Adds missing channel entries in channels.ts for the new feature.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/index.ts Registers the message scanner for job posting detection on added channels.
src/features/message-scanner.ts Implements bot detection logic using regex and keyword matching.
src/constants/channels.ts Extends the channel constants with new channel mappings.
Comments suppressed due to low confidence (2)
src/features/message-scanner.ts:62

• CHANNELS.jobsLog and CHANNELS.lookingForGroup are referenced in the warning message but are not defined in the channels constants. Please update the message to reference only the valid, defined channels.

const warningMsg = `Oops <@${msg.author.id}>! This message looks more like a job/collaboration/advice post. Mind sharing that in <#${CHANNELS.jobsLog}> or <#${CHANNELS.lookingForGroup}> or <#${CHANNELS.jobsAdvice}> instead? If this was a mistake, please try again and ask your question. Appreciate you helping us keep channels on-topic 🙌`;

src/constants/channels.ts:24

• Local channel IDs for generalTech, jobsAdvice, generalReact, helpReactNative, helpStyling, and helpBackend share the same value. Please verify if each channel should have a unique identifier.

  generalTech: "1166062952084942940",

Yes, the channels have already been added and are all valid. In addition, there are duplicate channel IDS but only in the local channels constant. This is intentional as it is just for development testing purposes, and the duplicate IDS are copied from existing duplicate IDS in the local channel constants such as resumeReview.