Fix documentation typos, grammar, and correctness issues

- blockifier_reexecution/README.md: Fix "blockier" -> "blockifier",
  "permisionless" -> "permissionless", "dowload-files" -> "download-files",
  "neccessary" -> "necessary", "from the GC Bucket" -> "to the GC Bucket"
- apollo_config/src/dumping.rs: Fix missing "be" in "will not included"
  and fix "a None sub configs, its" -> "None sub configs, their"
- apollo_storage/src/lib.rs: Fix "actually" -> "actual", "non clonable" ->
  "non-clonable"
- apollo_infra_utils/src/tracing.rs: Remove extra backtick in doc comment
- mempool.md: Fix "Transaction prior" -> "Transactions prior", remove
  double space
- swarm_trait.rs, worker_pool.rs: Fix "occured" -> "occurred"
- peer_manager/mod.rs, p2p_sync server/test.rs: Fix "reciever" -> "receiver"
- db_test.rs: Fix "charachter" -> "character"
- blockifier_versioned_constants.rs: Fix "seperate" -> "separate"

https://claude.ai/code/session_01H7Rdt1uDEga9dFsKSzPhBe

Author: claude

crates/apollo_config/src/dumping.rs

@@ -157,7 +157,7 @@ fn chain_param_paths(param_paths: &[&str]) -> ParamPath {
 pub trait SerializeConfig {
     /// Conversion of a configuration to a mapping of flattened parameters to their descriptions and
     /// values.
-    /// Note, in the case of a None sub configs, its elements will not included in the flatten map.
+    /// Note, in the case of a None sub configs, its elements will not be included in the flatten map.
     fn dump(&self) -> BTreeMap<ParamPath, SerializedParam>;
 
     /// Serialization of a configuration into a JSON file.
@@ -196,7 +196,7 @@ pub trait SerializeConfig {
     /// let file_path = dir.path().join("config.json");
     /// ConfigExample { key: 42 }.dump_to_file(&vec![], &HashSet::new(), file_path.to_str().unwrap());
     /// ```
-    /// Note, in the case of a None sub configs, its elements will not be included in the file.
+    /// Note, in the case of None sub configs, their elements will not be included in the file.
     fn dump_to_file(
         &self,
         config_pointers: &ConfigPointers,

crates/apollo_infra_utils/src/tracing.rs

@@ -137,7 +137,7 @@ macro_rules! trace_every_n {
 ///
 /// # Arguments
 ///
-/// * `$n`: Number of milliseconds (e.g., `2` for not logging a message again for `2`` milliseconds
+/// * `$n`: Number of milliseconds (e.g., `2` for not logging a message again for `2` milliseconds
 ///   from when it last logged).
 /// * `$($arg:tt)*`: The arguments to pass to `tracing::info!`, e.g., a format string and its
 ///   corresponding values.

crates/apollo_network/src/network_manager/swarm_trait.rs

@@ -111,7 +111,7 @@ impl SwarmTrait for Swarm<mixed_behaviour::MixedBehaviour> {
             // TODO(shahak): Consider reporting to the subscriber broadcast failures or retrying
             // upon failure.
             warn!(
-                "Error occured while broadcasting a message to the topic with hash \
+                "Error occurred while broadcasting a message to the topic with hash \
                  {topic_hash:?}: {err:?}"
             );
         }

crates/apollo_network/src/peer_manager/mod.rs

@@ -183,7 +183,7 @@ impl PeerManager {
             // TODO(shahak): remove this code block, since we check that peer has connection id
             } else {
                 // In case we have a race condition where the connection is closed after we added to
-                // the pending list, the reciever will get an error and will need to ask for
+                // the pending list, the receiver will get an error and will need to ask for
                 // re-assignment
                 if let Some(sessions) = self.peers_pending_dial_with_sessions.get_mut(peer_id) {
                     sessions.push(outbound_session_id);

crates/apollo_p2p_sync/src/server/test.rs

@@ -359,7 +359,7 @@ async fn run_test<T, F, TQuery>(
         step: 1,
     };
     let query = TQuery::from(query);
-    let (server_query_manager, _report_sender, response_reciever) =
+    let (server_query_manager, _report_sender, response_receiver) =
         create_test_server_query_manager(query);
 
     register_query::<T, TQuery>(
@@ -374,7 +374,7 @@ async fn run_test<T, F, TQuery>(
         _never = p2p_sync_server.run() => {
             unreachable!("Return type Never should never be constructed");
         },
-        mut res = response_reciever.collect::<Vec<_>>() => {
+        mut res = response_receiver.collect::<Vec<_>>() => {
             assert_eq!(DataOrFin(None), res.pop().unwrap());
             let filtered_res: Vec<T> = res.into_iter()
                     .map(|data| data.0.expect("P2pSyncServer returned Fin and then returned another response"))

crates/apollo_protobuf/src/proto/p2p/proto/mempool/mempool.md

@@ -19,8 +19,8 @@ Our choice of using Gossipsub might change in the future if we encounter perform
 
 ## Transaction
 The object that we're passing is a [MempoolTransaction](./transaction.proto). A few notes about it:
-* It contains support only for V3 transactions. Transaction prior to V3 can't be a part of a mempool
-because they don't have the required fields to  enter the fee market
+* It contains support only for V3 transactions. Transactions prior to V3 can't be a part of a mempool
+because they don't have the required fields to enter the fee market
 * It doesn't contain a variant for L1Handler transactions. The reason is that mempools can learn
 about existing L1Handler transactions by simply looking at L1
 * In Declare, it contains the SIERRA of the class being declared. The reason it doesn't contain the

crates/apollo_storage/src/db/db_test.rs

@@ -54,7 +54,7 @@ fn open_env_with_enforce_file_exists() {
     let result: DbResult<(DbReader, DbWriter)> = open_env(&db_config);
     assert_matches!(result, Ok(_));
 
-    // Add some charachter to the path to make it invalid.
+    // Add some character to the path to make it invalid.
     // Fourth and final call to `open_env` with path enforcement should fail because the path is
     // invalid.
     db_config.path_prefix = db_config.path_prefix.join("2");

crates/apollo_storage/src/lib.rs

@@ -520,7 +520,7 @@ impl StorageReader {
 }
 
 /// A struct for starting RW transactions ([`StorageTxn`]) to the storage.
-/// There is a single non clonable writer instance, to make sure there is only one write transaction
+/// There is a single non-clonable writer instance, to make sure there is only one write transaction
 /// at any given moment.
 pub struct StorageWriter {
     db_writer: DbWriter,
@@ -568,7 +568,7 @@ impl Drop for MetricsHandler {
 }
 
 /// A struct for interacting with the storage.
-/// The actually functionality is implemented on the transaction in multiple traits.
+/// The actual functionality is implemented on the transaction in multiple traits.
 pub struct StorageTxn<'env, Mode: TransactionKind> {
     txn: DbTransaction<'env, Mode>,
     file_handlers: FileHandlers<Mode>,

crates/blockifier/src/blockifier_versioned_constants.rs

@@ -559,7 +559,7 @@ pub struct OsResources {
     // Mapping from every syscall to its execution resources in the OS (e.g., amount of Cairo
     // steps).
     // TODO(Arni, 14/6/2023): Update `GetBlockHash` values.
-    // TODO(ilya): Consider moving the resources of a keccak round to a seperate dict.
+    // TODO(ilya): Consider moving the resources of a keccak round to a separate dict.
     execute_syscalls: HashMap<SyscallSelector, ResourcesParams>,
     // Mapping from every transaction to its extra execution resources in the OS,
     // i.e., resources that don't count during the execution itself.

crates/blockifier/src/concurrency/worker_pool.rs

@@ -141,7 +141,7 @@ impl<S: StateReader> WorkerThread<S> {
             panic!("Another thread panicked. Aborting.");
         }
 
-        // Making sure that the program will abort if a panic occured while halting
+        // Making sure that the program will abort if a panic occurred while halting
         // the scheduler.
         let abort_guard = AbortIfPanic;
         // If a panic is not handled or the handling logic itself panics, then we