fix(core): nx v15 prefers output paths that start with {workspaceRoot}

Author: AgentEnder

demo/apps/web-frontend/jest.config.ts

@@ -8,4 +8,5 @@ export default {
   },
   moduleFileExtensions: ['ts', 'tsx', 'js', 'jsx'],
   coverageDirectory: '../../coverage//demo/apps/web-frontend',
+  testEnvironment: 'jsdom',
 };

e2e/core-e2e/tests/nx-dotnet.spec.ts

@@ -376,35 +376,24 @@ describe('nx-dotnet e2e', () => {
       });
     });
 
-    it('should work with workspace.json + project.json', () => {
-      const relativeProjectPath = joinPathFragments('apps', api);
+    it('should work with project.json', () => {
       writeFileSync(
         join(projectFolder, 'project.json'),
         JSON.stringify({
-          root: relativeProjectPath,
+          targets: {},
         }),
       );
-      updateFile('workspace.json', (c) => {
-        const json = JSON.parse(c);
-        json.projects[api] = relativeProjectPath;
-        return JSON.stringify(json, null, 2);
-      });
       expect(() => runNxCommand(`build ${api}`)).not.toThrow();
     });
 
-    it('should work without workspace.json or project.json', () => {
-      const workspaceJsonContents = readFile('workspace.json');
-      unlinkSync(join(e2eDir, 'workspace.json'));
-
+    it('should work without project.json', () => {
       const projectJsonContents = readFile(
         joinPathFragments('apps', api, 'project.json'),
       );
       unlinkSync(join(projectFolder, 'project.json'));
 
       expect(() => runNxCommand(`build ${api}`)).not.toThrow();
 
-      writeFileSync(join(e2eDir, 'workspace.json'), workspaceJsonContents);
-
       writeFileSync(join(projectFolder, 'project.json'), projectJsonContents);
     });
   });

e2e/utils/index.ts

@@ -15,7 +15,7 @@ import {
 import * as path from 'path';
 import { dirSync } from 'tmp';
 
-import isCI = require('is-ci');
+import * as isCI from 'is-ci';
 import { workspaceConfigName } from 'nx/src/config/workspaces';
 import { detectPackageManager } from '@nrwl/devkit';
 interface RunCmdOpts {
@@ -143,11 +143,6 @@ export function runCLI(
       console.log(r);
     }
 
-    const needsMaxWorkers = /g.*(express|nest|node|web|react):app.*/;
-    if (command && needsMaxWorkers.test(command)) {
-      setMaxWorkers();
-    }
-
     return r;
   } catch (e: any) {
     if (opts.silenceError) {
@@ -181,39 +176,6 @@ export function runCommand(command: string): string {
   }
 }
 
-/**
- * Sets maxWorkers in CI on all projects that require it
- * so that it doesn't try to run it with 34 workers
- *
- * maxWorkers required for: node, web, jest
- */
-function setMaxWorkers() {
-  if (isCI) {
-    const workspaceFile = workspaceConfigName(tmpProjPath());
-    const workspace = readJson(workspaceFile);
-
-    Object.keys(workspace.projects).forEach((appName) => {
-      const project = workspace.projects[appName];
-      const { build } = project.targets ?? project.architect;
-
-      if (!build) {
-        return;
-      }
-
-      const executor = build.builder ?? build.executor;
-      if (
-        executor.startsWith('@nrwl/node') ||
-        executor.startsWith('@nrwl/web') ||
-        executor.startsWith('@nrwl/jest')
-      ) {
-        build.options.maxWorkers = 4;
-      }
-    });
-
-    updateFile(workspaceFile, JSON.stringify(workspace));
-  }
-}
-
 export function createFile(f: string, content: string = ''): void {
   const path = tmpProjPath(f);
   createFileSync(path);

jest.config.ts

@@ -1,16 +1,5 @@
+import { getJestProjects } from '@nrwl/jest';
+
 export default {
-  projects: [
-    '<rootDir>/e2e/core-e2e',
-    '<rootDir>/e2e/nx-ghpages-e2e',
-    '<rootDir>/e2e/nxdoc-e2e',
-    '<rootDir>/packages/core',
-    '<rootDir>/packages/dotnet',
-    '<rootDir>/packages/nx-ghpages',
-    '<rootDir>/packages/nxdoc',
-    '<rootDir>/packages/utils',
-    '<rootDir>/smoke/core',
-    '<rootDir>/demo/libs/generated/webapi-types',
-    '<rootDir>/demo/apps/web-frontend',
-    '<rootDir>/demo/apps/web-frontend',
-  ],
+  projects: getJestProjects(),
 };

jest.preset.js

@@ -1,3 +1,3 @@
 const nxPreset = require('@nrwl/jest/preset').default;
 
-module.exports = { ...nxPreset };
+module.exports = { ...nxPreset, testEnvironment: 'node' };

package.json

@@ -44,18 +44,18 @@
     "@commitlint/cli": "^16.2.1",
     "@commitlint/config-conventional": "^17.1.0",
     "@commitlint/types": "^16.2.1",
-    "@nrwl/cypress": "15.0.0-rc1",
-    "@nrwl/devkit": "15.0.0-rc1",
-    "@nrwl/eslint-plugin-nx": "15.0.0-rc1",
-    "@nrwl/jest": "15.0.0-rc1",
-    "@nrwl/js": "15.0.0-rc1",
-    "@nrwl/linter": "15.0.0-rc1",
-    "@nrwl/node": "15.0.0-rc1",
-    "@nrwl/nx-cloud": "14.7.0",
-    "@nrwl/nx-plugin": "15.0.0-rc1",
-    "@nrwl/react": "15.0.0-rc1",
-    "@nrwl/web": "15.0.0-rc1",
-    "@nrwl/workspace": "15.0.0-rc1",
+    "@nrwl/cypress": "15.0.1",
+    "@nrwl/devkit": "15.0.1",
+    "@nrwl/eslint-plugin-nx": "15.0.1",
+    "@nrwl/jest": "15.0.1",
+    "@nrwl/js": "15.0.1",
+    "@nrwl/linter": "15.0.1",
+    "@nrwl/node": "15.0.1",
+    "@nrwl/nx-cloud": "15.0.0",
+    "@nrwl/nx-plugin": "15.0.1",
+    "@nrwl/react": "15.0.1",
+    "@nrwl/web": "15.0.1",
+    "@nrwl/workspace": "15.0.1",
     "@nx-plus/docusaurus": "14.0.0",
     "@semantic-release/changelog": "^6.0.1",
     "@semantic-release/commit-analyzer": "^9.0.2",
@@ -93,11 +93,12 @@
     "fs-extra": "^10.1.0",
     "husky": "^8.0.1",
     "jest": "28.1.3",
+    "jest-environment-jsdom": "28.1.3",
     "jsonc-eslint-parser": "^2.1.0",
     "kill-port": "^1.6.1",
     "lint-staged": "^13.0.3",
     "node-fetch": "2.6.6",
-    "nx": "15.0.0-rc1",
+    "nx": "15.0.1",
     "openapi-types": "^11.0.1",
     "prettier": "2.7.1",
     "react-test-renderer": "18.2.0",

packages/core/src/generators/swagger-typescript/generator.spec.ts

@@ -121,7 +121,7 @@ describe('swagger-typescript generator', () => {
   let tree: Tree;
 
   beforeEach(() => {
-    tree = createTreeWithEmptyWorkspace(2);
+    tree = createTreeWithEmptyWorkspace();
     libraryGenerator(tree, { name: 'generated-ts' });
     tree.write('swagger.json', MOCK_SWAGGER_JSON);
   });

packages/core/src/generators/utils/generate-project.spec.ts

@@ -1,8 +1,6 @@
 import { readProjectConfiguration, Tree, writeJson } from '@nrwl/devkit';
 import { createTreeWithEmptyWorkspace } from '@nrwl/devkit/testing';
 
-import { resolve } from 'path';
-
 import { DotNetClient, mockDotnetFactory } from '@nx-dotnet/dotnet';
 
 import { NxDotnetProjectGeneratorSchema } from '../../models';
@@ -25,7 +23,7 @@ describe('nx-dotnet project generator', () => {
   let options: NxDotnetProjectGeneratorSchema;
 
   beforeEach(() => {
-    appTree = createTreeWithEmptyWorkspace(2);
+    appTree = createTreeWithEmptyWorkspace();
     dotnetClient = new DotNetClient(mockDotnetFactory());
 
     const packageJson = { scripts: {}, devDependencies: {} };
@@ -86,10 +84,7 @@ describe('nx-dotnet project generator', () => {
     const outputPath = config.targets?.build.outputs || [];
     expect(outputPath.length).toBe(1);
 
-    const absoluteDistPath = resolve(appTree.root, outputPath[0]);
-    const expectedDistPath = resolve(appTree.root, './dist/apps/test');
-
-    expect(absoluteDistPath).toEqual(expectedDistPath);
+    expect(outputPath[0]).toEqual('{workspaceRoot}/dist/apps/test');
   });
 
   it('should include serve target for applications', async () => {

packages/core/src/generators/utils/generate-test-project.spec.ts

@@ -7,7 +7,6 @@ import {
 import { createTreeWithEmptyWorkspace } from '@nrwl/devkit/testing';
 
 import * as fs from 'fs';
-import { resolve } from 'path';
 
 import { DotNetClient, mockDotnetFactory } from '@nx-dotnet/dotnet';
 import * as utils from '@nx-dotnet/utils';
@@ -100,15 +99,9 @@ describe('nx-dotnet test project generator', () => {
     const config = readProjectConfiguration(appTree, testProjectName);
     console.log(config.targets?.build.options);
     const outputPath = config.targets?.build.outputs?.[0];
-    expect(outputPath).toBeTruthy();
-
-    const absoluteDistPath = resolve(appTree.root, outputPath || '');
-    const expectedDistPath = resolve(
-      appTree.root,
-      './dist/apps/domain/existing-app-test',
+    expect(outputPath).toEqual(
+      '{workspaceRoot}/dist/apps/domain/existing-app-test',
     );
-
-    expect(absoluteDistPath).toEqual(expectedDistPath);
   });
 
   it('should include lint target', async () => {

packages/core/src/generators/utils/get-path-to-startup-assembly.ts

@@ -3,36 +3,49 @@ import {
   joinPathFragments,
   ProjectConfiguration,
   TargetConfiguration,
+  workspaceRoot,
 } from '@nrwl/devkit';
 
 import { execSync } from 'child_process';
 import { sync } from 'fast-glob';
 import { existsSync } from 'fs';
-import { basename } from 'path';
+import { basename, relative, resolve } from 'path';
 
 export function buildStartupAssemblyPath(
   projectName: string,
   project: ProjectConfiguration,
   csProjFilePath: string,
 ) {
   const [target, configuration] = findBuildTarget(project);
-  const outputDirectory = configuration?.outputs?.[0];
+  let outputDirectory = configuration?.outputs?.[0];
   if (!outputDirectory) {
     throw new Error(`Specify the output directory for ${project.root}
       
       To generate swagger with Nx, the outputs must be captured. This is also necessary for Nx's caching mechanism.
       Simply update the outputs array in project.json with the location of your build artifacts.
       `);
   }
+  outputDirectory = resolve(
+    outputDirectory
+      .replace('{workspaceRoot}', workspaceRoot)
+      .replace('{projectRoot}', project.root),
+  );
   if (!existsSync(outputDirectory)) {
     execSync(`${getPackageManagerCommand().exec} nx ${target} ${projectName}`);
   }
-  console.log('before dll name');
   const dllName = basename(csProjFilePath).replace(
     /(?:\.csproj|\.vbproj|\.fsproj)$/,
     '.dll',
   );
-  console.log('dll name', dllName, outputDirectory);
+  const foundDll = sync(`**/${dllName}`, { cwd: outputDirectory })[0];
+  if (!foundDll) {
+    throw new Error(
+      `[nx-dotnet] Unable to locate ${dllName} in ${relative(
+        workspaceRoot,
+        outputDirectory,
+      )}`,
+    );
+  }
   return joinPathFragments(
     outputDirectory,
     sync(`**/${dllName}`, { cwd: outputDirectory })[0],