eslint-plugin-primer-react 2.0.0-rc.022b246 → 2.0.0-rc.9730bcf

package/CHANGELOG.md

@@ -6,10 +6,6 @@
 
 - [#39](https://github.com/primer/eslint-plugin-primer-react/pull/39) [`96a675d`](https://github.com/primer/eslint-plugin-primer-react/commit/96a675d8865450d3be556135947a29181a56551c) Thanks [@colebemis](https://github.com/colebemis)! - Add `direct-slot-children` rule
 
-### Minor Changes
-
-- [#21](https://github.com/primer/eslint-plugin-primer-react/pull/21) [`5f68eb7`](https://github.com/primer/eslint-plugin-primer-react/commit/5f68eb7e222e2a41c527b5036b1308ff293e7a0d) Thanks [@SferaDev](https://github.com/SferaDev)! - `no-system-props`: Add `skipImportCheck` option
-
 ## 1.0.1
 
 ### Patch Changes

package/docs/rules/no-system-props.md

@@ -1,4 +1,4 @@
-# Disallow use of styled-system props (no-system-props)
+# Disallow use of styled-system props (no-system-colors)
 
 🔧 The `--fix` option on the [ESLint CLI](https://eslint.org/docs/user-guide/command-line-interface#fixing-problems) can automatically fix some of the problems reported by this rule.
 
@@ -41,10 +41,6 @@ import {Avatar} from 'some-other-library'
 
 ## Options
 
-- `skipImportCheck` (default: `false`)
-
-  By default, the `no-system-props` rule will only check for styled system props used in functions and components that are imported from `@primer/react`. You can disable this behavior by setting `skipImportCheck` to `true`. This is useful for linting custom components that pass styled system props down to Primer React components.
-
 - `includeUtilityComponents` (default: `false`)
 
   By default, `Box` and `Text` are excluded because styled system props are not deprecated in our utility components. If you prefer to avoid styled system props there as well for consistency, you can set `includeUtilityComponents` to `true`.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-primer-react",
-  "version": "2.0.0-rc.022b246",
+  "version": "2.0.0-rc.9730bcf",
   "description": "ESLint rules for Primer React",
   "main": "src/index.js",
   "scripts": {

package/src/rules/no-system-props.js

@@ -13,16 +13,7 @@ const utilityComponents = new Set(['Box', 'Text'])
 // Components for which we allow a set of prop names
 const excludedComponentProps = new Map([
   ['AnchoredOverlay', new Set(['width', 'height'])],
-  ['Avatar', new Set(['size'])],
-  ['AvatarToken', new Set(['size'])],
-  ['CircleOcticon', new Set(['size'])],
   ['Dialog', new Set(['width', 'height'])],
-  ['IssueLabelToken', new Set(['size'])],
-  ['ProgressBar', new Set(['bg'])],
-  ['Spinner', new Set(['size'])],
-  ['StyledOcticon', new Set(['size'])],
-  ['PointerBox', new Set(['bg'])],
-  ['Token', new Set(['size'])],
   ['PageLayout', new Set(['padding'])],
   ['ProgressBar', new Set(['bg'])],
   ['PointerBox', new Set(['bg'])]
@@ -37,9 +28,6 @@ module.exports = {
     schema: [
       {
         properties: {
-          skipImportCheck: {
-            type: 'boolean'
-          },
           includeUtilityComponents: {
             type: 'boolean'
           }
@@ -51,10 +39,6 @@ module.exports = {
     }
   },
   create(context) {
-    // If `skipImportCheck` is true, this rule will check for deprecated styled system props
-    // used in any components (not just ones that are imported from `@primer/components`).
-    const skipImportCheck = context.options[0] ? context.options[0].skipImportCheck : false
-
     const includeUtilityComponents = context.options[0] ? context.options[0].includeUtilityComponents : false
 
     const excludedComponents = new Set([
@@ -64,7 +48,7 @@ module.exports = {
 
     return {
       JSXOpeningElement(jsxNode) {
-        if (!skipImportCheck && !isPrimerComponent(jsxNode.name, context.getScope(jsxNode))) return
+        if (!isPrimerComponent(jsxNode.name, context.getScope(jsxNode))) return
         if (excludedComponents.has(jsxNode.name.name)) return
 
         // Create an object mapping from prop name to the AST node for that attribute
@@ -80,7 +64,7 @@ module.exports = {
         // Create an array of system prop attribute nodes
         let systemProps = Object.values(pick(propsByNameObject))
 
-        const excludedProps = excludedComponentProps.has(jsxNode.name.name)
+        let excludedProps = excludedComponentProps.has(jsxNode.name.name)
           ? new Set([...alwaysExcludedProps, ...excludedComponentProps.get(jsxNode.name.name)])
           : alwaysExcludedProps
 
@@ -158,12 +142,12 @@ const excludeSxEntriesFromStyleMap = (stylesMap, sxProp) => {
   if (
     !sxProp.value ||
     sxProp.value.type !== 'JSXExpressionContainer' ||
-    sxProp.value.expression.type !== 'ObjectExpression'
+    sxProp.value.expression.type != 'ObjectExpression'
   ) {
     return stylesMap
   }
   return new Map(
-    [...stylesMap].filter(([key]) => {
+    [...stylesMap].filter(([key, _value]) => {
       return !some(sxProp.value.expression.properties, p => p.type === 'Property' && p.key.name === key)
     })
   )