Update watcher to fetch historical logs

[K-Ho]

Description

Updates the watcher to fetch historical logs as well

Questions

Metadata

Fixes

• Fixes # [Link to Issue]

Contributing Agreement

• I have read and understood the Optimism Contributing Guide and Code of Conduct and am following those guidelines in this pull request.

[karlfloersch]

This looks awesome!!! Feels suuper clean to me -- a function which gives the tx receipt on the other layer. Great!

🚢

[karlfloersch]

Probably worth adding the specific type (TransactionReceipt?) here to make people's auto-correct happy.

[K-Ho]

This is quite possibly broken. We will definitely need this for phase C tho. Closing for now

[tynes]

CI is breaking due to a dependency issue:

error nanoid@3.1.12: The engine "node" is incompatible with this module. Expected version "^10 || ^12 || >=13.7". Got "11.15.0"
error Found incompatible module

If some of the dependencies don't work on node 12, one solution is to use node 10. Note that node 10 is in maintenance mode until April 2021, meaning that we should definitely upgrade to a newer version of node by that time. See https://nodejs.org/en/about/releases/

This error wasn't happening before, so another solution would be to downgrade nanoid, but I don't think that is an ideal thing to do. Its likely a transitive dependency that was automatically upgraded due to semver.

Another option is to use the the yarn option --ignore-engines, but that could cause issues if a dependency actually doesn't support the version we are using.

[tynes]

There can be an uncaught exception here, I think it would be better to try/catch the getTransactionReceipt and reject(e) in the catch(e)

[tynes]

I think that its a good idea to turn this function signature into a constant

[tynes]

You could do the combination of the filter + map in a single iteration using a loop.

const data = []
for (const log of receipt.logs) {
  if (log.address === layer.messengerAddress && log.topics[0] === ethers.utils.id('SentMessage(bytes32)')) {
    data.push(log.data)
  }
}
return data