Use import/export type

change/beachball-68ef8117-b338-4a39-996b-803a38057146.json

@@ -0,0 +1,7 @@
+{
+  "type": "minor",
+  "comment": "Use import/export type",
+  "packageName": "beachball",
+  "email": "elcraig@microsoft.com",
+  "dependentChangeType": "patch"
+}

src/__e2e__/bump.test.ts

@@ -4,10 +4,10 @@ import path from 'path';
 import { generateChangeFiles, getChangeFiles } from '../__fixtures__/changeFiles';
 import { readChangelogJson } from '../__fixtures__/changelog';
 import { initMockLogs } from '../__fixtures__/mockLogs';
-import { RepoFixture, RepositoryFactory } from '../__fixtures__/repositoryFactory';
+import { type RepoFixture, RepositoryFactory } from '../__fixtures__/repositoryFactory';
 import { bump } from '../commands/bump';
 import { getPackageInfos } from '../monorepo/getPackageInfos';
-import { BeachballOptions, HooksOptions } from '../types/BeachballOptions';
+import type { BeachballOptions, HooksOptions } from '../types/BeachballOptions';
 import type { Repository } from '../__fixtures__/repository';
 import { getDefaultOptions } from '../options/getDefaultOptions';
 import type { PackageJson } from '../types/PackageInfo';

src/__e2e__/change.test.ts

@@ -3,14 +3,14 @@ import fs from 'fs-extra';
 import type prompts from 'prompts';
 import { getChangeFiles } from '../__fixtures__/changeFiles';
 import { initMockLogs } from '../__fixtures__/mockLogs';
-import { RepoFixture, RepositoryFactory } from '../__fixtures__/repositoryFactory';
+import { type RepoFixture, RepositoryFactory } from '../__fixtures__/repositoryFactory';
 import { change } from '../commands/change';
-import { BeachballOptions } from '../types/BeachballOptions';
+import type { BeachballOptions } from '../types/BeachballOptions';
 import { defaultBranchName } from '../__fixtures__/gitDefaults';
 import { MockStdout } from '../__fixtures__/mockStdout';
 import { MockStdin } from '../__fixtures__/mockStdin';
 import type { ChangeFileInfo, ChangeInfoMultiple } from '../types/ChangeInfo';
-import { Repository } from '../__fixtures__/repository';
+import type { Repository } from '../__fixtures__/repository';
 import { getDefaultOptions } from '../options/getDefaultOptions';
 
 // prompts writes to stdout (not console) in a way that can't really be mocked with spies,

src/__e2e__/getChangedPackages.test.ts

@@ -2,7 +2,7 @@ import { describe, expect, it, afterEach } from '@jest/globals';
 import { defaultBranchName } from '../__fixtures__/gitDefaults';
 import { RepositoryFactory } from '../__fixtures__/repositoryFactory';
 import { getPackageInfos } from '../monorepo/getPackageInfos';
-import { BeachballOptions } from '../types/BeachballOptions';
+import type { BeachballOptions } from '../types/BeachballOptions';
 import { getChangedPackages } from '../changefile/getChangedPackages';
 import { initMockLogs } from '../__fixtures__/mockLogs';
 import { generateChangeFiles } from '../__fixtures__/changeFiles';

src/__e2e__/publishE2E.test.ts

@@ -6,11 +6,11 @@ import { generateChangeFiles } from '../__fixtures__/changeFiles';
 import { defaultBranchName, defaultRemoteBranchName } from '../__fixtures__/gitDefaults';
 import { initMockLogs } from '../__fixtures__/mockLogs';
 import { npmShow } from '../__fixtures__/npmShow';
-import { Repository } from '../__fixtures__/repository';
-import { PackageJsonFixture, RepositoryFactory } from '../__fixtures__/repositoryFactory';
+import type { Repository } from '../__fixtures__/repository';
+import { type PackageJsonFixture, RepositoryFactory } from '../__fixtures__/repositoryFactory';
 import { publish } from '../commands/publish';
 import { getDefaultOptions } from '../options/getDefaultOptions';
