Refactor grefplus to use ES modules and update alias paths

Author: begin-again

src/alias.sh

@@ -11,7 +11,7 @@ alias ll='ls -ltrh'
 ## tools
 alias resource='source $HOME/.bashrc'
 alias clean='node $DEVROOT/dev-tools/src/clean/index.js'
-alias grefp='node $DEVROOT/dev-tools/src/grefplus'
+alias grefp='node $DEVROOT/dev-tools/src/grefplus/index.mjs'
 alias branches='node $DEVROOT/dev-tools/src/branches'
 alias nt='node $DEVROOT/dev-tools/src/node-tools'
 alias yn='node $DEVROOT/dev-tools/src/yarn'

src/grefplus/cmdline.js renamed to src/grefplus/cmdline.mjs

@@ -1,6 +1,6 @@
-const { accessSync, constants, lstatSync } = require('fs');
-const yargs = require('yargs');
-const { DateTime } = require('luxon');
+import { accessSync, constants, lstatSync } from 'node:fs';
+import yargs from 'yargs/yargs';
+import { DateTime } from 'luxon';
 
 const options = {
     dateOptions: 'yyyy-MM-dd hh:mm:ss a'
@@ -32,7 +32,7 @@ const cmdKeys = {
         alias: 'r'
         , describe: 'root folder of development environment (/c/blah/blah). Default is DEVROOT'
         , type: 'array'
-        // eslint-disable-next-line no-process-env
+
         , default: process.env.DEVROOT
     }
     , 'date': {
@@ -46,13 +46,14 @@ const cmdKeys = {
 /**
  * Validates from and to dates,
  *
- * @param {String} checkDate
+ * @param {string} checkDate
+ * @param {string} msg
  * @throws if date is not valid
  * @private
  */
 const validateDate = (checkDate, msg) => {
     try {
-        const parsed = DateTime.fromFormat(checkDate, options.allowedFormat);
+        const parsed = DateTime.fromFormat(checkDate.toString(), options.allowedFormat);
         if(!parsed.isValid) {
             throw new Error('unknown format');
         }
@@ -69,11 +70,11 @@ const validateDate = (checkDate, msg) => {
 /**
  * Aborts build if dev root path does not exist
  *
- * @param {String} devRoot
+ * @param {string[]} devRoot
  * @throws if path not accessible
  * @private
  */
-const validatePath = ({ devRoot }) => {
+const validatePath = (devRoot) => {
     let problematicRoot = null;
 
     devRoot.forEach(root => {
@@ -111,21 +112,23 @@ const isFuture = (checkDate) => {
 /**
  * Parse command line and configures options
  *
- * @param {Boolean} test - used for testing only
+ * @param {Boolean} [test] - used for testing only
  */
 const setOptions = (test) => {
-    const argv = test || yargs
+    const argv = test || yargs(process.argv.slice(2))
+        // @ts-ignore
         .options(cmdKeys)
         .version(false)
         .help(true)
         .strict(true)
         .check((_argv) => {
             // super secret shortcut
             if(!_argv.date && !_argv.fromDate && !_argv.toDate && Number.isInteger(_argv._[0])) {
+                const days = Number(_argv._[0]);
                 const _date = DateTime
-                    .fromFormat(_argv._[0], options.allowedFormat)
-                    .plus({ days: _argv._[0] })
-                    .format('MM/DD/YY');
+                    .fromFormat(days.toString(), options.allowedFormat)
+                    .plus({ days })
+                    .toFormat('MM/DD/YY');
                 _argv.date = _date;
             }
             else {
@@ -138,53 +141,66 @@ const setOptions = (test) => {
             }
             return true;
         })
-        .check(({ date }) => {
-            if(date) {
-                validateDate(date, '--date');
-                if(isFuture(date)) {
+        .check((argv) => {
+            /** @type {string} */
+            const dt = `${argv.date}` || '';
+            if(dt) {
+                validateDate(dt, '--date');
+                if(isFuture(dt)) {
                     throw new Error('--date cannot exceed current date');
                 }
             }
             return true;
         })
-        .check(({ fromDate }) => {
-            if(fromDate) {
-                validateDate(fromDate, '--from-date');
-                if(isFuture(fromDate)) {
+        .check((argv) => {
+            /** @type {string} */
+            const dt = `${argv.fromDate}` || '';
+            if(dt) {
+                validateDate(dt, '--from-date');
+                if(isFuture(dt)) {
                     throw new Error('--from-date cannot exceed current date');
                 }
             }
             return true;
         })
-        .check(({ toDate }) => {
-            if(toDate) {
-                validateDate(toDate, '--to-date');
+        .check((argv) => {
+            /** @type {string} */
+            const dt = `${argv.toDate}` || '';
+            if(dt) {
+                validateDate(dt, '--to-date');
             }
             return true;
         })
-        .check(({ folderNames }) => {
+        .check((argv) => {
+            const folderNames = Array.isArray(argv.folderNames) ? argv.folderNames : [];
             if(folderNames && !folderNames.length) {
                 throw new Error('--folder-names requires at least one name');
             }
             return true;
         })
-        .check(validatePath)
+        .check(argv => {
+            const devRoot = `${argv.devRoot}`;
+            return validatePath(devRoot);
+        })
         .argv;
 
+    // @ts-ignore
     if(argv.date) {
+        // @ts-ignore
         const date = DateTime.fromFormat(argv.date, options.allowedFormat);
-        module.exports.options.fromDate = date.startOf('day');
-        module.exports.options.toDate = date.endOf('day');
+        options.fromDate = date.startOf('day');
+        options.toDate = date.endOf('day');
     }
     else {
-        module.exports.options.fromDate = argv.fromDate ? DateTime.fromFormat(argv.fromDate, options.allowedFormat).startOf('day') : null;
-        module.exports.options.toDate = argv.toDate ? DateTime.fromFormat(argv.toDate, options.allowedFormat).endOf('day') : null;
+        // @ts-ignore
+        options.fromDate = argv.fromDate ? DateTime.fromFormat(argv.fromDate, options.allowedFormat).startOf('day') : null;
+        // @ts-ignore
+        options.toDate = argv.toDate ? DateTime.fromFormat(argv.toDate, options.allowedFormat).endOf('day') : null;
     }
-    module.exports.options.devRoot = argv.devRoot;
-    module.exports.options.folderNames = argv.folderNames || [];
+    // @ts-ignore
+    options.devRoot = argv.devRoot;
+    // @ts-ignore
+    options.folderNames = argv.folderNames || [];
 };
 
-module.exports = {
-    options
-    , setOptions
-};
+export { options, setOptions };

src/grefplus/index.js renamed to src/grefplus/index.mjs

@@ -1,13 +1,20 @@
 /* eslint no-console:off */
-require('./cmdline').setOptions();
-const { basename } = require('path');
-const { allRepoPaths } = require('../common/repos');
-const { promisify } = require('util');
-const _exec = promisify(require('child_process').exec);
-const { DateTime } = require('luxon');
-const { options } = require('./cmdline');
+
+import { setOptions, options } from './cmdline.mjs';
+import { basename } from 'node:path';
+import { promisify } from 'node:util';
+import { exec } from 'node:child_process';
+const _exec = promisify(exec);
+
+import { DateTime } from 'luxon';
+
+import repos from '../common/repos.js';
+const { allRepoPaths } = repos;
+
 const DateLength = 6;
 
+setOptions();
+
 /**
  * Creates command string
  * @param  {String} repo - repository base name