fix replacing "export default" in strings in user code -- fixes #34

Author: rixo

lib/make-hot.js

@@ -109,6 +109,15 @@ if (typeof add_css !== 'undefined' && !document.getElementById(${json(
 }
 `
 
+// replace from last occurrence of the splitter
+const replaceLast = (string, splitter, regex, replacer) => {
+  const lastIndex = string.lastIndexOf(splitter)
+  return (
+    string.slice(0, lastIndex) +
+    string.slice(lastIndex).replace(regex, replacer)
+  )
+}
+
 // https://github.com/darkskyapp/string-hash/blob/master/index.js
 // (via https://github.com/sveltejs/svelte/blob/91d758e35b2b2154512ddd11e6b6d9d65708a99e/src/compiler/compile/utils/hash.ts#L2)
 const stringHashcode = str => {
@@ -447,11 +456,13 @@ const createMakeHot = ({ resolveAbsoluteImport, pkg = {} }) => ({
       emitCss,
     })
 
-    return compiledCode.replace(
+    // NOTE export default can appear in strings in use code
+    // see: https://github.com/rixo/svelte-hmr/issues/34
+    return replaceLast(
+      compiledCode,
+      'export default',
       /(\n?export default ([^;]*);)/,
-      (match, $1, $2) => {
-        return replacement.replace(/\$2/g, () => $2)
-      }
+      (match, $1, $2) => replacement.replace(/\$2/g, () => $2)
     )
   }