Merge branch 'dev' into fix-docker-cleanup-empty-path

Pure move of the post-apply pipeline (tec handling, invariants,
balance rewards, weak hooks, commit) into a helper so it can be
invoked more than once. No behaviour change.

.github/workflows/build-in-docker.yml

@@ -91,5 +91,11 @@ jobs:
       - name: Cleanup workspace
         run: |
           CHECKOUT_PATH="${{ needs.checkout.outputs.checkout_path }}"
+          # A run cancelled before checkout has no path; an empty one would
+          # make the rm below delete every run's checkout on the runner.
+          if [[ -z "${CHECKOUT_PATH}" ]]; then
+            echo "No checkout path (cancelled before checkout); nothing to clean"
+            exit 0
+          fi
           echo "Cleaning workspace for ${CHECKOUT_PATH}"
           rm -rf "${{ github.workspace }}/${CHECKOUT_PATH}"

src/xrpld/app/tx/detail/Transactor.cpp

@@ -2050,6 +2050,16 @@ Transactor::operator()()
     if (auto stream = j_.trace())
         stream << "preclaim result: " << transToken(result);
 
+    return finishApply(result, hooksEnabled, aawMap, tsh);
+}
+
+ApplyResult
+Transactor::finishApply(
+    TER result,
+    bool hooksEnabled,
+    std::map<AccountID, std::set<uint256>>& aawMap,
+    std::vector<std::pair<AccountID, bool>>& tsh)
+{
     bool applied = isTesSuccess(result);
 
     auto fee = ctx_.tx.getFieldAmount(sfFee).xrp();

src/xrpld/app/tx/detail/Transactor.h

@@ -228,6 +228,15 @@ protected:
                              // deduced until after apply i.e. pathing
                              // participants, crossed offers
 
+    // The post-apply pipeline of operator(): tec handling (reset),
+    // invariants, balance rewards, weak hooks and commit.
+    ApplyResult
+    finishApply(
+        TER result,
+        bool hooksEnabled,
+        std::map<AccountID, std::set<uint256>>& aawMap,
+        std::vector<std::pair<AccountID, bool>>& tsh);
+
     ///////////////////////////////////////////////////
 
     TER