Feature: systematic chunks recovery

.clang-format

@@ -22,7 +22,7 @@ QualifierAlignment: Custom
 QualifierOrder: ['inline', 'static', 'constexpr', 'const', 'volatile', 'type', 'restrict']
 ReferenceAlignment: Right
 ReflowComments: true
-RemoveSemicolon: true
+#RemoveSemicolon: true
 InsertNewlineAtEOF: true
 
 

.clang-tidy

@@ -8,7 +8,6 @@
 Checks:          '-*,clang-analyzer-*,clang-diagnostic-*,readability-*,modernize-*,boost-*,bugprone-*,cppcoreguidelines-*,google-*,hicpp-*,performance-*,readability-*,-google-readability-namespace-comments,-readability-inconsistent-declaration-parameter-name,-readability-braces-around-statements,-hicpp-signed-bitwise,-google-runtime-references,-cppcoreguidelines-avoid-magic-numbers,-cppcoreguidelines-pro-bounds-array-to-pointer-decay,-readability-magic-numbers,-hicpp-explicit-conversions,-hicpp-uppercase-literal-suffix,-readability-uppercase-literal-suffix,-hicpp-no-array-decay,-hicpp-special-member-functions,-bugprone-narrowing-conversions,-modernize-use-nodiscard,-google-readability-braces-around-statements,-hicpp-braces-around-statements,-bugprone-suspicious-semicolon,-readability-named-parameter,-hicpp-named-parameter,-readability-identifier-naming,-modernize-use-trailing-return-type'
 WarningsAsErrors: 'modernize-*,cppcoreguidelines-*,boost-*,performance-*,google-build-using-namespace,readability-else-after-return,google-readability-todo'
 HeaderFilterRegex: '\.(hpp|h)'
-AnalyzeTemporaryDtors: false
 FormatStyle:     .clang-format
 User:            bogdan
 CheckOptions:

.github/workflows/test.yml

@@ -218,6 +218,63 @@ jobs:
           BUILD_TYPE: "${{ matrix.options.build-type }}"
         run: ./housekeeping/docker/kagome-dev/make.sh
 
+  kagome_dev_docker_build_tidy:
+    runs-on: ubuntu-latest
+    timeout-minutes: 180
+    name: "Linux: clang-tidy GCE"
+
+    steps:
+      - name: "Checkout repository"
+        uses: actions/checkout@v4
+
+      - name: "Authenticate with Google Cloud"
+        uses: 'google-github-actions/auth@v2'
+        with:
+          credentials_json: ${{ secrets.GCP_SERVICE_ACCOUNT_KEY }}
+
+      - name: "Set up Cloud SDK"
+        uses: 'google-github-actions/setup-gcloud@v2'
+
+      - name: "Configure Docker for GCR"
+        run: |
+          gcloud auth configure-docker --quiet
+          gcloud auth configure-docker ${{ secrets.GCP_REGISTRY }} --quiet
+
+      - name: "Check version"
+        working-directory: ./housekeeping/docker/kagome-dev
+        run: |
+          SHORT_COMMIT_HASH=$(grep 'short_commit_hash:' commit_hash.txt | cut -d ' ' -f 2)
+          echo "short_commit_hash=${SHORT_COMMIT_HASH}" >> $GITHUB_ENV
+
+      - name: "Cache dependencies"
+        id: cache-restore
+        uses: actions/cache/restore@v4
+        with:
+          path: ${{ env.CACHE_PATH }}
+          key: ${{ github.job }}-${{ env.CACHE_VERSION }}-tidy-${{ env.short_commit_hash }}
+          restore-keys: |
+            ${{ github.job }}-${{ env.CACHE_VERSION }}-tidy-
+
+      - name: "Build target"
+        working-directory: ./housekeeping/docker/kagome-dev
+        run:
+          make kagome_dev_docker_build_tidy \
+          DOCKER_REGISTRY_PATH=${DOCKER_REGISTRY_PATH} \
+          GITHUB_HUNTER_USERNAME=${{ secrets.HUNTER_USERNAME }} \
+          GITHUB_HUNTER_TOKEN=${{ secrets.HUNTER_TOKEN }}
+
+      - name: "Cleaning cache"
+        run: |
+          find ${{ env.CACHE_PATH }} -name '*.pdf' -exec rm {} \;
+
+      - name: "Always Save Cache"
+        id: cache-save
+        if: always() && (steps.cache-restore.outputs.cache-hit != 'true' || env.package_exist != 'True')
+        uses: actions/cache/save@v4
+        with:
+          path: ${{ env.CACHE_PATH }}
+          key: ${{ steps.cache-restore.outputs.cache-primary-key }}
+
   kagome_dev_docker_build:
     runs-on: [ actions-runner-controller ]
     timeout-minutes: 180

