fix(ci): resolve all quick-checks failures

- Remove missing cli crate from workspace members
- Fix TOML array syntax errors in pvf and benchmarking-cli Cargo.toml
- Fix Rust import ordering with cargo fmt
- Fix feature propagation with zepter (try-runtime, runtime-benchmarks, std)

pezbridges/bin/runtime-common/Cargo.toml

@@ -55,16 +55,16 @@ pezpallet-bridge-messages = { features = [
 pezsp-core = { workspace = true, default-features = true }
 
 [features]
-default = ["std"]
+default = [ "std" ]
 std = [
+	"codec/std",
 	"pezbp-header-pez-chain/std",
 	"pezbp-messages/std",
 	"pezbp-pezkuwi-core/std",
 	"pezbp-relayers/std",
+	"pezbp-runtime/std",
 	"pezbp-test-utils/std",
 	"pezbp-teyrchains/std",
-	"codec/std",
-	"pezbp-runtime/std",
 	"pezframe-support/std",
 	"pezframe-system/std",
 	"pezpallet-bridge-grandpa/std",
@@ -88,10 +88,10 @@ runtime-benchmarks = [
 	"pezbp-messages/runtime-benchmarks",
 	"pezbp-pezkuwi-core/runtime-benchmarks",
 	"pezbp-relayers/runtime-benchmarks",
-	"pezbp-test-utils/runtime-benchmarks",
-	"pezbp-teyrchains/runtime-benchmarks",
 	"pezbp-runtime/runtime-benchmarks",
 	"pezbp-runtime/test-helpers",
+	"pezbp-test-utils/runtime-benchmarks",
+	"pezbp-teyrchains/runtime-benchmarks",
 	"pezframe-support/runtime-benchmarks",
 	"pezframe-system/runtime-benchmarks",
 	"pezpallet-balances/runtime-benchmarks",
@@ -108,5 +108,5 @@ runtime-benchmarks = [
 	"pezsp-trie?/runtime-benchmarks",
 	"xcm/runtime-benchmarks",
 ]
-integrity-test = ["static_assertions"]
-test-helpers = ["pezbp-runtime/test-helpers", "pezsp-trie"]
+integrity-test = [ "static_assertions" ]
+test-helpers = [ "pezbp-runtime/test-helpers", "pezsp-trie" ]

pezbridges/bin/runtime-common/src/extensions.rs

@@ -19,8 +19,8 @@
 //! checks.
 
 use pezbp_relayers::ExplicitOrAccountParams;
-use pezbp_teyrchains::SubmitTeyrchainHeadsInfo;
 use pezbp_runtime::Teyrchain;
+use pezbp_teyrchains::SubmitTeyrchainHeadsInfo;
 use pezpallet_bridge_grandpa::{
 	BridgedBlockNumber, CallSubType as GrandpaCallSubType, SubmitFinalityProofHelper,
 };
@@ -377,14 +377,14 @@ macro_rules! generate_bridge_reject_obsolete_headers_and_messages {
 mod tests {
 	use super::*;
 	use crate::mock::*;
+	use codec::{Decode, Encode, MaxEncodedLen};
 	use pezbp_header_pez_chain::StoredHeaderDataBuilder;
 	use pezbp_messages::{InboundLaneData, MessageNonce, OutboundLaneData};
 	use pezbp_pezkuwi_core::teyrchains::{ParaHeadsProof, ParaId};
 	use pezbp_relayers::{RewardsAccountOwner, RewardsAccountParams};
+	use pezbp_runtime::HeaderId;
 	use pezbp_test_utils::{make_default_justification, test_keyring, TEST_GRANDPA_SET_ID};
 	use pezbp_teyrchains::{BestParaHeadHash, ParaInfo};
-	use codec::{Decode, Encode, MaxEncodedLen};
-	use pezbp_runtime::HeaderId;
 	use pezframe_support::{assert_err, assert_ok, traits::fungible::Mutate};
 	use pezpallet_bridge_grandpa::{Call as GrandpaCall, StoredAuthoritySet};
 	use pezpallet_bridge_teyrchains::Call as TeyrchainsCall;

pezbridges/bin/runtime-common/src/integrity.rs

@@ -19,9 +19,9 @@
 //! Most of the tests in this module assume that the bridge is using standard (see `crate::messages`
 //! module for details) configuration.
 
+use codec::Encode;
 use pezbp_header_pez_chain::ChainWithGrandpa;
 use pezbp_messages::{ChainWithMessages, InboundLaneData, MessageNonce};
-use codec::Encode;
 use pezbp_runtime::{AccountIdOf, Chain};
 use pezframe_support::{storage::generator::StorageValue, traits::Get, weights::Weight};
 use pezframe_system::limits;

pezbridges/bin/runtime-common/src/messages_benchmarking.rs

@@ -19,12 +19,12 @@
 
 #![cfg(feature = "runtime-benchmarks")]
 
+use codec::Encode;
 use pezbp_messages::{
 	source_chain::FromBridgedChainMessagesDeliveryProof,
 	target_chain::FromBridgedChainMessagesProof, MessagePayload,
 };
 use pezbp_pezkuwi_core::teyrchains::ParaHash;
-use codec::Encode;
 use pezbp_runtime::{AccountIdOf, Chain, HashOf, Teyrchain};
 use pezframe_support::weights::Weight;
 use pezpallet_bridge_messages::{

pezbridges/bin/runtime-common/src/mock.rs

@@ -20,15 +20,15 @@
 #![cfg(test)]
 #![allow(dead_code)]
 
+use codec::Encode;
 use pezbp_header_pez_chain::ChainWithGrandpa;
 use pezbp_messages::{
 	target_chain::{DispatchMessage, MessageDispatch},
 	ChainWithMessages, HashedLaneId, LaneIdType, MessageNonce,
 };
 use pezbp_relayers::{PayRewardFromAccount, RewardsAccountParams};
-use pezbp_teyrchains::SingleParaStoredHeaderDataBuilder;
-use codec::Encode;
 use pezbp_runtime::{messages::MessageDispatchResult, Chain, ChainId, Teyrchain};
+use pezbp_teyrchains::SingleParaStoredHeaderDataBuilder;
 use pezframe_support::{
 	derive_impl, parameter_types,
 	weights::{ConstantMultiplier, IdentityFee, RuntimeDbWeight, Weight},

pezbridges/bin/runtime-common/src/teyrchains_benchmarking.rs

@@ -20,14 +20,14 @@
 
 use crate::messages_benchmarking::insert_header_to_grandpa_pallet;
 
-use pezbp_pezkuwi_core::teyrchains::{ParaHash, ParaHead, ParaHeadsProof, ParaId};
-use pezbp_teyrchains::{
-	teyrchain_head_storage_key_at_source, RelayBlockHash, RelayBlockHasher, RelayBlockNumber,
-};
 use codec::Encode;
+use pezbp_pezkuwi_core::teyrchains::{ParaHash, ParaHead, ParaHeadsProof, ParaId};
 use pezbp_runtime::{
 	grow_storage_value, record_all_trie_keys, Chain, UnverifiedStorageProofParams,
 };
+use pezbp_teyrchains::{
+	teyrchain_head_storage_key_at_source, RelayBlockHash, RelayBlockHasher, RelayBlockNumber,
+};
 use pezframe_support::traits::Get;
 use pezsp_std::prelude::*;
 use pezsp_trie::{trie_types::TrieDBMutBuilderV1, LayoutV1, MemoryDB, TrieMut};