@nx/js 16.6.0-beta.0 → 16.6.0-beta.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/js",
-  "version": "16.6.0-beta.0",
+  "version": "16.6.0-beta.1",
   "private": false,
   "description": "The JS plugin for Nx contains executors and generators that provide the best experience for developing JavaScript and TypeScript projects. ",
   "repository": {
@@ -39,9 +39,9 @@
     "@babel/preset-env": "^7.15.0",
     "@babel/preset-typescript": "^7.15.0",
     "@babel/runtime": "^7.14.8",
-    "@nrwl/js": "16.6.0-beta.0",
-    "@nx/devkit": "16.6.0-beta.0",
-    "@nx/workspace": "16.6.0-beta.0",
+    "@nrwl/js": "16.6.0-beta.1",
+    "@nx/devkit": "16.6.0-beta.1",
+    "@nx/workspace": "16.6.0-beta.1",
     "@phenomnomnominal/tsquery": "~5.0.1",
     "babel-plugin-const-enum": "^1.0.1",
     "babel-plugin-macros": "^2.8.0",
@@ -69,5 +69,5 @@
     "access": "public"
   },
   "types": "./src/index.d.ts",
-  "gitHead": "5a7ffb93c9e84bffbf52f6ae9c6c1e3b8edb9d35"
+  "gitHead": "cf9f5cbe6a7ce3b916ec6505dc8488f669cbc52b"
 }

package/src/executors/swc/schema.json

@@ -69,13 +69,15 @@
     "updateBuildableProjectDepsInPackageJson": {
       "type": "boolean",
       "description": "Whether to update the buildable project dependencies in the build output package.json.",
-      "default": false
+      "default": false,
+      "x-deprecated": "Configure the project to use the '@nx/dependency-checks' ESLint rule instead (https://nx.dev/packages/eslint-plugin/documents/dependency-checks). It will be removed in v17."
     },
     "buildableProjectDepsInPackageJsonType": {
       "type": "string",
       "description": "When `updateBuildableProjectDepsInPackageJson` is `true`, this adds dependencies to either `peerDependencies` or `dependencies`.",
       "enum": ["dependencies", "peerDependencies"],
-      "default": "peerDependencies"
+      "default": "peerDependencies",
+      "x-deprecated": "Configure the project to use the '@nx/dependency-checks' ESLint rule instead (https://nx.dev/packages/eslint-plugin/documents/dependency-checks). It will be removed in v17."
     },
     "external": {
       "description": "A list projects to be treated as external. This feature is experimental",

package/src/executors/tsc/schema.json

@@ -57,13 +57,15 @@
     "updateBuildableProjectDepsInPackageJson": {
       "type": "boolean",
       "description": "Whether to update the buildable project dependencies in the build output package.json.",
-      "default": false
+      "default": false,
+      "x-deprecated": "Configure the project to use the '@nx/dependency-checks' ESLint rule instead (https://nx.dev/packages/eslint-plugin/documents/dependency-checks). It will be removed in v17."
     },
     "buildableProjectDepsInPackageJsonType": {
       "type": "string",
       "description": "When `updateBuildableProjectDepsInPackageJson` is `true`, this adds dependencies to either `peerDependencies` or `dependencies`.",
       "enum": ["dependencies", "peerDependencies"],
-      "default": "peerDependencies"
+      "default": "peerDependencies",
+      "x-deprecated": "Configure the project to use the '@nx/dependency-checks' ESLint rule instead (https://nx.dev/packages/eslint-plugin/documents/dependency-checks). It will be removed in v17."
     },
     "external": {
       "description": "A list projects to be treated as external. This feature is experimental",

package/src/executors/tsc/tsc.batch-impl.js

@@ -43,20 +43,21 @@ function tscBatchExecutor(taskGraph, inputs, overrides, context) {
                 }
             },
         };