-import { BeachballOptions } from '../types/BeachballOptions';
+import type { BeachballOptions } from '../types/BeachballOptions';
 import { _mockNpmPublish, initNpmMock } from '../__fixtures__/mockNpm';
 import type { PackageJson } from '../types/PackageInfo';
 

src/__e2e__/publishGit.test.ts

@@ -3,13 +3,13 @@ import fs from 'fs-extra';
 import { defaultRemoteBranchName } from '../__fixtures__/gitDefaults';
 import { generateChangeFiles, getChangeFiles } from '../__fixtures__/changeFiles';
 import { initMockLogs } from '../__fixtures__/mockLogs';
-import { Repository } from '../__fixtures__/repository';
+import type { Repository } from '../__fixtures__/repository';
 import { RepositoryFactory } from '../__fixtures__/repositoryFactory';
 import { bumpAndPush } from '../publish/bumpAndPush';
 import { publish } from '../commands/publish';
 import { gatherBumpInfo } from '../bump/gatherBumpInfo';
-import { BeachballOptions } from '../types/BeachballOptions';
-import { ChangeFileInfo } from '../types/ChangeInfo';
+import type { BeachballOptions } from '../types/BeachballOptions';
+import type { ChangeFileInfo } from '../types/ChangeInfo';
 import { getPackageInfos } from '../monorepo/getPackageInfos';
 import { getDefaultOptions } from '../options/getDefaultOptions';
 import type { PackageJson } from '../types/PackageInfo';

src/__e2e__/publishRegistry.test.ts

@@ -3,11 +3,11 @@ import { defaultRemoteBranchName } from '../__fixtures__/gitDefaults';
 import { generateChangeFiles } from '../__fixtures__/changeFiles';
 import { initMockLogs } from '../__fixtures__/mockLogs';
 import { npmShow } from '../__fixtures__/npmShow';
-import { Repository } from '../__fixtures__/repository';
+import type { Repository } from '../__fixtures__/repository';
 import { RepositoryFactory } from '../__fixtures__/repositoryFactory';
 import { publish } from '../commands/publish';
 import { getDefaultOptions } from '../options/getDefaultOptions';
-import { BeachballOptions } from '../types/BeachballOptions';
+import type { BeachballOptions } from '../types/BeachballOptions';
 import { initNpmMock } from '../__fixtures__/mockNpm';
 
 // Spawning actual npm to run commands against a fake registry is extremely slow, so mock it for

src/__e2e__/syncE2E.test.ts

@@ -2,13 +2,13 @@ import { describe, expect, it, afterEach, jest } from '@jest/globals';
 import fs from 'fs-extra';
 import { defaultRemoteBranchName } from '../__fixtures__/gitDefaults';
 import { initMockLogs } from '../__fixtures__/mockLogs';
-import { Repository } from '../__fixtures__/repository';
+import type { Repository } from '../__fixtures__/repository';
 import { RepositoryFactory } from '../__fixtures__/repositoryFactory';
 import { sync } from '../commands/sync';
 import { getPackageInfos } from '../monorepo/getPackageInfos';
-import { packagePublish } from '../packageManager/packagePublish';
+import type { packagePublish } from '../packageManager/packagePublish';
 import { getDefaultOptions } from '../options/getDefaultOptions';
-import { BeachballOptions } from '../types/BeachballOptions';
+import type { BeachballOptions } from '../types/BeachballOptions';
 import { initNpmMock } from '../__fixtures__/mockNpm';
 
 // Spawning actual npm to run commands against a fake registry is extremely slow, so mock it for

src/__e2e__/validate.test.ts

@@ -1,7 +1,7 @@
-import { describe, expect, it, afterAll, jest, beforeAll, afterEach } from '@jest/globals';
+import { describe, expect, it, afterAll, type jest, beforeAll, afterEach } from '@jest/globals';
 import { defaultBranchName } from '../__fixtures__/gitDefaults';
 import { RepositoryFactory } from '../__fixtures__/repositoryFactory';
