feat(importer): ensure the user is aware of any script changes we make

electron · Jan 10, 2017 · cbb73e7 · cbb73e7
1 parent 272d9b1
commit cbb73e7
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/src/api/import.js b/src/api/import.js
@@ -111,9 +111,19 @@ export default async (providedOptions = {}) => {
   packageJSON.scripts = packageJSON.scripts || {};
   d('reading current scripts object:', packageJSON.scripts);
 
-  packageJSON.scripts.start = 'electron-forge start';
-  packageJSON.scripts.package = 'electron-forge package';
-  packageJSON.scripts.make = 'electron-forge make';
+  const updatePackageScript = async (scriptName, newValue) => {
+    if (packageJSON.scripts[scriptName] !== newValue) {
+      // eslint-disable-next-line max-len
+      const shouldUpdate = await confirmIfInteractive(interactive, `Do you want us to update the "${scriptName}" script to instead call the electron-forge task "${newValue}"`);
+      if (shouldUpdate) {
+        packageJSON.scripts[scriptName] = newValue;
+      }
+    }
+  };
+
+  await updatePackageScript('start', 'electron-forge start');
+  await updatePackageScript('package', 'electron-forge package');
+  await updatePackageScript('make', 'electron-forge make');
 
   d('forgified scripts object:', packageJSON.scripts);