npm-update-package 0.45.15 → 0.46.1

package/README.md

@@ -1,7 +1,7 @@
 [![npm version](https://badge.fury.io/js/npm-update-package.svg)](https://badge.fury.io/js/npm-update-package)
 [![JavaScript Style Guide](https://img.shields.io/badge/code_style-standard-brightgreen.svg)](https://standardjs.com)
-![eslint](https://github.com/npm-update-package/npm-update-package/actions/workflows/eslint.yml/badge.svg)
-![test](https://github.com/npm-update-package/npm-update-package/actions/workflows/test.yml/badge.svg)
+[![eslint](https://github.com/npm-update-package/npm-update-package/actions/workflows/eslint.yml/badge.svg)](https://github.com/npm-update-package/npm-update-package/actions/workflows/eslint.yml)
+[![test](https://github.com/npm-update-package/npm-update-package/actions/workflows/test.yml/badge.svg)](https://github.com/npm-update-package/npm-update-package/actions/workflows/test.yml)
 ![Coverage:statements](docs/badge-statements.svg)
 ![Coverage:branches](docs/badge-branches.svg)
 ![Coverage:functions](docs/badge-functions.svg)
@@ -22,6 +22,23 @@ npx npm-update-package --github-token $GITHUB_TOKEN
 You can customize behavior via command-line options.  
 Some options can embed variables like `{{packageName}}`(HTML-escaped) or `{{{packageName}}}`(not HTML-escaped).
 
+### `--assignees`
+
+User names to assign to pull request.
+
+|Name|Value|
+|---|---|
+|type|string[]|
+|required|false|
+
+#### Example
+
+```sh
+npx npm-update-package \
+  --github-token $GITHUB_TOKEN \
+  --assignees npm-update-package npm-update-package-bot
+```
+
 ### `--commit-message`
 
 Commit message template.

package/dist/package.json

@@ -1,6 +1,6 @@
 {
     "name": "npm-update-package",
-    "version": "0.45.15",
+    "version": "0.46.1",
     "description": "CLI tool for creating pull requests to update npm packages",
     "scripts": {
         "badges": "jest-coverage-badges output docs",
@@ -23,7 +23,7 @@
         "http-status-codes": "2.2.0",
         "io-ts": "2.2.16",
         "lodash": "4.14.2",
-        "log4js": "6.4.2",
+        "log4js": "6.4.3",
         "mustache": "4.1.0",
         "node-fetch": "2.6.7",
         "npm-check-updates": "12.5.3",
@@ -36,13 +36,13 @@
         "@jest/types": "27.4.2",
         "@tsconfig/node12": "1.0.9",
         "@types/jest": "27.4.0",
-        "@types/lodash": "4.14.179",
+        "@types/lodash": "4.14.180",
         "@types/mustache": "4.1.2",
         "@types/node": "12.20.40",
         "@types/node-fetch": "2.6.1",
         "@types/parse-github-url": "1.0.0",
         "@types/semver": "7.3.9",
-        "@typescript-eslint/eslint-plugin": "5.14.0",
+        "@typescript-eslint/eslint-plugin": "5.15.0",
         "eslint": "8.11.0",
         "eslint-config-standard-with-typescript": "21.0.1",
         "eslint-plugin-import": "2.25.4",
@@ -53,7 +53,7 @@
         "husky": "7.0.4",
         "jest": "27.4.7",
         "jest-coverage-badges": "1.1.2",
-        "lint-staged": "12.3.5",
+        "lint-staged": "12.3.6",
         "npm-run-all": "4.1.5",
         "rimraf": "3.0.2",
         "ts-jest": "27.1.3",

package/dist/src/core/OutdatedPackageProcessor.js

@@ -5,9 +5,8 @@ const Either_1 = require("fp-ts/lib/Either");
 const git_1 = require("../git");
 // TODO: add test
 class OutdatedPackageProcessor {
-    constructor({ git, ncu, packageManager, pullRequestCreator, branchFinder, logger, commitMessageCreator, pullRequestFinder, pullRequestCloser }) {
+    constructor({ git, packageManager, pullRequestCreator, branchFinder, logger, commitMessageCreator, pullRequestFinder, pullRequestCloser, packageUpdater }) {
         this.git = git;
-        this.ncu = ncu;
         this.packageManager = packageManager;
         this.pullRequestCreator = pullRequestCreator;
         this.branchFinder = branchFinder;
@@ -15,6 +14,7 @@ class OutdatedPackageProcessor {
         this.commitMessageCreator = commitMessageCreator;
         this.pullRequestFinder = pullRequestFinder;
         this.pullRequestCloser = pullRequestCloser;
+        this.packageUpdater = packageUpdater;
     }
     /**
      * Don't run in parallel because it includes file operations.
@@ -33,7 +33,7 @@ class OutdatedPackageProcessor {
         this.logger.info(`${branchName} branch has created.`);
         try {
             try {
-                await this.updatePackage(outdatedPackage);
+                await this.packageUpdater.update(outdatedPackage);
             }
             catch (error) {
                 this.logger.error(error);
@@ -66,13 +66,6 @@ class OutdatedPackageProcessor {
             this.logger.info(`${branchName} branch has removed.`);
         }
     }
-    async updatePackage(outdatedPackage) {
-        const updatedPackages = await this.ncu.update(outdatedPackage);
-        if (updatedPackages.length !== 1) {
-            throw new Error(`Failed to update ${outdatedPackage.name}.`);
-        }
-        await this.packageManager.install();
-    }
     async closeOldPullRequests(outdatedPackage) {
         const pullRequests = this.pullRequestFinder.findByPackageName(outdatedPackage.name);
         this.logger.debug(`pullRequests=${JSON.stringify(pullRequests)}`);

package/dist/src/core/PackageUpdater.js

@@ -0,0 +1,17 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.PackageUpdater = void 0;
+class PackageUpdater {
+    constructor({ packageManager, ncu }) {
+        this.packageManager = packageManager;
+        this.ncu = ncu;
+    }
+    async update(outdatedPackage) {
+        const updatedPackages = await this.ncu.update(outdatedPackage);
+        if (updatedPackages.length !== 1) {
+            throw new Error(`Failed to update ${outdatedPackage.name}.`);
+        }
+        await this.packageManager.install();
+    }
+}
+exports.PackageUpdater = PackageUpdater;

package/dist/src/core/index.js

@@ -18,4 +18,5 @@ __exportStar(require("./FailedResult"), exports);
 __exportStar(require("./OutdatedPackage"), exports);
 __exportStar(require("./OutdatedPackageProcessor"), exports);
 __exportStar(require("./OutdatedPackagesProcessor"), exports);
+__exportStar(require("./PackageUpdater"), exports);
 __exportStar(require("./SucceededResult"), exports);

package/dist/src/file/readFile.js

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

package/dist/src/github/GitHub.js

@@ -6,6 +6,14 @@ class GitHub {
     constructor(octokit) {
         this.octokit = octokit;
     }
+    async addAssignees({ owner, repo, issueNumber, assignees }) {
+        await this.octokit.issues.addAssignees({
+            owner,
+            repo,
+            issue_number: issueNumber,
+            assignees
+        });
+    }
     async addLabels({ owner, repo, issueNumber, labels }) {
         await this.octokit.issues.addLabels({
             owner,

package/dist/src/github/label/creator/LabelCreator.js

@@ -2,7 +2,6 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.LabelCreator = void 0;
 const errors_1 = require("../../errors");
-// TODO: Add test
 class LabelCreator {
     constructor({ github, gitRepo, logger }) {
         this.github = github;
@@ -15,6 +14,14 @@ class LabelCreator {
             this.logger.info(`Skip creating ${name} label because it already exists.`);
             return;
         }
+        await this.createLabel({
+            name,
+            description,
+            color
+        });
+        this.logger.info(`${name} label has created.`);
+    }
+    async createLabel({ name, description, color }) {
         await this.github.createLabel({
             owner: this.gitRepo.owner,
             repo: this.gitRepo.name,
@@ -22,7 +29,6 @@ class LabelCreator {
             description,
             color
         });
-        this.logger.info(`${name} label has created.`);
     }
     async fetchLabel(name) {
         try {

package/dist/src/github/pull-request/creator/PullRequestCreator.js

@@ -2,7 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.PullRequestCreator = void 0;
 class PullRequestCreator {
-    constructor({ github, gitRepo, githubRepo, pullRequestTitleCreator, pullRequestBodyCreator, logger, reviewers }) {
+    constructor({ github, gitRepo, githubRepo, pullRequestTitleCreator, pullRequestBodyCreator, logger, reviewers, assignees }) {
         this.github = github;
         this.gitRepo = gitRepo;
         this.githubRepo = githubRepo;
@@ -10,6 +10,7 @@ class PullRequestCreator {
         this.pullRequestBodyCreator = pullRequestBodyCreator;
         this.logger = logger;
         this.reviewers = reviewers;
+        this.assignees = assignees;
     }
     async create({ outdatedPackage, branchName }) {
         const title = this.pullRequestTitleCreator.create(outdatedPackage);
@@ -31,6 +32,14 @@ class PullRequestCreator {
             issueNumber: pullRequest.number,
             labels: ['npm-update-package']
         });
+        if (this.assignees !== undefined) {
+            await this.github.addAssignees({
+                owner: this.gitRepo.owner,
+                repo: this.gitRepo.name,
+                issueNumber: pullRequest.number,
+                assignees: this.assignees
+            });
+        }
         if (this.reviewers !== undefined) {
             await this.github.requestReviewers({
                 owner: this.gitRepo.owner,

package/dist/src/logger/createLogger.js

@@ -2,7 +2,6 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.createLogger = void 0;
 const log4js_1 = require("log4js");
-// TODO: add test
 const createLogger = (logLevel) => {
     const logger = (0, log4js_1.getLogger)();
     logger.level = logLevel;

package/dist/src/main.js

@@ -77,21 +77,26 @@ const main = async ({ options, logger }) => {
         pullRequestTitleCreator,
         pullRequestBodyCreator,
         logger,
-        reviewers: options.reviewers
+        reviewers: options.reviewers,
+        assignees: options.assignees
     });
     const commitMessageCreator = new git_1.CommitMessageCreator(options.commitMessage);
     const pullRequestFinder = new github_1.PullRequestFinder(pullRequests);
     const pullRequestCloser = new github_1.PullRequestCloser(github);
+    const packageUpdater = new core_1.PackageUpdater({
+        packageManager,
+        ncu
+    });
     const outdatedPackageProcessor = new core_1.OutdatedPackageProcessor({
         git,
-        ncu,
         packageManager,
         pullRequestCreator,
         branchFinder,
         logger,
         commitMessageCreator,
         pullRequestFinder,
-        pullRequestCloser
+        pullRequestCloser,
+        packageUpdater
     });
     const outdatedPackagesProcessor = new core_1.OutdatedPackagesProcessor({
         outdatedPackageProcessor,

package/dist/src/options/Options.js

@@ -24,6 +24,7 @@ const Options = (0, io_ts_1.intersection)([
         prTitle: io_ts_1.string
     }),
     (0, io_ts_1.partial)({
+        assignees: (0, io_ts_1.array)(io_ts_1.string),
         ignorePackages: (0, io_ts_1.array)(io_ts_1.string),
         prBodyNotes: io_ts_1.string,
         reviewers: (0, io_ts_1.array)(io_ts_1.string)

package/dist/src/options/cliOptions.js

@@ -5,6 +5,12 @@ const logger_1 = require("../logger");
 const package_manager_1 = require("../package-manager");
 const OptionType_1 = require("./OptionType");
 exports.cliOptions = [
+    {
+        name: 'assignees',
+        description: 'User names to assign to pull request',
+        type: OptionType_1.OptionType.StringArray,
+        required: false
+    },
     {
         name: 'commit-message',
         description: 'Commit message template',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "npm-update-package",
-  "version": "0.45.15",
+  "version": "0.46.1",
   "description": "CLI tool for creating pull requests to update npm packages",
   "scripts": {
     "badges": "jest-coverage-badges output docs",
@@ -23,7 +23,7 @@
     "http-status-codes": "2.2.0",
     "io-ts": "2.2.16",
     "lodash": "4.14.2",
-    "log4js": "6.4.2",
+    "log4js": "6.4.3",
     "mustache": "4.1.0",
     "node-fetch": "2.6.7",
     "npm-check-updates": "12.5.3",
@@ -36,13 +36,13 @@
     "@jest/types": "27.4.2",
     "@tsconfig/node12": "1.0.9",
     "@types/jest": "27.4.0",
-    "@types/lodash": "4.14.179",
+    "@types/lodash": "4.14.180",
     "@types/mustache": "4.1.2",
     "@types/node": "12.20.40",
     "@types/node-fetch": "2.6.1",
     "@types/parse-github-url": "1.0.0",
     "@types/semver": "7.3.9",
-    "@typescript-eslint/eslint-plugin": "5.14.0",
+    "@typescript-eslint/eslint-plugin": "5.15.0",
     "eslint": "8.11.0",
     "eslint-config-standard-with-typescript": "21.0.1",
     "eslint-plugin-import": "2.25.4",
@@ -53,7 +53,7 @@
     "husky": "7.0.4",
     "jest": "27.4.7",
     "jest-coverage-badges": "1.1.2",
-    "lint-staged": "12.3.5",
+    "lint-staged": "12.3.6",
     "npm-run-all": "4.1.5",
     "rimraf": "3.0.2",
     "ts-jest": "27.1.3",