throw a TypeError if identifier validation fails

[dentrado]

Q	A
Fixed Issues?
Patch: Bug Fix?	Yes
Major: Breaking Change?	No
Minor: New Feature?	No
Tests Added + Pass?	No
Documentation PR Link
Any Dependency Changes?	No
License	MIT

I noticed that t.identifier(name) accepts invalid identifiers, this PR fixes that bug. Thanks to @JLHwung for finding the reason so quickly!

[buildsize]

File name	Previous Size	New Size	Change
babel-preset-env.js	2.77 MB	2.77 MB	815 bytes (0%)
babel-preset-env.min.js	1.67 MB	1.67 MB	399 bytes (0%)
babel.js	2.95 MB	2.95 MB	817 bytes (0%)
babel.min.js	1.63 MB	1.63 MB	400 bytes (0%)
test262.tap	4.84 MB	[deleted]

[buildsize]

File name	Previous Size	New Size	Change
babel-preset-env.js	2.77 MB	2.77 MB	815 bytes (0%)
babel-preset-env.min.js	1.67 MB	1.67 MB	399 bytes (0%)
babel.js	2.95 MB	2.95 MB	817 bytes (0%)
babel.min.js	1.63 MB	1.63 MB	400 bytes (0%)
test262.tap	4.84 MB	[deleted]

[jridgewell]

🙌

I've had to depend on and import esutils in a few projects because this was never exposed. Yay!

[JLHwung]

@jridgewell We do have plan to refactor it. In the near future babel-types will not depend on esutils anymore, instead it will reuse the big identifier name regex in babel-parser.

[nicolo-ribaudo]

Thanks!