Skip to content

Feature InventoryCleaner module#229

Open
IceTank wants to merge 8 commits intolambda-client:1.21.11from
IceTank:feature/module/easy-trash
Open

Feature InventoryCleaner module#229
IceTank wants to merge 8 commits intolambda-client:1.21.11from
IceTank:feature/module/easy-trash

Conversation

@IceTank
Copy link
Contributor

@IceTank IceTank commented Jan 26, 2026

This module adds a feature to auto drop items when you don't have space in your inventory

@IceTank
Copy link
Contributor Author

IceTank commented Jan 26, 2026

Auto.drop.mp4
Auto.inv.drop.mp4

@beanbag44 beanbag44 self-requested a review January 26, 2026 23:30
Copy link
Member

@beanbag44 beanbag44 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really handy module, just some small nit picks and changes

@IceTank IceTank requested a review from beanbag44 January 27, 2026 18:56
@IceTank IceTank changed the title feature easy-trash module Feature InventoryCleaner module Feb 21, 2026
Copy link
Member

@beanbag44 beanbag44 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

when clicking an empty slot, it still throws a stack to make space

""".trimIndent(),
tag = ModuleTag.PLAYER
) {
private val itemsCanTrash by setting(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

settings should be written on one line each

}

fun SafeContext.checkShouldTrashSomething() {
if (player.health > 0.0f && !player.isSpectator && dropToPickup && !player.isCreative) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could probably replace the health check for player.isAlive. I'd also just check if the players in survival

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.

3 participants