@nx/js 17.3.0-beta.5 → 17.3.0-beta.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/js",
-  "version": "17.3.0-beta.5",
+  "version": "17.3.0-beta.8",
   "private": false,
   "description": "The JS plugin for Nx contains executors and generators that provide the best experience for developing JavaScript and TypeScript projects. ",
   "repository": {
@@ -52,14 +52,14 @@
     "tsconfig-paths": "^4.1.2",
     "ignore": "^5.0.4",
     "js-tokens": "^4.0.0",
-    "minimatch": "3.0.5",
+    "minimatch": "9.0.3",
     "ora": "5.3.0",
     "semver": "7.5.3",
     "source-map-support": "0.5.19",
     "tslib": "^2.3.0",
-    "@nx/devkit": "17.3.0-beta.5",
-    "@nx/workspace": "17.3.0-beta.5",
-    "@nrwl/js": "17.3.0-beta.5"
+    "@nx/devkit": "17.3.0-beta.8",
+    "@nx/workspace": "17.3.0-beta.8",
+    "@nrwl/js": "17.3.0-beta.8"
   },
   "peerDependencies": {
     "verdaccio": "^5.0.4"

package/src/executors/verdaccio/verdaccio.impl.js

@@ -149,7 +149,7 @@ function getYarnUnsafeHttpWhitelist(isYarnV1) {
         : null;
 }
 function setYarnUnsafeHttpWhitelist(currentWhitelist, options) {
-    if (currentWhitelist.size > 1) {
+    if (currentWhitelist.size > 0) {
         (0, child_process_1.execSync)(`yarn config set unsafeHttpWhitelist --json '${JSON.stringify(Array.from(currentWhitelist))}'` + (options.location === 'user' ? ' --home' : ''));
     }
     else {

package/src/generators/init/init.js

@@ -50,7 +50,7 @@ async function initGenerator(tree, schema) {
         '@swc-node/register': versions_1.swcNodeVersion,
         '@swc/core': versions_1.swcCoreVersion,
     };
-    if (!schema.js) {
+    if (!schema.js && !schema.keepExistingVersions) {
         const installedTsVersion = await getInstalledTypescriptVersion(tree);
         if (!installedTsVersion ||
             !(0, semver_2.satisfies)(installedTsVersion, versions_1.supportedTypescriptVersions, {
@@ -96,7 +96,7 @@ async function initGenerator(tree, schema) {
         });
     }
     const installTask = !schema.skipPackageJson
-        ? (0, devkit_1.addDependenciesToPackageJson)(tree, {}, devDependencies)
+        ? (0, devkit_1.addDependenciesToPackageJson)(tree, {}, devDependencies, undefined, schema.keepExistingVersions)
         : () => { };
     tasks.push(installTask);
     (0, devkit_1.ensurePackage)('prettier', versions_1.prettierVersion);

package/src/generators/init/schema.d.ts

@@ -2,5 +2,6 @@ export interface InitSchema {
   js?: boolean;
   skipFormat?: boolean;
   skipPackageJson?: boolean;
+  keepExistingVersions?: boolean;
   tsConfigName?: string;
 }

package/src/generators/init/schema.json

@@ -21,6 +21,12 @@
       "description": "Skip adding package.json dependencies",
       "x-priority": "internal"
     },
+    "keepExistingVersions": {
+      "type": "boolean",
+      "x-priority": "internal",
+      "description": "Keep existing dependencies versions",
+      "default": false
+    },
     "tsConfigName": {
       "type": "string",
       "description": "Customize the generated tsconfig file name.",

package/src/generators/library/library.js

@@ -163,7 +163,7 @@ function addProject(tree, options) {
 async function addLint(tree, options) {
     const { lintProjectGenerator } = (0, devkit_1.ensurePackage)('@nx/eslint', versions_1.nxVersion);
     const projectConfiguration = (0, devkit_1.readProjectConfiguration)(tree, options.name);
-    const task = lintProjectGenerator(tree, {
+    const task = await lintProjectGenerator(tree, {
         project: options.name,
         linter: options.linter,
         skipFormat: true,

package/src/generators/release-version/release-version.js

@@ -25,6 +25,14 @@ async function releaseVersionGenerator(tree, options) {
             // The node semver library classes a leading `v` as valid, but we want to ensure it is not present in the final version
             options.specifier = options.specifier.replace(/^v/, '');
         }
+        if (options.versionPrefix &&
+            version_1.validReleaseVersionPrefixes.indexOf(options.versionPrefix) === -1) {
+            throw new Error(`Invalid value for version.generatorOptions.versionPrefix: "${options.versionPrefix}"
+
+Valid values are: ${version_1.validReleaseVersionPrefixes
+                .map((s) => `"${s}"`)
+                .join(', ')}`);
+        }
         if (options.firstRelease) {
             // always use disk as a fallback for the first release
             options.fallbackCurrentVersionResolver = 'disk';
@@ -242,7 +250,7 @@ To fix this you will either need to add a package.json file at that location, or
             versionData[projectName] = {
                 currentVersion,
                 dependentProjects,
-                newVersion: null, // will stay as null in the final result the case that no changes are detected
+                newVersion: null, // will stay as null in the final result in the case that no changes are detected
             };
             if (!specifier) {
                 log(`🚫 Skipping versioning "${projectPackageJson.name}" as no changes were detected.`);
@@ -262,8 +270,23 @@ To fix this you will either need to add a package.json file at that location, or
             }
             for (const dependentProject of dependentProjects) {
                 (0, devkit_1.updateJson)(tree, (0, devkit_1.joinPathFragments)(projectNameToPackageRootMap.get(dependentProject.source), 'package.json'), (json) => {
-                    json[dependentProject.dependencyCollection][packageName] =
-                        newVersion;
+                    // Auto (i.e.infer existing) by default
+                    let versionPrefix = options.versionPrefix ?? 'auto';
+                    // For auto, we infer the prefix based on the current version of the dependent
+                    if (versionPrefix === 'auto') {
+                        versionPrefix = ''; // we don't want to end up printing auto
+                        const current = json[dependentProject.dependencyCollection][packageName];
+                        if (current) {
+                            const prefixMatch = current.match(/^[~^]/);
+                            if (prefixMatch) {
+                                versionPrefix = prefixMatch[0];
+                            }
+                            else {
+                                versionPrefix = '';
+                            }
+                        }
+                    }
+                    json[dependentProject.dependencyCollection][packageName] = `${versionPrefix}${newVersion}`;
                     return json;
                 });
             }

package/src/generators/setup-build/generator.js

@@ -5,6 +5,7 @@ const devkit_1 = require("@nx/devkit");
 const add_swc_config_1 = require("../../utils/swc/add-swc-config");
 const add_swc_dependencies_1 = require("../../utils/swc/add-swc-dependencies");
 const versions_1 = require("../../utils/versions");
+const add_build_target_defaults_1 = require("@nx/devkit/src/generators/add-build-target-defaults");
 async function setupBuildGenerator(tree, options) {
     const tasks = [];
     const project = (0, devkit_1.readProjectConfiguration)(tree, options.project);
@@ -101,6 +102,7 @@ async function setupBuildGenerator(tree, options) {
             break;
         }
         case 'tsc': {
+            (0, add_build_target_defaults_1.addBuildTargetDefaults)(tree, '@nx/js:tsc');
             const outputPath = (0, devkit_1.joinPathFragments)('dist', project.root);
             project.targets[buildTarget] = {
                 executor: `@nx/js:tsc`,
@@ -116,6 +118,7 @@ async function setupBuildGenerator(tree, options) {
             break;
         }
         case 'swc': {
+            (0, add_build_target_defaults_1.addBuildTargetDefaults)(tree, '@nx/js:swc');
             const outputPath = (0, devkit_1.joinPathFragments)('dist', project.root);
             project.targets[buildTarget] = {
                 executor: `@nx/js:swc`,

package/src/utils/assets/copy-assets-handler.js

@@ -1,7 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.CopyAssetsHandler = exports.defaultFileEventHandler = void 0;
-const minimatch = require("minimatch");
+const minimatch_1 = require("minimatch");
 const path = require("path");
 const fse = require("fs-extra");
 const ignore_1 = require("ignore");
@@ -114,8 +114,8 @@ class CopyAssetsHandler {
         for (const event of events) {
             const pathFromRoot = path.relative(this.rootDir, event.path);
             for (const ag of this.assetGlobs) {
-                if (minimatch(pathFromRoot, ag.pattern) &&
-                    !ag.ignore?.some((ig) => minimatch(pathFromRoot, ig)) &&
+                if ((0, minimatch_1.minimatch)(pathFromRoot, ag.pattern) &&
+                    !ag.ignore?.some((ig) => (0, minimatch_1.minimatch)(pathFromRoot, ig)) &&
                     !this.ignore.ignores(pathFromRoot)) {
                     const relPath = path.relative(ag.input, pathFromRoot);
                     const destPath = relPath.startsWith('..') ? pathFromRoot : relPath;
@@ -134,7 +134,7 @@ class CopyAssetsHandler {
     }
     filesToEvent(files, assetGlob) {
         return files.reduce((acc, src) => {
-            if (!assetGlob.ignore?.some((ig) => minimatch(src, ig)) &&
+            if (!assetGlob.ignore?.some((ig) => (0, minimatch_1.minimatch)(src, ig)) &&
                 !this.ignore.ignores(src)) {
                 const relPath = path.relative(assetGlob.input, src);
                 const dest = relPath.startsWith('..') ? src : relPath;