fix live updates

Author: christianalfoni

packages/app/src/app/overmind/effects/vscode/ModelsHandler.ts

@@ -30,8 +30,7 @@ export type OnFileChangeData = {
   moduleShortid: string;
   title: string;
   code: string;
-  event?: any[];
-  model?: any;
+  model: any;
 };
 
 export type OnOperationAppliedData = {
@@ -446,7 +445,6 @@ export class ModelsHandler {
           moduleShortid: module.shortid,
           title: module.title,
           code: model.getValue(),
-          event: e,
           model,
         });
       } catch (err) {

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

@@ -188,32 +188,9 @@ export const onOperationApplied: Action<{
 export const codeChanged: Action<{
   moduleShortid: string;
   code: string;
-  event?: any;
-}> = ({ effects, state, actions }, { code, moduleShortid, event }) => {
+}> = ({ effects, state, actions }, { code, moduleShortid }) => {
   effects.analytics.trackOnce('Change Code');
 
-  if (
-    state.live.isLive &&
-    moduleShortid === state.editor.currentModuleShortid &&
-    event &&
-    event.changes
-  ) {
-    try {
-      actions.live.onTransformMade({
-        moduleShortid,
-        event,
-        code: state.editor.currentModule.code,
-      });
-    } catch (e) {
-      // Something went wrong while composing the operation, so we're opting for a full sync
-      console.error(e);
-
-      if (this.props.onModuleStateMismatch) {
-        this.props.onModuleStateMismatch();
-      }
-    }
-  }
-
   const module = state.editor.currentSandbox.modules.find(
     m => m.shortid === moduleShortid
   );
@@ -223,9 +200,7 @@ export const codeChanged: Action<{
   }
 
   if (state.live.isLive) {
-    state.live.receivingCode = true;
-    effects.live.sendCodeUpdate(moduleShortid, module.code || '', code);
-    state.live.receivingCode = false;
+    effects.live.sendCodeUpdate(moduleShortid, module.code, code);
   }
 
   actions.editor.internal.setModuleCode({

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

@@ -3,7 +3,6 @@ import { Action, AsyncAction, Operator } from 'app/overmind';
 import { withLoadApp } from 'app/overmind/factories';
 import { filter, fork, pipe } from 'overmind';
 
-import eventToTransform from '../../utils/event-to-transform';
 import * as internalActions from './internalActions';
 import * as liveMessage from './liveMessageOperators';
 
@@ -68,34 +67,6 @@ export const liveMessageReceived: Operator<LiveMessage> = pipe(
   })
 );
 
-export const onTransformMade: Action<{
-  event: any;
-  moduleShortid: string;
-  code: string;
-}> = ({ effects, state }, { event, moduleShortid, code }) => {
-  if (!state.live.isCurrentEditor) {
-    return;
-  }
-
-  const { operation } = eventToTransform(event, code);
-
-  if (!operation) {
-    return;
-  }
-
-  try {
-    effects.live.applyClient(moduleShortid, operation.toJSON());
-  } catch (e) {
-    // Something went wrong, probably a sync mismatch. Request new version
-    console.error(
-      'Something went wrong with applying OT operation',
-      moduleShortid,
-      operation
-    );
-    effects.live.sendModuleState();
-  }
-};
-
 export const applyTransformation: Action<{
   operation: any;
   moduleShortid: string;