remove redundent code comments

Author: nerolation

src/ethereum/osaka/block_access_lists/__init__.py

@@ -0,0 +1,51 @@
+"""
+Block Access Lists (EIP-7928) implementation for Ethereum Osaka fork.
+"""
+
+from .builder import (
+    BlockAccessListBuilder,
+    add_balance_change,
+    add_code_change,
+    add_nonce_change,
+    add_storage_read,
+    add_storage_write,
+    add_touched_account,
+    build,
+)
+from .tracker import (
+    StateChangeTracker,
+    set_transaction_index,
+    track_address_access,
+    track_balance_change,
+    track_code_change,
+    track_nonce_change,
+    track_storage_read,
+    track_storage_write,
+)
+from .utils import (
+    compute_bal_hash,
+    ssz_encode_block_access_list,
+    validate_bal_against_execution,
+)
+
+__all__ = [
+    "BlockAccessListBuilder",
+    "StateChangeTracker",
+    "add_balance_change",
+    "add_code_change",
+    "add_nonce_change",
+    "add_storage_read",
+    "add_storage_write",
+    "add_touched_account",
+    "build",
+    "compute_bal_hash",
+    "set_transaction_index",
+    "ssz_encode_block_access_list",
+    "track_address_access",
+    "track_balance_change",
+    "track_code_change",
+    "track_nonce_change",
+    "track_storage_read",
+    "track_storage_write",
+    "validate_bal_against_execution",
+]

src/ethereum/osaka/block_access_lists/builder.py

@@ -2,7 +2,7 @@
 Block Access List Builder for EIP-7928
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 
-This module implements the BAL builder that tracks all account and storage
+This module implements the Block Access List builder that tracks all account and storage
 accesses during block execution and constructs the final BlockAccessList.
 """
 

src/ethereum/osaka/block_access_lists/tracker.py

@@ -1,5 +1,5 @@
 """
-BAL State Change Tracker for EIP-7928
+Block Access List State Change Tracker for EIP-7928
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 
 This module tracks state changes during transaction execution to build Block Access Lists.
@@ -27,7 +27,7 @@
 @dataclass
 class StateChangeTracker:
     """
-    Tracks state changes during transaction execution for BAL construction.
+    Tracks state changes during transaction execution for Block Access List construction.
     """
     block_access_list_builder: BlockAccessListBuilder
     pre_storage_cache: Dict[tuple, U256] = field(default_factory=dict)

src/ethereum/osaka/block_access_lists/utils.py

@@ -31,7 +31,7 @@ def compute_bal_hash(bal: BlockAccessList) -> Hash32:
     """
     Compute the hash of a Block Access List.
     
-    The BAL is SSZ-encoded and then hashed with keccak256.
+    The Block Access List is SSZ-encoded and then hashed with keccak256.
     
     Parameters
     ----------
@@ -41,7 +41,7 @@ def compute_bal_hash(bal: BlockAccessList) -> Hash32:
     Returns
     -------
     hash :
-        The keccak256 hash of the SSZ-encoded BAL.
+        The keccak256 hash of the SSZ-encoded Block Access List.
     """
     bal_bytes = ssz_encode_block_access_list(bal)
     return keccak256(bal_bytes)
@@ -247,20 +247,20 @@ def validate_bal_against_execution(
     block_access_list_builder: Optional['BlockAccessListBuilder'] = None
 ) -> bool:
     """
-    Validate that a BAL is structurally correct and optionally matches a builder's state.
+    Validate that a Block Access List is structurally correct and optionally matches a builder's state.
     
     Parameters
     ----------
     bal :
         The Block Access List to validate.
     block_access_list_builder :
-        Optional BAL builder to validate against. If provided, checks that the BAL
-        hash matches what would be built from the builder's current state.
+        Optional Block Access List builder to validate against. If provided, checks that the
+        Block Access List hash matches what would be built from the builder's current state.
         
     Returns
     -------
     valid :
-        True if the BAL is structurally valid and matches the builder (if provided).
+        True if the Block Access List is structurally valid and matches the builder (if provided).
     """
     # 1. Validate structural constraints
 
