fix(build) registered language names should use - not _ (#3264)

* Fix registered language name for grammar names with dash, fixes #3263.

Author: fredrikekre

src/stub.js

@@ -5,7 +5,11 @@ import * as builtIns from "builtInLanguages";
 const hljs = HighlightJS;
 
 for (const key of Object.keys(builtIns)) {
-  const languageName = key.replace("grmr_", "");
+  // our builtInLanguages Rollup plugin has to use `_` to allow identifiers to be
+  // compatible with `export` naming conventions, so we need to convert the
+  // identifiers back into the more typical dash style that we use for language
+  // naming via the API
+  const languageName = key.replace("grmr_", "").replace("_", "-");
   hljs.registerLanguage(languageName, builtIns[key]);
 }
 // console.log(hljs.listLanguages());

test/builds/browser_build_as_commonjs.js

@@ -21,4 +21,13 @@ API.forEach(n => {
   assert(_ => keys.includes(n), `API should include ${n}`);
 });
 
+// See e.g. highlightjs/highlight.js#3263
+const langs = ["python", "python-repl"]
+langs.forEach(n => {
+  assert(_ => {
+    res = hljs.getLanguage(n);
+    return typeof res === 'object' && res !== null
+  })
+})
+
 console.log("Pass: browser build works with Node.js just fine.")