Token

In order to clearly identify the Solidity version with which the contracts will be compiled, pragma directives should be fixed and consistent across files within a project.

To avoid unexpected changes in bytecode across deployments the pragma version of the BTCXDigitalCurrency contract should be fixed instead of the current unfixed ^0.8.27 specification.

Fix the exact version of Solidity that will be used to deploy the BTCX token in the BTCXDigitalCurrency file.

BTCX Team: Resolved.

The initial supply for the BTCXDigitalCurrency contract is 1.2 Billion tokens and this is represented as 1200000000 * 10 ** decimals().

However this formatting may prove not optimally readable for verifiers or readers.

Consider using underscores to clearly display the supply as 1.2 Billion as follows:

1_200_000_000 * 10 ** decimals()

BTCX Team: Resolved.

The BTCXDigitalCurrency inherits from ERC20 as well as ERC20Burnable and ERC20Permit.

However, the direct is ERC20 inheritance is not strictly necessary since both the ERC20Burnable and ERC20Permit contracts inherit from ERC20 themselves.

Consider removing the direct is ERC20 inheritance as it is not strictly necessary, however there is no harm in keeping it.

BTCX Team: Acknowledged.

The constructor emits an InitialMint event upon minting the 1.2 Billion initial supply of tokens.

However, this initial event emission is not necessary because the _mint function itself emits a Transfer event where the from address is address(0) to indicate that this is a mint.

As this is the only time a mint can occur with the BTCX token, this is sufficient to represent the initial mint.

Consider removing the InitialMint event as it is redundant with the event emitted by _mint.

BTCX Team: Acknowledged.