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

Add a Dart-based WebIDL parser #341

Open
ykmnkmi opened this issue Feb 17, 2025 · 3 comments
Open

Add a Dart-based WebIDL parser #341

ykmnkmi opened this issue Feb 17, 2025 · 3 comments

Comments

@ykmnkmi
Copy link

ykmnkmi commented Feb 17, 2025

Currently, web_generator relies on the @webref/* JS libraries for parsing WebIDL definitions. This introduces a dependency on JS, introduces complexities in the build process, and slows down the development process.

@kevmoo
Copy link
Member

kevmoo commented Feb 17, 2025

Any suggestions on how to improve this?

@ykmnkmi
Copy link
Author

ykmnkmi commented Feb 18, 2025

I'm trying to work on variadic arguments, and each run requires compilation. I'm looking to implement it myself. There are cons, like keeping it updated, but I don't think the IDL format and web specs change that much.

@srujzs
Copy link
Contributor

srujzs commented Feb 18, 2025

I'd be more worried we aren't handling corner cases correctly. I think reimplementing something like webidl2 is going to be quite a bit of effort and I would want it to be its own separate maintained package if we were to add it.

Note that we would still have to do something for MDN compat data even if we were to add this before we could move fully away from using any JS libraries.

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

3 participants