Ban wildcard imports via clippy::wildcard_imports

Cargo.toml

@@ -314,6 +314,7 @@ unexpected_cfgs = { level = "allow", check-cfg = [
 assertions_on_result_states = "warn"
 allow_attributes = "warn"
 mod_module_files = "deny"
+wildcard_imports = "deny"
 
 [workspace.lints.rustdoc]
 broken_intra_doc_links = "deny"

crates/contract/src/crypto_shared/types.rs

@@ -159,7 +159,10 @@ impl TryFrom<dtos::PublicKey> for PublicKeyExtended {
 }
 
 pub mod k256_types {
-    use super::*;
+    use super::{
+        AffinePoint, BorshDeserialize, BorshSerialize, CurveArithmetic, Deserialize, PrimeField,
+        Secp256k1, Serialize,
+    };
     use k256::Scalar;
 
     pub type PublicKey = <Secp256k1 as CurveArithmetic>::AffinePoint;
@@ -249,7 +252,7 @@ pub mod k256_types {
 }
 
 pub mod ed25519_types {
-    use super::*;
+    use super::{BorshDeserialize, BorshSerialize, Deserialize, PrimeField, Serialize, serde_as};
     use curve25519_dalek::Scalar;
 
     // Is there a better way to force a borsh serialization?

crates/contract/src/dto_mapping.rs

@@ -606,7 +606,10 @@ impl TryFrom<near_mpc_contract_interface::types::Config> for Config {
 // Gated behind `test-utils` so they stay out of the production/WASM surface.
 #[cfg(feature = "test-utils")]
 mod test_conversions {
-    use super::*;
+    use super::{
+        GovernanceThresholdParameters, IntoContractType, IntoInterfaceType, ParticipantInfo,
+        ProposedGovernanceThresholdParameters, ProtocolContractState, dtos,
+    };
 
     impl From<ProtocolContractState> for dtos::ProtocolContractState {
         fn from(state: ProtocolContractState) -> Self {

crates/devnet/src/types.rs

@@ -219,7 +219,9 @@ pub mod near_crypto_compatible_serialization {
     const ED25519_PREFIX: &str = "ed25519";
 
     pub mod signing_keys {
-        use super::*;
+        use super::{
+            Deserialize, Deserializer, ED25519_PREFIX, Serialize, Serializer, SigningKey, de,
+        };
 
         pub fn serialize<S>(keys: &[SigningKey], serializer: S) -> Result<S::Ok, S::Error>
         where
@@ -264,7 +266,9 @@ pub mod near_crypto_compatible_serialization {
     pub mod verifying_key {
         use anyhow::Context;
 
-        use super::*;
+        use super::{
+            Deserialize, Deserializer, ED25519_PREFIX, Serialize, Serializer, VerifyingKey, de,
+        };
 
         pub fn serialize<S>(key: &VerifyingKey, serializer: S) -> Result<S::Ok, S::Error>
         where

crates/near-mpc-bounded-collections/src/bounded_vec.rs

@@ -599,7 +599,7 @@ impl<T, const L: usize, const U: usize> OptBoundedVecToVec<T>
 }
 
 mod borsh_impl {
-    use super::*;
+    use super::{BoundedVec, witnesses};
     use borsh::{BorshDeserialize, BorshSerialize};
 
     impl<T: BorshSerialize, const L: usize, const U: usize, W> BorshSerialize
@@ -632,7 +632,7 @@ mod borsh_impl {
 
     #[cfg(feature = "abi")]
     mod schema {
-        use super::*;
+        use super::BoundedVec;
         use borsh::BorshSchema;
         use borsh::schema::{Declaration, Definition, add_definition};
         use std::collections::BTreeMap;
@@ -656,7 +656,7 @@ mod borsh_impl {
 }
 
 mod serde_impl {
-    use super::*;
+    use super::{BoundedVec, TryFrom, UpperBoundedVec};
     use serde::{Deserialize, Serialize};
 
     // direct impl to unify serde in one place instead of doing attribute on declaration and deserialize here
@@ -693,7 +693,7 @@ mod serde_impl {
 
     #[cfg(all(feature = "abi", not(target_arch = "wasm32")))]
     mod schema {
-        use super::*;
+        use super::{BoundedVec, TryFrom};
         use schemars::JsonSchema;
 
         impl<T: JsonSchema, const L: usize, const U: usize, W> JsonSchema for BoundedVec<T, L, U, W> {
@@ -739,7 +739,7 @@ mod serde_impl {
 /// }
 /// ```
 pub mod hex_serde {
-    use super::*;
+    use super::{BoundedVec, BoundedVecOutOfBounds, TryFrom, TryInto};
     use serde::Deserialize;
 
     #[cfg(all(feature = "abi", not(target_arch = "wasm32")))]