@@ -326,10 +326,10 @@ def validate_bal_against_execution(
             if len(code_change.new_code) > MAX_CODE_SIZE:
                 return False
 
-    # 4. If BAL builder provided, validate against it by comparing hashes
+    # 4. If Block Access List builder provided, validate against it by comparing hashes
     if block_access_list_builder is not None:
         from .builder import build
-        # Build a BAL from the builder
+        # Build a Block Access List from the builder
         expected_bal = build(block_access_list_builder)
 
         # Compare hashes - much simpler!

src/ethereum/osaka/blocks.py

@@ -243,7 +243,7 @@ class Header:
 
     bal_hash: Hash32
     """
-    Hash of the Block Access List (BAL) containing all accounts and storage
+    Hash of the Block Access List containing all accounts and storage
     locations accessed during block execution. Introduced in [EIP-7928].
 
     [EIP-7928]: https://eips.ethereum.org/EIPS/eip-7928

src/ethereum/osaka/fork.py

@@ -763,7 +763,7 @@ def apply_body(
     """
     block_output = vm.BlockOutput()
 
-    # Initialize BAL state change tracker
+    # Initialize Block Access List state change tracker
     change_tracker = StateChangeTracker(block_output.block_access_list_builder)
 
     process_unchecked_system_transaction(

src/ethereum/osaka/ssz_types.py

@@ -2,7 +2,7 @@
 SSZ Types for EIP-7928 Block-Level Access Lists
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 
-This module defines the SSZ data structures for Block-Level Access Lists (BALs)
+This module defines the SSZ data structures for Block-Level Access Lists
 as specified in EIP-7928. These structures enable efficient encoding and
 decoding of all accounts and storage locations accessed during block execution.
 """

src/ethereum/osaka/state.py

@@ -510,9 +510,7 @@ def increase_recipient_balance(recipient: Account) -> None:
     modify_state(state, sender_address, reduce_sender_balance)
     modify_state(state, recipient_address, increase_recipient_balance)
 
-    # Track balance changes for BAL
     if change_tracker is not None:
-        # Track new balances after the transfer
         sender_new_balance = get_account(state, sender_address).balance
         recipient_new_balance = get_account(state, recipient_address).balance
 
@@ -549,7 +547,6 @@ def set_balance(account: Account) -> None:
 
     modify_state(state, address, set_balance)
 
-    # Track balance change for BAL
     if change_tracker is not None:
         change_tracker.track_balance_change(address, amount, state)
 
@@ -575,7 +572,7 @@ def increase_nonce(sender: Account) -> None:
 
     modify_state(state, address, increase_nonce)
 
-    # Track nonce change for BAL (for ALL accounts and ALL nonce changes)
+    # Track nonce change for Block Access List (for ALL accounts and ALL nonce changes)
     # This includes:
     # - EOA senders (transaction nonce increments)
     # - Contracts performing CREATE/CREATE2

src/ethereum/osaka/vm/instructions/environment.py

@@ -87,7 +87,6 @@ def balance(evm: Evm) -> None:
     # Non-existent accounts default to EMPTY_ACCOUNT, which has balance 0.
     balance = get_account(evm.message.block_env.state, address).balance
 
-    # BAL tracking for address access
     if evm.message.change_tracker:
         evm.message.change_tracker.track_address_access(address)
 
@@ -357,7 +356,6 @@ def extcodesize(evm: Evm) -> None:
     # OPERATION
     code = get_account(evm.message.block_env.state, address).code
 
-    # BAL tracking for address access
     if evm.message.change_tracker:
         evm.message.change_tracker.track_address_access(address)
 
@@ -403,7 +401,6 @@ def extcodecopy(evm: Evm) -> None:
     evm.memory += b"\x00" * extend_memory.expand_by
     code = get_account(evm.message.block_env.state, address).code
 
-    # BAL tracking for address access
     if evm.message.change_tracker:
         evm.message.change_tracker.track_address_access(address)
 
@@ -493,7 +490,6 @@ def extcodehash(evm: Evm) -> None:
     # OPERATION
     account = get_account(evm.message.block_env.state, address)
 
-    # BAL tracking for address access
     if evm.message.change_tracker:
         evm.message.change_tracker.track_address_access(address)
 

src/ethereum/osaka/vm/instructions/storage.py

@@ -60,7 +60,6 @@ def sload(evm: Evm) -> None:
         evm.message.block_env.state, evm.message.current_target, key
     )
 
-    # BAL tracking
     if evm.message.change_tracker:
         evm.message.change_tracker.track_storage_read(
             evm.message.current_target, key, evm.message.block_env.state
@@ -134,7 +133,6 @@ def sstore(evm: Evm) -> None:
         raise WriteInStaticContext
     set_storage(state, evm.message.current_target, key, new_value)
 
-    # BAL tracking
     if evm.message.change_tracker:
         evm.message.change_tracker.track_storage_write(
             evm.message.current_target, key, new_value, state