-import { BeachballOptions } from '../types/BeachballOptions';
+import type { BeachballOptions } from '../types/BeachballOptions';
 import { initMockLogs } from '../__fixtures__/mockLogs';
 import { validate } from '../validation/validate';
 import type { Repository } from '../__fixtures__/repository';

src/__fixtures__/changeFiles.ts

@@ -2,7 +2,7 @@ import fs from 'fs';
 import path from 'path';
 import { writeChangeFiles } from '../changefile/writeChangeFiles';
 import { getChangePath } from '../paths';
-import { ChangeFileInfo, ChangeType } from '../types/ChangeInfo';
+import type { ChangeFileInfo, ChangeType } from '../types/ChangeInfo';
 import type { BeachballOptions } from '../types/BeachballOptions';
 
 /** Change file with `packageName` required and other props optional */

src/__fixtures__/changelog.ts

@@ -1,6 +1,6 @@
 import fs from 'fs-extra';
 import path from 'path';
-import { ChangelogJson } from '../types/ChangeLog';
+import type { ChangelogJson } from '../types/ChangeLog';
 import { markerComment } from '../changelog/renderChangelog';
 
 /** Placeholder commit as replaced by cleanChangelogJson */

src/__fixtures__/gitDefaults.ts

@@ -1,4 +1,4 @@
-import { git, gitFailFast } from 'workspace-tools';
+import { type git, gitFailFast } from 'workspace-tools';
 
 export const defaultBranchName = 'master';
 export const defaultRemoteName = 'origin';

src/__fixtures__/mockNpm.test.ts

@@ -4,8 +4,8 @@
 
 import { afterAll, afterEach, beforeAll, describe, expect, it, jest } from '@jest/globals';
 import fs from 'fs-extra';
-import { NpmResult, npm } from '../packageManager/npm';
-import { PackageJson } from '../types/PackageInfo';
+import { type NpmResult, npm } from '../packageManager/npm';
+import type { PackageJson } from '../types/PackageInfo';
 import { initNpmMock, _makeRegistryData, _mockNpmPublish, _mockNpmShow, type MockNpmResult } from './mockNpm';
 
 jest.mock('fs-extra');

src/__fixtures__/mockNpm.ts

@@ -1,11 +1,11 @@
-import { afterAll, afterEach, beforeAll, jest } from '@jest/globals';
+import { afterAll, afterEach, beforeAll, type jest } from '@jest/globals';
 import fs from 'fs-extra';
 import _ from 'lodash';
 import path from 'path';
 import semver from 'semver';
-import { NpmShowResult } from '../packageManager/listPackageVersions';
+import type { NpmShowResult } from '../packageManager/listPackageVersions';
 import { npm, NpmResult } from '../packageManager/npm';
-import { PackageJson } from '../types/PackageInfo';
+import type { PackageJson } from '../types/PackageInfo';
 import type { PackageManagerOptions } from '../packageManager/packageManager';
 
 /** Published versions and dist-tags for a package */

src/__fixtures__/npmShow.ts

@@ -1,9 +1,9 @@
 import { expect } from '@jest/globals';
 import os from 'os';
 import { env } from '../env';
-import { NpmShowResult } from '../packageManager/listPackageVersions';
+import type { NpmShowResult } from '../packageManager/listPackageVersions';
 import { npm } from '../packageManager/npm';
