npm-update-package 0.10.0 → 0.11.0

package/README.md

@@ -9,6 +9,10 @@ CLI tool for creating pull request to update npm packages
 
 This package is currently under development, so the major version is 0 yet.
 
+## Requirements
+
+- Git v2.23.0 or higher
+
 ## Installation
 
 If you are using npm:
@@ -45,7 +49,7 @@ Template strings such as `--branch-name` can embed variables like `{{packageName
 Branch name template
 
 - value: template string
-- embeddable variables:
+- variables:
   - `currentVersion`
   - `newVersion`
   - `packageName`
@@ -58,7 +62,7 @@ Branch name template
 Commit message template
 
 - value: template string
-- embeddable variables:
+- variables:
   - `currentVersion`
   - `newVersion`
   - `packageName`
@@ -78,6 +82,7 @@ GitHub token
 Log level to show
 
 - value: string
+  - `error`
   - `info`
   - `debug`
 - required: false
@@ -98,7 +103,7 @@ Package manager of your project
 Pull request body template
 
 - value: template string
-- embeddable variables:
+- variables:
   - `appName`
   - `appVersion`
   - `appWeb`
@@ -125,7 +130,7 @@ This PR has been generated by [{{{appName}}}]({{{appWeb}}}) v{{appVersion}}
 Pull request title template
 
 - value: template string
-- embeddable variables:
+- variables:
   - `currentVersion`
   - `newVersion`
   - `packageName`

package/dist/app.js

@@ -3,6 +3,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.app = void 0;
 exports.app = {
     name: 'npm-update-package',
-    version: '0.10.0',
+    version: '0.11.0',
     web: 'https://github.com/npm-update-package/npm-update-package'
 };

package/dist/bin.js

@@ -12,8 +12,10 @@ logger.info(`Start ${app_1.app.name} v${app_1.app.version}`);
     options,
     logger
 })
-    .then(() => logger.info(`End ${app_1.app.name} v${app_1.app.version}`))
-    .catch((e) => {
-    // TODO: improve error handling
-    logger.fatal('Unexpected error has occurred.', e);
+    .then(() => {
+    logger.info(`End ${app_1.app.name} v${app_1.app.version}`);
+})
+    .catch((error) => {
+    logger.fatal(error);
+    throw error;
 });

package/dist/git/Git.js

@@ -10,14 +10,11 @@ class Git {
     async add(...files) {
         await this.terminal.run('git', 'add', ...files);
     }
-    async checkout(branchName) {
-        await this.terminal.run('git', 'checkout', branchName);
-    }
     async commit(message) {
         await this.terminal.run('git', 'commit', '--message', message);
     }
     async createBranch(branchName) {
-        await this.terminal.run('git', 'checkout', '-b', branchName);
+        await this.terminal.run('git', 'switch', '-c', branchName);
     }
     async getConfig(key) {
         const { stdout } = await this.terminal.run('git', 'config', key);
@@ -41,8 +38,14 @@ class Git {
     async removeBranch(branchName) {
         await this.terminal.run('git', 'branch', '-D', branchName);
     }
+    async restore(...files) {
+        await this.terminal.run('git', 'restore', '--worktree', '--staged', ...files);
+    }
     async setConfig(key, value) {
         await this.terminal.run('git', 'config', key, value);
     }
+    async switch(branchName) {
+        await this.terminal.run('git', 'switch', branchName);
+    }
 }
 exports.Git = Git;

package/dist/logger/LogLevel.js

@@ -2,8 +2,9 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.isLogLevel = exports.LogLevel = void 0;
 exports.LogLevel = {
-    Info: 'info',
-    Debug: 'debug'
+    Debug: 'debug',
+    Error: 'error',
+    Info: 'info'
 };
 const logLevels = Object.values(exports.LogLevel);
 const isLogLevel = (value) => logLevels.includes(value);

package/dist/main.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.main = void 0;
+const Either_1 = require("fp-ts/lib/Either");
 const git_1 = require("./git");
 const github_1 = require("./github");
 const ncu_1 = require("./ncu");
@@ -75,14 +76,24 @@ const main = async ({ options, logger }) => {
     });
     const results = await outdatedPackagesProcessor.process(outdatedPackages);
     logger.debug(`results=${JSON.stringify(results)}`);
-    const updatedPackages = results
+    const succeededResults = results.filter(Either_1.isRight).map(({ right }) => right);
+    logger.debug(`succeededResults=${JSON.stringify(succeededResults)}`);
+    const updatedPackages = succeededResults
         .filter(({ updated }) => updated)
         .map(({ outdatedPackage }) => outdatedPackage);
     logger.debug(`updatedPackages=${JSON.stringify(updatedPackages)}`);
-    const skippedPackages = results
+    const skippedPackages = succeededResults
         .filter(({ skipped }) => skipped)
         .map(({ outdatedPackage }) => outdatedPackage);
     logger.debug(`skippedPackages=${JSON.stringify(skippedPackages)}`);
-    logger.info(`${updatedPackages.length} packages has updated. ${skippedPackages.length} packages has skipped.`);
+    const failedResults = results.filter(Either_1.isLeft).map(({ left }) => left);
+    logger.debug(`failedResults=${JSON.stringify(failedResults)}`);
+    const failedPackages = failedResults.map(({ outdatedPackage }) => outdatedPackage);
+    logger.debug(`failedPackages=${JSON.stringify(failedPackages)}`);
+    // TODO: show as table
+    logger.info(`Processed ${succeededResults.length + failedPackages.length} packages:
+- ${updatedPackages.length} packages has updated: ${updatedPackages.map(({ name }) => name).join(',')}
+- ${skippedPackages.length} packages has skipped: ${skippedPackages.map(({ name }) => name).join(',')}
+- ${failedPackages.length} packages has failed: ${failedPackages.map(({ name }) => name).join(',')}`);
 };
 exports.main = main;

package/dist/options/Options.js

@@ -8,7 +8,7 @@ const Options = (0, io_ts_1.type)({
     branchName: io_ts_1.string,
     commitMessage: io_ts_1.string,
     githubToken: io_ts_1.string,
-    logLevel: (0, io_ts_1.union)([(0, io_ts_1.literal)(logger_1.LogLevel.Info), (0, io_ts_1.literal)(logger_1.LogLevel.Debug)]),
+    logLevel: (0, io_ts_1.union)([(0, io_ts_1.literal)(logger_1.LogLevel.Error), (0, io_ts_1.literal)(logger_1.LogLevel.Info), (0, io_ts_1.literal)(logger_1.LogLevel.Debug)]),
     packageManager: (0, io_ts_1.union)([(0, io_ts_1.literal)(package_manager_1.PackageManagerName.Npm), (0, io_ts_1.literal)(package_manager_1.PackageManagerName.Yarn)]),
     pullRequestBody: io_ts_1.string,
     pullRequestTitle: io_ts_1.string

package/dist/options/initOptions.js

@@ -15,6 +15,7 @@ const initOptions = () => {
         .requiredOption('--github-token <value>', 'GitHub token')
         .addOption(new commander_1.Option('--log-level <value>', 'Log level to show')
         .choices([
+        logger_1.LogLevel.Error,
         logger_1.LogLevel.Info,
         logger_1.LogLevel.Debug
     ])

package/dist/processors/OutdatedPackageProcessor.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.OutdatedPackageProcessor = void 0;
+const Either_1 = require("fp-ts/lib/Either");
 // TODO: add test
 class OutdatedPackageProcessor {
     constructor({ git, ncu, packageManager, pullRequestCreator, remoteBranchExistenceChecker, logger, branchNameCreator, commitMessageCreator }) {
@@ -21,35 +22,49 @@ class OutdatedPackageProcessor {
         this.logger.debug(`branchName=${branchName}`);
         if (this.remoteBranchExistenceChecker.check(branchName)) {
             this.logger.info(`Skip ${outdatedPackage.name} because ${branchName} branch already exists on remote.`);
-            return {
+            return (0, Either_1.right)({
                 outdatedPackage,
                 skipped: true
-            };
+            });
         }
         await this.git.createBranch(branchName);
         this.logger.info(`${branchName} branch has created.`);
-        const updatedPackages = await this.ncu.update(outdatedPackage);
-        if (updatedPackages.length !== 1) {
-            throw new Error(`Failed to update ${outdatedPackage.name}.`);
+        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();
+            }
+            catch (error) {
+                this.logger.error(error);
+                return (0, Either_1.left)({
+                    outdatedPackage,
+                    error
+                });
+            }
+            this.logger.info(`${outdatedPackage.name} has updated from v${outdatedPackage.currentVersion.version} to v${outdatedPackage.newVersion.version}`);
+            await this.git.add(...this.packageManager.packageFiles);
+            const message = this.commitMessageCreator.create(outdatedPackage);
+            this.logger.debug(`message=${message}`);
+            await this.git.commit(message);
+            await this.git.push(branchName);
+            await this.pullRequestCreator.create({
+                outdatedPackage,
+                branchName
+            });
+            return (0, Either_1.right)({
+                outdatedPackage,
+                updated: true
+            });
+        }
+        finally {
+            await this.git.restore(...this.packageManager.packageFiles);
+            await this.git.switch('-');
+            await this.git.removeBranch(branchName);
+            this.logger.info(`${branchName} branch has removed.`);
         }
-        await this.packageManager.install();
-        this.logger.info(`${outdatedPackage.name} has updated from v${outdatedPackage.currentVersion.version} to v${outdatedPackage.newVersion.version}`);
-        await this.git.add(...this.packageManager.packageFiles);
-        const message = this.commitMessageCreator.create(outdatedPackage);
-        this.logger.debug(`message=${message}`);
-        await this.git.commit(message);
-        await this.git.push(branchName);
-        await this.pullRequestCreator.create({
-            outdatedPackage,
-            branchName
-        });
-        await this.git.checkout('-');
-        await this.git.removeBranch(branchName);
-        this.logger.info(`${branchName} branch has removed.`);
-        return {
-            outdatedPackage,
-            updated: true
-        };
     }
 }
 exports.OutdatedPackageProcessor = OutdatedPackageProcessor;

package/dist/processors/SucceededResult.js

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "npm-update-package",
-  "version": "0.10.0",
+  "version": "0.11.0",
   "description": "CLI tool for creating pull request to update npm packages",
   "scripts": {
     "build": "tsc --project tsconfig.build.json",