+        const processTaskPostCompilation = (tsConfig) => {
+            if (tsConfigTaskInfoMap[tsConfig]) {
+                const taskInfo = tsConfigTaskInfoMap[tsConfig];
+                taskInfo.assetsHandler.processAllAssetsOnceSync();
+                (0, update_package_json_1.updatePackageJson)(taskInfo.options, taskInfo.context, taskInfo.projectGraphNode, taskInfo.buildableProjectNodeDependencies);
+                taskInfo.endTime = Date.now();
+            }
+        };
         const typescriptCompilation = (0, lib_1.compileTypescriptSolution)(tsCompilationContext, shouldWatch, logger, {
             beforeProjectCompilationCallback: (tsConfig) => {
                 if (tsConfigTaskInfoMap[tsConfig]) {
                     tsConfigTaskInfoMap[tsConfig].startTime = Date.now();
                 }
             },
-            afterProjectCompilationCallback: (tsConfig) => {
-                if (tsConfigTaskInfoMap[tsConfig]) {
-                    const taskInfo = tsConfigTaskInfoMap[tsConfig];
-                    taskInfo.assetsHandler.processAllAssetsOnceSync();
-                    (0, update_package_json_1.updatePackageJson)(taskInfo.options, taskInfo.context, taskInfo.projectGraphNode, taskInfo.buildableProjectNodeDependencies);
-                    taskInfo.endTime = Date.now();
-                }
-            },
+            afterProjectCompilationCallback: processTaskPostCompilation,
         });
         if (shouldWatch) {
             const taskInfos = Object.values(tsConfigTaskInfoMap);
@@ -79,21 +80,54 @@ function tscBatchExecutor(taskGraph, inputs, overrides, context) {
                 return { value: undefined, done: true };
             }))))));
         }
+        const toBatchExecutorTaskResult = (tsConfig, success) => ({
+            task: tsConfigTaskInfoMap[tsConfig].task,
+            result: {
+                success: success,
+                terminalOutput: tsConfigTaskInfoMap[tsConfig].terminalOutput,
+                startTime: tsConfigTaskInfoMap[tsConfig].startTime,
+                endTime: tsConfigTaskInfoMap[tsConfig].endTime,
+            },
+        });
+        let isCompilationDone = false;
+        const taskTsConfigsToReport = new Set(Object.keys(taskGraph.tasks).map((t) => taskInMemoryTsConfigMap[t].path));
+        let tasksToReportIterator;
+        const processSkippedTasks = () => {
+            const { value: tsConfig, done } = tasksToReportIterator.next();
+            if (done) {
+                return { value: undefined, done: true };
+            }
+            tsConfigTaskInfoMap[tsConfig].startTime = Date.now();
+            processTaskPostCompilation(tsConfig);
+            return { value: toBatchExecutorTaskResult(tsConfig, true), done: false };
+        };
         return yield tslib_1.__await(yield tslib_1.__await(yield* tslib_1.__asyncDelegator(tslib_1.__asyncValues(mapAsyncIterable(typescriptCompilation, (iterator) => tslib_1.__awaiter(this, void 0, void 0, function* () {
+            if (isCompilationDone) {
+                return processSkippedTasks();
+            }
             const { value, done } = yield iterator.next();
             if (done) {
-                return { value, done: true };
+                if (taskTsConfigsToReport.size > 0) {
+                    /**
+                     * TS compilation is done but we still have tasks to report. This can
+                     * happen if, for example, a project is identified as affected, but
+                     * no file in the TS project is actually changed or if running a
+                     * task with `--skip-nx-cache` and the outputs are already there. There
+                     * can still be changes to assets or other files we need to process.
+                     *
+                     * Switch to handle the iterator for the tasks we still need to report.
+                     */
+                    isCompilationDone = true;
+                    tasksToReportIterator = taskTsConfigsToReport.values();
+                    return processSkippedTasks();
+                }
+                return { value: undefined, done: true };
             }
-            const taskResult = {
-                task: tsConfigTaskInfoMap[value.tsConfig].task,
-                result: {
-                    success: value.success,
-                    terminalOutput: tsConfigTaskInfoMap[value.tsConfig].terminalOutput,
-                    startTime: tsConfigTaskInfoMap[value.tsConfig].startTime,
-                    endTime: tsConfigTaskInfoMap[value.tsConfig].endTime,
-                },
+            taskTsConfigsToReport.delete(value.tsConfig);
+            return {
+                value: toBatchExecutorTaskResult(value.tsConfig, value.success),
+                done: false,
             };
-            return { value: taskResult, done: false };
         }))))));
     });
 }

