bug(cheatcodes): readFile / readAddress not consistent depending of OS (Linux / Mac)

[vjuge]

Context

i'm using this piece of code to persist contract addresses on a deploy script:

function updateContractAddressJson(string memory contractName, string memory __chainid, address __address) private returns (string memory outputObj1) {
        string memory deployments = vm.readFile("./contracts/broadcast/deployments.json");

        address contract31337 = deployments.readAddress(string.concat("$.",contractName, ".31337"));
        address contract1 = deployments.readAddress(string.concat("$.",contractName, ".1"));
        address contract80001 = deployments.readAddress(string.concat("$.",contractName, ".80001"));

         // [...]       
        vm.serializeAddress(obj, "31337", contract31337);
         // [...]       
        outputObj1 = vm.serializeAddress(obj, __chainid, __address);
        console.log(outputObj1);

}

to read/update the file ./contracts/broadcast/deployments.json:

{
  "DAOTreasury": {
    "1": "0x0000000000000000000000000000000000000020",
    "31337": "0xe7f1725E7734CE288F8367e1Bb143E90bb3F0512",
    "80001": "0x0000000000000000000000000000000000000020"
  },
  "MockERC20": {
    "1": "0x0000000000000000000000000000000000000020",
    "31337": "0x5FbDB2315678afecb367f032d93F642f64180aa3",
    "80001": "0xdaAe8d27687155dD49B7D6dBc33369fd00081255"
  }
}

Issue

I used to launch this script either from a linux PC(fedora) or from a macbook, and the json path is not interpreted the same depending of the OS the script runs from.

Caution

in some case we need to have a prefix "$.", in some case not.

linux working code:

        address contract31337 = deployments.readAddress(string.concat("$.",contractName, ".31337"));

macos working code:

        address contract31337 = deployments.readAddress(string.concat(contractName, ".31337"));

Otherwise, I have this kind of error (sample running without the "$."prefix on linux):

    ├─ [0] VM::parseJson(<JSON file>, MockERC20.31337) [staticcall]
    │   └─ ← "path error: 
$MockERC20.31337

As you can see, on linux we need to have the prefix "$.".

Is this something i'm missing or a bug ?

[grandizzy]

should be fixed with #9107 by using new cheatcodes