@form8ion/javascript 8.0.0 → 8.1.0

package/lib/index.mjs

@@ -27,6 +27,7 @@ import * as huskyPlugin from '@form8ion/husky';
 import { scaffold as scaffold$4 } from '@form8ion/husky';
 import { scaffold as scaffold$2 } from '@form8ion/prettier';
 import { scaffold as scaffold$1, lift as lift$4, test as test$2 } from '@form8ion/eslint';
+import sortObjectKeys from 'sort-object-keys';
 
 const questionNames$1 = {
   UNIT_TEST_FRAMEWORK: 'unitTestFramework',
@@ -284,7 +285,6 @@ function predicate ({projectRoot}) {
 }
 
 async function scaffoldTypescript ({config, projectType, projectRoot, testFilenamePattern}) {
-  const eslintConfigs = ['typescript'];
   const shareableTsConfigPackage = `${config.scope}/tsconfig`;
 
   await writeConfigFile({
@@ -307,8 +307,7 @@ async function scaffoldTypescript ({config, projectType, projectRoot, testFilena
   });
 
   return {
-    eslint: {configs: eslintConfigs},
-    eslintConfigs,
+    eslint: {configs: ['typescript']},
     devDependencies: ['typescript', shareableTsConfigPackage],
     vcsIgnore: {files: ['tsconfig.tsbuildinfo']}
   };
@@ -425,6 +424,13 @@ async function scaffoldPackage ({
   return {homepage: packageData.homepage};
 }
 
+function sortPackageProperties (packageContents) {
+  return sortObjectKeys(
+    packageContents,
+    ['name', 'keywords', 'engines', 'scripts', 'dependencies', 'devDependencies', 'peerDependencies']
+  );
+}
+
 const details = {
   [packageManagers.NPM]: {
     installationCommand: 'install',
@@ -488,13 +494,13 @@ async function liftPackage ({
 
     await writePackageJson({
       projectRoot,
-      config: {
+      config: sortPackageProperties({
         ...existingPackageJsonContents,
         scripts: liftScripts({existingScripts: existingPackageJsonContents.scripts, scripts}),
         ...tags && {
           keywords: existingPackageJsonContents.keywords ? [...existingPackageJsonContents.keywords, ...tags] : tags
         }
-      }
+      })
     });
   }
 
@@ -1125,7 +1131,6 @@ async function scaffoldPackageType ({
   return deepmerge.all([
     {
       documentation: scaffoldPackageDocumentation({packageName, visibility, scope, packageManager}),
-      eslintConfigs: [],
       nextSteps: [
         {summary: 'Add the appropriate `save` flag to the installation instructions in the README'},
         {summary: 'Publish pre-release versions to npm until package is stable enough to publish v1.0.0'}
@@ -1156,7 +1161,6 @@ async function scaffoldApplicationType ({projectRoot}) {
     devDependencies: ['rimraf'],
     vcsIgnore: {files: ['.env'], directories: [`/${buildDirectory}/`]},
     buildDirectory,
-    eslintConfigs: [],
     nextSteps: []
   };
 }
@@ -1167,7 +1171,6 @@ async function scaffoldMonorepoType ({projectRoot}) {
   await mergeIntoExistingPackageJson({projectRoot, config: {private: true}});
 
   return {
-    eslintConfigs: [],
     nextSteps: [{
       summary: 'Add packages to your new monorepo',
       description: 'Leverage [@form8ion/add-package-to-monorepo](https://npm.im/@form8ion/add-package-to-monorepo)'
@@ -1208,7 +1211,6 @@ async function scaffoldCliType ({packageName, visibility, projectRoot, dialect,
       vcsIgnore: {files: [], directories: [`/${defaultBuildDirectory}/`]},
       buildDirectory: defaultBuildDirectory,
       badges: defineBadges(packageName, visibility),
-      eslintConfigs: [],
       nextSteps: []
     }
   );
@@ -1250,9 +1252,7 @@ async function scaffoldProjectType ({
     case projectTypes.MONOREPO:
       return scaffoldMonorepoType({projectRoot});
     case 'Other':
-      return {
-        eslintConfigs: []
-      };
+      return {};
     default:
       throw new Error(`The project-type of ${projectType} is invalid`);
   }
@@ -1493,7 +1493,7 @@ async function scaffolder (options) {
     decisions,
     pathWithinParent
   });
-  const [nodeVersion, npmResults, dialectResults, codeStyleResults, projectTypePluginResults] = await Promise.all([
+  const [nodeVersion, npmResults, dialectResults, codeStyleResults] = await Promise.all([
     scaffoldNodeVersion({projectRoot, nodeVersionCategory}),
     scaffoldNpmConfig({projectType, projectRoot, registries}),
     scaffoldDialect({
@@ -1503,24 +1503,24 @@ async function scaffolder (options) {
       projectType,
       testFilenamePattern: verificationResults.testFilenamePattern
     }),
-    scaffoldCodeStyle({projectRoot, projectType, dialect, configs, vcs, configureLinting}),
-    scaffoldProjectTypePlugin({
-      projectRoot,
-      projectType,
-      projectName,
-      packageName,
-      packageManager,
-      scope,
-      dialect,
-      tests,
-      decisions,
-      plugins: {
-        [projectTypes.PACKAGE]: packageTypes,
-        [projectTypes.APPLICATION]: applicationTypes,
-        [projectTypes.MONOREPO]: monorepoTypes
-      }
-    })
+    scaffoldCodeStyle({projectRoot, projectType, dialect, configs, vcs, configureLinting})
   ]);
+  const projectTypePluginResults = await scaffoldProjectTypePlugin({
+    projectRoot,
+    projectType,
+    projectName,
+    packageName,
+    packageManager,
+    scope,
+    dialect,
+    tests,
+    decisions,
+    plugins: {
+      [projectTypes.PACKAGE]: packageTypes,
+      [projectTypes.APPLICATION]: applicationTypes,
+      [projectTypes.MONOREPO]: monorepoTypes
+    }
+  });
   const mergedContributions = deepmerge.all([
     ...(await Promise.all([
       scaffoldChoice(