@form8ion/javascript 15.8.3 → 16.0.0-beta.2

package/src/options/validator.test.js

@@ -68,7 +68,8 @@ describe('options validator', () => {
           packageTypes: pluginsSchema,
           monorepoTypes: pluginsSchema,
           hosts: pluginsSchema,
-          ciServices: pluginsSchema
+          ciServices: pluginsSchema,
+          registries: pluginsSchema
         }
       })
       .thenReturn({required: joiRequiredObject});

package/src/package/lifter.js

@@ -1,6 +1,5 @@
 import {promises as fs} from 'node:fs';
 import deepmerge from 'deepmerge';
-import {info} from '@travi/cli-messages';
 import {writePackageJson} from '@form8ion/javascript-core';
 
 import sortPackageProperties from './property-sorter.js';
@@ -17,8 +16,8 @@ export default 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 fs.readFile(`${projectRoot}/package.json`, 'utf-8'));
   const {scripts: liftedScripts, dependencies: scriptDependencies} = liftScripts({
@@ -43,5 +42,5 @@ export default async function liftPackageJson({
     devDependencies,
     projectRoot,
     packageManager
-  });
+  }, {logger});
 }

package/src/package/lifter.test.js

@@ -34,6 +34,7 @@ describe('package.json lifter', () => {
   const config = any.simpleObject();
   const tags = any.listOf(any.word);
   const scriptDependencies = any.simpleObject();
+  const logger = {info: () => undefined};
 
   beforeEach(() => {
     when(defineVcsHostDetails).calledWith(vcs, pathWithinParent).thenReturn(vcsDetails);
@@ -53,7 +54,10 @@ describe('package.json lifter', () => {
       .calledWith({...existingPackageContents, ...vcsDetails, scripts: liftedScripts})
       .thenReturn(config);
 
-    await liftPackage({dependencies, devDependencies, projectRoot, packageManager, vcs, pathWithinParent, scripts});
+    await liftPackage(
+      {dependencies, devDependencies, projectRoot, packageManager, vcs, pathWithinParent, scripts},
+      {logger}
+    );
 
     expect(writePackageJson).toHaveBeenCalledWith({projectRoot, config});
     expect(processDependencies).toHaveBeenCalledWith({
@@ -61,7 +65,7 @@ describe('package.json lifter', () => {
       devDependencies,
       projectRoot,
       packageManager
-    });
+    }, {logger});
   });
 
   it('should update keywords if tags are provided', async () => {
@@ -73,7 +77,7 @@ describe('package.json lifter', () => {
       .calledWith({...existingPackageContents, ...vcsDetails, scripts: liftedScripts, keywords: tags})
       .thenReturn(config);
 
-    await liftPackage({dependencies, projectRoot, packageManager, vcs, pathWithinParent, scripts, tags});
+    await liftPackage({dependencies, projectRoot, packageManager, vcs, pathWithinParent, scripts, tags}, {logger});
 
     expect(writePackageJson).toHaveBeenCalledWith({projectRoot, config});
   });
@@ -93,7 +97,7 @@ describe('package.json lifter', () => {
       })
       .thenReturn(config);
 
-    await liftPackage({dependencies, projectRoot, packageManager, vcs, pathWithinParent, scripts, tags});
+    await liftPackage({dependencies, projectRoot, packageManager, vcs, pathWithinParent, scripts, tags}, {logger});
 
     expect(writePackageJson).toHaveBeenCalledWith({projectRoot, config});
   });

package/src/package/scaffolder.js

@@ -1,4 +1,3 @@
-import {info} from '@travi/cli-messages';
 import {writePackageJson} from '@form8ion/javascript-core';
 
 import buildPackageName from './package-name.js';
@@ -12,8 +11,8 @@ export default async function scaffoldPackage({
   license,
   author,
   description
-}) {
-  info('Configuring package.json');
+}, {logger}) {
+  logger.info('Configuring package.json');
 
   const packageName = buildPackageName(projectName, scope);
 

package/src/package/scaffolder.test.js

@@ -13,6 +13,8 @@ vi.mock('./package-name.js');
 vi.mock('./details.js');
 
 describe('package scaffolder', () => {
+  const logger = {info: () => undefined};
+
   it('should create the package file', async () => {
     const projectName = any.string();
     const packageName = any.string();
@@ -40,7 +42,7 @@ describe('package scaffolder', () => {
       license,
       author,
       description
-    })).toEqual({packageName});
+    }, {logger})).toEqual({packageName});
     expect(writePackageJson).toHaveBeenCalledWith({projectRoot, config: packageDetails});
   });
 });

