npm-update-package 0.42.0 → 0.43.2

package/README.md

@@ -37,6 +37,13 @@ Commit message template
 - type: string
 - required: true
 
+### `--ignore-packages`
+
+Package names to ignore
+
+- type: string[]
+- required: false
+
 ### `--log-level`
 
 Log level to show

package/dist/package.json

@@ -1,6 +1,6 @@
 {
     "name": "npm-update-package",
-    "version": "0.42.0",
+    "version": "0.43.2",
     "description": "CLI tool for creating pull requests to update npm packages",
     "scripts": {
         "build": "tsc --project tsconfig.build.json",
@@ -20,7 +20,7 @@
         "fp-ts": "2.11.8",
         "http-status-codes": "2.2.0",
         "io-ts": "2.2.16",
-        "log4js": "6.4.1",
+        "log4js": "6.4.2",
         "mustache": "4.1.0",
         "npm-check-updates": "12.5.0",
         "parse-github-url": "1.0.2",
@@ -49,7 +49,7 @@
         "npm-run-all": "4.1.5",
         "rimraf": "3.0.2",
         "ts-jest": "27.1.3",
-        "ts-node": "10.5.0",
+        "ts-node": "10.6.0",
         "typescript": "4.5.5",
         "utility-types": "3.10.0"
     },

package/dist/src/core/OutdatedPackageProcessor.js

@@ -5,7 +5,7 @@ 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, ncu, packageManager, pullRequestCreator, branchFinder, logger, commitMessageCreator, pullRequestFinder, pullRequestCloser, options }) {
         this.git = git;
         this.ncu = ncu;
         this.packageManager = packageManager;
@@ -15,11 +15,21 @@ class OutdatedPackageProcessor {
         this.commitMessageCreator = commitMessageCreator;
         this.pullRequestFinder = pullRequestFinder;
         this.pullRequestCloser = pullRequestCloser;
+        this.options = options;
     }
     /**
      * Don't run in parallel because it includes file operations.
      */
     async process(outdatedPackage) {
+        var _a;
+        const ignoredPackages = (_a = this.options.ignorePackages) !== null && _a !== void 0 ? _a : [];
+        if (ignoredPackages.includes(outdatedPackage.name)) {
+            this.logger.info(`Skip ${outdatedPackage.name} because ignorePackages contains it.`);
+            return (0, Either_1.right)({
+                outdatedPackage,
+                skipped: true
+            });
+        }
         const branchName = (0, git_1.createBranchName)(outdatedPackage);
         this.logger.debug(`branchName=${branchName}`);
         if (this.branchFinder.findByName(branchName) !== undefined) {
@@ -33,11 +43,7 @@ class OutdatedPackageProcessor {
         this.logger.info(`${branchName} branch has created.`);
         try {
             try {
-                const updatedPackages = await this.ncu.update(outdatedPackage);
-                if (updatedPackages.length !== 1) {
-                    throw new Error(`Failed to update ${outdatedPackage.name}.`);
-                }
-                await this.packageManager.install();
+                await this.updatePackage(outdatedPackage);
             }
             catch (error) {
                 this.logger.error(error);
@@ -57,12 +63,7 @@ class OutdatedPackageProcessor {
                 branchName
             });
             this.logger.info(`Pull request for ${outdatedPackage.name} has created. ${pullRequest.html_url}`);
-            const pullRequests = this.pullRequestFinder.findByPackageName(outdatedPackage.name);
-            this.logger.debug(`pullRequests=${JSON.stringify(pullRequests)}`);
-            await Promise.all(pullRequests.map(async (pullRequest) => {
-                await this.pullRequestCloser.close(pullRequest);
-                this.logger.info(`Pull request for ${outdatedPackage.name} has closed. ${pullRequest.html_url}`);
-            }));
+            await this.closeOldPullRequests(outdatedPackage);
             return (0, Either_1.right)({
                 outdatedPackage,
                 created: true
@@ -75,5 +76,20 @@ 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)}`);
+        await Promise.all(pullRequests.map(async (pullRequest) => {
+            await this.pullRequestCloser.close(pullRequest);
+            this.logger.info(`Pull request for ${outdatedPackage.name} has closed. ${pullRequest.html_url}`);
+        }));
+    }
 }
 exports.OutdatedPackageProcessor = OutdatedPackageProcessor;

package/dist/src/git/Git.js

@@ -16,14 +16,6 @@ class Git {
     async createBranch(branchName) {
         await this.terminal.run('git', 'checkout', '-b', branchName);
     }
-    async getConfig(key) {
-        const { stdout } = await this.terminal.run('git', 'config', key);
-        return stdout.trim();
-    }
-    async getCurrentBranch() {
-        const { stdout } = await this.terminal.run('git', 'rev-parse', '--abbrev-ref', 'HEAD');
-        return stdout.trim();
-    }
     async getRemoteUrl() {
         const { stdout } = await this.terminal.run('git', 'remote', 'get-url', '--push', 'origin');
         return stdout.trim();
@@ -41,9 +33,6 @@ class Git {
     async restore(...files) {
         await this.terminal.run('git', 'checkout', ...files);
     }
-    async setConfig(key, value) {
-        await this.terminal.run('git', 'config', key, value);
-    }
     async switch(branchName) {
         await this.terminal.run('git', 'checkout', branchName);
     }

package/dist/src/github/errors/NotFoundError.js

@@ -3,7 +3,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.isNotFoundError = void 0;
 const request_error_1 = require("@octokit/request-error");
 const http_status_codes_1 = require("http-status-codes");
-// TODO: Add test
 const isNotFoundError = (value) => {
     return value instanceof request_error_1.RequestError && value.status === http_status_codes_1.StatusCodes.NOT_FOUND;
 };

package/dist/src/github/pull-request/closer/PullRequestCloser.js

@@ -6,14 +6,16 @@ class PullRequestCloser {
         this.github = github;
     }
     async close(pullRequest) {
+        const owner = pullRequest.base.repo.owner.login;
+        const repo = pullRequest.base.repo.name;
         await this.github.closePullRequest({
-            owner: pullRequest.base.repo.owner.login,
-            repo: pullRequest.base.repo.name,
+            owner,
+            repo,
             pullNumber: pullRequest.number
         });
         await this.github.deleteBranch({
-            owner: pullRequest.base.repo.owner.login,
-            repo: pullRequest.base.repo.name,
+            owner,
+            repo,
             branch: pullRequest.head.ref
         });
     }

package/dist/src/main.js

@@ -89,7 +89,8 @@ const main = async ({ options, logger }) => {
         logger,
         commitMessageCreator,
         pullRequestFinder,
-        pullRequestCloser
+        pullRequestCloser,
+        options
     });
     const outdatedPackagesProcessor = new core_1.OutdatedPackagesProcessor({
         outdatedPackageProcessor,

package/dist/src/options/Options.js

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

package/dist/src/options/cliOptions.js

@@ -18,6 +18,12 @@ exports.cliOptions = [
         type: OptionType_1.OptionType.String,
         required: true
     },
+    {
+        name: 'ignore-packages',
+        description: 'Package names to ignore',
+        type: OptionType_1.OptionType.StringArray,
+        required: false
+    },
     {
         name: 'log-level',
         description: 'Log level to show',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "npm-update-package",
-  "version": "0.42.0",
+  "version": "0.43.2",
   "description": "CLI tool for creating pull requests to update npm packages",
   "scripts": {
     "build": "tsc --project tsconfig.build.json",
@@ -20,7 +20,7 @@
     "fp-ts": "2.11.8",
     "http-status-codes": "2.2.0",
     "io-ts": "2.2.16",
-    "log4js": "6.4.1",
+    "log4js": "6.4.2",
     "mustache": "4.1.0",
     "npm-check-updates": "12.5.0",
     "parse-github-url": "1.0.2",
@@ -49,7 +49,7 @@
     "npm-run-all": "4.1.5",
     "rimraf": "3.0.2",
     "ts-jest": "27.1.3",
-    "ts-node": "10.5.0",
+    "ts-node": "10.6.0",
     "typescript": "4.5.5",
     "utility-types": "3.10.0"
   },