cmake/Hunter/config.cmake

@@ -55,8 +55,8 @@ if ("${WASM_COMPILER}" STREQUAL "WAVM")
       LLVM
       VERSION 12.0.1-p4
       CMAKE_ARGS
-      LLVM_ENABLE_PROJECTS=ir
-      KEEP_PACKAGE_SOURCES
+        LLVM_ENABLE_PROJECTS=ir
+        KEEP_PACKAGE_SOURCES
   )
 
   if (CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
@@ -69,7 +69,7 @@ if ("${WASM_COMPILER}" STREQUAL "WAVM")
       wavm
       VERSION 1.0.14
       CMAKE_ARGS
-      WAVM_CXX_FLAGS=${WAVM_CXX_FLAGS}
+        WAVM_CXX_FLAGS=${WAVM_CXX_FLAGS}
       KEEP_PACKAGE_SOURCES
   )
 endif ()
@@ -84,5 +84,14 @@ hunter_config(
     libsecp256k1
     VERSION 0.5.1
     CMAKE_ARGS
-        SECP256K1_ENABLE_MODULE_RECOVERY=ON
+      SECP256K1_ENABLE_MODULE_RECOVERY=ON
+)
+
+hunter_config(
+    erasure_coding_crust
+#    VERSION 0.0.8
+    URL  https://github.com/qdrvm/erasure-coding-crust/archive/refs/tags/v0.0.8.tar.gz
+    SHA1 6bcdb6327f5da2dcec5c70f2fa63b95a44925af0
+    KEEP_PACKAGE_SOURCES
 )
+

cmake/clang-tidy.cmake

@@ -1,6 +1,6 @@
 if(NOT CLANG_TIDY_BIN)
   find_program(CLANG_TIDY_BIN
-    NAMES clang-tidy clang-tidy-10 clang-tidy-9 clang-tidy-8 clang-tidy-7
+    NAMES clang-tidy clang-tidy-20 clang-tidy-19 clang-tidy-18 clang-tidy-17 clang-tidy-16
     DOC "Path to clang-tidy executable"
   )
 endif()

core/api/service/author/requests/has_key.hpp

