[private-classnames] Remove package and move everything to utils (mui#32758)

Author: mnajdova

.codesandbox/ci.json

@@ -28,7 +28,6 @@
     "@mui/styled-engine-sc": "packages/mui-styled-engine-sc/build",
     "@mui/system": "packages/mui-system/build",
     "@mui/private-theming": "packages/mui-private-theming/build",
-    "@mui/private-classnames": "packages/mui-private-classnames/build",
     "@mui/types": "packages/mui-types/build",
     "@mui/utils": "packages/mui-utils/build",
     "@mui/base": "packages/mui-base/build",

babel.config.js

@@ -18,7 +18,6 @@ const defaultAlias = {
   '@mui/styles': resolveAliasPath('./packages/mui-styles/src'),
   '@mui/system': resolveAliasPath('./packages/mui-system/src'),
   '@mui/private-theming': resolveAliasPath('./packages/mui-private-theming/src'),
-  '@mui/private-classnames': resolveAliasPath('./packages/mui-private-classnames/src'),
   '@mui/base': resolveAliasPath('./packages/mui-base/src'),
   '@mui/utils': resolveAliasPath('./packages/mui-utils/src'),
   '@mui/material-next': resolveAliasPath('./packages/mui-material-next/src'),

docs/babel.config.js

@@ -25,7 +25,6 @@ const alias = {
   // '@mui/styled-engine': '../packages/mui-styled-engine-sc/src',
   '@mui/system': '../packages/mui-system/src',
   '@mui/private-theming': '../packages/mui-private-theming/src',
-  '@mui/private-classnames': '../packages/mui-private-classnames/src',
   '@mui/utils': '../packages/mui-utils/src',
   '@mui/base': '../packages/mui-base/src',
   '@mui/material-next': '../packages/mui-material-next/src',

docs/next.config.js

@@ -137,7 +137,6 @@ module.exports = {
                         '@mui/styles': '../packages/mui-styles/src',
                         '@mui/system': '../packages/mui-system/src',
                         '@mui/private-theming': '../packages/mui-private-theming/src',
-                        '@mui/private-classnames': '../packages/mui-private-classnames/src',
                         '@mui/utils': '../packages/mui-utils/src',
                         '@mui/base': '../packages/mui-base/src',
                         '@mui/material-next': '../packages/mui-material-next/src',

docs/src/modules/utils/helpers.ts

@@ -169,7 +169,6 @@ export function getDependencies(
     '@mui/styles': getMuiPackageVersion('styles', muiCommitRef),
     '@mui/system': getMuiPackageVersion('system', muiCommitRef),
     '@mui/private-theming': getMuiPackageVersion('theming', muiCommitRef),
-    '@mui/private-classnames': getMuiPackageVersion('classnames', muiCommitRef),
     '@mui/base': getMuiPackageVersion('base', muiCommitRef),
     '@mui/utils': getMuiPackageVersion('utils', muiCommitRef),
     '@mui/material-next': getMuiPackageVersion('material-next', muiCommitRef),

packages/mui-base/package.json

@@ -53,7 +53,6 @@
   "dependencies": {
     "@babel/runtime": "^7.17.2",
     "@emotion/is-prop-valid": "^1.1.2",
-    "@mui/private-classnames": "^5.7.0",
     "@mui/types": "^7.1.3",
     "@mui/utils": "^5.7.0",
     "@popperjs/core": "^2.11.5",

packages/mui-base/src/className/index.ts

@@ -1,2 +1,2 @@
 // eslint-disable-next-line import/prefer-default-export
-export { unstable_ClassNameGenerator } from '@mui/private-classnames';
+export { unstable_ClassNameGenerator } from '@mui/utils';

packages/mui-base/src/composeClasses/index.ts

@@ -1 +1 @@
-export { unstable_composeClasses as default } from '@mui/private-classnames';
+export { unstable_composeClasses as default } from '@mui/utils';

packages/mui-base/src/generateUtilityClass/index.ts

@@ -1,2 +1,2 @@
-export { generateUtilityClass as default } from '@mui/private-classnames';
-export type { GlobalStateSlot } from '@mui/private-classnames';
+export { unstable_generateUtilityClass as default } from '@mui/utils';
+export type { GlobalStateSlot } from '@mui/utils';

packages/mui-base/src/generateUtilityClasses/index.ts

@@ -1 +1 @@
-export { generateUtilityClasses as default } from '@mui/private-classnames';
+export { unstable_generateUtilityClasses as default } from '@mui/utils';

packages/mui-base/tsconfig.build.json

@@ -11,6 +11,5 @@
     "rootDir": "./src"
   },
   "include": ["src/**/*.ts*"],
-  "exclude": ["src/**/*.spec.ts*", "src/**/*.test.ts*"],
-  "references": [{ "path": "../mui-private-classnames/tsconfig.build.json" }]
+  "exclude": ["src/**/*.spec.ts*", "src/**/*.test.ts*"]
 }

packages/mui-material/package.json

@@ -62,7 +62,6 @@
   "dependencies": {
     "@babel/runtime": "^7.17.2",
     "@mui/base": "5.0.0-alpha.80",
-    "@mui/private-classnames": "^5.7.0",
     "@mui/system": "^5.7.0",
     "@mui/types": "^7.1.3",
     "@mui/utils": "^5.7.0",

packages/mui-material/scripts/rollup.config.js

@@ -102,10 +102,6 @@ const nestedFolder = {
       return resolveNestedImport('mui-private-theming', importee);
     }
 
-    if (importee.indexOf('@mui/private-classnames/') === 0) {
-      return resolveNestedImport('mui-private-classnames', importee);
-    }
-
     if (importee.indexOf('@mui/styled-engine/') === 0) {
       return resolveNestedImport('mui-styled-engine', importee);
     }

packages/mui-material/src/className/index.ts

@@ -1,2 +1,2 @@
 // eslint-disable-next-line import/prefer-default-export
-export { unstable_ClassNameGenerator } from '@mui/private-classnames';
+export { unstable_ClassNameGenerator } from '@mui/utils';

packages/mui-private-classnames/README.md

@@ -1,4 +1,7 @@
-import { generateUtilityClass, generateUtilityClasses } from '@mui/private-classnames';
+import {
+  unstable_generateUtilityClass as generateUtilityClass,
+  unstable_generateUtilityClasses as generateUtilityClasses,
+} from '@mui/utils';
 
 export interface ContainerClasses {
   /** Styles applied to the root element. */

packages/mui-private-classnames/package.json

@@ -2,12 +2,12 @@ import * as React from 'react';
 import PropTypes from 'prop-types';
 import clsx from 'clsx';
 import { Interpolation, StyledComponent } from '@mui/styled-engine';
-import { unstable_capitalize as capitalize } from '@mui/utils';
 import { OverridableComponent } from '@mui/types';
 import {
+  unstable_capitalize as capitalize,
   unstable_composeClasses as composeClasses,
-  generateUtilityClass,
-} from '@mui/private-classnames';
+  unstable_generateUtilityClass as generateUtilityClass,
+} from '@mui/utils';
 import { ContainerProps, ContainerTypeMap } from './ContainerProps';
 import useThemePropsSystem from '../useThemeProps';
 import systemStyled from '../styled';

packages/mui-private-classnames/src/index.d.ts

@@ -11,6 +11,5 @@
     "rootDir": "./src"
   },
   "include": ["src/**/*.ts*"],
-  "exclude": ["src/**/*.spec.ts", "src/**/*.test.ts"],
-  "references": [{ "path": "../mui-private-classnames/tsconfig.build.json" }]
+  "exclude": ["src/**/*.spec.ts", "src/**/*.test.ts"]
 }

packages/mui-private-classnames/src/index.js

@@ -26,14 +26,14 @@
     "url": "https://opencollective.com/mui"
   },
   "scripts": {
-    "build": "yarn build:legacy && yarn build:modern && yarn build:node && yarn build:stable && yarn build:copy-files && yarn build:types",
+    "build": "yarn build:legacy && yarn build:modern && yarn build:node && yarn build:stable && yarn build:types && yarn build:copy-files",
     "build:legacy": "node ../../scripts/build legacy",
     "build:modern": "node ../../scripts/build modern",
     "build:node": "node ../../scripts/build node",
     "build:stable": "node ../../scripts/build stable",
     "build:copy-files": "node ../../scripts/copy-files.js",
     "build:types": "node ../../scripts/buildTypes",
-    "prebuild": "rimraf build",
+    "prebuild": "rimraf build tsconfig.build.tsbuildinfo",
     "release": "yarn build && npm publish build",
     "test": "cd ../../ && cross-env NODE_ENV=test mocha 'packages/mui-utils/**/*.test.{js,ts,tsx}'",
     "typescript": "tslint -p tsconfig.json \"{src,test}/**/*.{spec,d}.{ts,tsx}\" && tsc -p tsconfig.json"

packages/mui-private-classnames/tsconfig.build.json

@@ -1,5 +1,5 @@
 import { expect } from 'chai';
-import { unstable_composeClasses as composeClasses } from '@mui/private-classnames';
+import { unstable_composeClasses as composeClasses } from '@mui/utils';
 
 describe('composeClasses', () => {
   it('should generate the classes based on the slots', () => {

packages/mui-private-classnames/tsconfig.json

@@ -1,8 +1,8 @@
 import { expect } from 'chai';
 import {
-  generateUtilityClass,
+  unstable_generateUtilityClass as generateUtilityClass,
   unstable_ClassNameGenerator as ClassNameGenerator,
-} from '@mui/private-classnames';
+} from '@mui/utils';
 
 describe('generateUtilityClass', () => {
   it('should generate the class correctly', () => {
@@ -27,14 +27,14 @@ describe('generateUtilityClass', () => {
     });
 
     it('able to set custom generator', () => {
-      const generator = (name) => `foo-bar-${name}`;
+      const generator = (name: string) => `foo-bar-${name}`;
       ClassNameGenerator.configure(generator);
 
       expect(generateUtilityClass('MuiTest', 'slot')).to.equal('foo-bar-MuiTest-slot');
     });
 
     it('does not affect state class', () => {
-      const generator = (name) => `foo-bar-${name}`;
+      const generator = (name: string) => `foo-bar-${name}`;
       ClassNameGenerator.configure(generator);
 
       expect(generateUtilityClass('MuiTest', 'active')).to.equal('Mui-active');

packages/mui-system/src/Container/containerClasses.ts

@@ -1,5 +1,5 @@
 import { expect } from 'chai';
-import { generateUtilityClasses } from '@mui/private-classnames';
+import { unstable_generateUtilityClasses as generateUtilityClasses } from '@mui/utils';
 
 describe('generateUtilityClasses', () => {
   it('should generate the classes correctly', () => {

packages/mui-system/src/Container/createContainer.tsx

@@ -34,3 +34,8 @@ export { default as usePreviousProps } from './usePreviousProps';
 export { default as visuallyHidden } from './visuallyHidden';
 export { default as integerPropType } from './integerPropType';
 export { default as internal_resolveProps } from './resolveProps';
+export { default as unstable_composeClasses } from './composeClasses';
+export { default as unstable_generateUtilityClass } from './generateUtilityClass';
+export * from './generateUtilityClass';
+export { default as unstable_generateUtilityClasses } from './generateUtilityClasses';
+export { default as unstable_ClassNameGenerator } from './ClassNameGenerator';

packages/mui-system/tsconfig.build.json

@@ -216,10 +216,6 @@ function createWebpackConfig(entry, environment) {
         '@mui/styles': path.join(workspaceRoot, 'packages/mui-styles/build'),
         '@mui/system': path.join(workspaceRoot, 'packages/mui-system/build'),
         '@mui/private-theming': path.join(workspaceRoot, 'packages/mui-private-theming/build'),
-        '@mui/private-classnames': path.join(
-          workspaceRoot,
-          'packages/mui-private-classnames/build',
-        ),
         '@mui/utils': path.join(workspaceRoot, 'packages/mui-utils/build'),
         '@mui/base': path.join(workspaceRoot, 'packages/mui-base/build'),
         '@mui/material-next': path.join(workspaceRoot, 'packages/mui-material-next/build'),

packages/mui-utils/package.json

@@ -156,7 +156,6 @@ module.exports = function setKarmaConfig(config) {
                         '@mui/styles': './packages/mui-styles/src',
                         '@mui/system': './packages/mui-system/src',
                         '@mui/private-theming': './packages/mui-private-theming/src',
-                        '@mui/private-classnames': './packages/mui-private-classnames/src',
                         '@mui/utils': './packages/mui-utils/src',
                         '@mui/base': './packages/mui-base/src',
                         '@mui/material-next': './packages/mui-material-next/src',

packages/mui-private-classnames/src/ClassNameGenerator/ClassNameGenerator.ts renamed to packages/mui-utils/src/ClassNameGenerator/ClassNameGenerator.ts

@@ -28,8 +28,6 @@
       "@mui/system": ["./packages/mui-system/src"],
       "@mui/private-theming": ["./packages/mui-private-theming/src"],
       "@mui/private-theming/*": ["./packages/mui-private-theming/src/*"],
-      "@mui/private-classnames": ["./packages/mui-private-classnames/src"],
-      "@mui/private-classnames/*": ["./packages/mui-private-classnames/src/*"],
       "@mui/types": ["./packages/mui-types"],
       "@mui/base": ["./packages/mui-base/src"],
       "@mui/base/*": ["./packages/mui-base/src/*"],

packages/mui-private-classnames/src/ClassNameGenerator/index.ts renamed to packages/mui-utils/src/ClassNameGenerator/index.ts

@@ -18,7 +18,6 @@ module.exports = {
       '@mui/styles': path.resolve(__dirname, './packages/mui-styles/src'),
       '@mui/system': path.resolve(__dirname, './packages/mui-system/src'),
       '@mui/private-theming': path.resolve(__dirname, './packages/mui-private-theming/src'),
-      '@mui/private-classnames': path.resolve(__dirname, './packages/mui-private-classnames/src'),
       '@mui/base': path.resolve(__dirname, './packages/mui-base/src'),
       '@mui/utils': path.resolve(__dirname, './packages/mui-utils/src'),
       '@mui/material-next': path.resolve(__dirname, './packages/mui-material-next/src'),