package/src/project-type/application/scaffolder.js

@@ -1,10 +1,9 @@
 import {mergeIntoExistingPackageJson} from '@form8ion/javascript-core';
-import {info} from '@travi/cli-messages';
 
 const defaultBuildDirectory = 'public';
 
-export default async function scaffoldApplication({projectRoot}) {
-  info('Scaffolding Application Details');
+export default async function scaffoldApplication({projectRoot}, {logger}) {
+  logger.info('Scaffolding Application Details');
 
   await mergeIntoExistingPackageJson({projectRoot, config: {private: true}});
 

package/src/project-type/application/scaffolder.test.js

@@ -1,6 +1,6 @@
 import {mergeIntoExistingPackageJson} from '@form8ion/javascript-core';
 
-import {afterEach, describe, expect, it, vi} from 'vitest';
+import {describe, expect, it, vi} from 'vitest';
 import any from '@travi/any';
 
 import scaffoldApplication from './scaffolder.js';
@@ -8,15 +8,13 @@ import scaffoldApplication from './scaffolder.js';
 vi.mock('@form8ion/javascript-core');
 
 describe('application project-type scaffolder', () => {
-  afterEach(() => {
-    vi.clearAllMocks();
-  });
+  const logger = {info: () => undefined};
 
   it('should scaffold the details specific to an application project-type', async () => {
     const projectRoot = any.string();
     const buildDirectory = 'public';
 
-    expect(await scaffoldApplication({projectRoot})).toEqual({
+    expect(await scaffoldApplication({projectRoot}, {logger})).toEqual({
       scripts: {
         clean: `rimraf ./${buildDirectory}`,
         start: `node ./${buildDirectory}/index.js`,

package/src/project-type/monorepo/scaffolder.js

@@ -1,8 +1,7 @@
 import {mergeIntoExistingPackageJson} from '@form8ion/javascript-core';
-import {info} from '@travi/cli-messages';
 
-export default async function scaffoldMonorepo({projectRoot}) {
-  info('Scaffolding Monorepo Details');
+export default async function scaffoldMonorepo({projectRoot}, {logger}) {
+  logger.info('Scaffolding Monorepo Details');
 
   await mergeIntoExistingPackageJson({projectRoot, config: {private: true}});
 

package/src/project-type/monorepo/scaffolder.test.js

@@ -1,6 +1,6 @@
 import {mergeIntoExistingPackageJson} from '@form8ion/javascript-core';
 
-import {describe, vi, it, expect, afterEach} from 'vitest';
+import {describe, expect, it, vi} from 'vitest';
 import any from '@travi/any';
 
 import scaffoldMonorepo from './scaffolder.js';
@@ -9,13 +9,10 @@ vi.mock('@form8ion/javascript-core');
 
 describe('monorepo project-type scaffolder', () => {
   const projectRoot = any.string();
-
-  afterEach(() => {
-    vi.clearAllMocks();
-  });
+  const logger = {info: () => undefined};
 
   it('should scaffold the monorepo specific project details', async () => {
-    expect(await scaffoldMonorepo({projectRoot})).toEqual({
+    expect(await scaffoldMonorepo({projectRoot}, {logger})).toEqual({
       nextSteps: [{
         summary: 'Add packages to your new monorepo',
         description: 'Leverage [@form8ion/add-package-to-monorepo](https://npm.im/@form8ion/add-package-to-monorepo)'

package/src/project-type/package/scaffolder.js

@@ -1,5 +1,4 @@
 import deepmerge from 'deepmerge';
-import {info} from '@travi/cli-messages';
 import {dialects, mergeIntoExistingPackageJson} from '@form8ion/javascript-core';
 
 import determinePackageAccessLevelFromProjectVisibility from '../publishable/access-level.js';
@@ -19,8 +18,8 @@ export default async function scaffoldPackageProjectType({
   dialect,
   provideExample,
   publishRegistry
-}) {
-  info('Scaffolding Package Details');
+}, {logger}) {
+  logger.info('Scaffolding Package Details');
 
   const packageAccessLevel = determinePackageAccessLevelFromProjectVisibility({projectVisibility: visibility});
   const [detailsForBuild, publishableResults] = await Promise.all([

package/src/project-type/package/scaffolder.test.js

@@ -35,6 +35,7 @@ describe('package project-type scaffolder', () => {
   const documentation = any.simpleObject();
   const decisions = any.simpleObject();
   const buildDetailsResults = any.simpleObject();
+  const logger = {info: () => undefined};
 
   beforeEach(() => {
     when(documentationScaffolder.default)
@@ -68,7 +69,7 @@ describe('package project-type scaffolder', () => {
       decisions,
       dialect,
       provideExample
-    })).toEqual({
+    }, {logger})).toEqual({
       ...publishableResults,
       ...buildDetailsResults,
       documentation,
@@ -113,7 +114,7 @@ describe('package project-type scaffolder', () => {
       packageBundlers,
       decisions,
       provideExample
-    })).toEqual({
+    }, {logger})).toEqual({
       ...publishableResults,
       ...buildDetailsResults,
       documentation,
@@ -153,7 +154,7 @@ describe('package project-type scaffolder', () => {
       decisions,
       dialect,
       provideExample
-    })).toEqual({
+    }, {logger})).toEqual({
       ...publishableResults,
       ...buildDetailsResults,
       documentation,
@@ -199,7 +200,7 @@ describe('package project-type scaffolder', () => {
       packageBundlers,
       dialect,
       provideExample
-    })).toEqual({
+    }, {logger})).toEqual({
       ...publishableResults,
       ...buildDetailsResults,
       documentation,
@@ -239,7 +240,7 @@ describe('package project-type scaffolder', () => {
       dialect,
       provideExample,
       packageBundlers
-    });
+    }, {logger});
 
     expect(mergeIntoExistingPackageJson).toHaveBeenCalledWith({
       projectRoot,

package/src/project-type/publishable/access-level.js

@@ -1,3 +1,3 @@
 export default function mapProjectVisibility({projectVisibility}) {
-  return 'Public' === projectVisibility ? 'public' : 'restricted';
+  return 'OSS' === projectVisibility ? 'public' : 'restricted';
 }

package/src/project-type/publishable/access-level.test.js

@@ -1,13 +1,15 @@
 import {describe, expect, it} from 'vitest';
+import any from '@travi/any';
 
 import determinePackageAccessLevelFromProjectVisibility from './access-level.js';
 
 describe('package access level', () => {
-  it('should return `public` when project visibility is `Public`', () => {
-    expect(determinePackageAccessLevelFromProjectVisibility({projectVisibility: 'Public'})).toEqual('public');
+  it('should return `public` when project visibility is `OSS`', () => {
+    expect(determinePackageAccessLevelFromProjectVisibility({projectVisibility: 'OSS'})).toEqual('public');
   });
 
-  it('should return `restricted` when the project visibility is `Private`', () => {
-    expect(determinePackageAccessLevelFromProjectVisibility({projectVisibility: 'Private'})).toEqual('restricted');
+  it('should return `restricted` when the project visibility is `ISS` or `CS`', () => {
+    expect(determinePackageAccessLevelFromProjectVisibility({projectVisibility: any.fromList(['ISS', 'CS'])}))
+      .toEqual('restricted');
   });
 });

package/src/project-type/scaffolder.js

@@ -18,7 +18,7 @@ export default async function scaffoldProjectType({
   dialect,
   provideExample,
   publishRegistry
-}) {
+}, {logger}) {
   switch (projectType) {
     case projectTypes.PACKAGE:
       return scaffoldPackageType({
@@ -33,9 +33,9 @@ export default async function scaffoldProjectType({
         dialect,
         provideExample,
         publishRegistry
-      });
+      }, {logger});
     case projectTypes.APPLICATION:
-      return scaffoldApplicationType({projectRoot});
+      return scaffoldApplicationType({projectRoot}, {logger});
     case projectTypes.CLI:
       return scaffoldCliType({
         visibility,
@@ -46,7 +46,7 @@ export default async function scaffoldProjectType({
         packageBundlers
       });
     case projectTypes.MONOREPO:
-      return scaffoldMonorepoType({projectRoot});
+      return scaffoldMonorepoType({projectRoot}, {logger});
     case 'Other':
       return {};
     default:

package/src/project-type/scaffolder.test.js

@@ -1,6 +1,6 @@
 import {projectTypes} from '@form8ion/javascript-core';
 
-import {afterEach, describe, expect, it, vi} from 'vitest';
+import {describe, expect, it, vi} from 'vitest';
 import any from '@travi/any';
 import {when} from 'vitest-when';
 
@@ -27,10 +27,7 @@ describe('project-type scaffolder', () => {
   const dialect = any.word();
   const provideExample = any.boolean();
   const packageBundlers = any.simpleObject();
-
-  afterEach(() => {
-    vi.clearAllMocks();
-  });
+  const logger = {info: () => undefined};
 
   it('should apply the package-type scaffolder when the project-type is `Package`', async () => {
     const scope = any.word();
@@ -46,7 +43,7 @@ describe('project-type scaffolder', () => {
       dialect,
       provideExample,
       publishRegistry
-    }).thenResolve(results);
+    }, {logger}).thenResolve(results);
 
     expect(await projectTypeScaffolder({
       projectType: projectTypes.PACKAGE,
@@ -61,11 +58,11 @@ describe('project-type scaffolder', () => {
       dialect,
       provideExample,
       publishRegistry
-    })).toEqual(results);
+    }, {logger})).toEqual(results);
   });
 
   it('should apply the application-type scaffolder when the project-type is `Application`', async () => {
-    when(scaffoldApplicationType).calledWith({projectRoot}).thenResolve(results);
+    when(scaffoldApplicationType).calledWith({projectRoot}, {logger}).thenResolve(results);
 
     expect(await projectTypeScaffolder({
       projectType: projectTypes.APPLICATION,
@@ -75,7 +72,7 @@ describe('project-type scaffolder', () => {
       packageManager,
       decisions,
       visibility
-    })).toEqual(results);
+    }, {logger})).toEqual(results);
   });
 
   it('should apply the cli-type scaffolder when the project-type is `CLI`', async () => {
@@ -91,24 +88,26 @@ describe('project-type scaffolder', () => {
       publishRegistry,
       decisions,
       packageBundlers
-    })).toEqual(results);
+    }, {logger})).toEqual(results);
   });
 
   it('should apply the monorepo-type scaffolder when the project-type is `Monorepo`', async () => {
-    when(scaffoldMonorepoType).calledWith({projectRoot}).thenResolve(results);
+    when(scaffoldMonorepoType).calledWith({projectRoot}, {logger}).thenResolve(results);
 
-    expect(await projectTypeScaffolder({projectRoot, projectType: projectTypes.MONOREPO, packageManager, decisions}))
-      .toEqual(results);
+    expect(await projectTypeScaffolder(
+      {projectRoot, projectType: projectTypes.MONOREPO, packageManager, decisions},
+      {logger}
+    )).toEqual(results);
   });
 
   it('should not throw an error when the project-type is `Other`', async () => {
-    expect(await projectTypeScaffolder({projectType: 'Other'})).toEqual({});
+    expect(await projectTypeScaffolder({projectType: 'Other'}, {logger})).toEqual({});
   });
 
   it('should throw an error for an unknown project-type', async () => {
     const projectType = any.word();
 
-    await expect(() => projectTypeScaffolder({projectType}))
-      .rejects.toThrowError(`The project-type of ${projectType} is invalid`);
+    await expect(() => projectTypeScaffolder({projectType}, {logger}))
+      .rejects.toThrow(`The project-type of ${projectType} is invalid`);
   });
 });

package/src/prompts/conditionals.js

@@ -16,7 +16,7 @@ export function projectIsApplication(answers) {
 }
 
 function packageShouldBeScoped(visibility, answers) {
-  return 'Private' === visibility || answers[questionNames.SHOULD_BE_SCOPED];
+  return ['ISS', 'CS'].includes(visibility) || answers[questionNames.SHOULD_BE_SCOPED];
 }
 
 function willBePublishedToNpm(answers) {

package/src/prompts/conditionals.test.js

@@ -33,8 +33,15 @@ describe('javascript prompt conditionals', () => {
       })).toBe(true);
     });
 
-    it('should present a scope prompt when a package is private, because they must be scoped', () => {
-      expect(scopePromptShouldBePresentedFactory('Private')({
+    it('should present a scope prompt when a package is inner source, because they must be scoped', () => {
+      expect(scopePromptShouldBePresentedFactory('ISS')({
+        [questionNames.SHOULD_BE_SCOPED]: false,
+        [questionNames.PROJECT_TYPE]: projectTypes.PACKAGE
+      })).toBe(true);
+    });
+
+    it('should present a scope prompt when a package is closed source, because they must be scoped', () => {
+      expect(scopePromptShouldBePresentedFactory('CS')({
         [questionNames.SHOULD_BE_SCOPED]: false,
         [questionNames.PROJECT_TYPE]: projectTypes.PACKAGE
       })).toBe(true);
@@ -47,15 +54,29 @@ describe('javascript prompt conditionals', () => {
       })).toBe(true);
     });
 
-    it('should present a scope prompt when a CLI is private, because they must be scoped', () => {
-      expect(scopePromptShouldBePresentedFactory('Private')({
+    it('should present a scope prompt when a CLI is closed source, because they must be scoped', () => {
+      expect(scopePromptShouldBePresentedFactory('CS')({
         [questionNames.SHOULD_BE_SCOPED]: false,
         [questionNames.PROJECT_TYPE]: projectTypes.CLI
       })).toBe(true);
     });
 
-    it('should not present a scope prompt when an application is private', () => {
-      expect(scopePromptShouldBePresentedFactory('Private')({
+    it('should present a scope prompt when a CLI is inner source, because they must be scoped', () => {
+      expect(scopePromptShouldBePresentedFactory('ISS')({
+        [questionNames.SHOULD_BE_SCOPED]: false,
+        [questionNames.PROJECT_TYPE]: projectTypes.CLI
+      })).toBe(true);
+    });
+
+    it('should not present a scope prompt when an application is closed source', () => {
+      expect(scopePromptShouldBePresentedFactory('CS')({
+        [questionNames.SHOULD_BE_SCOPED]: false,
+        [questionNames.PROJECT_TYPE]: projectTypes.APPLICATION
+      })).toBe(false);
+    });
+
+    it('should not present a scope prompt when an application is inner source', () => {
+      expect(scopePromptShouldBePresentedFactory('ISS')({
         [questionNames.SHOULD_BE_SCOPED]: false,
         [questionNames.PROJECT_TYPE]: projectTypes.APPLICATION
       })).toBe(false);

package/src/prompts/questions.js

@@ -1,7 +1,6 @@
 import {packageManagers, projectTypes} from '@form8ion/javascript-core';
 import {prompt as promptWithInquirer} from '@form8ion/overridable-prompts';
 import {questionNames as commonQuestionNames, questions as commonQuestions} from '@travi/language-scaffolder-prompts';
-import {warn} from '@travi/cli-messages';
 
 import {execa} from 'execa';
 import npmConfFactory from '../../thirdparty-wrappers/npm-conf.js';
@@ -43,7 +42,8 @@ export async function prompt(
   vcs,
   decisions,
   configs,
-  pathWithinParent
+  pathWithinParent,
+  {logger}
 ) {
   const npmConf = npmConfFactory();
 
@@ -52,7 +52,7 @@ export async function prompt(
     maybeLoggedInNpmUsername = (await execa('npm', ['whoami'])).stdout;
   } catch (failedExecutionResult) {
     if (!decisions[questionNames.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.');
     }
   }
@@ -101,7 +101,7 @@ export async function prompt(
       choices: [...Object.values(projectTypes), 'Other'],
       default: projectTypes.PACKAGE
     },
-    ...'Private' === visibility ? [] : [{
+    ...['ISS', 'CS'].includes(visibility) ? [] : [{
       name: questionNames.SHOULD_BE_SCOPED,
       message: 'Should this package be scoped?',
       type: 'confirm',

package/src/prompts/questions.test.js

@@ -59,6 +59,7 @@ describe('prompts', () => {
     [questionNames.DIALECT]: dialect,
     [questionNames.PROVIDE_EXAMPLE]: provideExample
   };
+  const logger = {info: () => undefined, warn: () => undefined};
 
   beforeEach(() => {
     when(commonPrompts.questions)
@@ -166,7 +167,7 @@ describe('prompts', () => {
       ], decisions)
       .thenResolve({...answers, [questionNames.CONFIGURE_LINTING]: any.word()});
 
-    expect(await prompt(ciServices, hosts, visibility, vcs, decisions, configs)).toEqual({
+    expect(await prompt(ciServices, hosts, visibility, vcs, decisions, configs, undefined, {logger})).toEqual({
       tests,
       projectType,
       ci,
@@ -188,7 +189,7 @@ describe('prompts', () => {
     when(execa).calledWith('npm', ['whoami']).thenResolve({stdout: npmUser});
     prompts.prompt.mockResolvedValue({...answers, [questionNames.CONFIGURE_LINTING]: false});
 
-    expect(await prompt(ciServices, {}, visibility, vcs, decisions)).toEqual({
+    expect(await prompt(ciServices, {}, visibility, vcs, decisions, undefined, undefined, {logger})).toEqual({
       tests,
       projectType,
       ci,
@@ -211,13 +212,13 @@ describe('prompts', () => {
       .thenReturn(commonQuestions);
     prompts.prompt.mockResolvedValue(answers);
 
-    await prompt(ciServices, {}, 'Private', vcs, null, null, pathWithinParent);
+    await prompt(ciServices, {}, 'CS', vcs, null, null, pathWithinParent, {logger});
 
     const [questions] = prompts.prompt.mock.lastCall;
     expect(questions.filter(question => questionNames.NODE_VERSION_CATEGORY === question.name).length).toEqual(0);
   });
 
-  it('should not ask whether private packages should be scoped', async () => {
+  it('should not ask whether closed source packages should be scoped', async () => {
     when(execa).calledWith('npm', ['whoami']).thenResolve({stdout: any.word()});
     npmConfFactory.mockReturnValue({get: () => undefined});
     when(commonPrompts.questions)
@@ -225,7 +226,21 @@ describe('prompts', () => {
       .thenReturn(commonQuestions);
     prompts.prompt.mockResolvedValue(answers);
 
-    await prompt(ciServices, {}, 'Private', vcs, null, null, pathWithinParent);
+    await prompt(ciServices, {}, 'CS', vcs, null, null, pathWithinParent, {logger});
+
+    const [questions] = prompts.prompt.mock.lastCall;
+    expect(questions.filter(question => questionNames.SHOULD_BE_SCOPED === question.name).length).toEqual(0);
+  });
+
+  it('should not ask whether inner source packages should be scoped', async () => {
+    when(execa).calledWith('npm', ['whoami']).thenResolve({stdout: any.word()});
+    npmConfFactory.mockReturnValue({get: () => undefined});
+    when(commonPrompts.questions)
+      .calledWith({vcs, ciServices, pathWithinParent})
+      .thenReturn(commonQuestions);
+    prompts.prompt.mockResolvedValue(answers);
+
+    await prompt(ciServices, {}, 'ISS', vcs, null, null, pathWithinParent, {logger});
 
     const [questions] = prompts.prompt.mock.lastCall;
     expect(questions.filter(question => questionNames.SHOULD_BE_SCOPED === question.name).length).toEqual(0);
@@ -239,7 +254,7 @@ describe('prompts', () => {
       .thenReturn(commonQuestions);
     prompts.prompt.mockResolvedValue(answers);
 
-    await prompt(ciServices, {}, 'Public', vcs, {}, null, pathWithinParent);
+    await prompt(ciServices, {}, 'OSS', vcs, {}, null, pathWithinParent, {logger});
 
     const [questions] = prompts.prompt.mock.lastCall;
     expect(questions.filter(question => questionNames.SHOULD_BE_SCOPED === question.name).length).toEqual(1);

package/src/prompts/validators.js

@@ -1,7 +1,12 @@
 export function scope(visibility) {
   return input => {
-    if (!input && 'Private' === visibility) {
-      return 'Private packages must be scoped (https://docs.npmjs.com/private-modules/intro#setting-up-your-package)';
+    if (!input) {
+      if ('CS' === visibility) {
+        return 'Closed source packages must be scoped';
+      }
+      if ('ISS' === visibility) {
+        return 'Inner source packages must be scoped';
+      }
     }
 
     return true;

package/src/prompts/validators.test.js

@@ -1,19 +1,22 @@
 import {expect, it, describe} from 'vitest';
+import any from '@travi/any';
 
 import {scope} from './validators.js';
 
 describe('question validators', () => {
-  it('should require a scope for private project', () => {
-    expect(scope('Private')()).toEqual(
-      'Private packages must be scoped (https://docs.npmjs.com/private-modules/intro#setting-up-your-package)'
-    );
+  it('should require a scope for a closed-source project', () => {
+    expect(scope('CS')()).toEqual('Closed source packages must be scoped');
+  });
+
+  it('should require a scope for an inner-source project', () => {
+    expect(scope('ISS')()).toEqual('Inner source packages must be scoped');
   });
 
   it('it should consider a provided value to be a valid answer to private projects', () => {
-    expect(scope('Public')()).toBe(true);
+    expect(scope('OSS')(any.word())).toBe(true);
   });
 
   it('it should consider an empty value to be a valid answer to public projects', () => {
-    expect(scope('Public')()).toBe(true);
+    expect(scope('OSS')()).toBe(true);
   });
 });