requested changes

contracts/TrustedVerifier.sol

@@ -19,14 +19,6 @@ contract TrustedVerifier is IGatewayVerifier {
 
     mapping(address fetcher => Config) _configs;
 
-    function gatewayURLs() external view returns (string[] memory) {
-        return _configs[msg.sender].urls;
-    }
-
-    function getLatestContext() external view returns (bytes memory) {
-        return abi.encode(block.timestamp, msg.sender);
-    }
-
     modifier _canModifyFetcher(address op, address fetcher) {
         if (fetcher == op) {
             require(fetcher.code.length != 0, 'Trusted: not code');
@@ -82,6 +74,14 @@ contract TrustedVerifier is IGatewayVerifier {
         return _configs[sender].signers[signer];
     }
 
+    function gatewayURLs() external view returns (string[] memory) {
+        return _configs[msg.sender].urls;
+    }
+
+    function getLatestContext() external view returns (bytes memory) {
+        return abi.encode(block.timestamp, msg.sender);
+    }
+
     struct GatewayProof {
         bytes signature;
         uint64 signedAt;

scripts/serve.ts

@@ -343,6 +343,9 @@ function getProverFactory(chain: Chain): LatestProverFactory<AbstractProver> {
     case CHAINS.ZKSYNC_SEPOLIA:
       return ZKSyncProver;
     // NOTE: linea should use eth_getProof instead of linea_getProof
+    // NOTE: this probably needs "--latest" cli option too
+    // rollup => SMT w/Mimc root using linea_getProof
+    // chain => PMT w/Keccak root using eth_getProof
     // case CHAINS.LINEA:
     // case CHAINS.LINEA_SEPOLIA:
     //   return LineaProver;

src/GatewayProvider.ts

@@ -51,10 +51,13 @@ export class GatewayProvider extends JsonRpcProvider {
     let backoff = 250;
     for (let attempt = 0; ; attempt++) {
       try {
-        // ethers bug: this type is wrong
+        // ethers bug: return type is wrong
+        // expected: (JsonRpcResult | JsonRpcErrror)[]
         const results: any[] = await super._send(payload);
         if (!results.some((x) => x.error?.code === 429)) {
-          // what the fuck is this
+          // handle alchemy weirdness
+          // note: this is only supposed to happen over WebSocket
+          // note: there is no header "retry-after"
           // https://docs.alchemy.com/reference/throughput
           return results;
         }

src/chains.ts

@@ -47,10 +47,6 @@ export const CHAINS = {
   APE: 33139n,
 } as const satisfies Record<string, Chain>;
 
-// const NAMES = new Map<Chain, string>(
-//   Object.entries(CHAINS).map(([a, b]) => [b, a])
-// );
-
 export function chainName(chain: Chain): string {
   for (const [name, c] of Object.entries(CHAINS)) {
     if (c === chain) return name;

src/linea/LineaProver.ts

@@ -154,7 +154,7 @@ export class LineaProver extends BlockProver {
         // 20240825: most cloud providers seem to reject batched getProof
         // since we aren't in control of provider construction (ie. batchMaxSize)
         // sendImmediate is a temporary hack to avoid this issue
-        // 20241027: switch to RetryProvider
+        // 20241027: use GatewayProvider
         this.provider.send('linea_getProof', [
           target,
           slots

src/vm.ts

@@ -474,7 +474,7 @@ function checkSize(size: bigint | number, limit: number) {
 }
 
 const GATEWAY_EXT_ABI = new Interface([
-  // IReadBytesAt.sol
+  // ReadBytesAt.sol
   'function readBytesAt(uint256 slot) view returns (bytes)',
 ]);
 
@@ -560,7 +560,10 @@ export abstract class AbstractProver {
       storageProofs: Array.from(order.subarray(1), (i) => proofs[i]),
     };
   }
-  // non-essential
+  // NOTE: if a prover cannot provide this value, throw
+  // eg. LineaProver stateRoot is part of the rollup machinery
+  // a block-derived LineaProver doesn't have a stateRoot
+  // whereas LineaRollup => getCommit() => prover does (from L1)
   abstract fetchStateRoot(): Promise<HexString32>;
 
   // machine interface

test/gateway/common.ts

@@ -208,6 +208,7 @@ export function testTrustedEth(chain2: Chain, opts: TestOptions) {
         EthProver,
         new SigningKey(randomBytes(32))
       );
+      rollup.latestBlockTag = 'latest';
       afterAll(foundry.shutdown);
       const gateway = new Gateway(rollup);
       const ccip = await serve(gateway, { protocol: 'raw', log: !!opts.log });