@nx/js 17.0.4 → 17.0.6

package/src/generators/library/library.js

@@ -3,21 +3,17 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.addLint = exports.libraryGeneratorInternal = exports.libraryGenerator = void 0;
 const devkit_1 = require("@nx/devkit");
 const project_name_and_root_utils_1 = require("@nx/devkit/src/generators/project-name-and-root-utils");
-const add_build_target_defaults_1 = require("@nx/devkit/src/generators/add-build-target-defaults");
-const log_show_project_command_1 = require("@nx/devkit/src/utils/log-show-project-command");
-const find_matching_projects_1 = require("nx/src/utils/find-matching-projects");
+const ts_config_1 = require("../../utils/typescript/ts-config");
 const path_1 = require("path");
+const minimal_publish_script_1 = require("../../utils/minimal-publish-script");
 const add_swc_config_1 = require("../../utils/swc/add-swc-config");
 const add_swc_dependencies_1 = require("../../utils/swc/add-swc-dependencies");
-const create_ts_config_1 = require("../../utils/typescript/create-ts-config");
-const ts_config_1 = require("../../utils/typescript/ts-config");
 const versions_1 = require("../../utils/versions");
 const init_1 = require("../init/init");
 const generator_1 = require("../setup-verdaccio/generator");
-const defaultOutputDirectory = 'dist';
+const create_ts_config_1 = require("../../utils/typescript/create-ts-config");
 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',