-import { Registry } from './registry';
+import type { Registry } from './registry';
 
 /**
  * Runs `npm show <packageName>` on the fake registry and returns the result.

src/__fixtures__/packageInfos.ts

@@ -1,6 +1,6 @@
 import _ from 'lodash';
-import { BeachballOptions } from '../types/BeachballOptions';
-import { PackageInfo, PackageInfos } from '../types/PackageInfo';
+import type { BeachballOptions } from '../types/BeachballOptions';
+import type { PackageInfo, PackageInfos } from '../types/PackageInfo';
 import { getDefaultOptions } from '../options/getDefaultOptions';
 
 const defaultOptions = getDefaultOptions();

src/__fixtures__/repositoryFactory.ts

@@ -1,8 +1,8 @@
 import fs from 'fs-extra';
 import path from 'path';
-import { PackageJson } from '../types/PackageInfo';
-import { Repository, RepositoryCloneOptions } from './repository';
-import { BeachballOptions } from '../types/BeachballOptions';
+import type { PackageJson } from '../types/PackageInfo';
+import { Repository, type RepositoryCloneOptions } from './repository';
+import type { BeachballOptions } from '../types/BeachballOptions';
 import { tmpdir } from './tmpdir';
 import { gitFailFast } from 'workspace-tools';
 import { setDefaultBranchName } from './gitDefaults';

src/__functional__/changefile/readChangeFiles.test.ts

@@ -6,10 +6,10 @@ import { RepositoryFactory } from '../../__fixtures__/repositoryFactory';
 
 import { getPackageInfos } from '../../monorepo/getPackageInfos';
 import { readChangeFiles } from '../../changefile/readChangeFiles';
-import { BeachballOptions } from '../../types/BeachballOptions';
+import type { BeachballOptions } from '../../types/BeachballOptions';
 import type { Repository } from '../../__fixtures__/repository';
 import { getDefaultOptions } from '../../options/getDefaultOptions';
-import { ChangeInfo } from '../../types/ChangeInfo';
+import type { ChangeInfo } from '../../types/ChangeInfo';
 
 describe('readChangeFiles', () => {
   let repositoryFactory: RepositoryFactory;

src/__functional__/git/fetch.test.ts

@@ -2,10 +2,10 @@ import { describe, expect, it, beforeAll, afterAll, jest, afterEach } from '@jes
 import * as workspaceTools from 'workspace-tools';
 import { RepositoryFactory } from '../../__fixtures__/repositoryFactory';
 import { initMockLogs } from '../../__fixtures__/mockLogs';
-import { Repository } from '../../__fixtures__/repository';
+import type { Repository } from '../../__fixtures__/repository';
 import { gitFetch } from '../../git/fetch';
 import { defaultBranchName, defaultRemoteName } from '../../__fixtures__/gitDefaults';
-import { GitProcessOutput } from 'workspace-tools';
+import type { GitProcessOutput } from 'workspace-tools';
 
 // required for `jest.spyOn('workspace-tools', git)` to work
 jest.mock('workspace-tools', () => {

src/__functional__/monorepo/getScopedPackages.test.ts

@@ -1,8 +1,8 @@
 import { describe, expect, it, beforeAll, afterAll } from '@jest/globals';
-import { Repository } from '../../__fixtures__/repository';
+import type { Repository } from '../../__fixtures__/repository';
 import { RepositoryFactory } from '../../__fixtures__/repositoryFactory';
 import { getScopedPackages } from '../../monorepo/getScopedPackages';
-import { PackageInfos } from '../../types/PackageInfo';
+import type { PackageInfos } from '../../types/PackageInfo';
 import { getPackageInfos } from '../../monorepo/getPackageInfos';
 
 describe('getScopedPackages', () => {

src/__functional__/options/getOptions.test.ts

@@ -1,6 +1,6 @@
 import { describe, expect, it, beforeAll, afterEach, afterAll } from '@jest/globals';
 import fs from 'fs-extra';
-import { Repository } from '../../__fixtures__/repository';
+import type { Repository } from '../../__fixtures__/repository';
 import { RepositoryFactory } from '../../__fixtures__/repositoryFactory';
 import { getOptions } from '../../options/getOptions';
 

src/__functional__/packageManager/packagePublish.test.ts

@@ -7,8 +7,8 @@ import { Registry } from '../../__fixtures__/registry';
 import { tmpdir } from '../../__fixtures__/tmpdir';
 import * as npmModule from '../../packageManager/npm';
 import { packagePublish } from '../../packageManager/packagePublish';
-import { PackageInfo } from '../../types/PackageInfo';
-import { npm, NpmResult } from '../../packageManager/npm';
+import type { PackageInfo } from '../../types/PackageInfo';
+import type { npm, NpmResult } from '../../packageManager/npm';
 import type { PackageOptions } from '../../types/BeachballOptions';
 
 const testTag = 'testbeachballtag';

src/__functional__/validation/areChangeFilesDeleted.test.ts

@@ -3,9 +3,9 @@ import fs from 'fs-extra';
 import { generateChangeFiles } from '../../__fixtures__/changeFiles';
 import { defaultRemoteBranchName } from '../../__fixtures__/gitDefaults';
 import { initMockLogs } from '../../__fixtures__/mockLogs';
-import { Repository } from '../../__fixtures__/repository';
+import type { Repository } from '../../__fixtures__/repository';
 import { RepositoryFactory } from '../../__fixtures__/repositoryFactory';
-import { BeachballOptions } from '../../types/BeachballOptions';
+import type { BeachballOptions } from '../../types/BeachballOptions';
 import { areChangeFilesDeleted } from '../../validation/areChangeFilesDeleted';
 import { getChangePath } from '../../paths';
 import { getDefaultOptions } from '../../options/getDefaultOptions';

src/__functional__/validation/isChangeFileNeeded.test.ts

@@ -1,10 +1,10 @@
 import { describe, expect, it, beforeAll, beforeEach, afterAll } from '@jest/globals';
 import { defaultRemoteBranchName, defaultRemoteName } from '../../__fixtures__/gitDefaults';
 import { initMockLogs } from '../../__fixtures__/mockLogs';
-import { Repository } from '../../__fixtures__/repository';
+import type { Repository } from '../../__fixtures__/repository';
 import { RepositoryFactory } from '../../__fixtures__/repositoryFactory';
 import { isChangeFileNeeded } from '../../validation/isChangeFileNeeded';
-import { BeachballOptions } from '../../types/BeachballOptions';
+import type { BeachballOptions } from '../../types/BeachballOptions';
 import { getPackageInfos } from '../../monorepo/getPackageInfos';
 import { getDefaultOptions } from '../../options/getDefaultOptions';
 

src/__tests__/bump/updateRelatedChangeType.test.ts

@@ -1,8 +1,8 @@
 import { describe, expect, it } from '@jest/globals';
 import { updateRelatedChangeType } from '../../bump/updateRelatedChangeType';
-import { BumpInfo } from '../../types/BumpInfo';
+import type { BumpInfo } from '../../types/BumpInfo';
 import _ from 'lodash';
-import { ChangeInfo, ChangeSet, ChangeType } from '../../types/ChangeInfo';
+import type { ChangeInfo, ChangeSet, ChangeType } from '../../types/ChangeInfo';
 import { makePackageInfos } from '../../__fixtures__/packageInfos';
 
 type DeepPartial<T> = {

src/__tests__/changefile/changeTypes.test.ts

@@ -1,6 +1,6 @@
 import { describe, expect, it } from '@jest/globals';
 import { getMaxChangeType, initializePackageChangeTypes } from '../../changefile/changeTypes';
-import { ChangeSet } from '../../types/ChangeInfo';
+import type { ChangeSet } from '../../types/ChangeInfo';
 
 describe('getMaxChangeType', () => {
   it('handles equal change types', () => {

src/__tests__/changefile/getDisallowedChangeTypes.test.ts

@@ -1,6 +1,6 @@
 import { describe, expect, it } from '@jest/globals';
 import { getDisallowedChangeTypes } from '../../changefile/getDisallowedChangeTypes';
-import { PackageGroups } from '../../types/PackageInfo';
+import type { PackageGroups } from '../../types/PackageInfo';
 import { makePackageInfos } from '../../__fixtures__/packageInfos';
 
 describe('getDisallowedChangeTypes', () => {

src/__tests__/changefile/getQuestionsForPackage.test.ts

@@ -1,7 +1,7 @@
 import { describe, expect, it, jest } from '@jest/globals';
-import prompts from 'prompts';
+import type prompts from 'prompts';
 import { getQuestionsForPackage } from '../../changefile/getQuestionsForPackage';
-import { ChangeFilePromptOptions } from '../../types/ChangeFilePrompt';
+import type { ChangeFilePromptOptions } from '../../types/ChangeFilePrompt';
 import { initMockLogs } from '../../__fixtures__/mockLogs';
 import { makePackageInfos } from '../../__fixtures__/packageInfos';
 import type { ChangeType } from '../../types/ChangeInfo';

src/__tests__/changefile/promptForChange.test.ts

@@ -1,7 +1,7 @@
 import { afterEach, beforeEach, describe, expect, it, jest } from '@jest/globals';
-import prompts from 'prompts';
+import type prompts from 'prompts';
 import { promptForChange } from '../../changefile/promptForChange';
-import { ChangeFilePromptOptions } from '../../types/ChangeFilePrompt';
+import type { ChangeFilePromptOptions } from '../../types/ChangeFilePrompt';
 import { initMockLogs } from '../../__fixtures__/mockLogs';
 import { MockStdin } from '../../__fixtures__/mockStdin';
 import { MockStdout } from '../../__fixtures__/mockStdout';

src/__tests__/changefile/promptForChange_promptForPackageChange.test.ts

@@ -1,5 +1,5 @@
 import { afterEach, beforeEach, describe, expect, it, jest } from '@jest/globals';
-import prompts from 'prompts';
+import type prompts from 'prompts';
 import { _promptForPackageChange } from '../../changefile/promptForChange';
 import { initMockLogs } from '../../__fixtures__/mockLogs';
 import { MockStdin } from '../../__fixtures__/mockStdin';

src/__tests__/changelog/getPackageChangelogs.test.ts

@@ -1,8 +1,8 @@
 import { describe, expect, it, jest } from '@jest/globals';
 import { getPackageChangelogs } from '../../changelog/getPackageChangelogs';
-import { BumpInfo } from '../../types/BumpInfo';
-import { ChangeFileInfo, ChangeSet } from '../../types/ChangeInfo';
-import { PackageInfos } from '../../types/PackageInfo';
+import type { BumpInfo } from '../../types/BumpInfo';
+import type { ChangeFileInfo, ChangeSet } from '../../types/ChangeInfo';
+import type { PackageInfos } from '../../types/PackageInfo';
 import { makePackageInfos } from '../../__fixtures__/packageInfos';
 
 const commit = 'deadbeef';

src/__tests__/changelog/mergeChangelogs.test.ts

@@ -1,7 +1,7 @@
 import { describe, expect, it } from '@jest/globals';
-import { PackageChangelog } from '../../types/ChangeLog';
+import type { PackageChangelog } from '../../types/ChangeLog';
 import { mergeChangelogs } from '../../changelog/mergeChangelogs';
-import { PackageInfo } from '../../types/PackageInfo';
+import type { PackageInfo } from '../../types/PackageInfo';
 
 describe('mergeChangelogs', () => {
   const primaryPackageInfo = {

src/__tests__/changelog/renderChangelog.test.ts

@@ -1,7 +1,7 @@
 import { describe, expect, it, jest } from '@jest/globals';
 import { initMockLogs } from '../../__fixtures__/mockLogs';
 import {
-  MarkdownChangelogRenderOptions,
+  type MarkdownChangelogRenderOptions,
   renderChangelog,
   markerComment,
   _trimPreviousLog,

src/__tests__/changelog/renderJsonChangelog.test.ts

@@ -1,5 +1,5 @@
 import { describe, expect, it } from '@jest/globals';
-import { ChangelogJson, PackageChangelog } from '../..';
+import type { ChangelogJson, PackageChangelog } from '../../types/ChangeLog';
 import { renderJsonChangelog } from '../../changelog/renderJsonChangelog';
 
 describe('renderJsonChangelog', () => {