meteor-client icon indicating copy to clipboard operation
meteor-client copied to clipboard

auto-tool improvements, split anti-break into its own module

Open chri-k opened this issue 3 months ago • 3 comments

Type of change

  • [X] Bug fix
  • [X] New feature

Description

auto-tool changes

  • setting to break glass with silk touch tools if possible
  • settings to allow using tools from the entire inventory
  • setting to try to switch to non-tool item when no tool is found

new module ToolSaver

  • replaces anti-break functionality everywhere other than AutoFish
  • setting to allow breaking tools anyway if a better one is present in inventory (very useful in conjunction with auto-tool), or breaking tools which don't have mending
  • other modules should check whether this module will allow a tool to be used before selecting it (KillAura still does not due to a few reasons)

Related issues

resolves #5707 (by accident) resolves #1805 resolves #4785 partially resolves #2255 superset of #3992 (did not know that existed)

possibly others

How Has This Been Tested?

direct functionality and interaction with some other modules was tested, but there is probably at least one buggy edge-case somewhere

Checklist:

  • [X] Code follows the style guidelines of this project.
  • [X] Comments exist in more complex areas
  • [X] Tested in both development and production environments.

chri-k avatar Sep 20 '25 18:09 chri-k

why did the check not run (it did, but it took over a minute for it to even be queued for some reason)

chri-k avatar Sep 20 '25 18:09 chri-k

How will this affect the legacy of popbob?

elijah-yum avatar Sep 21 '25 01:09 elijah-yum

don't spam random github comments

chri-k avatar Sep 21 '25 18:09 chri-k