@form8ion/javascript 15.8.2 → 16.0.0-beta.1

package/README.md

@@ -175,7 +175,7 @@ $ npm test
 
 [renovate-link]: https://renovatebot.com
 
-[renovate-badge]: https://img.shields.io/badge/renovate-enabled-brightgreen.svg?logo=renovatebot
+[renovate-badge]: https://img.shields.io/badge/renovate-enabled-brightgreen.svg?logo=renovate
 
 [github-actions-ci-link]: https://github.com/form8ion/javascript/actions?query=workflow%3A%22Node.js+CI%22+branch%3Amaster
 
@@ -197,6 +197,6 @@ $ npm test
 
 [coverage-link]: https://codecov.io/github/form8ion/javascript
 
-[coverage-badge]: https://img.shields.io/codecov/c/github/form8ion/javascript?logo=codecov
+[coverage-badge]: https://img.shields.io/codecov/c/github/form8ion/javascript/master?logo=codecov
 
 [slsa-badge]: https://slsa.dev/images/gh-badge-level2.svg

package/lib/index.js

@@ -1,11 +1,10 @@
 import { questionNames as questionNames$2, questions } from '@travi/language-scaffolder-prompts';
 import deepmerge from 'deepmerge';
-import { warn, info, error } from '@travi/cli-messages';
 import { scaffoldChoice, dialects as dialects$1, projectTypes as projectTypes$1, packageManagers as packageManagers$1, writePackageJson, PROD_DEPENDENCY_TYPE, DEV_DEPENDENCY_TYPE, mergeIntoExistingPackageJson } from '@form8ion/javascript-core';
 import * as commitConventionPlugin from '@form8ion/commit-convention';
 import { scaffold as scaffold$5 } from '@form8ion/commit-convention';
 import joi from 'joi';
-import { fileExists, directoryExists, optionsSchemas, validateOptions, fileTypes, loadConfigFile, writeConfigFile, applyEnhancers } from '@form8ion/core';
+import { fileExists, directoryExists, optionsSchemas, validateOptions, visibilityOptions, fileTypes, loadConfigFile, writeConfigFile, applyEnhancers } from '@form8ion/core';
 import { prompt as prompt$1 } from '@form8ion/overridable-prompts';
 import { execa } from 'execa';
 import npmConf from 'npm-conf';
@@ -145,7 +144,7 @@ const nameBasedConfigSchema = joi.object({
 
 const registriesSchema = joi.object().pattern(joi.string(), joi.string().uri()).default({});
 
-const visibilitySchema = joi.string().valid('Public', 'Private').required();
+const visibilitySchema = joi.string().valid(...Object.keys(visibilityOptions)).required();
 
 const projectNameSchema = joi.string().regex(/^@\w*\//, {invert: true}).required();
 
@@ -270,7 +269,8 @@ async function prompt(
   vcs,
   decisions,
   configs,
-  pathWithinParent
+  pathWithinParent,
+  {logger}
 ) {
   const npmConf$1 = npmConf();
 
@@ -279,7 +279,7 @@ async function prompt(
     maybeLoggedInNpmUsername = (await execa('npm', ['whoami'])).stdout;
   } catch (failedExecutionResult) {
     if (!decisions[questionNames$1.SCOPE]) {
-      warn('No logged in user found with `npm whoami`. Login with `npm login` '
+      logger.warn('No logged in user found with `npm whoami`. Login with `npm login` '
         + 'to use your npm account name as the package scope default.');
     }
   }
@@ -657,8 +657,8 @@ $ ${packageManager} test
   };
 }
 
-async function determineLatestVersionOf(nodeVersionCategory) {
-  info('Determining version of node', {level: 'secondary'});
+async function determineLatestVersionOf(nodeVersionCategory, {logger}) {
+  logger.info('Determining version of node', {level: 'secondary'});
 
   const {stdout: nvmLsOutput} = await execa(
     `. ~/.nvm/nvm.sh && nvm ls-remote${('LTS' === nodeVersionCategory) ? ' --lts' : ''}`,
@@ -671,25 +671,25 @@ async function determineLatestVersionOf(nodeVersionCategory) {
   return lsLine.match(/(v[0-9]+)\.[0-9]+\.[0-9]+/)[1];
 }
 
-function install(nodeVersionCategory) {
-  info(`Installing ${nodeVersionCategory} version of node using nvm`, {level: 'secondary'});
+function install(nodeVersionCategory, {logger}) {
+  logger.info(`Installing ${nodeVersionCategory} version of node using nvm`, {level: 'secondary'});
 
   const subprocess = execa('. ~/.nvm/nvm.sh && nvm install', {shell: true});
   subprocess.stdout.pipe(process.stdout);
   return subprocess;
 }
 
-async function scaffoldNodeVersion({projectRoot, nodeVersionCategory}) {
+async function scaffoldNodeVersion({projectRoot, nodeVersionCategory}, {logger}) {
   if (!nodeVersionCategory) return undefined;
 
   const lowerCaseCategory = nodeVersionCategory.toLowerCase();
-  info(`Configuring ${lowerCaseCategory} version of node`);
+  logger.info(`Configuring ${lowerCaseCategory} version of node`);
 
-  const version = await determineLatestVersionOf(nodeVersionCategory);
+  const version = await determineLatestVersionOf(nodeVersionCategory, {logger});
 
   await promises.writeFile(`${projectRoot}/.nvmrc`, version);
 
-  await install(nodeVersionCategory);
+  await install(nodeVersionCategory, {logger});
 
   return version;
 }
@@ -741,8 +741,8 @@ async function scaffoldPackage({
   license,
   author,
   description
-}) {
-  info('Configuring package.json');
+}, {logger}) {
+  logger.info('Configuring package.json');
 
   const packageName = determinePackageName(projectName, scope);
 
@@ -845,10 +845,11 @@ async function installDependencies(
   dependencies,
   dependenciesType,
   projectRoot,
-  packageManager = packageManagers$1.NPM
+  packageManager,
+  {logger}
 ) {
   if (dependencies.length) {
-    info(`Installing ${dependenciesType} dependencies`, {level: 'secondary'});
+    logger.info(`Installing ${dependenciesType} dependencies`, {level: 'secondary'});
 
     await execa(
       `. ~/.nvm/nvm.sh && nvm use && ${packageManager} ${
@@ -858,19 +859,22 @@ async function installDependencies(
       }`,
       {shell: true, cwd: projectRoot}
     );
-  } else warn(`No ${dependenciesType} dependencies to install`);
+  } else logger.warn(`No ${dependenciesType} dependencies to install`);
 }
 
-async function removeDependencies({packageManager, dependencies}) {
+async function removeDependencies({packageManager, dependencies}, {logger}) {
   if (dependencies.length) {
-    info('Removing dependencies dependencies', {level: 'secondary'});
+    logger.info('Removing dependencies dependencies', {level: 'secondary'});
 
     await execa(packageManager, ['remove', ...dependencies]);
   }
 }
 
-async function processDependencies({dependencies = {}, devDependencies, projectRoot, packageManager}) {
-  info('Processing dependencies');
+async function processDependencies(
+  {dependencies = {}, devDependencies, projectRoot, packageManager},
+  {logger}
+) {
+  logger.info('Processing dependencies');
 
   if (Array.isArray(devDependencies)) {
     throw new Error(
@@ -885,12 +889,12 @@ async function processDependencies({dependencies = {}, devDependencies, projectR
   const {javascript: {production = [], development = [], remove: dependenciesToRemove = []} = {}} = dependencies;
 
   try {
-    await installDependencies(production, PROD_DEPENDENCY_TYPE, projectRoot, packageManager);
-    await installDependencies(development, DEV_DEPENDENCY_TYPE, projectRoot, packageManager);
-    await removeDependencies({packageManager, dependencies: dependenciesToRemove});
+    await installDependencies(production, PROD_DEPENDENCY_TYPE, projectRoot, packageManager, {logger});
+    await installDependencies(development, DEV_DEPENDENCY_TYPE, projectRoot, packageManager, {logger});
+    await removeDependencies({packageManager, dependencies: dependenciesToRemove}, {logger});
   } catch (e) {
-    error('Failed to update dependencies');
-    error(e, {level: 'secondary'});
+    logger.error('Failed to update dependencies');
+    logger.error(e, {level: 'secondary'});
   }
 }
 
@@ -982,8 +986,8 @@ async function liftPackageJson({
   packageManager,
   vcs,
   pathWithinParent
-}) {
-  info('Updating `package.json`', {level: 'secondary'});
+}, {logger}) {
+  logger.info('Updating `package.json`', {level: 'secondary'});
 
   const existingPackageJsonContents = JSON.parse(await promises.readFile(`${projectRoot}/package.json`, 'utf-8'));
   const {scripts: liftedScripts, dependencies: scriptDependencies} = liftScripts({
@@ -1008,11 +1012,11 @@ async function liftPackageJson({
     devDependencies,
     projectRoot,
     packageManager
-  });
+  }, {logger});
 }
 
 function mapProjectVisibility({projectVisibility}) {
-  return 'Public' === projectVisibility ? 'public' : 'restricted';
+  return 'OSS' === projectVisibility ? 'public' : 'restricted';
 }
 
 function isScope(firstPart) {
@@ -1037,14 +1041,14 @@ function buildNpmBadgeImageUrl(packageName, customRegistry) {
   return `https://img.shields.io/npm/v/${packageName}?${params}`;
 }
 
-function scaffoldPublishableBadges({packageName, accessLevel, customRegistry}) {
+function scaffoldPublishableBadges({packageName, accessLevel, customRegistry, registryPage}) {
   return {
     consumer: {
       ...'public' === accessLevel && {
         npm: {
           img: buildNpmBadgeImageUrl(packageName, customRegistry),
           text: 'npm',
-          link: `https://www.npmjs.com/package/${packageName}`
+          link: registryPage
         }
       }
     },
@@ -1083,17 +1087,22 @@ async function liftProvenance({projectRoot, packageDetails, customRegistry}) {
 async function liftPublishable({projectRoot, packageDetails, configs}) {
   const {name: packageName, publishConfig: {access: packageAccessLevel}} = packageDetails;
   const customRegistry = resolveRegistry(packageName, configs.registries);
-  const homepage = `https://npm.im/${packageName}`;
+  const registryPage = `https://www.npmjs.com/package/${packageName}`;
 
-  await mergeIntoExistingPackageJson({projectRoot, config: {homepage}});
+  await mergeIntoExistingPackageJson({projectRoot, config: {homepage: registryPage}});
 
   return deepmerge(
     await liftProvenance({packageDetails, projectRoot, customRegistry}),
     {
-      homepage,
+      homepage: registryPage,
       dependencies: {javascript: {development: ['publint']}},
       scripts: {'lint:publish': 'publint --strict'},
-      badges: scaffoldPublishableBadges({packageName, accessLevel: packageAccessLevel, customRegistry})
+      badges: scaffoldPublishableBadges({
+        packageName,
+        accessLevel: packageAccessLevel,
+        customRegistry,
+        registryPage
+      })
     }
   );
 }
@@ -1215,8 +1224,8 @@ async function scaffoldPackageProjectType({
   dialect,
   provideExample,
   publishRegistry
-}) {
-  info('Scaffolding Package Details');
+}, {logger}) {
+  logger.info('Scaffolding Package Details');
 
   const packageAccessLevel = mapProjectVisibility({projectVisibility: visibility});
   const [detailsForBuild, publishableResults] = await Promise.all([
@@ -1284,15 +1293,15 @@ function liftPackage({projectRoot, packageDetails, configs}) {
 }
 
 async function projectIsPackage({
-  packageDetails: {exports: exports$1, publishConfig, bin, private: projectMarkedPrivate}
+  packageDetails: {exports, publishConfig, bin, private: projectMarkedPrivate}
 }) {
-  return !projectMarkedPrivate && (!!exports$1 || (!!publishConfig && !bin));
+  return !projectMarkedPrivate && (!!exports || (!!publishConfig && !bin));
 }
 
 const defaultBuildDirectory$1 = 'public';
 
-async function scaffoldApplication({projectRoot}) {
-  info('Scaffolding Application Details');
+async function scaffoldApplication({projectRoot}, {logger}) {
+  logger.info('Scaffolding Application Details');
 
   await mergeIntoExistingPackageJson({projectRoot, config: {private: true}});
 
@@ -1315,8 +1324,8 @@ function projectIsApplication({packageDetails}) {
   return !!packageDetails.private;
 }
 
-async function scaffoldMonorepo({projectRoot}) {
-  info('Scaffolding Monorepo Details');
+async function scaffoldMonorepo({projectRoot}, {logger}) {
+  logger.info('Scaffolding Monorepo Details');
 
   await mergeIntoExistingPackageJson({projectRoot, config: {private: true}});
 
@@ -1395,7 +1404,7 @@ async function scaffoldProjectType({
   dialect,
   provideExample,
   publishRegistry
-}) {
+}, {logger}) {
   switch (projectType) {
     case projectTypes$1.PACKAGE:
       return scaffoldPackageProjectType({
@@ -1410,9 +1419,9 @@ async function scaffoldProjectType({
         dialect,
         provideExample,
         publishRegistry
-      });
+      }, {logger});
     case projectTypes$1.APPLICATION:
-      return scaffoldApplication({projectRoot});
+      return scaffoldApplication({projectRoot}, {logger});
     case projectTypes$1.CLI:
       return scaffoldCli({
         visibility,
@@ -1423,7 +1432,7 @@ async function scaffoldProjectType({
         packageBundlers
       });
     case projectTypes$1.MONOREPO:
-      return scaffoldMonorepo({projectRoot});
+      return scaffoldMonorepo({projectRoot}, {logger});
     case 'Other':
       return {};
     default:
@@ -1680,8 +1689,8 @@ async function scaffoldCodeStyle({
   ].filter(Boolean)));
 }
 
-function liftCodeStyle(options) {
-  return applyEnhancers({options, enhancers: [eslintPlugin]});
+function liftCodeStyle(options, dependencies) {
+  return applyEnhancers({options, enhancers: [eslintPlugin], dependencies});
 }
 
 function codeStyleConfigured(options) {
@@ -1695,8 +1704,8 @@ var codeStylePlugin = /*#__PURE__*/Object.freeze({
   test: codeStyleConfigured
 });
 
-async function scaffoldJavascript(options) {
-  info('Initializing JavaScript project');
+async function scaffoldJavascript(options, {logger}) {
+  logger.info('Initializing JavaScript project');
 
   const {
     projectRoot,
@@ -1732,9 +1741,9 @@ async function scaffoldJavascript(options) {
     provideExample,
     packageManager,
     dialect
-  } = await prompt(ciServices, hosts, visibility, vcs, decisions, configs, pathWithinParent);
+  } = await prompt(ciServices, hosts, visibility, vcs, decisions, configs, pathWithinParent, {logger});
 
-  info('Writing project files', {level: 'secondary'});
+  logger.info('Writing project files', {level: 'secondary'});
 
   const {packageName} = await scaffoldPackage({
     projectRoot,
@@ -1744,7 +1753,7 @@ async function scaffoldJavascript(options) {
     license,
     author,
     description
-  });
+  }, {logger});
   const projectTypeResults = await scaffoldProjectType({
     projectType,
     projectRoot,
@@ -1758,7 +1767,7 @@ async function scaffoldJavascript(options) {
     dialect,
     provideExample,
     publishRegistry: configs.registries.publish
-  });
+  }, {logger});
   const verificationResults = await scaffoldVerification({
     projectRoot,
     dialect,
@@ -1772,7 +1781,7 @@ async function scaffoldJavascript(options) {
     pathWithinParent
   });
   const [nodeVersion, npmResults, dialectResults, codeStyleResults] = await Promise.all([
-    scaffoldNodeVersion({projectRoot, nodeVersionCategory}),
+    scaffoldNodeVersion({projectRoot, nodeVersionCategory}, {logger}),
     scaffoldNpmConfiguration({projectType, projectRoot}),
     scaffoldDialect({
       dialect,
@@ -1908,8 +1917,8 @@ async function liftJavaScript({
   pathWithinParent,
   enhancers = {},
   configs = {}
-}) {
-  info('Lifting JavaScript-specific details');
+}, dependencies) {
+  dependencies.logger.info('Lifting JavaScript-specific details');
 
   const [packageManager, packageContents] = await Promise.all([
     resolveCurrentPackageManager({projectRoot, packageManager: results.packageManager}),
@@ -1931,15 +1940,19 @@ async function liftJavaScript({
       packageManagers,
       registriesPlugin
     },
-    options: {packageManager, projectRoot, vcs, packageDetails: JSON.parse(packageContents), configs}
+    options: {packageManager, projectRoot, vcs, packageDetails: JSON.parse(packageContents), configs},
+    dependencies
   });
 
-  await liftPackageJson(deepmerge.all([{projectRoot, packageManager, vcs, pathWithinParent}, enhancerResults]));
+  await liftPackageJson(
+    deepmerge.all([{projectRoot, packageManager, vcs, pathWithinParent}, enhancerResults]),
+    dependencies
+  );
 
   return enhancerResults;
 }
 
-async function projectUsesJavaScript({projectRoot}) {
+async function projectUsesJavaScript({projectRoot}, {logger}) {
   const [nvmFound, jsPackageManagerFound] = await Promise.all([
     nodeVersionMangerInUse({projectRoot}),
     packageManagerInUse({projectRoot})
@@ -1947,7 +1960,7 @@ async function projectUsesJavaScript({projectRoot}) {
 
   const jsProjectFound = nvmFound || jsPackageManagerFound;
 
-  if (jsProjectFound) info('JavaScript Project Detected');
+  if (jsProjectFound) logger.info('JavaScript Project Detected');
 
   return jsProjectFound;
 }