Fix deleting module and directory (codesandbox#2403)

Author: christianalfoni

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

@@ -1,10 +1,10 @@
-import { Action, AsyncAction } from '.';
-import * as internalActions from './internalActions';
-import { withLoadApp } from './factories';
 import {
   NotificationType,
   convertTypeToStatus,
 } from '@codesandbox/common/lib/utils/notifications';
+import { Action, AsyncAction } from '.';
+import * as internalActions from './internalActions';
+import { withLoadApp } from './factories';
 
 export const internal = internalActions;
 

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

@@ -225,10 +225,9 @@ export const addedFileToSandbox: AsyncAction<{
   });
 };
 
-export const deletedUploadedFile: AsyncAction<string> = async (
-  { state, effects },
-  id
-) => {
+export const deletedUploadedFile: AsyncAction<{
+  id: string;
+}> = async ({ state, effects }, { id }) => {
   try {
     await effects.api.deleteUploadedFile(id);
     state.uploadedFiles = null;
@@ -481,8 +480,10 @@ export const syncSandbox: AsyncAction<any[]> = async (
   }
 };
 
-export const removeModule: AsyncAction<string> = withOwnedSandbox(
-  async ({ state, effects, actions }, moduleShortid) => {
+export const removeModule: AsyncAction<{
+  moduleShortid: string;
+}> = withOwnedSandbox(
+  async ({ state, effects, actions }, { moduleShortid }) => {
     if (state.editor.currentModule.shortid === moduleShortid) {
       actions.editor.internal.setCurrentModule(state.editor.mainModule);
     }
@@ -504,10 +505,9 @@ export const removeModule: AsyncAction<string> = withOwnedSandbox(
   }
 );
 
-export const removeDirectory: AsyncAction<string> = async (
-  { state },
-  directoryShortid
-) => {
+export const removeDirectory: AsyncAction<{
+  directoryShortid: string;
+}> = async ({ state }, { directoryShortid }) => {
   const sandbox = state.editor.currentSandbox;
   state.editor.currentModuleShortid = state.editor.mainModule.shortid;
   const directory = sandbox.directories.find(

packages/app/src/app/overmind/namespaces/live/liveMessageOperators.ts

@@ -177,7 +177,9 @@ export const onModuleDeleted: Operator<
     return;
   }
   // Do not think this really works? Cause this would fork the sandbox
-  actions.files.removeModule(data.moduleShortid);
+  actions.files.removeModule({
+    moduleShortid: data.moduleShortid,
+  });
 });
 
 export const onDirectoryCreated: Operator<
@@ -219,7 +221,9 @@ export const onDirectoryDeleted: Operator<
   }
   state.editor.currentModuleShortid = state.editor.mainModule.shortid;
   // Again, this does not work very well?
-  actions.files.removeDirectory(data.directoryShortid);
+  actions.files.removeDirectory({
+    directoryShortid: data.directoryShortid,
+  });
 });
 
 export const onUserSelection: Operator<