Typescriptify the main process code (11/N)

Summary: Converted static/compilePlugins.js to typescript Reviewed By: passy Differential Revision: D20070227 fbshipit-source-id: ba360b944440babedef94af0d0bc577711165ff6
2020-02-27 05:28:02 -08:00
parent 2d551f6b4a
commit aff433c8ad
4 changed files with 147 additions and 107 deletions
--- a/scripts/build-utils.ts
+++ b/scripts/build-utils.ts
@@ -8,7 +8,7 @@
 */

 const Metro = require('../static/node_modules/metro');
-const compilePlugins = require('../static/compilePlugins');
+import compilePlugins from '../static/compilePlugins';
 import util from 'util';
 import tmp from 'tmp';
 import path from 'path';
@@ -48,7 +48,7 @@ export function compileDefaultPlugins(
    defaultPluginDir,
    {force: true, failSilently: false, recompileOnChanges: false},
  )
-    .then((defaultPlugins: any[]) =>
+    .then(defaultPlugins =>
      fs.writeFileSync(
        path.join(defaultPluginDir, 'index.json'),
        JSON.stringify(
--- a/static/compilePlugins.ts
+++ b/static/compilePlugins.ts
@@ -8,28 +8,54 @@
 * @flow strict-local
 */

-const path = require('path');
-const fs = require('fs');
+import path from 'path';
+import fs from 'fs';
 const Metro = require('metro');
-const util = require('util');
-const recursiveReaddir = require('recursive-readdir');
-const expandTilde = require('expand-tilde');
-const pMap = require('p-map');
-const HOME_DIR = require('os').homedir();
+import util from 'util';
+import recursiveReaddir from 'recursive-readdir';
+import expandTilde from 'expand-tilde';
+import pMap from 'p-map';
+import {homedir} from 'os';
 const Watchman = require('./watchman');

-const DEFAULT_COMPILE_OPTIONS = {
+const HOME_DIR = homedir();
+
+const DEFAULT_COMPILE_OPTIONS: CompileOptions = {
  force: false,
  failSilently: true,
  recompileOnChanges: true,
 };

-module.exports = async (
-  reloadCallback,
-  pluginPaths,
-  pluginCache,
-  options = DEFAULT_COMPILE_OPTIONS,
-) => {
+export type CompileOptions = {
+  force: boolean;
+  failSilently: boolean;
+  recompileOnChanges: boolean;
+};
+
+type DynamicCompileOptions = CompileOptions & {force: boolean};
+
+export type PluginManifest = {
+  version: string;
+  name: string;
+  main?: string;
+  [key: string]: any;
+};
+
+type PluginInfo = {
+  rootDir: string;
+  name: string;
+  entry: string;
+  manifest: PluginManifest;
+};
+
+export type CompiledPluginInfo = PluginManifest & {out: string};
+
+export default async function(
+  reloadCallback: (() => void) | null,
+  pluginPaths: string[],
+  pluginCache: string,
+  options: CompileOptions = DEFAULT_COMPILE_OPTIONS,
+) {
  options = Object.assign({}, DEFAULT_COMPILE_OPTIONS, options);
  const plugins = pluginEntryPoints(pluginPaths);
  if (!fs.existsSync(pluginCache)) {
@@ -41,20 +67,27 @@ module.exports = async (
  const compilations = pMap(
    Object.values(plugins),
    plugin => {
-      const dynamicOptions = Object.assign(options, {force: false});
+      const dynamicOptions: DynamicCompileOptions = Object.assign(options, {
+        force: false,
+      });
      return compilePlugin(plugin, pluginCache, dynamicOptions);
    },
    {concurrency: 4},
  );

-  const dynamicPlugins = (await compilations).filter(c => c != null);
+  const dynamicPlugins = (await compilations).filter(
+    c => c !== null,
+  ) as CompiledPluginInfo[];
  console.log('✅  Compiled all plugins.');
  return dynamicPlugins;
-};
+}

-async function startWatchingPluginsUsingWatchman(plugins, onPluginChanged) {
+async function startWatchingPluginsUsingWatchman(
+  plugins: PluginInfo[],
+  onPluginChanged: (plugin: PluginInfo) => void,
+) {
  // Initializing a watchman for each folder containing plugins
-  const watchmanRootMap = {};
+  const watchmanRootMap: {[key: string]: any} = {};
  await Promise.all(
    plugins.map(async plugin => {
      const watchmanRoot = path.resolve(plugin.rootDir, '..');
@@ -81,24 +114,26 @@ async function startWatchingPluginsUsingWatchman(plugins, onPluginChanged) {
 }

 async function startWatchChanges(
-  plugins,
-  reloadCallback,
-  pluginCache,
-  options = DEFAULT_COMPILE_OPTIONS,
+  plugins: {[key: string]: PluginInfo},
+  reloadCallback: (() => void) | null,
+  pluginCache: string,
+  options: CompileOptions = DEFAULT_COMPILE_OPTIONS,
 ) {
  // eslint-disable-next-line no-console
  console.log('🕵️‍  Watching for plugin changes');

-  const delayedCompilation = {};
+  const delayedCompilation: {[key: string]: NodeJS.Timeout | null} = {};
  const kCompilationDelayMillis = 1000;
-  const onPluginChanged = plugin => {
+  const onPluginChanged = (plugin: PluginInfo) => {
    if (!delayedCompilation[plugin.name]) {
      delayedCompilation[plugin.name] = setTimeout(() => {
        delayedCompilation[plugin.name] = null;
        // eslint-disable-next-line no-console
        console.log(`🕵️‍  Detected changes in ${plugin.name}`);
        const watchOptions = Object.assign(options, {force: true});
-        compilePlugin(plugin, pluginCache, watchOptions).then(reloadCallback);
+        compilePlugin(plugin, pluginCache, watchOptions).then(
+          reloadCallback ?? (() => {}),
+        );
      }, kCompilationDelayMillis);
    }
  };
@@ -117,7 +152,7 @@ async function startWatchChanges(
    );
  }
 }
-function hash(string) {
+function hash(string: string) {
  let hash = 0;
  if (string.length === 0) {
    return hash;
@@ -131,7 +166,7 @@ function hash(string) {
  return hash;
 }
 const fileToIdMap = new Map();
-const createModuleIdFactory = () => filePath => {
+const createModuleIdFactory = () => (filePath: string) => {
  if (filePath === '__prelude__') {
    return 0;
  }
@@ -142,7 +177,7 @@ const createModuleIdFactory = () => filePath => {
  }
  return id;
 };
-function pluginEntryPoints(additionalPaths = []) {
+function pluginEntryPoints(additionalPaths: string[] = []) {
  const defaultPluginPath = path.join(HOME_DIR, '.flipper', 'node_modules');
  const entryPoints = entryPointForPluginFolder(defaultPluginPath);
  if (typeof additionalPaths === 'string') {
@@ -156,7 +191,7 @@ function pluginEntryPoints(additionalPaths = []) {
  });
  return entryPoints;
 }
-function entryPointForPluginFolder(pluginPath) {
+function entryPointForPluginFolder(pluginPath: string) {
  pluginPath = expandTilde(pluginPath);
  if (!fs.existsSync(pluginPath)) {
    return {};
@@ -174,13 +209,14 @@ function entryPointForPluginFolder(pluginPath) {
      } catch (e) {}
      if (packageJSON) {
        try {
-          const pkg = JSON.parse(packageJSON);
-          return {
-            packageJSON: pkg,
+          const pkg = JSON.parse(packageJSON) as PluginManifest;
+          const plugin: PluginInfo = {
+            manifest: pkg,
            name: pkg.name,
            entry: path.join(pluginPath, name, pkg.main || 'index.js'),
            rootDir: path.join(pluginPath, name),
          };
+          return plugin;
        } catch (e) {
          console.error(
            `Could not load plugin "${pluginPath}", because package.json is invalid.`,
@@ -189,39 +225,41 @@ function entryPointForPluginFolder(pluginPath) {
          return null;
        }
      }
+      return null;
    })
    .filter(Boolean)
-    .reduce((acc, cv) => {
-      acc[cv.name] = cv;
+    .reduce<{[key: string]: PluginInfo}>((acc, cv) => {
+      acc[cv!.name] = cv!;
      return acc;
    }, {});
 }
-function mostRecentlyChanged(dir) {
-  return util
-    .promisify(recursiveReaddir)(dir)
-    .then(files =>
-      files
+async function mostRecentlyChanged(dir: string) {
+  const files = await util.promisify<string, string[]>(recursiveReaddir)(dir);
+  return files
    .map(f => fs.lstatSync(f).ctime)
-        .reduce((a, b) => (a > b ? a : b), new Date(0)),
-    );
+    .reduce((a, b) => (a > b ? a : b), new Date(0));
 }
 async function compilePlugin(
-  {rootDir, name, entry, packageJSON},
-  pluginCache,
-  options,
-) {
+  pluginInfo: PluginInfo,
+  pluginCache: string,
+  options: DynamicCompileOptions,
+): Promise<CompiledPluginInfo | null> {
+  const {rootDir, manifest, entry, name} = pluginInfo;
  const isPreBundled = fs.existsSync(path.join(rootDir, 'dist'));
-  const result = Object.assign({}, packageJSON, {rootDir, name, entry});
  if (isPreBundled) {
    // eslint-disable-next-line no-console
    console.log(`🥫  Using pre-built version of ${name}...`);
-    result.out = path.resolve(rootDir, result.main);
-    return result;
-  }
-  const fileName = `${name}@${packageJSON.version || '0.0.0'}.js`;
-  const out = path.join(pluginCache, fileName);
-  result.out = out;
-  // check if plugin needs to be compiled
+    const out = path.join(
+      rootDir,
+      manifest.main ?? path.join('dist', 'index.js'),
+    );
+    return Object.assign({}, pluginInfo.manifest, {out});
+  } else {
+    const out = path.join(
+      pluginCache,
+      `${name}@${manifest.version || '0.0.0'}.js`,
+    );
+    const result = Object.assign({}, pluginInfo.manifest, {out});
    const rootDirCtime = await mostRecentlyChanged(rootDir);
    if (
      !options.force &&
@@ -240,7 +278,7 @@ async function compilePlugin(
            projectRoot: rootDir,
            watchFolders: [__dirname, rootDir],
            serializer: {
-            getRunModuleStatement: moduleID =>
+              getRunModuleStatement: (moduleID: string) =>
                `module.exports = global.__r(${moduleID}).default;`,
              createModuleIdFactory,
            },
@@ -278,3 +316,4 @@ async function compilePlugin(
      return result;
    }
  }
+}
--- a/static/icons.json
+++ b/static/icons.json
@@ -195,6 +195,7 @@
    16
  ],
  "rocket": [
+    12,
    20
  ],
  "settings": [
--- a/static/main.ts
+++ b/static/main.ts
@@ -16,7 +16,7 @@ import url from 'url';
 import fs from 'fs';
 import fixPath from 'fix-path';
 import {exec} from 'child_process';
-const compilePlugins = require('./compilePlugins');
+import compilePlugins from './compilePlugins';
 import setup from './setup';
 import delegateToLauncher from './launcher';
 import expandTilde from 'expand-tilde';
@@ -120,7 +120,7 @@ compilePlugins(
  },
  pluginPaths,
  path.join(flipperDir, 'plugins'),
-).then((dynamicPlugins: string[]) => {
+).then(dynamicPlugins => {
  ipcMain.on('get-dynamic-plugins', event => {
    event.returnValue = dynamicPlugins;
  });