Better source maps (#567)

* Add a new local server task

* Fix sourcemap

Author: nettofarah

Makefile

@@ -35,4 +35,10 @@ build-and-upload: build
 
 build-and-publish: build
 	make publish-assets
-.PHONY: build-and-publish
+.PHONY: build-and-publish
+
+local-server:
+	yarn build
+	node ./scripts/build-shells.js
+	npx serve
+.PHONY: local-server

scripts/build-shells.js

@@ -22,13 +22,14 @@ files.forEach(file => {
   const libraryName = last(libraryPath.split('/')).replace('.js', '');
   const vendorPath = last(vendor[0].split('build'));
 
-  const dynamic = `
-window['${libraryName}Deps'] = ["${vendorPath}"]
+  // We need this to remain a single line in order to not break source maps,
+  // as well as put the bracket at the very end, this is so we don't change the line numbers
+  // for the dynamic file
 
-window['${libraryName}Loader'] = function() {
-  return ${fs.readFileSync(file).toString()}
-}
-`.trim();
+  // prettier-ignore
+  const dynamic = `
+  window['${libraryName}Deps'] = ["${vendorPath}"];window['${libraryName}Loader'] = function() { return ${fs.readFileSync(file).toString()}
+};`.trim();
 
   const filename = file.replace(/\.js$/, '.dynamic.js');
   fs.writeFileSync(filename + '.gz', zlib.gzipSync(dynamic));