Fixed missing whitespace and covered more documentation

Author: psg-19

src/api.rs

@@ -11,12 +11,19 @@ pub use bitcoin::{
 
 use serde::Deserialize;
 
+/// It contains the value and scriptpubkey of a
+/// previous transaction output that a transaction
+/// input can reference.
 #[derive(Deserialize, Clone, Debug, PartialEq, Eq)]
 pub struct PrevOut {
     pub value: u64,
     pub scriptpubkey: ScriptBuf,
 }
 
+/// Represents the transaction input containing
+/// a Previous output (or none if coinbase) and includes
+/// the unlocking script, witness data, sequence number,
+/// and a flag indicating if it is a coinbase input.
 #[derive(Deserialize, Clone, Debug, PartialEq, Eq)]
 pub struct Vin {
     pub txid: Txid,
@@ -30,13 +37,15 @@ pub struct Vin {
     pub is_coinbase: bool,
 }
 
+/// Represents the transaction output containing a value and
+/// a scriptpubkey.
 #[derive(Deserialize, Clone, Debug, PartialEq, Eq)]
 pub struct Vout {
     pub value: u64,
     pub scriptpubkey: ScriptBuf,
 }
 
-///Represents Transaction Status.
+/// Represents Transaction Status.
 #[derive(Deserialize, Clone, Debug, PartialEq, Eq)]
 pub struct TxStatus {
     pub confirmed: bool,
@@ -45,6 +54,10 @@ pub struct TxStatus {
     pub block_time: Option<u64>,
 }
 
+/// It holds the block height, a Merkle path of
+/// transaction IDs, and the position of the 
+/// transaction in the tree to verify its inclusion 
+/// in a block.
 #[derive(Deserialize, Clone, Debug, PartialEq, Eq)]
 pub struct MerkleProof {
     pub block_height: u32,
@@ -61,7 +74,7 @@ pub struct OutputStatus {
     pub status: Option<TxStatus>,
 }
 
-///A Struct represents the status of a block in the blockchain.
+/// This Struct represents the status of a block in the blockchain.
 /// `in_best_chain` - a boolean that shows whether the block is part of the main chain.
 /// `height` - Optional field that shows the height of the block if block is in main chain.
 /// `next_best` - Optional field that contains `BlockHash` of the next block that may represent 
@@ -73,7 +86,7 @@ pub struct BlockStatus {
     pub next_best: Option<BlockHash>,
 }
 
-///Structure represents a complete transaction
+/// Structure represents a complete transaction
 #[derive(Deserialize, Clone, Debug, PartialEq, Eq)]
 pub struct Tx {
     pub txid: Txid,
@@ -89,14 +102,14 @@ pub struct Tx {
     pub fee: u64,
 }
 
-///Returns timing information of a Block
+/// Returns timing information of a Block
 /// containg `timestamp` and `height` of block
 #[derive(Deserialize, Clone, Debug, PartialEq, Eq)]
 pub struct BlockTime {
     pub timestamp: u64,
     pub height: u32,
 }
-///Provides a Summary of a  Bitcoin block which includes
+/// Provides a Summary of a  Bitcoin block which includes
 /// `BlockHash`, `BlockTime`, `previousblockhash`, `merkle_root`.
 #[derive(Debug, Clone, Deserialize, PartialEq, Eq)]
 pub struct BlockSummary {
@@ -135,7 +148,7 @@ pub struct AddressTxsSummary {
 }
 
 impl Tx {
-    ///Converts a transaction into a standard `Bitcoin transaction`.
+    /// Converts a transaction into a standard `Bitcoin transaction`.
     pub fn to_tx(&self) -> Transaction {
         Transaction {
             version: transaction::Version::non_standard(self.version),
@@ -166,7 +179,7 @@ impl Tx {
         }
     }
 
-    ///Checks Transaction status, returns a `BlockTime` struct contaning
+    /// Checks Transaction status, returns a `BlockTime` struct contaning
     /// `height` and `timestamp` if transaction has been confirmed or
     /// `None` otherwise.
     pub fn confirmation_time(&self) -> Option<BlockTime> {
@@ -181,7 +194,7 @@ impl Tx {
         }
     }
 
-    ///Takes Transaction as input 
+    /// Takes Transaction as input 
     /// iterates through all the inputs present in the transaction
     /// and checks for prevout field and creates a `TxOut` Struct for each input if it exists
     /// then returns all optional TxOut values  as a vector.
@@ -197,12 +210,12 @@ impl Tx {
             })
             .collect()
     }
-     ///Takes Transaction as input and returns 
+     /// Takes Transaction as input and returns 
      /// the `Weight instance` of the weight present in Transaction.
     pub fn weight(&self) -> Weight {
         Weight::from_wu(self.weight)
     }
-    ///Takes Transaction as input and returns 
+    /// Takes Transaction as input and returns 
     /// the `Amount instance` of the satoshis present in Transaction.
     pub fn fee(&self) -> Amount {
         Amount::from_sat(self.fee)

src/async.rs

@@ -34,6 +34,9 @@ use crate::{
     BASE_BACKOFF_MILLIS, RETRYABLE_ERROR_CODES,
 };
 
+/// Struct that an HTTP client configured with an
+/// Esplora server URL, max retries, and a type
+/// marker for asynchronous sleep behavior.
 #[derive(Debug, Clone)]
 pub struct AsyncClient<S = DefaultSleeper> {
     /// The URL of the Esplora Server.
@@ -81,7 +84,7 @@ impl<S: Sleeper> AsyncClient<S> {
             marker: PhantomData,
         })
     }
-    ///A constructor for creating a new instance of `AsyncClient`.
+    /// A constructor for creating a new instance of `AsyncClient`.
     pub fn from_client(url: String, client: Client) -> Self {
         AsyncClient {
             url,
@@ -482,6 +485,9 @@ fn is_status_retryable(status: reqwest::StatusCode) -> bool {
     RETRYABLE_ERROR_CODES.contains(&status.as_u16())
 }
 
+/// The Sleeper trait defines an asynchronous
+/// sleep mechanism with an associated future
+/// type that completes after a given duration.
 pub trait Sleeper: 'static {
     type Sleep: std::future::Future<Output = ()>;
     fn sleep(dur: std::time::Duration) -> Self::Sleep;