All you need is this mate
You need to make use of your onLoop, No need to do multiple actions in 1 onLoop cycle ^^
if (getInventory().contains("Logs")) {
getInventory().drop("Logs");
} else {
if (!myPlayer().isAnimating()) {
RS2Object tree = getObjects().closest("Tree");
if (tree != null) {
if (tree.interact("Chop down")) {
new ConditionalSleep(random(4000, 5000)) {
@Override
public boolean condition() throws InterruptedException {
return myPlayer().isAnimating();
}
}.sleep();
}
} else {
//No Valid Tree Found
}
} else {
//Do nothing, We are Chopping, Just wait ...
}
}