@@ -20,7 +20,7 @@ namespace kagome::api::author::request {
    public:
     explicit HasKey(std::shared_ptr<AuthorApi> api) : api_(std::move(api)) {
       BOOST_ASSERT(api_);
-    };
+    }
 
     outcome::result<Return> execute() override {
       OUTCOME_TRY(public_key, common::unhexWith0x(getParam<0>()));

core/api/service/author/requests/has_session_keys.hpp

@@ -19,7 +19,7 @@ namespace kagome::api::author::request {
     explicit HasSessionKeys(std::shared_ptr<AuthorApi> api)
         : api_(std::move(api)) {
       BOOST_ASSERT(api_);
-    };
+    }
 
     outcome::result<Return> execute() override {
       OUTCOME_TRY(keys, common::unhexWith0x(getParam<0>()));

core/api/service/author/requests/insert_key.hpp

@@ -25,7 +25,7 @@ namespace kagome::api::author::request {
    public:
     explicit InsertKey(std::shared_ptr<AuthorApi> api) : api_(std::move(api)) {
       BOOST_ASSERT(api_);
-    };
+    }
 
     outcome::result<Return> execute() override {
       auto &seed_hex = getParam<1>();

core/api/service/author/requests/rotate_keys.hpp

@@ -18,7 +18,7 @@ namespace kagome::api::author::request {
    public:
     explicit RotateKeys(std::shared_ptr<AuthorApi> api) : api_(std::move(api)) {
       BOOST_ASSERT(api_);
-    };
+    }
 
     outcome::result<Return> execute() override {
       return api_->rotateKeys();

core/api/service/author/requests/submit_and_watch_extrinsic.hpp

@@ -9,6 +9,7 @@
 #include <jsonrpc-lean/request.h>
 
 #include "api/service/author/author_api.hpp"
+#include "api/service/base_request.hpp"
 #include "primitives/extrinsic.hpp"
 
 namespace kagome::api::author::request {
@@ -19,7 +20,7 @@ namespace kagome::api::author::request {
     explicit SubmitAndWatchExtrinsic(std::shared_ptr<AuthorApi> api)
         : api_(std::move(api)) {
       BOOST_ASSERT(api_);
-    };
+    }
 
     outcome::result<primitives::SubscriptionId> execute() override {
       auto ext_hex = getParam<0>();

core/api/service/author/requests/submit_extrinsic.hpp

@@ -9,6 +9,7 @@
 #include <jsonrpc-lean/request.h>
 
 #include "api/service/author/author_api.hpp"
+#include "api/service/base_request.hpp"
 #include "primitives/extrinsic.hpp"
 
 namespace kagome::api::author::request {
@@ -19,7 +20,7 @@ namespace kagome::api::author::request {
     explicit SubmitExtrinsic(std::shared_ptr<AuthorApi> api)
         : api_(std::move(api)) {
       BOOST_ASSERT(api_);
-    };
+    }
 
     outcome::result<common::Hash256> execute() override {
       auto ext_hex = getParam<0>();

core/api/service/chain/requests/get_block_hash.cpp

@@ -89,6 +89,6 @@ namespace kagome::api::chain::request {
 
           return results;
         });
-  };
+  }
 
 }  // namespace kagome::api::chain::request

core/api/service/child_state/requests/get_keys.hpp

@@ -30,7 +30,7 @@ namespace kagome::api::child_state::request {
     explicit GetKeys(std::shared_ptr<ChildStateApi> api)
         : api_(std::move(api)) {
       BOOST_ASSERT(api_);
-    };
+    }
     ~GetKeys() = default;
 
     outcome::result<void> init(const jsonrpc::Request::Parameters &params);

core/api/service/child_state/requests/get_keys_paged.hpp

@@ -30,7 +30,7 @@ namespace kagome::api::child_state::request {
     explicit GetKeysPaged(std::shared_ptr<ChildStateApi> api)
         : api_(std::move(api)) {
       BOOST_ASSERT(api_);
-    };
+    }
     ~GetKeysPaged() = default;
 
     outcome::result<void> init(const jsonrpc::Request::Parameters &params);

core/api/service/rpc/requests/methods.hpp

@@ -32,7 +32,7 @@ namespace kagome::api::rpc::request {
 
     explicit Methods(std::shared_ptr<RpcApi> api) : api_(std::move(api)) {
       BOOST_ASSERT(api_);
-    };
+    }
     ~Methods() = default;
 
     outcome::result<void> init(const jsonrpc::Request::Parameters &params);

core/api/service/state/requests/call.cpp

@@ -12,7 +12,7 @@ namespace kagome::api::state::request {
 
   Call::Call(std::shared_ptr<StateApi> api) : api_(std::move(api)) {
     BOOST_ASSERT(api_);
-  };
+  }
 
   outcome::result<void> Call::init(const jsonrpc::Request::Parameters &params) {
     if (params.size() < 2 or params.size() > 3) {

core/api/service/state/requests/get_keys_paged.hpp

@@ -30,7 +30,7 @@ namespace kagome::api::state::request {
     explicit GetKeysPaged(std::shared_ptr<StateApi> api)
         : api_(std::move(api)) {
       BOOST_ASSERT(api_);
-    };
+    }
     ~GetKeysPaged() = default;
 
     outcome::result<void> init(const jsonrpc::Request::Parameters &params);

core/api/service/state/requests/query_storage.hpp

@@ -44,7 +44,7 @@ namespace kagome::api::state::request {
     explicit QueryStorage(std::shared_ptr<StateApi> api)
         : api_(std::move(api)) {
       BOOST_ASSERT(api_);
-    };
+    }
 
     outcome::result<std::vector<StateApi::StorageChangeSet>> execute()
         override {

core/api/service/system/requests/account_next_index.hpp

@@ -17,7 +17,7 @@ namespace kagome::api::system::request {
     explicit AccountNextIndex(std::shared_ptr<SystemApi> api)
         : api_(std::move(api)) {
       BOOST_ASSERT(api_);
-    };
+    }
 
     outcome::result<primitives::AccountNonce> execute() override {
       return api_->getNonceFor(getParam<0>());

core/application/app_configuration.hpp

@@ -211,8 +211,8 @@ namespace kagome::application {
      * List of telemetry endpoints specified via CLI argument or config file
      * @return a vector of parsed telemetry endpoints
      */
-    virtual const std::vector<telemetry::TelemetryEndpoint>
-        &telemetryEndpoints() const = 0;
+    virtual const std::vector<telemetry::TelemetryEndpoint> &
+    telemetryEndpoints() const = 0;
 
     /**
      * @return enum constant of the chosen sync method

core/application/chain_spec.hpp

@@ -38,8 +38,8 @@ namespace kagome::application {
     virtual const std::vector<libp2p::multi::Multiaddress> &bootNodes()
         const = 0;
 
-    virtual const std::vector<std::pair<std::string, size_t>>
-        &telemetryEndpoints() const = 0;
+    virtual const std::vector<std::pair<std::string, size_t>> &
+    telemetryEndpoints() const = 0;
 
     virtual const std::string &protocolId() const = 0;
 

core/authority_discovery/query/query_impl.cpp

@@ -283,7 +283,7 @@ namespace kagome::authority_discovery {
     }
 
     std::ignore = host_.getPeerRepository().getAddressRepository().addAddresses(
-        peer.id, peer.addresses, libp2p::peer::ttl::kRecentlyConnected);
+        peer.id, peer.addresses, libp2p::peer::ttl::kDay);
 
     peer_to_auth_cache_.insert_or_assign(peer.id, authority);
     auth_to_peer_cache_.insert_or_assign(

core/blockchain/impl/block_tree_impl.cpp

@@ -425,18 +425,18 @@ namespace kagome::blockchain {
       std::shared_ptr<storage::trie_pruner::TriePruner> state_pruner,
       common::MainThreadPool &main_thread_pool)
       : block_tree_data_{BlockTreeData{
-            .header_repo_ = std::move(header_repo),
-            .storage_ = std::move(storage),
-            .state_pruner_ = std::move(state_pruner),
-            .tree_ = std::make_unique<CachedTree>(finalized),
-            .extrinsic_observer_ = std::move(extrinsic_observer),
-            .hasher_ = std::move(hasher),
-            .extrinsic_event_key_repo_ = std::move(extrinsic_event_key_repo),
-            .justification_storage_policy_ =
-                std::move(justification_storage_policy),
-            .genesis_block_hash_ = {},
-            .blocks_pruning_ = {app_config.blocksPruning(), finalized.number},
-        }},
+          .header_repo_ = std::move(header_repo),
+          .storage_ = std::move(storage),
+          .state_pruner_ = std::move(state_pruner),
+          .tree_ = std::make_unique<CachedTree>(finalized),
+          .extrinsic_observer_ = std::move(extrinsic_observer),
+          .hasher_ = std::move(hasher),
+          .extrinsic_event_key_repo_ = std::move(extrinsic_event_key_repo),
+          .justification_storage_policy_ =
+              std::move(justification_storage_policy),
+          .genesis_block_hash_ = {},
+          .blocks_pruning_ = {app_config.blocksPruning(), finalized.number},
+      }},
         main_pool_handler_{main_thread_pool.handlerStarted()} {
     block_tree_data_.sharedAccess([&](const BlockTreeData &p) {
       BOOST_ASSERT(p.header_repo_ != nullptr);

core/common/variant_builder.hpp

@@ -82,7 +82,7 @@ namespace kagome::common {
       static_assert(dynamic_variant::is_boost_variant<Variant>::value);
       dynamic_variant::functors_vector_builder<Variant> builder = {&funcs_};
       boost::mpl::for_each<Types>(builder);
-    };
+    }
 
     /**
      * Initializes the referenced variant with default constructed instance of

core/consensus/grandpa/impl/voting_round_impl.hpp

@@ -167,7 +167,7 @@ namespace kagome::consensus::grandpa {
      */
     std::shared_ptr<VotingRound> getPreviousRound() const override {
       return previous_round_;
-    };
+    }
 
     /**
      * Removes previous round to limit chain of rounds
@@ -226,7 +226,7 @@ namespace kagome::consensus::grandpa {
      */
     const std::optional<BlockInfo> &finalizedBlock() const override {
       return finalized_;
-    };
+    }
 
     /**
      * @return state containing round number, last finalized block, votes, and

core/consensus/grandpa/types/equivocation_proof.hpp

@@ -48,7 +48,7 @@ namespace kagome::consensus::grandpa {
       BOOST_ASSERT((first.is<Prevote>() and second.is<Prevote>())
                    or (first.is<Precommit>() and second.is<Precommit>()));
       BOOST_ASSERT(first.id == second.id);
-    };
+    }
 
     AuthorityId offender() const {
       return first.id;

core/consensus/timeline/impl/consensus_selector_impl.cpp

@@ -120,7 +120,7 @@ namespace kagome::consensus {
       // TODO: Code for trying to select another consensus
     }
     BOOST_UNREACHABLE_RETURN({});
-  };
+  }
 
   std::shared_ptr<FinalityConsensus>
   ConsensusSelectorImpl::getFinalityConsensus(
@@ -139,6 +139,6 @@ namespace kagome::consensus {
       // TODO: Code for trying to select another consensus
     }
     BOOST_UNREACHABLE_RETURN({});
-  };
+  }
 
 }  // namespace kagome::consensus

core/consensus/timeline/impl/timeline_impl.cpp

@@ -421,7 +421,7 @@ namespace kagome::consensus {
             }
           }
         });
-  };
+  }
 
   bool TimelineImpl::updateSlot(TimePoint now) {
     best_block_ = block_tree_->bestBlock();