eslint-plugin-primer-react 6.1.1 → 6.1.2-rc.b701a6b

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-primer-react",
-  "version": "6.1.1",
+  "version": "6.1.2-rc.b701a6b",
   "description": "ESLint rules for Primer React",
   "main": "src/index.js",
   "scripts": {

package/src/rules/__tests__/no-wildcard-imports.test.js

@@ -71,13 +71,13 @@ ruleTester.run('no-wildcard-imports', rule, {
 
     // Test default import
     {
-      code: `import useIsomorphicLayoutEffect from '@primer/react/lib-esm/useIsomorphicLayoutEffect'`,
+      code: `import useIsomorphicLayoutEffect from '@primer/react/lib-esm/utils/useIsomorphicLayoutEffect'`,
       output: `import {useIsomorphicLayoutEffect} from '@primer/react'`,
       errors: [
         {
           messageId: 'wildcardMigration',
           data: {
-            wildcardEntrypoint: '@primer/react/lib-esm/useIsomorphicLayoutEffect',
+            wildcardEntrypoint: '@primer/react/lib-esm/utils/useIsomorphicLayoutEffect',
           },
         },
       ],
@@ -85,8 +85,8 @@ ruleTester.run('no-wildcard-imports', rule, {
 
     // Test multiple wildcard imports into single entrypoint
     {
-      code: `import useResizeObserver from '@primer/react/lib-esm/hooks/useResizeObserver'
-    import useIsomorphicLayoutEffect from '@primer/react/lib-esm/useIsomorphicLayoutEffect'`,
+      code: `import {useResizeObserver} from '@primer/react/lib-esm/hooks/useResizeObserver'
+    import useIsomorphicLayoutEffect from '@primer/react/lib-esm/utils/useIsomorphicLayoutEffect'`,
       output: `import {useResizeObserver} from '@primer/react'
     import {useIsomorphicLayoutEffect} from '@primer/react'`,
       errors: [
@@ -99,7 +99,21 @@ ruleTester.run('no-wildcard-imports', rule, {
         {
           messageId: 'wildcardMigration',
           data: {
-            wildcardEntrypoint: '@primer/react/lib-esm/useIsomorphicLayoutEffect',
+            wildcardEntrypoint: '@primer/react/lib-esm/utils/useIsomorphicLayoutEffect',
+          },
+        },
+      ],
+    },
+
+    // Test renamed wildcard imports
+    {
+      code: `import type {ItemProps} from '@primer/react/lib-esm/deprecated/ActionList/Item'`,
+      output: `import type {ActionListItemProps as ItemProps} from '@primer/react/deprecated'`,
+      errors: [
+        {
+          messageId: 'wildcardMigration',
+          data: {
+            wildcardEntrypoint: '@primer/react/lib-esm/deprecated/ActionList/Item',
           },
         },
       ],
@@ -154,7 +168,7 @@ import type {ButtonBaseProps} from '@primer/react'`,
     },
     {
       code: `import {SelectPanel} from '@primer/react/lib-esm/SelectPanel/SelectPanel'`,
-      output: `import {SelectPanel} from '@primer/react/experimental'`,
+      output: `import {SelectPanel} from '@primer/react'`,
       errors: [
         {
           messageId: 'wildcardMigration',
@@ -166,7 +180,7 @@ import type {ButtonBaseProps} from '@primer/react'`,
     },
     {
       code: `import type {SelectPanelProps} from '@primer/react/lib-esm/SelectPanel/SelectPanel'`,
-      output: `import type {SelectPanelProps} from '@primer/react/experimental'`,
+      output: `import type {SelectPanelProps} from '@primer/react'`,
       errors: [
         {
           messageId: 'wildcardMigration',
@@ -226,7 +240,7 @@ import type {ButtonBaseProps} from '@primer/react'`,
     },
     {
       code: `import type {ItemProps} from '@primer/react/lib-esm/deprecated/ActionList'`,
-      output: `import type {ActionListItemProps} from '@primer/react/deprecated'`,
+      output: `import type {ActionListItemProps as ItemProps} from '@primer/react/deprecated'`,
       errors: [
         {
           messageId: 'wildcardMigration',
@@ -238,7 +252,7 @@ import type {ButtonBaseProps} from '@primer/react'`,
     },
     {
       code: `import type {GroupedListProps} from '@primer/react/lib-esm/deprecated/ActionList/List'`,
-      output: `import type {ActionListGroupedListProps} from '@primer/react/deprecated'`,
+      output: `import type {ActionListGroupedListProps as GroupedListProps} from '@primer/react/deprecated'`,
       errors: [
         {
           messageId: 'wildcardMigration',
@@ -250,7 +264,7 @@ import type {ButtonBaseProps} from '@primer/react'`,
     },
     {
       code: `import {ItemInput} from '@primer/react/lib-esm/deprecated/ActionList/List'`,
-      output: `import {ActionListItemInput} from '@primer/react/deprecated'`,
+      output: `import {ActionListItemInput as ItemInput} from '@primer/react/deprecated'`,
       errors: [
         {
           messageId: 'wildcardMigration',
@@ -262,7 +276,7 @@ import type {ButtonBaseProps} from '@primer/react'`,
     },
     {
       code: `import type {ItemProps} from '@primer/react/lib-esm/deprecated/ActionList/Item'`,
-      output: `import type {ActionListItemProps} from '@primer/react/deprecated'`,
+      output: `import type {ActionListItemProps as ItemProps} from '@primer/react/deprecated'`,
       errors: [
         {
           messageId: 'wildcardMigration',
@@ -277,13 +291,13 @@ import type {ButtonBaseProps} from '@primer/react'`,
 
     // @primer/react/lib-esm/useIsomorphicLayoutEffect
     {
-      code: `import useIsomorphicLayoutEffect from '@primer/react/lib-esm/useIsomorphicLayoutEffect'`,
+      code: `import useIsomorphicLayoutEffect from '@primer/react/lib-esm/utils/useIsomorphicLayoutEffect'`,
       output: `import {useIsomorphicLayoutEffect} from '@primer/react'`,
       errors: [
         {
           messageId: 'wildcardMigration',
           data: {
-            wildcardEntrypoint: '@primer/react/lib-esm/useIsomorphicLayoutEffect',
+            wildcardEntrypoint: '@primer/react/lib-esm/utils/useIsomorphicLayoutEffect',
           },
         },
       ],
@@ -291,7 +305,7 @@ import type {ButtonBaseProps} from '@primer/react'`,
 
     // @primer/react/lib-esm/hooks/useResizeObserver
     {
-      code: `import useResizeObserver from '@primer/react/lib-esm/hooks/useResizeObserver'`,
+      code: `import {useResizeObserver} from '@primer/react/lib-esm/hooks/useResizeObserver'`,
       output: `import {useResizeObserver} from '@primer/react'`,
       errors: [
         {
@@ -305,7 +319,7 @@ import type {ButtonBaseProps} from '@primer/react'`,
 
     // @primer/react/lib-esm/hooks/useProvidedRefOrCreate
     {
-      code: `import useProvidedRefOrCreate from '@primer/react/lib-esm/hooks/useProvidedRefOrCreate'`,
+      code: `import {useProvidedRefOrCreate} from '@primer/react/lib-esm/hooks/useProvidedRefOrCreate'`,
       output: `import {useProvidedRefOrCreate} from '@primer/react'`,
       errors: [
         {
@@ -319,7 +333,7 @@ import type {ButtonBaseProps} from '@primer/react'`,
 
     // @primer/react/lib-esm/hooks/useResponsiveValue
     {
-      code: `import useResponsiveValue from '@primer/react/lib-esm/hooks/useResponsiveValue'`,
+      code: `import {useResponsiveValue} from '@primer/react/lib-esm/hooks/useResponsiveValue'`,
       output: `import {useResponsiveValue} from '@primer/react'`,
       errors: [
         {

package/src/rules/no-wildcard-imports.js

@@ -35,6 +35,11 @@ const wildcardImports = new Map([
         name: 'Dialog',
         from: '@primer/react/experimental',
       },
+      {
+        name: 'DialogHeaderProps',
+        from: '@primer/react/experimental',
+        type: 'type',
+      },
     ],
   ],
   [
@@ -42,12 +47,12 @@ const wildcardImports = new Map([
     [
       {
         name: 'SelectPanel',
-        from: '@primer/react/experimental',
+        from: '@primer/react',
       },
       {
         type: 'type',
         name: 'SelectPanelProps',
-        from: '@primer/react/experimental',
+        from: '@primer/react',
       },
     ],
   ],
@@ -132,7 +137,7 @@ const wildcardImports = new Map([
 
   // Hooks
   [
-    '@primer/react/lib-esm/useIsomorphicLayoutEffect',
+    '@primer/react/lib-esm/utils/useIsomorphicLayoutEffect',
     [
       {
         name: 'default',
@@ -145,9 +150,8 @@ const wildcardImports = new Map([
     '@primer/react/lib-esm/hooks/useResizeObserver',
     [
       {
-        name: 'default',
+        name: 'useResizeObserver',
         from: '@primer/react',
-        as: 'useResizeObserver',
       },
     ],
   ],
@@ -155,9 +159,8 @@ const wildcardImports = new Map([
     '@primer/react/lib-esm/hooks/useProvidedRefOrCreate',
     [
       {
-        name: 'default',
+        name: 'useProvidedRefOrCreate',
         from: '@primer/react',
-        as: 'useProvidedRefOrCreate',
       },
     ],
   ],
@@ -165,9 +168,8 @@ const wildcardImports = new Map([
     '@primer/react/lib-esm/hooks/useResponsiveValue',
     [
       {
-        name: 'default',
+        name: 'useResponsiveValue',
         from: '@primer/react',
-        as: 'useResponsiveValue',
       },
     ],
   ],
@@ -282,7 +284,7 @@ module.exports = {
           }
 
           if (migration.as) {
-            changes.get(migration.from).push([migration.as, migration.as, migration.type])
+            changes.get(migration.from).push([migration.as, specifier.local.name, migration.type])
           } else {
             changes.get(migration.from).push([migration.name, specifier.local.name, migration.type])
           }
@@ -357,7 +359,7 @@ module.exports = {
                   }
                   return imported
                 })
-                yield fixer.insertTextAfter(node, `import type {${specifiers.join(', ')}} from '${entrypoint}'`)
+                yield fixer.insertTextAfter(node, `\nimport type {${specifiers.join(', ')}} from '${entrypoint}'`)
               }
             }
           },

package/src/rules/use-deprecated-from-deprecated.js

@@ -7,22 +7,50 @@ const components = [
     identifier: 'Dialog',
     entrypoint: '@primer/react',
   },
+  {
+    identifier: 'DialogProps',
+    entrypoint: '@primer/react',
+  },
+  {
+    identifier: 'DialogHeaderProps',
+    entrypoint: '@primer/react',
+  },
   {
     identifier: 'Octicon',
     entrypoint: '@primer/react',
   },
+  {
+    identifier: 'OcticonProps',
+    entrypoint: '@primer/react',
+  },
   {
     identifier: 'Pagehead',
     entrypoint: '@primer/react',
   },
+  {
+    identifier: 'PageheadProps',
+    entrypoint: '@primer/react',
+  },
   {
     identifier: 'TabNav',
     entrypoint: '@primer/react',
   },
+  {
+    identifier: 'TabNavProps',
+    entrypoint: '@primer/react',
+  },
+  {
+    identifier: 'TabNavLinkProps',
+    entrypoint: '@primer/react',
+  },
   {
     identifier: 'Tooltip',
     entrypoint: '@primer/react',
   },
+  {
+    identifier: 'TooltipProps',
+    entrypoint: '@primer/react',
+  },
 ]
 
 const entrypoints = new Map()