style: Update pre-commit hooks

.github/workflows/check-libxrpl.yml

@@ -33,6 +33,9 @@ jobs:
         with:
           enable_ccache: false
 
+      - name: Print build environment
+        uses: XRPLF/actions/print-env@8f811899319ef002e7a088c77c57d95aba602a97
+
       - name: Update libXRPL version requirement
         run: |
           sed -i.bak -E "s|'xrpl/[a-zA-Z0-9\\.\\-]+'|'xrpl/${{ github.event.client_payload.conan_ref }}'|g" conanfile.py

.github/workflows/clang-tidy.yml

@@ -20,7 +20,7 @@ concurrency:
 
 env:
   CONAN_PROFILE: clang
-  LLVM_TOOLS_VERSION: 20
+  LLVM_TOOLS_VERSION: 21
 
 defaults:
   run:
@@ -31,7 +31,7 @@ jobs:
     if: github.event_name != 'push' || contains(github.event.head_commit.message, 'clang-tidy auto fixes')
     runs-on: heavy
     container:
-      image: ghcr.io/xrplf/clio-ci:14342e087ceb8b593027198bf9ef06a43833c696
+      image: ghcr.io/xrplf/clio-ci:f174b47f4909ae41b80406d836ab52adc39eacc6
 
     permissions:
       contents: write
@@ -48,6 +48,9 @@ jobs:
         with:
           enable_ccache: false
 
+      - name: Print build environment
+        uses: XRPLF/actions/print-env@8f811899319ef002e7a088c77c57d95aba602a97
+
       - name: Run conan
         uses: ./.github/actions/conan
         with:
@@ -84,6 +87,19 @@ jobs:
           pre-commit run --all-files fix-local-includes || true
           pre-commit run --all-files clang-format || true
 
+      - name: Generate git diff
+        if: ${{ steps.files_changed.outcome != 'success' }}
+        run: |
+          git diff | tee clang-tidy-git-diff.txt
+
+      - name: Upload clang-tidy diff output
+        if: ${{ steps.files_changed.outcome != 'success' }}
+        uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1
+        with:
+          name: clang-tidy-git-diff
+          path: clang-tidy-git-diff.txt
+          retention-days: 30
+
       - name: Create an issue
         if: ${{ (steps.clang_tidy.outcome != 'success' || steps.files_changed.outcome != 'success') && github.event_name != 'pull_request' }}
         id: create_issue

.github/workflows/docs.yml

@@ -31,6 +31,9 @@ jobs:
         with:
           enable_ccache: false
 
+      - name: Print build environment
+        uses: XRPLF/actions/print-env@8f811899319ef002e7a088c77c57d95aba602a97
+
       - name: Create build directory
         run: mkdir build_docs
 

.github/workflows/pre-commit-autoupdate.yml

@@ -12,7 +12,7 @@ on:
 
 jobs:
   auto-update:
-    uses: XRPLF/actions/.github/workflows/pre-commit-autoupdate.yml@a249154199805d6809359fe99fa8ba09dd804e3d
+    uses: XRPLF/actions/.github/workflows/pre-commit-autoupdate.yml@5e942d61bf32f7557a7c159cfac4712a687b3e3a
     with:
       sign_commit: true
       committer: "Clio CI <skuznetsov@ripple.com>"

.github/workflows/pre-commit.yml

@@ -8,7 +8,7 @@ on:
 
 jobs:
   run-hooks:
-    uses: XRPLF/actions/.github/workflows/pre-commit.yml@9307df762265e15c745ddcdb38a581c989f7f349
+    uses: XRPLF/actions/.github/workflows/pre-commit.yml@5e942d61bf32f7557a7c159cfac4712a687b3e3a
     with:
       runs_on: heavy
       container: '{ "image": "ghcr.io/xrplf/clio-pre-commit:14342e087ceb8b593027198bf9ef06a43833c696" }'

.github/workflows/reusable-build.yml

@@ -99,6 +99,9 @@ jobs:
         with:
           enable_ccache: ${{ inputs.download_ccache }}
 
