-
Notifications
You must be signed in to change notification settings - Fork 76
⭐️ LIT-4211 - V8 auth and lit client refactor #800
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
Draft
MaximusHaximus
wants to merge
71
commits into
feat/naga-fix-bls-wasm-cleanup
Choose a base branch
from
LIT-4211-auth-refactor
base: feat/naga-fix-bls-wasm-cleanup
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
⭐️ LIT-4211 - V8 auth and lit client refactor #800
MaximusHaximus
wants to merge
71
commits into
feat/naga-fix-bls-wasm-cleanup
from
LIT-4211-auth-refactor
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… @lit-protocol/networks
- Define localStorage storage plugin w/ tests
- Define localStorage storage plugin w/ tests
…rors in preparation for moving providers to new auth package as authenticators
…r module to auth package
- Migrated `lit-auth-client` providers and relayer module to auth package - Collapsed `auth-browser` into `auth` package as an authenticator - `MetamaskAuthenticator`
…e-client` into a single package in preparation for introduction of `lit-client`
…into LIT-4211-auth-refactor
… moving and code refactors
…ons are extensible on machine construction and referenceable in machine definition
… add this if we want for the team
…is updated after import
…e using both `Jest` and `Bun` as they are automatically imported in the global scope. However, a Jest polyfill is required in the `jest.setup.js` because the global `Request` API (part of the Fetch API) is not available (or not properly configured?)
…rts, gonna use vNaga code as reference
…lit-network-components Feat/lit chain manager and lit network components
…ecessary-packages
…cessary-packages Feature/lit 4238 clean unnecessary packages
…a-revamp-fix-import-paths-issues-for-builds Feature/lit 4373 js sdk naga revamp fix import paths issues for builds
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR is a major refactor for v8 sdk onwards; some packages have been combined, others removed, and new APIs are introduced in this release to simplify interaction with the SDK and eliminate existing codebase-level technical debt across the SDK.
Major points
lit-client
instead of a lit node client directly, and the lit-client will use an internal lit node client and/or lit chain client as neededauth
packageauth
packageAuthProviders
toAuthenticators
for clarityauth-browser
has been moved intoauth
as a single, metamask-specific authenticatornodejs
version of thelit-node-client
due to migration of embedded metamask-specific logic inlit-node-client
to an explicit authenticatorType of change
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist: