Fix code being undefined in browserfs

Author: CompuIves

packages/app/config/build.js

@@ -29,6 +29,13 @@ const staticAssets = [
     from: 'packages/app/static',
     to: 'static',
   },
+  {
+    from: isDev
+      ? 'standalone-packages/codesandbox-browserfs/build'
+      : 'standalone-packages/codesandbox-browserfs/dist',
+    to: 'static/browserfs6',
+  },
+  // For caching purposes
   {
     from: isDev
       ? 'standalone-packages/codesandbox-browserfs/build'

packages/app/src/app/components/CodeEditor/Monaco/workers/linter/index.js

@@ -3,7 +3,7 @@ import Linter from 'eslint/lib/linter';
 import monkeypatch from './monkeypatch-babel-eslint';
 
 self.importScripts(
-  `${process.env.CODESANDBOX_HOST}/static/browserfs5/browserfs.min.js`
+  `${process.env.CODESANDBOX_HOST}/static/browserfs6/browserfs.min.js`
 );
 
 /* eslint-disable global-require */

packages/app/src/app/index.html

@@ -48,7 +48,7 @@
 	</script>
 	<!-- End Google Tag Manager -->
 	<!-- <script async src="//cdn.headwayapp.co/widget.js"></script> -->
-	<script src="<%= webpackConfig.output.publicPath %>static/browserfs5/browserfs<%=process.env.NODE_ENV
+	<script src="<%= webpackConfig.output.publicPath %>static/browserfs6/browserfs<%=process.env.NODE_ENV
       === 'development' ? '' : '.min'%>.js"
 	 type="text/javascript"></script>
 

packages/app/src/app/vscode/extensionHostWorker/common/global.ts

@@ -20,7 +20,7 @@ export const initializePolyfills = () => {
 
 export const loadBrowserFS = () => {
   ctx.importScripts(
-    `${process.env.CODESANDBOX_HOST}/static/browserfs5/browserfs.js`
+    `${process.env.CODESANDBOX_HOST}/static/browserfs6/browserfs.min.js`
   );
 };
 

packages/app/src/embed/index.html

@@ -38,7 +38,7 @@
     </script>
     <!-- End Google Tag Manager -->
 
-    <script src="<%= webpackConfig.output.publicPath %>static/browserfs5/browserfs<%=process.env.NODE_ENV
+    <script src="<%= webpackConfig.output.publicPath %>static/browserfs6/browserfs<%=process.env.NODE_ENV
       === 'development' ? '' : '.min'%>.js"
       type="text/javascript"></script>
 

packages/app/src/sandbox/eval/transpilers/babel/worker/index.js

@@ -3,7 +3,7 @@ import loadPolyfills from '@codesandbox/common/lib/load-dynamic-polyfills';
 require('app/config/polyfills');
 
 self.importScripts(
-  `${process.env.CODESANDBOX_HOST}/static/browserfs5/browserfs.min.js`
+  `${process.env.CODESANDBOX_HOST}/static/browserfs6/browserfs.min.js`
 );
 
 self.process = self.BrowserFS.BFSRequire('process');

packages/app/src/sandbox/eval/transpilers/sass/worker/index.js

@@ -1,5 +1,5 @@
 self.importScripts(
-  `${process.env.CODESANDBOX_HOST}/static/browserfs5/browserfs.min.js`
+  `${process.env.CODESANDBOX_HOST}/static/browserfs6/browserfs.min.js`
 );
 
 self.process = self.BrowserFS.BFSRequire('process');

packages/app/src/sandbox/index.html

@@ -7,7 +7,7 @@
     <title>Sandbox - CodeSandbox</title>
     <link rel="manifest" href="/manifest.json">
     <link rel="shortcut icon" href="/favicon.ico">
-    <script src="<%= webpackConfig.output.publicPath %>static/browserfs5/browserfs.min.js"
+    <script src="<%= webpackConfig.output.publicPath %>static/browserfs6/browserfs.min.js"
       type="text/javascript"></script>
 
     <script>

standalone-packages/codesandbox-browserfs/src/backend/CodeSandboxEditorFS.ts

@@ -62,9 +62,9 @@ export interface IManager {
 
 const warn = console.warn;
 
-function getCode(savedCode: string | null, code: string | undefined) {
+function getCode(savedCode: string | null | undefined, code: string | undefined) {
   if (savedCode !== null) {
-    return savedCode;
+    return savedCode || '';
   }
 
   return code || '';

standalone-packages/monaco-typescript/src/tsWorker.ts

@@ -40,7 +40,7 @@ declare global {
 // @ts-ignore
 const oldamd = self.define.amd;
 (self as any).define.amd = null;
-(self as any).importScripts(`/static/browserfs5/browserfs.min.js`);
+(self as any).importScripts(`/static/browserfs6/browserfs.min.js`);
 (self as any).define.amd = oldamd;
 
 (self as any).BrowserFS = BrowserFS;