@nx/js 17.0.0-beta.8 → 17.0.0-rc.1

package/babel.d.ts

@@ -4,6 +4,9 @@ export interface NxWebBabelPresetOptions {
         decoratorsBeforeExport?: boolean;
         legacy?: boolean;
     };
+    loose?: boolean;
+    /** @deprecated Use `loose` option instead of `classProperties.loose`
+     */
     classProperties?: {
         loose?: boolean;
     };

package/babel.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 const path_1 = require("path");
+const devkit_1 = require("@nx/devkit");
 module.exports = function (api, options = {}) {
     api.assertVersion(7);
     const isModern = api.caller((caller) => caller?.isModern);
@@ -9,9 +10,13 @@ module.exports = function (api, options = {}) {
     // This is set by `@nx/rollup:rollup` executor
     const isNxPackage = api.caller((caller) => caller?.isNxPackage);
     const emitDecoratorMetadata = api.caller((caller) => caller?.emitDecoratorMetadata ?? true);
-    // Determine settings  for `@babel/plugin-proposal-class-properties`,
+    // Determine settings  for `@babel//babel-plugin-transform-class-properties`,
     // so that we can sync the `loose` option with `@babel/preset-env`.
-    const classProperties = options.classProperties ?? { loose: true };
+    // TODO(v18): Remove classProperties since it's no longer needed, now that the class props transform is in preset-env.
+    const loose = options.classProperties?.loose ?? options.loose ?? true;
+    if (options.classProperties) {
+        devkit_1.logger.warn(`Use =\`loose\` option instead of \`classProperties.loose\`. The \`classProperties\` option will be removed in Nx 18`);
+    }
     return {
         presets: [
             // Support module/nomodule pattern.
@@ -33,7 +38,7 @@ module.exports = function (api, options = {}) {
                         // Exclude transforms that make all code slower
                         exclude: ['transform-typeof-symbol'],
                         // This must match the setting for `@babel/plugin-proposal-class-properties`
-                        loose: classProperties.loose,
+                        loose,
                     },
             ],
             [
@@ -65,10 +70,6 @@ module.exports = function (api, options = {}) {
                 require.resolve('@babel/plugin-proposal-decorators'),
                 options.decorators ?? { legacy: true },
             ],
-            [
-                require.resolve('@babel/plugin-proposal-class-properties'),
-                classProperties,
-            ],
         ].filter(Boolean),
         overrides: [
             // Convert `const enum` to `enum`. The former cannot be supported by babel

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/js",
-  "version": "17.0.0-beta.8",
+  "version": "17.0.0-rc.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": {
@@ -31,7 +31,6 @@
   "executors": "./executors.json",
   "dependencies": {
     "@babel/core": "^7.22.9",
-    "@babel/plugin-proposal-class-properties": "^7.18.6",
     "@babel/plugin-proposal-decorators": "^7.22.7",
     "@babel/plugin-transform-runtime": "^7.22.9",
     "@babel/preset-env": "^7.22.9",
@@ -57,9 +56,9 @@
     "semver": "7.5.3",
     "source-map-support": "0.5.19",
     "tslib": "^2.3.0",
-    "@nx/devkit": "17.0.0-beta.8",
-    "@nx/workspace": "17.0.0-beta.8",
-    "@nrwl/js": "17.0.0-beta.8"
+    "@nx/devkit": "17.0.0-rc.1",
+    "@nx/workspace": "17.0.0-rc.1",
+    "@nrwl/js": "17.0.0-rc.1"
   },
   "peerDependencies": {
     "verdaccio": "^5.0.4"

package/src/utils/package-json/get-npm-scope.js

@@ -6,11 +6,6 @@ const devkit_1 = require("@nx/devkit");
  * Read the npm scope that a workspace should use by default
  */
 function getNpmScope(tree) {
-    const nxJson = (0, devkit_1.readNxJson)(tree);
-    // TODO(v17): Remove reading this from nx.json
-    if (nxJson?.npmScope) {
-        return nxJson.npmScope;
-    }
     const { name } = tree.exists('package.json')
         ? (0, devkit_1.readJson)(tree, 'package.json')
         : { name: null };