Inconsistent decimal handling in liquidity calculation

The _calcAddLiquidity function inconsistently handles decimal places during liquidity calculations:

1. Initial Calculation (Correct):

   • Uses provDecimals (from price feed) when calculating valueChange:
   solidityCopy

   (tokenPrice, provDecimals) = _getLastPrice(_token);
   valueChange = (_amountIn * tokenPrice).toDecimals(tokenDecimals + provDecimals, 18);

2. LP Amount Calculation (Incorrect):

   • Hardcodes 8 decimals instead of using provDecimals:
   solidityCopy

   lpAmount = lpAmount.toDecimals(tokenDecimals + 8, plpDecimals);  // Should use provDecimals

This inconsistency occurs in both branches of the LP amount calculation (when lpSupply == 0 and the else case). The function assumes price feed decimals are always 8, which may not be true for all oracle providers.

Recommendations

Use proveDecimals consistently throughout the codebase to standardize handling of different token price decimal formats.