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

side-channel native replacement #123

Open
justinfagnani opened this issue Jul 27, 2024 · 0 comments
Open

side-channel native replacement #123

justinfagnani opened this issue Jul 27, 2024 · 0 comments

Comments

@justinfagnani
Copy link
Contributor

side-channel can be replaced by a WeakMap - I think.

There aren't any docs, so I'm a bit confused how this package has 1400 dependents. How do they know what they're using? It seems like the vast majority of dependents are create-next-app packages that don't directly depend on this in their source, but do in the published package. SO that must be something about create-next-app or Next.js's build system.

But the usages I did see mostly look like they could easily be replaced by a WeakMap. The side-channel tests do have some other APIs, like assert(), but those can probably be one-liners as well.

It has a pretty large dependency tree: https://deptree.rschristian.dev/?q=side-channel

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant