@mui/codemod 6.0.1 → 6.1.0

package/README.md

@@ -1103,19 +1103,6 @@ npx @mui/codemod@latest deprecations/form-control-label-props <path>
 npx @mui/codemod@latest deprecations/list-item-props <path>
 ```
 
-#### `grid-props`
-
-```diff
- <Grid
--  wrap="nowrap"
-+  flexWrap="nowrap"
- />
-```
-
-```bash
-npx @mui/codemod@latest deprecations/grid-props <path>
-```
-
 #### `image-list-item-bar-classes`
 
 JS transforms:

package/node/deprecations/all/deprecations-all.js

@@ -17,7 +17,6 @@ var _circularProgressClasses = _interopRequireDefault(require("../circular-progr
 var _dividerProps = _interopRequireDefault(require("../divider-props"));
 var _filledInputProps = _interopRequireDefault(require("../filled-input-props"));
 var _formControlLabelProps = _interopRequireDefault(require("../form-control-label-props"));
-var _gridProps = _interopRequireDefault(require("../grid-props"));
 var _imageListItemBarClasses = _interopRequireDefault(require("../image-list-item-bar-classes"));
 var _inputBaseProps = _interopRequireDefault(require("../input-base-props"));
 var _inputProps = _interopRequireDefault(require("../input-props"));
@@ -47,7 +46,6 @@ function deprecationsAll(file, api, options) {
   file.source = (0, _dividerProps.default)(file, api, options);
   file.source = (0, _filledInputProps.default)(file, api, options);
   file.source = (0, _formControlLabelProps.default)(file, api, options);
-  file.source = (0, _gridProps.default)(file, api, options);
   file.source = (0, _imageListItemBarClasses.default)(file, api, options);
   file.source = (0, _inputBaseProps.default)(file, api, options);
   file.source = (0, _inputProps.default)(file, api, options);

package/node/v5.0.0/theme-spacing.test/large-actual.js

@@ -328,7 +328,7 @@ function AppSearch() {
       }), /*#__PURE__*/(0, _jsxRuntime.jsxs)(Shortcut, {
         children: [macOS ? '⌘' : 'Ctrl+', "K"]
       })]
-    }), isOpen && /*#__PURE__*/(0, _reactDom.createPortal)( /*#__PURE__*/(0, _jsxRuntime.jsx)(_react2.DocSearchModal, {
+    }), isOpen && /*#__PURE__*/(0, _reactDom.createPortal)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_react2.DocSearchModal, {
       initialQuery: initialQuery,
       appId: 'TZGZ85B9TB',
       apiKey: '8177dfb3e2be72b241ffb8c5abafa899',

package/node/v5.0.0/theme-spacing.test/large-expected.js

@@ -328,7 +328,7 @@ function AppSearch() {
       }), /*#__PURE__*/(0, _jsxRuntime.jsxs)(Shortcut, {
         children: [macOS ? '⌘' : 'Ctrl+', "K"]
       })]
-    }), isOpen && /*#__PURE__*/(0, _reactDom.createPortal)( /*#__PURE__*/(0, _jsxRuntime.jsx)(_react2.DocSearchModal, {
+    }), isOpen && /*#__PURE__*/(0, _reactDom.createPortal)(/*#__PURE__*/(0, _jsxRuntime.jsx)(_react2.DocSearchModal, {
       initialQuery: initialQuery,
       appId: 'TZGZ85B9TB',
       apiKey: '8177dfb3e2be72b241ffb8c5abafa899',

package/node/v6.0.0/list-item-button-prop/list-item-button-prop.js

@@ -41,6 +41,7 @@ function transformer(file, api, options) {
   });
   const openTaggedNotHavingButtonProp = new Set();
   const openTaggedHavingButtonProp = new Set();
+  let addedListItemButton = false;
   // Rename components that have ListItem button to ListItemButton
   (0, _findComponentJSX.default)(j, {
     root,
@@ -50,6 +51,7 @@ function transformer(file, api, options) {
     // The ListItem has a button prop
     if (index !== -1) {
       openTaggedHavingButtonProp.add(elementPath.node.openingElement.name.name);
+      addedListItemButton = true;
       elementPath.node.openingElement.name.name = `ListItemButton`;
       elementPath.node.openingElement.attributes.splice(index, 1);
     } else {
@@ -81,9 +83,10 @@ function transformer(file, api, options) {
     }
     return false;
   }).remove();
-  // If ListItemButton does not already exist, add it at the end
+
+  // If ListItemButton import does not already exist, add it at the end
   const imports = root.find(j.ImportDeclaration).filter(path => path.node.source.value === '@mui/material/ListItemButton');
-  if (imports.length === 0) {
+  if (addedListItemButton && imports.length === 0) {
     const lastImport = root.find(j.ImportDeclaration).at(-1);
 
     // Insert the import for 'ListItemButton' after the last import declaration

package/node/v6.0.0/list-item-button-prop/test-cases/not-related.actual.js

@@ -0,0 +1,4 @@
+"use strict";
+
+var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault").default;
+var _ListItem = _interopRequireDefault(require("@component/ListItem"));

package/node/v6.0.0/list-item-button-prop/test-cases/not-related.expected.js

@@ -0,0 +1,4 @@
+"use strict";
+
+var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault").default;
+var _ListItem = _interopRequireDefault(require("@component/ListItem"));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mui/codemod",
-  "version": "6.0.1",
+  "version": "6.1.0",
   "bin": "./codemod.js",
   "private": false,
   "author": "MUI Team",
@@ -25,16 +25,16 @@
   },
   "dependencies": {
     "@babel/core": "^7.25.2",
-    "@babel/runtime": "^7.25.0",
-    "@babel/traverse": "^7.25.3",
+    "@babel/runtime": "^7.25.6",
+    "@babel/traverse": "^7.25.6",
     "jscodeshift": "^0.16.1",
     "jscodeshift-add-imports": "^1.0.11",
-    "postcss": "^8.4.41",
+    "postcss": "^8.4.45",
     "postcss-cli": "^11.0.0",
     "yargs": "^17.7.2"
   },
   "devDependencies": {
-    "@types/chai": "^4.3.17",
+    "@types/chai": "^4.3.19",
     "@types/jscodeshift": "0.11.11",
     "chai": "^4.5.0"
   },

package/node/deprecations/grid-props/grid-props.js

@@ -1,51 +0,0 @@
-"use strict";
-
-var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault").default;
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.default = transformer;
-var _findComponentJSX = _interopRequireDefault(require("../../util/findComponentJSX"));
-/**
- * @param {import('jscodeshift').FileInfo} file
- * @param {import('jscodeshift').API} api
- */
-function transformer(file, api, options) {
-  const j = api.jscodeshift;
-  const root = j(file.source);
-  const printOptions = options.printOptions;
-  const filterJSXAttr = (attr, name) => attr.type === 'JSXAttribute' && attr.name.name === name;
-  (0, _findComponentJSX.default)(j, {
-    root,
-    componentName: 'Grid'
-  }, gridElement => {
-    const attrs = gridElement.node.openingElement.attributes;
-    const wrapPropIndex = attrs.findIndex(attr => filterJSXAttr(attr, 'wrap'));
-    if (wrapPropIndex !== -1) {
-      if (attrs.findIndex(attr => filterJSXAttr(attr, 'flexWrap')) !== -1) {
-        attrs.splice(wrapPropIndex, 1);
-      } else {
-        attrs[wrapPropIndex].name.name = 'flexWrap';
-      }
-    }
-  });
-  root.find(j.ObjectProperty, {
-    key: {
-      name: 'MuiGrid'
-    }
-  }).forEach(path => {
-    const defaultProps = path.value.value.properties.find(({
-      key
-    }) => key.name === 'defaultProps');
-    const props = defaultProps.value.properties;
-    const wrapPropIndex = props.findIndex(prop => prop.key.name === 'wrap');
-    if (wrapPropIndex !== -1) {
-      if (props.findIndex(prop => prop.key.name === 'flexWrap') !== -1) {
-        props.splice(wrapPropIndex, 1);
-      } else {
-        props[wrapPropIndex].key.name = 'flexWrap';
-      }
-    }
-  });
-  return root.toSource(printOptions);
-}

package/node/deprecations/grid-props/index.js

@@ -1,13 +0,0 @@
-"use strict";
-
-var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault").default;
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-Object.defineProperty(exports, "default", {
-  enumerable: true,
-  get: function () {
-    return _gridProps.default;
-  }
-});
-var _gridProps = _interopRequireDefault(require("./grid-props"));

package/node/deprecations/grid-props/test-cases/actual.js

@@ -1,25 +0,0 @@
-"use strict";
-
-var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault").default;
-var _Grid = _interopRequireDefault(require("@mui/material/Grid"));
-var _material = require("@mui/material");
-var _jsxRuntime = require("react/jsx-runtime");
-/*#__PURE__*/(0, _jsxRuntime.jsx)(_Grid.default, {
-  wrap: "wrap-reverse"
-});
-/*#__PURE__*/(0, _jsxRuntime.jsx)(_material.Grid, {
-  wrap: "wrap-reverse"
-});
-/*#__PURE__*/(0, _jsxRuntime.jsx)(_Grid.default, {
-  wrap: "wrap-reverse",
-  flexWrap: "wrap"
-});
-/*#__PURE__*/(0, _jsxRuntime.jsx)(_material.Grid, {
-  wrap: "wrap-reverse",
-  flexWrap: "wrap"
-});
-
-// should skip non MUI components
-/*#__PURE__*/(0, _jsxRuntime.jsx)(NonMuiGrid, {
-  wrap: "wrap-reverse"
-});

package/node/deprecations/grid-props/test-cases/expected.js

@@ -1,23 +0,0 @@
-"use strict";
-
-var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault").default;
-var _Grid = _interopRequireDefault(require("@mui/material/Grid"));
-var _material = require("@mui/material");
-var _jsxRuntime = require("react/jsx-runtime");
-/*#__PURE__*/(0, _jsxRuntime.jsx)(_Grid.default, {
-  flexWrap: "wrap-reverse"
-});
-/*#__PURE__*/(0, _jsxRuntime.jsx)(_material.Grid, {
-  flexWrap: "wrap-reverse"
-});
-/*#__PURE__*/(0, _jsxRuntime.jsx)(_Grid.default, {
-  flexWrap: "wrap"
-});
-/*#__PURE__*/(0, _jsxRuntime.jsx)(_material.Grid, {
-  flexWrap: "wrap"
-});
-
-// should skip non MUI components
-/*#__PURE__*/(0, _jsxRuntime.jsx)(NonMuiGrid, {
-  wrap: "wrap-reverse"
-});

package/node/deprecations/grid-props/test-cases/theme.actual.js

@@ -1,24 +0,0 @@
-"use strict";
-
-fn({
-  MuiGrid: {
-    defaultProps: {
-      wrap: 'wrap'
-    }
-  }
-});
-fn({
-  MuiGrid: {
-    defaultProps: {
-      wrap: 'wrap-reverse'
-    }
-  }
-});
-fn({
-  MuiGrid: {
-    defaultProps: {
-      wrap: 'wrap-reverse',
-      flexWrap: 'wrap'
-    }
-  }
-});

package/node/deprecations/grid-props/test-cases/theme.expected.js

@@ -1,23 +0,0 @@
-"use strict";
-
-fn({
-  MuiGrid: {
-    defaultProps: {
-      flexWrap: 'wrap'
-    }
-  }
-});
-fn({
-  MuiGrid: {
-    defaultProps: {
-      flexWrap: 'wrap-reverse'
-    }
-  }
-});
-fn({
-  MuiGrid: {
-    defaultProps: {
-      flexWrap: 'wrap'
-    }
-  }
-});