Why if I wrap my if revert condition in {}, on OracleLib.sol, it breaksDSCEngine.sol ?

[yawarasuuna]

Does anyone know why if I wrap my if revert condition in {}, on OracleLib.sol, it breaksDSCEngine.sol ?
My AI buddies got confused every time and started hallucinating

It throws error

Member "staleCheckLatestRoundData" not found or not visible after argument-dependent lookup in contract AggregatorV3Interface.solidity(9582)

This code works

pragma solidity ^0.8.18;

import {AggregatorV3Interface} from "@chainlink/contracts/src/v0.8/interfaces/AggregatorV3Interface.sol";

/**
 * @title OracleLib
 * @author yawarasuuna
 * @notice This library is used to check Chainlink Oracle for stale data.
 * If a price is stale, the function will revert, and render the DSCEngine unusable - this is by design
 * We want DSCEngine to freeze if prices become stale.
 *
 * Known bug: if Chainlink network explodes and you have a lot of money locked in the protocol... too bad.
 */
library OracleLib {
    error OracleLib__StalePrice();

    // TODO instead of hardcoding, do it programatically
    uint256 private constant TIMEOUT = 3 hours; // 3 * 60 * 60 = 10800 seconds

    function staleCheckLatestRoundData(AggregatorV3Interface priceFeed) public view returns (uint80, int256, uint256, uint256, uint80) {
        (uint80 roundId, int256 answer, uint256 startedAt, uint256 updatedAt, uint80 answeredInRound) =
            priceFeed.latestRoundData();

        uint256 secondsSince = block.timestamp - updatedAt; // seconds since priceFeed was last updated
        if (secondsSince > TIMEOUT) revert OracleLib__StalePrice();
        return (roundId, answer, startedAt, updatedAt, answeredInRound);
    }
}

this one does not

pragma solidity ^0.8.18;

import {AggregatorV3Interface} from "@chainlink/contracts/src/v0.8/interfaces/AggregatorV3Interface.sol";

/**
 * @title OracleLib
 * @author yawarasuuna
 * @notice This library is used to check Chainlink Oracle for stale data.
 * If a price is stale, the function will revert, and render the DSCEngine unusable - this is by design
 * We want DSCEngine to freeze if prices become stale.
 *
 * Known bug: if Chainlink network explodes and you have a lot of money locked in the protocol... too bad.
 */
library OracleLib {
    error OracleLib__StalePrice();

    // TODO instead of hardcoding, do it programatically
    uint256 private constant TIMEOUT = 3 hours; // 3 * 60 * 60 = 10800 seconds

    function staleCheckLatestRoundData(AggregatorV3Interface priceFeed) public view returns (uint80, int256, uint256, uint256, uint80) {
        (uint80 roundId, int256 answer, uint256 startedAt, uint256 updatedAt, uint80 answeredInRound) =
            priceFeed.latestRoundData();

        uint256 secondsSince = block.timestamp - updatedAt; // seconds since priceFeed was last updated
        if (secondsSince > TIMEOUT) {
            revert OracleLib__StalePrice();
        }
        return (roundId, answer, startedAt, updatedAt, answeredInRound);
    }
}

[EngrPips]

Hey @yawarasuuna, This error seems weird, and it is tough to say what went wrong, Please delete your cache folder, run forge clean and then run foundryup then retry, and let's see if the issue persists.