fix(misc): migration for run-commands output-path should handle null … (

#17040)

packages/nx/src/migrations/update-16-2-0/remove-run-commands-output-path.spec.ts

@@ -43,6 +43,24 @@ describe('removeRunCommandsOutputPath', () => {
     expect(migratedTargets.other).toEqual(startingTargets.other);
   });
 
+  it('should handle null options correctly', () => {
+    const tree = createTreeWithEmptyWorkspace();
+    const startingTargets: Record<string, TargetConfiguration> = {
+      build: {
+        executor: 'nx:run-commands',
+        outputs: ['dist/some/path'],
+      },
+    };
+    addProjectConfiguration(tree, 'my-app', {
+      root: 'apps/my-app',
+      targets: startingTargets,
+    });
+    expect(() => removeRunCommandsOutputPath(tree)).not.toThrow();
+    const migratedTargets = readProjectConfiguration(tree, 'my-app').targets;
+    expect(migratedTargets.build).toEqual(startingTargets.build);
+    expect(migratedTargets.other).toEqual(startingTargets.other);
+  });
+
   it('should migrate target defaults correctly', () => {
     const tree = createTreeWithEmptyWorkspace();
     const startingTargetDefaults: Record<string, TargetConfiguration> = {

packages/nx/src/migrations/update-16-2-0/remove-run-commands-output-path.ts

@@ -32,7 +32,7 @@ export default async function removeRunCommandsOutputPath(tree: Tree) {
 
 function updateTargetBlock(target: TargetConfiguration): boolean {
   let changed = false;
-  if (target.executor === 'nx:run-commands' && target.options.outputPath) {
+  if (target.executor === 'nx:run-commands' && target.options?.outputPath) {
     changed = true;
     const outputs = new Set(target.outputs ?? []);
     outputs.delete('{options.outputPath}');