Skip to content
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

Shared Libraries - Reduce maintenance burden for core wallet functionality - Q423 #1780

Closed
3 of 5 tasks
mcmire opened this issue Oct 5, 2023 · 0 comments
Closed
3 of 5 tasks

Comments

@mcmire
Copy link
Contributor

mcmire commented Oct 5, 2023

2023/Q4/O1/KR2 Description:

5 libraries are migrated to core monorepo
Some of the core wallet functionality is held in utility libraries. We #1079 the set of libraries that need to be migrated into the core monorepo.

In order to fully complete this task, we may need to raise the code quality of these libraries by converting the existing code to TypeScript, upgrading the minimum Node version, upgrading Yarn and Jest, upgrading our ESLint packages along with any configuration, upgrading other dependencies to their latest versions, and aligning the project to match our module template.

Slated libraries for migration (we were able to start on these at the very end of Q3 so we will add a few more to this list to replace the ones we finish ahead of time):

  • @metamask/utils
  • @metamask/json-rpc-engine
  • @metamask/eth-json-rpc-provider
  • @metamask/json-rpc-middleware-stream

2023/Q4/O1/KR1 Description:

5 extension controllers are merged into core monorepo
Much of the core wallet functionality is managed by specific libraries known as “controllers”. We previously designated the set of controllers used by the extension that were eligible for merging into the core monorepo. We will choose a few from this list that we are confident we can complete without a need to collaborate with any other team.
In order to fully complete this task, we may need to raise the code quality of these controllers by refactoring the existing code, simplifying the existing API, adding missing tests, improving the quality of existing tests, or adding missing documentation.

Controllers to be merged:

  • AccountTracker (core) / CachedBalances (extension) [there may be some overlap with accounts]
  • TokenBalances
  • TokenDetection (core) / DetectTokens (extension)
  • PermissionLog
  • ENS [stretch goal]
@mcmire mcmire added the Epic label Oct 5, 2023
@desi desi changed the title Shared Libraries - 2023/Q4/O1/KR1 - 5 extension controllers are merged into core monorepo Shared Libraries - Core MM Wallet Jan 11, 2024
@desi desi changed the title Shared Libraries - Core MM Wallet Shared Libraries - Reduce maintenance burden for core wallet functionality Jan 24, 2024
@desi desi changed the title Shared Libraries - Reduce maintenance burden for core wallet functionality Shared Libraries - Reduce maintenance burden for core wallet functionality - Q423 Jan 24, 2024
@desi desi closed this as completed Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants