@form8ion/javascript 15.8.3 → 16.0.0-beta.1

package/package.json

@@ -2,15 +2,15 @@
   "name": "@form8ion/javascript",
   "description": "JavaScript language plugin for the @form8ion toolset",
   "license": "MIT",
-  "version": "15.8.3",
+  "version": "16.0.0-beta.1",
   "type": "module",
   "engines": {
-    "node": "^20.19.0 || >=22.14.0"
+    "node": "^22.21.0 || >=24.12"
   },
   "author": "Matt Travi <npm@travi.org> (https://matt.travi.org)",
   "repository": "form8ion/javascript",
   "bugs": "https://github.com/form8ion/javascript/issues",
-  "homepage": "https://npm.im/@form8ion/javascript",
+  "homepage": "https://www.npmjs.com/package/@form8ion/javascript",
   "runkitExampleFilename": "./example.js",
   "exports": {
     "module": "./lib/index.js",
@@ -22,7 +22,7 @@
     "pretest": "run-s build",
     "test": "npm-run-all --print-label --parallel lint:* --parallel test:*",
     "lint:engines": "ls-engines",
-    "lint:gherkin": "gherkin-lint",
+    "lint:gherkin": "gplint --config .gplintrc.json",
     "lint:js": "eslint . --cache",
     "lint:js:fix": "run-s 'lint:js -- --fix'",
     "lint:lockfile": "lockfile-lint",
@@ -59,26 +59,25 @@
     "access": "public",
     "provenance": true
   },
-  "packageManager": "npm@11.12.1+sha512.cdca14b85d647b3192028d02aadbe82d75f79a446aceea9874be98e6d768f20ebd3555770a48d0e9906106007877bbc690f715e9372f2e2dc644a3c3157fb14c",
+  "packageManager": "npm@11.15.0+sha512.fa4d2d93b9519e93143e70bb913b94ff2ad5fe69c5a0f84943be557cbb59e9fc1bcce55768fb1313f225f4f9f50c78b8f366f2332aa41effd7b10efa98d8d72f",
   "dependencies": {
     "@form8ion/c8": "^1.0.0-beta.2",
     "@form8ion/codecov": "^7.0.0-beta.1",
     "@form8ion/commit-convention": "^8.0.0-beta.6",
     "@form8ion/config-file": "^1.1.1",
-    "@form8ion/core": "^4.0.0",
+    "@form8ion/core": "^5.0.0-beta.9",
     "@form8ion/eslint": "^7.0.0-beta.1",
     "@form8ion/husky": "^7.0.0-beta.1",
     "@form8ion/javascript-core": "^12.0.0-beta.1",
     "@form8ion/overridable-prompts": "^1.2.0",
     "@form8ion/prettier": "^3.0.0",
     "@hapi/hoek": "^11.0.0",
-    "@travi/cli-messages": "^1.1.0",
     "@travi/language-scaffolder-prompts": "^2.0.0-beta.1",
     "camelcase": "^9.0.0",
     "deepmerge": "^4.2.2",
     "execa": "^9.5.1",
     "ini": "^6.0.0",
-    "joi": "^17.5.0",
+    "joi": "^18.2.1",
     "mustache": "^4.2.0",
     "npm-conf": "^1.1.3",
     "sort-object-keys": "^2.0.0",
@@ -86,34 +85,35 @@
     "validate-npm-package-name": "^7.0.0"
   },
   "devDependencies": {
-    "@cucumber/cucumber": "12.7.0",
-    "@form8ion/commitlint-config": "2.0.11",
+    "@cucumber/cucumber": "12.9.0",
+    "@form8ion/commitlint-config": "2.0.15",
     "@form8ion/eslint-config": "7.1.0-beta.1",
     "@form8ion/eslint-config-cucumber": "1.4.1",
     "@form8ion/eslint-config-vitest": "1.1.0",
     "@form8ion/remark-lint-preset": "6.0.7",
     "@rollup/plugin-node-resolve": "16.0.3",
-    "@travi/any": "3.1.3",
-    "@vitest/coverage-v8": "4.1.4",
+    "@travi/any": "3.3.0",
+    "@vitest/coverage-v8": "4.1.7",
     "ban-sensitive-files": "1.10.11",
     "chai": "6.2.2",
     "cz-conventional-changelog": "3.3.0",
-    "gherkin-lint": "4.2.4",
+    "debug": "4.4.3",
+    "gplint": "2.5.2",
     "husky": "9.1.7",
     "js-yaml": "4.1.1",
     "lockfile-lint": "5.0.0",
     "ls-engines": "0.9.4",
     "mock-fs": "5.5.0",
-    "npm-run-all2": "8.0.4",
-    "publint": "0.3.18",
+    "npm-run-all2": "9.0.1",
+    "publint": "0.3.21",
     "remark-cli": "12.0.1",
     "remark-toc": "9.0.0",
     "remark-usage": "11.0.1",
     "rimraf": "6.1.3",
-    "rollup": "4.60.1",
+    "rollup": "4.60.4",
     "rollup-plugin-auto-external": "2.0.0",
     "testdouble": "3.20.2",
-    "vitest": "4.1.4",
+    "vitest": "4.1.7",
     "vitest-when": "0.10.0"
   }
 }

