fix: ensure script flow continues after successful pathfinding

[Roll1r]

fixed problem when after successful completion nodes with pathfinding flow of script doesn't continue

[Copilot]

Pull request overview

This PR fixes a control-flow bug in several script action nodes where successful pathfinding could complete without emitting a result, preventing the script from continuing.

Changes:

• Replace .map(...) with .thenReturn(...) after Mono.fromFuture(PathExecutor.executePathfinding(...)) so a success result is emitted even when the underlying CompletableFuture<Void> completes with null.
• Apply the same fix consistently across pathfinding-based action nodes (pathfind-to, pathfind-away, break-block, place-block).

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File	Description
mod/src/main/java/com/soulfiremc/server/script/nodes/action/PlaceBlockNode.java	Ensure success output is emitted after pathfinding completion by using thenReturn.
mod/src/main/java/com/soulfiremc/server/script/nodes/action/PathfindToNode.java	Ensure success output is emitted after pathfinding completion by using thenReturn.
mod/src/main/java/com/soulfiremc/server/script/nodes/action/PathfindAwayFromNode.java	Ensure success output is emitted after pathfinding completion by using thenReturn.
mod/src/main/java/com/soulfiremc/server/script/nodes/action/BreakBlockNode.java	Ensure success output is emitted after pathfinding completion by using thenReturn.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.