contracts-bedrock: Add OPSM file I/O for superchain deployments

packages/contracts-bedrock/foundry.toml

@@ -49,7 +49,8 @@ fs_permissions = [
   { access='read', path = './forge-artifacts/' },
   { access='write', path='./semver-lock.json' },
   { access='read-write', path='./.testdata/' },
-  { access='read', path='./kout-deployment' }
+  { access='read', path='./kout-deployment' },
+  { access='read', path='./test/fixtures' },
 ]
 libs = ["node_modules", "lib"]
 

packages/contracts-bedrock/scripts/DeploySuperchain.s.sol

@@ -2,6 +2,7 @@
 pragma solidity 0.8.15;
 
 import { Script } from "forge-std/Script.sol";
+import { CommonBase } from "forge-std/Base.sol";
 
 import { SuperchainConfig } from "src/L1/SuperchainConfig.sol";
 import { ProtocolVersions, ProtocolVersion } from "src/L1/ProtocolVersions.sol";
@@ -57,19 +58,20 @@ import { Solarray } from "scripts/libraries/Solarray.sol";
  * scripts from the existing ones that "Config" and "Artifacts" terminology.
  */
 
-contract DeploySuperchainInput {
+contract DeploySuperchainInput is CommonBase {
     // The input struct contains all the input data required for the deployment.
+    // The fields must be in alphabetical order for vm.parseToml to work.
     struct Input {
-        Roles roles;
         bool paused;
-        ProtocolVersion requiredProtocolVersion;
         ProtocolVersion recommendedProtocolVersion;
+        ProtocolVersion requiredProtocolVersion;
+        Roles roles;
     }
 
     struct Roles {
-        address proxyAdminOwner;
-        address protocolVersionsOwner;
         address guardian;
+        address protocolVersionsOwner;
+        address proxyAdminOwner;
     }
 
     // This flag tells us if all inputs have been set. An `input()` getter method that returns all
@@ -84,10 +86,10 @@ contract DeploySuperchainInput {
 
     // Load the input from a TOML file.
     function loadInputFile(string memory _infile) public {
-        _infile;
-        Input memory parsedInput;
+        string memory toml = vm.readFile(_infile);
+        bytes memory data = vm.parseToml(toml);
+        Input memory parsedInput = abi.decode(data, (Input));
         loadInput(parsedInput);
-        require(false, "DeploySuperchainInput: not implemented");
     }
 
     // Load the input from a struct.
@@ -153,7 +155,7 @@ contract DeploySuperchainInput {
     }
 }
 
-contract DeploySuperchainOutput {
+contract DeploySuperchainOutput is CommonBase {
     // The output struct contains all the output data from the deployment.
     struct Output {
         ProxyAdmin superchainProxyAdmin;
@@ -182,9 +184,14 @@ contract DeploySuperchainOutput {
     }
 
     // Save the output to a TOML file.
-    function writeOutputFile(string memory _outfile) public pure {
-        _outfile;
-        require(false, "DeploySuperchainOutput: not implemented");
+    function writeOutputFile(string memory _outfile) public {
+        string memory key = "dso-outfile";
+        vm.serializeAddress(key, "proxyAdmin", address(outputs.superchainProxyAdmin));
+        vm.serializeAddress(key, "superchainConfigImpl", address(outputs.superchainConfigImpl));
+        vm.serializeAddress(key, "superchainConfigProxy", address(outputs.superchainConfigProxy));
+        vm.serializeAddress(key, "protocolVersionsImpl", address(outputs.protocolVersionsImpl));
+        string memory out = vm.serializeAddress(key, "protocolVersionsProxy", address(outputs.protocolVersionsProxy));
+        vm.writeToml(out, _outfile);
     }
 
     function output() public view returns (Output memory) {
@@ -238,7 +245,7 @@ contract DeploySuperchain is Script {
     // This entrypoint is for end-users to deploy from an input file and write to an output file.
     // In this usage, we don't need the input and output contract functionality, so we deploy them
     // here and abstract that architectural detail away from the end user.
-    function run(string memory _infile) public {
+    function run(string memory _infile, string memory _outfile) public {
         // End-user without file IO, so etch the IO helper contracts.
         (DeploySuperchainInput dsi, DeploySuperchainOutput dso) = etchIOContracts();
 
@@ -248,10 +255,8 @@ contract DeploySuperchain is Script {
         // Run the deployment script and write outputs to the DeploySuperchainOutput contract.
         run(dsi, dso);
 
-        // Write the output data to a file. The file
-        string memory outfile = ""; // This will be derived from input file name, e.g. `foo.in.toml` -> `foo.out.toml`
-        dso.writeOutputFile(outfile);
-        require(false, "DeploySuperchain: run is not implemented");
+        // Write the output data to a file.
+        dso.writeOutputFile(_outfile);
     }
 
     // This entrypoint is for use with Solidity tests, where the input and outputs are structs.

packages/contracts-bedrock/test/DeploySuperchain.t.sol

@@ -32,27 +32,15 @@ contract DeploySuperchainInput_Test is Test {
         // parameters to e.g. avoid the zero address. Therefore we hardcode a concrete test case
         // which is simpler and still sufficient.
         dsi.loadInput(input);
+        _assertLoadInput();
+    }
 
-        assertTrue(dsi.inputSet(), "100");
-
-        // Compare the test input struct to the getter methods.
-        assertEq(input.roles.proxyAdminOwner, dsi.proxyAdminOwner(), "200");
-        assertEq(input.roles.protocolVersionsOwner, dsi.protocolVersionsOwner(), "300");
-        assertEq(input.roles.guardian, dsi.guardian(), "400");
-        assertEq(input.paused, dsi.paused(), "500");
-        assertEq(
-            ProtocolVersion.unwrap(input.requiredProtocolVersion),
-            ProtocolVersion.unwrap(dsi.requiredProtocolVersion()),
-            "600"
-        );
-        assertEq(
-            ProtocolVersion.unwrap(input.recommendedProtocolVersion),
-            ProtocolVersion.unwrap(dsi.recommendedProtocolVersion()),
-            "700"
-        );
+    function test_loadInputFile_succeeds() public {
+        string memory root = vm.projectRoot();
+        string memory path = string.concat(root, "/test/fixtures/test-deploy-superchain.toml");
 
-        // Compare the test input struct to the `input` getter method.
-        assertEq(keccak256(abi.encode(input)), keccak256(abi.encode(dsi.input())), "800");
+        dsi.loadInputFile(path);
+        _assertLoadInput();
     }
 
     function test_getters_whenNotSet_revert() public {
@@ -76,6 +64,29 @@ contract DeploySuperchainInput_Test is Test {
         vm.expectRevert(expectedErr);
         dsi.recommendedProtocolVersion();
     }
+
+    function _assertLoadInput() internal {
+        assertTrue(dsi.inputSet(), "100");
+
+        // Compare the test input struct to the getter methods.
+        assertEq(input.roles.proxyAdminOwner, dsi.proxyAdminOwner(), "200");
+        assertEq(input.roles.protocolVersionsOwner, dsi.protocolVersionsOwner(), "300");
+        assertEq(input.roles.guardian, dsi.guardian(), "400");
+        assertEq(input.paused, dsi.paused(), "500");
+        assertEq(
+            ProtocolVersion.unwrap(input.requiredProtocolVersion),
+            ProtocolVersion.unwrap(dsi.requiredProtocolVersion()),
+            "600"
+        );
+        assertEq(
+            ProtocolVersion.unwrap(input.recommendedProtocolVersion),
+            ProtocolVersion.unwrap(dsi.recommendedProtocolVersion()),
+            "700"
+        );
+
+        // Compare the test input struct to the `input` getter method.
+        assertEq(keccak256(abi.encode(input)), keccak256(abi.encode(dsi.input())), "800");
+    }
 }
 
 contract DeploySuperchainOutput_Test is Test {
@@ -165,6 +176,35 @@ contract DeploySuperchainOutput_Test is Test {
         vm.expectRevert(expectedErr);
         dso.protocolVersionsProxy();
     }
+
+    function test_writeOutputFile_succeeds() public {
+        DeploySuperchainOutput.Output memory output = DeploySuperchainOutput.Output({
+            superchainProxyAdmin: ProxyAdmin(makeAddr("superchainProxyAdmin")),
+            superchainConfigImpl: SuperchainConfig(makeAddr("superchainConfigImpl")),
+            superchainConfigProxy: SuperchainConfig(makeAddr("superchainConfigProxy")),
+            protocolVersionsImpl: ProtocolVersions(makeAddr("protocolVersionsImpl")),
+            protocolVersionsProxy: ProtocolVersions(makeAddr("protocolVersionsProxy"))
+        });
+
+        dso.set(dso.superchainProxyAdmin.selector, address(output.superchainProxyAdmin));
+        dso.set(dso.superchainConfigImpl.selector, address(output.superchainConfigImpl));
+        dso.set(dso.superchainConfigProxy.selector, address(output.superchainConfigProxy));
+        dso.set(dso.protocolVersionsImpl.selector, address(output.protocolVersionsImpl));
+        dso.set(dso.protocolVersionsProxy.selector, address(output.protocolVersionsProxy));
+
+        string memory root = vm.projectRoot();
+        string memory path = string.concat(root, "/.testdata/test-deploy-superchain-output.toml");
+        dso.writeOutputFile(path);
+        string memory data = vm.readFile(path);
+
+        // Clean up before asserting so that we don't leave any files behind.
+        vm.removeFile(path);
+
+        assertEq(
+            keccak256(abi.encode(data)),
+            bytes32(0x496d8cafe47414684c9ccf13231a39c825f3740e0ed19854c54e1df691591fbd)
+        );
+    }
 }
 
 contract DeploySuperchain_Test is Test {
@@ -193,7 +233,7 @@ contract DeploySuperchain_Test is Test {
         return ProtocolVersion.unwrap(_pv);
     }
 
-    function test_run_succeeds(DeploySuperchainInput.Input memory _input) public {
+    function test_run_memory_succeeds(DeploySuperchainInput.Input memory _input) public {
         vm.assume(_input.roles.proxyAdminOwner != address(0));
         vm.assume(_input.roles.protocolVersionsOwner != address(0));
         vm.assume(_input.roles.guardian != address(0));
@@ -244,6 +284,22 @@ contract DeploySuperchain_Test is Test {
         dso.checkOutput();
     }
 
+    function test_run_io_succeeds() public {
+        string memory root = vm.projectRoot();
+        string memory inpath = string.concat(root, "/test/fixtures/test-deploy-superchain.toml");
+        string memory outpath = string.concat(root, "/.testdata/test-deploy-superchain-out.toml");
+
+        deploySuperchain.run(inpath, outpath);
+
+        string memory outdata = vm.readFile(outpath);
+        // Clean up before asserting so that we don't leave any files behind.
+        vm.removeFile(outpath);
+        assertEq(
+            keccak256(abi.encode(outdata)),
+            bytes32(0x08aee2fe779ae629438776065e99398edaec136e7b57da9fd8574581aa7a0b3d)
+        );
+    }
+
     function test_run_ZeroAddressRoleInput_reverts() public {
         // Snapshot the state so we can revert to the default `input` struct between assertions.
         uint256 snapshotId = vm.snapshot();

packages/contracts-bedrock/test/fixtures/test-deploy-superchain.toml

@@ -0,0 +1,8 @@
+paused = false
+requiredProtocolVersion = 1
+recommendedProtocolVersion = 2
+
+[roles]
+proxyAdminOwner = "0x51f0348a9fA2aAbaB45E82825Fbd13d406e04497"
+protocolVersionsOwner = "0xeEB4cc05dC0dE43c465f97cfc703D165418CA93A"
+guardian = "0xE5DbA98c65F4B9EB0aeEBb3674fE64f88509a1eC"