@nx/rollup 23.1.0-beta.3 → 23.1.0-beta.5

package/dist/src/generators/configuration/configuration.js

@@ -119,9 +119,7 @@ function updatePackageJson(tree, options, outputConfig, isTsSolutionSetup) {
             // target must exist if we don't receive an outputConfig
             const projectTarget = project.targets[options.buildTarget];
             const nxJson = (0, devkit_1.readNxJson)(tree);
-            const mergedTarget = (0, devkit_internals_1.mergeTargetConfigurations)(projectTarget, (projectTarget.executor
-                ? nxJson.targetDefaults?.[projectTarget.executor]
-                : undefined) ?? nxJson.targetDefaults?.[options.buildTarget]);
+            const mergedTarget = (0, devkit_internals_1.mergeTargetConfigurations)(projectTarget, (0, internal_1.readTargetDefaultsForTarget)(options.buildTarget, nxJson.targetDefaults, projectTarget.executor));
             ({ main, outputPath } = mergedTarget.options);
         }
         packageJson = (0, js_1.getUpdatedPackageJsonContent)(packageJson, {

package/dist/src/generators/convert-to-inferred/convert-to-inferred.js

@@ -20,9 +20,12 @@ async function convertToInferred(tree, options) {
             return;
         // Since targetDefaults for '@nx/rollup:rollup' will no longer apply, we want to copy them to the target options.
         const nxJson = (0, devkit_1.readNxJson)(tree);
-        const defaults = nxJson.targetDefaults['@nx/rollup:rollup'];
+        const defaults = (0, internal_1.findTargetDefault)(nxJson?.targetDefaults, {
+            executor: '@nx/rollup:rollup',
+        });
         if (defaults) {
-            for (const [key, value] of Object.entries(defaults)) {
+            const { target: _t, executor: _e, projects: _p, plugin: _pl, ...rest } = defaults;
+            for (const [key, value] of Object.entries(rest)) {
                 target[key] ??= value;
             }
         }

package/dist/src/migrations/update-23-0-0/remove-use-legacy-typescript-plugin.js

@@ -59,6 +59,11 @@ async function default_1(tree) {
         const ROLLUP_EXECUTORS = ['@nx/rollup:rollup', '@nrwl/rollup:rollup'];
         let nxJsonChanged = false;
         for (const [targetOrExecutor, targetDefault] of Object.entries(nxJson.targetDefaults)) {
+            // This migration predates the filtered array value form, so values are
+            // always plain objects here; skip arrays defensively to stay type-safe.
+            if (Array.isArray(targetDefault)) {
+                continue;
+            }
             if (!ROLLUP_EXECUTORS.includes(targetOrExecutor) &&
                 !ROLLUP_EXECUTORS.includes(targetDefault.executor)) {
                 continue;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/rollup",
-  "version": "23.1.0-beta.3",
+  "version": "23.1.0-beta.5",
   "private": false,
   "type": "commonjs",
   "files": [
@@ -91,8 +91,8 @@
     "postcss": "^8.4.38",
     "postcss-modules": "^6.0.1",
     "tslib": "^2.3.0",
-    "@nx/devkit": "23.1.0-beta.3",
-    "@nx/js": "23.1.0-beta.3"
+    "@nx/devkit": "23.1.0-beta.5",
+    "@nx/js": "23.1.0-beta.5"
   },
   "peerDependencies": {
     "rollup": "^3.0.0 || ^4.0.0"
@@ -104,7 +104,7 @@
   },
   "devDependencies": {
     "source-map-js": "^1.2.0",
-    "nx": "23.1.0-beta.3"
+    "nx": "23.1.0-beta.5"
   },
   "publishConfig": {
     "access": "public"