search

consensus-shipyard / ipc-actors

Rust implementation of the IPC actors for FVM
MIT License
6 stars 3 forks source link

Code refactor in preparation for IPC M2 #21

Closed adlrocha closed 1 year ago

adlrocha commented 1 year ago

Background

The code for IPC actors is scattered in several different repos, mainly:

This has been a consequence of the different stages of development of these actors. We first started with the gateway as part of the builtin-actor and the subnet actor as a user-defined actor; to then having both as user-defined actors; to having to decouple the atomic execution logic from the gateway, etc. As the code (and our plans) are starting to stabilize, I think we would benefit from having all the IPC actors code, primitives and sdk in the same repo. The protocol will evolve as a whole, and it will ease the deployment and reduce the maintenance burden in the future. Dependency updates and new releases will be easier to handle. This may also ease the transition for the case that we end up going "full system actors" for IPC (see https://github.com/consensus-shipyard/ipc-gateway/issues/19).

Goal

The goal would be to have all IPC actor-related code in the same repo with (at least initially) the following crates:

Tasks

adlrocha commented 1 year ago

@cryptoAtwill, @sergefdrv, I would love your thoughts. I would try to take this one ASAP if possible. Thanks!

sergefdrv commented 1 year ago

I like this idea! Though I would first wait until all open pull requests in other repos get merged before moving the code.

adlrocha commented 1 year ago

Closed by #23 and #33