Skip to content

Comments

refactor: simplify inventory ID usage and clean up imports in MInvent…#1545

Merged
chsami merged 1 commit intomainfrom
development
Oct 1, 2025
Merged

refactor: simplify inventory ID usage and clean up imports in MInvent…#1545
chsami merged 1 commit intomainfrom
development

Conversation

@chsami
Copy link
Owner

@chsami chsami commented Oct 1, 2025

…orySetupsPlugin and SessionManager

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 1, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

  • Bumps runelite-client/pom.xml property microbot.version from 2.0.16 to 2.0.16.1 for the shaded microbot artifact.
  • SessionManager.java refactors imports: consolidates several explicit java.io and related imports into wildcard/organized imports without modifying class signatures or logic.
  • MInventorySetupsPlugin.java adds a static import for InventoryID.INV and replaces fully qualified references to use INV across inventory-related code paths, updating container retrieval and size calculations in methods such as getNormalizedContainer, addInventorySetup, and updateCurrentSetup.

Possibly related PRs

✨ Finishing touches
  • 📝 Generate Docstrings

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cb45581 and fc5056e.

📒 Files selected for processing (3)
  • runelite-client/pom.xml (1 hunks)
  • runelite-client/src/main/java/net/runelite/client/account/SessionManager.java (1 hunks)
  • runelite-client/src/main/java/net/runelite/client/plugins/microbot/inventorysetups/MInventorySetupsPlugin.java (7 hunks)

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 and usage tips.

@chsami chsami merged commit 58b6451 into main Oct 1, 2025
2 of 3 checks passed
@coderabbitai coderabbitai bot mentioned this pull request Oct 14, 2025
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