mythix 2.8.2 → 2.8.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mythix",
-  "version": "2.8.2",
+  "version": "2.8.4",
   "description": "Mythix is a NodeJS web-app framework",
   "main": "src/index",
   "scripts": {

package/src/application.js

@@ -235,14 +235,18 @@ class Application extends EventEmitter {
   }
 
   loadConfig(configPath) {
-    try {
-      let appOptions = this.getOptions();
+    let appOptions  = this.getOptions();
+    let environment = (appOptions.environment || process.NODE_ENV || 'development');
+
+    if (!configPath)
+      return wrapConfig(Object.assign({}, { environment }));
 
+    try {
       let config = require(configPath);
       if (config.__esModule)
         config = config['default'];
 
-      return wrapConfig(Object.assign({}, config || {}, { environment: (appOptions.environment || config.environment || 'development')}));
+      return wrapConfig(Object.assign({}, config || {}, { environment }));
     } catch (error) {
       this.getLogger().error(`Error while trying to load application configuration ${configPath}: `, error);
       throw error;

package/src/cli/cli-utils.js

@@ -324,6 +324,9 @@ function loadCommand(name) {
 
 function getCommandFiles(commandsPath, filterFunc) {
   try {
+    if (!commandsPath)
+      return [];
+
     return walkDir(commandsPath, {
       filter: (fullFileName, fileName, stats) => {
         if (typeof filterFunc === 'function')
@@ -390,7 +393,7 @@ function resolveConfig(config) {
 
 function loadMythixConfig(_mythixConfigPath, _appRootPath) {
   let mythixConfigPath = _mythixConfigPath;
-  let configPath        = mythixConfigPath;
+  let configPath       = mythixConfigPath;
 
   try {
     let stats = FileSystem.statSync(mythixConfigPath);

package/src/cli/deploy-command.js

@@ -288,6 +288,10 @@ module.exports = defineCommand('deploy', ({ Parent }) => {
       }
     }
 
+    async postCloneProject() {
+
+    }
+
     async copyFile(dryRun, sourceFilePath, targetFilePath) {
       let targetDir = Path.dirname(targetFilePath);
       if (!FileSystem.existsSync(targetDir))
@@ -361,6 +365,7 @@ module.exports = defineCommand('deploy', ({ Parent }) => {
 
     async archiveProject(deployConfig) {
       let {
+        dryRun,
         tempLocation,
         version,
       } = deployConfig;
@@ -368,7 +373,7 @@ module.exports = defineCommand('deploy', ({ Parent }) => {
       let archiveLocation = deployConfig.archiveLocation = Path.join(tempLocation, `${version}.tar.gz`);
 
       await this.spawnCommand(
-        false,
+        dryRun,
         'tar',
         [
           '-czf',
@@ -653,26 +658,28 @@ module.exports = defineCommand('deploy', ({ Parent }) => {
         relativeConfigPath,
       } = deployConfig;
 
-      let deployLocation    = this.joinUnixPath(decodeURIComponent(target.pathname), '' + deployConfig.version);
-      let targetConfigPath  = this.joinUnixPath(decodeURIComponent(target.pathname), 'shared', 'config');
-      let sourceConfigPath  = this.joinUnixPath(deployLocation, relativeConfigPath);
-      let nodeModulesLink   = this.joinUnixPath(targetConfigPath, 'node_modules');
+      if (relativeConfigPath) {
+        let deployLocation    = this.joinUnixPath(decodeURIComponent(target.pathname), '' + deployConfig.version);
+        let targetConfigPath  = this.joinUnixPath(decodeURIComponent(target.pathname), 'shared', 'config');
+        let sourceConfigPath  = this.joinUnixPath(deployLocation, relativeConfigPath);
+        let nodeModulesLink   = this.joinUnixPath(targetConfigPath, 'node_modules');
 
-      // Ensure shared/config exists
-      await this.executeRemoteCommands(target, deployConfig, [
-        { command: 'test', args: [ '!', '-d', `"${targetConfigPath}"`, '&&', `${this.sudo(deployConfig)}cp -a "${sourceConfigPath}" "${targetConfigPath}"`, '|| true' ] },
-      ]);
+        // Ensure shared/config exists
+        await this.executeRemoteCommands(target, deployConfig, [
+          { command: 'test', args: [ '!', '-d', `"${targetConfigPath}"`, '&&', `${this.sudo(deployConfig)}cp -a "${sourceConfigPath}" "${targetConfigPath}"`, '|| true' ] },
+        ]);
 
-      // Ensure shared/config/node_modules symlink exists
-      await this.executeRemoteCommands(target, deployConfig, [
-        { command: 'test', args: [ '!', '-e', `"${nodeModulesLink}"`, '&&', `{ cd "${targetConfigPath}"; ${this.sudo(deployConfig)}ln -s "../../current/node_modules" "node_modules"; }`, '|| true' ] },
-      ]);
+        // Ensure shared/config/node_modules symlink exists
+        await this.executeRemoteCommands(target, deployConfig, [
+          { command: 'test', args: [ '!', '-e', `"${nodeModulesLink}"`, '&&', `{ cd "${targetConfigPath}"; ${this.sudo(deployConfig)}ln -s "../../current/node_modules" "node_modules"; }`, '|| true' ] },
+        ]);
 
-      // Remove app/config and symlink to ../shared/app/config
-      await this.executeRemoteCommands(target, deployConfig, [
-        { command: 'rm', args: [ '-fr', `"${sourceConfigPath}"` ] },
-        { command: 'ln', args: [ '-s', `"${targetConfigPath}"`, `"${sourceConfigPath}"` ] },
-      ]);
+        // Remove app/config and symlink to ../shared/app/config
+        await this.executeRemoteCommands(target, deployConfig, [
+          { command: 'rm', args: [ '-fr', `"${sourceConfigPath}"` ] },
+          { command: 'ln', args: [ '-s', `"${targetConfigPath}"`, `"${sourceConfigPath}"` ] },
+        ]);
+      }
 
       await this.installModulesForApp(target, deployConfig);
     }
@@ -692,7 +699,7 @@ module.exports = defineCommand('deploy', ({ Parent }) => {
         { command: 'ls', args: [ '-1', `"${deployLocation}"`, '|', 'grep', '-P', '"\\d+"' ] },
       ]);
 
-      let versions          = result.stdout.split(/\n+/g).map((part) => part.trim()).filter(Boolean).sort();
+      let versions          = (result.stdout || '').split(/\n+/g).map((part) => part.trim()).filter(Boolean).sort();
       let versionsToRemove  = versions.slice(0, -LATEST_DEPLOY_VERSIONS_TO_KEEP);
 
       if (Nife.isNotEmpty(versionsToRemove)) {
@@ -809,7 +816,8 @@ module.exports = defineCommand('deploy', ({ Parent }) => {
         if (!deployConfig.rootPath)
           deployConfig.rootPath = Path.dirname(configPath);
 
-        deployConfig.relativeConfigPath = appOptions.configPath.substring(deployConfig.rootPath.length).replace(/^[/\\.]+/, '').replace(/[/\\]+$/, '');
+        if (appOptions.configPath)
+          deployConfig.relativeConfigPath = appOptions.configPath.substring(deployConfig.rootPath.length).replace(/^[/\\.]+/, '').replace(/[/\\]+$/, '');
 
         let git = deployConfig.git || {};
         if (git.useGitIgnore) {
@@ -894,7 +902,7 @@ module.exports = defineCommand('deploy', ({ Parent }) => {
 
       console.log(`-------- Deploy ${deployConfig.version} --------`);
       console.log(`  Application name: ${appName}`);
-      console.log(`  Application config location: ./${deployConfig.relativeConfigPath}`);
+      console.log(`  Application config location: ${(deployConfig.relativeConfigPath) ? `./${deployConfig.relativeConfigPath}` : '<none>'}`);
       console.log(`  Temporary file location: ${deployConfig.tempLocation}`);
       console.log(`  Project file location: ${deployConfig.rootPath}`);
       console.log('');
@@ -916,8 +924,8 @@ module.exports = defineCommand('deploy', ({ Parent }) => {
       console.log('  Command log:');
 
       try {
-        this.mkdirSync(dryRun, tempLocation);
-        this.mkdirSync(dryRun, Path.join(tempLocation, deployConfig.version));
+        this.mkdirSync(false, tempLocation);
+        this.mkdirSync(false, Path.join(tempLocation, deployConfig.version));
 
         if (Nife.isNotEmpty(branch) && Nife.isEmpty(repository)) {
           repository = await this.getRepositoryURL(remoteName);
@@ -926,6 +934,7 @@ module.exports = defineCommand('deploy', ({ Parent }) => {
         }
 
         await this.cloneProject(deployConfig);
+        await this.postCloneProject(deployConfig);
         await this.copyProjectFilesToDeployFolder(deployConfig);
         await this.prepProjectPreDeploy(deployConfig);
         await this.archiveProject(deployConfig);

package/src/cli/migrate-command.js

@@ -59,16 +59,17 @@ module.exports = defineCommand('migrate', ({ Parent }) => {
       let revision = ('' + _revision);
 
       let index = migrationFiles.findIndex((fullFileName) => {
-        let fileName = Path.basename(fullFileName);
+        let fileName        = Path.basename(fullFileName);
+        let migrationFileTS = fileName.substring(0, TIMESTAMP_LENGTH);
 
-        if (fileName.substring(0, TIMESTAMP_LENGTH) === revision)
+        if (BigInt(revision) >= BigInt(migrationFileTS))
           return true;
 
         return false;
       });
 
       if (index < 0)
-        throw new Error(`Error, migration revision ${revision} not found. Aborting...`);
+        return migrationFiles.slice();
 
       return migrationFiles.slice((isRollback) ? index : index + 1);
     }