@nx/js 18.0.0-beta.1 → 18.0.0-beta.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/js",
-  "version": "18.0.0-beta.1",
+  "version": "18.0.0-beta.2",
   "private": false,
   "description": "The JS plugin for Nx contains executors and generators that provide the best experience for developing JavaScript and TypeScript projects. ",
   "repository": {
@@ -57,9 +57,9 @@
     "semver": "^7.5.3",
     "source-map-support": "0.5.19",
     "tslib": "^2.3.0",
-    "@nx/devkit": "18.0.0-beta.1",
-    "@nx/workspace": "18.0.0-beta.1",
-    "@nrwl/js": "18.0.0-beta.1"
+    "@nx/devkit": "18.0.0-beta.2",
+    "@nx/workspace": "18.0.0-beta.2",
+    "@nrwl/js": "18.0.0-beta.2"
   },
   "peerDependencies": {
     "verdaccio": "^5.0.4"

package/src/generators/library/library.d.ts

@@ -11,6 +11,6 @@ export interface NormalizedSchema extends LibraryGeneratorSchema {
     parsedTags: string[];
     importPath?: string;
 }
-export type AddLintOptions = Pick<NormalizedSchema, 'name' | 'linter' | 'projectRoot' | 'unitTestRunner' | 'js' | 'setParserOptionsProject' | 'rootProject' | 'bundler'>;
+export type AddLintOptions = Pick<NormalizedSchema, 'name' | 'linter' | 'projectRoot' | 'unitTestRunner' | 'js' | 'setParserOptionsProject' | 'rootProject' | 'bundler' | 'addPlugin'>;
 export declare function addLint(tree: Tree, options: AddLintOptions): Promise<GeneratorCallback>;
 export default libraryGenerator;

package/src/generators/library/library.js

@@ -13,8 +13,10 @@ const init_1 = require("../init/init");
 const generator_1 = require("../setup-verdaccio/generator");
 const create_ts_config_1 = require("../../utils/typescript/create-ts-config");
 const log_show_project_command_1 = require("@nx/devkit/src/utils/log-show-project-command");
+const add_build_target_defaults_1 = require("@nx/devkit/src/generators/add-build-target-defaults");
 async function libraryGenerator(tree, schema) {
     return await libraryGeneratorInternal(tree, {
+        addPlugin: false,
         // provide a default projectNameAndRootFormat to avoid breaking changes
         // to external generators invoking this one
         projectNameAndRootFormat: 'derived',
@@ -49,6 +51,7 @@ async function libraryGeneratorInternal(tree, schema) {
             includeLib: true,
             skipFormat: true,
             testEnvironment: options.testEnvironment,
+            addPlugin: options.addPlugin,
         });
         tasks.push(viteTask);
         createOrEditViteConfig(tree, {
@@ -117,8 +120,10 @@ function addProject(tree, options) {
         options.bundler !== 'none' &&
         options.config !== 'npm-scripts') {
         const outputPath = getOutputPath(options);
+        const executor = getBuildExecutor(options.bundler);
+        (0, add_build_target_defaults_1.addBuildTargetDefaults)(tree, executor);
         projectConfiguration.targets.build = {
-            executor: getBuildExecutor(options.bundler),
+            executor,
             outputs: ['{options.outputPath}'],
             options: {
                 outputPath,
@@ -177,6 +182,7 @@ async function addLint(tree, options) {
         unitTestRunner: options.unitTestRunner,
         setParserOptionsProject: options.setParserOptionsProject,
         rootProject: options.rootProject,
+        addPlugin: options.addPlugin,
     });
     const { addOverrideToLintConfig, lintConfigHasOverride, isEslintConfigSupported, updateOverrideInLintConfig,
     // nx-ignore-next-line
@@ -388,6 +394,7 @@ function replaceJestConfig(tree, options, filesDir) {
     });
 }
 async function normalizeOptions(tree, options) {
+    options.addPlugin ??= process.env.NX_ADD_PLUGINS !== 'false';
     /**
      * We are deprecating the compiler and the buildable options.
      * However, we want to keep the existing behavior for now.

package/src/generators/release-version/utils/update-lock-file.js

@@ -14,23 +14,30 @@ async function updateLockFile(cwd, { dryRun, verbose, generatorOptions, }) {
         }
         return [];
     }
+    const packageManager = (0, devkit_1.detectPackageManager)(cwd);
+    if (packageManager === 'yarn' &&
+        !(0, semver_1.gte)((0, devkit_1.getPackageManagerVersion)(packageManager), '2.0.0')) {
+        // yarn classic does not store workspace data in the lock file, so we don't need to update it
+        if (verbose) {
+            console.log('\nSkipped lock file update because it is not necessary for Yarn Classic.');
+        }
+        return [];
+    }
     const isDaemonEnabled = client_1.daemonClient.enabled();
     if (isDaemonEnabled) {
         // temporarily stop the daemon, as it will error if the lock file is updated
         await client_1.daemonClient.stop();
     }
-    const packageManager = (0, devkit_1.detectPackageManager)(cwd);
     const packageManagerCommands = (0, devkit_1.getPackageManagerCommand)(packageManager);
     let installArgs = generatorOptions?.installArgs || '';
     devkit_1.output.logSingleLine(`Updating ${packageManager} lock file`);
     let env = {};
     if (generatorOptions?.installIgnoreScripts) {
-        if (packageManager === 'yarn' &&
-            (0, semver_1.gte)((0, devkit_1.getPackageManagerVersion)(packageManager), '2.0.0')) {
+        if (packageManager === 'yarn') {
             env = { YARN_ENABLE_SCRIPTS: 'false' };
         }
         else {
-            // npm, pnpm, and yarn classic all use the same --ignore-scripts option
+            // npm and pnpm use the same --ignore-scripts option
             installArgs = `${installArgs} --ignore-scripts`.trim();
         }
     }

package/src/utils/schema.d.ts

@@ -33,6 +33,7 @@ export interface LibraryGeneratorSchema {
   minimal?: boolean;
   rootProject?: boolean;
   simpleName?: boolean;
+  addPlugin?: boolean;
 }
 
 export interface ExecutorOptions {