feat(devkit): mark ExecutorContext props required

docs/generated/devkit/ExecutorContext.md

@@ -47,23 +47,19 @@ Enable verbose logging
 
 ### nxJsonConfiguration
 
-• `Optional` **nxJsonConfiguration**: [`NxJsonConfiguration`](../../devkit/documents/NxJsonConfiguration)<`string`[] \| `"*"`\>
+• **nxJsonConfiguration**: [`NxJsonConfiguration`](../../devkit/documents/NxJsonConfiguration)<`string`[] \| `"*"`\>
 
 The contents of nx.json.
 
-@todo(vsavkin): mark this as required for v17
-
 ---
 
 ### projectGraph
 
-• `Optional` **projectGraph**: [`ProjectGraph`](../../devkit/documents/ProjectGraph)
+• **projectGraph**: [`ProjectGraph`](../../devkit/documents/ProjectGraph)
 
 A snapshot of the project graph as
 it existed when the Nx command was kicked off
 
-@todo(vsavkin) mark this required for v17
-
 ---
 
 ### projectName
@@ -76,12 +72,10 @@ The name of the project being executed on
 
 ### projectsConfigurations
 
-• `Optional` **projectsConfigurations**: [`ProjectsConfigurations`](../../devkit/documents/ProjectsConfigurations)
+• **projectsConfigurations**: [`ProjectsConfigurations`](../../devkit/documents/ProjectsConfigurations)
 
 Projects config
 
-@todo(vsavkin): mark this as required for v17
-
 ---
 
 ### root
@@ -119,8 +113,7 @@ it existed when the Nx command was kicked off
 
 ### workspace
 
-• `Optional` **workspace**: [`ProjectsConfigurations`](../../devkit/documents/ProjectsConfigurations) & [`NxJsonConfiguration`](../../devkit/documents/NxJsonConfiguration)<`string`[] \| `"*"`\>
+• **workspace**: [`ProjectsConfigurations`](../../devkit/documents/ProjectsConfigurations) & [`NxJsonConfiguration`](../../devkit/documents/NxJsonConfiguration)<`string`[] \| `"*"`\>
 
 Deprecated. Use projectsConfigurations or nxJsonConfiguration
 The full workspace configuration
-@todo(vsavkin): remove after v17

packages/angular/src/builders/module-federation-dev-server/module-federation-dev-server.impl.ts

@@ -9,11 +9,7 @@ import { scheduleTarget } from 'nx/src/adapter/ngcli-adapter';
 import { executeWebpackDevServerBuilder } from '../webpack-dev-server/webpack-dev-server.impl';
 import { readProjectsConfigurationFromProjectGraph } from 'nx/src/project-graph/project-graph';
 import { getExecutorInformation } from 'nx/src/command-line/run/executor-utils';
-import {
-  getDynamicRemotes,
-  getStaticRemotes,
-  validateDevRemotes,
-} from '../utilities/module-federation';
+import { validateDevRemotes } from '../utilities/module-federation';
 import { existsSync } from 'fs';
 import { extname, join } from 'path';
 import {
@@ -64,6 +60,10 @@ export function executeModuleFederationDevServerBuilder(
           projectsConfigurations:
             readProjectsConfigurationFromProjectGraph(projectGraph),
           nxJsonConfiguration: readNxJson(),
+          workspace: {
+            version: 2,
+            projects: workspaceProjects,
+          },
         }
       )
     )

packages/angular/src/builders/webpack-dev-server/webpack-dev-server.impl.ts

@@ -2,7 +2,6 @@ import {
   joinPathFragments,
   parseTargetString,
   readCachedProjectGraph,
-  readNxJson,
 } from '@nx/devkit';
 import { WebpackNxBuildCoordinationPlugin } from '@nx/webpack/src/plugins/webpack-nx-build-coordination-plugin';
 import { DependentBuildableProjectNode } from '@nx/js/src/utils/buildable-libs-utils';
@@ -19,7 +18,6 @@ import { createTmpTsConfigForBuildableLibs } from '../utilities/buildable-libs';
 import { from } from 'rxjs';
 import { switchMap } from 'rxjs/operators';
 import { getRootTsConfigPath } from '@nx/js';
