@tangelo/tangelo-configuration-toolkit 1.8.0-beta.2 → 1.8.0-beta.3

package/index.js

@@ -5,9 +5,10 @@ String.prototype.toFws = function(){
 };
 
 
-const findUp  = require('find-up');
-const fs      = require('fs-extra');
-const path    = require('path');
+const findUp    = require('find-up');
+const fs        = require('fs-extra');
+const {getPath} = require('global-modules-path');
+const path      = require('path');
 
 const execGitCommand = require('./src/lib/exec-git-command');
 
@@ -56,8 +57,12 @@ _paths.repoconfig = path.join(_paths.repo, appname+'-repoconfig.json');
 _paths.apply = process.cwd().replace(_paths.repo, '').substr(1);
 
 
-global._package = require('./package.json');
-global._appdata = fs.readJsonSync(_paths.appdata, {throws: false}) || {versionLastChecked: 0, versionIsLatest: true};
+global._appdata  = fs.readJsonSync(_paths.appdata, {throws: false}) || {};
+global._packages = {
+  TCT: {name: '@tangelo/tangelo-configuration-toolkit', version: require('./package.json')?.version},
+  FDT: {name: '@fontoxml/fontoxml-development-tools'}
+}
+_packages.FDT.version = require(`${getPath(_packages.FDT.name)}/package.json`)?.version;
 
 try { global._appconfig = _paths.appconfig && fs.readJsonSync(_paths.appconfig) || {}; }
 catch({message}) { _error('Error in '+message); }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tangelo/tangelo-configuration-toolkit",
