-
-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Document high-level MetaMask architecture (from a package/controller perspective) #27
Comments
Hey team! Please add your planning poker estimate with Zenhub @cryptodev-2s @Gudahtt @kanthesha @MajorLift @mikesposito |
For a past example of architecture documentation: https://github.com/MetaMask/metamask-extension#architecture Lots of inaccuracies or misleading details in that diagram, but it's mostly correct and shows how the different processes interact |
Some ideas that would be nice to have for this:
It would be great to avoid getting into too much detail on each of these as well, so that the documentation is less likely to go stale as we change things. |
Someone new to MetaMask might look at all of the repos that we have and (after they get over their initial shock) ask, how do all of these pieces fit together?
There are a few ways we could answer that question, but to start, we could create an architecture graph that includes the extension, mobile app, and packages that are used along the way (including core controllers, but also libraries like
eth-sig-util
). To provide another view, we could have a list below the graph which summarizes all of the packages with a short summary of the role that they play. Then we put both of these things in a new document.The text was updated successfully, but these errors were encountered: