ignore old automation & fix minor vrf

contracts/.solhintignore

@@ -1,6 +1,11 @@
 # 344 warnings
 #./src/v0.8/automation
 
+# Ignore frozen Automation code
+./src/v0.8/automation/v1_2
+./src/v0.8/automation/v1_3
+./src/v0.8/automation/v2_0
+
 # Ignore Functions v1.0.0 code that was frozen after audit
 ./src/v0.8/functions/v1_0_0
 

contracts/package.json

@@ -18,7 +18,7 @@
     "prepublishOnly": "pnpm compile && ./scripts/prepublish_generate_abi_folder",
     "publish-beta": "pnpm publish --tag beta",
     "publish-prod": "npm dist-tag add @chainlink/[email protected] latest",
-    "solhint": "solhint --max-warnings 376 \"./src/v0.8/**/*.sol\""
+    "solhint": "solhint --max-warnings 180 \"./src/v0.8/**/*.sol\""
   },
   "files": [
     "src/v0.8",

contracts/src/v0.8/ValidatorProxy.sol

@@ -167,6 +167,7 @@ contract ValidatorProxy is AggregatorValidatorInterface, TypeAndVersionInterface
     current = s_currentAggregator.target;
     hasProposal = s_currentAggregator.hasNewProposal;
     proposed = s_proposedAggregator;
+    return (current, hasProposal, proposed);
   }
 
   /** VALIDATOR CONFIGURATION FUNCTIONS **/
@@ -216,6 +217,7 @@ contract ValidatorProxy is AggregatorValidatorInterface, TypeAndVersionInterface
     current = s_currentValidator.target;
     hasProposal = s_currentValidator.hasNewProposal;
     proposed = s_proposedValidator;
+    return (current, hasProposal, proposed);
   }
 
   /**

contracts/src/v0.8/transmission/dev/ERC-4337/Paymaster.sol

@@ -103,6 +103,7 @@ contract Paymaster is IPaymaster, ConfirmedOwner {
         extraCost = directFundingData.topupAmount;
       }
     }
+    return extraCost;
   }
 
   /// @dev Deducts user subscription balance after execution.
@@ -116,6 +117,7 @@ contract Paymaster is IPaymaster, ConfirmedOwner {
 
   function _getCostJuels(uint256 costWei) internal view returns (uint256 costJuels) {
     costJuels = (1e18 * costWei) / uint256(_getFeedData());
+    return costJuels;
   }
 
   function _getFeedData() internal view returns (int256) {

contracts/src/v0.8/transmission/dev/ERC-4337/SCALibrary.sol

@@ -31,6 +31,7 @@ library SCALibrary {
         hashOfEncoding
       )
     );
+    return fullHash;
   }
 
   function _recoverSignature(bytes memory signature, bytes32 fullHash) internal pure returns (address) {

contracts/src/v0.8/transmission/dev/ERC-4337/SmartContractAccountFactory.sol

@@ -27,5 +27,7 @@ contract SmartContractAccountFactory {
     }
 
     emit ContractCreated(scaAddress);
+
+    return scaAddress;
   }
 }

contracts/src/v0.8/transmission/dev/testhelpers/SmartContractAccountHelper.sol

@@ -18,6 +18,7 @@ library SmartContractAccountHelper {
       SCA.executeTransactionFromEntryPoint.selector,
       abi.encode(endContract, value, block.timestamp + deadline, data)
     );
+    return encoding;
   }
 
   function getFullHashForSigning(bytes32 userOpHash, address scaAddress) public view returns (bytes32) {
@@ -29,6 +30,7 @@ library SmartContractAccountHelper {
     address entryPoint
   ) public pure returns (bytes memory initCode) {
     initCode = bytes.concat(INITIALIZE_CODE, abi.encode(owner, entryPoint));
+    return initCode;
   }
 
   function getInitCode(
@@ -46,6 +48,7 @@ library SmartContractAccountHelper {
         initializeCodeWithConstructor
       )
     );
+    return initCode;
   }
 
   /// @dev Computes the smart contract address that results from a CREATE2 operation, per EIP-1014.

contracts/src/v0.8/vrf/KeepersVRFConsumer.sol

@@ -61,6 +61,7 @@ contract KeepersVRFConsumer is KeeperCompatibleInterface, VRFConsumerBaseV2 {
    * @return upkeepNeeded true if and only if at least UPKEEP_INTERVAL seconds have elapsed since the last upkeep or since construction
    * of the contract.
    */
+  // solhint-disable-next-line chainlink-solidity/explicit-returns
   function checkUpkeep(
     bytes calldata /* checkData */
   ) external view override returns (bool upkeepNeeded, bytes memory /* performData */) {

contracts/src/v0.8/vrf/VRF.sol

@@ -205,6 +205,7 @@ contract VRF {
     while (x_ >= FIELD_SIZE) {
       x_ = uint256(keccak256(abi.encodePacked(x_)));
     }
+    return x_;
   }
 
   // Hash b to a random point which hopefully lies on secp256k1. The y ordinate
@@ -223,6 +224,7 @@ contract VRF {
         p[1] = FIELD_SIZE - p[1];
       }
     }
+    return p;
   }
 
   // Domain-separation tag for initial hash in _hashToCurve. Corresponds to
@@ -248,6 +250,7 @@ contract VRF {
     while (!_isOnCurve(rv)) {
       rv = _newCandidateSecp256k1Point(abi.encodePacked(rv[0]));
     }
+    return rv;
   }
 
   /** *********************************************************************
@@ -294,6 +297,7 @@ contract VRF {
       uint256 num2 = mulmod(FIELD_SIZE - x2, z1, FIELD_SIZE);
       (x3, z3) = (addmod(num1, num2, FIELD_SIZE), mulmod(z1, z2, FIELD_SIZE));
     }
+    return (x3, z3);
   }
 
   // Returns x1/z1*x2/z2=(x1x2)/(z1z2), in projective coordinates on P¹(𝔽ₙ)
@@ -304,6 +308,7 @@ contract VRF {
     uint256 z2
   ) internal pure returns (uint256 x3, uint256 z3) {
     (x3, z3) = (mulmod(x1, x2, FIELD_SIZE), mulmod(z1, z2, FIELD_SIZE));
+    return (x3, z3);
   }
 
   /** **************************************************************************
@@ -385,6 +390,7 @@ contract VRF {
         sz = dx;
       }
     }
+    return (sx, sy, sz);
   }
 
   // p1+p2, as affine points on secp256k1.
@@ -577,5 +583,6 @@ contract VRF {
       proof.zInv
     );
     output = uint256(keccak256(abi.encode(VRF_RANDOM_OUTPUT_HASH_PREFIX, proof.gamma)));
+    return output;
   }
 }

contracts/src/v0.8/vrf/VRFCoordinatorV2.sol

@@ -491,6 +491,7 @@ contract VRFCoordinatorV2 is VRF, ConfirmedOwner, TypeAndVersionInterface, VRFCo
     // The seed actually used by the VRF machinery, mixing in the blockhash
     uint256 actualSeed = uint256(keccak256(abi.encodePacked(proof.seed, blockHash)));
     randomness = VRF._randomValueFromVRFProof(proof, actualSeed); // Reverts on failure
+    return (keyHash, requestId, randomness);
   }
 
   /*