Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add FreeRolesAnomalyDetector.ts (not system-tested yet) #4

Open
wants to merge 2 commits into
base: gmdi-rebase
Choose a base branch
from

Conversation

AbdiHaryadi
Copy link
Contributor

(The previous pull request has been cancelled because of signing issue)

This pull request contains a FreeRolesAnomalyDetector class for the second-rule bot prevention that was talked in 10 March 2024. Here is the summary for the changes:

  • The component can be imported as object for usage.
  • The component has been integrated in src/events.
  • The component can be configured in src/handler/Config.ts. Currently, the free role list is arbitrary.
  • The component has been unit tested, but not system tested.
  • The component callback can be customized in src/handler/FreeRolesAnomalyDetector.ts. Currently, it just a logging process.

Future actions:

  • Test the entire system with this component.
  • Configure the free roles.
  • Integrate the callback to bot prevention manager that handle other rules.

@AbdiHaryadi AbdiHaryadi marked this pull request as ready for review March 16, 2024 15:58
@AbdiHaryadi
Copy link
Contributor Author

AbdiHaryadi commented Mar 16, 2024

How to solve the signing issue? .-.

Edit: It's solved.

@AbdiHaryadi AbdiHaryadi marked this pull request as draft March 16, 2024 20:35
@AbdiHaryadi AbdiHaryadi marked this pull request as ready for review March 16, 2024 20:43
@ray-1337 ray-1337 self-assigned this Mar 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants