Live fixes (codesandbox#2405)

* Fix selection issue

* Fix updated messages for modules and directories

* Fix moving directories and refactor a bit

Author: christianalfoni

packages/app/src/app/overmind/effects/live/index.ts

@@ -149,17 +149,18 @@ export default {
       }
     });
   },
-  sendModuleUpdate(moduleShortid: string, module?: Module) {
-    return this.send('module:saved', {
+  sendModuleUpdate(module: Module) {
+    return this.send('module:updated', {
       type: 'module',
-      moduleShortid,
+      moduleShortid: module.shortid,
       module,
     });
   },
-  sendDirectoryUpdate(directoryShortid: string) {
-    return this.send('module:saved', {
-      type: 'module',
-      directoryShortid,
+  sendDirectoryUpdate(directory: Directory) {
+    return this.send('directory:updated', {
+      type: 'directory',
+      directoryShortid: directory.shortid,
+      module: directory,
     });
   },
   sendCodeUpdate(moduleShortid: string, currentCode: string, code: string) {

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

@@ -566,7 +566,7 @@ export const renameModule: AsyncAction<{
       );
 
       if (state.live.isCurrentEditor) {
-        effects.live.sendModuleUpdate(moduleShortid);
+        effects.live.sendModuleUpdate(module);
       }
     } catch (error) {
       module.title = oldTitle;

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

@@ -103,7 +103,7 @@ export const saveCode: AsyncAction<{
     }
 
     if (state.live.isLive && state.live.isCurrentEditor) {
-      effects.live.sendModuleUpdate(module.shortid);
+      effects.live.sendModuleUpdate(module);
     }
 
     await actions.editor.internal.updateCurrentTemplate();

packages/app/src/app/overmind/namespaces/files/actions.ts

@@ -31,7 +31,7 @@ export const moduleRenamed: AsyncAction<{
       await effects.api.saveModuleTitle(sandbox.id, moduleShortid, title);
 
       if (state.live.isCurrentEditor) {
-        effects.live.sendModuleUpdate(moduleShortid, module);
+        effects.live.sendModuleUpdate(module);
       }
     } catch (error) {
       module.title = oldTitle;
@@ -104,7 +104,7 @@ export const moduleMovedToDirectory: AsyncAction<{
         moduleShortid,
         directoryShortid
       );
-      effects.live.sendModuleUpdate(moduleShortid);
+      effects.live.sendModuleUpdate(module);
     } catch (error) {
       module.directoryShortid = currentDirectoryShortid;
 
@@ -131,7 +131,7 @@ export const directoryMovedToDirectory: AsyncAction<{
         shortid,
         directoryShortid
       );
-      effects.live.sendDirectoryUpdate(directoryShortid);
+      effects.live.sendDirectoryUpdate(directoryToMove);
     } catch (error) {
       directoryToMove.directoryShortid = shortid;
       effects.notificationToast.error('Could not save new directory location');
@@ -171,7 +171,7 @@ export const directoryRenamed: AsyncAction<{
   title: string;
   directoryShortid: string;
 }> = withOwnedSandbox(
-  async ({ actions, effects, state }, { title, directoryShortid }) => {
+  async ({ effects, state }, { title, directoryShortid }) => {
     const sandbox = state.editor.currentSandbox;
     const directory = sandbox.directories.find(
       directoryEntry => directoryEntry.shortid === directoryShortid
@@ -184,7 +184,7 @@ export const directoryRenamed: AsyncAction<{
       await effects.api.saveDirectoryTitle(sandbox.id, directoryShortid, title);
 
       if (state.live.isCurrentEditor) {
-        effects.live.sendDirectoryUpdate(directoryShortid);
+        effects.live.sendDirectoryUpdate(directory);
       }
     } catch (error) {
       directory.title = oldTitle;