The only issue I see with this is that if log storage fills up with a fresh axe, your bot could fell 500 trees before exiting the "until hands empty" loop.
nope he loops only sometimes after loading the game, if you setup this in a active save the bot stops fine. i play for houres with it and mostly need to search for him when he stops. i already extended this code so he can work on a apple tree farm as primary, sourced on apple storage and switches over to pine trees until Logs are full and stops when the log storage is near full.