Skip to content

fix(antiban): default mouse speed to low - #1853

Closed
itsBOTzilla wants to merge 1 commit into
chsami:mainfrom
itsBOTzilla:fix/antiban-low-mouse-speed
Closed

fix(antiban): default mouse speed to low#1853
itsBOTzilla wants to merge 1 commit into
chsami:mainfrom
itsBOTzilla:fix/antiban-low-mouse-speed

Conversation

@itsBOTzilla

Copy link
Copy Markdown
Contributor

Antiban initializes Mouse Speed to Extreme when the plugin starts and when settings are reset. Set both paths to a shared Low default, initialize the activity intensity to Low, and align the Mouse Speed slider defaults with it.

Bump the Antiban plugin version to 1.0.1. Explicit user and script speed selections remain available.

Validation: :client:runUnitTests --tests net.runelite.client.plugins.microbot.util.antiban.Rs2AntibanDefaultsTest (includes client compilation); focused reset-default test passed. Source review completed. No live-client validation performed.

@coderabbitai

coderabbitai Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Team

Run ID: 90d324fc-6edc-42a1-aa80-8aaf937182f8

📥 Commits

Reviewing files that changed from the base of the PR and between 2c0721c and 9369604.

📒 Files selected for processing (4)
  • runelite-client/src/main/java/net/runelite/client/plugins/microbot/util/antiban/AntibanPlugin.java
  • runelite-client/src/main/java/net/runelite/client/plugins/microbot/util/antiban/Rs2Antiban.java
  • runelite-client/src/main/java/net/runelite/client/plugins/microbot/util/antiban/ui/MousePanel.java
  • runelite-client/src/test/java/net/runelite/client/plugins/microbot/util/antiban/Rs2AntibanDefaultsTest.java

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.


Walkthrough

The antiban plugin descriptor version is set to 1.0.1. ActivityIntensity.LOW is now the shared default for startup, initialization, and reset operations. Mouse-panel defaults and fallback conversions also use LOW. A JUnit test verifies that resetting settings changes the activity intensity from HIGH to LOW.

Merge Risk: ⚪ Minimal · up to 93696

The plugin now consistently defaults mouse activity intensity to Low at startup and after reset while retaining explicit selections. The updated UI fallbacks and reset coverage align with this behavior, with no remaining merge-blocking risk identified.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 42.86% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 7 functions across 4 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description check ✅ Passed The description accurately explains the Low default changes, version bump, preserved explicit selections, and validation performed.
Title check ✅ Passed The title clearly summarizes the main change: setting the Antiban default mouse speed to Low.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
  • Fix all pre-merge checks with AI

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@itsBOTzilla itsBOTzilla closed this Sep 4, 2026
@itsBOTzilla
itsBOTzilla deleted the fix/antiban-low-mouse-speed branch September 4, 2026 20:28
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.

1 participant