code review

Author: mrsdizzie

.eslintrc

@@ -28,7 +28,7 @@ globals:
   Tribute: false
 
 overrides:
-  - files: ["web_src/**/*.worker.js", "web_src/js/serviceworker.js"]
+  - files: ["web_src/js/serviceworker.js"]
     env:
       worker: true
 

modules/highlight/highlight.go

@@ -39,6 +39,10 @@ func NewContext() {
 // Code returns a HTML version of code string with chroma syntax highlighting classes
 func Code(fileName, code string) string {
 	NewContext()
+	// don't highlight over 25kb
+	if len(code) > 25000 {
+		return plainText(string(code), numLines)
+	}
 	formatter := html.New(html.WithClasses(true),
 		html.WithLineNumbers(false),
 		html.PreventSurroundingPre(true),
@@ -80,8 +84,10 @@ func Code(fileName, code string) string {
 // File returns map with line lumbers and HTML version of code with chroma syntax highlighting classes
 func File(numLines int, fileName string, code []byte) map[int]string {
 	NewContext()
-	m := make(map[int]string, numLines)
-
+	// don't highlight over 25kb
+	if len(code) > 25000 {
+		return plainText(string(code), numLines)
+	}
 	formatter := html.New(html.WithClasses(true),
 		html.WithLineNumbers(false),
 		html.PreventSurroundingPre(true),
@@ -120,6 +126,7 @@ func File(numLines int, fileName string, code []byte) map[int]string {
 	}
 
 	htmlw.Flush()
+	m := make(map[int]string, numLines)
 	for k, v := range strings.SplitN(htmlbuf.String(), "\n", numLines) {
 		line := k + 1
 		m[line] = string(v)

package-lock.json

@@ -47,7 +47,6 @@
     "webpack-fix-style-only-entries": "0.5.0",
     "workbox-routing": "5.1.3",
     "workbox-strategies": "5.1.3",
-    "worker-loader": "2.0.0",
     "wrap-ansi": "7.0.0"
   },
   "devDependencies": {

package.json

@@ -102,20 +102,6 @@ module.exports = {
         exclude: /node_modules/,
         loader: 'vue-loader',
       },
-      {
-        test: /\.worker\.js$/,
-        exclude: /monaco/,
-        use: [
-          {
-            loader: 'worker-loader',
-            options: {
-              name: '[name].js',
-              inline: true,
-              fallback: false,
-            },
-          },
-        ],
-      },
       {
         test: /\.js$/,
         exclude: /node_modules/,