-  "version": "1.8.0-beta.2",
+  "version": "1.8.0-beta.3",
   "description": "Tangelo Configuration Toolkit is a command-line toolkit which offers support for developing a Tangelo configuration.",
   "bin": {
     "tct": "bin/index.js",
@@ -25,6 +25,7 @@
     "event-stream": "^4.0.1",
     "find-up": "^5.0.0",
     "fs-extra": "^10.0.0",
+    "global-modules-path": "^2.3.1",
     "globby": "^6.1.0",
     "gulp": "^4.0.2",
     "gulp-babel": "^7.0.1",

package/src/cli.js

@@ -1,13 +1,31 @@
+const {compare} = require('compare-versions');
+const exec  = require('util').promisify(require('child_process').exec);
 const fs    = require('fs-extra');
 const yargs = require('yargs');
-const {getCurrentPackageVersion, getLatestPackageVersion, updatePackage} = require('./lib/npm-package-update');
+
+
+const updateAppdata = (data) => Object.assign(_appdata, data, {_changed: true});
+
+const checkForPackageUpdate = (package) => (
+  exec(`npm view -g ${_packages[package].name} version`)
+  .then(r => {
+    const versionAvailable = r.stdout.match(/([\d/.]+)/)[1];
+    if (compare(_packages[package].version, versionAvailable, '<')) {
+      updateAppdata({[`updateCheck${package}`]: {executed: new Date(), versionAvailable}});
+      return versionAvailable;
+    }
+    else updateAppdata({[`updateCheck${package}`]: {executed: new Date()}});
+  })
+  .catch(e => _warn(`Failed checking latest version of ${package}.`))
+);
+
 
 module.exports = function cli () {
 
   _write();
 
   const txtTitle    = 'Tangelo Configuration Toolkit'.bold.underline.cyan;
-  const txtVersion  = `v${_package.version}${_appdata.versionIsLatest ? '' : ' (new version available)'}`.lblack;
+  const txtVersion  = `v${_packages.TCT.version}`.lblack;
 
   const {argv} = yargs
   .scriptName('tct')
@@ -95,8 +113,6 @@ module.exports = function cli () {
   })
   .recommendCommands()
   .option('config', {alias: 'c', desc: 'Show loaded appconfig', global: false})
-  .option('update', {alias: 'u', desc: 'Update this package', global: false})
-  .option('update-fdt', {alias: 'uf', desc: 'Update the FDT package', global: false})
   .version(false)
   .help(false)
   .example([
@@ -123,53 +139,36 @@ module.exports = function cli () {
       _write(_appconfig);
     }
 
-    if (argv.update) {
-      _info('Updating TCT...');
-      getLatestPackageVersion({package: 'TCT', version: _package.version})
-      .then(updatePackage)
-      ;
-    }
-
-    if (argv.updateFdt) {
-      _info('Updating FDT...');
-      getCurrentPackageVersion({package: 'FDT'})
-      .then(getLatestPackageVersion)
-      .then(updatePackage)
-      ;
-    }
-
   }
 
 
   let checkUpdatesDone = false;
 
   process.on('beforeExit', () => {
-
-    if (_appdata._changed) {
-      delete _appdata._changed;
-      fs.writeJsonSync(_paths.appdata, _appdata, {spaces: 2});
-    }
-
     _write(); // print empty line before and after update check
 
-    if (!(argv.update || argv.updateFdt) && // not when doing update
-        (!_appdata.updateCheckTCT?.versionAvailable || !_appdata.updateCheckFDT?.versionAvailable) && // not if already known that update is available
-        ( new Date() - new Date(_appdata.updateCheckTCT?.executed || 0) > 1000*3600*24*3 ||
-          new Date() - new Date(_appdata.updateCheckFDT?.executed || 0) > 1000*3600*24*3
-        )  && // check every 3 days
-        !checkUpdatesDone // check if updatecheck has ran before because async calls below trigger beforeExit again
-    ) {
+    if (!checkUpdatesDone) { // check if updatecheck has ran before because async calls below trigger beforeExit again
       checkUpdatesDone = true;
-      _info('Checking for updates...');
 
-      // check tct version
-      getLatestPackageVersion({package: 'TCT', version: _package.version});
+      ['TCT', 'FDT'].forEach(p => {
+        const updateMsg = (va) => `| Update ${p} to ${va} | ` + `npm i -g ${_packages[p].name}`.white;
+        const {versionAvailable} = _appdata[`updateCheck${p}`] || {};
+
+        if (new Date() - new Date(_appdata[`updateCheck${p}`]?.executed || 0) > 1000*3600*24*7) { // check every week
+          checkForPackageUpdate(p).then(r => r && _warn(updateMsg(r)));
+        }
+        else if (versionAvailable) {
+          if (compare(_packages[p].version, versionAvailable, '<')) _warn(updateMsg(versionAvailable));
+          else updateAppdata({[`updateCheck${p}`]: {executed: new Date()}});
+        }
+      });
+    }
 
-      // check fdt version
-      getCurrentPackageVersion({package: 'FDT'})
-      .then(getLatestPackageVersion)
-      ;
+    if (_appdata._changed) {
+      delete _appdata._changed;
+      fs.writeJsonSync(_paths.appdata, _appdata, {spaces: 2});
     }
+
   });
 
 }

package/src/lib/npm-package-update.js

@@ -1,51 +0,0 @@
-const {compare} = require('compare-versions');
-const util = require('util');
-const exec = util.promisify(require('child_process').exec);
-
-
-const packageNames = {
-  TCT: '@tangelo/tangelo-configuration-toolkit',
-  FDT: '@fontoxml/fontoxml-development-tools'
-};
-
-const updateAppdata = (data) => Object.assign(_appdata, data, {_changed: true});
-
-const getCurrentPackageVersion = ({package}) => (
-  exec(`npm list -g ${packageNames[package]}`)
-  .then(r => ({package, version: r.stdout.match(/@([\d/.]+)/)[1]}))
-  .catch(e => _warn(`Failed checking current version of ${package}.`))
-);
-
-const getLatestPackageVersion = ({package, version}) => (
-  exec(`npm view -g ${packageNames[package]} version`)
-  .then(r => {
-    const versionAvailable = r.stdout.match(/([\d/.]+)/)[1];
-    if (compare(version, versionAvailable, '>=')) {
-      updateAppdata({[`updateCheck${package}`]: {executed: new Date()}});
-      _write(`Using latest version of ${package}.`);
-    }
-    else {
-      updateAppdata({[`updateCheck${package}`]: {executed: new Date(), versionAvailable}});
-      _write(`New version of ${package} available!`);
-    }
-    return {package, version, versionAvailable};
-  })
-  .catch(e => _warn(`Failed checking latest version of ${package}.`))
-);
-
-const updatePackage = ({package, version, versionAvailable}) => (
-  compare(version, versionAvailable, '<') &&
-  exec(`npm install -g ${packageNames[package]}`)
-  .then(r => {
-    updateAppdata({[`updateCheck${package}`]: {executed: new Date()}});
-    _write(`Updated ${package} to version ${versionAvailable}.`);
-  })
-  .catch(e => _warn(`Failed updating latest version of ${package}.`))
-);
-
-
-module.exports = {
-  getCurrentPackageVersion,
-  getLatestPackageVersion,
-  updatePackage
-};