-
Couldn't load subscription status.
- Fork 1.4k
Replace arrgv with args-tokenizer #3364
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
base: main
Are you sure you want to change the base?
Conversation
Recently published args-tokenizer package. We use it at Webstudio for parsing cURL on paste. Used at https://github.com/antfu-collective/bumpp. There are two benefits - published with type definitions - parser is forgiving to unclosed quotes which can be more user friendly The package also is esm only but used only in CLI so should not be a problem with commonjs. Though I can add cjs support if necessary. https://github.com/TrySound/args-tokenizer
|
I don't think silently forgiving invalid input is an improvement. |
|
Will "strict" option work for you? I can add it |
|
Upgraded to new strict by default version and reverted test changes |
|
@TrySound apologies for ignoring this for so long. What would give more confidence is if you incorporate |
To follow up on this, you're asking to replace a dependency that as far as I know hasn't caused problems with a modern version you wrote. That's great! But the less work for me to verify the behavior is exactly the same, the better. |
|
Of course! Thanks for your response. Will update PR when release 1.0 |
|
{ |
Recently published args-tokenizer package. We use it at Webstudio for parsing cURL on paste. Also used at https://github.com/antfu-collective/bumpp.
There are a few benefits
parser is forgiving to unclosed quotes which can be more user friendlyThe package also is esm only but used only in CLI so should not be a problem with commonjs. Though I can add cjs support if necessary.
https://github.com/TrySound/args-tokenizer