Merge branch 'tapanito/transaction-invariant' into tapanito/transactor-invariant-pt2

include/xrpl/tx/applySteps.h

@@ -3,8 +3,6 @@
 #include <xrpl/beast/utility/Journal.h>
 #include <xrpl/ledger/ApplyViewImpl.h>
 
-#include <memory>
-
 namespace xrpl {
 
 class ServiceRegistry;

src/libxrpl/tx/Transactor.cpp

@@ -1264,6 +1264,8 @@ Transactor::operator()()
 
     if (applied)
     {
+        // Check invariants: if `tecINVARIANT_FAILED` is not returned, we can
+        // proceed to apply the tx
         result = checkInvariants(result, fee);
         if (result == tecINVARIANT_FAILED)
         {

src/libxrpl/tx/applySteps.cpp

@@ -295,6 +295,9 @@ invoke_apply(ApplyContext& ctx)
     }
 }
 
+// Test-only factory — not part of the public API.
+// The returned Transactor holds a raw reference to ctx; the caller must ensure
+// the ApplyContext outlives the Transactor.
 std::unique_ptr<Transactor>
 makeTransactor(ApplyContext& ctx)
 {