Querz / mcaselector

A tool to select chunks from Minecraft worlds for deletion or export.
MIT License
3.21k stars 175 forks source link

Not a fan of how specifying region/poi/entities works in CLI #482

Open TreeOfSelf opened 3 months ago

TreeOfSelf commented 3 months ago

Describe the bug Less of a bug more of a request, I have been trying to track down why the hell entities have been randomly being deleted in the overworld at server restart. During server restart, I delete chunks <1min inhabitedTime automatically.

On one of my servers, it doesn't have the nice clear world,world_nether, world_the_end, its got that stupid world/DIM1, world/DIM-1 and I have no idea why Minecraft insists on doing it like this sometimes, but it is what it is. At one point, I wrote the command to delete the chunks like

java -jar /usr/local/bin/mcaselector.jar --mode delete --world 'world' --region 'world/DIM-1/region' --query 'InhabitedTime < 1min'

Maybe you can already spot the issue with that? That will check the Nether chunks for inhabitedTime and delete them, BUT IT WILL AUTOMATICALLY USE THE OVERWORLD ENTITIES AND POI FOLDER. I can't really see when you would want to use a seperate entities/poi folder? So why not just supply it the DIM1 folder altogether instead of all three seperately, or at least have it as an option. Maybe a warning or something? If you supply region/poi/entities make you supply all three? Something.

This led to a huge headache of a bug-chase, players crying about their entities being deleted, me scratching my head what the heck is going on until I finally just now figured it out.

To Reproduce Clearly described in "bug report"

Expected behavior Just a feature request, nothing "Broken" just not a fan of how it works.

Environment (please complete the following information): Any, under CLI

Additional Hey, maybe I am the dumb one, but this felt pretty sneaky. I suppose I should have raised an eyebrow at the fact all three commands showed the same amount of progress bar roughly. But I feel like this could be done a bit better if nothing else, for people dumb like me.