@@ -26,6 +22,7 @@ async function libraryGenerator(tree, schema) {
 }
 exports.libraryGenerator = libraryGenerator;
 async function libraryGeneratorInternal(tree, schema) {
+    const filesDir = (0, path_1.join)(__dirname, './files');
     const tasks = [];
     tasks.push(await (0, init_1.default)(tree, {
         ...schema,
@@ -33,11 +30,9 @@ async function libraryGeneratorInternal(tree, schema) {
         tsConfigName: schema.rootProject ? 'tsconfig.json' : 'tsconfig.base.json',
     }));
     const options = await normalizeOptions(tree, schema);
-    createFiles(tree, options);
-    await addProject(tree, options);
-    if (!options.skipPackageJson) {
-        tasks.push(addProjectDependencies(tree, options));
-    }
+    createFiles(tree, options, `${filesDir}/lib`);
+    addProject(tree, options);
+    tasks.push(addProjectDependencies(tree, options));
     if (options.publishable) {
         tasks.push(await (0, generator_1.default)(tree, { ...options, skipFormat: true }));
     }
@@ -51,7 +46,6 @@ async function libraryGeneratorInternal(tree, schema) {
             includeLib: true,
             skipFormat: true,
             testEnvironment: options.testEnvironment,
-            addPlugin: options.addPlugin,
         });
         tasks.push(viteTask);
         createOrEditViteConfig(tree, {
@@ -69,7 +63,7 @@ async function libraryGeneratorInternal(tree, schema) {
         const jestCallback = await addJest(tree, options);
         tasks.push(jestCallback);
         if (options.bundler === 'swc' || options.bundler === 'rollup') {
-            replaceJestConfig(tree, options);
+            replaceJestConfig(tree, options, `${filesDir}/jest-config`);
         }
     }
     else if (options.unitTestRunner === 'vitest' &&
@@ -79,7 +73,7 @@ async function libraryGeneratorInternal(tree, schema) {
         const vitestTask = await vitestGenerator(tree, {
             project: options.name,
             uiFramework: 'none',
-            coverageProvider: 'v8',
+            coverageProvider: 'c8',
             skipFormat: true,
             testEnvironment: options.testEnvironment,
         });
@@ -102,18 +96,10 @@ async function libraryGeneratorInternal(tree, schema) {
     if (!options.skipFormat) {
         await (0, devkit_1.formatFiles)(tree);
     }
-    if (options.publishable) {
-        tasks.push(() => {
-            logNxReleaseDocsInfo();
-        });
-    }
-    tasks.push(() => {
-        (0, log_show_project_command_1.logShowProjectCommand)(options.name);
-    });
     return (0, devkit_1.runTasksInSerial)(...tasks);
 }
 exports.libraryGeneratorInternal = libraryGeneratorInternal;
-async function addProject(tree, options) {
+function addProject(tree, options) {
     const projectConfiguration = {
         root: options.projectRoot,
         sourceRoot: (0, devkit_1.joinPathFragments)(options.projectRoot, 'src'),
@@ -125,10 +111,8 @@ async 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,
+            executor: getBuildExecutor(options.bundler),
             outputs: ['{options.outputPath}'],
             options: {
                 outputPath,
@@ -156,24 +140,11 @@ async function addProject(tree, options) {
             projectConfiguration.targets.build.options.assets.push((0, devkit_1.joinPathFragments)(options.projectRoot, '*.md'));
         }
         if (options.publishable) {
-            const packageRoot = (0, path_1.join)(defaultOutputDirectory, '{projectRoot}');
-            projectConfiguration.targets ??= {};
-            projectConfiguration.targets['nx-release-publish'] = {
-                options: {
-                    packageRoot,
-                },
-            };
-            projectConfiguration.release = {
-                version: {
-                    generatorOptions: {
-                        packageRoot,
-                        // using git tags to determine the current version is required here because
-                        // the version in the package root is overridden with every build
-                        currentVersionResolver: 'git-tag',
-                    },
-                },
+            const publishScriptPath = (0, minimal_publish_script_1.addMinimalPublishScript)(tree);
+            projectConfiguration.targets.publish = {
+                command: `node ${publishScriptPath} ${options.name} {args.ver} {args.tag}`,
+                dependsOn: ['build'],
             };
-            await addProjectToNxReleaseConfig(tree, options, projectConfiguration);
         }
     }
     if (options.config === 'workspace' || options.config === 'project') {
@@ -189,8 +160,11 @@ async function addProject(tree, options) {
 }
 async function addLint(tree, options) {
     const { lintProjectGenerator } = (0, devkit_1.ensurePackage)('@nx/eslint', versions_1.nxVersion);
+    const { mapLintPattern } = 
+    // nx-ignore-next-line
+    require('@nx/eslint/src/generators/lint-project/lint-project');
     const projectConfiguration = (0, devkit_1.readProjectConfiguration)(tree, options.name);
-    const task = await lintProjectGenerator(tree, {
+    const task = lintProjectGenerator(tree, {
         project: options.name,
         linter: options.linter,
         skipFormat: true,
@@ -198,9 +172,11 @@ async function addLint(tree, options) {
             (0, devkit_1.joinPathFragments)(options.projectRoot, 'tsconfig.lib.json'),
         ],
         unitTestRunner: options.unitTestRunner,
+        eslintFilePatterns: [
+            mapLintPattern(options.projectRoot, options.js ? 'js' : 'ts', options.rootProject),
+        ],
         setParserOptionsProject: options.setParserOptionsProject,
         rootProject: options.rootProject,
-        addPlugin: options.addPlugin,
     });
     const { addOverrideToLintConfig, lintConfigHasOverride, isEslintConfigSupported, updateOverrideInLintConfig,
     // nx-ignore-next-line
@@ -302,10 +278,10 @@ function addBabelRc(tree, options) {
     };
     (0, devkit_1.writeJson)(tree, (0, path_1.join)(options.projectRoot, filename), babelrc);
 }
-function createFiles(tree, options) {
+function createFiles(tree, options, filesDir) {
     const { className, name, propertyName } = (0, devkit_1.names)(options.projectNames.projectFileName);
     createProjectTsConfigJson(tree, options);
-    (0, devkit_1.generateFiles)(tree, (0, path_1.join)(__dirname, './files/lib'), options.projectRoot, {
+    (0, devkit_1.generateFiles)(tree, filesDir, options.projectRoot, {
         ...options,
         dot: '.',
         className,
@@ -319,22 +295,6 @@ function createFiles(tree, options) {
         buildable: options.bundler && options.bundler !== 'none',
         hasUnitTestRunner: options.unitTestRunner !== 'none',
     });
-    if (!options.rootProject) {
-        (0, devkit_1.generateFiles)(tree, (0, path_1.join)(__dirname, './files/readme'), options.projectRoot, {
-            ...options,
-            dot: '.',
-            className,
-            name,
-            propertyName,
-            js: !!options.js,
-            cliCommand: 'nx',
-            strict: undefined,
-            tmpl: '',
-            offsetFromRoot: (0, devkit_1.offsetFromRoot)(options.projectRoot),
-            buildable: options.bundler && options.bundler !== 'none',
-            hasUnitTestRunner: options.unitTestRunner !== 'none',
-        });
-    }
     if (options.bundler === 'swc' || options.bundler === 'rollup') {
         (0, add_swc_dependencies_1.addSwcDependencies)(tree);
         (0, add_swc_config_1.addSwcConfig)(tree, options.projectRoot, options.bundler === 'swc' ? 'commonjs' : 'es6');
@@ -358,9 +318,6 @@ function createFiles(tree, options) {
             if (json.private && (options.publishable || options.rootProject)) {
                 delete json.private;
             }
-            if (!options.publishable && !options.rootProject) {
-                json.private = true;
-            }
             return {
                 ...json,
                 dependencies: {
@@ -372,16 +329,12 @@ function createFiles(tree, options) {
         });
     }
     else {
-        const packageJson = {
+        (0, devkit_1.writeJson)(tree, packageJsonPath, {
             name: options.importPath,
             version: '0.0.1',
             dependencies: determineDependencies(options),
             ...determineEntryFields(options),
-        };
-        if (!options.publishable && !options.rootProject) {
-            packageJson.private = true;
-        }
-        (0, devkit_1.writeJson)(tree, packageJsonPath, packageJson);
+        });
     }
     if (options.config === 'npm-scripts') {
         (0, devkit_1.updateJson)(tree, packageJsonPath, (json) => {
@@ -418,8 +371,7 @@ async function addJest(tree, options) {
                 : undefined,
     });
 }
-function replaceJestConfig(tree, options) {
-    const filesDir = (0, path_1.join)(__dirname, './files/jest-config');
+function replaceJestConfig(tree, options, filesDir) {
     // the existing config has to be deleted otherwise the new config won't overwrite it
     const existingJestConfig = (0, devkit_1.joinPathFragments)(filesDir, `jest.config.${options.js ? 'js' : 'ts'}`);
     if (tree.exists(existingJestConfig)) {
@@ -436,10 +388,6 @@ function replaceJestConfig(tree, options) {
     });
 }
 async function normalizeOptions(tree, options) {
-    const nxJson = (0, devkit_1.readNxJson)(tree);
-    const addPlugin = process.env.NX_ADD_PLUGINS !== 'false' &&
-        nxJson.useInferencePlugins !== false;
-    options.addPlugin ??= addPlugin;
     /**
      * We are deprecating the compiler and the buildable options.
      * However, we want to keep the existing behavior for now.
@@ -474,7 +422,7 @@ async function normalizeOptions(tree, options) {
             options.bundler = 'tsc';
         }
     }
-    // This is to preserve old behavior, buildable: false
+    // This is to preserve old behaviour, buildable: false
     if (options.publishable === false && options.buildable === false) {
         options.bundler = 'none';
     }
@@ -566,7 +514,7 @@ function getBuildExecutor(bundler) {
     }
 }
 function getOutputPath(options) {
-    const parts = [defaultOutputDirectory];
+    const parts = ['dist'];
     if (options.projectRoot === '.') {
         parts.push(options.name);
     }
@@ -658,94 +606,4 @@ function determineEntryFields(options) {
         }
     }
 }
-function projectsConfigMatchesProject(projectsConfig, project) {
-    if (!projectsConfig) {
-        return false;
-    }
-    if (typeof projectsConfig === 'string') {
-        projectsConfig = [projectsConfig];
-    }
-    const graph = {
-        [project.name]: project,
-    };
-    const matchingProjects = (0, find_matching_projects_1.findMatchingProjects)(projectsConfig, graph);
-    return matchingProjects.includes(project.name);
-}
-async function addProjectToNxReleaseConfig(tree, options, projectConfiguration) {
-    const nxJson = (0, devkit_1.readNxJson)(tree);
-    const addPreVersionCommand = () => {
-        const pmc = (0, devkit_1.getPackageManagerCommand)();
-        nxJson.release = {
-            ...nxJson.release,
-            version: {
-                preVersionCommand: `${pmc.dlx} nx run-many -t build`,
-                ...nxJson.release?.version,
-            },
-        };
-    };
-    if (!nxJson.release || (!nxJson.release.projects && !nxJson.release.groups)) {
-        // skip adding any projects configuration since the new project should be
-        // automatically included by nx release's default project detection logic
-        addPreVersionCommand();
-        (0, devkit_1.writeJson)(tree, 'nx.json', nxJson);
-        return;
-    }
-    const project = {
-        name: options.name,
-        type: 'lib',
-        data: {
-            root: projectConfiguration.root,
-            tags: projectConfiguration.tags,
-        },
-    };
-    if (projectsConfigMatchesProject(nxJson.release.projects, project)) {
-        devkit_1.output.log({
-            title: `Project already included in existing release configuration`,
-        });
-        addPreVersionCommand();
-        (0, devkit_1.writeJson)(tree, 'nx.json', nxJson);
-        return;
-    }
-    if (Array.isArray(nxJson.release.projects)) {
-        nxJson.release.projects.push(options.name);
-        addPreVersionCommand();
-        (0, devkit_1.writeJson)(tree, 'nx.json', nxJson);
-        devkit_1.output.log({
-            title: `Added project to existing release configuration`,
-        });
-    }
-    if (nxJson.release.groups) {
-        const allGroups = Object.entries(nxJson.release.groups);
-        for (const [name, group] of allGroups) {
-            if (projectsConfigMatchesProject(group.projects, project)) {
-                addPreVersionCommand();
-                (0, devkit_1.writeJson)(tree, 'nx.json', nxJson);
-                return `Project already included in existing release configuration for group ${name}`;
-            }
-        }
-        devkit_1.output.warn({
-            title: `Could not find a release group that includes ${options.name}`,
-            bodyLines: [
-                `Ensure that ${options.name} is included in a release group's "projects" list in nx.json so it can be published with "nx release"`,
-            ],
-        });
-        addPreVersionCommand();
-        (0, devkit_1.writeJson)(tree, 'nx.json', nxJson);
-        return;
-    }
-    if (typeof nxJson.release.projects === 'string') {
-        nxJson.release.projects = [nxJson.release.projects, options.name];
-        addPreVersionCommand();
-        (0, devkit_1.writeJson)(tree, 'nx.json', nxJson);
-        devkit_1.output.log({
-            title: `Added project to existing release configuration`,
-        });
-        return;
-    }
-}
-function logNxReleaseDocsInfo() {
-    devkit_1.output.log({
-        title: `📦 To learn how to publish this library, see https://nx.dev/core-features/manage-releases.`,
-    });
-}
 exports.default = libraryGenerator;

package/src/generators/library/schema.json

@@ -1,5 +1,5 @@
 {
-  "$schema": "https://json-schema.org/schema",
+  "$schema": "http://json-schema.org/schema",
   "$id": "NxTypescriptLibrary",
   "cli": "nx",
   "title": "Create a TypeScript Library",
@@ -48,12 +48,6 @@
       "default": false,
       "x-priority": "internal"
     },
-    "skipPackageJson": {
-      "description": "Do not add dependencies to `package.json`.",
-      "type": "boolean",
-      "default": false,
-      "x-priority": "internal"
-    },
     "skipTsConfig": {
       "type": "boolean",
       "description": "Do not update tsconfig.json for development experience.",
@@ -89,7 +83,7 @@
     "publishable": {
       "type": "boolean",
       "default": false,
-      "description": "Configure the library ready for use with `nx release` (https://nx.dev/core-features/manage-releases).",
+      "description": "Generate a publishable library.",
       "x-priority": "important"
     },
     "importPath": {

package/src/generators/release-version/release-version.d.ts

@@ -1,5 +1,4 @@
 import { Tree } from '@nx/devkit';
-import { ReleaseVersionGeneratorResult } from 'nx/src/command-line/release/version';
 import { ReleaseVersionGeneratorSchema } from './schema';
-export declare function releaseVersionGenerator(tree: Tree, options: ReleaseVersionGeneratorSchema): Promise<ReleaseVersionGeneratorResult>;
+export declare function releaseVersionGenerator(tree: Tree, options: ReleaseVersionGeneratorSchema): Promise<void>;
 export default releaseVersionGenerator;