style: rename vault decimals

Author: QGarchery

src/ChainlinkOracle.sol

@@ -18,9 +18,9 @@ contract ChainlinkOracle is IOracle {
 
     /// @notice Vault.
     IERC4626 public immutable VAULT;
-    /// @notice Vault precision. The number of decimals used to price shares of the vault.
-    /// @notice Should be chosen such that converting `10 ** VAULT_PRECISION` to assets has enough precision.
-    uint256 public immutable VAULT_PRECISION;
+    /// @notice Conversion sample decimals. The decimals of the shares sample used to convert to the underlying asset.
+    /// @notice Should be chosen such that converting `10 ** CONVERSION_SAMPLE_DECIMALS` to assets has enough precision.
+    uint256 public immutable CONVERSION_SAMPLE_DECIMALS;
     /// @notice First base feed.
     AggregatorV3Interface public immutable BASE_FEED_1;
     /// @notice Second base feed.
@@ -39,7 +39,7 @@ contract ChainlinkOracle is IOracle {
     /// @param baseFeed2 Second base feed. Pass address zero if the price = 1.
     /// @param quoteFeed1 First quote feed. Pass address zero if the price = 1.
     /// @param quoteFeed2 Second quote feed. Pass address zero if the price = 1.
-    /// @param vaultPrecision Vault precision. Pass 0 if the oracle does not use a vault.
+    /// @param conversionSampleDecimals Conversion sample decimals. Pass 0 if the oracle does not use a vault.
     /// @param baseTokenDecimals Base token decimals.
     /// @param quoteTokenDecimals Quote token decimals.
     constructor(
@@ -48,15 +48,15 @@ contract ChainlinkOracle is IOracle {
         AggregatorV3Interface baseFeed2,
         AggregatorV3Interface quoteFeed1,
         AggregatorV3Interface quoteFeed2,
-        uint256 vaultPrecision,
+        uint256 conversionSampleDecimals,
         uint256 baseTokenDecimals,
         uint256 quoteTokenDecimals
     ) {
         // The vault parameter is used for ERC4626 tokens, to price its shares.
-        // It is used to price `10 ** VAULT_PRECISION` of the vault shares, so it requires dividing by that number,
-        // hence the `VAULT_PRECISION` subtraction in the `SCALE_FACTOR` definition.
+        // It is used to price `10 ** CONVERSION_SAMPLE_DECIMALS` of the vault shares, so it requires dividing
+        // by that number, hence the `CONVERSION_SAMPLE_DECIMALS` subtraction in the `SCALE_FACTOR` definition.
         VAULT = vault;
-        VAULT_PRECISION = vaultPrecision;
+        CONVERSION_SAMPLE_DECIMALS = conversionSampleDecimals;
         BASE_FEED_1 = baseFeed1;
         BASE_FEED_2 = baseFeed2;
         QUOTE_FEED_1 = quoteFeed1;
@@ -78,15 +78,16 @@ contract ChainlinkOracle is IOracle {
         SCALE_FACTOR = 10
             ** (
                 36 + quoteTokenDecimals + quoteFeed1.getDecimals() + quoteFeed2.getDecimals() - baseTokenDecimals
-                    - baseFeed1.getDecimals() - baseFeed2.getDecimals() - vaultPrecision
+                    - baseFeed1.getDecimals() - baseFeed2.getDecimals() - CONVERSION_SAMPLE_DECIMALS
             );
     }
 
     /* PRICE */
 
     /// @inheritdoc IOracle
     function price() external view returns (uint256) {
-        return (VAULT.getAssets(10 ** VAULT_PRECISION) * BASE_FEED_1.getPrice() * BASE_FEED_2.getPrice() * SCALE_FACTOR)
+        uint256 sample = 10 ** CONVERSION_SAMPLE_DECIMALS;
+        return (VAULT.getAssets(sample) * BASE_FEED_1.getPrice() * BASE_FEED_2.getPrice() * SCALE_FACTOR)
             / (QUOTE_FEED_1.getPrice() * QUOTE_FEED_2.getPrice());
     }
 }