npm-update-package 0.53.1 → 0.54.0

package/README.md

@@ -247,14 +247,14 @@ npx npm-update-package \
 
 ### packageManager
 
-Package manager of your project.
+Package manager of your project.  
+Since npm-update-package automatically determines which package manager to use, it is usually not necessary to specify this option.
 
 |Name|Value|
 |---|---|
 |cli|`--package-manager`|
 |type|string|
 |required|false|
-|default|`npm`|
 
 Allowed values:
 

package/dist/package.json

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

package/dist/src/file/canReadWrite.js

@@ -0,0 +1,18 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.canReadWrite = void 0;
+const fs_1 = __importDefault(require("fs"));
+// TODO: Add test
+const canReadWrite = async (path) => {
+    try {
+        await fs_1.default.promises.access(path, fs_1.default.constants.R_OK | fs_1.default.constants.W_OK);
+    }
+    catch {
+        return false;
+    }
+    return true;
+};
+exports.canReadWrite = canReadWrite;

package/dist/src/file/index.js

@@ -14,4 +14,5 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
+__exportStar(require("./canReadWrite"), exports);
 __exportStar(require("./readFile"), exports);

package/dist/src/main.js

@@ -63,10 +63,11 @@ const main = async ({ options, logger }) => {
         color: 'A00F21'
     });
     const branchFinder = new github_1.BranchFinder(branches);
-    const packageManager = (0, package_manager_1.createPackageManager)({
-        terminal,
-        packageManager: options.packageManager
+    const packageManagerCreator = new package_manager_1.PackageManagerCreator({
+        options,
+        logger
     });
+    const packageManager = await packageManagerCreator.create(terminal);
     const pullRequestTitleCreator = new github_1.PullRequestTitleCreator(options.prTitle);
     const releasesFetcher = new github_1.ReleasesFetcher({
         options,

package/dist/src/ncu/Ncu.js

@@ -15,12 +15,14 @@ class Ncu {
     }
     async check() {
         return await this.run({
+            packageManager: this.options.packageManager,
             jsonUpgraded: true,
             reject: this.options.ignorePackages
         });
     }
     async update(outdatedPackage) {
         return await this.run({
+            packageManager: this.options.packageManager,
             jsonUpgraded: true,
             filter: outdatedPackage.name,
             upgrade: true

package/dist/src/options/Options.js

@@ -19,10 +19,6 @@ const Options = (0, io_ts_1.intersection)([
             (0, io_ts_1.literal)(logger_1.LogLevel.Debug),
             (0, io_ts_1.literal)(logger_1.LogLevel.Trace)
         ]),
-        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)
-        ]),
         prTitle: io_ts_1.string
     }),
     (0, io_ts_1.partial)({
@@ -30,6 +26,10 @@ const Options = (0, io_ts_1.intersection)([
         gitUserEmail: io_ts_1.string,
         gitUserName: io_ts_1.string,
         ignorePackages: (0, io_ts_1.array)(io_ts_1.string),
+        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)
+        ]),
         prBodyNotes: io_ts_1.string,
         reviewers: (0, io_ts_1.array)(io_ts_1.string)
     })

package/dist/src/options/cliOptions.js

@@ -80,8 +80,7 @@ exports.cliOptions = [
         choices: [
             package_manager_1.PackageManagerName.Npm,
             package_manager_1.PackageManagerName.Yarn
-        ],
-        default: package_manager_1.PackageManagerName.Npm
+        ]
     },
     {
         name: 'pr-body-notes',

package/dist/src/package-manager/PackageManagerCreator.js

@@ -0,0 +1,33 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.PackageManagerCreator = void 0;
+const detectPackageManager_1 = require("./detectPackageManager");
+const Npm_1 = require("./Npm");
+const PackageManagerName_1 = require("./PackageManagerName");
+const Yarn_1 = require("./Yarn");
+class PackageManagerCreator {
+    constructor({ options, logger }) {
+        this.options = options;
+        this.logger = logger;
+    }
+    async create(terminal) {
+        const packageManagerName = await this.getPackageManagerName();
+        this.logger.trace(`packageManagerName=${packageManagerName}`);
+        switch (packageManagerName) {
+            case PackageManagerName_1.PackageManagerName.Npm:
+                this.logger.info('Use npm as package manager');
+                return new Npm_1.Npm(terminal);
+            case PackageManagerName_1.PackageManagerName.Yarn:
+                this.logger.info('Use Yarn as package manager');
+                return new Yarn_1.Yarn(terminal);
+        }
+    }
+    async getPackageManagerName() {
+        if (this.options.packageManager !== undefined) {
+            return this.options.packageManager;
+        }
+        this.logger.info('Try to detect package manager from lock file.');
+        return await (0, detectPackageManager_1.detectPackageManager)();
+    }
+}
+exports.PackageManagerCreator = PackageManagerCreator;

package/dist/src/package-manager/detectPackageManager.js

@@ -0,0 +1,15 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.detectPackageManager = void 0;
+const file_1 = require("../file");
+const PackageManagerName_1 = require("./PackageManagerName");
+const detectPackageManager = async () => {
+    if (await (0, file_1.canReadWrite)('package-lock.json')) {
+        return PackageManagerName_1.PackageManagerName.Npm;
+    }
+    if (await (0, file_1.canReadWrite)('yarn.lock')) {
+        return PackageManagerName_1.PackageManagerName.Yarn;
+    }
+    throw new Error('No lock file exists.');
+};
+exports.detectPackageManager = detectPackageManager;

package/dist/src/package-manager/index.js

@@ -14,8 +14,8 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-__exportStar(require("./createPackageManager"), exports);
 __exportStar(require("./Npm"), exports);
 __exportStar(require("./PackageManager"), exports);
+__exportStar(require("./PackageManagerCreator"), exports);
 __exportStar(require("./PackageManagerName"), exports);
 __exportStar(require("./Yarn"), exports);

package/package.json

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

package/dist/src/package-manager/createPackageManager.js

@@ -1,15 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.createPackageManager = void 0;
-const Npm_1 = require("./Npm");
-const PackageManagerName_1 = require("./PackageManagerName");
-const Yarn_1 = require("./Yarn");
-const createPackageManager = ({ terminal, packageManager }) => {
-    switch (packageManager) {
-        case PackageManagerName_1.PackageManagerName.Npm:
-            return new Npm_1.Npm(terminal);
-        case PackageManagerName_1.PackageManagerName.Yarn:
-            return new Yarn_1.Yarn(terminal);
-    }
-};
-exports.createPackageManager = createPackageManager;