Add cache deletion

Author: CompuIves

packages/app/src/sandbox/compile.js

@@ -22,7 +22,7 @@ import {
 } from './boilerplates';
 
 import loadDependencies from './npm';
-import { consumeCache, saveCache } from './eval/cache';
+import { consumeCache, saveCache, deleteAPICache } from './eval/cache';
 import getDefinition from '../../../common/templates/index';
 
 let initializedResizeListener = false;
@@ -41,6 +41,7 @@ export function getCurrentManager(): ?Manager {
 
 let firstLoad = true;
 let hadError = false;
+let changedModuleCount = 0;
 
 const DEPENDENCY_ALIASES = {
   '@vue/cli-plugin-babel': '@vue/babel-preset-app',
@@ -251,7 +252,9 @@ async function updateManager(
   }
 
   manager.updateConfigurations(configurations);
-  return manager.updateData(managerModules);
+  return manager.updateData(managerModules).then(x => {
+    changedModuleCount = x.length;
+  });
 }
 
 function initializeResizeListener() {
@@ -535,13 +538,23 @@ async function compile({
       type: 'success',
     });
 
-    saveCache(sandboxId, managerModuleToTranspile, manager, firstLoad);
+    saveCache(
+      sandboxId,
+      managerModuleToTranspile,
+      manager,
+      changedModuleCount,
+      firstLoad
+    );
   } catch (e) {
     console.log('Error in sandbox:');
     console.error(e);
 
     if (manager) {
       manager.clearCache();
+
+      if (firstLoad && changedModuleCount === 0) {
+        deleteAPICache(manager.id);
+      }
     }
 
     if (firstLoad) {

packages/app/src/sandbox/eval/cache.js

@@ -9,6 +9,8 @@ const debug = _debug('cs:compiler:cache');
 
 const host = process.env.CODESANDBOX_HOST;
 
+let APICacheUsed = false;
+
 try {
   localforage.config({
     name: 'CodeSandboxApp',
@@ -24,8 +26,8 @@ try {
   console.warn(e);
 }
 
-function shouldSaveOnlineCache(firstRun: boolean) {
-  if (!firstRun) {
+function shouldSaveOnlineCache(firstRun: boolean, changes: number) {
+  if (!firstRun || changes > 0) {
     return false;
   }
 
@@ -40,6 +42,7 @@ export async function saveCache(
   sandboxId: string,
   managerModuleToTranspile: any,
   manager: Manager,
+  changes: number,
   firstRun: boolean
 ) {
   const managerState = {
@@ -65,7 +68,7 @@ export async function saveCache(
     manager.clearCache();
   }
 
-  if (shouldSaveOnlineCache(firstRun)) {
+  if (shouldSaveOnlineCache(firstRun, changes)) {
     const stringifiedManagerState = JSON.stringify(managerState);
 
     debug(
@@ -95,6 +98,29 @@ export async function saveCache(
   return Promise.resolve(false);
 }
 
+export function deleteAPICache(sandboxId: string) {
+  if (APICacheUsed) {
+    debug('Deleting cache of API');
+    return window
+      .fetch(`${host}/api/v1/sandboxes/${sandboxId}/cache`, {
+        method: 'DELETE',
+        body: JSON.stringify({
+          version: SCRIPT_VERSION,
+        }),
+        headers: {
+          'Content-Type': 'application/json',
+        },
+      })
+      .then(x => x.json())
+      .catch(e => {
+        console.error('Something went wrong while deleting cache.');
+        console.error(e);
+      });
+  }
+
+  return Promise.resolve(false);
+}
+
 function findCacheToUse(cache1, cache2) {
   if (!cache1 && !cache2) {
     return null;
@@ -136,6 +162,12 @@ export async function consumeCache(manager: Manager) {
       const version = SCRIPT_VERSION;
 
       if (cache.version === version) {
+        if (cache === localData) {
+          APICacheUsed = false;
+        } else {
+          APICacheUsed = true;
+        }
+
         debug(
           `Loading cache from ${cache === localData ? 'localStorage' : 'API'}`,
           cache

packages/app/src/sandbox/eval/manager.js

@@ -23,7 +23,7 @@ import TestRunner from './tests/jest-lite';
 import dependenciesToQuery from '../npm/dependencies-to-query';
 import isESModule from './utils/is-es-module';
 
-import { ignoreNextCache } from './cache';
+import { ignoreNextCache, deleteAPICache } from './cache';
 
 type Externals = {
   [name: string]: string,
@@ -829,9 +829,13 @@ export default class Manager {
     }
   }
 
+  deleteAPICache() {
+    ignoreNextCache();
+    deleteAPICache(this.id);
+  }
+
   clearCache() {
     try {
-      ignoreNextCache();
       localforage.clear();
     } catch (ex) {
       if (process.env.NODE_ENV === 'development') {

packages/app/src/sandbox/eval/transpiled-module.js

@@ -657,6 +657,7 @@ export default class TranspiledModule {
             // We're in a reload loop! Ignore all caches!
 
             manager.clearCache();
+            manager.deleteAPICache();
           }
 
           location.reload();