package/src/code-style/lifter.js

@@ -1,6 +1,6 @@
 import {applyEnhancers} from '@form8ion/core';
 import * as eslintPlugin from '@form8ion/eslint';
 
-export default function liftCodeStyle(options) {
-  return applyEnhancers({options, enhancers: [eslintPlugin]});
+export default function liftCodeStyle(options, dependencies) {
+  return applyEnhancers({options, enhancers: [eslintPlugin], dependencies});
 }

package/src/dependencies/installer.js

@@ -1,5 +1,4 @@
-import {info, warn} from '@travi/cli-messages';
-import {DEV_DEPENDENCY_TYPE, packageManagers} from '@form8ion/javascript-core';
+import {DEV_DEPENDENCY_TYPE} from '@form8ion/javascript-core';
 
 import {execa} from 'execa';
 import {getDependencyTypeFlag, getExactFlag, getInstallationCommandFor} from './package-managers.js';
@@ -8,10 +7,11 @@ export default async function installDependencies(
   dependencies,
   dependenciesType,
   projectRoot,
-  packageManager = packageManagers.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} ${
@@ -21,5 +21,5 @@ export default async function installDependencies(
       }`,
       {shell: true, cwd: projectRoot}
     );
-  } else warn(`No ${dependenciesType} dependencies to install`);
+  } else logger.warn(`No ${dependenciesType} dependencies to install`);
 }

package/src/dependencies/installer.test.js

@@ -1,11 +1,11 @@
 import {execa} from 'execa';
-import {DEV_DEPENDENCY_TYPE, packageManagers} from '@form8ion/javascript-core';
+import {DEV_DEPENDENCY_TYPE} from '@form8ion/javascript-core';
 
-import {vi, it, describe, expect, beforeEach} from 'vitest';
+import {beforeEach, describe, expect, it, vi} from 'vitest';
 import any from '@travi/any';
 import {when} from 'vitest-when';
 
-import {getDependencyTypeFlag, getInstallationCommandFor, getExactFlag} from './package-managers.js';
+import {getDependencyTypeFlag, getExactFlag, getInstallationCommandFor} from './package-managers.js';
 import install from './installer.js';
 
 vi.mock('execa');
@@ -20,13 +20,14 @@ describe('dependencies installer', () => {
   const uniqueDependencies = any.listOf(any.word);
   const dependencies = [duplicateDependency, ...uniqueDependencies, duplicateDependency];
   const exactFlag = any.word();
+  const logger = {info: () => undefined, warn: () => undefined};
 
   beforeEach(() => {
     when(getInstallationCommandFor).calledWith(packageManager).thenReturn(installationCommand);
   });
 
   it('should avoid execution when there are no dependencies to install', async () => {
-    await install([]);
+    await install([], undefined, undefined, undefined, {logger});
 
     expect(execa).not.toHaveBeenCalled();
   });
@@ -35,7 +36,7 @@ describe('dependencies installer', () => {
     const dependenciesType = any.word();
     when(getDependencyTypeFlag).calledWith(packageManager, dependenciesType).thenReturn(typeFlag);
 
-    await install(dependencies, dependenciesType, projectRoot, packageManager);
+    await install(dependencies, dependenciesType, projectRoot, packageManager, {logger});
 
     expect(execa).toHaveBeenCalledWith(
       `. ~/.nvm/nvm.sh && nvm use && ${packageManager} ${installationCommand} ${
@@ -46,11 +47,11 @@ describe('dependencies installer', () => {
   });
 
   it('should pin versions when installing dev-dependencies', async () => {
-    await install(dependencies, DEV_DEPENDENCY_TYPE, projectRoot, packageManager);
+    await install(dependencies, DEV_DEPENDENCY_TYPE, projectRoot, packageManager, {logger});
     when(getDependencyTypeFlag).calledWith(packageManager, DEV_DEPENDENCY_TYPE).thenReturn(typeFlag);
     when(getExactFlag).calledWith(packageManager).thenReturn(exactFlag);
 
-    await install(dependencies, DEV_DEPENDENCY_TYPE, projectRoot, packageManager);
+    await install(dependencies, DEV_DEPENDENCY_TYPE, projectRoot, packageManager, {logger});
 
     expect(execa).toHaveBeenCalledWith(
       `. ~/.nvm/nvm.sh && nvm use && ${packageManager} ${installationCommand} ${
@@ -59,19 +60,4 @@ describe('dependencies installer', () => {
       {shell: true, cwd: projectRoot}
     );
   });
-
-  it('should default to `npm` when the package-manager is not specified', async () => {
-    when(getDependencyTypeFlag).calledWith(packageManagers.NPM, DEV_DEPENDENCY_TYPE).thenReturn(typeFlag);
-    when(getExactFlag).calledWith(packageManagers.NPM).thenReturn(exactFlag);
-    when(getInstallationCommandFor).calledWith(packageManagers.NPM).thenReturn(installationCommand);
-
-    await install(dependencies, DEV_DEPENDENCY_TYPE, projectRoot);
-
-    expect(execa).toHaveBeenCalledWith(
-      `. ~/.nvm/nvm.sh && nvm use && ${packageManagers.NPM} ${installationCommand} ${
-        [duplicateDependency, ...uniqueDependencies].join(' ')
-      } --${typeFlag} --${exactFlag}`,
-      {shell: true, cwd: projectRoot}
-    );
-  });
 });

package/src/dependencies/processor.js

@@ -1,11 +1,13 @@
-import {error, info} from '@travi/cli-messages';
 import {DEV_DEPENDENCY_TYPE, PROD_DEPENDENCY_TYPE} from '@form8ion/javascript-core';
 
 import install from './installer.js';
 import remove from './remover.js';
 
-export default async function processDependencies({dependencies = {}, devDependencies, projectRoot, packageManager}) {
-  info('Processing dependencies');
+export default async function processDependencies(
+  {dependencies = {}, devDependencies, projectRoot, packageManager},
+  {logger}
+) {
+  logger.info('Processing dependencies');
 
   if (Array.isArray(devDependencies)) {
     throw new Error(
@@ -20,11 +22,11 @@ export default async function processDependencies({dependencies = {}, devDepende
   const {javascript: {production = [], development = [], remove: dependenciesToRemove = []} = {}} = dependencies;
 
   try {
-    await install(production, PROD_DEPENDENCY_TYPE, projectRoot, packageManager);
-    await install(development, DEV_DEPENDENCY_TYPE, projectRoot, packageManager);
-    await remove({packageManager, dependencies: dependenciesToRemove});
+    await install(production, PROD_DEPENDENCY_TYPE, projectRoot, packageManager, {logger});
+    await install(development, DEV_DEPENDENCY_TYPE, projectRoot, packageManager, {logger});
+    await remove({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'});
   }
 }

package/src/dependencies/processor.test.js

@@ -13,6 +13,7 @@ vi.mock('./remover.js');
 describe('dependencies processor', () => {
   const projectRoot = any.string();
   const packageManager = any.word();
+  const logger = {info: () => undefined, error: () => undefined};
 
   it('should process the provided dependency lists', async () => {
     const production = any.listOf(any.word);
@@ -23,52 +24,64 @@ describe('dependencies processor', () => {
       dependencies: {javascript: {production, development, remove}},
       projectRoot,
       packageManager
-    });
+    }, {logger});
 
-    expect(installDependencies).toHaveBeenCalledWith(production, PROD_DEPENDENCY_TYPE, projectRoot, packageManager);
-    expect(installDependencies).toHaveBeenCalledWith(development, DEV_DEPENDENCY_TYPE, projectRoot, packageManager);
-    expect(removeDependencies).toHaveBeenCalledWith({packageManager, dependencies: remove});
+    expect(installDependencies).toHaveBeenCalledWith(
+      production,
+      PROD_DEPENDENCY_TYPE,
+      projectRoot,
+      packageManager,
+      {logger}
+    );
+    expect(installDependencies).toHaveBeenCalledWith(
+      development,
+      DEV_DEPENDENCY_TYPE,
+      projectRoot,
+      packageManager,
+      {logger}
+    );
+    expect(removeDependencies).toHaveBeenCalledWith({packageManager, dependencies: remove}, {logger});
   });
 
   it('should process as empty lists when dependencies are not provided', async () => {
-    await processDependencies({projectRoot, packageManager});
+    await processDependencies({projectRoot, packageManager}, {logger});
 
-    expect(installDependencies).toHaveBeenCalledWith([], PROD_DEPENDENCY_TYPE, projectRoot, packageManager);
-    expect(installDependencies).toHaveBeenCalledWith([], DEV_DEPENDENCY_TYPE, projectRoot, packageManager);
+    expect(installDependencies).toHaveBeenCalledWith([], PROD_DEPENDENCY_TYPE, projectRoot, packageManager, {logger});
+    expect(installDependencies).toHaveBeenCalledWith([], DEV_DEPENDENCY_TYPE, projectRoot, packageManager, {logger});
   });
 
   it('should process as empty lists when javascript dependencies are not provided', async () => {
-    await processDependencies({projectRoot, packageManager, dependencies: {}});
+    await processDependencies({projectRoot, packageManager, dependencies: {}}, {logger});
 
-    expect(installDependencies).toHaveBeenCalledWith([], PROD_DEPENDENCY_TYPE, projectRoot, packageManager);
-    expect(installDependencies).toHaveBeenCalledWith([], DEV_DEPENDENCY_TYPE, projectRoot, packageManager);
+    expect(installDependencies).toHaveBeenCalledWith([], PROD_DEPENDENCY_TYPE, projectRoot, packageManager, {logger});
+    expect(installDependencies).toHaveBeenCalledWith([], DEV_DEPENDENCY_TYPE, projectRoot, packageManager, {logger});
   });
 
   it('should process as empty lists when dependency types are not provided', async () => {
-    await processDependencies({projectRoot, packageManager, dependencies: {javascript: {}}});
+    await processDependencies({projectRoot, packageManager, dependencies: {javascript: {}}}, {logger});
 
-    expect(installDependencies).toHaveBeenCalledWith([], PROD_DEPENDENCY_TYPE, projectRoot, packageManager);
-    expect(installDependencies).toHaveBeenCalledWith([], DEV_DEPENDENCY_TYPE, projectRoot, packageManager);
-    expect(removeDependencies).toHaveBeenCalledWith({packageManager, dependencies: []});
+    expect(installDependencies).toHaveBeenCalledWith([], PROD_DEPENDENCY_TYPE, projectRoot, packageManager, {logger});
+    expect(installDependencies).toHaveBeenCalledWith([], DEV_DEPENDENCY_TYPE, projectRoot, packageManager, {logger});
+    expect(removeDependencies).toHaveBeenCalledWith({packageManager, dependencies: []}, {logger});
   });
 
   it('should prevent an installation error from bubbling', async () => {
     installDependencies.mockRejectedValue(new Error());
 
-    await expect(() => processDependencies({dependencies: {javascript: {}}})).not.toThrowError();
+    expect(() => processDependencies({dependencies: {javascript: {}}}, {logger})).not.toThrow();
   });
 
   it('should throw an error if dependencies is defined as an array rather than an object', async () => {
     const dependencies = any.listOf(any.word);
 
-    await expect(() => processDependencies({dependencies}))
-      .rejects.toThrowError(`Expected dependencies to be an object. Instead received: ${dependencies}`);
+    await expect(() => processDependencies({dependencies}, {logger}))
+      .rejects.toThrow(`Expected dependencies to be an object. Instead received: ${dependencies}`);
   });
 
   it('should throw an error if devDependencies is defined as an array', async () => {
     const devDependencies = any.listOf(any.word);
 
-    await expect(() => processDependencies({devDependencies})).rejects.toThrowError(
+    await expect(() => processDependencies({devDependencies}, {logger})).rejects.toThrowError(
       `devDependencies provided as: ${devDependencies}. Instead, provide under dependencies.javascript.development`
     );
   });

package/src/dependencies/remover.js

@@ -1,9 +1,8 @@
 import {execa} from 'execa';
-import {info} from '@travi/cli-messages';
 
-export default async function removeDependencies({packageManager, dependencies}) {
+export default 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]);
   }

package/src/dependencies/remover.test.js

@@ -8,11 +8,13 @@ import removeDependencies from './remover.js';
 vi.mock('execa');
 
 describe('dependency remover', () => {
+  const logger = {info: () => undefined};
+
   it('should remove the dependencies using the named package manager', async () => {
     const packageManager = any.word();
     const dependencies = any.listOf(any.word);
 
-    await removeDependencies({packageManager, dependencies});
+    await removeDependencies({packageManager, dependencies}, {logger});
 
     expect(execa).toHaveBeenCalledWith(packageManager, ['remove', ...dependencies]);
   });
@@ -21,7 +23,7 @@ describe('dependency remover', () => {
     const packageManager = any.word();
     const dependencies = [];
 
-    await removeDependencies({packageManager, dependencies});
+    await removeDependencies({packageManager, dependencies}, {logger});
 
     expect(execa).not.toHaveBeenCalled();
   });

package/src/lifter.js

@@ -1,6 +1,5 @@
 import {promises as fs} from 'node:fs';
 import deepmerge from 'deepmerge';
-import {info} from '@travi/cli-messages';
 import {applyEnhancers} from '@form8ion/core';
 import * as huskyPlugin from '@form8ion/husky';
 import * as commitConventionPlugin from '@form8ion/commit-convention';
@@ -23,8 +22,8 @@ export default 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([
     resolvePackageManager({projectRoot, packageManager: results.packageManager}),
@@ -46,10 +45,14 @@ export default 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 liftPackage(deepmerge.all([{projectRoot, packageManager, vcs, pathWithinParent}, enhancerResults]));
+  await liftPackage(
+    deepmerge.all([{projectRoot, packageManager, vcs, pathWithinParent}, enhancerResults]),
+    dependencies
+  );
 
   return enhancerResults;
 }

package/src/lifter.test.js

@@ -29,13 +29,12 @@ describe('lift', () => {
   const projectRoot = any.string();
   const scripts = any.simpleObject();
   const tags = any.listOf(any.word);
-  const dependencies = any.simpleObject();
-  const devDependencies = any.simpleObject();
+  const dependencies = {...any.simpleObject(), logger: {info: () => undefined}};
   const packageManager = any.word();
   const manager = any.word();
   const enhancerResults = any.simpleObject();
   const vcsDetails = any.simpleObject();
-  const results = {...any.simpleObject(), scripts, tags, dependencies, devDependencies, packageManager: manager};
+  const results = {...any.simpleObject(), scripts, tags, packageManager: manager};
   const pathWithinParent = any.string();
   const packageDetails = any.simpleObject();
   const internalEnhancers = {
@@ -66,17 +65,18 @@ describe('lift', () => {
       .calledWith({
         results,
         enhancers: internalEnhancers,
-        options: {projectRoot, packageManager, vcs: vcsDetails, packageDetails, configs}
+        options: {projectRoot, packageManager, vcs: vcsDetails, packageDetails, configs},
+        dependencies
       })
       .thenResolve(enhancerResults);
 
-    const liftResults = await lift({projectRoot, vcs: vcsDetails, results, pathWithinParent, configs});
+    const liftResults = await lift({projectRoot, vcs: vcsDetails, results, pathWithinParent, configs}, dependencies);
 
     expect(liftResults).toEqual(enhancerResults);
-    expect(liftPackage).toHaveBeenCalledWith(deepmerge.all([
-      {projectRoot, packageManager, vcs: vcsDetails, pathWithinParent},
-      enhancerResults
-    ]));
+    expect(liftPackage).toHaveBeenCalledWith(
+      deepmerge.all([{projectRoot, packageManager, vcs: vcsDetails, pathWithinParent}, enhancerResults]),
+      dependencies
+    );
   });
 
   it('should apply provided enhancers', async () => {
@@ -85,11 +85,12 @@ describe('lift', () => {
       .calledWith({
         results,
         enhancers: {...enhancers, ...internalEnhancers},
-        options: {projectRoot, packageManager, vcs: vcsDetails, packageDetails, configs: {}}
+        options: {projectRoot, packageManager, vcs: vcsDetails, packageDetails, configs: {}},
+        dependencies
       })
       .thenResolve(enhancerResults);
 
-    const liftResults = await lift({projectRoot, vcs: vcsDetails, results, pathWithinParent, enhancers});
+    const liftResults = await lift({projectRoot, vcs: vcsDetails, results, pathWithinParent, enhancers}, dependencies);
 
     expect(liftResults).toEqual(enhancerResults);
   });

package/src/node-version/scaffolder.js

@@ -1,19 +1,18 @@
 import {promises as fs} from 'node:fs';
-import {info} from '@travi/cli-messages';
 
 import {determineLatestVersionOf, install as installNodeVersion} from './tasks.js';
 
-export default async function scaffoldNodeVersion({projectRoot, nodeVersionCategory}) {
+export default 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 fs.writeFile(`${projectRoot}/.nvmrc`, version);
 
-  await installNodeVersion(nodeVersionCategory);
+  await installNodeVersion(nodeVersionCategory, {logger});
 
   return version;
 }

package/src/node-version/scaffolder.test.js

@@ -12,6 +12,7 @@ vi.mock('./tasks.js');
 
 describe('node-version scaffolder', () => {
   const projectRoot = any.string();
+  const logger = {info: () => undefined};
 
   afterEach(() => {
     vi.clearAllMocks();
@@ -20,14 +21,14 @@ describe('node-version scaffolder', () => {
   it('should configure nvm with the desired version', async () => {
     const nodeVersionCategory = any.word();
     const version = any.word();
-    when(determineLatestVersionOf).calledWith(nodeVersionCategory).thenResolve(version);
+    when(determineLatestVersionOf).calledWith(nodeVersionCategory, {logger}).thenResolve(version);
 
-    expect(await scaffoldNodeVersion({projectRoot, nodeVersionCategory})).toEqual(version);
-    expect(installNodeVersion).toHaveBeenCalledWith(nodeVersionCategory);
+    expect(await scaffoldNodeVersion({projectRoot, nodeVersionCategory}, {logger})).toEqual(version);
+    expect(installNodeVersion).toHaveBeenCalledWith(nodeVersionCategory, {logger});
     expect(fs.writeFile).toHaveBeenCalledWith(`${projectRoot}/.nvmrc`, version);
   });
 
   it('should return `undefined` when a category is not defined', async () => {
-    expect(await scaffoldNodeVersion({projectRoot})).toBe(undefined);
+    expect(await scaffoldNodeVersion({projectRoot}, {logger})).toBe(undefined);
   });
 });

package/src/node-version/tasks.js

@@ -1,9 +1,7 @@
-import {info} from '@travi/cli-messages';
-
 import {execa} from 'execa';
 
-export async function determineLatestVersionOf(nodeVersionCategory) {
-  info('Determining version of node', {level: 'secondary'});
+export 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' : ''}`,
@@ -16,8 +14,8 @@ export async function determineLatestVersionOf(nodeVersionCategory) {
   return lsLine.match(/(v[0-9]+)\.[0-9]+\.[0-9]+/)[1];
 }
 
-export function install(nodeVersionCategory) {
-  info(`Installing ${nodeVersionCategory} version of node using nvm`, {level: 'secondary'});
+export 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);

package/src/node-version/tasks.test.js

@@ -1,6 +1,6 @@
 import {execa} from 'execa';
 
-import {it, expect, describe, afterEach, vi} from 'vitest';
+import {describe, expect, it, vi} from 'vitest';
 import {when} from 'vitest-when';
 import any from '@travi/any';
 
@@ -11,17 +11,14 @@ vi.mock('execa');
 describe('node-version tasks', () => {
   const majorVersion = any.integer();
   const version = `v${majorVersion}.${any.integer()}.${any.integer()}`;
-
-  afterEach(() => {
-    vi.clearAllMocks();
-  });
+  const logger = {info: () => undefined};
 
   it('should return the latest node version when latest is requested', async () => {
     when(execa)
       .calledWith('. ~/.nvm/nvm.sh && nvm ls-remote', {shell: true})
       .thenResolve({stdout: [...any.listOf(any.word), version, ''].join('\n')});
 
-    expect(await determineLatestVersionOf(any.word())).toEqual(`v${majorVersion}`);
+    expect(await determineLatestVersionOf(any.word(), {logger})).toEqual(`v${majorVersion}`);
   });
 
   it('should return the latest LTS node version when LTS is requested', async () => {
@@ -29,14 +26,14 @@ describe('node-version tasks', () => {
       .calledWith('. ~/.nvm/nvm.sh && nvm ls-remote --lts', {shell: true})
       .thenResolve({stdout: [...any.listOf(any.word), version, ''].join('\n')});
 
-    expect(await determineLatestVersionOf('LTS')).toEqual(`v${majorVersion}`);
+    expect(await determineLatestVersionOf('LTS', {logger})).toEqual(`v${majorVersion}`);
   });
 
   it('should install the node version', async () => {
     const pipe = vi.fn();
     when(execa).calledWith('. ~/.nvm/nvm.sh && nvm install', {shell: true}).thenReturn({stdout: {pipe}});
 
-    await install();
+    await install(undefined, {logger});
 
     expect(pipe).toHaveBeenCalledWith(process.stdout);
   });

package/src/options/schemas.js

@@ -1,4 +1,5 @@
 import joi from 'joi';
+import {visibilityOptions} from '@form8ion/core';
 
 export const scopeBasedConfigSchema = joi.object({scope: joi.string().regex(/^@[a-z0-9-]+$/i, 'scope').required()});
 
@@ -9,6 +10,6 @@ export const nameBasedConfigSchema = joi.object({
 
 export const registriesSchema = joi.object().pattern(joi.string(), joi.string().uri()).default({});
 
-export const visibilitySchema = joi.string().valid('Public', 'Private').required();
+export const visibilitySchema = joi.string().valid(...Object.keys(visibilityOptions)).required();
 
 export const projectNameSchema = joi.string().regex(/^@\w*\//, {invert: true}).required();

package/src/options/schemas.test.js

@@ -104,21 +104,27 @@ describe('options schemas', () => {
       expect(() => validateOptions(visibilitySchema)).toThrowError('"value" is required');
     });
 
-    it('should allow `Public` as a valid value', () => {
-      const visibility = 'Public';
+    it('should allow "Open Source" as a valid value', () => {
+      const visibility = 'OSS';
 
       expect(validateOptions(visibilitySchema, visibility)).toEqual(visibility);
     });
 
-    it('should allow `Private` as a valid value', () => {
-      const visibility = 'Private';
+    it('should allow "Inner Source" as a valid value', () => {
+      const visibility = 'ISS';
+
+      expect(validateOptions(visibilitySchema, visibility)).toEqual(visibility);
+    });
+
+    it('should allow "Closed Source" as a valid value', () => {
+      const visibility = 'CS';
 
       expect(validateOptions(visibilitySchema, visibility)).toEqual(visibility);
     });
 
     it('should consider values other than `Public` and `Private` as invalid', () => {
       expect(() => validateOptions(visibilitySchema, any.word()))
-        .toThrowError('"value" must be one of [Public, Private]');
+        .toThrowError('"value" must be one of [OSS, ISS, CS]');
     });
   });