-import { join } from 'path';
 
 type BuildTargetOptions = {
   tsConfig: string;
@@ -36,12 +34,19 @@ export function executeWebpackDevServerBuilder(
 
   const options = normalizeOptions(rawOptions);
 
+  const projectsConfigurations = {
+    version: 2,
+    projects: {},
+  };
   const parsedBrowserTarget = parseTargetString(options.browserTarget, {
     cwd: context.currentDirectory,
     projectGraph: readCachedProjectGraph(),
     projectName: context.target.project,
     root: context.workspaceRoot,
     isVerbose: false,
+    projectsConfigurations,
+    workspace: projectsConfigurations,
+    nxJsonConfiguration: {},
   });
   const browserTargetProjectConfiguration = readCachedProjectConfiguration(
     parsedBrowserTarget.project

packages/cypress/src/utils/ct-helpers.ts

@@ -124,5 +124,9 @@ export function createExecutorContext(
     projectName,
     projectsConfigurations,
     nxJsonConfiguration,
+    workspace: {
+      version: 2,
+      projects: projectsConfigurations.projects,
+    },
   };
 }

packages/cypress/src/utils/find-target-options.ts

@@ -244,5 +244,9 @@ function createExecutorContext(
     projectName,
     projectsConfigurations,
     nxJsonConfiguration,
+    workspace: {
+      version: 2,
+      projects: projectsConfigurations.projects,
+    },
   };
 }

packages/devkit/src/executors/parse-target-string.spec.ts

@@ -38,6 +38,15 @@ describe('parseTargetString', () => {
       externalNodes: {},
       version: '',
     },
+    workspace: {
+      version: 2,
+      projects: {},
+    },
+    projectsConfigurations: {
+      version: 2,
+      projects: {},
+    },
+    nxJsonConfiguration: {},
   };
 
   it.each(cases)('$input -> $expected', ({ input, expected }) => {

packages/devkit/src/utils/convert-nx-executor.ts

@@ -51,6 +51,10 @@ export function convertNxExecutor(executor: Executor) {
         projectGraph: null,
         taskGraph: null,
         isVerbose: false,
+        workspace: {
+          version: 2,
+          projects: projectsConfigurations.projects,
+        },
       };
       return executor(options, context);
     };

packages/esbuild/src/executors/esbuild/lib/build-esbuild-options.spec.ts

@@ -3,16 +3,17 @@ import { ExecutorContext } from 'nx/src/config/misc-interfaces';
 import path = require('path');
 
 describe('buildEsbuildOptions', () => {
-  const context: ExecutorContext = {
-    projectName: 'myapp',
-    projectsConfigurations: {
-      version: 2,
-      projects: {
-        myapp: {
-          root: 'apps/myapp',
-        },
+  const projectsConfigurations = {
+    version: 2,
+    projects: {
+      myapp: {
+        root: 'apps/myapp',
       },
     },
+  };
+  const context: ExecutorContext = {
+    projectName: 'myapp',
+    projectsConfigurations,
     projectGraph: {
       nodes: {
         myapp: {
@@ -33,6 +34,7 @@ describe('buildEsbuildOptions', () => {
         outputPath: 'dist/apps/myapp',
       },
     },
+    workspace: projectsConfigurations,
   };
 
   it('should include environment variables for platform === browser', () => {

packages/esbuild/src/executors/esbuild/lib/normalize.spec.ts

@@ -19,6 +19,15 @@ describe('normalizeOptions', () => {
       },
       dependencies: {},
     },
+    workspace: {
+      version: 2,
+      projects: {},
+    },
+    projectsConfigurations: {
+      version: 2,
+      projects: {},
+    },
+    nxJsonConfiguration: {},
   };
 
   it('should handle single entry point options', () => {

packages/eslint/src/executors/lint/lint.impl.spec.ts

@@ -104,6 +104,14 @@ describe('Linter Builder', () => {
         },
       },
       nxJsonConfiguration: {},
+      workspace: {
+        version: 2,
+        projects: {},
+      },
+      projectGraph: {
+        nodes: {},
+        dependencies: {},
+      },
       isVerbose: false,
     };
     mockLintFiles.mockImplementation(() => mockReports);

packages/jest/src/executors/jest/jest.impl.spec.ts

@@ -56,6 +56,14 @@ describe('Jest Executor', () => {
       },
       cwd: '/root',
       isVerbose: true,
+      workspace: {
+        version: 2,
+        projects: {},
+      },
+      projectGraph: {
+        nodes: {},
+        dependencies: {},
+      },
     };
   });
 

packages/js/src/executors/tsc/tsc.impl.spec.ts

@@ -16,6 +16,14 @@ describe('tscExecutor', () => {
         projects: {},
       },
       nxJsonConfiguration: {},
+      projectGraph: {
+        nodes: {},
+        dependencies: {},
+      },
+      workspace: {
+        version: 2,
+        projects: {},
+      },
       isVerbose: false,
       projectName: 'example',
       targetName: 'build',

packages/js/src/utils/package-json/update-package-json.spec.ts

@@ -419,6 +419,15 @@ describe('updatePackageJson', () => {
     cwd: '',
     targetName: 'build',
     projectGraph,
+    nxJsonConfiguration: {},
+    projectsConfigurations: {
+      version: 2,
+      projects: {},
+    },
+    workspace: {
+      version: 2,
+      projects: {},
+    },
   };
 
   it('should generate new package if missing', () => {

packages/next/plugins/component-testing.ts

@@ -58,12 +58,19 @@ export function nxComponentTestingPreset(
   let buildFileReplacements = [];
   let buildOuputPath = `dist/${ctProjectName}/.next`;
   if (buildTarget) {
+    const projectsConfigurations = {
+      version: 2,
+      projects: {},
+    };
     const parsedBuildTarget = parseTargetString(buildTarget, {
       cwd: process.cwd(),
       root: workspaceRoot,
       isVerbose: false,
       projectName: ctProjectName,
       projectGraph: graph,
+      projectsConfigurations,
+      workspace: projectsConfigurations,
+      nxJsonConfiguration: {},
     });
     const buildProjectConfig = graph.nodes[parsedBuildTarget.project]?.data;
     const buildExecutorContext = createExecutorContext(

packages/nx/src/config/misc-interfaces.ts

@@ -195,16 +195,13 @@ export interface ExecutorContext {
   /**
    * Projects config
    *
-   * @todo(vsavkin): mark this as required for v17
    */
-  projectsConfigurations?: ProjectsConfigurations;
+  projectsConfigurations: ProjectsConfigurations;
 
   /**
    * The contents of nx.json.
-   *
-   * @todo(vsavkin): mark this as required for v17
    */
-  nxJsonConfiguration?: NxJsonConfiguration;
+  nxJsonConfiguration: NxJsonConfiguration;
 
   /**
    * The current working directory
@@ -219,10 +216,8 @@ export interface ExecutorContext {
   /**
    * A snapshot of the project graph as
    * it existed when the Nx command was kicked off
-   *
-   * @todo(vsavkin) mark this required for v17
    */
-  projectGraph?: ProjectGraph;
+  projectGraph: ProjectGraph;
 
   /**
    * A snapshot of the task graph as
@@ -233,7 +228,6 @@ export interface ExecutorContext {
   /**
    * Deprecated. Use projectsConfigurations or nxJsonConfiguration
    * The full workspace configuration
-   * @todo(vsavkin): remove after v17
    */
-  workspace?: ProjectsConfigurations & NxJsonConfiguration;
+  workspace: ProjectsConfigurations & NxJsonConfiguration;
 }

packages/rollup/src/executors/rollup/rollup.impl.spec.ts

@@ -22,6 +22,14 @@ describe('rollupExecutor', () => {
       isVerbose: false,
       projectName: 'example',
       targetName: 'build',
+      projectGraph: {
+        nodes: {},
+        dependencies: {},
+      },
+      workspace: {
+        version: 2,
+        projects: {},
+      },
     };
     testOptions = {
       compiler: 'babel',