+      - name: Print build environment
+        uses: XRPLF/actions/print-env@8f811899319ef002e7a088c77c57d95aba602a97
+
       - name: Setup conan on macOS
         if: ${{ runner.os == 'macOS' }}
         run: ./.github/scripts/conan/init.sh

.github/workflows/upload-conan-deps.yml

@@ -82,6 +82,9 @@ jobs:
         with:
           enable_ccache: false
 
+      - name: Print build environment
+        uses: XRPLF/actions/print-env@8f811899319ef002e7a088c77c57d95aba602a97
+
       - name: Setup conan on macOS
         if: ${{ runner.os == 'macOS' }}
         run: ./.github/scripts/conan/init.sh

.pre-commit-config.yaml

@@ -29,7 +29,7 @@ repos:
 
   # Autoformat: YAML, JSON, Markdown, etc.
   - repo: https://github.com/rbubley/mirrors-prettier
-    rev: c2bc67fe8f8f549cc489e00ba8b45aa18ee713b1 # frozen: v3.8.1
+    rev: 515f543f5718ebfd6ce22e16708bb32c68ff96e1  # frozen: v3.8.3
     hooks:
       - id: prettier
 
@@ -64,7 +64,7 @@ repos:
       - id: black
 
   - repo: https://github.com/scop/pre-commit-shfmt
-    rev: e26a818fd47b4f33cefa99035d1265b0849f4b47 # frozen: v3.13.0-1
+    rev: 05c1426671b9237fb5e1444dd63aa5731bec0dfb  # frozen: v3.13.1-1
     hooks:
       - id: shfmt
         args: ["-i", "4", "--write"]
@@ -94,7 +94,7 @@ repos:
         language: script
 
   - repo: https://github.com/pre-commit/mirrors-clang-format
-    rev: 07a0f7667439f60724899f6ae288e4a4f572e0e1 # frozen: v22.1.2
+    rev: 39233709be54124a7371a50cbfc5325bd4fb9d90  # frozen: v22.1.4
     hooks:
       - id: clang-format
         args: [--style=file]

docker/ci/Dockerfile

@@ -61,7 +61,7 @@ RUN pip install -q --no-cache-dir \
         pre-commit
 
 # Install LLVM tools
-ARG LLVM_TOOLS_VERSION=20
+ARG LLVM_TOOLS_VERSION=21
 
 RUN echo "deb http://apt.llvm.org/focal/ llvm-toolchain-focal-${LLVM_TOOLS_VERSION} main" >> /etc/apt/sources.list \
     && wget --progress=dot:giga -O - https://apt.llvm.org/llvm-snapshot.gpg.key | apt-key add -

docker/ci/README.md

@@ -14,6 +14,7 @@ The image is based on Ubuntu 20.04 and contains:
 - GDB 17.1
 - gh 2.83.2
 - git-cliff 2.11.0
+- LLVM Tools 21
 - mold 2.40.4
 - Ninja 1.13.2
 - Python 3.8

src/migration/cassandra/impl/Spec.hpp

@@ -13,7 +13,7 @@ concept TableSpec = requires {
     // Check that 'row' exists and is a tuple
     // keys types are at the beginning and the other fields types sort in alphabetical order
     typename T::Row;
-    requires std::tuple_size<typename T::Row>::value >= 0;  // Ensures 'row' is a tuple
+    requires std::tuple_size_v<typename T::Row> >= 0;  // Ensures 'row' is a tuple
 
     // Check that static constexpr members 'partitionKey' and 'tableName' exist
     { T::kPARTITION_KEY } -> std::convertible_to<char const*>;

tests/unit/feed/SubscriptionManagerTests.cpp

@@ -120,10 +120,10 @@ TEST_F(SubscriptionManagerTest, ReportCurrentSubscriber)
             "book_changes": 2
         })JSON";
     web::SubscriptionContextPtr const session1 = std::make_shared<MockSession>();
