pnpm 3.7.5 → 3.8.1

package/README.md

@@ -16,7 +16,7 @@ Features:
 
 * **Fast.** As fast as npm and Yarn.
 * **Efficient.** One version of a package is saved only ever once on a disk.
-* **Great for multi-package repositories (a.k.a. monorepos).** See the [recursive](https://pnpm.js.org/docs/en/pnpm-recursive.html) commands.
+* **Great for multi-package repositories (a.k.a. monorepos).** See the [recursive](https://pnpm.js.org/en/cli/recursive) commands.
 * **Strict.** A package can access only dependencies that are specified in its `package.json`.
 * **Deterministic.** Has a lockfile called `pnpm-lock.yaml`.
 * **Works everywhere.** Works on Windows, Linux and OS X.
@@ -56,16 +56,16 @@ curl -L https://unpkg.com/@pnpm/self-installer | node
 Via npm:
 
 ```
-npm install -g pnpm
+npm add -g pnpm
 ```
 
 Once you first installed pnpm, you can upgrade it using pnpm:
 
 ```
-pnpm install -g pnpm
+pnpm add -g pnpm
 ```
 
-> Do you wanna use pnpm on CI servers? See: [Continuous Integration](https://pnpm.js.org/docs/en/continuous-integration.html).
+> Do you wanna use pnpm on CI servers? See: [Continuous Integration](https://pnpm.js.org/en/continuous-integration).
 
 ## Usage
 
@@ -74,10 +74,10 @@ pnpm install -g pnpm
 Just use pnpm in place of npm. For instance, to install run:
 
 ```
-pnpm install lodash
+pnpm install
 ```
 
-For more advanced usage, read [pnpm CLI](https://pnpm.js.org/docs/en/pnpm-cli.html) on our website.
+For more advanced usage, read [pnpm CLI](https://pnpm.js.org/en/pnpm-cli) on our website.
 
 For using the programmatic API, use pnpm's engine: [supi](packages/supi).
 
@@ -103,7 +103,7 @@ Benchmarks on a React app:
 
 ## Support
 
-- [Frequently Asked Questions](https://pnpm.js.org/docs/en/faq.html)
+- [Frequently Asked Questions](https://pnpm.js.org/en/faq)
 - [Stack Overflow](https://stackoverflow.com/questions/tagged/pnpm)
 - [Gitter chat](https://gitter.im/pnpm/pnpm)
 - [Twitter](https://twitter.com/pnpmjs)

package/lib/bin/pnpx.js

@@ -9,5 +9,5 @@ const PNPM_PATH = path.join(__dirname, 'pnpm.js');
     if (workspaceRoot) {
         process.env.PATH = `${path.join(workspaceRoot, 'node_modules/.bin')}${path.delimiter}${process.env.PATH}`;
     }
-    npx(Object.assign({}, npx.parseArgs(process.argv, PNPM_PATH), { installerStdio: 'inherit' }));
+    npx(Object.assign(Object.assign({}, npx.parseArgs(process.argv, PNPM_PATH)), { installerStdio: 'inherit' }));
 })();

package/lib/cmd/help.js

@@ -3,27 +3,19 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const constants_1 = require("@pnpm/constants");
 const common_tags_1 = require("common-tags");
 const getCommandFullName_1 = require("../getCommandFullName");
+const pnpmPkgJson_1 = require("../pnpmPkgJson");
 function default_1(input) {
     const cmdName = getCommandFullName_1.default(input[0]);
-    console.log(getHelpText(cmdName));
+    console.log(`Version ${pnpmPkgJson_1.default.version}\n${getHelpText(cmdName)}`);
 }
 exports.default = default_1;
 function getHelpText(command) {
     switch (getCommandFullName_1.default(command)) {
         case 'install':
             return common_tags_1.stripIndent `
-        pnpm install (with no args, in package dir)
-        pnpm install [<@scope>/]<name>
-        pnpm install [<@scope>/]<name>@<tag>
-        pnpm install [<@scope>/]<name>@<version>
-        pnpm install [<@scope>/]<name>@<version range>
-        pnpm install <git-host>:<git-user>/<repo-name>
-        pnpm install <git repo url>
-        pnpm install <tarball file>
-        pnpm install <tarball url>
-        pnpm install <folder>
-
-        Aliases: i, install, add
+        pnpm install
+
+        Aliases: i
 
         Installs all dependencies of the project in the current working directory.
         To install dependencies in every project of a monorepo, run \`pnpm recursive install\`
@@ -31,12 +23,6 @@ function getHelpText(command) {
 
         Options:
 
-          -P, --save-prod                    save package to your \`dependencies\`. The default behavior
-          -D, --save-dev                     save package to your \`devDependencies\`
-          -O, --save-optional                save package to your \`optionalDependencies\`
-          --save-peer                        save package to your \`peerDependencies\` and \`devDependencies\`
-          -E, --save-exact                   install exact version
-          -g, --global                       install as a global package
           -r                                 run installation recursively in every package found in subdirectories
                                              or in every workspace package, when executed inside a workspace.
                                              For options that may be used with \`-r\`, see "pnpm help recursive"
@@ -111,6 +97,32 @@ function getHelpText(command) {
 
             .
               Includes all packages that are under the current working directory.
+      `;
+        case 'add':
+            return common_tags_1.stripIndent `
+        pnpm add [<@scope>/]<name>
+        pnpm add [<@scope>/]<name>@<tag>
+        pnpm add [<@scope>/]<name>@<version>
+        pnpm add [<@scope>/]<name>@<version range>
+        pnpm add <git-host>:<git-user>/<repo-name>
+        pnpm add <git repo url>
+        pnpm add <tarball file>
+        pnpm add <tarball url>
+        pnpm add <folder>
+
+        Installs a package and any packages that it depends on.
+
+        Options:
+
+          -P, --save-prod                    save package to your \`dependencies\`. The default behavior
+          -D, --save-dev                     save package to your \`devDependencies\`
+          -O, --save-optional                save package to your \`optionalDependencies\`
+          --save-peer                        save package to your \`peerDependencies\` and \`devDependencies\`
+          -E, --save-exact                   install exact version
+          -g, --global                       install as a global package
+          -r                                 run installation recursively in every package found in subdirectories
+                                             or in every workspace package, when executed inside a workspace.
+                                             For options that may be used with \`-r\`, see "pnpm help recursive"
       `;
         case 'import':
             return common_tags_1.stripIndent `
@@ -392,6 +404,8 @@ function getHelpText(command) {
 
           install
 
+          add
+
           update
 
           uninstall [<@scope>/]<pkg>...
@@ -468,6 +482,7 @@ function getHelpText(command) {
 
         manage your dependencies:
           - install
+          - add
           - update
           - uninstall
           - link
@@ -496,6 +511,7 @@ function getHelpText(command) {
         manage you monorepo:
           - recursive exec
           - recursive install
+          - recursive add
           - recursive list
           - recursive outdated
           - recursive rebuild

package/lib/cmd/import.js

@@ -17,7 +17,7 @@ async function installCmd(input, opts) {
     getAllVersionsByPackageNames(npmPackageLock, versionsByPackageNames);
     const preferredVersions = getPreferredVersions(versionsByPackageNames);
     const store = await createStoreController_1.default(opts);
-    const installOpts = Object.assign({}, opts, { lockfileOnly: true, preferredVersions, store: store.path, storeController: store.ctrl });
+    const installOpts = Object.assign(Object.assign({}, opts), { lockfileOnly: true, preferredVersions, store: store.path, storeController: store.ctrl });
     await supi_1.install(await read_importer_manifest_1.readImporterManifestOnly(opts.prefix), installOpts);
 }
 exports.default = installCmd;

package/lib/cmd/index.js

@@ -19,6 +19,7 @@ const uninstall_1 = require("./uninstall");
 const unlink_1 = require("./unlink");
 const update_1 = require("./update");
 exports.default = {
+    add: install_1.default,
     help: help_1.default,
     import: import_1.default,
     install: install_1.default,

package/lib/cmd/install.js

@@ -19,7 +19,7 @@ const OVERWRITE_UPDATE_OPTIONS = {
  * @example
  *     installCmd([ 'lodash', 'foo' ], { silent: true })
  */
-async function installCmd(input, opts) {
+async function installCmd(input, opts, invocation) {
     // `pnpm install ""` is going to be just `pnpm install`
     input = input.filter(Boolean);
     const prefix = opts.prefix || process.cwd();
@@ -30,7 +30,7 @@ async function installCmd(input, opts) {
         opts.hooks = requireHooks_1.default(opts.lockfileDirectory || prefix, opts);
     }
     const store = await createStoreController_1.default(opts);
-    const installOpts = Object.assign({}, opts, { 
+    const installOpts = Object.assign(Object.assign({}, opts), { 
         // In case installation is done in a multi-package repository
         // The dependencies should be built first,
         // so ignoring scripts for now
@@ -52,6 +52,9 @@ async function installCmd(input, opts) {
         delete installOpts.include;
     }
     if (!input || !input.length) {
+        if (invocation === 'add' && opts.useBetaCli) {
+            throw new error_1.default('MISSING_PACKAGE_NAME', '`pnpm add` requires the package name');
+        }
         await supi_1.install(manifest, installOpts);
     }
     else {
@@ -74,7 +77,7 @@ async function installCmd(input, opts) {
         // TODO: reuse somehow the previous read of packages
         // this is not optimal
         const allWorkspacePkgs = await findWorkspacePackages_1.default(opts.workspacePrefix, opts);
-        await recursive_1.recursive(allWorkspacePkgs, [], Object.assign({}, opts, OVERWRITE_UPDATE_OPTIONS, { ignoredPackages: new Set([prefix]), packageSelectors: [
+        await recursive_1.recursive(allWorkspacePkgs, [], Object.assign(Object.assign(Object.assign({}, opts), OVERWRITE_UPDATE_OPTIONS), { ignoredPackages: new Set([prefix]), packageSelectors: [
                 {
                     matcher: prefix,
                     scope: 'dependencies',
@@ -89,7 +92,7 @@ async function installCmd(input, opts) {
                 manifest: await readImporterManifest_1.readImporterManifestOnly(opts.prefix, opts),
                 prefix: opts.prefix,
             },
-        ], Object.assign({}, opts, { pending: true, store: store.path, storeController: store.ctrl }));
+        ], Object.assign(Object.assign({}, opts), { pending: true, store: store.path, storeController: store.ctrl }));
     }
 }
 exports.default = installCmd;

package/lib/cmd/link.js

@@ -31,7 +31,7 @@ exports.default = async (input, opts) => {
     // pnpm link
     if (!input || !input.length) {
         const { manifest, writeImporterManifest } = await readImporterManifest_1.tryReadImporterManifest(opts.globalPrefix, opts);
-        const newManifest = await supi_1.linkToGlobal(cwd, Object.assign({}, linkOpts, { manifest: manifest || {} }));
+        const newManifest = await supi_1.linkToGlobal(cwd, Object.assign(Object.assign({}, linkOpts), { manifest: manifest || {} }));
         await writeImporterManifest(newManifest);
         return;
     }
@@ -55,10 +55,10 @@ exports.default = async (input, opts) => {
     }
     await Promise.all(pkgPaths.map((prefix) => installLimit(async () => {
         const s = await createStoreController_1.cached(storeControllerCache, opts);
-        await supi_1.install(await readImporterManifest_1.readImporterManifestOnly(prefix, opts), Object.assign({}, await getConfigs_1.default(Object.assign({}, opts.cliArgs, { prefix }), {
+        await supi_1.install(await readImporterManifest_1.readImporterManifestOnly(prefix, opts), Object.assign(Object.assign({}, await getConfigs_1.default(Object.assign(Object.assign({}, opts.cliArgs), { prefix }), {
             command: ['link'],
             excludeReporter: true,
-        }), { localPackages, store: s.path, storeController: s.ctrl }));
+        })), { localPackages, store: s.path, storeController: s.ctrl }));
     })));
     const { manifest, writeImporterManifest } = await readImporterManifest_1.default(cwd, opts);
     // When running `pnpm link --production ../source`
@@ -66,7 +66,7 @@ exports.default = async (input, opts) => {
     // The target directory should keep its existing dependencies.
     // Except the ones that are replaced by the link.
     delete linkOpts.include;
-    const newManifest = await supi_1.link(pkgPaths, path.join(cwd, 'node_modules'), Object.assign({}, linkOpts, { manifest }));
+    const newManifest = await supi_1.link(pkgPaths, path.join(cwd, 'node_modules'), Object.assign(Object.assign({}, linkOpts), { manifest }));
     await writeImporterManifest(newManifest);
     await Promise.all(Array.from(storeControllerCache.values())
         .map(async (storeControllerPromise) => {

package/lib/cmd/list.js

@@ -2,7 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const list_1 = require("@pnpm/list");
 async function default_1(args, opts, command) {
-    const output = await render([opts.prefix], args, Object.assign({}, opts, { lockfileDirectory: opts.lockfileDirectory || opts.prefix }), command);
+    const output = await render([opts.prefix], args, Object.assign(Object.assign({}, opts), { lockfileDirectory: opts.lockfileDirectory || opts.prefix }), command);
     if (output)
         console.log(output);
 }

package/lib/cmd/outdated.js

@@ -3,12 +3,27 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const error_1 = require("@pnpm/error");
 const lockfile_file_1 = require("@pnpm/lockfile-file");
 const outdated_1 = require("@pnpm/outdated");
+const semver_diff_1 = require("@pnpm/semver-diff");
 const store_path_1 = require("@pnpm/store-path");
 const chalk_1 = require("chalk");
-const stripColor = require("strip-color");
-const table = require("text-table");
-const createLatestVersionGetter_1 = require("../createLatestVersionGetter");
+const R = require("ramda");
+const table_1 = require("table");
+const wrapAnsi = require("wrap-ansi");
+const createLatestManifestGetter_1 = require("../createLatestManifestGetter");
 const readImporterManifest_1 = require("../readImporterManifest");
+exports.TABLE_OPTIONS = {
+    border: {
+        bodyLeft: '',
+        bottomLeft: '',
+        joinLeft: '',
+        topLeft: '',
+    },
+    columns: {
+        0: {
+            paddingLeft: 0,
+        },
+    },
+};
 async function default_1(args, opts, command) {
     const packages = [
         {
@@ -19,17 +34,111 @@ async function default_1(args, opts, command) {
     const { outdatedPackages } = (await outdatedDependenciesOfWorkspacePackages(packages, args, opts))[0];
     if (!outdatedPackages.length)
         return;
-    const columnNames = ['Package', 'Current', 'Wanted', 'Latest'].map((txt) => chalk_1.default.underline(txt));
-    console.log(table([columnNames].concat(outdatedPackages.map((outdatedPkg) => [
-        chalk_1.default.yellow(outdatedPkg.packageName),
-        outdatedPkg.current || 'missing',
-        chalk_1.default.green(outdatedPkg.wanted),
-        chalk_1.default.magenta(outdatedPkg.latest || ''),
-    ])), {
-        stringLength: (s) => stripColor(s).length,
-    }));
+    const columnNames = [
+        'Package',
+        'Current',
+        'Latest',
+        'Details',
+    ];
+    let columnFns = [
+        renderPackageName,
+        renderCurrent,
+        renderLatest,
+        renderDetails,
+    ];
+    return table_1.table([
+        columnNames,
+        ...R.sortWith([
+            sortBySemverChange,
+            (o1, o2) => o1.packageName.localeCompare(o2.packageName),
+        ], outdatedPackages.map(toOutdatedWithVersionDiff))
+            .map((outdatedPkg) => columnFns.map((fn) => fn(outdatedPkg))),
+    ], exports.TABLE_OPTIONS);
 }
 exports.default = default_1;
+function getCellWidth(data, columnNumber, maxWidth) {
+    return Math.min(maxWidth, data.reduce((maxWidth, row) => Math.max(maxWidth, row[columnNumber].length), 0));
+}
+exports.getCellWidth = getCellWidth;
+function toOutdatedWithVersionDiff(outdated) {
+    if (outdated.latestManifest) {
+        return Object.assign(Object.assign({}, outdated), semver_diff_1.default(outdated.wanted, outdated.latestManifest.version));
+    }
+    return Object.assign(Object.assign({}, outdated), { change: 'unknown' });
+}
+exports.toOutdatedWithVersionDiff = toOutdatedWithVersionDiff;
+function renderPackageName({ belongsTo, packageName }) {
+    switch (belongsTo) {
+        case 'devDependencies': return `${packageName} ${chalk_1.default.dim('(dev)')}`;
+        case 'optionalDependencies': return `${packageName} ${chalk_1.default.dim('(optional)')}`;
+        default: return packageName;
+    }
+}
+exports.renderPackageName = renderPackageName;
+function renderCurrent({ current, wanted }) {
+    let output = current || 'missing';
+    if (current === wanted)
+        return output;
+    return `${output} (wanted ${wanted})`;
+}
+exports.renderCurrent = renderCurrent;
+const DIFF_COLORS = {
+    feature: chalk_1.default.yellowBright.bold,
+    fix: chalk_1.default.greenBright.bold,
+};
+function renderLatest({ latestManifest, change, diff }) {
+    if (!latestManifest)
+        return '';
+    if (change === null || !diff) {
+        return latestManifest.deprecated
+            ? chalk_1.default.redBright.bold('Deprecated')
+            : latestManifest.version;
+    }
+    const highlight = DIFF_COLORS[change] || chalk_1.default.redBright.bold;
+    const same = joinVersionTuples(diff[0], 0);
+    const other = highlight(joinVersionTuples(diff[1], diff[0].length));
+    if (!same)
+        return other;
+    if (!other) {
+        // Happens when current is 1.0.0-rc.0 and latest is 1.0.0
+        return same;
+    }
+    return diff[0].length === 3 ? `${same}-${other}` : `${same}.${other}`;
+}
+exports.renderLatest = renderLatest;
+function joinVersionTuples(versionTuples, startIndex) {
+    const neededForSemver = 3 - startIndex;
+    if (versionTuples.length <= neededForSemver || neededForSemver === 0) {
+        return versionTuples.join('.');
+    }
+    return `${versionTuples.slice(0, neededForSemver).join('.')}-${versionTuples.slice(neededForSemver).join('.')}`;
+}
+function sortBySemverChange(outdated1, outdated2) {
+    return pkgPriority(outdated1) - pkgPriority(outdated2);
+}
+exports.sortBySemverChange = sortBySemverChange;
+function pkgPriority(pkg) {
+    switch (pkg.change) {
+        case null: return 0;
+        case 'fix': return 1;
+        case 'feature': return 2;
+        case 'breaking': return 3;
+        default: return 4;
+    }
+}
+function renderDetails({ latestManifest }) {
+    if (!latestManifest)
+        return '';
+    const outputs = [];
+    if (latestManifest.deprecated) {
+        outputs.push(wrapAnsi(chalk_1.default.redBright(latestManifest.deprecated), 40));
+    }
+    if (latestManifest.homepage) {
+        outputs.push(chalk_1.default.underline(latestManifest.homepage));
+    }
+    return outputs.join('\n');
+}
+exports.renderDetails = renderDetails;
 async function outdatedDependenciesOfWorkspacePackages(pkgs, args, opts) {
     const lockfileDirectory = opts.lockfileDirectory || opts.prefix;
     const currentLockfile = await lockfile_file_1.readCurrentLockfile(lockfileDirectory, { ignoreIncompatible: false });
@@ -38,18 +147,19 @@ async function outdatedDependenciesOfWorkspacePackages(pkgs, args, opts) {
         throw new error_1.default('OUTDATED_NO_LOCKFILE', 'No lockfile in this directory. Run `pnpm install` to generate one.');
     }
     const store = await store_path_1.default(opts.prefix, opts.store);
-    const getLatestVersion = createLatestVersionGetter_1.default(Object.assign({}, opts, { lockfileDirectory,
+    const getLatestManifest = createLatestManifestGetter_1.default(Object.assign(Object.assign({}, opts), { lockfileDirectory,
         store }));
     return Promise.all(pkgs.map(async ({ manifest, path }) => {
         const optsForOutdated = {
             currentLockfile,
-            getLatestVersion,
+            getLatestManifest,
             lockfileDirectory,
             manifest,
             prefix: path,
             wantedLockfile,
         };
         return {
+            manifest,
             outdatedPackages: args.length
                 ? await outdated_1.forPackages(args, optsForOutdated)
                 : await outdated_1.default(optsForOutdated),

package/lib/cmd/prune.js

@@ -13,5 +13,5 @@ exports.default = async (input, opts) => {
             prefix: process.cwd(),
             pruneDirectDependencies: true,
         },
-    ], Object.assign({}, opts, { pruneStore: true, store: store.path, storeController: store.ctrl }));
+    ], Object.assign(Object.assign({}, opts), { pruneStore: true, store: store.path, storeController: store.ctrl }));
 };

package/lib/cmd/publish.js

@@ -68,7 +68,7 @@ async function fakeRegularManifest(opts, fn) {
     await Promise.all(copiedLicenses.map((copiedLicense) => fs.unlink(copiedLicense)));
 }
 async function makePublishManifest(prefix, originalManifest) {
-    const publishManifest = Object.assign({}, originalManifest, { dependencies: await makePublishDependencies(prefix, originalManifest.dependencies), optionalDependencies: await makePublishDependencies(prefix, originalManifest.optionalDependencies) });
+    const publishManifest = Object.assign(Object.assign({}, originalManifest), { dependencies: await makePublishDependencies(prefix, originalManifest.dependencies), optionalDependencies: await makePublishDependencies(prefix, originalManifest.optionalDependencies) });
     if (originalManifest.publishConfig) {
         if (originalManifest.publishConfig.main) {
             publishManifest.main = originalManifest.publishConfig.main;

package/lib/cmd/recursive/exec.js

@@ -14,7 +14,7 @@ exports.default = async (packageChunks, graph, args, cmd, opts) => {
             try {
                 await execa(args[0], args.slice(1), {
                     cwd: prefix,
-                    env: Object.assign({}, process.env, { PNPM_PACKAGE_NAME: graph[prefix].package.manifest.name }),
+                    env: Object.assign(Object.assign({}, process.env), { PNPM_PACKAGE_NAME: graph[prefix].package.manifest.name }),
                     stdio: 'inherit',
                 });
                 result.passes++;

package/lib/cmd/recursive/index.js

@@ -30,6 +30,7 @@ const outdated_1 = require("./outdated");
 const recursiveSummary_1 = require("./recursiveSummary");
 const run_1 = require("./run");
 const supportedRecursiveCommands = new Set([
+    'add',
     'install',
     'uninstall',
     'update',
@@ -107,6 +108,11 @@ async function recursive(allPkgs, input, opts, cmdFullName, cmd) {
         case 'outdated':
             await outdated_1.default(pkgs, input, cmd, opts); // tslint:disable-line:no-any
             return true;
+        case 'add':
+            if (opts.useBetaCli && (!input || !input.length)) {
+                throw new error_1.default('MISSING_PACKAGE_NAME', '`pnpm recursive add` requires the package name');
+            }
+            break;
     }
     const chunks = opts.sort
         ? sortPackages(pkgGraphResult.graph)
@@ -119,7 +125,7 @@ async function recursive(allPkgs, input, opts, cmdFullName, cmd) {
             throwOnFail(await run_1.default(chunks, pkgGraphResult.graph, input, cmd, opts)); // tslint:disable-line:no-any
             return true;
         case 'update':
-            opts = Object.assign({}, opts, { update: true, allowNew: false }); // tslint:disable-line:no-any
+            opts = Object.assign(Object.assign({}, opts), { update: true, allowNew: false }); // tslint:disable-line:no-any
             break;
         case 'exec':
             throwOnFail(await exec_1.default(chunks, pkgGraphResult.graph, input, cmd, opts)); // tslint:disable-line:no-any
@@ -131,7 +137,7 @@ async function recursive(allPkgs, input, opts, cmdFullName, cmd) {
     // That's why saveState that is passed to the install engine
     // does nothing.
     const saveState = store.ctrl.saveState;
-    const storeController = Object.assign({}, store.ctrl, { saveState: async () => undefined });
+    const storeController = Object.assign(Object.assign({}, store.ctrl), { saveState: async () => undefined });
     const localPackages = opts.linkWorkspacePackages && cmdFullName !== 'unlink'
         ? findWorkspacePackages_1.arrayOfLocalPackagesToMap(allPkgs)
         : {};
@@ -173,7 +179,7 @@ async function recursive(allPkgs, input, opts, cmdFullName, cmd) {
     }
     if (cmdFullName !== 'rebuild') {
         // For a workspace with shared lockfile
-        if (opts.lockfileDirectory && ['install', 'uninstall', 'update'].includes(cmdFullName)) {
+        if (opts.lockfileDirectory && ['add', 'install', 'uninstall', 'update'].includes(cmdFullName)) {
             if (opts.shamefullyFlatten) {
                 logger_1.default.info({ message: 'Only the root workspace package is going to have a flat node_modules', prefix: opts.lockfileDirectory });
             }
@@ -219,7 +225,7 @@ async function recursive(allPkgs, input, opts, cmdFullName, cmd) {
                         return;
                     case 'installSome':
                         mutatedImporters.push({
-                            allowNew: cmdFullName === 'install',
+                            allowNew: cmdFullName === 'install' || cmdFullName === 'add',
                             dependencySelectors: currentInput,
                             manifest,
                             mutation,
@@ -244,7 +250,7 @@ async function recursive(allPkgs, input, opts, cmdFullName, cmd) {
                         return;
                 }
             }));
-            const mutatedPkgs = await supi_1.mutateModules(mutatedImporters, Object.assign({}, installOpts, { hooks, storeController: store.ctrl }));
+            const mutatedPkgs = await supi_1.mutateModules(mutatedImporters, Object.assign(Object.assign({}, installOpts), { hooks, storeController: store.ctrl }));
             await Promise.all(mutatedPkgs
                 .filter((mutatedPkg, index) => mutatedImporters[index].mutation !== 'install')
                 .map(({ manifest, prefix }, index) => writeImporterManifests[index](manifest)));
@@ -294,10 +300,10 @@ async function recursive(allPkgs, input, opts, cmdFullName, cmd) {
                         break;
                 }
                 const localConfigs = await memReadLocalConfigs(prefix);
-                const newManifest = await action(manifest, Object.assign({}, installOpts, localConfigs, { bin: path.join(prefix, 'node_modules', '.bin'), hooks, ignoreScripts: true, pinnedVersion: getPinnedVersion_1.default({
+                const newManifest = await action(manifest, Object.assign(Object.assign(Object.assign({}, installOpts), localConfigs), { bin: path.join(prefix, 'node_modules', '.bin'), hooks, ignoreScripts: true, pinnedVersion: getPinnedVersion_1.default({
                         saveExact: typeof localConfigs.saveExact === 'boolean' ? localConfigs.saveExact : opts.saveExact,
                         savePrefix: typeof localConfigs.savePrefix === 'string' ? localConfigs.savePrefix : opts.savePrefix,
-                    }), prefix, rawNpmConfig: Object.assign({}, installOpts.rawNpmConfig, localConfigs), storeController }));
+                    }), prefix, rawNpmConfig: Object.assign(Object.assign({}, installOpts.rawNpmConfig), localConfigs), storeController }));
                 if (action !== supi_1.install) {
                     await writeImporterManifest(newManifest);
                 }
@@ -320,14 +326,17 @@ async function recursive(allPkgs, input, opts, cmdFullName, cmd) {
         await saveState();
     }
     if (cmdFullName === 'rebuild' ||
-        !opts.lockfileOnly && !opts.ignoreScripts && (cmdFullName === 'install' || cmdFullName === 'update' || cmdFullName === 'unlink')) {
+        !opts.lockfileOnly && !opts.ignoreScripts && (cmdFullName === 'add' ||
+            cmdFullName === 'install' ||
+            cmdFullName === 'update' ||
+            cmdFullName === 'unlink')) {
         const action = (cmdFullName !== 'rebuild' || input.length === 0
             ? supi_1.rebuild
             : (importers, opts) => supi_1.rebuildPkgs(importers, input, opts) // tslint:disable-line
         );
         if (opts.lockfileDirectory) {
             const importers = await getImporters();
-            await action(importers, Object.assign({}, installOpts, { pending: cmdFullName !== 'rebuild' || opts.pending === true }));
+            await action(importers, Object.assign(Object.assign({}, installOpts), { pending: cmdFullName !== 'rebuild' || opts.pending === true }));
             return true;
         }
         const limitRebuild = p_limit_1.default(opts.workspaceConcurrency);
@@ -344,7 +353,7 @@ async function recursive(allPkgs, input, opts, cmdFullName, cmd) {
                             manifest: manifestsByPath[prefix].manifest,
                             prefix,
                         },
-                    ], Object.assign({}, installOpts, localConfigs, { bin: path.join(prefix, 'node_modules', '.bin'), pending: cmdFullName !== 'rebuild' || opts.pending === true, prefix, rawNpmConfig: Object.assign({}, installOpts.rawNpmConfig, localConfigs) }));
+                    ], Object.assign(Object.assign(Object.assign({}, installOpts), localConfigs), { bin: path.join(prefix, 'node_modules', '.bin'), pending: cmdFullName !== 'rebuild' || opts.pending === true, prefix, rawNpmConfig: Object.assign(Object.assign({}, installOpts.rawNpmConfig), localConfigs) }));
                     result.passes++;
                 }
                 catch (err) {

package/lib/cmd/recursive/list.js

@@ -4,13 +4,13 @@ const logger_1 = require("@pnpm/logger");
 const list_1 = require("../list");
 exports.default = async (pkgs, args, cmd, opts) => {
     if (opts.lockfileDirectory) {
-        console.log(await list_1.render(pkgs.map((pkg) => pkg.path), args, Object.assign({}, opts, { alwaysPrintRootPackage: opts.depth === -1, lockfileDirectory: opts.lockfileDirectory }), cmd));
+        console.log(await list_1.render(pkgs.map((pkg) => pkg.path), args, Object.assign(Object.assign({}, opts), { alwaysPrintRootPackage: opts.depth === -1, lockfileDirectory: opts.lockfileDirectory }), cmd));
         return;
     }
     const outputs = [];
     for (const { path } of pkgs) {
         try {
-            const output = await list_1.render([path], args, Object.assign({}, opts, { alwaysPrintRootPackage: opts.depth === -1, lockfileDirectory: opts.lockfileDirectory || path }), cmd);
+            const output = await list_1.render([path], args, Object.assign(Object.assign({}, opts), { alwaysPrintRootPackage: opts.depth === -1, lockfileDirectory: opts.lockfileDirectory || path }), cmd);
             if (!output)
                 continue;
             outputs.push(output);