refactor

src/account.rs

@@ -44,12 +44,12 @@ impl KeyStoreBackend {
             Self::Os(key) => {
                 let priv_key = get_priv_key_from_os(key)?;
                 let signature: Signature = priv_key.try_sign(data).unwrap();
-                Ok(signature.to_bytes().try_into()?)
+                Ok(signature.to_bytes().into())
             }
             Self::Memory(key) => {
                 let priv_key = get_priv_key_from_memory(key)?;
                 let signature: Signature = priv_key.try_sign(data).unwrap();
-                Ok(signature.to_bytes().try_into()?)
+                Ok(signature.to_bytes().into())
             }
         }
     }

src/cli/tx.rs

@@ -172,7 +172,7 @@ impl Transaction {
                             let rewards = crate::query::get_rewards(&account, rpc_endpoint).await?;
 
                             let validator = delegations
-                                .get(0)
+                                .first()
                                 .context("delegator should have atleast one validator")?
                                 .0
                                 .clone();
@@ -229,7 +229,7 @@ impl Transaction {
                                         crate::query::get_delegated(&account, rpc_endpoint).await?;
 
                                     delegations
-                                        .get(0)
+                                        .first()
                                         .context("delegator should have atleast one validator")?
                                         .0
                                         .clone()
@@ -409,7 +409,8 @@ impl Transaction {
                                 timeout_height: None,
                                 timeout_timestamp: (chrono::Utc::now()
                                     + chrono::Duration::minutes(10))
-                                .timestamp_nanos()
+                                .timestamp_nanos_opt()
+                                .context("error while converting time")?
                                     as u64,
                             };
                             (account_acc, vec![Any::try_pack(ibc_transfer)?])

src/cli/utils.rs

@@ -55,7 +55,11 @@ pub fn custom_coin(coin_str: &str) -> Result<Coin> {
 
 pub fn custom_io_string(json_str: &str) -> Result<String> {
     Ok(match json_str {
-        "-" => std::io::stdin().lock().lines().flatten().collect(),
+        "-" => std::io::stdin()
+            .lock()
+            .lines()
+            .map_while(std::io::Result::ok)
+            .collect(),
         _ if json_str.starts_with('@') => {
             std::fs::read_to_string(json_str.strip_prefix('@').context("should never arise")?)?
         }

src/endpoint.rs

@@ -86,15 +86,14 @@ pub async fn get_cosmos_directory_name(chain_id: &str) -> Result<String> {
         .as_array()
         .context("no vec value")?
         .iter()
-        .filter_map(|v| {
-            (v.pointer("/chain_id")?.as_str()? == chain_id).then(|| {
-                v.pointer("/name")
-                    .and_then(|k| k.as_str())
-                    .expect("name in chains data")
-            })
+        .filter_map(|v| (v.pointer("/chain_id")?.as_str()? == chain_id).then_some(v))
+        .map(|v| {
+            v.pointer("/name")
+                .and_then(|k| k.as_str())
+                .context("name in chains data")
         })
         .next()
-        .context("at least one chain")?;
+        .context("at least one chain")??;
 
     Ok(format!("https://rpc.cosmos.directory/{chain_name}"))
 }