Introduce Rs2Player.isIdle() and waitForXpDrop() overloads

[afss0]

This PR introduces:

1. Skill unspecific waitForXpDrop overloads, that triggers when the player receives experience in any skill.
2. Rs2Player.isIdle(), to check if the player is idle (not performing any action).

The player is considered idle if all of the following are true:

• Not currently animating (animation is IDLE and no animation within the last 3000ms)
• Not in combat
• Not moving
• Not interacting with a bank, deposit box, grand exchange, or chat dialog

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 726fcab6-4908-4ead-960f-83684cfc2c40

📥 Commits

Reviewing files that changed from the base of the PR and between 9fda631 and c30f098.

📒 Files selected for processing (1)

• runelite-client/src/main/java/net/runelite/client/plugins/microbot/util/player/Rs2Player.java

---

Walkthrough

This PR extends the Rs2Player utility class with imports and two new public methods for player state detection. Four UI/state helper imports (Rs2Bank, Rs2DepositBox, Rs2Dialogue, Rs2GrandExchange) are added to support blocking-condition checks. Four overloads of waitForXpDrop are introduced to detect overall experience gains (not per-skill), with optional timeout and inventory-full early-exit parameters. A new isIdle() method polls up to 3 seconds to verify the player is idle by confirming no XP gain, animation, combat, movement, and no open bank/deposit-box/grand-exchange/dialogue screens.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately and concisely describes the two main additions: Rs2Player.isIdle() method and waitForXpDrop() overloads.
Description check	✅ Passed	The description is directly related to the changeset, clearly explaining the two features being introduced and the idle criteria with specific technical details.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
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.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[afss0]

ClientThreadGuardrailTest claims net.runelite.api.Client#getOverallExperience() requires to be called in the client thread, however I didn't detect any exceptions during runtime. I'll review this further.