Replies: 1 comment 1 reply
-
I like this idea, Are you going to seperate every connector: |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
This RFC (Request for Comments) document outlines the updates to the current architecture of adapters (aka connectors) to minimize the package bloating, yet enabling other projects to use all adapters (aka connectors) in a single package to implement a better wallet connection experience.
Also, since the focus of this RFC is connectors, it is proposed to implement all the connectors that exist in
@cosmos-kit/react
as such proposes a variety of connectors, whilegraz
implements only some of them.Motivation
Given that
wagmi
efficiently manages its connectors by separating all of them under@wagmi/connectors
package, it seems beneficial forgraz
to adopt a similar approach. This would reduce amount of packages, yet would still tree-shake effectively sincegraz
would not introduceindex.ts
barrel files for such, reducing the consumer's bundle size, and keeping thegraz
as a codebase more granular.As well as adding more connectors would inspire more FE engineers to use the library since it would offer the same variety of connectors that
@cosmos-kit/react
has, yet offering a better developer experience using the variety of hooksgraz
offers.Goals
The primary objectives are:
@graz-sh/connectors
package which would have all the existing adapters moved into;@cosmos-kit/react
offers to have a better variety of wallets to connect to;Adapter
toConnector
, as well as getting rid of unnecessary interface extension sinceConnector
is not being used anywhere, thus being an useless export, and getting rid ofGraz
prefix as it is implicitly known for a consumer that such a type comes fromgraz
:graz/packages/graz/src/adapter/index.ts
Lines 11 to 23 in 137b2dc
Beta Was this translation helpful? Give feedback.
All reactions