feat: Support json names containing non-alphanumeric characters #520
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.
I had a protobuf definition that needed to match an external JSON format, which contains the property
error-codes
(with hyphen):This compiled into the following invalid TypeScript:
This PR fixes that issue.
It will use array-syntax for all properties that don't follow the format
(letter /underscore / dollar) + (letter / number / underscore / $)
.Technically, ecma allows many more patterns than this, but that regex is insane, and it's not such a problem to over-use the bracket syntax, in my opinion.
This PR also includes a fix for a bug in the
watch
task. There was an error in the chokidar event handler (signature is different when watchingall
as compared tochange
.) Hope that's ok.