improve: remove bot protection macro

[nekiro]

Description

This macro does very poor job at blocking any bot activity and additionally it also blocks some internal lua calls (for example #1028 doesn't work, because of this macro).

It basically just checks the stacktrace of a method and checks if it comes from lua.

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

Checklist

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I checked the PR checks reports
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works

[Nottinghster]

@kokekanon Can you test it?
@nekiro Are we good to go?

[nekiro]

@kokekanon Can you test it? @nekiro Are we good to go?

From my side, yeah. Rebased with master branch and is good to go.

[kokekanon]

I don't use bot much, but this is important in vbot4.8.

https://github.com/mehah/otclient/pull/1037/files#diff-1b975e904a1c9756dc22f83fbc9884012958e3c3f158bc081aaa6c49dfa2f585L806-L815

expected result

[nekiro]

I don't use bot much, but this is important in vbot4.8.

Alright, I can add it back.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud