Skip to content

Update Looting Bag Stored XP: diagnostic logging#11699

Closed
ZNPKOH wants to merge 1 commit into
runelite:masterfrom
ZNPKOH:update-bag-diagnostic-logging
Closed

Update Looting Bag Stored XP: diagnostic logging#11699
ZNPKOH wants to merge 1 commit into
runelite:masterfrom
ZNPKOH:update-bag-diagnostic-logging

Conversation

@ZNPKOH
Copy link
Copy Markdown
Contributor

@ZNPKOH ZNPKOH commented Apr 28, 2026

Adds INFO-level diagnostic logging to help diagnose why items aren't being detected on some clients. Logs container and widget item counts every 3 seconds while the bag widget is open.

@runelite-github-app
Copy link
Copy Markdown

@runelite-github-app
Copy link
Copy Markdown

This plugin requires reviews from Plugin Hub reviewers. The reviewer will request any additional changes if needed.


Internal use only: Reviewer details Maintainer details

@coopermor
Copy link
Copy Markdown
Contributor

Are you aware you can test your plugin in an IDE without releasing to the plugin hub?

@ZNPKOH
Copy link
Copy Markdown
Contributor Author

ZNPKOH commented Apr 29, 2026

Thanks for the heads-up @coopermor — apologies for the noise. I've now set up local IDE testing with insecure-write-credentials and identified the issue: the looting bag stores items with noted IDs (id+1) and we needed ItemManager.canonicalize. The bug is fixed locally and verified working.

I'll close this diagnostic-only PR and submit a clean update with the fix shortly.

@ZNPKOH
Copy link
Copy Markdown
Contributor Author

ZNPKOH commented Apr 29, 2026

Closing — diagnostic logging no longer needed. A clean update PR is incoming.

@ZNPKOH ZNPKOH closed this Apr 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants