merge master

Author: christianalfoni

packages/app/src/app/overmind/namespaces/editor/internalActions.ts

@@ -6,6 +6,10 @@ import {
   ServerContainerStatus,
   TabType,
 } from '@codesandbox/common/lib/types';
+import {
+  captureException,
+  logBreadcrumb,
+} from '@codesandbox/common/lib/utils/analytics/sentry';
 import slugify from '@codesandbox/common/lib/utils/slugify';
 import {
   editorUrl,
@@ -63,9 +67,17 @@ export const setModuleSavedCode: Action<{
   );
 
   if (moduleIndex > -1) {
-    state.editor.sandboxes[sandbox.id].modules[
-      moduleIndex
-    ].savedCode = savedCode;
+    const module = state.editor.sandboxes[sandbox.id].modules[moduleIndex];
+
+    if (savedCode === undefined) {
+      logBreadcrumb({
+        type: 'error',
+        message: `SETTING UNDEFINED SAVEDCODE FOR CODE: ${module.code}`,
+      });
+      captureException(new Error('SETTING UNDEFINED SAVEDCODE'));
+    }
+
+    module.savedCode = savedCode;
   }
 };
 
@@ -90,8 +102,18 @@ export const saveCode: AsyncAction<{
 
     module.insertedAt = updatedModule.insertedAt;
     module.updatedAt = updatedModule.updatedAt;
-    module.savedCode =
+
+    const savedCode =
       updatedModule.code === module.code ? null : updatedModule.code;
+    if (savedCode === undefined) {
+      logBreadcrumb({
+        type: 'error',
+        message: `SETTING UNDEFINED SAVEDCODE FOR CODE: ${updatedModule.code}`,
+      });
+      captureException(new Error('SETTING UNDEFINED SAVEDCODE'));
+    }
+
+    module.savedCode = savedCode;
 
     effects.vscode.fs.writeFile(state.editor.modulesByPath, module);
     effects.moduleRecover.remove(sandbox.id, module);