@elastic/eslint-plugin-eui 0.2.0 → 2.0.0

package/README.md

@@ -128,33 +128,29 @@ function MyComponent() {
 
 It's worth pointing out that although the examples provided are specific to EUI components, this rule applies to all JSX elements.
 
-### `@elastic/eui/prefer-css-attributes-for-eui-components`
-
-This rule warns about the use of `style` attribute and encourages to replace it with `css` attribute. Using the `css` attribute ensures better integration with Emotion's styling capabilities.
-
-#### Example
-
-The following code:
+## Testing
 
-```jsx
-<EuiCode style={{ color: '#dd4040' }}>This is a test</EuiCode>
-```
+### Running unit tests
 
-will raise an ESLint report and suggest replacing the `style` attribute with `css`:
+Run unit tests using the following command:
 
-```jsx
-<EuiCode css={{ color: '#dd4040' }}>This is a test</EuiCode>
+```bash
+yarn test
 ```
 
-## Testing
+Unit tests are written using `RuleTester` from `@typescript-eslint/rule-tester`.
 
 ### Against an existing package
 
 To test the local changes to the plugin, you must:
 
-1. Run `yarn pack` in the directory.
-2. In your project's `package.json`, point `@elastic/eslint-plugin-eui` to `file:/path/to/package.tgz`.
-3. Install dependencies: `yarn kbn bootstrap --no-validate`.
+1. Install `yalc` globally if you haven't already: `npm install -g yalc`.
+2. Open a terminal and navigate to this folder: `cd packages/eslint-plugin`
+3. Build the package: `yarn build`
+4. Run `yalc publish` in the plugin's directory to publish it locally.
+5. In your project's directory, run `yalc add @elastic/eslint-plugin-eui` to link the locally published package.
+6. Install dependencies: `yarn` (if you're a Kibana contributor, run `yarn kbn bootstrap --no-validate`).
+7. After making further changes to the plugin, repeat the steps from 3.
 
 ## Publishing
 

package/lib/cjs/index.js

@@ -3,7 +3,6 @@
 var _href_or_on_click = require("./rules/href_or_on_click");
 var _no_restricted_eui_imports = require("./rules/no_restricted_eui_imports");
 var _no_css_color = require("./rules/no_css_color");
-var _prefer_css_attribute_for_eui_components = require("./rules/prefer_css_attribute_for_eui_components");
 /*
  * Licensed to Elasticsearch B.V. under one or more contributor
  * license agreements. See the NOTICE file distributed with
@@ -27,8 +26,7 @@ const config = {
   rules: {
     'href-or-on-click': _href_or_on_click.HrefOnClick,
     'no-restricted-eui-imports': _no_restricted_eui_imports.NoRestrictedEuiImports,
-    'no-css-color': _no_css_color.NoCssColor,
-    'prefer-css-attributes-for-eui-components': _prefer_css_attribute_for_eui_components.PreferCSSAttributeForEuiComponents
+    'no-css-color': _no_css_color.NoCssColor
   },
   configs: {
     recommended: {
@@ -36,8 +34,7 @@ const config = {
       rules: {
         '@elastic/eui/href-or-on-click': 'warn',
         '@elastic/eui/no-restricted-eui-imports': 'warn',
-        '@elastic/eui/no-css-color': 'warn',
-        '@elastic/eui/prefer-css-attributes-for-eui-components': 'warn'
+        '@elastic/eui/no-css-color': 'warn'
       }
     }
   }

package/lib/cjs/utils/resolve_member_expression_root.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"resolve_member_expression_root.d.ts","sourceRoot":"","sources":["../../../src/utils/resolve_member_expression_root.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,sCAAsC,CAAC;AAEhE,eAAO,MAAM,2BAA2B,SAChC,QAAQ,CAAC,gBAAgB,KAC9B,QAAQ,CAAC,UAMX,CAAC"}
+{"version":3,"file":"resolve_member_expression_root.d.ts","sourceRoot":"","sources":["../../../src/utils/resolve_member_expression_root.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,sCAAsC,CAAC;AAEhE,eAAO,MAAM,2BAA2B,GACtC,MAAM,QAAQ,CAAC,gBAAgB,KAC9B,QAAQ,CAAC,UAMX,CAAC"}

package/lib/esm/index.js

@@ -21,13 +21,11 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const href_or_on_click_1 = require("./rules/href_or_on_click");
 const no_restricted_eui_imports_1 = require("./rules/no_restricted_eui_imports");
 const no_css_color_1 = require("./rules/no_css_color");
-const prefer_css_attribute_for_eui_components_1 = require("./rules/prefer_css_attribute_for_eui_components");
 const config = {
     rules: {
         'href-or-on-click': href_or_on_click_1.HrefOnClick,
         'no-restricted-eui-imports': no_restricted_eui_imports_1.NoRestrictedEuiImports,
         'no-css-color': no_css_color_1.NoCssColor,
-        'prefer-css-attributes-for-eui-components': prefer_css_attribute_for_eui_components_1.PreferCSSAttributeForEuiComponents,
     },
     configs: {
         recommended: {
@@ -36,7 +34,6 @@ const config = {
                 '@elastic/eui/href-or-on-click': 'warn',
                 '@elastic/eui/no-restricted-eui-imports': 'warn',
                 '@elastic/eui/no-css-color': 'warn',
-                '@elastic/eui/prefer-css-attributes-for-eui-components': 'warn',
             },
         },
     },

package/lib/esm/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../../src/index.ts"],"names":[],"mappings":";AAAA;;;;;;;;;;;;;;;;;GAiBG;;AAEH,+DAAuD;AACvD,iFAA2E;AAC3E,uDAAkD;AAClD,6GAAqG;AAErG,MAAM,MAAM,GAAG;IACb,KAAK,EAAE;QACL,kBAAkB,EAAE,8BAAW;QAC/B,2BAA2B,EAAE,kDAAsB;QACnD,cAAc,EAAE,yBAAU;QAC1B,0CAA0C,EACxC,4EAAkC;KACrC;IACD,OAAO,EAAE;QACP,WAAW,EAAE;YACX,OAAO,EAAE,CAAC,4BAA4B,CAAC;YACvC,KAAK,EAAE;gBACL,+BAA+B,EAAE,MAAM;gBACvC,wCAAwC,EAAE,MAAM;gBAChD,2BAA2B,EAAE,MAAM;gBACnC,uDAAuD,EAAE,MAAM;aAChE;SACF;KACF;CACF,CAAC;AAEF,MAAM,CAAC,OAAO,GAAG,MAAM,CAAC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../src/index.ts"],"names":[],"mappings":";AAAA;;;;;;;;;;;;;;;;;GAiBG;;AAEH,+DAAuD;AACvD,iFAA2E;AAC3E,uDAAkD;AAElD,MAAM,MAAM,GAAG;IACb,KAAK,EAAE;QACL,kBAAkB,EAAE,8BAAW;QAC/B,2BAA2B,EAAE,kDAAsB;QACnD,cAAc,EAAE,yBAAU;KAC3B;IACD,OAAO,EAAE;QACP,WAAW,EAAE;YACX,OAAO,EAAE,CAAC,4BAA4B,CAAC;YACvC,KAAK,EAAE;gBACL,+BAA+B,EAAE,MAAM;gBACvC,wCAAwC,EAAE,MAAM;gBAChD,2BAA2B,EAAE,MAAM;aACpC;SACF;KACF;CACF,CAAC;AAEF,MAAM,CAAC,OAAO,GAAG,MAAM,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@elastic/eslint-plugin-eui",
-  "version": "0.2.0",
+  "version": "2.0.0",
   "license": "Apache-2.0",
   "repository": {
     "type": "git",
@@ -27,11 +27,11 @@
     "@types/dedent": "^0.7.2",
     "@types/jest": "^29.5.14",
     "@types/micromatch": "^4.0.9",
-    "@typescript-eslint/eslint-plugin": "^8.22.0",
-    "@typescript-eslint/parser": "^8.22.0",
-    "@typescript-eslint/rule-tester": "^8.22.0",
-    "@typescript-eslint/typescript-estree": "^8.22.0",
-    "@typescript-eslint/utils": "^8.22.0",
+    "@typescript-eslint/eslint-plugin": "^8.31.1",
+    "@typescript-eslint/parser": "^8.31.1",
+    "@typescript-eslint/rule-tester": "^8.31.1",
+    "@typescript-eslint/typescript-estree": "^8.31.1",
+    "@typescript-eslint/utils": "^8.31.1",
     "cssstyle": "^4.2.1",
     "dedent": "^1.5.3",
     "eslint": "^8.57.0",
@@ -40,7 +40,7 @@
     "rimraf": "^6.0.1",
     "ts-jest": "^29.2.5",
     "ts-node": "^10.9.2",
-    "typescript": "^5.7.3"
+    "typescript": "^5.8.3"
   },
   "scripts": {
     "test": "jest src",
@@ -48,8 +48,7 @@
     "build:clean": "rimraf lib/",
     "build:compile:esm": "tsc --project ./tsconfig.esm.json",
     "build:compile": "NODE_ENV=production babel src --out-dir=lib/cjs --extensions .js,.ts,.tsx",
-    "build:types": "NODE_ENV=production tsc --project tsconfig.types.json",
-    "build:pack": "yarn build && npm pack"
+    "build:types": "NODE_ENV=production tsc --project tsconfig.types.json"
   },
   "files": [
     "lib",

package/lib/cjs/rules/prefer_css_attribute_for_eui_components.d.ts

@@ -1,3 +0,0 @@
-import { ESLintUtils } from '@typescript-eslint/utils';
-export declare const PreferCSSAttributeForEuiComponents: ESLintUtils.RuleModule<"preferCSSAttributeForEuiComponents", [], unknown, ESLintUtils.RuleListener>;
-//# sourceMappingURL=prefer_css_attribute_for_eui_components.d.ts.map

package/lib/cjs/rules/prefer_css_attribute_for_eui_components.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"prefer_css_attribute_for_eui_components.d.ts","sourceRoot":"","sources":["../../../src/rules/prefer_css_attribute_for_eui_components.ts"],"names":[],"mappings":"AAmBA,OAAO,EAAY,WAAW,EAAE,MAAM,0BAA0B,CAAC;AAEjE,eAAO,MAAM,kCAAkC,qGA0D3C,CAAC"}

package/lib/cjs/rules/prefer_css_attribute_for_eui_components.js

@@ -1,63 +0,0 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.PreferCSSAttributeForEuiComponents = void 0;
-var _utils = require("@typescript-eslint/utils");
-/*
- * Licensed to Elasticsearch B.V. under one or more contributor
- * license agreements. See the NOTICE file distributed with
- * this work for additional information regarding copyright
- * ownership. Elasticsearch B.V. licenses this file to you under
- * the Apache License, Version 2.0 (the "License"); you may
- * not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *    http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied.  See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
-
-const PreferCSSAttributeForEuiComponents = exports.PreferCSSAttributeForEuiComponents = _utils.ESLintUtils.RuleCreator.withoutDocs({
-  create(context) {
-    const isNamedEuiComponentRegex = /^Eui[A-Z]*/;
-    return {
-      JSXOpeningElement(node) {
-        if (node.name.type === 'JSXIdentifier' && isNamedEuiComponentRegex.test(node.name.name)) {
-          let styleAttrNode;
-          if (styleAttrNode = node.attributes.filter(attr => attr.type === 'JSXAttribute').find(attr => attr.name.name === 'style')) {
-            context.report({
-              node: styleAttrNode?.parent,
-              messageId: 'preferCSSAttributeForEuiComponents',
-              fix(fixer) {
-                const cssAttr = node.attributes.find(attr => attr.type === 'JSXAttribute' && attr.name.name === 'css');
-                if (cssAttr) {
-                  return null;
-                }
-                return fixer.replaceTextRange(styleAttrNode?.name?.range, 'css');
-              }
-            });
-          }
-        }
-      }
-    };
-  },
-  meta: {
-    type: 'suggestion',
-    docs: {
-      description: 'Prefer the JSX css attribute for EUI components'
-    },
-    messages: {
-      preferCSSAttributeForEuiComponents: 'Prefer the css attribute for EUI components'
-    },
-    fixable: 'code',
-    schema: []
-  },
-  defaultOptions: []
-});

package/lib/esm/rules/prefer_css_attribute_for_eui_components.d.ts

@@ -1,2 +0,0 @@
-import { ESLintUtils } from '@typescript-eslint/utils';
-export declare const PreferCSSAttributeForEuiComponents: ESLintUtils.RuleModule<"preferCSSAttributeForEuiComponents", [], unknown, ESLintUtils.RuleListener>;

package/lib/esm/rules/prefer_css_attribute_for_eui_components.js

@@ -1,63 +0,0 @@
-"use strict";
-/*
- * Licensed to Elasticsearch B.V. under one or more contributor
- * license agreements. See the NOTICE file distributed with
- * this work for additional information regarding copyright
- * ownership. Elasticsearch B.V. licenses this file to you under
- * the Apache License, Version 2.0 (the "License"); you may
- * not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *    http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied.  See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.PreferCSSAttributeForEuiComponents = void 0;
-const utils_1 = require("@typescript-eslint/utils");
-exports.PreferCSSAttributeForEuiComponents = utils_1.ESLintUtils.RuleCreator.withoutDocs({
-    create(context) {
-        const isNamedEuiComponentRegex = /^Eui[A-Z]*/;
-        return {
-            JSXOpeningElement(node) {
-                if (node.name.type === 'JSXIdentifier' &&
-                    isNamedEuiComponentRegex.test(node.name.name)) {
-                    let styleAttrNode;
-                    if ((styleAttrNode = node.attributes
-                        .filter((attr) => attr.type === 'JSXAttribute')
-                        .find((attr) => attr.name.name === 'style'))) {
-                        context.report({
-                            node: styleAttrNode?.parent,
-                            messageId: 'preferCSSAttributeForEuiComponents',
-                            fix(fixer) {
-                                const cssAttr = node.attributes.find((attr) => attr.type === 'JSXAttribute' && attr.name.name === 'css');
-                                if (cssAttr) {
-                                    return null;
-                                }
-                                return fixer.replaceTextRange(styleAttrNode?.name?.range, 'css');
-                            },
-                        });
-                    }
-                }
-            },
-        };
-    },
-    meta: {
-        type: 'suggestion',
-        docs: {
-            description: 'Prefer the JSX css attribute for EUI components',
-        },
-        messages: {
-            preferCSSAttributeForEuiComponents: 'Prefer the css attribute for EUI components',
-        },
-        fixable: 'code',
-        schema: [],
-    },
-    defaultOptions: [],
-});
-//# sourceMappingURL=prefer_css_attribute_for_eui_components.js.map

package/lib/esm/rules/prefer_css_attribute_for_eui_components.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"prefer_css_attribute_for_eui_components.js","sourceRoot":"","sources":["../../../src/rules/prefer_css_attribute_for_eui_components.ts"],"names":[],"mappings":";AAAA;;;;;;;;;;;;;;;;;GAiBG;;;AAEH,oDAAiE;AAEpD,QAAA,kCAAkC,GAC7C,mBAAW,CAAC,WAAW,CAAC,WAAW,CAAC;IAClC,MAAM,CAAC,OAAO;QACZ,MAAM,wBAAwB,GAAG,YAAY,CAAC;QAE9C,OAAO;YACL,iBAAiB,CAAC,IAAgC;gBAChD,IACE,IAAI,CAAC,IAAI,CAAC,IAAI,KAAK,eAAe;oBAClC,wBAAwB,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,EAC7C,CAAC;oBACD,IAAI,aAAgD,CAAC;oBAErD,IACE,CAAC,aAAa,GAAG,IAAI,CAAC,UAAU;yBAC7B,MAAM,CACL,CAAC,IAAI,EAAiC,EAAE,CACtC,IAAI,CAAC,IAAI,KAAK,cAAc,CAC/B;yBACA,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,KAAK,OAAO,CAAC,CAAC,EAC9C,CAAC;wBACD,OAAO,CAAC,MAAM,CAAC;4BACb,IAAI,EAAE,aAAa,EAAE,MAAM;4BAC3B,SAAS,EAAE,oCAAoC;4BAC/C,GAAG,CAAC,KAAK;gCACP,MAAM,OAAO,GAAG,IAAI,CAAC,UAAU,CAAC,IAAI,CAClC,CAAC,IAAI,EAAE,EAAE,CACP,IAAI,CAAC,IAAI,KAAK,cAAc,IAAI,IAAI,CAAC,IAAI,CAAC,IAAI,KAAK,KAAK,CAC3D,CAAC;gCAEF,IAAI,OAAO,EAAE,CAAC;oCACZ,OAAO,IAAI,CAAC;gCACd,CAAC;gCAED,OAAO,KAAK,CAAC,gBAAgB,CAC3B,aAAa,EAAE,IAAI,EAAE,KAAM,EAC3B,KAAK,CACN,CAAC;4BACJ,CAAC;yBACF,CAAC,CAAC;oBACL,CAAC;gBACH,CAAC;YACH,CAAC;SACF,CAAC;IACJ,CAAC;IACD,IAAI,EAAE;QACJ,IAAI,EAAE,YAAY;QAClB,IAAI,EAAE;YACJ,WAAW,EAAE,iDAAiD;SAC/D;QACD,QAAQ,EAAE;YACR,kCAAkC,EAChC,6CAA6C;SAChD;QACD,OAAO,EAAE,MAAM;QACf,MAAM,EAAE,EAAE;KACX;IACD,cAAc,EAAE,EAAE;CACnB,CAAC,CAAC"}