@mui/x-codemod 8.0.0-alpha.9 → 8.0.0-beta.0

package/README.md

@@ -173,6 +173,24 @@ Replace `row` and `column` values by `horizontal` and `vertical` respectively.
  />
 ```
 
+#### `replace-legend-position-values`
+
+Replace `"left" | "middle" | "right"` values `"start" | "center" | "end"` respectively.
+This is to align with the CSS values and reflect the RTL ability of the legend component.
+
+```diff
+ <BarChart
+    slotProps={{
+      legend: {
+        position: {
+-          horizontal: "left",
++          horizontal: "start",
+        }
+      }
+    }}
+ />
+```
+
 #### `rename-responsive-chart-container`
 
 Renames `ResponsiveChartContainer` and `ResponsiveChartContainerPro` by `ChartContainer` and `ChartContainerPro` which have the same behavior in v8.
@@ -188,6 +206,15 @@ Renames `ResponsiveChartContainer` and `ResponsiveChartContainerPro` by `ChartCo
 +</ChartContainer>
 ```
 
+#### `rename-legend-position-type`
+
+Renames `LegendPosition` to `Position`.
+
+```diff
+-import { LegendPosition } from '@mui/x-charts/ChartsLegend';
++import { Position } from '@mui/x-charts/models';
+```
+
 > [!WARNING]
 > If you imported both `ResponsiveChartContainer` and `ChartContainer` in the same file, you might end up with duplicated import.
 > Verify the git diff to remove the duplicate.
@@ -215,6 +242,73 @@ Renames `labelFontSize` and `tickFontSize` props to the corresponding `xxxStyle`
  />
 ```
 
+#### `remove-on-axis-click-handler`
+
+Remove the `<ChartsOnAxisClickHandler />` and move the associated `onAxisClick` prop to its parent.
+
+> [!WARNING]
+> This codemode does not work if component got renamed or if the handler is not a direct child of the container.
+
+```diff
++ <ChartContainer onAxisClick={() => {}}>
+- <ChartContainer>
+-   <ChartsOnAxisClickHandler onAxisClick={() => {}} />
+ </ChartContainer>
+```
+
+#### `rename-unstable-use-series`
+
+Remove `unstable_` prefix from `useSeries` and `use*Series` hooks, as they have now become stable.
+
+```diff
+  import {
+-   unstable_useSeries,
++   useSeries,
+-   unstable_usePieSeries,
++   usePieSeries,
+-   unstable_useLineSeries,
++   useLineSeries,
+-   unstable_useBarSeries,
++   useBarSeries,
+-   unstable_useScatterSeries,
++   useScatterSeries,
+  } from '@mui/x-charts/hooks';
+  import {
+-   unstable_useHeatmapSeries,
++   useHeatmapSeries,
+  } from '@mui/x-charts-pro/hooks';
+```
+
+#### `rename-sparkline-colors-to-color`
+
+Renames the `colors` prop of a `SparkLineChart` to `color` and accesses its first element.
+
+```diff
+ <SparkLineChart
+-  colors={['red']}
++  color={'red'}
+ />
+```
+
+If `colors` is a function, it will be wrapped in another function that returns its first element.
+
+```diff
+ <SparkLineChart
+-  colors={fn}
++  color={typeof fn === 'function' ? mode => fn(mode)[0] : fn[0]}
+ />
+```
+
+If there are cases that the codemod cannot handle, you should see a comment with a `mui-x-codemod` prefix in the code.
+
+```diff
+ <SparkLineChart
+-  colors={(mode) => (mode === 'light' ? ['black'] : ['white'])}
++  /* mui-x-codemod: We renamed the `colors` prop to `color`, but didn't change the value. Please ensure sure this prop receives a string or a function that returns a string. */
++  color={(mode) => (mode === 'light' ? ['black'] : ['white'])}
+ />
+```
+
 ### Data Grid codemods
 
 #### `preset-safe` for Data Grid v8.0.0
