npm-update-package 0.5.0 → 0.7.0

package/src/logger/createLogger.ts

@@ -1,6 +1,6 @@
 import { getLogger } from 'log4js'
-import type { LogLevel } from '../enums'
 import type { Logger } from './Logger'
+import type { LogLevel } from './LogLevel'
 
 // TODO: add test
 export const createLogger = (logLevel: LogLevel): Logger => {

package/src/logger/index.ts

@@ -1,2 +1,6 @@
 export { createLogger } from './createLogger'
 export type { Logger } from './Logger'
+export {
+  isLogLevel,
+  LogLevel
+} from './LogLevel'

package/src/main.ts

@@ -1,20 +1,27 @@
-import { BranchNameCreator } from './branch-name-creator'
-import { CommitMessageCreator } from './commit-message-creator'
 import {
+  BranchNameCreator,
+  CommitMessageCreator,
   Committer,
   Git
 } from './git'
 import {
   createGitHub,
   PullRequestCreator,
+  PullRequestTitleCreator,
   RemoteBranchExistenceChecker
 } from './github'
 import type { Logger } from './logger'
 import { Ncu } from './ncu'
 import type { Options } from './options'
-import { OutdatedPackageProcessor } from './outdated-package-processor'
-import { OutdatedPackagesProcessor } from './outdated-packages-processor'
+import {
+  PackageJsonParser,
+  PackageJsonReader
+} from './package-json'
 import { createPackageManager } from './package-manager'
+import {
+  OutdatedPackageProcessor,
+  OutdatedPackagesProcessor
+} from './processors'
 import { Terminal } from './terminal'
 
 // TODO: add test
@@ -27,7 +34,12 @@ export const main = async ({
 }): Promise<void> => {
   logger.debug(`options=${JSON.stringify(options)}`)
 
-  const ncu = new Ncu()
+  const packageJsonParser = new PackageJsonParser(logger)
+  const packageJsonReader = new PackageJsonReader({
+    packageJsonParser,
+    logger
+  })
+  const ncu = new Ncu(packageJsonReader)
   const outdatedPackages = await ncu.check()
   logger.debug(`outdatedPackages=${JSON.stringify(outdatedPackages)}`)
 
@@ -71,10 +83,12 @@ export const main = async ({
     terminal,
     packageManager: options.packageManager
   })
+  const pullRequestTitleCreator = new PullRequestTitleCreator(options.pullRequestTitle)
   const pullRequestCreator = new PullRequestCreator({
     github,
     gitRepo,
     githubRepo,
+    pullRequestTitleCreator,
     logger
   })
   const branchNameCreator = new BranchNameCreator(options.branchName)

package/src/ncu/Ncu.ts

@@ -1,12 +1,14 @@
 import { run } from 'npm-check-updates'
 import type { Options } from 'npm-check-updates/build/src/types'
-import { readPackageJson } from '../read-package-json'
-import type { OutdatedPackage } from '../types'
+import type { OutdatedPackage } from '../ncu'
+import type { PackageJsonReader } from '../package-json'
 import { isNcuOutdatedPackages } from './isNcuOutdatedPackages'
 import { NcuOutdatedPackagesConverter } from './NcuOutdatedPackagesConverter'
 
 // TODO: add test
 export class Ncu {
+  constructor (private readonly packageJsonReader: PackageJsonReader) {}
+
   async check (): Promise<OutdatedPackage[]> {
     return await this.run({
       jsonUpgraded: true
@@ -22,7 +24,7 @@ export class Ncu {
   }
 
   private async run (options: Options): Promise<OutdatedPackage[]> {
-    const pkg = await readPackageJson('./package.json')
+    const pkg = await this.packageJsonReader.read('./package.json')
     const currentDependencies = {
       ...pkg.dependencies,
       ...pkg.devDependencies,

package/src/ncu/NcuOutdatedPackagesConverter.ts

@@ -1,7 +1,7 @@
-import type { PackageDependencies } from '../read-package-json'
-import type { OutdatedPackage } from '../types'
-import { PackageVersion } from '../values'
+import type { PackageDependencies } from '../package-json'
 import type { NcuOutdatedPackages } from './NcuOutdatedPackages'
+import type { OutdatedPackage } from './OutdatedPackage'
+import { PackageVersion } from './PackageVersion'
 import { toUpdateType } from './toUpdateType'
 
 // TODO: add test

package/src/{types → ncu}/OutdatedPackage.ts

@@ -1,5 +1,5 @@
-import type { UpdateType } from '../enums/UpdateType'
-import type { PackageVersion } from '../values'
+import type { PackageVersion } from './PackageVersion'
+import type { UpdateType } from './UpdateType'
 
 export interface OutdatedPackage {
   name: string

package/src/ncu/index.ts

@@ -1 +1,7 @@
 export { Ncu } from './Ncu'
+export type { OutdatedPackage } from './OutdatedPackage'
+export { PackageVersion } from './PackageVersion'
+export {
+  isUpdateType,
+  UpdateType
+} from './UpdateType'

package/src/ncu/toUpdateType.test.ts

@@ -1,6 +1,6 @@
-import { UpdateType } from '../enums'
-import { PackageVersion } from '../values'
+import { PackageVersion } from './PackageVersion'
 import { toUpdateType } from './toUpdateType'
+import { UpdateType } from './UpdateType'
 
 describe('toUpdateType', () => {
   const currentVersion = PackageVersion.of('1.0.0')

package/src/ncu/toUpdateType.ts

@@ -1,5 +1,5 @@
-import { UpdateType } from '../enums'
-import type { PackageVersion } from '../values'
+import type { PackageVersion } from './PackageVersion'
+import { UpdateType } from './UpdateType'
 
 export const toUpdateType = (currentVersion: PackageVersion, newVersion: PackageVersion): UpdateType => {
   if (currentVersion.major !== newVersion.major) {

package/src/options/Options.ts

@@ -7,14 +7,17 @@ import {
   union
 } from 'io-ts'
 import type { TypeOf } from 'io-ts'
+import { LogLevel } from '../logger'
+import { PackageManagerName } from '../package-manager'
 
-export const Options = intersection([
+const Options = intersection([
   type({
     branchName: string,
     commitMessage: string,
     githubToken: string,
-    logLevel: union([literal('info'), literal('debug')]),
-    packageManager: union([literal('npm'), literal('yarn')])
+    logLevel: union([literal(LogLevel.Info), literal(LogLevel.Debug)]),
+    packageManager: union([literal(PackageManagerName.Npm), literal(PackageManagerName.Yarn)]),
+    pullRequestTitle: string
   }),
   partial({
     gitUserEmail: string,

package/src/options/initOptions.ts

@@ -3,10 +3,8 @@ import {
   program
 } from 'commander'
 import { app } from '../app'
-import {
-  LogLevel,
-  PackageManagerName
-} from '../enums'
+import { LogLevel } from '../logger'
+import { PackageManagerName } from '../package-manager'
 import { isOptions } from './Options'
 import type { Options } from './Options'
 
@@ -35,6 +33,7 @@ export const initOptions = (): Options => {
         ])
         .default(PackageManagerName.Npm)
     )
+    .option('--pull-request-title <value>', 'Pull request title template', 'chore(deps): {{updateType}} update {{{packageName}}} to v{{newVersion}}')
   program.parse(process.argv)
   const options = program.opts()
 

package/src/{read-package-json → package-json}/Package.ts

@@ -7,7 +7,7 @@ import {
 import type { TypeOf } from 'io-ts'
 import { PackageDependencies } from './PackageDependencies'
 
-export const Package = intersection([
+const Package = intersection([
   type({
     name: string,
     version: string

package/src/package-json/PackageJsonParser.ts

@@ -0,0 +1,19 @@
+import type { Logger } from '../logger'
+import { isPackage } from './Package'
+import type { Package } from './Package'
+
+// TODO: add test
+export class PackageJsonParser {
+  constructor (private readonly logger: Logger) {}
+
+  parse (json: string): Package {
+    const parsed: unknown = JSON.parse(json)
+    this.logger.debug(`parsed=${JSON.stringify(parsed)}`)
+
+    if (isPackage(parsed)) {
+      return parsed
+    } else {
+      throw new Error(`Failed to parse package.json. json=${json}`)
+    }
+  }
+}

package/src/package-json/PackageJsonReader.ts

@@ -0,0 +1,27 @@
+import fs from 'fs'
+import type { Logger } from '../logger'
+import type { Package } from './Package'
+import type { PackageJsonParser } from './PackageJsonParser'
+
+// TODO: add test
+export class PackageJsonReader {
+  private readonly packageJsonParser: PackageJsonParser
+  private readonly logger: Logger
+
+  constructor ({
+    packageJsonParser,
+    logger
+  }: {
+    packageJsonParser: PackageJsonParser
+    logger: Logger
+  }) {
+    this.packageJsonParser = packageJsonParser
+    this.logger = logger
+  }
+
+  async read (filePath: string): Promise<Package> {
+    const json = await fs.promises.readFile(filePath, 'utf8')
+    this.logger.debug(`json=${json}`)
+    return this.packageJsonParser.parse(json)
+  }
+}

package/src/package-json/index.ts

@@ -0,0 +1,3 @@
+export { PackageJsonParser } from './PackageJsonParser'
+export { PackageJsonReader } from './PackageJsonReader'
+export type { PackageDependencies } from './PackageDependencies'

package/src/package-manager/createPackageManager.ts

@@ -1,7 +1,7 @@
-import type { PackageManagerName } from '../enums'
 import type { Terminal } from '../terminal'
 import { Npm } from './Npm'
 import type { PackageManager } from './PackageManager'
+import type { PackageManagerName } from './PackageManagerName'
 import { Yarn } from './Yarn'
 
 // TODO: add test

package/src/package-manager/index.ts

@@ -1,4 +1,8 @@
 export { createPackageManager } from './createPackageManager'
 export { Npm } from './Npm'
 export type { PackageManager } from './PackageManager'
+export {
+  isPackageManagerName,
+  PackageManagerName
+} from './PackageManagerName'
 export { Yarn } from './Yarn'

package/src/{outdated-package-processor → processors}/OutdatedPackageProcessor.ts

@@ -1,6 +1,6 @@
-import type { BranchNameCreator } from '../branch-name-creator'
-import type { CommitMessageCreator } from '../commit-message-creator'
 import type {
+  BranchNameCreator,
+  CommitMessageCreator,
   Committer,
   Git
 } from '../git'
@@ -9,12 +9,12 @@ import type {
   RemoteBranchExistenceChecker
 } from '../github'
 import type { Logger } from '../logger'
-import type { Ncu } from '../ncu'
-import type { PackageManager } from '../package-manager'
 import type {
-  OutdatedPackage,
-  Result
-} from '../types'
+  Ncu,
+  OutdatedPackage
+} from '../ncu'
+import type { PackageManager } from '../package-manager'
+import type { Result } from './Result'
 
 // TODO: add test
 export class OutdatedPackageProcessor {

package/src/{outdated-packages-processor → processors}/OutdatedPackagesProcessor.ts

@@ -1,9 +1,7 @@
 import type { Logger } from '../logger'
-import type { OutdatedPackageProcessor } from '../outdated-package-processor'
-import type {
-  OutdatedPackage,
-  Result
-} from '../types'
+import type { OutdatedPackage } from '../ncu'
+import type { OutdatedPackageProcessor } from './OutdatedPackageProcessor'
+import type { Result } from './Result'
 
 export class OutdatedPackagesProcessor {
   private readonly outdatedPackageProcessor: OutdatedPackageProcessor

package/src/{types → processors}/Result.ts

@@ -1,4 +1,4 @@
-import type { OutdatedPackage } from './OutdatedPackage'
+import type { OutdatedPackage } from '../ncu'
 
 export interface Result {
   outdatedPackage: OutdatedPackage

package/src/{outdated-packages-processor → processors}/index.ts

@@ -1 +1,2 @@
+export { OutdatedPackageProcessor } from './OutdatedPackageProcessor'
 export { OutdatedPackagesProcessor } from './OutdatedPackagesProcessor'

package/dist/branch-name-creator/index.js

@@ -1,5 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.BranchNameCreator = void 0;
-var BranchNameCreator_1 = require("./BranchNameCreator");
-Object.defineProperty(exports, "BranchNameCreator", { enumerable: true, get: function () { return BranchNameCreator_1.BranchNameCreator; } });

package/dist/commit-message-creator/index.js

@@ -1,5 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.CommitMessageCreator = void 0;
-var CommitMessageCreator_1 = require("./CommitMessageCreator");
-Object.defineProperty(exports, "CommitMessageCreator", { enumerable: true, get: function () { return CommitMessageCreator_1.CommitMessageCreator; } });

package/dist/enums/index.js

@@ -1,12 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.UpdateType = exports.isUpdateType = exports.PackageManagerName = exports.isPackageManagerName = exports.LogLevel = exports.isLogLevel = void 0;
-var LogLevel_1 = require("./LogLevel");
-Object.defineProperty(exports, "isLogLevel", { enumerable: true, get: function () { return LogLevel_1.isLogLevel; } });
-Object.defineProperty(exports, "LogLevel", { enumerable: true, get: function () { return LogLevel_1.LogLevel; } });
-var PackageManagerName_1 = require("./PackageManagerName");
-Object.defineProperty(exports, "isPackageManagerName", { enumerable: true, get: function () { return PackageManagerName_1.isPackageManagerName; } });
-Object.defineProperty(exports, "PackageManagerName", { enumerable: true, get: function () { return PackageManagerName_1.PackageManagerName; } });
-var UpdateType_1 = require("./UpdateType");
-Object.defineProperty(exports, "isUpdateType", { enumerable: true, get: function () { return UpdateType_1.isUpdateType; } });
-Object.defineProperty(exports, "UpdateType", { enumerable: true, get: function () { return UpdateType_1.UpdateType; } });

package/dist/github/createPullRequestTitle.js

@@ -1,10 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.createPullRequestTitle = void 0;
-const createPullRequestTitle = (outdatedPackage) => {
-    const packageName = outdatedPackage.name;
-    const newVersion = outdatedPackage.newVersion.version;
-    const updateType = outdatedPackage.type;
-    return `chore(deps): ${updateType} update ${packageName} to v${newVersion}`;
-};
-exports.createPullRequestTitle = createPullRequestTitle;

package/dist/outdated-package-processor/index.js

@@ -1,5 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.OutdatedPackageProcessor = void 0;
-var OutdatedPackageProcessor_1 = require("./OutdatedPackageProcessor");
-Object.defineProperty(exports, "OutdatedPackageProcessor", { enumerable: true, get: function () { return OutdatedPackageProcessor_1.OutdatedPackageProcessor; } });

package/dist/read-package-json/index.js

@@ -1,5 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.readPackageJson = void 0;
-var readPackageJson_1 = require("./readPackageJson");
-Object.defineProperty(exports, "readPackageJson", { enumerable: true, get: function () { return readPackageJson_1.readPackageJson; } });

package/dist/read-package-json/parsePackageJson.js

@@ -1,15 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.parsePackageJson = void 0;
-const Package_1 = require("./Package");
-// TODO: add test
-const parsePackageJson = (json) => {
-    const parsed = JSON.parse(json);
-    if ((0, Package_1.isPackage)(parsed)) {
-        return parsed;
-    }
-    else {
-        throw new Error(`Failed to parse package.json. json=${json}`);
-    }
-};
-exports.parsePackageJson = parsePackageJson;

package/dist/read-package-json/readFile.js

@@ -1,12 +0,0 @@
-"use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.readFile = void 0;
-const fs_1 = __importDefault(require("fs"));
-// TODO: add test
-const readFile = async (filePath) => {
-    return await fs_1.default.promises.readFile(filePath, 'utf8');
-};
-exports.readFile = readFile;

package/dist/read-package-json/readPackageJson.js

@@ -1,11 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.readPackageJson = void 0;
-const parsePackageJson_1 = require("./parsePackageJson");
-const readFile_1 = require("./readFile");
-// TODO: add test
-const readPackageJson = async (filePath) => {
-    const json = await (0, readFile_1.readFile)(filePath);
-    return (0, parsePackageJson_1.parsePackageJson)(json);
-};
-exports.readPackageJson = readPackageJson;

package/dist/types/index.js

@@ -1,2 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });

package/dist/values/index.js

@@ -1,5 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.PackageVersion = void 0;
-var PackageVersion_1 = require("./PackageVersion");
-Object.defineProperty(exports, "PackageVersion", { enumerable: true, get: function () { return PackageVersion_1.PackageVersion; } });

package/src/branch-name-creator/index.ts

@@ -1 +0,0 @@
-export { BranchNameCreator } from './BranchNameCreator'

package/src/commit-message-creator/index.ts

@@ -1 +0,0 @@
-export { CommitMessageCreator } from './CommitMessageCreator'

package/src/enums/index.ts

@@ -1,12 +0,0 @@
-export {
-  isLogLevel,
-  LogLevel
-} from './LogLevel'
-export {
-  isPackageManagerName,
-  PackageManagerName
-} from './PackageManagerName'
-export {
-  isUpdateType,
-  UpdateType
-} from './UpdateType'

package/src/github/createPullRequestTitle.test.ts

@@ -1,43 +0,0 @@
-import { UpdateType } from '../enums'
-import { PackageVersion } from '../values'
-import { createPullRequestTitle } from './createPullRequestTitle'
-
-describe('createPullRequestTitle', () => {
-  const currentVersion = PackageVersion.of('1.0.0')
-
-  describe('if update type is patch', () => {
-    it('returns pull request title', () => {
-      const actual = createPullRequestTitle({
-        name: '@typescript-eslint/eslint-plugin',
-        currentVersion,
-        newVersion: PackageVersion.of('1.0.1'),
-        type: UpdateType.Patch
-      })
-      expect(actual).toBe('chore(deps): patch update @typescript-eslint/eslint-plugin to v1.0.1')
-    })
-  })
-
-  describe('if update type is minor', () => {
-    it('returns pull request title', () => {
-      const actual = createPullRequestTitle({
-        name: '@typescript-eslint/eslint-plugin',
-        currentVersion,
-        newVersion: PackageVersion.of('1.1.0'),
-        type: UpdateType.Minor
-      })
-      expect(actual).toBe('chore(deps): minor update @typescript-eslint/eslint-plugin to v1.1.0')
-    })
-  })
-
-  describe('if update type is major', () => {
-    it('returns pull request title', () => {
-      const actual = createPullRequestTitle({
-        name: '@typescript-eslint/eslint-plugin',
-        currentVersion,
-        newVersion: PackageVersion.of('2.0.0'),
-        type: UpdateType.Major
-      })
-      expect(actual).toBe('chore(deps): major update @typescript-eslint/eslint-plugin to v2.0.0')
-    })
-  })
-})

package/src/github/createPullRequestTitle.ts

@@ -1,8 +0,0 @@
-import type { OutdatedPackage } from '../types'
-
-export const createPullRequestTitle = (outdatedPackage: OutdatedPackage): string => {
-  const packageName = outdatedPackage.name
-  const newVersion = outdatedPackage.newVersion.version
-  const updateType = outdatedPackage.type
-  return `chore(deps): ${updateType} update ${packageName} to v${newVersion}`
-}

package/src/outdated-package-processor/index.ts

@@ -1 +0,0 @@
-export { OutdatedPackageProcessor } from './OutdatedPackageProcessor'

package/src/read-package-json/index.ts

@@ -1,2 +0,0 @@
-export type { PackageDependencies } from './PackageDependencies'
-export { readPackageJson } from './readPackageJson'

package/src/read-package-json/parsePackageJson.ts

@@ -1,13 +0,0 @@
-import { isPackage } from './Package'
-import type { Package } from './Package'
-
-// TODO: add test
-export const parsePackageJson = (json: string): Package => {
-  const parsed: unknown = JSON.parse(json)
-
-  if (isPackage(parsed)) {
-    return parsed
-  } else {
-    throw new Error(`Failed to parse package.json. json=${json}`)
-  }
-}

package/src/read-package-json/readFile.ts

@@ -1,6 +0,0 @@
-import fs from 'fs'
-
-// TODO: add test
-export const readFile = async (filePath: string): Promise<string> => {
-  return await fs.promises.readFile(filePath, 'utf8')
-}

package/src/read-package-json/readPackageJson.ts

@@ -1,9 +0,0 @@
-import type { Package } from './Package'
-import { parsePackageJson } from './parsePackageJson'
-import { readFile } from './readFile'
-
-// TODO: add test
-export const readPackageJson = async (filePath: string): Promise<Package> => {
-  const json = await readFile(filePath)
-  return parsePackageJson(json)
-}

package/src/types/index.ts

@@ -1,2 +0,0 @@
-export type { OutdatedPackage } from './OutdatedPackage'
-export type { Result } from './Result'

package/src/values/index.ts

@@ -1 +0,0 @@
-export { PackageVersion } from './PackageVersion'