-    MockSession* mockSession1 = dynamic_cast<MockSession*>(session1.get());
+    auto const* mockSession1 = dynamic_cast<MockSession const*>(session1.get());
 
     web::SubscriptionContextPtr session2 = std::make_shared<MockSession>();
-    MockSession* mockSession2 = dynamic_cast<MockSession*>(session2.get());
+    auto const* mockSession2 = dynamic_cast<MockSession const*>(session2.get());
     std::vector<web::SubscriptionContextInterface::OnDisconnectSlot> session2OnDisconnectSlots;
     ON_CALL(*mockSession2, onDisconnect).WillByDefault([&session2OnDisconnectSlots](auto slot) {
         session2OnDisconnectSlots.push_back(slot);

tests/unit/rpc/handlers/ServerInfoTests.cpp

@@ -166,8 +166,8 @@ TEST_F(RPCServerInfoHandlerTest, NoFeesErrorsOutWithInternal)
 TEST_F(RPCServerInfoHandlerTest, DefaultOutputIsPresent)
 {
     MockLoadBalancer* rawBalancerPtr = mockLoadBalancerPtr_.get();
-    MockCounters* rawCountersPtr = mockCountersPtr_.get();
-    MockETLService* rawETLServicePtr = mockETLServicePtr_.get();
+    MockCounters const* rawCountersPtr = mockCountersPtr_.get();
+    MockETLService const* rawETLServicePtr = mockETLServicePtr_.get();
 
     auto const ledgerHeader = createLedgerHeader(kLEDGER_HASH, 30, 3);  // 3 seconds old
     EXPECT_CALL(*backend_, fetchLedgerBySequence).WillOnce(Return(ledgerHeader));
@@ -209,8 +209,8 @@ TEST_F(RPCServerInfoHandlerTest, DefaultOutputIsPresent)
 TEST_F(RPCServerInfoHandlerTest, AmendmentBlockedIsPresentIfSet)
 {
     MockLoadBalancer* rawBalancerPtr = mockLoadBalancerPtr_.get();
-    MockCounters* rawCountersPtr = mockCountersPtr_.get();
-    MockETLService* rawETLServicePtr = mockETLServicePtr_.get();
+    MockCounters const* rawCountersPtr = mockCountersPtr_.get();
+    MockETLService const* rawETLServicePtr = mockETLServicePtr_.get();
 
     auto const ledgerHeader = createLedgerHeader(kLEDGER_HASH, 30, 3);  // 3 seconds old
     EXPECT_CALL(*backend_, fetchLedgerBySequence).WillOnce(Return(ledgerHeader));
@@ -250,8 +250,8 @@ TEST_F(RPCServerInfoHandlerTest, AmendmentBlockedIsPresentIfSet)
 TEST_F(RPCServerInfoHandlerTest, CorruptionDetectedIsPresentIfSet)
 {
     MockLoadBalancer* rawBalancerPtr = mockLoadBalancerPtr_.get();
-    MockCounters* rawCountersPtr = mockCountersPtr_.get();
-    MockETLService* rawETLServicePtr = mockETLServicePtr_.get();
+    MockCounters const* rawCountersPtr = mockCountersPtr_.get();
+    MockETLService const* rawETLServicePtr = mockETLServicePtr_.get();
 
     auto const ledgerHeader = createLedgerHeader(kLEDGER_HASH, 30, 3);  // 3 seconds old
     EXPECT_CALL(*backend_, fetchLedgerBySequence).WillOnce(Return(ledgerHeader));
@@ -291,7 +291,7 @@ TEST_F(RPCServerInfoHandlerTest, CorruptionDetectedIsPresentIfSet)
 TEST_F(RPCServerInfoHandlerTest, CacheReportsEnabledFlagCorrectly)
 {
     MockLoadBalancer* rawBalancerPtr = mockLoadBalancerPtr_.get();
-    MockCounters* rawCountersPtr = mockCountersPtr_.get();
+    MockCounters const* rawCountersPtr = mockCountersPtr_.get();
 
     auto const ledgerHeader = createLedgerHeader(kLEDGER_HASH, 30, 3);  // 3 seconds old
     EXPECT_CALL(*backend_, fetchLedgerBySequence).Times(2).WillRepeatedly(Return(ledgerHeader));
@@ -346,9 +346,9 @@ TEST_F(RPCServerInfoHandlerTest, CacheReportsEnabledFlagCorrectly)
 
 TEST_F(RPCServerInfoHandlerTest, AdminSectionPresentWhenAdminFlagIsSet)
 {
-    MockLoadBalancer* rawBalancerPtr = mockLoadBalancerPtr_.get();
-    MockCounters* rawCountersPtr = mockCountersPtr_.get();
-    MockETLService* rawETLServicePtr = mockETLServicePtr_.get();
+    MockLoadBalancer const* rawBalancerPtr = mockLoadBalancerPtr_.get();
+    MockCounters const* rawCountersPtr = mockCountersPtr_.get();
+    MockETLService const* rawETLServicePtr = mockETLServicePtr_.get();
 
     auto const empty = json::object{};
     auto const ledgerHeader = createLedgerHeader(kLEDGER_HASH, 30, 3);  // 3 seconds old
@@ -389,9 +389,9 @@ TEST_F(RPCServerInfoHandlerTest, AdminSectionPresentWhenAdminFlagIsSet)
 
 TEST_F(RPCServerInfoHandlerTest, BackendCountersPresentWhenRequestWithParam)
 {
-    MockLoadBalancer* rawBalancerPtr = mockLoadBalancerPtr_.get();
-    MockCounters* rawCountersPtr = mockCountersPtr_.get();
-    MockETLService* rawETLServicePtr = mockETLServicePtr_.get();
+    MockLoadBalancer const* rawBalancerPtr = mockLoadBalancerPtr_.get();
+    MockCounters const* rawCountersPtr = mockCountersPtr_.get();
+    MockETLService const* rawETLServicePtr = mockETLServicePtr_.get();
 
     auto const empty = json::object{};
     auto const ledgerHeader = createLedgerHeader(kLEDGER_HASH, 30, 3);  // 3 seconds old
@@ -439,9 +439,9 @@ TEST_F(RPCServerInfoHandlerTest, BackendCountersPresentWhenRequestWithParam)
 
 TEST_F(RPCServerInfoHandlerTest, RippledForwardedValuesPresent)
 {
-    MockLoadBalancer* rawBalancerPtr = mockLoadBalancerPtr_.get();
-    MockCounters* rawCountersPtr = mockCountersPtr_.get();
-    MockETLService* rawETLServicePtr = mockETLServicePtr_.get();
+    MockLoadBalancer const* rawBalancerPtr = mockLoadBalancerPtr_.get();
+    MockCounters const* rawCountersPtr = mockCountersPtr_.get();
+    MockETLService const* rawETLServicePtr = mockETLServicePtr_.get();
 
     auto const empty = json::object{};
     auto const ledgerHeader = createLedgerHeader(kLEDGER_HASH, 30, 3);  // 3 seconds old
@@ -493,9 +493,9 @@ TEST_F(RPCServerInfoHandlerTest, RippledForwardedValuesPresent)
 
 TEST_F(RPCServerInfoHandlerTest, RippledForwardedValuesMissingNoExceptionThrown)
 {
-    MockLoadBalancer* rawBalancerPtr = mockLoadBalancerPtr_.get();
-    MockCounters* rawCountersPtr = mockCountersPtr_.get();
-    MockETLService* rawETLServicePtr = mockETLServicePtr_.get();
+    MockLoadBalancer const* rawBalancerPtr = mockLoadBalancerPtr_.get();
+    MockCounters const* rawCountersPtr = mockCountersPtr_.get();
+    MockETLService const* rawETLServicePtr = mockETLServicePtr_.get();
 
     auto const empty = json::object{};
     auto const ledgerHeader = createLedgerHeader(kLEDGER_HASH, 30, 3);  // 3 seconds old