@@ -230,6 +324,8 @@ npx @mui/x-codemod@next v8.0.0/data-grid/preset-safe <path|folder>
 The list includes these transformers
 
 - [`remove-stabilized-v8-experimentalFeatures`](#remove-stabilized-v8-experimentalFeatures)
+- [`remove-props`](#remove-props)
+- [`rename-props`](#rename-props)
 
 #### `remove-stabilized-v8-experimentalFeatures`
 
@@ -249,6 +345,49 @@ Remove feature flags for stabilized `experimentalFeatures`.
 npx @mui/x-codemod@next v8.0.0/data-grid/remove-stabilized-experimentalFeatures <path>
 ```
 
+#### `remove-props`
+
+Remove props that are no longer supported.
+
+The list includes these props:
+
+- `indeterminateCheckboxAction`
+- `rowPositionsDebounceMs`
+
+```diff
+ <DataGrid
+-  indeterminateCheckboxAction="deselect"
+-  rowPositionsDebounceMs={100}
+ />
+```
+
+<!-- #default-branch-switch -->
+
+```bash
+npx @mui/x-codemod@next v8.0.0/data-grid/remove-props <path>
+```
+
+#### `rename-props`
+
+Rename props to the new ones.
+
+The list includes these props:
+
+- `unstable_rowSpanning` to `rowSpanning`
+
+```diff
+ <DataGrid
+-  unstable_rowSpanning
++  rowSpanning
+ />
+```
+
+<!-- #default-branch-switch -->
+
+```bash
+npx @mui/x-codemod@next v8.0.0/data-grid/rename-props <path>
+```
+
 ### Pickers codemods
 
 #### `preset-safe` for Pickers v8.0.0
@@ -264,7 +403,7 @@ npx @mui/x-codemod@next v8.0.0/pickers/preset-safe <path|folder>
 The list includes these transformers
 
 - [`rename-adapter-date-fns-imports`](#rename-adapter-date-fns-imports)
-- [`rename-and-move-field-value-type`](#rename-and-move-field-value-type)
+- [`rename-type-imports`](#rename-type-imports)
 
 #### `rename-adapter-date-fns-imports`
 
@@ -296,13 +435,25 @@ The list includes these transformers
 npx @mui/x-codemod@next v8.0.0/pickers/rename-adapter-date-fns-imports <path>
 ```
 
-#### `rename-and-move-field-value-type`
+#### `rename-type-imports`
+
+Renames:
 
-Renames `FieldValueType` to `PickerValueType`.
+- `usePickersTranslations` to `usePickerTranslations`
+- `usePickersContext` to `usePickerContext`
+- `FieldValueType` to `PickerValueType`
+- `RangeFieldSection` to `FieldRangeSection`
+- `PickerShortcutChangeImportance` to `PickerChangeImportance`
 
 ```diff
+-import { usePickersTranslations, usePickersContext } from '@mui/x-date-pickers/hooks';
++import { usePickerTranslations, usePickerContext } from '@mui/x-date-pickers/hooks';
 -import { FieldValueType } from '@mui/x-date-pickers';
 +import { PickerValueType } from '@mui/x-date-pickers';
+-import { RangeFieldSection } from '@mui/x-date-pickers-pro/models';
++import { FieldRangeSection } from '@mui/x-date-pickers-pro/models';
+-import { PickerShortcutChangeImportance } from '@mui/x-date-pickers/PickersShortcuts';
++import { PickerChangeImportance } from '@mui/x-date-pickers/models';
 
  interface MyComponentProps {
 -  valueType: FieldValueType;
@@ -315,7 +466,7 @@ Renames `FieldValueType` to `PickerValueType`.
 <!-- #default-branch-switch -->
 
 ```bash
-npx @mui/x-codemod@next v8.0.0/pickers/rename-and-move-field-value-type <path>
+npx @mui/x-codemod@next v8.0.0/pickers/rename-type-imports <path>
 ```
 
 ## v7.0.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mui/x-codemod",
-  "version": "8.0.0-alpha.9",
+  "version": "8.0.0-beta.0",
   "bin": "./codemod.js",
   "private": false,
   "author": "MUI Team",
@@ -25,17 +25,17 @@
     "url": "https://opencollective.com/mui-org"
   },
   "dependencies": {
-    "@babel/core": "^7.26.0",
-    "@babel/runtime": "^7.26.0",
-    "@babel/traverse": "^7.26.5",
+    "@babel/core": "^7.26.10",
+    "@babel/runtime": "^7.26.10",
+    "@babel/traverse": "^7.26.10",
     "jscodeshift": "17.1.2",
     "yargs": "^17.7.2",
-    "@mui/x-internals": "8.0.0-alpha.9"
+    "@mui/x-internals": "8.0.0-beta.0"
   },
   "devDependencies": {
     "@types/jscodeshift": "^0.12.0",
     "dayjs": "^1.11.13",
-    "moment-timezone": "^0.5.46",
+    "moment-timezone": "^0.5.47",
     "rimraf": "^6.0.1"
   },
   "sideEffects": false,

package/v8.0.0/charts/preset-safe/index.js

@@ -9,11 +9,20 @@ var _renameLegendToSlotsLegend = _interopRequireDefault(require("../rename-legen
 var _renameResponsiveChartContainer = _interopRequireDefault(require("../rename-responsive-chart-container"));
 var _renameLabelAndTickFontSize = _interopRequireDefault(require("../rename-label-and-tick-font-size"));
 var _replaceLegendDirectionValues = _interopRequireDefault(require("../replace-legend-direction-values"));
+var _replaceLegendPositionValues = _interopRequireDefault(require("../replace-legend-position-values"));
+var _removeExperimentalMarkRendering = _interopRequireDefault(require("../remove-experimental-mark-rendering"));
+var _renameLegendPositionType = _interopRequireDefault(require("../rename-legend-position-type"));
+var _removeOnAxisClickHandler = _interopRequireDefault(require("../remove-on-axis-click-handler"));
+var _renameUnstableUseSeries = _interopRequireDefault(require("../rename-unstable-use-series"));
 function transformer(file, api, options) {
   file.source = (0, _renameLegendToSlotsLegend.default)(file, api, options);
   file.source = (0, _renameResponsiveChartContainer.default)(file, api, options);
   file.source = (0, _renameLabelAndTickFontSize.default)(file, api, options);
-  file.source = (0, _renameLabelAndTickFontSize.default)(file, api, options);
   file.source = (0, _replaceLegendDirectionValues.default)(file, api, options);
+  file.source = (0, _replaceLegendPositionValues.default)(file, api, options);
+  file.source = (0, _removeExperimentalMarkRendering.default)(file, api, options);
+  file.source = (0, _renameLegendPositionType.default)(file, api, options);
+  file.source = (0, _removeOnAxisClickHandler.default)(file, api, options);
+  file.source = (0, _renameUnstableUseSeries.default)(file, api, options);
   return file.source;
 }

package/v8.0.0/charts/remove-on-axis-click-handler/index.js

@@ -0,0 +1,74 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.default = transformer;
+/**
+ * @param {import('jscodeshift').FileInfo} file
+ * @param {import('jscodeshift').API} api
+ */
+function transformer(file, api, options) {
+  const j = api.jscodeshift;
+  const printOptions = options.printOptions;
+  const root = j(file.source);
+
+  // Move the handler to the container
+  root.findJSXElements().filter(path => Boolean(path.node.openingElement) && path.node.openingElement.name.type === 'JSXIdentifier' && path.node.openingElement.name.name.includes('ChartContainer')).forEach(path => {
+    // We find the <ChartsOnAxisClickHandler /> node
+    const clickHandler = path.node.children?.find(child => {
+      if (child.type !== 'JSXElement') {
+        return false;
+      }
+      if (child.openingElement.name.type !== 'JSXIdentifier') {
+        return false;
+      }
+      return child.openingElement.name.name === 'ChartsOnAxisClickHandler';
+    });
+    if (!clickHandler) {
+      return;
+    }
+    const clickCallback = clickHandler.openingElement.attributes?.find(attr => attr.type === 'JSXAttribute' && attr.name.name === 'onAxisClick');
+    if (!clickCallback) {
+      return;
+    }
+
+    // Move the callback to the container
+    path.node.openingElement.attributes?.push(clickCallback);
+
+    // Remove the children
+    path.node.children = path.node.children?.filter(child => {
+      if (child.type !== 'JSXElement') {
+        return true;
+      }
+      if (child.openingElement.name.type !== 'JSXIdentifier') {
+        return true;
+      }
+      return child.openingElement.name.name !== 'ChartsOnAxisClickHandler';
+    });
+  });
+
+  // Remove nested import
+  root.find(j.ImportDeclaration, {
+    source: {
+      value: '@mui/x-charts/ChartsOnAxisClickHandler'
+    }
+  }).remove();
+
+  // Remove global import
+  root.find(j.ImportDeclaration).forEach(path => {
+    if (typeof path.node.source.value !== 'string') {
+      return;
+    }
+    if (!path.node.source.value.includes('@mui/x-charts')) {
+      return;
+    }
+    path.node.specifiers = path.node.specifiers?.filter(specifier => {
+      if (specifier.type !== 'ImportSpecifier') {
+        return true;
+      }
+      return specifier.imported.name !== 'ChartsOnAxisClickHandler';
+    });
+  });
+  return root.toSource(printOptions);
+}

package/v8.0.0/{pickers/rename-and-move-field-value-type → charts/rename-legend-position-type}/index.js

@@ -10,16 +10,18 @@ function transformer(file, api, options) {
   const root = j(file.source);
   const printOptions = options.printOptions || {
     quote: 'single',
-    trailingComma: true
+    trailingComma: true,
+    wrapColumn: 40
   };
   (0, _renameImports.renameImports)({
     j,
     root,
-    packageNames: ['@mui/x-date-pickers', '@mui/x-date-pickers-pro'],
+    packageNames: ['@mui/x-charts', '@mui/x-charts-pro'],
     imports: [{
-      oldEndpoint: 'models',
+      oldEndpoint: 'ChartsLegend',
+      newEndpoint: 'models',
       importsMapping: {
-        FieldValueType: 'PickerValueType'
+        LegendPosition: 'Position'
       }
     }]
   });

package/v8.0.0/charts/rename-sparkline-colors-to-color/index.js

@@ -0,0 +1,40 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.default = transformer;
+/**
+ * @param {import('jscodeshift').FileInfo} file
+ * @param {import('jscodeshift').API} api
+ */
+function transformer(file, api, options) {
+  const j = api.jscodeshift;
+  const printOptions = options.printOptions;
+  const root = j(file.source);
+  const componentNames = ['SparkLineChart'];
+  const props = {
+    colors: 'color'
+  };
+  const colorAttributes = root.find(j.JSXElement).filter(path => {
+    return componentNames.includes(path.value.openingElement.name.name);
+  }).find(j.JSXAttribute).filter(attribute => Object.keys(props).includes(attribute.node.name.name));
+  return colorAttributes.forEach(attribute => {
+    const colorsAttributeExpression = attribute.node.value?.type === 'JSXExpressionContainer' ? attribute.node.value.expression : null;
+    let colorAttributeExpression;
+    if (colorsAttributeExpression?.type === 'ArrayExpression') {
+      colorAttributeExpression = colorsAttributeExpression.elements[0];
+    } else if (colorsAttributeExpression?.type === 'Identifier') {
+      colorAttributeExpression = j.conditionalExpression(j.binaryExpression('===', j.unaryExpression('typeof', colorsAttributeExpression), j.literal('function')), j.arrowFunctionExpression([j.identifier('mode')], j.chainExpression(j.optionalMemberExpression(j.callExpression(colorsAttributeExpression, [j.identifier('mode')]), j.literal(0)))), colorsAttributeExpression);
+    } else {
+      // Don't know how to handle this case
+    }
+
+    // Only transform the value if we know how to handle it, otherwise rename the prop and add a comment
+    if (colorAttributeExpression) {
+      j(attribute).replaceWith(j.jsxAttribute(j.jsxIdentifier(props[attribute.node.name.name]), j.jsxExpressionContainer(colorAttributeExpression)));
+    } else {
+      j(attribute).replaceWith(j.jsxAttribute(j.jsxIdentifier(props[attribute.node.name.name]), attribute.node.value)).insertBefore(j.commentBlock(" mui-x-codemod: We renamed the `colors` prop to `color`, but didn't change the value. Please ensure sure this prop receives a string or a function that returns a string. "));
+    }
+  }).toSource(printOptions);
+}

package/v8.0.0/charts/rename-unstable-use-series/index.js

@@ -0,0 +1,33 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.default = transformer;
+var _renameImports = require("../../../util/renameImports");
+function transformer(file, api, options) {
+  const j = api.jscodeshift;
+  const root = j(file.source);
+  const printOptions = options.printOptions || {
+    quote: 'single',
+    trailingComma: true,
+    wrapColumn: 40
+  };
+  (0, _renameImports.renameImports)({
+    j,
+    root,
+    packageNames: ['@mui/x-charts', '@mui/x-charts-pro'],
+    imports: [{
+      oldEndpoint: 'hooks',
+      importsMapping: {
+        unstable_useSeries: 'useSeries',
+        unstable_usePieSeries: 'usePieSeries',
+        unstable_useLineSeries: 'useLineSeries',
+        unstable_useBarSeries: 'useBarSeries',
+        unstable_useScatterSeries: 'useScatterSeries',
+        unstable_useHeatmapSeries: 'useHeatmapSeries'
+      }
+    }]
+  });
+  return root.toSource(printOptions);
+}

package/v8.0.0/charts/replace-legend-position-values/index.js

@@ -0,0 +1,60 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.default = transformer;
+var _addComponentsSlots = require("../../../util/addComponentsSlots");
+/**
+ * @param {import('jscodeshift').FileInfo} file
+ * @param {import('jscodeshift').API} api
+ */
+function transformer(file, api, options) {
+  const j = api.jscodeshift;
+  const printOptions = options.printOptions;
+  const root = j(file.source);
+  root.find(j.ImportDeclaration).filter(({
+    node
+  }) => {
+    return typeof node.source.value === 'string' && node.source.value.startsWith('@mui/x-charts');
+  }).forEach(path => {
+    path.node.specifiers?.forEach(node => {
+      root.findJSXElements(node.local?.name).forEach(elementPath => {
+        if (elementPath.node.type !== 'JSXElement') {
+          return;
+        }
+        const slotProps = elementPath.node.openingElement.attributes?.find(elementNode => elementNode.type === 'JSXAttribute' && elementNode.name.name === 'slotProps');
+        if (slotProps === null) {
+          // No slotProps to manage
+          return;
+        }
+        const position = slotProps?.value?.expression?.properties
+        // @ts-expect-error
+        ?.find(v => v?.key?.name === 'legend')
+        // @ts-expect-error
+        ?.value?.properties?.find(v => v?.key?.name === 'position');
+        const horizontal = position?.value?.properties?.find(v => v?.key?.name === 'horizontal');
+        if (horizontal === undefined || horizontal?.value === undefined || horizontal?.value?.value === undefined) {
+          return;
+        }
+        const horizontalValue = horizontal.value;
+        horizontalValue.value = mapFix(horizontalValue.value);
+        (0, _addComponentsSlots.transformNestedProp)(elementPath, 'slotProps', 'legend.position.horizontal', horizontalValue, j);
+      });
+    });
+  });
+  const transformed = root.findJSXElements();
+  return transformed.toSource(printOptions);
+}
+function mapFix(v) {
+  switch (v) {
+    case 'left':
+      return 'start';
+    case 'right':
+      return 'end';
+    case 'middle':
+      return 'center';
+    default:
+      return v;
+  }
+}

package/v8.0.0/data-grid/preset-safe/index.js

@@ -6,7 +6,11 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = transformer;
 var _removeStabilizedExperimentalFeatures = _interopRequireDefault(require("../remove-stabilized-experimentalFeatures"));
+var _removeProps = _interopRequireDefault(require("../remove-props"));
+var _renameProps = _interopRequireDefault(require("../rename-props"));
 function transformer(file, api, options) {
   file.source = (0, _removeStabilizedExperimentalFeatures.default)(file, api, options);
+  file.source = (0, _renameProps.default)(file, api, options);
+  file.source = (0, _removeProps.default)(file, api, options);
   return file.source;
 }

package/v8.0.0/data-grid/remove-props/index.js

@@ -0,0 +1,24 @@
+"use strict";
+
+var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault").default;
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.default = transformer;
+var _removeProps = _interopRequireDefault(require("../../../util/removeProps"));
+const componentNames = ['DataGrid', 'DataGridPro', 'DataGridPremium'];
+const props = ['indeterminateCheckboxAction', 'rowPositionsDebounceMs'];
+function transformer(file, api, options) {
+  const j = api.jscodeshift;
+  const root = j(file.source);
+  const printOptions = options.printOptions || {
+    quote: 'single',
+    trailingComma: true
+  };
+  return (0, _removeProps.default)({
+    root,
+    j,
+    props,
+    componentNames
+  }).toSource(printOptions);
+}

package/v8.0.0/data-grid/rename-props/index.js

@@ -0,0 +1,26 @@
+"use strict";
+
+var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault").default;
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.default = transformer;
+var _renameProps = _interopRequireDefault(require("../../../util/renameProps"));
+const componentNames = ['DataGrid', 'DataGridPro', 'DataGridPremium'];
+const props = {
+  unstable_rowSpanning: 'rowSpanning'
+};
+function transformer(file, api, options) {
+  const j = api.jscodeshift;
+  const root = j(file.source);
+  const printOptions = options.printOptions || {
+    quote: 'single',
+    trailingComma: true
+  };
+  return (0, _renameProps.default)({
+    root,
+    j,
+    props,
+    componentNames
+  }).toSource(printOptions);
+}

package/v8.0.0/pickers/preset-safe/index.js

@@ -6,9 +6,9 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = transformer;
 var _renameAdapterDateFnsImports = _interopRequireDefault(require("../rename-adapter-date-fns-imports"));
-var _renameAndMoveFieldValueType = _interopRequireDefault(require("../rename-and-move-field-value-type"));
+var _renameTypeImports = _interopRequireDefault(require("../rename-type-imports"));
 function transformer(file, api, options) {
   file.source = (0, _renameAdapterDateFnsImports.default)(file, api, options);
-  file.source = (0, _renameAndMoveFieldValueType.default)(file, api, options);
+  file.source = (0, _renameTypeImports.default)(file, api, options);
   return file.source;
 }

package/v8.0.0/pickers/rename-type-imports/index.js

@@ -0,0 +1,50 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.default = transformer;
+var _renameImports = require("../../../util/renameImports");
+function transformer(file, api, options) {
+  const j = api.jscodeshift;
+  let root = j(file.source);
+  const printOptions = options.printOptions || {
+    quote: 'single',
+    trailingComma: true
+  };
+  root = (0, _renameImports.renameImports)({
+    j,
+    root,
+    packageNames: ['@mui/x-date-pickers', '@mui/x-date-pickers-pro'],
+    imports: [{
+      oldEndpoint: 'hooks',
+      importsMapping: {
+        usePickersTranslations: 'usePickerTranslations',
+        usePickersContext: 'usePickerContext'
+      }
+    }, {
+      oldEndpoint: 'models',
+      importsMapping: {
+        FieldValueType: 'PickerValueType'
+      }
+    }, {
+      oldEndpoint: 'PickersShortcuts',
+      newEndpoint: 'models',
+      importsMapping: {
+        PickerShortcutChangeImportance: 'PickerChangeImportance'
+      }
+    }]
+  });
+  root = (0, _renameImports.renameImports)({
+    j,
+    root,
+    packageNames: ['@mui/x-date-pickers-pro'],
+    imports: [{
+      oldEndpoint: 'models',
+      importsMapping: {
+        RangeFieldSection: 'FieldRangeSection'
+      }
+    }]
+  });
+  return root.toSource(printOptions);
+}