Merge branch 'develop' of https://github.com/XRPLF/rippled into ripple/wasmi

Merge remote-tracking branch 'upstream/develop' into ripple/wasmi
fix: Handle WSClient write failure when server closes WebSocket (#6671 )
2026-04-19 18:32:23 +00:00 · 2026-04-08 13:48:41 -04:00 · 2026-04-07 16:02:56 -04:00 · 2026-04-07 16:01:26 -04:00 · 2026-04-07 16:01:26 -04:00 · 2026-04-07 16:01:26 -04:00
921 changed files with 2962 additions and 12438 deletions
--- a/.clang-format
+++ b/.clang-format
@@ -50,21 +50,20 @@ ForEachMacros: [Q_FOREACH, BOOST_FOREACH]
 IncludeBlocks: Regroup
 IncludeCategories:
  - Regex: "^<(test)/"
-    Priority: 1
+    Priority: 0
  - Regex: "^<(xrpld)/"
-    Priority: 2
+    Priority: 1
  - Regex: "^<(xrpl)/"
-    Priority: 3
+    Priority: 2
  - Regex: "^<(boost)/"
-    Priority: 4
+    Priority: 3
  - Regex: "^.*/"
-    Priority: 5
+    Priority: 4
  - Regex: '^.*\.h'
-    Priority: 6
+    Priority: 5
  - Regex: ".*"
-    Priority: 7
+    Priority: 6
 IncludeIsMainRegex: "$"
-MainIncludeChar: AngleBracket
 IndentCaseLabels: true
 IndentFunctionDeclarationAfterType: false
 IndentRequiresClause: true
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -10,26 +10,26 @@ Checks: "-*,
  bugprone-chained-comparison,
  bugprone-compare-pointer-to-member-virtual-function,
  bugprone-copy-constructor-init,
-  bugprone-crtp-constructor-accessibility,
+  # bugprone-crtp-constructor-accessibility, # has issues
  bugprone-dangling-handle,
  bugprone-dynamic-static-initializers,
-  bugprone-empty-catch,
+  # bugprone-empty-catch, # has issues
  bugprone-fold-init-type,
-  bugprone-forward-declaration-namespace,
-  bugprone-inaccurate-erase,
-  bugprone-inc-dec-in-conditions,
-  bugprone-incorrect-enable-if,
-  bugprone-incorrect-roundings,
-  bugprone-infinite-loop,
-  bugprone-integer-division,
+  # bugprone-forward-declaration-namespace, # has issues
+  # bugprone-inaccurate-erase,
+  # bugprone-inc-dec-in-conditions,
+  # bugprone-incorrect-enable-if,
+  # bugprone-incorrect-roundings,
+  # bugprone-infinite-loop,
+  # bugprone-integer-division,
  bugprone-lambda-function-name,
-  bugprone-macro-parentheses,
+  # bugprone-macro-parentheses, # has issues
  bugprone-macro-repeated-side-effects,
  bugprone-misplaced-operator-in-strlen-in-alloc,
  bugprone-misplaced-pointer-arithmetic-in-alloc,
  bugprone-misplaced-widening-cast,
  bugprone-move-forwarding-reference,
-  bugprone-multi-level-implicit-pointer-conversion,
+  # bugprone-multi-level-implicit-pointer-conversion, # has issues
  bugprone-multiple-new-in-one-expression,
  bugprone-multiple-statement-macro,
  bugprone-no-escape,
@@ -39,13 +39,13 @@ Checks: "-*,
  bugprone-pointer-arithmetic-on-polymorphic-object,
  bugprone-posix-return,
  bugprone-redundant-branch-condition,
-  bugprone-reserved-identifier,
-  bugprone-return-const-ref-from-parameter,
+  # bugprone-reserved-identifier, # has issues
+  # bugprone-return-const-ref-from-parameter, # has issues
  bugprone-shared-ptr-array-mismatch,
  bugprone-signal-handler,
  bugprone-signed-char-misuse,
  bugprone-sizeof-container,
-  bugprone-sizeof-expression,
+  # bugprone-sizeof-expression, # has issues
  bugprone-spuriously-wake-up-functions,
  bugprone-standalone-empty,
  bugprone-string-constructor,
@@ -62,7 +62,7 @@ Checks: "-*,
  bugprone-suspicious-string-compare,
  bugprone-suspicious-stringview-data-usage,
  bugprone-swapped-arguments,
-  bugprone-switch-missing-default-case,
+  # bugprone-switch-missing-default-case, # has issues
  bugprone-terminating-continue,
  bugprone-throw-keyword-missing,
  bugprone-too-small-loop-variable,
@@ -73,7 +73,7 @@ Checks: "-*,
  bugprone-unhandled-self-assignment,
  bugprone-unique-ptr-array-mismatch,
  bugprone-unsafe-functions,
-  bugprone-use-after-move, # has issues
+  # bugprone-use-after-move, # has issues
  bugprone-unused-raii,
  bugprone-unused-return-value,
  bugprone-unused-local-non-trivial-variable,
@@ -90,7 +90,6 @@ Checks: "-*,
  misc-const-correctness,
  misc-definitions-in-headers,
  misc-header-include-cycle,
-  misc-include-cleaner,
  misc-misplaced-const,
  misc-redundant-expression,
  misc-static-assert,
@@ -109,35 +108,37 @@ Checks: "-*,
  performance-move-constructor-init,
  performance-no-automatic-move,
  performance-trivially-destructible,
-  readability-avoid-nested-conditional-operator,
-  readability-avoid-return-with-void-value,
-  readability-braces-around-statements,
-  readability-const-return-type,
-  readability-container-contains,
-  readability-container-size-empty,
-  readability-convert-member-functions-to-static,
+  # readability-avoid-nested-conditional-operator, # has issues
+  # readability-avoid-return-with-void-value, # has issues
+  # readability-braces-around-statements, # has issues
+  # readability-const-return-type, # has issues
+  # readability-container-contains, # has issues
+  # readability-container-size-empty, # has issues
+  # readability-convert-member-functions-to-static, # has issues
  readability-duplicate-include,
-  readability-else-after-return,
-  readability-enum-initial-value,
-  readability-implicit-bool-conversion,
-  readability-make-member-function-const,
-  readability-math-missing-parentheses,
+  # readability-else-after-return, # has issues
+  # readability-enum-initial-value, # has issues
+  # readability-implicit-bool-conversion, # has issues
+  # readability-make-member-function-const, # has issues
+  # readability-math-missing-parentheses, # has issues
  readability-misleading-indentation,
  readability-non-const-parameter,
-  readability-redundant-casting,
-  readability-redundant-declaration,
-  readability-redundant-inline-specifier,
-  readability-redundant-member-init,
+  # readability-redundant-casting, # has issues
+  # readability-redundant-declaration, # has issues
+  # readability-redundant-inline-specifier, # has issues
+  # readability-redundant-member-init, # has issues
  readability-redundant-string-init,
  readability-reference-to-constructed-temporary,
-  readability-simplify-boolean-expr,
-  readability-static-definition-in-anonymous-namespace,
-  readability-suspicious-call-argument,
+  # readability-simplify-boolean-expr, # has issues
+  # readability-static-definition-in-anonymous-namespace, # has issues
+  # readability-suspicious-call-argument, # has issues
  readability-use-std-min-max
  "
 # ---
 # other checks that have issues that need to be resolved:
 #
+# misc-include-cleaner,
+#
 # readability-inconsistent-declaration-parameter-name, # in this codebase this check will break a lot of arg names
 # readability-static-accessed-through-instance,        # this check is probably unnecessary. it makes the code less readable
 # readability-identifier-naming,                       # https://github.com/XRPLF/rippled/pull/6571
@@ -193,7 +194,7 @@ CheckOptions:
  #   readability-identifier-naming.FunctionIgnoredRegexp: ".*tag_invoke.*"
  bugprone-unsafe-functions.ReportMoreUnsafeFunctions: true
  bugprone-unused-return-value.CheckedReturnTypes: ::std::error_code;::std::error_condition;::std::errc
-  misc-include-cleaner.IgnoreHeaders: ".*/(detail|impl)/.*;.*fwd\\.h(pp)?;time.h;stdlib.h;sqlite3.h;netinet/in\\.h;sys/resource\\.h;sys/sysinfo\\.h;linux/sysinfo\\.h;__chrono/.*;bits/.*;_abort\\.h;boost/uuid/uuid_hash.hpp;boost/beast/core/flat_buffer\\.hpp;boost/beast/http/field\\.hpp;boost/beast/http/dynamic_body\\.hpp;boost/beast/http/message\\.hpp;boost/beast/http/read\\.hpp;boost/beast/http/write\\.hpp;openssl/obj_mac\\.h"
+#   misc-include-cleaner.IgnoreHeaders: '.*/(detail|impl)/.*;.*(expected|unexpected).*;.*ranges_lower_bound\.h;time.h;stdlib.h;__chrono/.*;fmt/chrono.h;boost/uuid/uuid_hash.hpp'
 #
 HeaderFilterRegex: '^.*/(test|xrpl|xrpld)/.*\.(h|hpp)$'
 ExcludeHeaderFilterRegex: '^.*/protocol_autogen/.*\.(h|hpp)$'
--- a/.github/scripts/levelization/results/loops.txt
+++ b/.github/scripts/levelization/results/loops.txt
@@ -2,19 +2,19 @@ Loop: test.jtx test.toplevel
  test.toplevel > test.jtx

 Loop: test.jtx test.unit_test
-  test.unit_test ~= test.jtx
+  test.unit_test == test.jtx

 Loop: xrpld.app xrpld.overlay
-  xrpld.app > xrpld.overlay
+  xrpld.overlay ~= xrpld.app

 Loop: xrpld.app xrpld.peerfinder
-  xrpld.peerfinder ~= xrpld.app
+  xrpld.peerfinder == xrpld.app

 Loop: xrpld.app xrpld.rpc
  xrpld.rpc > xrpld.app

 Loop: xrpld.app xrpld.shamap
-  xrpld.shamap > xrpld.app
+  xrpld.shamap ~= xrpld.app

 Loop: xrpld.overlay xrpld.rpc
  xrpld.rpc ~= xrpld.overlay
--- a/.github/scripts/levelization/results/ordering.txt
+++ b/.github/scripts/levelization/results/ordering.txt
@@ -3,17 +3,13 @@ libxrpl.conditions > xrpl.basics
 libxrpl.conditions > xrpl.conditions
 libxrpl.core > xrpl.basics
 libxrpl.core > xrpl.core
-libxrpl.core > xrpl.json
 libxrpl.crypto > xrpl.basics
 libxrpl.json > xrpl.basics
 libxrpl.json > xrpl.json
 libxrpl.ledger > xrpl.basics
 libxrpl.ledger > xrpl.json
 libxrpl.ledger > xrpl.ledger
-libxrpl.ledger > xrpl.nodestore
 libxrpl.ledger > xrpl.protocol
-libxrpl.ledger > xrpl.server
-libxrpl.ledger > xrpl.shamap
 libxrpl.net > xrpl.basics
 libxrpl.net > xrpl.net
 libxrpl.nodestore > xrpl.basics
@@ -23,22 +19,19 @@ libxrpl.nodestore > xrpl.protocol
 libxrpl.protocol > xrpl.basics
 libxrpl.protocol > xrpl.json
 libxrpl.protocol > xrpl.protocol
+libxrpl.protocol_autogen > xrpl.protocol_autogen
 libxrpl.rdb > xrpl.basics
 libxrpl.rdb > xrpl.core
 libxrpl.rdb > xrpl.rdb
 libxrpl.resource > xrpl.basics
 libxrpl.resource > xrpl.json
-libxrpl.resource > xrpl.protocol
 libxrpl.resource > xrpl.resource
 libxrpl.server > xrpl.basics
-libxrpl.server > xrpl.core
 libxrpl.server > xrpl.json
 libxrpl.server > xrpl.protocol
 libxrpl.server > xrpl.rdb
-libxrpl.server > xrpl.resource
 libxrpl.server > xrpl.server
 libxrpl.shamap > xrpl.basics
-libxrpl.shamap > xrpl.nodestore
 libxrpl.shamap > xrpl.protocol
 libxrpl.shamap > xrpl.shamap
 libxrpl.tx > xrpl.basics
@@ -50,11 +43,12 @@ libxrpl.tx > xrpl.protocol
 libxrpl.tx > xrpl.server
 libxrpl.tx > xrpl.tx
 test.app > test.jtx
+test.app > test.rpc
+test.app > test.toplevel
 test.app > test.unit_test
 test.app > xrpl.basics
 test.app > xrpl.core
 test.app > xrpld.app
-test.app > xrpld.consensus
 test.app > xrpld.core
 test.app > xrpld.overlay
 test.app > xrpld.rpc
@@ -62,9 +56,9 @@ test.app > xrpl.json
 test.app > xrpl.ledger
 test.app > xrpl.nodestore
 test.app > xrpl.protocol
+test.app > xrpl.rdb
 test.app > xrpl.resource
 test.app > xrpl.server
-test.app > xrpl.shamap
 test.app > xrpl.tx
 test.basics > test.jtx
 test.basics > test.unit_test
@@ -77,17 +71,16 @@ test.beast > xrpl.basics
 test.conditions > xrpl.basics
 test.conditions > xrpl.conditions
 test.consensus > test.csf
-test.consensus > test.jtx
 test.consensus > test.toplevel
 test.consensus > test.unit_test
 test.consensus > xrpl.basics
 test.consensus > xrpld.app
 test.consensus > xrpld.consensus
+test.consensus > xrpl.json
 test.consensus > xrpl.ledger
-test.consensus > xrpl.protocol
-test.consensus > xrpl.shamap
 test.consensus > xrpl.tx
 test.core > test.jtx
+test.core > test.toplevel
 test.core > test.unit_test
 test.core > xrpl.basics
 test.core > xrpl.core
@@ -115,32 +108,27 @@ test.jtx > xrpl.resource
 test.jtx > xrpl.server
 test.jtx > xrpl.tx
 test.ledger > test.jtx
+test.ledger > test.toplevel
 test.ledger > xrpl.basics
-test.ledger > xrpl.core
-test.ledger > xrpld.app
 test.ledger > xrpld.core
-test.ledger > xrpl.json
 test.ledger > xrpl.ledger
 test.ledger > xrpl.protocol
 test.nodestore > test.jtx
+test.nodestore > test.toplevel
 test.nodestore > test.unit_test
 test.nodestore > xrpl.basics
-test.nodestore > xrpld.core
 test.nodestore > xrpl.nodestore
-test.nodestore > xrpl.protocol
 test.nodestore > xrpl.rdb
 test.overlay > test.jtx
+test.overlay > test.toplevel
 test.overlay > test.unit_test
 test.overlay > xrpl.basics
 test.overlay > xrpld.app
-test.overlay > xrpld.core
 test.overlay > xrpld.overlay
 test.overlay > xrpld.peerfinder
-test.overlay > xrpl.json
+test.overlay > xrpl.ledger
 test.overlay > xrpl.nodestore
 test.overlay > xrpl.protocol
-test.overlay > xrpl.resource
-test.overlay > xrpl.server
 test.overlay > xrpl.shamap
 test.peerfinder > test.beast
 test.peerfinder > test.unit_test
@@ -148,7 +136,7 @@ test.peerfinder > xrpl.basics
 test.peerfinder > xrpld.core
 test.peerfinder > xrpld.peerfinder
 test.peerfinder > xrpl.protocol
-test.protocol > test.jtx
+test.protocol > test.toplevel
 test.protocol > test.unit_test
 test.protocol > xrpl.basics
 test.protocol > xrpl.json
@@ -157,6 +145,7 @@ test.resource > test.unit_test
 test.resource > xrpl.basics
 test.resource > xrpl.resource
 test.rpc > test.jtx
+test.rpc > test.toplevel
 test.rpc > xrpl.basics
 test.rpc > xrpl.core
 test.rpc > xrpld.app
@@ -170,12 +159,13 @@ test.rpc > xrpl.resource
 test.rpc > xrpl.server
 test.rpc > xrpl.tx
 test.server > test.jtx
+test.server > test.toplevel
 test.server > test.unit_test
 test.server > xrpl.basics
 test.server > xrpld.app
 test.server > xrpld.core
+test.server > xrpld.rpc
 test.server > xrpl.json
-test.server > xrpl.protocol
 test.server > xrpl.server
 test.shamap > test.unit_test
 test.shamap > xrpl.basics
@@ -249,20 +239,19 @@ xrpld.consensus > xrpl.ledger
 xrpld.consensus > xrpl.protocol
 xrpld.core > xrpl.basics
 xrpld.core > xrpl.core
+xrpld.core > xrpl.json
 xrpld.core > xrpl.net
 xrpld.core > xrpl.protocol
 xrpld.core > xrpl.rdb
 xrpld.overlay > xrpl.basics
 xrpld.overlay > xrpl.core
-xrpld.overlay > xrpld.consensus
 xrpld.overlay > xrpld.core
 xrpld.overlay > xrpld.peerfinder
 xrpld.overlay > xrpl.json
-xrpld.overlay > xrpl.ledger
 xrpld.overlay > xrpl.protocol
+xrpld.overlay > xrpl.rdb
 xrpld.overlay > xrpl.resource
 xrpld.overlay > xrpl.server
-xrpld.overlay > xrpl.shamap
 xrpld.overlay > xrpl.tx
 xrpld.peerfinder > xrpl.basics
 xrpld.peerfinder > xrpld.core
@@ -272,7 +261,6 @@ xrpld.perflog > xrpl.basics
 xrpld.perflog > xrpl.core
 xrpld.perflog > xrpld.rpc
 xrpld.perflog > xrpl.json
-xrpld.perflog > xrpl.protocol
 xrpld.rpc > xrpl.basics
 xrpld.rpc > xrpl.core
 xrpld.rpc > xrpld.core
@@ -284,9 +272,5 @@ xrpld.rpc > xrpl.protocol
 xrpld.rpc > xrpl.rdb
 xrpld.rpc > xrpl.resource
 xrpld.rpc > xrpl.server
-xrpld.rpc > xrpl.shamap
 xrpld.rpc > xrpl.tx
-xrpld.shamap > xrpl.basics
-xrpld.shamap > xrpld.core
-xrpld.shamap > xrpl.protocol
 xrpld.shamap > xrpl.shamap
--- a/.github/workflows/publish-docs.yml
+++ b/.github/workflows/publish-docs.yml
@@ -82,7 +82,7 @@ jobs:

      - name: Create documentation artifact
        if: ${{ github.event.repository.visibility == 'public' && github.event_name == 'push' }}
-        uses: actions/upload-pages-artifact@fc324d3547104276b827a68afc52ff2a11cc49c9 # v5.0.0
+        uses: actions/upload-pages-artifact@7b1f4a764d45c48632c6b24a0339c27f5614fb0b # v4.0.0
        with:
          path: ${{ env.BUILD_DIR }}/docs/html

--- a/.github/workflows/reusable-build-test-config.yml
+++ b/.github/workflows/reusable-build-test-config.yml
@@ -153,32 +153,6 @@ jobs:
            ${CMAKE_ARGS} \
            ..

-      - name: Check protocol autogen files are up-to-date
-        working-directory: ${{ env.BUILD_DIR }}
-        env:
-          MESSAGE: |
-
-            The generated protocol wrapper classes are out of date.
-
-            This typically happens when the macro files or generator scripts
-            have changed but the generated files were not regenerated.
-
-            To fix this:
-              1. Run: cmake --build . --target setup_code_gen
-              2. Run: cmake --build . --target code_gen
-              3. Commit and push the regenerated files
-        run: |
-          set -e
-          cmake --build . --target setup_code_gen
-          cmake --build . --target code_gen
-          DIFF=$(git -C .. status --porcelain -- include/xrpl/protocol_autogen src/tests/libxrpl/protocol_autogen)
-          if [ -n "${DIFF}" ]; then
-            echo "::error::Generated protocol files are out of date"
-            git -C .. diff -- include/xrpl/protocol_autogen src/tests/libxrpl/protocol_autogen
-            echo "${MESSAGE}"
-            exit 1
-          fi
-
      - name: Build the binary
        working-directory: ${{ env.BUILD_DIR }}
        env:
@@ -192,6 +166,29 @@ jobs:
            --parallel "${BUILD_NPROC}" \
            --target "${CMAKE_TARGET}"

+      - name: Check protocol autogen files are up-to-date
+        env:
+          MESSAGE: |
+
+            The generated protocol wrapper classes are out of date.
+
+            This typically happens when your branch is behind develop and
+            the macro files or generator scripts have changed.
+
+            To fix this:
+              1. Update your branch from develop (merge or rebase)
+              2. Build with code generation enabled (XRPL_NO_CODEGEN=OFF)
+              3. Commit and push the regenerated files
+        run: |
+          set -e
+          DIFF=$(git status --porcelain -- include/xrpl/protocol_autogen src/tests/libxrpl/protocol_autogen)
+          if [ -n "${DIFF}" ]; then
+            echo "::error::Generated protocol files are out of date"
+            git diff -- include/xrpl/protocol_autogen src/tests/libxrpl/protocol_autogen
+            echo "${MESSAGE}"
+            exit 1
+          fi
+
      - name: Show ccache statistics
        if: ${{ inputs.ccache_enabled }}
        run: |
@@ -203,7 +200,7 @@ jobs:

      - name: Upload the binary (Linux)
        if: ${{ github.event.repository.visibility == 'public' && runner.os == 'Linux' }}
-        uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1
+        uses: actions/upload-artifact@bbbca2ddaa5d8feaa63e36b76fdaad77386f024f # v7.0.0
        with:
          name: xrpld-${{ inputs.config_name }}
          path: ${{ env.BUILD_DIR }}/xrpld
--- a/.github/workflows/reusable-clang-tidy-files.yml
+++ b/.github/workflows/reusable-clang-tidy-files.yml
@@ -84,7 +84,7 @@ jobs:

      - name: Upload clang-tidy output
        if: ${{ github.event.repository.visibility == 'public' && steps.run_clang_tidy.outcome != 'success' }}
-        uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1
+        uses: actions/upload-artifact@bbbca2ddaa5d8feaa63e36b76fdaad77386f024f # v7.0.0
        with:
          name: clang-tidy-results
          path: clang-tidy-output.txt
--- a/.gitignore
+++ b/.gitignore
@@ -13,7 +13,6 @@
 Debug/
 Release/
 /.build/
-/.venv/
 /build/
 /db/
 /out.txt
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -17,6 +17,7 @@ repos:
        args: [--maxkb=400, --enforce-all]
      - id: trailing-whitespace
      - id: end-of-file-fixer
+      - id: mixed-line-ending
      - id: check-merge-conflict
        args: [--assume-in-merge]

@@ -37,7 +38,6 @@ repos:
    rev: c2bc67fe8f8f549cc489e00ba8b45aa18ee713b1 # frozen: v3.8.1
    hooks:
      - id: prettier
-        args: [--end-of-line=auto]

  - repo: https://github.com/psf/black-pre-commit-mirror
    rev: ea488cebbfd88a5f50b8bd95d5c829d0bb76feb8 # frozen: 26.1.0
--- a/BUILD.md
+++ b/BUILD.md
@@ -459,21 +459,6 @@ install ccache --version 4.11.3 --allow-downgrade`.
   The location of `xrpld` binary in your build directory depends on your
   CMake generator. Pass `--help` to see the rest of the command line options.

-## Code generation
-
-The protocol wrapper classes in `include/xrpl/protocol_autogen/` are generated
-from macro definition files in `include/xrpl/protocol/detail/`. If you modify
-the macro files (e.g. `transactions.macro`, `ledger_entries.macro`) or the
-generation scripts/templates in `cmake/scripts/codegen/`, you need to regenerate the
-files:
-
-```
-cmake --build . --target setup_code_gen  # create venv and install dependencies (once)
-cmake --build . --target code_gen        # regenerate code
-```
-
-The regenerated files should be committed alongside your changes.
-
 ## Coverage report

 The coverage report is intended for developers using compilers GCC
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -93,6 +93,7 @@ find_package(OpenSSL REQUIRED)
 find_package(secp256k1 REQUIRED)
 find_package(SOCI REQUIRED)
 find_package(SQLite3 REQUIRED)
+find_package(wasmi REQUIRED)
 find_package(xxHash REQUIRED)

 target_link_libraries(
@@ -132,7 +133,6 @@ if(coverage)
 endif()

 include(XrplCore)
-include(XrplProtocolAutogen)
 include(XrplInstall)
 include(XrplValidatorKeys)

--- a/cmake/XrplCore.cmake
+++ b/cmake/XrplCore.cmake
@@ -67,6 +67,7 @@ target_link_libraries(
        Xrpl::opts
        Xrpl::syslibs
        secp256k1::secp256k1
+        wasmi::wasmi
        xrpl.libpb
        xxHash::xxhash
        $<$<BOOL:${voidstar}>:antithesis-sdk-cpp>
@@ -108,12 +109,24 @@ target_link_libraries(
 )

 # Level 05
+## Set up code generation for protocol_autogen module
+include(XrplProtocolAutogen)
+# Must call setup_protocol_autogen before add_module so that:
+# 1. Stale generated files are cleared before GLOB runs
+# 2. Output file list is known for custom commands
+setup_protocol_autogen()
+
 add_module(xrpl protocol_autogen)
 target_link_libraries(
    xrpl.libxrpl.protocol_autogen
    PUBLIC xrpl.libxrpl.protocol
 )

+# Ensure code generation runs before compiling protocol_autogen
+if(TARGET protocol_autogen_generate)
+    add_dependencies(xrpl.libxrpl.protocol_autogen protocol_autogen_generate)
+endif()
+
 # Level 06
 add_module(xrpl core)
 target_link_libraries(
--- a/cmake/XrplProtocolAutogen.cmake
+++ b/cmake/XrplProtocolAutogen.cmake
@@ -2,145 +2,308 @@
   Protocol Autogen - Code generation for protocol wrapper classes
 #]===================================================================]

+# Options for code generation
+option(
+    XRPL_NO_CODEGEN
+    "Disable code generation (use pre-generated files from repository)"
+    OFF
+)
 set(CODEGEN_VENV_DIR
-    "${CMAKE_CURRENT_SOURCE_DIR}/.venv"
+    ""
    CACHE PATH
-    "Path to a Python virtual environment for code generation. A venv will be created here by setup_code_gen and used to run generation scripts."
+    "Path to Python virtual environment for code generation. If provided, automatic venv setup is skipped."
 )

-# Directory paths
-set(MACRO_DIR "${CMAKE_CURRENT_SOURCE_DIR}/include/xrpl/protocol/detail")
-set(AUTOGEN_HEADER_DIR
-    "${CMAKE_CURRENT_SOURCE_DIR}/include/xrpl/protocol_autogen"
-)
-set(AUTOGEN_TEST_DIR
-    "${CMAKE_CURRENT_SOURCE_DIR}/src/tests/libxrpl/protocol_autogen"
-)
-set(SCRIPTS_DIR "${CMAKE_CURRENT_SOURCE_DIR}/cmake/scripts/codegen")
-
-# Input macro files
-set(TRANSACTIONS_MACRO "${MACRO_DIR}/transactions.macro")
-set(LEDGER_ENTRIES_MACRO "${MACRO_DIR}/ledger_entries.macro")
-set(SFIELDS_MACRO "${MACRO_DIR}/sfields.macro")
-
-# Python scripts and templates
-set(GENERATE_TX_SCRIPT "${SCRIPTS_DIR}/generate_tx_classes.py")
-set(GENERATE_LEDGER_SCRIPT "${SCRIPTS_DIR}/generate_ledger_classes.py")
-set(REQUIREMENTS_FILE "${SCRIPTS_DIR}/requirements.txt")
-set(MACRO_PARSER_COMMON "${SCRIPTS_DIR}/macro_parser_common.py")
-set(TX_TEMPLATE "${SCRIPTS_DIR}/templates/Transaction.h.mako")
-set(TX_TEST_TEMPLATE "${SCRIPTS_DIR}/templates/TransactionTests.cpp.mako")
-set(LEDGER_TEMPLATE "${SCRIPTS_DIR}/templates/LedgerEntry.h.mako")
-set(LEDGER_TEST_TEMPLATE "${SCRIPTS_DIR}/templates/LedgerEntryTests.cpp.mako")
-set(ALL_INPUT_FILES
-    "${TRANSACTIONS_MACRO}"
-    "${LEDGER_ENTRIES_MACRO}"
-    "${SFIELDS_MACRO}"
-    "${GENERATE_TX_SCRIPT}"
-    "${GENERATE_LEDGER_SCRIPT}"
-    "${REQUIREMENTS_FILE}"
-    "${MACRO_PARSER_COMMON}"
-    "${TX_TEMPLATE}"
-    "${TX_TEST_TEMPLATE}"
-    "${LEDGER_TEMPLATE}"
-    "${LEDGER_TEST_TEMPLATE}"
-)
-
-# Create output directories
-file(MAKE_DIRECTORY "${AUTOGEN_HEADER_DIR}/transactions")
-file(MAKE_DIRECTORY "${AUTOGEN_HEADER_DIR}/ledger_entries")
-file(MAKE_DIRECTORY "${AUTOGEN_TEST_DIR}/ledger_entries")
-file(MAKE_DIRECTORY "${AUTOGEN_TEST_DIR}/transactions")
-
-# Find Python3
-if(NOT Python3_EXECUTABLE)
-    find_package(Python3 COMPONENTS Interpreter QUIET)
-endif()
-
-if(NOT Python3_EXECUTABLE)
-    find_program(Python3_EXECUTABLE NAMES python3 python)
-endif()
-
-if(NOT Python3_EXECUTABLE)
-    message(
-        WARNING
-        "Python3 not found. The 'code_gen' and 'setup_code_gen' targets will not be available."
+# Function to set up code generation for protocol_autogen module
+# This runs at configure time to generate C++ wrapper classes from macro files
+function(setup_protocol_autogen)
+    # Directory paths
+    set(MACRO_DIR "${CMAKE_CURRENT_SOURCE_DIR}/include/xrpl/protocol/detail")
+    set(AUTOGEN_HEADER_DIR
+        "${CMAKE_CURRENT_SOURCE_DIR}/include/xrpl/protocol_autogen"
    )
-    return()
-endif()
-
-# Warn if pip is configured with a non-default index (may need VPN).
-execute_process(
-    COMMAND ${Python3_EXECUTABLE} -m pip config get global.index-url
-    OUTPUT_VARIABLE PIP_INDEX_URL
-    OUTPUT_STRIP_TRAILING_WHITESPACE
-    ERROR_QUIET
-    RESULT_VARIABLE PIP_CONFIG_RESULT
-)
-if(PIP_CONFIG_RESULT EQUAL 0 AND PIP_INDEX_URL)
-    if(
-        NOT PIP_INDEX_URL STREQUAL "https://pypi.org/simple"
-        AND NOT PIP_INDEX_URL STREQUAL "https://pypi.python.org/simple"
+    set(AUTOGEN_TEST_DIR
+        "${CMAKE_CURRENT_SOURCE_DIR}/src/tests/libxrpl/protocol_autogen"
    )
+    set(SCRIPTS_DIR "${CMAKE_CURRENT_SOURCE_DIR}/scripts")
+
+    # Input macro files
+    set(TRANSACTIONS_MACRO "${MACRO_DIR}/transactions.macro")
+    set(LEDGER_ENTRIES_MACRO "${MACRO_DIR}/ledger_entries.macro")
+    set(SFIELDS_MACRO "${MACRO_DIR}/sfields.macro")
+
+    # Python scripts and templates
+    set(GENERATE_TX_SCRIPT "${SCRIPTS_DIR}/generate_tx_classes.py")
+    set(GENERATE_LEDGER_SCRIPT "${SCRIPTS_DIR}/generate_ledger_classes.py")
+    set(REQUIREMENTS_FILE "${SCRIPTS_DIR}/requirements.txt")
+    set(MACRO_PARSER_COMMON "${SCRIPTS_DIR}/macro_parser_common.py")
+    set(TX_TEMPLATE "${SCRIPTS_DIR}/templates/Transaction.h.mako")
+    set(TX_TEST_TEMPLATE "${SCRIPTS_DIR}/templates/TransactionTests.cpp.mako")
+    set(LEDGER_TEMPLATE "${SCRIPTS_DIR}/templates/LedgerEntry.h.mako")
+    set(LEDGER_TEST_TEMPLATE
+        "${SCRIPTS_DIR}/templates/LedgerEntryTests.cpp.mako"
+    )
+
+    # Check if code generation is disabled
+    if(XRPL_NO_CODEGEN)
        message(
            WARNING
-            "Private pip index URL detected: ${PIP_INDEX_URL}\n"
-            "You may need to connect to VPN to access this URL."
+            "Protocol autogen: Code generation is disabled (XRPL_NO_CODEGEN=ON). "
+            "Generated files may be out of date."
+        )
+        return()
+    endif()
+
+    # Create output directories
+    file(MAKE_DIRECTORY "${AUTOGEN_HEADER_DIR}/transactions")
+    file(MAKE_DIRECTORY "${AUTOGEN_HEADER_DIR}/ledger_entries")
+    file(MAKE_DIRECTORY "${AUTOGEN_TEST_DIR}/ledger_entries")
+    file(MAKE_DIRECTORY "${AUTOGEN_TEST_DIR}/transactions")
+
+    # Find Python3 - check if already found by Conan or find it ourselves
+    if(NOT Python3_EXECUTABLE)
+        find_package(Python3 COMPONENTS Interpreter QUIET)
+    endif()
+
+    if(NOT Python3_EXECUTABLE)
+        # Try finding python3 executable directly
+        find_program(Python3_EXECUTABLE NAMES python3 python)
+    endif()
+
+    if(NOT Python3_EXECUTABLE)
+        message(
+            FATAL_ERROR
+            "Python3 not found. Code generation cannot proceed.\n"
+            "Please install Python 3, or set -DXRPL_NO_CODEGEN=ON to use existing generated files."
+        )
+        return()
+    endif()
+
+    message(STATUS "Using Python3 for code generation: ${Python3_EXECUTABLE}")
+
+    # Set up Python virtual environment for code generation
+    if(CODEGEN_VENV_DIR)
+        # User-provided venv - skip automatic setup
+        set(VENV_DIR "${CODEGEN_VENV_DIR}")
+        message(STATUS "Using user-provided Python venv: ${VENV_DIR}")
+    else()
+        # Use default venv in build directory
+        set(VENV_DIR "${CMAKE_CURRENT_BINARY_DIR}/codegen_venv")
+    endif()
+
+    # Determine the Python executable path in the venv
+    if(WIN32)
+        set(VENV_PYTHON "${VENV_DIR}/Scripts/python.exe")
+        set(VENV_PIP "${VENV_DIR}/Scripts/pip.exe")
+    else()
+        set(VENV_PYTHON "${VENV_DIR}/bin/python")
+        set(VENV_PIP "${VENV_DIR}/bin/pip")
+    endif()
+
+    # Only auto-setup venv if not user-provided
+    if(NOT CODEGEN_VENV_DIR)
+        # Check if venv needs to be created or updated
+        set(VENV_NEEDS_UPDATE FALSE)
+        if(NOT EXISTS "${VENV_PYTHON}")
+            set(VENV_NEEDS_UPDATE TRUE)
+            message(
+                STATUS
+                "Creating Python virtual environment for code generation..."
+            )
+        elseif(
+            "${REQUIREMENTS_FILE}"
+                IS_NEWER_THAN
+                "${VENV_DIR}/.requirements_installed"
+        )
+            set(VENV_NEEDS_UPDATE TRUE)
+            message(
+                STATUS
+                "Updating Python virtual environment (requirements changed)..."
+            )
+        endif()
+
+        # Create/update virtual environment if needed
+        if(VENV_NEEDS_UPDATE)
+            message(
+                STATUS
+                "Setting up Python virtual environment at ${VENV_DIR}"
+            )
+            execute_process(
+                COMMAND ${Python3_EXECUTABLE} -m venv "${VENV_DIR}"
+                RESULT_VARIABLE VENV_RESULT
+                ERROR_VARIABLE VENV_ERROR
+            )
+            if(NOT VENV_RESULT EQUAL 0)
+                message(
+                    FATAL_ERROR
+                    "Failed to create virtual environment: ${VENV_ERROR}"
+                )
+            endif()
+
+            # Check pip index URL configuration
+            execute_process(
+                COMMAND ${VENV_PIP} config get global.index-url
+                OUTPUT_VARIABLE PIP_INDEX_URL
+                OUTPUT_STRIP_TRAILING_WHITESPACE
+                ERROR_QUIET
+            )
+
+            # Default PyPI URL
+            set(DEFAULT_PIP_INDEX "https://pypi.org/simple")
+
+            # Show warning if using non-default index
+            if(PIP_INDEX_URL AND NOT PIP_INDEX_URL STREQUAL "")
+                if(NOT PIP_INDEX_URL STREQUAL DEFAULT_PIP_INDEX)
+                    message(
+                        WARNING
+                        "Private pip index URL detected: ${PIP_INDEX_URL}\n"
+                        "You may need to connect to VPN to access this URL."
+                    )
+                endif()
+            endif()
+
+            message(STATUS "Installing Python dependencies...")
+            execute_process(
+                COMMAND ${VENV_PIP} install --upgrade pip
+                RESULT_VARIABLE PIP_UPGRADE_RESULT
+                OUTPUT_QUIET
+                ERROR_VARIABLE PIP_UPGRADE_ERROR
+            )
+            if(NOT PIP_UPGRADE_RESULT EQUAL 0)
+                message(WARNING "Failed to upgrade pip: ${PIP_UPGRADE_ERROR}")
+            endif()
+
+            execute_process(
+                COMMAND ${VENV_PIP} install -r "${REQUIREMENTS_FILE}"
+                RESULT_VARIABLE PIP_INSTALL_RESULT
+                ERROR_VARIABLE PIP_INSTALL_ERROR
+            )
+            if(NOT PIP_INSTALL_RESULT EQUAL 0)
+                message(
+                    FATAL_ERROR
+                    "Failed to install Python dependencies: ${PIP_INSTALL_ERROR}"
+                )
+            endif()
+
+            # Mark requirements as installed
+            file(TOUCH "${VENV_DIR}/.requirements_installed")
+            message(STATUS "Python virtual environment ready")
+        endif()
+    endif()
+
+    # At configure time - get list of output files for transactions
+    execute_process(
+        COMMAND
+            ${VENV_PYTHON} "${GENERATE_TX_SCRIPT}" "${TRANSACTIONS_MACRO}"
+            --header-dir "${AUTOGEN_HEADER_DIR}/transactions" --test-dir
+            "${AUTOGEN_TEST_DIR}/transactions" --list-outputs
+        OUTPUT_VARIABLE TX_OUTPUT_FILES
+        OUTPUT_STRIP_TRAILING_WHITESPACE
+        RESULT_VARIABLE TX_LIST_RESULT
+        ERROR_VARIABLE TX_LIST_ERROR
+    )
+    if(NOT TX_LIST_RESULT EQUAL 0)
+        message(
+            FATAL_ERROR
+            "Failed to list transaction output files:\n${TX_LIST_ERROR}"
        )
    endif()
-endif()
+    # Convert newline-separated list to CMake list
+    string(REPLACE "\\" "/" TX_OUTPUT_FILES "${TX_OUTPUT_FILES}")
+    string(REPLACE "\n" ";" TX_OUTPUT_FILES "${TX_OUTPUT_FILES}")

-# Determine which Python interpreter to use for code generation.
-if(CODEGEN_VENV_DIR)
-    if(WIN32)
-        set(CODEGEN_PYTHON "${CODEGEN_VENV_DIR}/Scripts/python.exe")
-    else()
-        set(CODEGEN_PYTHON "${CODEGEN_VENV_DIR}/bin/python")
+    # At configure time - get list of output files for ledger entries
+    execute_process(
+        COMMAND
+            ${VENV_PYTHON} "${GENERATE_LEDGER_SCRIPT}" "${LEDGER_ENTRIES_MACRO}"
+            --header-dir "${AUTOGEN_HEADER_DIR}/ledger_entries" --test-dir
+            "${AUTOGEN_TEST_DIR}/ledger_entries" --list-outputs
+        OUTPUT_VARIABLE LEDGER_OUTPUT_FILES
+        OUTPUT_STRIP_TRAILING_WHITESPACE
+        RESULT_VARIABLE LEDGER_LIST_RESULT
+        ERROR_VARIABLE LEDGER_LIST_ERROR
+    )
+    if(NOT LEDGER_LIST_RESULT EQUAL 0)
+        message(
+            FATAL_ERROR
+            "Failed to list ledger entry output files:\n${LEDGER_LIST_ERROR}"
+        )
    endif()
-else()
-    set(CODEGEN_PYTHON "${Python3_EXECUTABLE}")
-    message(
-        WARNING
-        "CODEGEN_VENV_DIR is not set. Dependencies will be installed globally.\n"
-        "If this is not intended, reconfigure with:\n"
-        "  cmake . -UCODEGEN_VENV_DIR"
-    )
-endif()
+    # Convert newline-separated list to CMake list
+    string(REPLACE "\\" "/" LEDGER_OUTPUT_FILES "${LEDGER_OUTPUT_FILES}")
+    string(REPLACE "\n" ";" LEDGER_OUTPUT_FILES "${LEDGER_OUTPUT_FILES}")

-# Custom target to create a venv and install Python dependencies.
-# Run manually with: cmake --build . --target setup_code_gen
-if(CODEGEN_VENV_DIR)
-    add_custom_target(
-        setup_code_gen
-        COMMAND ${Python3_EXECUTABLE} -m venv "${CODEGEN_VENV_DIR}"
-        COMMAND ${CODEGEN_PYTHON} -m pip install -r "${REQUIREMENTS_FILE}"
+    # Custom command to generate transaction classes at build time
+    add_custom_command(
+        OUTPUT ${TX_OUTPUT_FILES}
+        COMMAND
+            ${VENV_PYTHON} "${GENERATE_TX_SCRIPT}" "${TRANSACTIONS_MACRO}"
+            --header-dir "${AUTOGEN_HEADER_DIR}/transactions" --test-dir
+            "${AUTOGEN_TEST_DIR}/transactions" --sfields-macro
+            "${SFIELDS_MACRO}"
        WORKING_DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}"
-        COMMENT "Creating venv and installing code generation dependencies..."
+        DEPENDS
+            "${TRANSACTIONS_MACRO}"
+            "${SFIELDS_MACRO}"
+            "${GENERATE_TX_SCRIPT}"
+            "${MACRO_PARSER_COMMON}"
+            "${TX_TEMPLATE}"
+            "${TX_TEST_TEMPLATE}"
+            "${REQUIREMENTS_FILE}"
+        COMMENT "Generating transaction classes from transactions.macro..."
+        VERBATIM
    )
-else()
-    add_custom_target(
-        setup_code_gen
-        COMMAND ${Python3_EXECUTABLE} -m pip install -r "${REQUIREMENTS_FILE}"
-        WORKING_DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}"
-        COMMENT "Installing code generation dependencies..."
-    )
-endif()

-# Custom target for code generation, excluded from ALL.
-# Run manually with: cmake --build . --target code_gen
-add_custom_target(
-    code_gen
-    COMMAND
-        ${CMAKE_COMMAND} -DCODEGEN_PYTHON=${CODEGEN_PYTHON}
-        -DGENERATE_TX_SCRIPT=${GENERATE_TX_SCRIPT}
-        -DGENERATE_LEDGER_SCRIPT=${GENERATE_LEDGER_SCRIPT}
-        -DTRANSACTIONS_MACRO=${TRANSACTIONS_MACRO}
-        -DLEDGER_ENTRIES_MACRO=${LEDGER_ENTRIES_MACRO}
-        -DSFIELDS_MACRO=${SFIELDS_MACRO}
-        -DAUTOGEN_HEADER_DIR=${AUTOGEN_HEADER_DIR}
-        -DAUTOGEN_TEST_DIR=${AUTOGEN_TEST_DIR} -P
-        "${CMAKE_CURRENT_SOURCE_DIR}/cmake/XrplProtocolAutogenRun.cmake"
-    WORKING_DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}"
-    COMMENT "Running protocol code generation..."
-    SOURCES ${ALL_INPUT_FILES}
-)
+    # Custom command to generate ledger entry classes at build time
+    add_custom_command(
+        OUTPUT ${LEDGER_OUTPUT_FILES}
+        COMMAND
+            ${VENV_PYTHON} "${GENERATE_LEDGER_SCRIPT}" "${LEDGER_ENTRIES_MACRO}"
+            --header-dir "${AUTOGEN_HEADER_DIR}/ledger_entries" --test-dir
+            "${AUTOGEN_TEST_DIR}/ledger_entries" --sfields-macro
+            "${SFIELDS_MACRO}"
+        WORKING_DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}"
+        DEPENDS
+            "${LEDGER_ENTRIES_MACRO}"
+            "${SFIELDS_MACRO}"
+            "${GENERATE_LEDGER_SCRIPT}"
+            "${MACRO_PARSER_COMMON}"
+            "${LEDGER_TEMPLATE}"
+            "${LEDGER_TEST_TEMPLATE}"
+            "${REQUIREMENTS_FILE}"
+        COMMENT "Generating ledger entry classes from ledger_entries.macro..."
+        VERBATIM
+    )
+
+    # Create a custom target that depends on all generated files
+    add_custom_target(
+        protocol_autogen_generate
+        DEPENDS ${TX_OUTPUT_FILES} ${LEDGER_OUTPUT_FILES}
+        COMMENT "Protocol autogen code generation"
+    )
+
+    # Extract test files from output lists (files ending in Tests.cpp)
+    set(PROTOCOL_AUTOGEN_TEST_SOURCES "")
+    foreach(FILE ${TX_OUTPUT_FILES} ${LEDGER_OUTPUT_FILES})
+        if(FILE MATCHES "Tests\\.cpp$")
+            list(APPEND PROTOCOL_AUTOGEN_TEST_SOURCES "${FILE}")
+        endif()
+    endforeach()
+    # Export test sources to parent scope for use in test CMakeLists.txt
+    set(PROTOCOL_AUTOGEN_TEST_SOURCES
+        "${PROTOCOL_AUTOGEN_TEST_SOURCES}"
+        CACHE INTERNAL
+        "Generated protocol_autogen test sources"
+    )
+
+    # Register dependencies so CMake reconfigures when macro files change
+    # (to update the list of output files)
+    set_property(
+        DIRECTORY
+        APPEND
+        PROPERTY
+            CMAKE_CONFIGURE_DEPENDS
+                "${TRANSACTIONS_MACRO}"
+                "${LEDGER_ENTRIES_MACRO}"
+    )
+endfunction()
--- a/cmake/XrplProtocolAutogenRun.cmake
+++ b/cmake/XrplProtocolAutogenRun.cmake
@@ -1,39 +0,0 @@
-#[===================================================================[
-   Protocol Autogen - Run script invoked by the 'code_gen' target
-#]===================================================================]
-
-# Generate transaction classes.
-execute_process(
-    COMMAND
-        ${CODEGEN_PYTHON} "${GENERATE_TX_SCRIPT}" "${TRANSACTIONS_MACRO}"
-        --header-dir "${AUTOGEN_HEADER_DIR}/transactions" --test-dir
-        "${AUTOGEN_TEST_DIR}/transactions" --sfields-macro "${SFIELDS_MACRO}"
-    RESULT_VARIABLE TX_RESULT
-    OUTPUT_VARIABLE TX_OUTPUT
-    ERROR_VARIABLE TX_ERROR
-)
-if(NOT TX_RESULT EQUAL 0)
-    message(
-        FATAL_ERROR
-        "Transaction code generation failed:\n${TX_OUTPUT}\n${TX_ERROR}\n${TX_RESULT}"
-    )
-endif()
-
-# Generate ledger entry classes.
-execute_process(
-    COMMAND
-        ${CODEGEN_PYTHON} "${GENERATE_LEDGER_SCRIPT}" "${LEDGER_ENTRIES_MACRO}"
-        --header-dir "${AUTOGEN_HEADER_DIR}/ledger_entries" --test-dir
-        "${AUTOGEN_TEST_DIR}/ledger_entries" --sfields-macro "${SFIELDS_MACRO}"
-    RESULT_VARIABLE LEDGER_RESULT
-    OUTPUT_VARIABLE LEDGER_OUTPUT
-    ERROR_VARIABLE LEDGER_ERROR
-)
-if(NOT LEDGER_RESULT EQUAL 0)
-    message(
-        FATAL_ERROR
-        "Ledger entry code generation failed:\n${LEDGER_OUTPUT}\n${LEDGER_ERROR}\n${TX_RESULT}"
-    )
-endif()
-
-message(STATUS "Protocol autogen: code generation complete")
--- a/conan.lock
+++ b/conan.lock
@@ -3,6 +3,7 @@
    "requires": [
        "zlib/1.3.1#cac0f6daea041b0ccf42934163defb20%1774439233.809",
        "xxhash/0.8.3#681d36a0a6111fc56e5e45ea182c19cc%1765850149.987",
+        "wasmi/1.0.9#1fecdab9b90c96698eb35ea99ca4f5cb%1772227278.324",
        "sqlite3/3.51.0#66aa11eabd0e34954c5c1c061ad44abe%1774467355.988",
        "soci/4.0.3#fe32b9ad5eb47e79ab9e45a68f363945%1774450067.231",
        "snappy/1.1.10#968fef506ff261592ec30c574d4a7809%1765850147.878",
@@ -11,7 +12,7 @@
        "re2/20251105#8579cfd0bda4daf0683f9e3898f964b4%1774398111.888",
        "protobuf/6.33.5#d96d52ba5baaaa532f47bda866ad87a5%1774467363.12",
        "openssl/3.6.1#e6399de266349245a4542fc5f6c71552%1774458290.139",
-        "nudb/2.0.9#11149c73f8f2baff9a0198fe25971fc7%1774883011.384",
+        "nudb/2.0.9#11149c73f8f2baff9a0198fe25971fc7%1775040983.408",
        "lz4/1.10.0#59fc63cac7f10fbe8e05c7e62c2f3504%1765850143.914",
        "libiconv/1.17#1e65319e945f2d31941a9d28cc13c058%1765842973.492",
        "libbacktrace/cci.20210118#a7691bfccd8caaf66309df196790a5a1%1765842973.03",
--- a/conanfile.py
+++ b/conanfile.py
@@ -35,6 +35,7 @@ class Xrpl(ConanFile):
        "openssl/3.6.1",
        "secp256k1/0.7.1",
        "soci/4.0.3",
+        "wasmi/1.0.9",
        "zlib/1.3.1",
    ]

@@ -220,6 +221,7 @@ class Xrpl(ConanFile):
            "soci::soci",
            "secp256k1::secp256k1",
            "sqlite3::sqlite",
+            "wasmi::wasmi",
            "xxhash::xxhash",
            "zlib::zlib",
        ]
--- a/include/xrpl/basics/BasicConfig.h
+++ b/include/xrpl/basics/BasicConfig.h
@@ -67,13 +67,9 @@ public:
    legacy(std::string value)
    {
        if (lines_.empty())
-        {
            lines_.emplace_back(std::move(value));
-        }
        else
-        {
            lines_[0] = std::move(value);
-        }
    }

    /**
@@ -88,10 +84,8 @@ public:
        if (lines_.empty())
            return "";
        if (lines_.size() > 1)
-        {
            Throw<std::runtime_error>(
                "A legacy value must have exactly one line. Section: " + name_);
-        }
        return lines_[0];
    }

@@ -302,7 +296,7 @@ set(T& target, std::string const& name, Section const& section)
        if ((found_and_valid = val.has_value()))
            target = *val;
    }
-    catch (boost::bad_lexical_cast const&)  // NOLINT(bugprone-empty-catch)
+    catch (boost::bad_lexical_cast&)
    {
    }
    return found_and_valid;
@@ -336,7 +330,7 @@ get(Section const& section, std::string const& name, T const& defaultValue = T{}
    {
        return section.value_or<T>(name, defaultValue);
    }
-    catch (boost::bad_lexical_cast const&)  // NOLINT(bugprone-empty-catch)
+    catch (boost::bad_lexical_cast&)
    {
    }
    return defaultValue;
@@ -351,7 +345,7 @@ get(Section const& section, std::string const& name, char const* defaultValue)
        if (val.has_value())
            return *val;
    }
-    catch (boost::bad_lexical_cast const&)  // NOLINT(bugprone-empty-catch)
+    catch (boost::bad_lexical_cast&)
    {
    }
    return defaultValue;
--- a/include/xrpl/basics/Buffer.h
+++ b/include/xrpl/basics/Buffer.h
@@ -24,8 +24,7 @@ public:
    Buffer() = default;

    /** Create an uninitialized buffer with the given size. */
-    explicit Buffer(std::size_t size)
-        : p_((size != 0u) ? new std::uint8_t[size] : nullptr), size_(size)
+    explicit Buffer(std::size_t size) : p_(size ? new std::uint8_t[size] : nullptr), size_(size)
    {
    }

@@ -37,7 +36,7 @@ public:
    */
    Buffer(void const* data, std::size_t size) : Buffer(size)
    {
-        if (size != 0u)
+        if (size)
            std::memcpy(p_.get(), data, size);
    }

@@ -115,7 +114,7 @@ public:

    operator Slice() const noexcept
    {
-        if (size_ == 0u)
+        if (!size_)
            return Slice{};
        return Slice{p_.get(), size_};
    }
@@ -156,7 +155,7 @@ public:
    {
        if (n != size_)
        {
-            p_.reset((n != 0u) ? new std::uint8_t[n] : nullptr);
+            p_.reset(n ? new std::uint8_t[n] : nullptr);
            size_ = n;
        }
        return p_.get();
@@ -200,7 +199,7 @@ operator==(Buffer const& lhs, Buffer const& rhs) noexcept
    if (lhs.size() != rhs.size())
        return false;

-    if (lhs.empty())
+    if (lhs.size() == 0)
        return true;

    return std::memcmp(lhs.data(), rhs.data(), lhs.size()) == 0;
--- a/include/xrpl/basics/CompressionAlgorithms.h
+++ b/include/xrpl/basics/CompressionAlgorithms.h
@@ -68,15 +68,12 @@ lz4Decompress(
    if (decompressedSize <= 0)
        Throw<std::runtime_error>("lz4Decompress: integer overflow (output)");

-    // NOLINTNEXTLINE(readability-suspicious-call-argument)
    if (LZ4_decompress_safe(
            reinterpret_cast<char const*>(in),
            reinterpret_cast<char*>(decompressed),
            inSize,
            decompressedSize) != decompressedSize)
-    {
        Throw<std::runtime_error>("lz4Decompress: failed");
-    }

    return decompressedSize;
 }
--- a/include/xrpl/basics/CountedObject.h
+++ b/include/xrpl/basics/CountedObject.h
@@ -112,6 +112,7 @@ private:
        return c;
    }

+public:
    CountedObject() noexcept
    {
        getCounter().increment();
@@ -125,13 +126,10 @@ private:
    CountedObject&
    operator=(CountedObject const&) noexcept = default;

-public:
    ~CountedObject() noexcept
    {
        getCounter().decrement();
    }
-
-    friend Object;
 };

 }  // namespace xrpl
--- a/include/xrpl/basics/DecayingSample.h
+++ b/include/xrpl/basics/DecayingSample.h
@@ -67,7 +67,7 @@ private:
            }
            else
            {
-                while ((elapsed--) != 0u)
+                while (elapsed--)
                    m_value -= (m_value + Window - 1) / Window;
            }
        }
--- a/include/xrpl/basics/IntrusiveRefCounts.h
+++ b/include/xrpl/basics/IntrusiveRefCounts.h
@@ -247,7 +247,7 @@ IntrusiveRefCounts::releaseStrongRef() const

    using enum ReleaseStrongRefAction;
    auto prevIntVal = refCounts.load(std::memory_order_acquire);
-    while (true)
+    while (1)
    {
        RefCountPair const prevVal{prevIntVal};
        XRPL_ASSERT(
@@ -298,7 +298,7 @@ IntrusiveRefCounts::addWeakReleaseStrongRef() const
    // Note: If this becomes a perf bottleneck, the `partialDestroyStartedMask`
    // may be able to be set non-atomically. But it is easier to reason about
    // the code if the flag is set atomically.
-    while (true)
+    while (1)
    {
        RefCountPair const prevVal{prevIntVal};
        // Converted the last strong pointer to a weak pointer.
@@ -343,7 +343,7 @@ IntrusiveRefCounts::releaseWeakRef() const
    RefCountPair prev = prevIntVal;
    if (prev.weak == 1 && prev.strong == 0)
    {
-        if (prev.partialDestroyStartedBit == 0u)
+        if (!prev.partialDestroyStartedBit)
        {
            // This case should only be hit if the partialDestroyStartedBit is
            // set non-atomically (and even then very rarely). The code is kept
@@ -352,7 +352,7 @@ IntrusiveRefCounts::releaseWeakRef() const
            prevIntVal = refCounts.load(std::memory_order_acquire);
            prev = RefCountPair{prevIntVal};
        }
-        if (prev.partialDestroyFinishedBit == 0u)
+        if (!prev.partialDestroyFinishedBit)
        {
            // partial destroy MUST finish before running a full destroy (when
            // using weak pointers)
@@ -372,7 +372,7 @@ IntrusiveRefCounts::checkoutStrongRefFromWeak() const noexcept
    while (!refCounts.compare_exchange_weak(curValue, desiredValue, std::memory_order_acq_rel))
    {
        RefCountPair const prev{curValue};
-        if (prev.strong == 0u)
+        if (!prev.strong)
            return false;

        desiredValue = curValue + strongDelta;
--- a/include/xrpl/basics/LocalValue.h
+++ b/include/xrpl/basics/LocalValue.h
@@ -42,10 +42,10 @@ struct LocalValues
    // Keys are the address of a LocalValue.
    std::unordered_map<void const*, std::unique_ptr<BasicValue>> values;

-    static void
+    static inline void
    cleanup(LocalValues* lvs)
    {
-        if ((lvs != nullptr) && !lvs->onCoro)
+        if (lvs && !lvs->onCoro)
            delete lvs;
    }
 };
@@ -89,7 +89,7 @@ T&
 LocalValue<T>::operator*()
 {
    auto lvs = detail::getLocalValues().get();
-    if (lvs == nullptr)
+    if (!lvs)
    {
        lvs = new detail::LocalValues();
        lvs->onCoro = false;
--- a/include/xrpl/basics/Log.h
+++ b/include/xrpl/basics/Log.h
@@ -226,7 +226,7 @@ private:
 // expensive argument lists if the stream is not active.
 #ifndef JLOG
 #define JLOG(x) \
-    if (!(x))   \
+    if (!x)     \
    {           \
    }           \
    else        \
@@ -235,7 +235,7 @@ private:

 #ifndef CLOG
 #define CLOG(ss) \
-    if (!(ss))   \
+    if (!ss)     \
        ;        \
    else         \
        *ss
--- a/include/xrpl/basics/Number.h
+++ b/include/xrpl/basics/Number.h
@@ -78,7 +78,7 @@ struct MantissaRange
    }

    rep min;
-    rep max{(min * 10) - 1};
+    rep max{min * 10 - 1};
    int log;
    mantissa_scale scale;

@@ -342,9 +342,7 @@ public:
    constexpr int
    signum() const noexcept
    {
-        if (negative_)
-            return -1;
-        return (mantissa_ != 0u) ? 1 : 0;
+        return negative_ ? -1 : (mantissa_ ? 1 : 0);
    }

    Number
@@ -404,19 +402,19 @@ public:
    static void
    setMantissaScale(MantissaRange::mantissa_scale scale);

-    static internalrep
+    inline static internalrep
    minMantissa()
    {
        return range_.get().min;
    }

-    static internalrep
+    inline static internalrep
    maxMantissa()
    {
        return range_.get().max;
    }

-    static int
+    inline static int
    mantissaLog()
    {
        return range_.get().log;
@@ -509,12 +507,16 @@ private:
    class Guard;
 };

-constexpr Number::Number(bool negative, internalrep mantissa, int exponent, unchecked) noexcept
+inline constexpr Number::Number(
+    bool negative,
+    internalrep mantissa,
+    int exponent,
+    unchecked) noexcept
    : negative_(negative), mantissa_{mantissa}, exponent_{exponent}
 {
 }

-constexpr Number::Number(internalrep mantissa, int exponent, unchecked) noexcept
+inline constexpr Number::Number(internalrep mantissa, int exponent, unchecked) noexcept
    : Number(false, mantissa, exponent, unchecked{})
 {
 }
@@ -546,7 +548,7 @@ inline Number::Number(rep mantissa) : Number{mantissa, 0}
 * Please see the "---- External Interface ----" section of the class
 * documentation for an explanation of why the internal value may be modified.
 */
-constexpr Number::rep
+inline constexpr Number::rep
 Number::mantissa() const noexcept
 {
    auto m = mantissa_;
@@ -567,7 +569,7 @@ Number::mantissa() const noexcept
 * Please see the "---- External Interface ----" section of the class
 * documentation for an explanation of why the internal value may be modified.
 */
-constexpr int
+inline constexpr int
 Number::exponent() const noexcept
 {
    auto e = exponent_;
@@ -582,13 +584,13 @@ Number::exponent() const noexcept
    return e;
 }

-constexpr Number
+inline constexpr Number
 Number::operator+() const noexcept
 {
    return *this;
 }

-constexpr Number
+inline constexpr Number
 Number::operator-() const noexcept
 {
    if (mantissa_ == 0)
@@ -703,19 +705,17 @@ Number::normalizeToRange(T minMantissa, T maxMantissa) const
    int exponent = exponent_;

    if constexpr (std::is_unsigned_v<T>)
-    {
        XRPL_ASSERT_PARTS(
            !negative,
            "xrpl::Number::normalizeToRange",
            "Number is non-negative for unsigned range.");
-    }
    Number::normalize(negative, mantissa, exponent, minMantissa, maxMantissa);

    auto const sign = negative ? -1 : 1;
    return std::make_pair(static_cast<T>(sign * mantissa), exponent);
 }

-constexpr Number
+inline constexpr Number
 abs(Number x) noexcept
 {
    if (x < Number{})
@@ -746,7 +746,7 @@ power(Number const& f, unsigned n, unsigned d);

 // Return 0 if abs(x) < limit, else returns x

-constexpr Number
+inline constexpr Number
 squelch(Number const& x, Number const& limit) noexcept
 {
    if (abs(x) < limit)
--- a/include/xrpl/basics/RangeSet.h
+++ b/include/xrpl/basics/RangeSet.h
@@ -117,32 +117,22 @@ from_string(RangeSet<T>& rs, std::string const& s)
            case 1: {
                T front;
                if (!beast::lexicalCastChecked(front, intervals.front()))
-                {
                    result = false;
-                }
                else
-                {
                    rs.insert(front);
-                }
                break;
            }
            case 2: {
                T front;
                if (!beast::lexicalCastChecked(front, intervals.front()))
-                {
                    result = false;
-                }
                else
                {
                    T back;
                    if (!beast::lexicalCastChecked(back, intervals.back()))
-                    {
                        result = false;
-                    }
                    else
-                    {
                        rs.insert(range(front, back));
-                    }
                }
                break;
            }
--- a/include/xrpl/basics/SlabAllocator.h
+++ b/include/xrpl/basics/SlabAllocator.h
@@ -60,7 +60,7 @@ class SlabAllocator
            {
                // Use memcpy to avoid unaligned UB
                // (will optimize to equivalent code)
-                std::memcpy(data, static_cast<void const*>(&l_), sizeof(std::uint8_t*));
+                std::memcpy(data, &l_, sizeof(std::uint8_t*));
                l_ = data;
                data += item;
            }
@@ -98,11 +98,11 @@ class SlabAllocator

                ret = l_;

-                if (ret != nullptr)
+                if (ret)
                {
                    // Use memcpy to avoid unaligned UB
                    // (will optimize to equivalent code)
-                    std::memcpy(static_cast<void*>(&l_), ret, sizeof(std::uint8_t*));
+                    std::memcpy(&l_, ret, sizeof(std::uint8_t*));
                }
            }

@@ -127,7 +127,7 @@ class SlabAllocator

            // Use memcpy to avoid unaligned UB
            // (will optimize to equivalent code)
-            std::memcpy(ptr, static_cast<void const*>(&l_), sizeof(std::uint8_t*));
+            std::memcpy(ptr, &l_, sizeof(std::uint8_t*));
            l_ = ptr;
        }
    };
@@ -159,7 +159,7 @@ public:
        std::size_t extra,
        std::size_t alloc = 0,
        std::size_t align = 0)
-        : itemAlignment_((align != 0u) ? align : alignof(Type))
+        : itemAlignment_(align ? align : alignof(Type))
        , itemSize_(boost::alignment::align_up(sizeof(Type) + extra, itemAlignment_))
        , slabSize_(alloc)
    {
@@ -215,7 +215,7 @@ public:
        // We want to allocate the memory at a 2 MiB boundary, to make it
        // possible to use hugepage mappings on Linux:
        auto buf = boost::alignment::aligned_alloc(megabytes(std::size_t(2)), size);
-        if (buf == nullptr) [[unlikely]]
+        if (!buf) [[unlikely]]
            return nullptr;

 #if BOOST_OS_LINUX
@@ -235,7 +235,7 @@ public:

        // This operation is essentially guaranteed not to fail but
        // let's be careful anyways.
-        if (boost::alignment::align(itemAlignment_, itemSize_, slabData, slabSize) == nullptr)
+        if (!boost::alignment::align(itemAlignment_, itemSize_, slabData, slabSize))
        {
            boost::alignment::aligned_free(buf);
            return nullptr;
--- a/include/xrpl/basics/Slice.h
+++ b/include/xrpl/basics/Slice.h
@@ -183,7 +183,7 @@ operator==(Slice const& lhs, Slice const& rhs) noexcept
    if (lhs.size() != rhs.size())
        return false;

-    if (lhs.empty())
+    if (lhs.size() == 0)
        return true;

    return std::memcmp(lhs.data(), rhs.data(), lhs.size()) == 0;
--- a/include/xrpl/basics/algorithm.h
+++ b/include/xrpl/basics/algorithm.h
@@ -23,10 +23,8 @@ generalized_set_intersection(
 {
    while (first1 != last1 && first2 != last2)
    {
-        if (comp(*first1, *first2))
-        {              // if *first1 < *first2
-            ++first1;  //     then reduce first range
-        }
+        if (comp(*first1, *first2))  // if *first1 < *first2
+            ++first1;                //     then reduce first range
        else
        {
            if (!comp(*first2, *first1))   // if *first1 == *first2
--- a/include/xrpl/basics/base_uint.h
+++ b/include/xrpl/basics/base_uint.h
@@ -183,17 +183,11 @@ private:
                return ParseResult::badChar;

            if (c >= 'a')
-            {
                nibble = static_cast<std::uint32_t>(c - 'a' + 0xA);
-            }
            else if (c >= 'A')
-            {
                nibble = static_cast<std::uint32_t>(c - 'A' + 0xA);
-            }
            else if (c <= '9')
-            {
                nibble = static_cast<std::uint32_t>(c - '0');
-            }

            if (nibble > 0xFu)
                return ParseResult::badChar;
@@ -314,10 +308,8 @@ public:
    signum() const
    {
        for (int i = 0; i < WIDTH; i++)
-        {
            if (data_[i] != 0)
                return 1;
-        }

        return 0;
    }
@@ -398,7 +390,7 @@ public:
        return *this;
    }

-    base_uint
+    base_uint const
    operator++(int)
    {
        // postfix operator
@@ -423,7 +415,7 @@ public:
        return *this;
    }

-    base_uint
+    base_uint const
    operator--(int)
    {
        // postfix operator
@@ -452,7 +444,7 @@ public:
    {
        std::uint64_t carry = 0;

-        for (int i = WIDTH - 1; i >= 0; i--)
+        for (int i = WIDTH; i--;)
        {
            std::uint64_t const n = carry + boost::endian::big_to_native(data_[i]) +
                boost::endian::big_to_native(b.data_[i]);
@@ -540,7 +532,7 @@ using uint256 = base_uint<256>;
 using uint192 = base_uint<192>;

 template <std::size_t Bits, class Tag>
-[[nodiscard]] constexpr std::strong_ordering
+[[nodiscard]] inline constexpr std::strong_ordering
 operator<=>(base_uint<Bits, Tag> const& lhs, base_uint<Bits, Tag> const& rhs)
 {
    // This comparison might seem wrong on a casual inspection because it
@@ -561,7 +553,7 @@ operator<=>(base_uint<Bits, Tag> const& lhs, base_uint<Bits, Tag> const& rhs)
 }

 template <std::size_t Bits, typename Tag>
-[[nodiscard]] constexpr bool
+[[nodiscard]] inline constexpr bool
 operator==(base_uint<Bits, Tag> const& lhs, base_uint<Bits, Tag> const& rhs)
 {
    return (lhs <=> rhs) == 0;
@@ -569,7 +561,7 @@ operator==(base_uint<Bits, Tag> const& lhs, base_uint<Bits, Tag> const& rhs)

 //------------------------------------------------------------------------------
 template <std::size_t Bits, class Tag>
-constexpr bool
+inline constexpr bool
 operator==(base_uint<Bits, Tag> const& a, std::uint64_t b)
 {
    return a == base_uint<Bits, Tag>(b);
@@ -577,28 +569,28 @@ operator==(base_uint<Bits, Tag> const& a, std::uint64_t b)

 //------------------------------------------------------------------------------
 template <std::size_t Bits, class Tag>
-constexpr base_uint<Bits, Tag>
+inline constexpr base_uint<Bits, Tag>
 operator^(base_uint<Bits, Tag> const& a, base_uint<Bits, Tag> const& b)
 {
    return base_uint<Bits, Tag>(a) ^= b;
 }

 template <std::size_t Bits, class Tag>
-constexpr base_uint<Bits, Tag>
+inline constexpr base_uint<Bits, Tag>
 operator&(base_uint<Bits, Tag> const& a, base_uint<Bits, Tag> const& b)
 {
    return base_uint<Bits, Tag>(a) &= b;
 }

 template <std::size_t Bits, class Tag>
-constexpr base_uint<Bits, Tag>
+inline constexpr base_uint<Bits, Tag>
 operator|(base_uint<Bits, Tag> const& a, base_uint<Bits, Tag> const& b)
 {
    return base_uint<Bits, Tag>(a) |= b;
 }

 template <std::size_t Bits, class Tag>
-constexpr base_uint<Bits, Tag>
+inline constexpr base_uint<Bits, Tag>
 operator+(base_uint<Bits, Tag> const& a, base_uint<Bits, Tag> const& b)
 {
    return base_uint<Bits, Tag>(a) += b;
--- a/include/xrpl/basics/partitioned_unordered_map.h
+++ b/include/xrpl/basics/partitioned_unordered_map.h
@@ -231,8 +231,7 @@ public:
    {
        // Set partitions to the number of hardware threads if the parameter
        // is either empty or set to 0.
-        partitions_ =
-            partitions && (*partitions != 0u) ? *partitions : std::thread::hardware_concurrency();
+        partitions_ = partitions && *partitions ? *partitions : std::thread::hardware_concurrency();
        map_.resize(partitions_);
        XRPL_ASSERT(
            partitions_,
--- a/include/xrpl/basics/safe_cast.h
+++ b/include/xrpl/basics/safe_cast.h
@@ -17,7 +17,7 @@ concept SafeToCast = (std::is_integral_v<Src> && std::is_integral_v<Dest>) &&
                                                               : sizeof(Dest) >= sizeof(Src));

 template <class Dest, class Src>
-constexpr std::enable_if_t<std::is_integral_v<Dest> && std::is_integral_v<Src>, Dest>
+inline constexpr std::enable_if_t<std::is_integral_v<Dest> && std::is_integral_v<Src>, Dest>
 safe_cast(Src s) noexcept
 {
    static_assert(
@@ -30,14 +30,14 @@ safe_cast(Src s) noexcept
 }

 template <class Dest, class Src>
-constexpr std::enable_if_t<std::is_enum_v<Dest> && std::is_integral_v<Src>, Dest>
+inline constexpr std::enable_if_t<std::is_enum_v<Dest> && std::is_integral_v<Src>, Dest>
 safe_cast(Src s) noexcept
 {
    return static_cast<Dest>(safe_cast<std::underlying_type_t<Dest>>(s));
 }

 template <class Dest, class Src>
-constexpr std::enable_if_t<std::is_integral_v<Dest> && std::is_enum_v<Src>, Dest>
+inline constexpr std::enable_if_t<std::is_integral_v<Dest> && std::is_enum_v<Src>, Dest>
 safe_cast(Src s) noexcept
 {
    return safe_cast<Dest>(static_cast<std::underlying_type_t<Src>>(s));
@@ -48,7 +48,7 @@ safe_cast(Src s) noexcept
 // underlying types become safe, it can be converted to a safe_cast.

 template <class Dest, class Src>
-constexpr std::enable_if_t<std::is_integral_v<Dest> && std::is_integral_v<Src>, Dest>
+inline constexpr std::enable_if_t<std::is_integral_v<Dest> && std::is_integral_v<Src>, Dest>
 unsafe_cast(Src s) noexcept
 {
    static_assert(
@@ -59,14 +59,14 @@ unsafe_cast(Src s) noexcept
 }

 template <class Dest, class Src>
-constexpr std::enable_if_t<std::is_enum_v<Dest> && std::is_integral_v<Src>, Dest>
+inline constexpr std::enable_if_t<std::is_enum_v<Dest> && std::is_integral_v<Src>, Dest>
 unsafe_cast(Src s) noexcept
 {
    return static_cast<Dest>(unsafe_cast<std::underlying_type_t<Dest>>(s));
 }

 template <class Dest, class Src>
-constexpr std::enable_if_t<std::is_integral_v<Dest> && std::is_enum_v<Src>, Dest>
+inline constexpr std::enable_if_t<std::is_integral_v<Dest> && std::is_enum_v<Src>, Dest>
 unsafe_cast(Src s) noexcept
 {
    return unsafe_cast<Dest>(static_cast<std::underlying_type_t<Src>>(s));
--- a/include/xrpl/beast/asio/io_latency_probe.h
+++ b/include/xrpl/beast/asio/io_latency_probe.h
@@ -184,7 +184,7 @@ private:
        void
        operator()() const
        {
-            if (m_probe == nullptr)
+            if (!m_probe)
                return;
            typename Clock::time_point const now(Clock::now());
            typename Clock::duration const elapsed(now - m_start);
@@ -202,7 +202,7 @@ private:
                // Calculate when we want to sample again, and
                // adjust for the expected latency.
                //
-                typename Clock::time_point const when(now + m_probe->m_period - (2 * elapsed));
+                typename Clock::time_point const when(now + m_probe->m_period - 2 * elapsed);

                if (when <= now)
                {
@@ -224,7 +224,7 @@ private:
        void
        operator()(boost::system::error_code const& ec)
        {
-            if (m_probe == nullptr)
+            if (!m_probe)
                return;
            typename Clock::time_point const now(Clock::now());
            boost::asio::post(
--- a/include/xrpl/beast/container/detail/aged_associative_container.h
+++ b/include/xrpl/beast/container/detail/aged_associative_container.h
@@ -26,7 +26,7 @@ struct aged_associative_container_extract_t<false>
    Value const&
    operator()(Value const& value) const
    {
-        return value;  // NOLINT(bugprone-return-const-ref-from-parameter)
+        return value;
    }
 };

--- a/include/xrpl/beast/container/detail/aged_container_iterator.h
+++ b/include/xrpl/beast/container/detail/aged_container_iterator.h
@@ -16,10 +16,10 @@ class aged_container_iterator
 {
 public:
    using iterator_category = typename std::iterator_traits<Iterator>::iterator_category;
-    using value_type = std::conditional_t<
+    using value_type = typename std::conditional<
        is_const,
        typename Iterator::value_type::stashed::value_type const,
-        typename Iterator::value_type::stashed::value_type>;
+        typename Iterator::value_type::stashed::value_type>::type;
    using difference_type = typename std::iterator_traits<Iterator>::difference_type;
    using pointer = value_type*;
    using reference = value_type&;
@@ -32,9 +32,9 @@ public:
    template <
        bool other_is_const,
        class OtherIterator,
-        class = std::enable_if_t<
-            (!other_is_const || is_const) &&
-            !static_cast<bool>(std::is_same_v<Iterator, OtherIterator>)>>
+        class = typename std::enable_if<
+            (other_is_const == false || is_const == true) &&
+            std::is_same<Iterator, OtherIterator>::value == false>::type>
    explicit aged_container_iterator(
        aged_container_iterator<other_is_const, OtherIterator> const& other)
        : m_iter(other.m_iter)
@@ -42,7 +42,9 @@ public:
    }

    // Disable constructing a const_iterator from a non-const_iterator.
-    template <bool other_is_const, class = std::enable_if_t<!other_is_const || is_const>>
+    template <
+        bool other_is_const,
+        class = typename std::enable_if<other_is_const == false || is_const == true>::type>
    aged_container_iterator(aged_container_iterator<other_is_const, Iterator> const& other)
        : m_iter(other.m_iter)
    {
@@ -51,8 +53,8 @@ public:
    // Disable assigning a const_iterator to a non-const iterator
    template <bool other_is_const, class OtherIterator>
    auto
-    operator=(aged_container_iterator<other_is_const, OtherIterator> const& other)
-        -> std::enable_if_t<!other_is_const || is_const, aged_container_iterator&>
+    operator=(aged_container_iterator<other_is_const, OtherIterator> const& other) -> typename std::
+        enable_if<other_is_const == false || is_const == true, aged_container_iterator&>::type
    {
        m_iter = other.m_iter;
        return *this;
--- a/include/xrpl/beast/container/detail/aged_ordered_container.h
+++ b/include/xrpl/beast/container/detail/aged_ordered_container.h
@@ -257,8 +257,7 @@ private:

        config_t(config_t&& other)
            : KeyValueCompare(std::move(other.key_compare()))
-            , beast::detail::empty_base_optimization<ElementAllocator>(std::move(
-                  static_cast<beast::detail::empty_base_optimization<ElementAllocator>&>(other)))
+            , beast::detail::empty_base_optimization<ElementAllocator>(std::move(other))
            , clock(other.clock)
        {
        }
--- a/include/xrpl/beast/core/List.h
+++ b/include/xrpl/beast/core/List.h
@@ -35,11 +35,9 @@ struct CopyConst<T const, U>
 template <typename T, typename Tag>
 class ListNode
 {
-    ListNode() = default;
-
+private:
    using value_type = T;

-    friend T;
    friend class List<T, Tag>;

    template <typename>
--- a/include/xrpl/beast/core/SemanticVersion.h
+++ b/include/xrpl/beast/core/SemanticVersion.h
@@ -40,12 +40,12 @@ public:
    std::string
    print() const;

-    bool
+    inline bool
    isRelease() const noexcept
    {
        return preReleaseIdentifiers.empty();
    }
-    bool
+    inline bool
    isPreRelease() const noexcept
    {
        return !isRelease();
--- a/include/xrpl/beast/hash/hash_append.h
+++ b/include/xrpl/beast/hash/hash_append.h
@@ -203,8 +203,7 @@ template <class Hasher, class T>
 inline std::enable_if_t<is_contiguously_hashable<T, Hasher>::value>
 hash_append(Hasher& h, T const& t) noexcept
 {
-    // NOLINTNEXTLINE(bugprone-sizeof-expression)
-    h(static_cast<void const*>(std::addressof(t)), sizeof(t));
+    h(std::addressof(t), sizeof(t));
 }

 template <class Hasher, class T>
--- a/include/xrpl/beast/hash/xxhasher.h
+++ b/include/xrpl/beast/hash/xxhasher.h
@@ -64,7 +64,7 @@ private:
    void
    flushToState(void const* data, std::size_t len)
    {
-        if (state_ == nullptr)
+        if (!state_)
        {
            state_ = allocState();
            if (seed_.has_value())
@@ -78,7 +78,7 @@ private:
        }
        XXH3_64bits_update(state_, readBuffer_.data(), readBuffer_.size());
        resetBuffers();
-        if ((data != nullptr) && (len != 0u))
+        if (data && len)
        {
            XXH3_64bits_update(state_, data, len);
        }
@@ -87,18 +87,22 @@ private:
    result_type
    retrieveHash()
    {
-        if (state_ != nullptr)
+        if (state_)
        {
            flushToState(nullptr, 0);
            return XXH3_64bits_digest(state_);
        }
-
-        if (seed_.has_value())
+        else
        {
-            return XXH3_64bits_withSeed(readBuffer_.data(), readBuffer_.size(), *seed_);
+            if (seed_.has_value())
+            {
+                return XXH3_64bits_withSeed(readBuffer_.data(), readBuffer_.size(), *seed_);
+            }
+            else
+            {
+                return XXH3_64bits(readBuffer_.data(), readBuffer_.size());
+            }
        }
-
-        return XXH3_64bits(readBuffer_.data(), readBuffer_.size());
    }

 public:
@@ -115,7 +119,7 @@ public:

    ~xxhasher() noexcept
    {
-        if (state_ != nullptr)
+        if (state_)
        {
            XXH3_freeState(state_);
        }
--- a/include/xrpl/beast/net/IPAddress.h
+++ b/include/xrpl/beast/net/IPAddress.h
@@ -70,13 +70,9 @@ hash_append(Hasher& h, beast::IP::Address const& addr) noexcept
 {
    using beast::hash_append;
    if (addr.is_v4())
-    {
        hash_append(h, addr.to_v4().to_bytes());
-    }
    else if (addr.is_v6())
-    {
        hash_append(h, addr.to_v6().to_bytes());
-    }
    else
    {
        // LCOV_EXCL_START
--- a/include/xrpl/beast/net/IPEndpoint.h
+++ b/include/xrpl/beast/net/IPEndpoint.h
@@ -69,12 +69,12 @@ public:
    {
        return m_addr.is_v6();
    }
-    AddressV4
+    AddressV4 const
    to_v4() const
    {
        return m_addr.to_v4();
    }
-    AddressV6
+    AddressV6 const
    to_v6() const
    {
        return m_addr.to_v6();
--- a/include/xrpl/beast/rfc2616.h
+++ b/include/xrpl/beast/rfc2616.h
@@ -53,9 +53,8 @@ is_white(char c)
        case '\t':
        case '\v':
            return true;
-        default:
-            return false;
    };
+    return false;
 }

 template <class FwdIter>
@@ -350,10 +349,8 @@ bool
 token_in_list(boost::string_ref const& value, boost::string_ref const& token)
 {
    for (auto const& item : make_list(value))
-    {
        if (ci_equal(item, token))
            return true;
-    }
    return false;
 }

@@ -362,10 +359,8 @@ bool
 is_keep_alive(boost::beast::http::message<isRequest, Body, Fields> const& m)
 {
    if (m.version() <= 10)
-    {
        return boost::beast::http::token_list{m[boost::beast::http::field::connection]}.exists(
            "keep-alive");
-    }
    return !boost::beast::http::token_list{m[boost::beast::http::field::connection]}.exists(
        "close");
 }
--- a/include/xrpl/beast/test/yield_to.h
+++ b/include/xrpl/beast/test/yield_to.h
@@ -44,7 +44,7 @@ public:
        : work_(boost::asio::make_work_guard(ios_))
    {
        threads_.reserve(concurrency);
-        while ((concurrency--) != 0u)
+        while (concurrency--)
            threads_.emplace_back([&] { ios_.run(); });
    }

--- a/include/xrpl/beast/type_name.h
+++ b/include/xrpl/beast/type_name.h
@@ -32,13 +32,9 @@ type_name()
    if (std::is_volatile<TR>::value)
        name += " volatile";
    if (std::is_lvalue_reference<T>::value)
-    {
        name += "&";
-    }
    else if (std::is_rvalue_reference<T>::value)
-    {
        name += "&&";
-    }

    return name;
 }
--- a/include/xrpl/beast/unit_test/recorder.h
+++ b/include/xrpl/beast/unit_test/recorder.h
@@ -50,7 +50,7 @@ private:
    virtual void
    on_case_end() override
    {
-        if (!m_case.tests.empty())
+        if (m_case.tests.size() > 0)
            m_suite.insert(std::move(m_case));
    }

--- a/include/xrpl/beast/unit_test/reporter.h
+++ b/include/xrpl/beast/unit_test/reporter.h
@@ -118,18 +118,18 @@ private:

 //------------------------------------------------------------------------------

-template <class Unused>
+template <class _>
 void
-reporter<Unused>::suite_results::add(case_results const& r)
+reporter<_>::suite_results::add(case_results const& r)
 {
    ++cases;
    total += r.total;
    failed += r.failed;
 }

-template <class Unused>
+template <class _>
 void
-reporter<Unused>::results::add(suite_results const& r)
+reporter<_>::results::add(suite_results const& r)
 {
    ++suites;
    total += r.total;
@@ -160,13 +160,13 @@ reporter<Unused>::results::add(suite_results const& r)

 //------------------------------------------------------------------------------

-template <class Unused>
-reporter<Unused>::reporter(std::ostream& os) : os_(os)
+template <class _>
+reporter<_>::reporter(std::ostream& os) : os_(os)
 {
 }

-template <class Unused>
-reporter<Unused>::~reporter()
+template <class _>
+reporter<_>::~reporter()
 {
    if (results_.top.size() > 0)
    {
@@ -180,9 +180,9 @@ reporter<Unused>::~reporter()
        << amount{results_.failed, "failure"} << std::endl;
 }

-template <class Unused>
+template <class _>
 std::string
-reporter<Unused>::fmtdur(typename clock_type::duration const& d)
+reporter<_>::fmtdur(typename clock_type::duration const& d)
 {
    using namespace std::chrono;
    auto const ms = duration_cast<milliseconds>(d);
@@ -193,46 +193,46 @@ reporter<Unused>::fmtdur(typename clock_type::duration const& d)
    return ss.str();
 }

-template <class Unused>
+template <class _>
 void
-reporter<Unused>::on_suite_begin(suite_info const& info)
+reporter<_>::on_suite_begin(suite_info const& info)
 {
    suite_results_ = suite_results{info.full_name()};
 }

-template <class Unused>
+template <class _>
 void
-reporter<Unused>::on_suite_end()
+reporter<_>::on_suite_end()
 {
    results_.add(suite_results_);
 }

-template <class Unused>
+template <class _>
 void
-reporter<Unused>::on_case_begin(std::string const& name)
+reporter<_>::on_case_begin(std::string const& name)
 {
    case_results_ = case_results(name);
    os_ << suite_results_.name << (case_results_.name.empty() ? "" : (" " + case_results_.name))
        << std::endl;
 }

-template <class Unused>
+template <class _>
 void
-reporter<Unused>::on_case_end()
+reporter<_>::on_case_end()
 {
    suite_results_.add(case_results_);
 }

-template <class Unused>
+template <class _>
 void
-reporter<Unused>::on_pass()
+reporter<_>::on_pass()
 {
    ++case_results_.total;
 }

-template <class Unused>
+template <class _>
 void
-reporter<Unused>::on_fail(std::string const& reason)
+reporter<_>::on_fail(std::string const& reason)
 {
    ++case_results_.failed;
    ++case_results_.total;
@@ -240,9 +240,9 @@ reporter<Unused>::on_fail(std::string const& reason)
        << std::endl;
 }

-template <class Unused>
+template <class _>
 void
-reporter<Unused>::on_log(std::string const& s)
+reporter<_>::on_log(std::string const& s)
 {
    os_ << s;
 }
--- a/include/xrpl/beast/unit_test/results.h
+++ b/include/xrpl/beast/unit_test/results.h
@@ -145,9 +145,9 @@ public:
    void
    insert(case_results&& r)
    {
+        cont().emplace_back(std::move(r));
        total_ += r.tests.total();
        failed_ += r.tests.failed();
-        cont().emplace_back(std::move(r));
    }

    void
--- a/include/xrpl/beast/unit_test/runner.h
+++ b/include/xrpl/beast/unit_test/runner.h
@@ -198,10 +198,8 @@ runner::run_if(FwdIter first, FwdIter last, Pred pred)
 {
    bool failed(false);
    for (; first != last; ++first)
-    {
        if (pred(*first))
            failed = run(*first) || failed;
-    }
    return failed;
 }

@@ -221,10 +219,8 @@ runner::run_each_if(SequenceContainer const& c, Pred pred)
 {
    bool failed(false);
    for (auto const& s : c)
-    {
        if (pred(s))
            failed = run(s) || failed;
-    }
    return failed;
 }

--- a/include/xrpl/beast/unit_test/suite.h
+++ b/include/xrpl/beast/unit_test/suite.h
@@ -36,7 +36,7 @@ make_reason(String const& reason, char const* file, int line)

 }  // namespace detail

-class Thread;
+class thread;

 enum abort_t { no_abort_on_fail, abort_on_fail };

@@ -295,7 +295,7 @@ public:
    }

 private:
-    friend class Thread;
+    friend class thread;

    static suite**
    p_this_suite()
@@ -309,7 +309,7 @@ private:
    run() = 0;

    void
-    propagate_abort() const;
+    propagate_abort();

    template <class = void>
    void
@@ -486,13 +486,9 @@ suite::unexpected(Condition shouldBeFalse, String const& reason)
 {
    bool const b = static_cast<bool>(shouldBeFalse);
    if (!b)
-    {
        pass();
-    }
    else
-    {
        fail(reason);
-    }
    return !b;
 }

@@ -526,7 +522,7 @@ suite::fail(String const& reason, char const* file, int line)
 }

 inline void
-suite::propagate_abort() const
+suite::propagate_abort()
 {
    if (abort_ && aborted_)
        BOOST_THROW_EXCEPTION(abort_exception());
@@ -542,7 +538,7 @@ suite::run(runner& r)
    {
        run();
    }
-    catch (abort_exception const&)  // NOLINT(bugprone-empty-catch)
+    catch (abort_exception const&)
    {
        // ends the suite
    }
--- a/include/xrpl/beast/unit_test/thread.h
+++ b/include/xrpl/beast/unit_test/thread.h
@@ -14,7 +14,7 @@ namespace beast {
 namespace unit_test {

 /** Replacement for std::thread that handles exceptions in unit tests. */
-class Thread
+class thread
 {
 private:
    suite* s_ = nullptr;
@@ -24,17 +24,17 @@ public:
    using id = std::thread::id;
    using native_handle_type = std::thread::native_handle_type;

-    Thread() = default;
-    Thread(Thread const&) = delete;
-    Thread&
-    operator=(Thread const&) = delete;
+    thread() = default;
+    thread(thread const&) = delete;
+    thread&
+    operator=(thread const&) = delete;

-    Thread(Thread&& other) : s_(other.s_), t_(std::move(other.t_))
+    thread(thread&& other) : s_(other.s_), t_(std::move(other.t_))
    {
    }

-    Thread&
-    operator=(Thread&& other)
+    thread&
+    operator=(thread&& other)
    {
        s_ = other.s_;
        t_ = std::move(other.t_);
@@ -42,10 +42,10 @@ public:
    }

    template <class F, class... Args>
-    explicit Thread(suite& s, F&& f, Args&&... args) : s_(&s)
+    explicit thread(suite& s, F&& f, Args&&... args) : s_(&s)
    {
        std::function<void(void)> b = std::bind(std::forward<F>(f), std::forward<Args>(args)...);
-        t_ = std::thread(&Thread::run, this, std::move(b));
+        t_ = std::thread(&thread::run, this, std::move(b));
    }

    bool
@@ -80,7 +80,7 @@ public:
    }

    void
-    swap(Thread& other)
+    swap(thread& other)
    {
        std::swap(s_, other.s_);
        std::swap(t_, other.t_);
@@ -94,7 +94,7 @@ private:
        {
            f();
        }
-        catch (suite::abort_exception const&)  // NOLINT(bugprone-empty-catch)
+        catch (suite::abort_exception const&)
        {
        }
        catch (std::exception const& e)
--- a/include/xrpl/beast/utility/Journal.h
+++ b/include/xrpl/beast/utility/Journal.h
@@ -13,13 +13,13 @@ enum Severity {
    kAll = 0,

    kTrace = kAll,
-    kDebug = 1,
-    kInfo = 2,
-    kWarning = 3,
-    kError = 4,
-    kFatal = 5,
+    kDebug,
+    kInfo,
+    kWarning,
+    kError,
+    kFatal,

-    kDisabled = 6,
+    kDisabled,
    kNone = kDisabled
 };
 }  // namespace severities
@@ -109,12 +109,12 @@ public:
    };

 #ifndef __INTELLISENSE__
-    static_assert(!std::is_default_constructible_v<Sink>, "");
-    static_assert(!std::is_copy_constructible_v<Sink>, "");
-    static_assert(!std::is_move_constructible_v<Sink>, "");
-    static_assert(!std::is_copy_assignable_v<Sink>, "");
-    static_assert(!std::is_move_assignable_v<Sink>, "");
-    static_assert(std::is_nothrow_destructible_v<Sink>, "");
+    static_assert(std::is_default_constructible<Sink>::value == false, "");
+    static_assert(std::is_copy_constructible<Sink>::value == false, "");
+    static_assert(std::is_move_constructible<Sink>::value == false, "");
+    static_assert(std::is_copy_assignable<Sink>::value == false, "");
+    static_assert(std::is_move_assignable<Sink>::value == false, "");
+    static_assert(std::is_nothrow_destructible<Sink>::value == true, "");
 #endif

    /** Returns a Sink which does nothing. */
@@ -165,12 +165,12 @@ public:
    };

 #ifndef __INTELLISENSE__
-    static_assert(!std::is_default_constructible_v<ScopedStream>, "");
-    static_assert(std::is_copy_constructible_v<ScopedStream>, "");
-    static_assert(std::is_move_constructible_v<ScopedStream>, "");
-    static_assert(!std::is_copy_assignable_v<ScopedStream>, "");
-    static_assert(!std::is_move_assignable_v<ScopedStream>, "");
-    static_assert(std::is_nothrow_destructible_v<ScopedStream>, "");
+    static_assert(std::is_default_constructible<ScopedStream>::value == false, "");
+    static_assert(std::is_copy_constructible<ScopedStream>::value == true, "");
+    static_assert(std::is_move_constructible<ScopedStream>::value == true, "");
+    static_assert(std::is_copy_assignable<ScopedStream>::value == false, "");
+    static_assert(std::is_move_assignable<ScopedStream>::value == false, "");
+    static_assert(std::is_nothrow_destructible<ScopedStream>::value == true, "");
 #endif

    //--------------------------------------------------------------------------
@@ -247,12 +247,12 @@ public:
    };

 #ifndef __INTELLISENSE__
-    static_assert(std::is_default_constructible_v<Stream>, "");
-    static_assert(std::is_copy_constructible_v<Stream>, "");
-    static_assert(std::is_move_constructible_v<Stream>, "");
-    static_assert(!std::is_copy_assignable_v<Stream>, "");
-    static_assert(!std::is_move_assignable_v<Stream>, "");
-    static_assert(std::is_nothrow_destructible_v<Stream>, "");
+    static_assert(std::is_default_constructible<Stream>::value == true, "");
+    static_assert(std::is_copy_constructible<Stream>::value == true, "");
+    static_assert(std::is_move_constructible<Stream>::value == true, "");
+    static_assert(std::is_copy_assignable<Stream>::value == false, "");
+    static_assert(std::is_move_assignable<Stream>::value == false, "");
+    static_assert(std::is_nothrow_destructible<Stream>::value == true, "");
 #endif

    //--------------------------------------------------------------------------
@@ -330,12 +330,12 @@ public:
 };

 #ifndef __INTELLISENSE__
-static_assert(!std::is_default_constructible_v<Journal>, "");
-static_assert(std::is_copy_constructible_v<Journal>, "");
-static_assert(std::is_move_constructible_v<Journal>, "");
-static_assert(std::is_copy_assignable_v<Journal>, "");
-static_assert(std::is_move_assignable_v<Journal>, "");
-static_assert(std::is_nothrow_destructible_v<Journal>, "");
+static_assert(std::is_default_constructible<Journal>::value == false, "");
+static_assert(std::is_copy_constructible<Journal>::value == true, "");
+static_assert(std::is_move_constructible<Journal>::value == true, "");
+static_assert(std::is_copy_assignable<Journal>::value == true, "");
+static_assert(std::is_move_assignable<Journal>::value == true, "");
+static_assert(std::is_nothrow_destructible<Journal>::value == true, "");
 #endif

 //------------------------------------------------------------------------------
--- a/include/xrpl/beast/utility/Zero.h
+++ b/include/xrpl/beast/utility/Zero.h
@@ -27,7 +27,7 @@ struct Zero
 };

 namespace {
-constexpr Zero zero{};
+static constexpr Zero zero{};
 }  // namespace

 /** Default implementation of signum calls the method on the class. */
--- a/include/xrpl/beast/xor_shift_engine.h
+++ b/include/xrpl/beast/xor_shift_engine.h
@@ -43,15 +43,15 @@ private:
    murmurhash3(result_type x);
 };

-template <class Unused>
-xor_shift_engine<Unused>::xor_shift_engine(result_type val)
+template <class _>
+xor_shift_engine<_>::xor_shift_engine(result_type val)
 {
    seed(val);
 }

-template <class Unused>
+template <class _>
 void
-xor_shift_engine<Unused>::seed(result_type seed)
+xor_shift_engine<_>::seed(result_type seed)
 {
    if (seed == 0)
        throw std::domain_error("invalid seed");
@@ -59,9 +59,9 @@ xor_shift_engine<Unused>::seed(result_type seed)
    s_[1] = murmurhash3(s_[0]);
 }

-template <class Unused>
+template <class _>
 auto
-xor_shift_engine<Unused>::operator()() -> result_type
+xor_shift_engine<_>::operator()() -> result_type
 {
    result_type s1 = s_[0];
    result_type const s0 = s_[1];
@@ -70,9 +70,9 @@ xor_shift_engine<Unused>::operator()() -> result_type
    return (s_[1] = (s1 ^ s0 ^ (s1 >> 17) ^ (s0 >> 26))) + s0;
 }

-template <class Unused>
+template <class _>
 auto
-xor_shift_engine<Unused>::murmurhash3(result_type x) -> result_type
+xor_shift_engine<_>::murmurhash3(result_type x) -> result_type
 {
    x ^= x >> 33;
    x *= 0xff51afd7ed558ccdULL;
--- a/include/xrpl/conditions/detail/utils.h
+++ b/include/xrpl/conditions/detail/utils.h
@@ -89,7 +89,7 @@ parsePreamble(Slice& s, std::error_code& ec)
    p.length = s[0];
    s += 1;

-    if ((p.length & 0x80) != 0u)
+    if (p.length & 0x80)
    {  // Long form length:
        std::size_t const cnt = p.length & 0x7F;

--- a/include/xrpl/core/ClosureCounter.h
+++ b/include/xrpl/core/ClosureCounter.h
@@ -34,9 +34,9 @@ template <typename Ret_t, typename... Args_t>
 class ClosureCounter
 {
 private:
-    std::mutex mutable mutex_;
-    std::condition_variable allClosuresDoneCond_;  // guard with mutex_
-    bool waitForClosures_{false};                  // guard with mutex_
+    std::mutex mutable mutex_{};
+    std::condition_variable allClosuresDoneCond_{};  // guard with mutex_
+    bool waitForClosures_{false};                    // guard with mutex_
    std::atomic<int> closureCount_{0};

    // Increment the count.
--- a/include/xrpl/core/PeerReservationTable.h
+++ b/include/xrpl/core/PeerReservationTable.h
@@ -20,7 +20,7 @@ struct PeerReservation final
 {
 public:
    PublicKey nodeId;
-    std::string description = {};  // NOLINT(readability-redundant-member-init)
+    std::string description{};

    auto
    toJson() const -> Json::Value;
@@ -68,7 +68,7 @@ public:
    contains(PublicKey const& nodeId)
    {
        std::lock_guard const lock(this->mutex_);
-        return table_.contains({.nodeId = nodeId, .description = {}});
+        return table_.find({nodeId}) != table_.end();
    }

    // Because `ApplicationImp` has two-phase initialization, so must we.
--- a/include/xrpl/json/to_string.h
+++ b/include/xrpl/json/to_string.h
@@ -1,11 +1,12 @@
 #pragma once

-#include <xrpl/json/json_writer.h>
-
+#include <ostream>
 #include <string>

 namespace Json {

+class Value;
+
 /** Writes a Json::Value to an std::string. */
 std::string
 to_string(Value const&);
@@ -14,4 +15,8 @@ to_string(Value const&);
 std::string
 pretty(Value const&);

+/** Output using the StyledStreamWriter. @see Json::operator>>(). */
+std::ostream&
+operator<<(std::ostream&, Value const& root);
+
 }  // namespace Json
--- a/include/xrpl/ledger/AmendmentTable.h
+++ b/include/xrpl/ledger/AmendmentTable.h
@@ -75,12 +75,10 @@ public:
    doValidatedLedger(std::shared_ptr<ReadView const> const& lastValidatedLedger)
    {
        if (needValidatedLedger(lastValidatedLedger->seq()))
-        {
            doValidatedLedger(
                lastValidatedLedger->seq(),
                getEnabledAmendments(*lastValidatedLedger),
                getMajorityAmendments(*lastValidatedLedger));
-        }
    }

    /** Called to determine whether the amendment logic needs to process
--- a/include/xrpl/ledger/CanonicalTXSet.h
+++ b/include/xrpl/ledger/CanonicalTXSet.h
@@ -29,31 +29,31 @@ private:
        friend bool
        operator<(Key const& lhs, Key const& rhs);

-        friend bool
+        inline friend bool
        operator>(Key const& lhs, Key const& rhs)
        {
            return rhs < lhs;
        }

-        friend bool
+        inline friend bool
        operator<=(Key const& lhs, Key const& rhs)
        {
            return !(lhs > rhs);
        }

-        friend bool
+        inline friend bool
        operator>=(Key const& lhs, Key const& rhs)
        {
            return !(lhs < rhs);
        }

-        friend bool
+        inline friend bool
        operator==(Key const& lhs, Key const& rhs)
        {
            return lhs.txId_ == rhs.txId_;
        }

-        friend bool
+        inline friend bool
        operator!=(Key const& lhs, Key const& rhs)
        {
            return !(lhs == rhs);
--- a/include/xrpl/ledger/PendingSaves.h
+++ b/include/xrpl/ledger/PendingSaves.h
@@ -65,7 +65,7 @@ public:
    pending(LedgerIndex seq)
    {
        std::lock_guard const lock(mutex_);
-        return map_.contains(seq);
+        return map_.find(seq) != map_.end();
    }

    /** Check if a ledger should be dispatched
--- a/include/xrpl/ledger/helpers/AMMHelpers.h
+++ b/include/xrpl/ledger/helpers/AMMHelpers.h
@@ -1,13 +1,8 @@
 #pragma once

-#include <xrpl/basics/Expected.h>
 #include <xrpl/basics/Log.h>
 #include <xrpl/basics/Number.h>
 #include <xrpl/beast/utility/Journal.h>
-#include <xrpl/ledger/ReadView.h>
-#include <xrpl/ledger/Sandbox.h>
-#include <xrpl/ledger/helpers/RippleStateHelpers.h>
-#include <xrpl/ledger/helpers/TokenHelpers.h>
 #include <xrpl/protocol/AMMCore.h>
 #include <xrpl/protocol/AmountConversions.h>
 #include <xrpl/protocol/Feature.h>
@@ -16,7 +11,6 @@
 #include <xrpl/protocol/Quality.h>
 #include <xrpl/protocol/Rules.h>
 #include <xrpl/protocol/STAmount.h>
-#include <xrpl/protocol/STLedgerEntry.h>

 namespace xrpl {

@@ -208,10 +202,10 @@ getAMMOfferStartWithTakerGets(

    // Try to reduce the offer size to improve the quality.
    // The quality might still not match the targetQuality for a tiny offer.
-    auto amounts = getAmounts(*nTakerGets);
-    if (Quality{amounts} < targetQuality)
+    if (auto amounts = getAmounts(*nTakerGets); Quality{amounts} < targetQuality)
        return getAmounts(detail::reduceOffer(amounts.out));
-    return amounts;
+    else
+        return amounts;
 }

 /** Generate AMM offer starting with takerPays when AMM pool
@@ -275,10 +269,10 @@ getAMMOfferStartWithTakerPays(

    // Try to reduce the offer size to improve the quality.
    // The quality might still not match the targetQuality for a tiny offer.
-    auto amounts = getAmounts(*nTakerPays);
-    if (Quality{amounts} < targetQuality)
+    if (auto amounts = getAmounts(*nTakerPays); Quality{amounts} < targetQuality)
        return getAmounts(detail::reduceOffer(amounts.in));
-    return amounts;
+    else
+        return amounts;
 }

 /**   Generate AMM offer so that either updated Spot Price Quality (SPQ)
@@ -318,12 +312,9 @@ changeSpotPriceQuality(
        auto const& a = f;
        auto const b = pool.in * (1 + f);
        Number const c = pool.in * pool.in - pool.in * pool.out * quality.rate();
-        auto const res = b * b - 4 * a * c;
-        if (res < 0)
-        {
+        if (auto const res = b * b - 4 * a * c; res < 0)
            return std::nullopt;  // LCOV_EXCL_LINE
-        }
-        if (auto const nTakerPaysPropose = (-b + root2(res)) / (2 * a); nTakerPaysPropose > 0)
+        else if (auto const nTakerPaysPropose = (-b + root2(res)) / (2 * a); nTakerPaysPropose > 0)
        {
            auto const nTakerPays = [&]() {
                // The fee might make the AMM offer quality less than CLOB
@@ -468,11 +459,13 @@ swapAssetIn(TAmounts<TIn, TOut> const& pool, TIn const& assetIn, std::uint16_t t

        return toAmount<TOut>(getAsset(pool.out), swapOut, Number::downward);
    }
-
-    return toAmount<TOut>(
-        getAsset(pool.out),
-        pool.out - (pool.in * pool.out) / (pool.in + assetIn * feeMult(tfee)),
-        Number::downward);
+    else
+    {
+        return toAmount<TOut>(
+            getAsset(pool.out),
+            pool.out - (pool.in * pool.out) / (pool.in + assetIn * feeMult(tfee)),
+            Number::downward);
+    }
 }

 /** Swap assetOut out of the pool and swap in a proportional amount
@@ -534,11 +527,13 @@ swapAssetOut(TAmounts<TIn, TOut> const& pool, TOut const& assetOut, std::uint16_

        return toAmount<TIn>(getAsset(pool.in), swapIn, Number::upward);
    }
-
-    return toAmount<TIn>(
-        getAsset(pool.in),
-        ((pool.in * pool.out) / (pool.out - assetOut) - pool.in) / feeMult(tfee),
-        Number::upward);
+    else
+    {
+        return toAmount<TIn>(
+            getAsset(pool.in),
+            ((pool.in * pool.out) / (pool.out - assetOut) - pool.in) / feeMult(tfee),
+            Number::upward);
+    }
 }

 /** Return square of n.
@@ -622,13 +617,9 @@ getRoundedAsset(Rules const& rules, STAmount const& balance, A const& frac, IsDe
    if (!rules.enabled(fixAMMv1_3))
    {
        if constexpr (std::is_same_v<A, STAmount>)
-        {
            return multiply(balance, frac, balance.asset());
-        }
        else
-        {
            return toSTAmount(balance.asset(), balance * frac);
-        }
    }
    auto const rm = detail::getAssetRounding(isDeposit);
    return multiply(balance, frac, rm);
@@ -722,94 +713,4 @@ adjustFracByTokens(
    STAmount const& tokens,
    Number const& frac);

-/** Get AMM pool balances.
- */
-std::pair<STAmount, STAmount>
-ammPoolHolds(
-    ReadView const& view,
-    AccountID const& ammAccountID,
-    Asset const& asset1,
-    Asset const& asset2,
-    FreezeHandling freezeHandling,
-    AuthHandling authHandling,
-    beast::Journal const j);
-
-/** Get AMM pool and LP token balances. If both optIssue are
- * provided then they are used as the AMM token pair issues.
- * Otherwise the missing issues are fetched from ammSle.
- */
-Expected<std::tuple<STAmount, STAmount, STAmount>, TER>
-ammHolds(
-    ReadView const& view,
-    SLE const& ammSle,
-    std::optional<Asset> const& optAsset1,
-    std::optional<Asset> const& optAsset2,
-    FreezeHandling freezeHandling,
-    AuthHandling authHandling,
-    beast::Journal const j);
-
-/** Get the balance of LP tokens.
- */
-STAmount
-ammLPHolds(
-    ReadView const& view,
-    Asset const& asset1,
-    Asset const& asset2,
-    AccountID const& ammAccount,
-    AccountID const& lpAccount,
-    beast::Journal const j);
-
-STAmount
-ammLPHolds(
-    ReadView const& view,
-    SLE const& ammSle,
-    AccountID const& lpAccount,
-    beast::Journal const j);
-
-/** Get AMM trading fee for the given account. The fee is discounted
- * if the account is the auction slot owner or one of the slot's authorized
- * accounts.
- */
-std::uint16_t
-getTradingFee(ReadView const& view, SLE const& ammSle, AccountID const& account);
-
-/** Returns total amount held by AMM for the given token.
- */
-STAmount
-ammAccountHolds(ReadView const& view, AccountID const& ammAccountID, Asset const& asset);
-
-/** Delete trustlines to AMM. If all trustlines are deleted then
- * AMM object and account are deleted. Otherwise tecINCOMPLETE is returned.
- */
-TER
-deleteAMMAccount(Sandbox& view, Asset const& asset, Asset const& asset2, beast::Journal j);
-
-/** Initialize Auction and Voting slots and set the trading/discounted fee.
- */
-void
-initializeFeeAuctionVote(
-    ApplyView& view,
-    std::shared_ptr<SLE>& ammSle,
-    AccountID const& account,
-    Asset const& lptAsset,
-    std::uint16_t tfee);
-
-/** Return true if the Liquidity Provider is the only AMM provider, false
- * otherwise. Return tecINTERNAL if encountered an unexpected condition,
- * for instance Liquidity Provider has more than one LPToken trustline.
- */
-Expected<bool, TER>
-isOnlyLiquidityProvider(ReadView const& view, Issue const& ammIssue, AccountID const& lpAccount);
-
-/** Due to rounding, the LPTokenBalance of the last LP might
- * not match the LP's trustline balance. If it's within the tolerance,
- * update LPTokenBalance to match the LP's trustline balance.
- */
-Expected<bool, TER>
-verifyAndAdjustLPTokenBalance(
-    Sandbox& sb,
-    STAmount const& lpTokens,
-    std::shared_ptr<SLE>& ammSle,
-    AccountID const& account);
-
 }  // namespace xrpl
--- a/include/xrpl/ledger/helpers/AMMUtils.h
+++ b/include/xrpl/ledger/helpers/AMMUtils.h
@@ -0,0 +1,108 @@
+#pragma once
+
+#include <xrpl/basics/Expected.h>
+#include <xrpl/beast/utility/Journal.h>
+#include <xrpl/ledger/View.h>
+#include <xrpl/ledger/helpers/RippleStateHelpers.h>
+#include <xrpl/protocol/STAmount.h>
+#include <xrpl/protocol/STLedgerEntry.h>
+#include <xrpl/protocol/TER.h>
+
+namespace xrpl {
+
+class ReadView;
+class ApplyView;
+class Sandbox;
+class NetClock;
+
+/** Get AMM pool balances.
+ */
+std::pair<STAmount, STAmount>
+ammPoolHolds(
+    ReadView const& view,
+    AccountID const& ammAccountID,
+    Asset const& asset1,
+    Asset const& asset2,
+    FreezeHandling freezeHandling,
+    AuthHandling authHandling,
+    beast::Journal const j);
+
+/** Get AMM pool and LP token balances. If both optIssue are
+ * provided then they are used as the AMM token pair issues.
+ * Otherwise the missing issues are fetched from ammSle.
+ */
+Expected<std::tuple<STAmount, STAmount, STAmount>, TER>
+ammHolds(
+    ReadView const& view,
+    SLE const& ammSle,
+    std::optional<Asset> const& optAsset1,
+    std::optional<Asset> const& optAsset2,
+    FreezeHandling freezeHandling,
+    AuthHandling authHandling,
+    beast::Journal const j);
+
+/** Get the balance of LP tokens.
+ */
+STAmount
+ammLPHolds(
+    ReadView const& view,
+    Asset const& asset1,
+    Asset const& asset2,
+    AccountID const& ammAccount,
+    AccountID const& lpAccount,
+    beast::Journal const j);
+
+STAmount
+ammLPHolds(
+    ReadView const& view,
+    SLE const& ammSle,
+    AccountID const& lpAccount,
+    beast::Journal const j);
+
+/** Get AMM trading fee for the given account. The fee is discounted
+ * if the account is the auction slot owner or one of the slot's authorized
+ * accounts.
+ */
+std::uint16_t
+getTradingFee(ReadView const& view, SLE const& ammSle, AccountID const& account);
+
+/** Returns total amount held by AMM for the given token.
+ */
+STAmount
+ammAccountHolds(ReadView const& view, AccountID const& ammAccountID, Asset const& asset);
+
+/** Delete trustlines to AMM. If all trustlines are deleted then
+ * AMM object and account are deleted. Otherwise tecIMPCOMPLETE is returned.
+ */
+TER
+deleteAMMAccount(Sandbox& view, Asset const& asset, Asset const& asset2, beast::Journal j);
+
+/** Initialize Auction and Voting slots and set the trading/discounted fee.
+ */
+void
+initializeFeeAuctionVote(
+    ApplyView& view,
+    std::shared_ptr<SLE>& ammSle,
+    AccountID const& account,
+    Asset const& lptAsset,
+    std::uint16_t tfee);
+
+/** Return true if the Liquidity Provider is the only AMM provider, false
+ * otherwise. Return tecINTERNAL if encountered an unexpected condition,
+ * for instance Liquidity Provider has more than one LPToken trustline.
+ */
+Expected<bool, TER>
+isOnlyLiquidityProvider(ReadView const& view, Issue const& ammIssue, AccountID const& lpAccount);
+
+/** Due to rounding, the LPTokenBalance of the last LP might
+ * not match the LP's trustline balance. If it's within the tolerance,
+ * update LPTokenBalance to match the LP's trustline balance.
+ */
+Expected<bool, TER>
+verifyAndAdjustLPTokenBalance(
+    Sandbox& sb,
+    STAmount const& lpTokens,
+    std::shared_ptr<SLE>& ammSle,
+    AccountID const& account);
+
+}  // namespace xrpl
--- a/include/xrpl/ledger/helpers/DirectoryHelpers.h
+++ b/include/xrpl/ledger/helpers/DirectoryHelpers.h
@@ -189,7 +189,7 @@ forEachItem(
    AccountID const& id,
    std::function<void(std::shared_ptr<SLE const> const&)> const& f)
 {
-    forEachItem(view, keylet::ownerDir(id), f);
+    return forEachItem(view, keylet::ownerDir(id), f);
 }

 /** Iterate all items after an item in an owner directory.
--- a/include/xrpl/ledger/helpers/PaymentChannelHelpers.h
+++ b/include/xrpl/ledger/helpers/PaymentChannelHelpers.h
@@ -1,8 +1,6 @@
 #pragma once

-#include <xrpl/beast/utility/Journal.h>
 #include <xrpl/ledger/ApplyView.h>
-#include <xrpl/protocol/TER.h>
 #include <xrpl/protocol/UintTypes.h>

 namespace xrpl {
--- a/include/xrpl/net/AutoSocket.h
+++ b/include/xrpl/net/AutoSocket.h
@@ -44,7 +44,7 @@ public:
    }

    bool
-    isSecure() const
+    isSecure()
    {
        return mSecure;
    }
@@ -126,9 +126,7 @@ public:
    async_shutdown(ShutdownHandler handler)
    {
        if (isSecure())
-        {
            mSocket->async_shutdown(handler);
-        }
        else
        {
            error_code ec;
@@ -136,7 +134,7 @@ public:
            {
                lowest_layer().shutdown(plain_socket::shutdown_both);
            }
-            catch (boost::system::system_error const& e)
+            catch (boost::system::system_error& e)
            {
                ec = e.code();
            }
@@ -149,13 +147,9 @@ public:
    async_read_some(Seq const& buffers, Handler handler)
    {
        if (isSecure())
-        {
            mSocket->async_read_some(buffers, handler);
-        }
        else
-        {
            PlainSocket().async_read_some(buffers, handler);
-        }
    }

    template <typename Seq, typename Condition, typename Handler>
@@ -163,13 +157,9 @@ public:
    async_read_until(Seq const& buffers, Condition condition, Handler handler)
    {
        if (isSecure())
-        {
            boost::asio::async_read_until(*mSocket, buffers, condition, handler);
-        }
        else
-        {
            boost::asio::async_read_until(PlainSocket(), buffers, condition, handler);
-        }
    }

    template <typename Allocator, typename Handler>
@@ -180,13 +170,9 @@ public:
        Handler handler)
    {
        if (isSecure())
-        {
            boost::asio::async_read_until(*mSocket, buffers, delim, handler);
-        }
        else
-        {
            boost::asio::async_read_until(PlainSocket(), buffers, delim, handler);
-        }
    }

    template <typename Allocator, typename MatchCondition, typename Handler>
@@ -197,13 +183,9 @@ public:
        Handler handler)
    {
        if (isSecure())
-        {
            boost::asio::async_read_until(*mSocket, buffers, cond, handler);
-        }
        else
-        {
            boost::asio::async_read_until(PlainSocket(), buffers, cond, handler);
-        }
    }

    template <typename Buf, typename Handler>
@@ -211,13 +193,9 @@ public:
    async_write(Buf const& buffers, Handler handler)
    {
        if (isSecure())
-        {
            boost::asio::async_write(*mSocket, buffers, handler);
-        }
        else
-        {
            boost::asio::async_write(PlainSocket(), buffers, handler);
-        }
    }

    template <typename Allocator, typename Handler>
@@ -225,13 +203,9 @@ public:
    async_write(boost::asio::basic_streambuf<Allocator>& buffers, Handler handler)
    {
        if (isSecure())
-        {
            boost::asio::async_write(*mSocket, buffers, handler);
-        }
        else
-        {
            boost::asio::async_write(PlainSocket(), buffers, handler);
-        }
    }

    template <typename Buf, typename Condition, typename Handler>
@@ -239,13 +213,9 @@ public:
    async_read(Buf const& buffers, Condition cond, Handler handler)
    {
        if (isSecure())
-        {
            boost::asio::async_read(*mSocket, buffers, cond, handler);
-        }
        else
-        {
            boost::asio::async_read(PlainSocket(), buffers, cond, handler);
-        }
    }

    template <typename Allocator, typename Condition, typename Handler>
@@ -253,13 +223,9 @@ public:
    async_read(boost::asio::basic_streambuf<Allocator>& buffers, Condition cond, Handler handler)
    {
        if (isSecure())
-        {
            boost::asio::async_read(*mSocket, buffers, cond, handler);
-        }
        else
-        {
            boost::asio::async_read(PlainSocket(), buffers, cond, handler);
-        }
    }

    template <typename Buf, typename Handler>
@@ -267,13 +233,9 @@ public:
    async_read(Buf const& buffers, Handler handler)
    {
        if (isSecure())
-        {
            boost::asio::async_read(*mSocket, buffers, handler);
-        }
        else
-        {
            boost::asio::async_read(PlainSocket(), buffers, handler);
-        }
    }

    template <typename Seq, typename Handler>
@@ -281,13 +243,9 @@ public:
    async_write_some(Seq const& buffers, Handler handler)
    {
        if (isSecure())
-        {
            mSocket->async_write_some(buffers, handler);
-        }
        else
-        {
            PlainSocket().async_write_some(buffers, handler);
-        }
    }

 protected:
--- a/include/xrpl/net/HTTPClientSSLContext.h
+++ b/include/xrpl/net/HTTPClientSSLContext.h
@@ -30,10 +30,8 @@ public:
            registerSSLCerts(ssl_context_, ec, j_);

            if (ec && sslVerifyDir.empty())
-            {
                Throw<std::runtime_error>(boost::str(
                    boost::format("Failed to set_default_verify_paths: %s") % ec.message()));
-            }
        }
        else
        {
@@ -45,10 +43,8 @@ public:
            ssl_context_.add_verify_path(sslVerifyDir, ec);

            if (ec)
-            {
                Throw<std::runtime_error>(
                    boost::str(boost::format("Failed to add verify path: %s") % ec.message()));
-            }
        }
    }

--- a/include/xrpl/nodestore/Types.h
+++ b/include/xrpl/nodestore/Types.h
@@ -22,11 +22,11 @@ enum {

 /** Return codes from Backend operations. */
 enum Status {
-    ok = 0,
-    notFound = 1,
-    dataCorrupt = 2,
-    unknown = 3,
-    backendError = 4,
+    ok,
+    notFound,
+    dataCorrupt,
+    unknown,
+    backendError,

    customCode = 100
 };
--- a/include/xrpl/nodestore/detail/codec.h
+++ b/include/xrpl/nodestore/detail/codec.h
@@ -113,11 +113,9 @@ nodeobject_decompress(void const* in, std::size_t in_size, BufferFactory&& bf)
        {
            auto const hs = field<std::uint16_t>::size;  // Mask
            if (in_size < hs + 32)
-            {
                Throw<std::runtime_error>(
                    "nodeobject codec v1: short inner node size: " + std::string("in_size = ") +
                    std::to_string(in_size) + " hs = " + std::to_string(hs));
-            }
            istream is(p, in_size);
            std::uint16_t mask = 0;
            read<std::uint16_t>(is, mask);  // Mask
@@ -138,12 +136,10 @@ nodeobject_decompress(void const* in, std::size_t in_size, BufferFactory&& bf)
                if (mask & bit)
                {
                    if (in_size < 32)
-                    {
                        Throw<std::runtime_error>(
                            "nodeobject codec v1: short inner node subsize: " +
                            std::string("in_size = ") + std::to_string(in_size) +
                            " i = " + std::to_string(i));
-                    }
                    std::memcpy(os.data(32), is(32), 32);
                    in_size -= 32;
                }
@@ -153,20 +149,16 @@ nodeobject_decompress(void const* in, std::size_t in_size, BufferFactory&& bf)
                }
            }
            if (in_size > 0)
-            {
                Throw<std::runtime_error>(
                    "nodeobject codec v1: long inner node, in_size = " + std::to_string(in_size));
-            }
            break;
        }
        case 3:  // full v1 inner node
        {
-            if (in_size != 16 * 32)
-            {  // hashes
+            if (in_size != 16 * 32)  // hashes
                Throw<std::runtime_error>(
                    "nodeobject codec v1: short full inner node, in_size = " +
                    std::to_string(in_size));
-            }
            istream is(p, in_size);
            result.second = 525;
            void* const out = bf(result.second);
@@ -222,7 +214,7 @@ nodeobject_compress(void const* in, std::size_t in_size, BufferFactory&& bf)
                void const* const h = is(32);
                if (std::memcmp(h, zero32(), 32) == 0)
                    continue;
-                std::memcpy(vh.data() + (32 * n), h, 32);
+                std::memcpy(vh.data() + 32 * n, h, 32);
                mask |= bit;
                ++n;
            }
@@ -233,7 +225,7 @@ nodeobject_compress(void const* in, std::size_t in_size, BufferFactory&& bf)
                auto const type = 2U;
                auto const vs = size_varint(type);
                result.second = vs + field<std::uint16_t>::size +  // mask
-                    (n * 32);                                      // hashes
+                    n * 32;                                        // hashes
                std::uint8_t* out = reinterpret_cast<std::uint8_t*>(bf(result.second));
                result.first = out;
                ostream os(out, result.second);
@@ -245,7 +237,7 @@ nodeobject_compress(void const* in, std::size_t in_size, BufferFactory&& bf)
            // 3 = full v1 inner node
            auto const type = 3U;
            auto const vs = size_varint(type);
-            result.second = vs + (n * 32);  // hashes
+            result.second = vs + n * 32;  // hashes
            std::uint8_t* out = reinterpret_cast<std::uint8_t*>(bf(result.second));
            result.first = out;
            ostream os(out, result.second);
--- a/include/xrpl/nodestore/detail/varint.h
+++ b/include/xrpl/nodestore/detail/varint.h
@@ -42,10 +42,8 @@ read_varint(void const* buf, std::size_t buflen, std::size_t& t)
    std::uint8_t const* p = reinterpret_cast<std::uint8_t const*>(buf);
    std::size_t n = 0;
    while (p[n] & 0x80)
-    {
        if (++n >= buflen)
            return 0;
-    }
    if (++n > buflen)
        return 0;
    // Special case for 0
--- a/include/xrpl/protocol/AmountConversions.h
+++ b/include/xrpl/protocol/AmountConversions.h
@@ -148,17 +148,11 @@ toAmount(Asset const& asset, Number const& n, Number::rounding_mode mode = Numbe
        Number::setround(mode);

    if constexpr (std::is_same_v<IOUAmount, T>)
-    {
        return IOUAmount(n);
-    }
    else if constexpr (std::is_same_v<XRPAmount, T>)
-    {
        return XRPAmount(static_cast<std::int64_t>(n));
-    }
    else if constexpr (std::is_same_v<MPTAmount, T>)
-    {
        return MPTAmount(static_cast<std::int64_t>(n));
-    }
    else if constexpr (std::is_same_v<STAmount, T>)
    {
        if (isXRP(asset))
@@ -177,17 +171,11 @@ T
 toMaxAmount(Asset const& asset)
 {
    if constexpr (std::is_same_v<IOUAmount, T>)
-    {
        return IOUAmount(STAmount::cMaxValue, STAmount::cMaxOffset);
-    }
    else if constexpr (std::is_same_v<XRPAmount, T>)
-    {
        return XRPAmount(static_cast<std::int64_t>(STAmount::cMaxNativeN));
-    }
    else if constexpr (std::is_same_v<MPTAmount, T>)
-    {
        return MPTAmount(maxMPTokenAmount);
-    }
    else if constexpr (std::is_same_v<STAmount, T>)
    {
        return asset.visit(
@@ -216,21 +204,13 @@ Asset
 getAsset(T const& amt)
 {
    if constexpr (std::is_same_v<IOUAmount, T>)
-    {
        return noIssue();
-    }
    else if constexpr (std::is_same_v<XRPAmount, T>)
-    {
        return xrpIssue();
-    }
    else if constexpr (std::is_same_v<MPTAmount, T>)
-    {
        return noMPT();
-    }
    else if constexpr (std::is_same_v<STAmount, T>)
-    {
        return amt.asset();
-    }
    else
    {
        constexpr bool alwaysFalse = !std::is_same_v<T, T>;
@@ -243,21 +223,13 @@ constexpr T
 get(STAmount const& a)
 {
    if constexpr (std::is_same_v<IOUAmount, T>)
-    {
        return a.iou();
-    }
    else if constexpr (std::is_same_v<XRPAmount, T>)
-    {
        return a.xrp();
-    }
    else if constexpr (std::is_same_v<MPTAmount, T>)
-    {
        return a.mpt();
-    }
    else if constexpr (std::is_same_v<STAmount, T>)
-    {
        return a;
-    }
    else
    {
        constexpr bool alwaysFalse = !std::is_same_v<T, T>;
--- a/include/xrpl/protocol/ApiVersion.h
+++ b/include/xrpl/protocol/ApiVersion.h
@@ -138,11 +138,9 @@ forApiVersions(Fn const& fn, Args&&... args)
 {
    constexpr auto size = maxVer + 1 - minVer;
    [&]<std::size_t... offset>(std::index_sequence<offset...>) {
-        // NOLINTBEGIN(bugprone-use-after-move)
        (((void)fn(
             std::integral_constant<unsigned int, minVer + offset>{}, std::forward<Args>(args)...)),
         ...);
-        // NOLINTEND(bugprone-use-after-move)
    }(std::make_index_sequence<size>{});
 }

--- a/include/xrpl/protocol/Asset.h
+++ b/include/xrpl/protocol/Asset.h
@@ -221,13 +221,9 @@ operator==(Asset const& lhs, Asset const& rhs)
    return std::visit(
        [&]<typename TLhs, typename TRhs>(TLhs const& issLhs, TRhs const& issRhs) {
            if constexpr (std::is_same_v<TLhs, TRhs>)
-            {
                return issLhs == issRhs;
-            }
            else
-            {
                return false;
-            }
        },
        lhs.issue_,
        rhs.issue_);
@@ -239,17 +235,11 @@ operator<=>(Asset const& lhs, Asset const& rhs)
    return std::visit(
        []<ValidIssueType TLhs, ValidIssueType TRhs>(TLhs const& lhs_, TRhs const& rhs_) {
            if constexpr (std::is_same_v<TLhs, TRhs>)
-            {
                return std::weak_ordering(lhs_ <=> rhs_);
-            }
            else if constexpr (is_issue_v<TLhs> && is_mptissue_v<TRhs>)
-            {
                return std::weak_ordering::greater;
-            }
            else
-            {
                return std::weak_ordering::less;
-            }
        },
        lhs.issue_,
        rhs.issue_);
@@ -277,17 +267,11 @@ equalTokens(Asset const& lhs, Asset const& rhs)
    return std::visit(
        [&]<typename TLhs, typename TRhs>(TLhs const& issLhs, TRhs const& issRhs) {
            if constexpr (std::is_same_v<TLhs, Issue> && std::is_same_v<TRhs, Issue>)
-            {
                return issLhs.currency == issRhs.currency;
-            }
            else if constexpr (std::is_same_v<TLhs, MPTIssue> && std::is_same_v<TRhs, MPTIssue>)
-            {
                return issLhs.getMptID() == issRhs.getMptID();
-            }
            else
-            {
                return false;
-            }
        },
        lhs.issue_,
        rhs.issue_);
@@ -308,6 +292,9 @@ validJSONAsset(Json::Value const& jv);
 Asset
 assetFromJson(Json::Value const& jv);

+Json::Value
+to_json(Asset const& asset);
+
 inline bool
 isConsistent(Asset const& asset)
 {
--- a/include/xrpl/protocol/Book.h
+++ b/include/xrpl/protocol/Book.h
@@ -53,7 +53,7 @@ reversed(Book const& book);

 /** Equality comparison. */
 /** @{ */
-[[nodiscard]] constexpr bool
+[[nodiscard]] inline constexpr bool
 operator==(Book const& lhs, Book const& rhs)
 {
    return (lhs.in == rhs.in) && (lhs.out == rhs.out) && (lhs.domain == rhs.domain);
@@ -62,7 +62,7 @@ operator==(Book const& lhs, Book const& rhs)

 /** Strict weak ordering. */
 /** @{ */
-[[nodiscard]] constexpr std::weak_ordering
+[[nodiscard]] inline constexpr std::weak_ordering
 operator<=>(Book const& lhs, Book const& rhs)
 {
    if (auto const c{lhs.in <=> rhs.in}; c != 0)
--- a/include/xrpl/protocol/Feature.h
+++ b/include/xrpl/protocol/Feature.h
@@ -107,8 +107,8 @@ validFeatureName(auto fn) -> bool
    return true;
 }

-enum class VoteBehavior : int { Obsolete = -1, DefaultNo = 0, DefaultYes = 1 };
-enum class AmendmentSupport : int { Retired = -1, Supported = 0, Unsupported = 1 };
+enum class VoteBehavior : int { Obsolete = -1, DefaultNo = 0, DefaultYes };
+enum class AmendmentSupport : int { Retired = -1, Supported = 0, Unsupported };

 /** All amendments libxrpl knows about. */
 std::map<std::string, AmendmentSupport> const&
@@ -125,12 +125,10 @@ namespace detail {
 #pragma push_macro("XRPL_RETIRE_FIX")
 #undef XRPL_RETIRE_FIX

-// NOLINTBEGIN(bugprone-macro-parentheses)
 #define XRPL_FEATURE(name, supported, vote) +1
 #define XRPL_FIX(name, supported, vote) +1
 #define XRPL_RETIRE_FEATURE(name) +1
 #define XRPL_RETIRE_FIX(name) +1
-// NOLINTEND(bugprone-macro-parentheses)

 // This value SHOULD be equal to the number of amendments registered in
 // Feature.cpp. Because it's only used to reserve storage, and determine how
@@ -375,10 +373,8 @@ void
 foreachFeature(FeatureBitset bs, F&& f)
 {
    for (size_t i = 0; i < bs.size(); ++i)
-    {
        if (bs[i])
            f(bitsetIndexToFeature(i));
-    }
 }

 #pragma push_macro("XRPL_FEATURE")
--- a/include/xrpl/protocol/IOUAmount.h
+++ b/include/xrpl/protocol/IOUAmount.h
@@ -151,9 +151,7 @@ operator bool() const noexcept
 inline int
 IOUAmount::signum() const noexcept
 {
-    if (mantissa_ < 0)
-        return -1;
-    return (mantissa_ != 0) ? 1 : 0;
+    return (mantissa_ < 0) ? -1 : (mantissa_ ? 1 : 0);
 }

 inline IOUAmount::exponent_type
--- a/include/xrpl/protocol/Issue.h
+++ b/include/xrpl/protocol/Issue.h
@@ -12,8 +12,8 @@ namespace xrpl {
 class Issue
 {
 public:
-    Currency currency;
-    AccountID account;
+    Currency currency{};
+    AccountID account{};

    Issue() = default;

@@ -68,7 +68,7 @@ hash_append(Hasher& h, Issue const& r)

 /** Equality comparison. */
 /** @{ */
-[[nodiscard]] constexpr bool
+[[nodiscard]] inline constexpr bool
 operator==(Issue const& lhs, Issue const& rhs)
 {
    return (lhs.currency == rhs.currency) && (isXRP(lhs.currency) || lhs.account == rhs.account);
--- a/include/xrpl/protocol/KnownFormats.h
+++ b/include/xrpl/protocol/KnownFormats.h
@@ -74,12 +74,10 @@ public:

        Derived classes will load the object with all the known formats.
    */
-private:
    KnownFormats() : name_(beast::type_name<Derived>())
    {
    }

-public:
    /** Destroy the known formats object.

        The defined formats are deleted.
@@ -183,7 +181,6 @@ private:

    boost::container::flat_map<std::string, Item const*> names_;
    boost::container::flat_map<KeyType, Item const*> types_;
-    friend Derived;
 };

 }  // namespace xrpl
--- a/include/xrpl/protocol/LedgerFormats.h
+++ b/include/xrpl/protocol/LedgerFormats.h
@@ -211,7 +211,7 @@ enum LedgerEntryType : std::uint16_t {
 //     lsfRequireDestTag = 0x00020000,
 //     ...
 // };
-#define TO_VALUE(name, value) name = (value),
+#define TO_VALUE(name, value) name = value,
 #define NULL_NAME(name, values) values
 #define NULL_OUTPUT(name, value)
 enum LedgerSpecificFlags : std::uint32_t { XMACRO(NULL_NAME, TO_VALUE, NULL_OUTPUT) };
--- a/include/xrpl/protocol/LedgerHeader.h
+++ b/include/xrpl/protocol/LedgerHeader.h
@@ -19,7 +19,7 @@ struct LedgerHeader
    //

    LedgerIndex seq = 0;
-    NetClock::time_point parentCloseTime;
+    NetClock::time_point parentCloseTime = {};

    //
    // For closed ledgers
@@ -49,7 +49,7 @@ struct LedgerHeader
    // closed. For open ledgers, the time the ledger
    // will close if there's no transactions.
    //
-    NetClock::time_point closeTime;
+    NetClock::time_point closeTime = {};
 };

 // ledger close flags
--- a/include/xrpl/protocol/MPTAmount.h
+++ b/include/xrpl/protocol/MPTAmount.h
@@ -109,9 +109,7 @@ operator bool() const noexcept
 constexpr int
 MPTAmount::signum() const noexcept
 {
-    if (value_ < 0)
-        return -1;
-    return (value_ != 0) ? 1 : 0;
+    return (value_ < 0) ? -1 : (value_ ? 1 : 0);
 }

 /** Returns the underlying value. Code SHOULD NOT call this
@@ -143,7 +141,7 @@ mulRatio(MPTAmount const& amt, std::uint32_t num, std::uint32_t den, bool roundU
 {
    using namespace boost::multiprecision;

-    if (den == 0u)
+    if (!den)
        Throw<std::runtime_error>("division by zero");

    int128_t const amt128(amt.value());
--- a/include/xrpl/protocol/MPTIssue.h
+++ b/include/xrpl/protocol/MPTIssue.h
@@ -47,14 +47,14 @@ public:
    friend constexpr std::weak_ordering
    operator<=>(MPTIssue const& lhs, MPTIssue const& rhs);

-    static bool
-    native()
+    bool
+    native() const
    {
        return false;
    }

-    static bool
-    integral()
+    bool
+    integral() const
    {
        return true;
    }
@@ -94,9 +94,9 @@ getMPTIssuer(MPTID const& mptid)
 }

 // Disallow temporary
-AccountID const&
+inline AccountID const&
 getMPTIssuer(MPTID const&&) = delete;
-AccountID const&
+inline AccountID const&
 getMPTIssuer(MPTID&&) = delete;

 inline MPTID
--- a/include/xrpl/protocol/MultiApiJson.h
+++ b/include/xrpl/protocol/MultiApiJson.h
@@ -74,14 +74,10 @@ struct MultiApiJson
    {
        int count = 0;
        for (auto& a : this->val)
-        {
            if (a.isMember(key))
                count += 1;
-        }

-        if (count == 0)
-            return none;
-        return count < size ? some : all;
+        return (count == 0 ? none : (count < size ? some : all));
    }

    static constexpr struct visitor_t final
--- a/include/xrpl/protocol/PathAsset.h
+++ b/include/xrpl/protocol/PathAsset.h
@@ -100,13 +100,9 @@ operator==(PathAsset const& lhs, PathAsset const& rhs)
    return std::visit(
        []<ValidPathAsset TLhs, ValidPathAsset TRhs>(TLhs const& lhs_, TRhs const& rhs_) {
            if constexpr (std::is_same_v<TLhs, TRhs>)
-            {
                return lhs_ == rhs_;
-            }
            else
-            {
                return false;
-            }
        },
        lhs.value(),
        rhs.value());
--- a/include/xrpl/protocol/Permissions.h
+++ b/include/xrpl/protocol/Permissions.h
@@ -20,7 +20,7 @@ enum GranularPermissionType : std::uint32_t {
 #pragma push_macro("PERMISSION")
 #undef PERMISSION

-#define PERMISSION(type, txType, value) type = (value),
+#define PERMISSION(type, txType, value) type = value,

 #include <xrpl/protocol/detail/permissions.macro>

--- a/include/xrpl/protocol/PublicKey.h
+++ b/include/xrpl/protocol/PublicKey.h
@@ -69,8 +69,8 @@ public:
        return buf_;
    }

-    static std::size_t
-    size() noexcept
+    std::size_t
+    size() const noexcept
    {
        return size_;
    }
--- a/include/xrpl/protocol/Quality.h
+++ b/include/xrpl/protocol/Quality.h
@@ -281,7 +281,7 @@ public:

        double const minVD = static_cast<double>(minVMantissa);
        double const maxVD =
-            (expDiff != 0) ? maxVMantissa * pow(10, expDiff) : static_cast<double>(maxVMantissa);
+            expDiff ? maxVMantissa * pow(10, expDiff) : static_cast<double>(maxVMantissa);

        // maxVD and minVD are scaled so they have the same exponents. Dividing
        // cancels out the exponents, so we only need to deal with the (scaled)
--- a/include/xrpl/protocol/RPCErr.h
+++ b/include/xrpl/protocol/RPCErr.h
@@ -1,7 +1,6 @@
 #pragma once

 #include <xrpl/json/json_value.h>
-#include <xrpl/protocol/ErrorCodes.h>

 namespace xrpl {

--- a/include/xrpl/protocol/SField.h
+++ b/include/xrpl/protocol/SField.h
@@ -84,7 +84,7 @@ class STCurrency;
 #pragma push_macro("TO_MAP")
 #undef TO_MAP

-#define TO_ENUM(name, value) name = (value),
+#define TO_ENUM(name, value) name = value,
 #define TO_MAP(name, value) {#name, value},

 enum SerializedTypeID { XMACRO(TO_ENUM) };
--- a/include/xrpl/protocol/STAmount.h
+++ b/include/xrpl/protocol/STAmount.h
@@ -48,7 +48,7 @@ public:
    // Maximum native value supported by the code
    constexpr static std::uint64_t cMinValue = 1'000'000'000'000'000ull;
    static_assert(isPowerOfTen(cMinValue));
-    constexpr static std::uint64_t cMaxValue = (cMinValue * 10) - 1;
+    constexpr static std::uint64_t cMaxValue = cMinValue * 10 - 1;
    static_assert(cMaxValue == 9'999'999'999'999'999ull);
    constexpr static std::uint64_t cMaxNative = 9'000'000'000'000'000'000ull;

@@ -359,13 +359,9 @@ inline STAmount::STAmount(IOUAmount const& amount, Issue const& issue)
    : mAsset(issue), mOffset(amount.exponent()), mIsNegative(amount < beast::zero)
 {
    if (mIsNegative)
-    {
        mValue = unsafe_cast<std::uint64_t>(-amount.mantissa());
-    }
    else
-    {
        mValue = unsafe_cast<std::uint64_t>(amount.mantissa());
-    }

    canonicalize();
 }
@@ -374,13 +370,9 @@ inline STAmount::STAmount(MPTAmount const& amount, MPTIssue const& mptIssue)
    : mAsset(mptIssue), mOffset(0), mIsNegative(amount < beast::zero)
 {
    if (mIsNegative)
-    {
        mValue = unsafe_cast<std::uint64_t>(-amount.value());
-    }
    else
-    {
        mValue = unsafe_cast<std::uint64_t>(amount.value());
-    }

    canonicalize();
 }
@@ -409,7 +401,7 @@ amountFromJsonNoThrow(STAmount& result, Json::Value const& jvSource);
 inline STAmount const&
 toSTAmount(STAmount const& a)
 {
-    return a;  // NOLINT(bugprone-return-const-ref-from-parameter)
+    return a;
 }

 //------------------------------------------------------------------------------
@@ -484,9 +476,7 @@ STAmount::getIssuer() const
 inline int
 STAmount::signum() const noexcept
 {
-    if (mValue == 0u)
-        return 0;
-    return mIsNegative ? -1 : 1;
+    return mValue ? (mIsNegative ? -1 : 1) : 0;
 }

 inline STAmount
--- a/include/xrpl/protocol/STBlob.h
+++ b/include/xrpl/protocol/STBlob.h
@@ -95,7 +95,7 @@ STBlob::size() const
 inline std::uint8_t const*
 STBlob::data() const
 {
-    return value_.data();
+    return reinterpret_cast<std::uint8_t const*>(value_.data());
 }

 inline STBlob&
--- a/include/xrpl/protocol/STCurrency.h
+++ b/include/xrpl/protocol/STCurrency.h
@@ -11,7 +11,7 @@ namespace xrpl {
 class STCurrency final : public STBase
 {
 private:
-    Currency currency_;
+    Currency currency_{};

 public:
    using value_type = Currency;
--- a/include/xrpl/protocol/STObject.h
+++ b/include/xrpl/protocol/STObject.h
@@ -704,7 +704,7 @@ class STObject::FieldErr : public std::runtime_error
 template <class T>
 STObject::Proxy<T>::Proxy(STObject* st, TypedField<T> const* f) : st_(st), f_(f)
 {
-    if (st_->mType != nullptr)
+    if (st_->mType)
    {
        // STObject has associated template
        if (!st_->peekAtPField(*f_))
@@ -770,13 +770,9 @@ STObject::Proxy<T>::assign(U&& u)
    }
    T* t = nullptr;
    if (style_ == soeINVALID)
-    {
        t = dynamic_cast<T*>(st_->getPField(*f_, true));
-    }
    else
-    {
        t = dynamic_cast<T*>(st_->makeFieldPresent(*f_));
-    }
    XRPL_ASSERT(t, "xrpl::STObject::Proxy::assign : type cast succeeded");
    *t = std::forward<U>(u);
 }
@@ -862,13 +858,9 @@ STObject::OptionalProxy<T>::operator=(
    -> OptionalProxy&
 {
    if (v)
-    {
        this->assign(std::move(*v));
-    }
    else
-    {
        disengage();
-    }
    return *this;
 }

@@ -877,13 +869,9 @@ auto
 STObject::OptionalProxy<T>::operator=(optional_type const& v) -> OptionalProxy&
 {
    if (v)
-    {
        this->assign(*v);
-    }
    else
-    {
        disengage();
-    }
    return *this;
 }

@@ -915,13 +903,9 @@ STObject::OptionalProxy<T>::disengage()
    if (this->style_ == soeREQUIRED || this->style_ == soeDEFAULT)
        Throw<STObject::FieldErr>("Template field error '" + this->f_->getName() + "'");
    if (this->style_ == soeINVALID)
-    {
        this->st_->delField(*this->f_);
-    }
    else
-    {
        this->st_->makeFieldAbsent(*this->f_);
-    }
 }

 template <class T>
@@ -1074,11 +1058,9 @@ STObject::at(TypedField<T> const& f) const
 {
    auto const b = peekAtPField(f);
    if (!b)
-    {
        // This is a free object (no constraints)
        // with no template
        Throw<STObject::FieldErr>("Missing field: " + f.getName());
-    }

    if (auto const u = dynamic_cast<T const*>(b))
        return u->value();
@@ -1156,13 +1138,9 @@ STObject::setFieldH160(SField const& field, base_uint<160, Tag> const& v)

    using Bits = STBitString<160>;
    if (auto cf = dynamic_cast<Bits*>(rf))
-    {
        cf->setValue(v);
-    }
    else
-    {
        Throw<std::runtime_error>("Wrong field type");
-    }
 }

 inline bool
@@ -1210,10 +1188,7 @@ STObject::getFieldByConstRef(SField const& field, V const& empty) const
    SerializedTypeID const id = rf->getSType();

    if (id == STI_NOTPRESENT)
-    {
-        // NOLINTNEXTLINE(bugprone-return-const-ref-from-parameter)
        return empty;  // optional field not present
-    }

    T const* cf = dynamic_cast<T const*>(rf);

--- a/include/xrpl/protocol/STValidation.h
+++ b/include/xrpl/protocol/STValidation.h
@@ -36,7 +36,7 @@ class STValidation final : public STObject, public CountedObject<STValidation>
    // that use manifests this will be derived from the master public key.
    NodeID const nodeID_;

-    NetClock::time_point seenTime_;
+    NetClock::time_point seenTime_ = {};

 public:
    /** Construct a STValidation from a peer from serialized data.
--- a/include/xrpl/protocol/STVector256.h
+++ b/include/xrpl/protocol/STVector256.h
@@ -148,7 +148,7 @@ STVector256::size() const
 inline void
 STVector256::resize(std::size_t n)
 {
-    mValue.resize(n);
+    return mValue.resize(n);
 }

 inline bool
@@ -220,7 +220,7 @@ STVector256::erase(std::vector<uint256>::iterator position)
 inline void
 STVector256::clear() noexcept
 {
-    mValue.clear();
+    return mValue.clear();
 }

 }  // namespace xrpl
--- a/include/xrpl/protocol/SecretKey.h
+++ b/include/xrpl/protocol/SecretKey.h
@@ -85,10 +85,10 @@ public:
    }
 };

-bool
+inline bool
 operator==(SecretKey const& lhs, SecretKey const& rhs) = delete;

-bool
+inline bool
 operator!=(SecretKey const& lhs, SecretKey const& rhs) = delete;

 //------------------------------------------------------------------------------
--- a/include/xrpl/protocol/Serializer.h
+++ b/include/xrpl/protocol/Serializer.h
@@ -33,7 +33,7 @@ public:
    {
        mData.resize(size);

-        if (size != 0u)
+        if (size)
        {
            XRPL_ASSERT(data, "xrpl::Serializer::Serializer(void const*) : non-null input");
            std::memcpy(mData.data(), data, size);
--- a/include/xrpl/protocol/TER.h
+++ b/include/xrpl/protocol/TER.h
@@ -344,6 +344,10 @@ enum TECcodes : TERUnderlyingType {
    tecLIMIT_EXCEEDED = 195,
    tecPSEUDO_ACCOUNT = 196,
    tecPRECISION_LOSS = 197,
+    // DEPRECATED: This error code tecNO_DELEGATE_PERMISSION is reserved for
+    // backward compatibility with historical data on non-prod networks, can be
+    // reclaimed after those networks reset.
+    tecNO_DELEGATE_PERMISSION = 198,
 };

 //------------------------------------------------------------------------------
--- a/include/xrpl/protocol/TxFlags.h
+++ b/include/xrpl/protocol/TxFlags.h
@@ -235,7 +235,7 @@ XMACRO(NULL_NAME, TO_VALUE, NULL_OUTPUT, NULL_MASK_ADJ)
 // The mask adjustment (maskAdj) allows adding flags back to the mask, making them invalid.
 // For example, Batch uses MASK_ADJ(tfInnerBatchTxn) to reject tfInnerBatchTxn on outer Batch.
 #define TO_MASK(name, values, maskAdj) \
-    inline constexpr FlagValue tf##name##Mask = ~(tfUniversal values) | (maskAdj);
+    inline constexpr FlagValue tf##name##Mask = ~(tfUniversal values) | maskAdj;
 #define VALUE_TO_MASK(name, value) | name
 #define MASK_ADJ_TO_MASK(value) value
 XMACRO(TO_MASK, VALUE_TO_MASK, VALUE_TO_MASK, MASK_ADJ_TO_MASK)
--- a/include/xrpl/protocol/Units.h
+++ b/include/xrpl/protocol/Units.h
@@ -267,9 +267,7 @@ public:
    constexpr int
    signum() const noexcept
    {
-        if (value_ < 0)
-            return -1;
-        return value_ ? 1 : 0;
+        return (value_ < 0) ? -1 : (value_ ? 1 : 0);
    }

    /** Returns the number of drops */
--- a/Show More
+++ b/Show More
Author	SHA1	Message	Date
Mayukha Vadari	1259c1d5ca	Merge branch 'develop' of https://github.com/XRPLF/rippled into ripple/wasmi	2026-04-08 13:48:41 -04:00
Mayukha Vadari	9917f96166	Merge remote-tracking branch 'upstream/develop' into ripple/wasmi	2026-04-07 16:02:56 -04:00
Pratik Mankawde	2cc9439fde	fix: Handle WSClient write failure when server closes WebSocket (#6671 ) Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>	2026-04-07 16:01:26 -04:00
Ayaz Salikhov	52af9582e2	ci: Change conditions for uploading artifacts in public/private/org repos (#6734 )	2026-04-07 16:01:26 -04:00
Bart	46e88dc732	refactor: Rename non-functional uses of `ripple(d)` to `xrpl(d)` (#6676 ) Co-authored-by: Bart <11445373+bthomee@users.noreply.github.com>	2026-04-07 16:01:26 -04:00
Mayukha Vadari	bc24f2e211	refactor: Move more helper files into `libxrpl/ledger/helpers` (#6731 ) Co-authored-by: xrplf-ai-reviewer[bot] <266832837+xrplf-ai-reviewer[bot]@users.noreply.github.com>	2026-04-07 16:01:26 -04:00
Mayukha Vadari	7a7c993b15	fix: Minor RPC fixes (#6730 )	2026-04-07 16:01:26 -04:00
Zhiyuan Wang	9733ca8f91	fix: Prevent deletion of MPTokens with active escrow (#6635 ) Co-authored-by: Bart <bthomee@users.noreply.github.com>	2026-04-07 16:01:26 -04:00
Vito Tumas	18d5e3e226	fix: Clamp VaultClawback to assetsAvailable for zero-amount clawback (#6646 )	2026-04-07 16:01:25 -04:00
Vito Tumas	b30b4e1d65	fix: Add assorted Lending Protocol fixes (#6678 ) Co-authored-by: Shawn Xie <35279399+shawnxie999@users.noreply.github.com>	2026-04-07 16:01:25 -04:00
Mayukha Vadari	d435893602	fix: Change variable signedness and correctly handle `std::optional` (#6657 )	2026-04-07 16:01:25 -04:00
Olek	00b0cf50f6	Update wasmi to 1.0.9 (#6727 )	2026-04-07 15:58:29 -04:00
Mayukha Vadari	1338062be7	Merge branch 'develop' of https://github.com/XRPLF/rippled into ripple/wasmi	2026-04-03 09:56:55 -04:00
Mayukha Vadari	1d4a3c00b8	Merge branch 'develop' of https://github.com/XRPLF/rippled into ripple/wasmi	2026-04-02 17:53:53 -04:00
Mayukha Vadari	2930ef217f	Merge remote-tracking branch 'upstream/develop' into ripple/wasmi	2026-03-24 10:20:38 -07:00
Mayukha Vadari	913e4b919e	Merge remote-tracking branch 'upstream/develop' into ripple/wasmi	2026-03-24 08:41:12 -07:00
Mayukha Vadari	bce5d91e45	Merge branch 'develop' into ripple/wasmi	2026-03-12 14:37:01 -04:00
Mayukha Vadari	654338fa66	Merge branch 'develop' into ripple/wasmi	2026-03-06 16:27:50 -04:00
Mayukha Vadari	3a825a41e1	Merge branch 'develop' into ripple/wasmi	2026-03-05 13:48:16 -04:00
Mayukha Vadari	61bcb7621f	Merge branch 'develop' into ripple/wasmi	2026-03-03 11:18:26 -04:00
Mayukha Vadari	a3f71b1774	Merge branch 'develop' into ripple/wasmi	2026-03-02 17:06:17 -05:00
Mayukha Vadari	b08bcf5d21	Merge branch 'develop' into ripple/wasmi	2026-02-27 16:41:44 -05:00
Mayukha Vadari	77dfd56ace	Merge branch 'develop' into ripple/wasmi	2026-02-27 13:49:25 -05:00
Mayukha Vadari	9c3c0280b1	Merge branch 'develop' into ripple/wasmi	2026-02-18 12:35:51 -05:00
Mayukha Vadari	bd16f7989d	Merge branch 'develop' into ripple/wasmi	2026-02-10 17:26:33 -05:00
Mayukha Vadari	2ec4a1114e	Merge branch 'develop' into ripple/wasmi	2026-02-04 18:13:00 -05:00
Mayukha Vadari	b2627039f6	Merge branch 'develop' into ripple/wasmi	2026-02-03 14:51:59 -05:00
Mayukha Vadari	e85e7b1b1a	Merge branch 'develop' into ripple/wasmi	2026-01-29 13:53:55 -05:00
Mayukha Vadari	72fffb6e51	Merge branch 'develop' into ripple/wasmi	2026-01-28 15:56:18 -05:00
Mayukha Vadari	f7ee580f01	Merge commit '5f638f55536def0d88b970d1018a465a238e55f4' into ripple/wasmi	2026-01-28 15:56:11 -05:00
Mayukha Vadari	122d405750	Merge commit '92046785d1fea5f9efe5a770d636792ea6cab78b' into ripple/wasmi	2026-01-28 15:56:04 -05:00
Mayukha Vadari	d7ed6d6512	Merge branch 'develop' into ripple/wasmi	2026-01-27 13:26:39 -05:00
Mayukha Vadari	8bc6f9cd70	Merge branch 'develop' into ripple/wasmi	2026-01-23 13:13:11 -05:00
Mayukha Vadari	ed5139d4e3	Merge branch 'develop' into ripple/wasmi	2026-01-21 12:57:29 -05:00
Mayukha Vadari	7a9d245950	Merge branch 'develop' into ripple/wasmi	2026-01-14 13:01:35 -05:00
Olek	d83ec96848	Switch to wasmi v1.0.6 (#6204 )	2026-01-12 13:36:02 -05:00
Mayukha Vadari	419d53ec4c	Merge branch 'develop' into ripple/wasmi	2026-01-12 13:10:58 -05:00
Mayukha Vadari	d4d70d5675	Merge branch 'develop' into ripple/wasmi	2026-01-12 12:27:48 -05:00
Mayukha Vadari	bbc28b3b1c	Merge branch 'develop' into ripple/wasmi	2026-01-08 11:42:28 -05:00
Mayukha Vadari	5aab274b7a	Merge branch 'develop' into ripple/wasmi	2026-01-07 16:52:10 -05:00
Mayukha Vadari	2c30e41191	use the develop hashes	2026-01-07 16:50:45 -05:00
Mayukha Vadari	8ea5106b0b	Merge branch 'develop' into ripple/wasmi	2026-01-07 14:34:49 -05:00
Mayukha Vadari	1977df9c2e	Merge remote-tracking branch 'upstream/develop' into ripple/wasmi	2026-01-05 18:43:49 -05:00
Mayukha Vadari	6c95548df5	Merge remote-tracking branch 'upstream/develop' into ripple/wasmi	2025-12-22 15:51:19 -08:00
Mayukha Vadari	90e0bbd0fc	Merge branch 'develop' into ripple/wasmi	2025-12-08 14:28:41 -05:00
Olek	b57df290de	Use conan repo for wasmi lib (#6109 ) * Use conan repo for wasmi lib * Generate lockfile	2025-12-08 13:02:01 -05:00
Mayukha Vadari	8a403f1241	Merge branch 'develop' into ripple/wasmi	2025-12-05 14:32:48 -05:00
Mayukha Vadari	6d2640871d	Merge branch 'develop' into ripple/wasmi	2025-12-02 18:40:54 -05:00
Olek	500bb68831	Fix win build (#6076 )	2025-11-24 16:56:23 -05:00
Mayukha Vadari	16087c9680	fix merge issue	2025-11-25 02:57:47 +05:30
Mayukha Vadari	25c3060fef	remove conan.lock (temporary)	2025-11-25 02:40:57 +05:30
Mayukha Vadari	ce9f0b38a4	Merge branch 'develop' into ripple/wasmi	2025-11-25 02:33:47 +05:30
Mayukha Vadari	35f7cbf772	update	2025-11-25 02:31:51 +05:30
Mayukha Vadari	0db564d261	WASMI data	2025-11-04 15:57:07 -05:00