package/src/generators/library/library.js

@@ -238,8 +238,8 @@ function createFiles(tree, options, filesDir) {
 }
 function addJest(tree, options) {
     return tslib_1.__awaiter(this, void 0, void 0, function* () {
-        const { jestProjectGenerator } = (0, devkit_1.ensurePackage)('@nx/jest', versions_1.nxVersion);
-        return yield jestProjectGenerator(tree, Object.assign(Object.assign({}, options), { project: options.name, setupFile: 'none', supportTsx: false, skipSerializers: true, testEnvironment: options.testEnvironment, skipFormat: true, compiler: options.bundler === 'swc' || options.bundler === 'tsc'
+        const { configurationGenerator } = (0, devkit_1.ensurePackage)('@nx/jest', versions_1.nxVersion);
+        return yield configurationGenerator(tree, Object.assign(Object.assign({}, options), { project: options.name, setupFile: 'none', supportTsx: false, skipSerializers: true, testEnvironment: options.testEnvironment, skipFormat: true, compiler: options.bundler === 'swc' || options.bundler === 'tsc'
                 ? options.bundler
                 : options.bundler === 'rollup'
                     ? 'swc'

package/src/migrations/update-16-6-0/explicitly-set-projects-to-update-buildable-deps.js

@@ -9,26 +9,29 @@ const executors = new Set([
     '@nrwl/js:tsc',
 ]);
 function default_1(tree) {
+    var _a;
+    var _b;
     return tslib_1.__awaiter(this, void 0, void 0, function* () {
-        const projects = (0, devkit_1.getProjects)(tree);
-        for (const [projectName, project] of projects) {
-            if (project.projectType !== 'library') {
+        // use project graph to get the expanded target configurations
+        const projectGraph = yield (0, devkit_1.createProjectGraphAsync)();
+        for (const [projectName, { data: projectData }] of Object.entries(projectGraph.nodes)) {
+            if (projectData.projectType !== 'library') {
                 continue;
             }
-            let updated = false;
-            for (const [, target] of Object.entries(project.targets || {})) {
+            for (const [targetName, target] of Object.entries(projectData.targets || {})) {
                 if (!executors.has(target.executor)) {
                     continue;
                 }
-                if (target.options &&
+                if (!target.options ||
                     target.options.updateBuildableProjectDepsInPackageJson === undefined) {
-                    target.options.updateBuildableProjectDepsInPackageJson = true;
-                    updated = true;
+                    // read the project configuration to write the explicit project configuration
+                    // and avoid writing the expanded target configuration
+                    const project = (0, devkit_1.readProjectConfiguration)(tree, projectName);
+                    (_a = (_b = project.targets[targetName]).options) !== null && _a !== void 0 ? _a : (_b.options = {});
+                    project.targets[targetName].options.updateBuildableProjectDepsInPackageJson = true;
+                    (0, devkit_1.updateProjectConfiguration)(tree, projectName, project);
                 }
             }
-            if (updated) {
-                (0, devkit_1.updateProjectConfiguration)(tree, projectName, project);
-            }
         }
         yield (0, devkit_1.formatFiles)(tree);
     });

package/src/utils/schema.d.ts

@@ -42,7 +42,17 @@ export interface ExecutorOptions {
   watch: boolean;
   clean?: boolean;
   transformers: TransformerEntry[];
+  /**
+   * @deprecated Configure the project to use the `@nx/dependency-checks` ESLint
+   * rule instead (https://nx.dev/packages/eslint-plugin/documents/dependency-checks).
+   * It will be removed in v17.
+   */
   updateBuildableProjectDepsInPackageJson?: boolean;
+  /**
+   * @deprecated Configure the project to use the `@nx/dependency-checks` ESLint
+   * rule instead (https://nx.dev/packages/eslint-plugin/documents/dependency-checks).
+   * It will be removed in v17.
+   */
   buildableProjectDepsInPackageJsonType?: 'dependencies' | 'peerDependencies';
   external?: 'all' | 'none' | string[];
   externalBuildTargets?: string[];