Merge pull request #1239 from opencomponents/cli-facade-ts

move to TS most of the cli/facade files

Author: ricardo-devis-agullo

src/cli/domain/watch.js renamed to src/cli/domain/watch.ts

@@ -1,18 +1,20 @@
-'use strict';
+import path from 'path';
+import chokidar from 'chokidar';
 
-const path = require('path');
-const chokidar = require('chokidar');
+import settings from '../../resources/settings';
 
-const settings = require('../../resources/settings').default;
-
-module.exports = function(dirs, baseDir, changed) {
+export default function watch(
+  dirs: string[],
+  baseDir: string,
+  changed: (err: Error | null, fileName: string, componentDir?: string) => void
+): void {
   const watcher = chokidar.watch(path.resolve(baseDir), {
     ignored: settings.filesToIgnoreOnDevWatch,
     persistent: true,
     ignoreInitial: true,
     usePolling: false
   });
-  const onChange = fileName => {
+  const onChange = (fileName: string) => {
     const componentDir = dirs.find(dir =>
       Boolean(fileName.match(escapeRegularExpression(dir + path.sep)))
     );
@@ -24,8 +26,8 @@ module.exports = function(dirs, baseDir, changed) {
     .on('change', onChange)
     .on('unlink', onChange)
     .on('error', changed);
-};
+}
 
-function escapeRegularExpression(text) {
+function escapeRegularExpression(text: string) {
   return text.replace(/[-[\]{}()*+?.,\\^$|#\s]/g, '\\$&');
 }

src/cli/facade/clean.js renamed to src/cli/facade/clean.ts

@@ -1,12 +1,17 @@
-'use strict';
+import read from 'read';
+import strings from '../../resources/index';
+import { Local } from '../../types';
+import { Logger } from '../logger';
 
-const read = require('read');
-
-const strings = require('../../resources/index').default;
-
-module.exports = function(dependencies) {
-  const { local, logger } = dependencies;
-  const { fetchList, remove } = local.clean;
+const clean = ({
+  local: {
+    clean: { fetchList, remove }
+  },
+  logger
+}: {
+  local: Local;
+  logger: Logger;
+}) => {
   const {
     cleanAlreadyClean,
     cleanList,
@@ -15,7 +20,7 @@ module.exports = function(dependencies) {
     cleanSuccess
   } = strings.messages.cli;
 
-  const prompt = cb =>
+  const prompt = (cb: (proceed: boolean) => void) =>
     read(
       { prompt: cleanPrompt, default: cleanPromptDefault },
       (err, result) => {
@@ -26,21 +31,24 @@ module.exports = function(dependencies) {
       }
     );
 
-  const removeFolders = (list, cb) =>
+  const removeFolders = (list: string[], cb: (err?: unknown) => void) =>
     remove(list, err => {
       if (err) {
-        logger.err(strings.errors.cli.cleanRemoveError(err));
+        logger.err(strings.errors.cli.cleanRemoveError(String(err)));
         return cb(err);
       }
 
       logger.ok(cleanSuccess);
       cb();
     });
 
-  return function(opts, callback) {
+  return (
+    opts: { dirPath: string; yes: boolean },
+    callback: (err?: unknown) => void
+  ) => {
     fetchList(opts.dirPath, (err, list) => {
       if (err) {
-        logger.err(strings.errors.generic(err));
+        logger.err(strings.errors.generic(String(err)));
         return callback(err);
       }
 
@@ -63,3 +71,7 @@ module.exports = function(dependencies) {
     });
   };
 };
+
+export default clean;
+
+module.exports = clean;