Skip to content

fix(no-wildcard-imports): use local name when changing imports #234

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 10 commits into from
Sep 24, 2024
Merged

fix(no-wildcard-imports): use local name when changing imports #234

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
5b26521
fix(no-wildcard-imports): use local name when changing imports
joshblack Sep 19, 2024
c2383db
chore: remove unused sourceCode
joshblack Sep 19, 2024
2d4125c
chore: add changeset
joshblack Sep 19, 2024
0cd574c
chore: fix imports for hooks
joshblack Sep 19, 2024
414c5fa
chore: add fix for when we insert multiple imports
joshblack Sep 19, 2024
e18bdd1
fix: move SelectPanel import to deprecated
joshblack Sep 23, 2024
88feb23
Merge branch 'main' into fix/use-local-name-for-changes
joshblack Sep 23, 2024
e9e9522
chore: add to stable
joshblack Sep 23, 2024
623cb8c
test: update test
joshblack Sep 23, 2024
7ae048b
test: update expected output
joshblack Sep 24, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/shy-news-care.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'eslint-plugin-primer-react': patch
---

Update no-wildcard-imports rule to use local name for updated imports
42 changes: 28 additions & 14 deletions src/rules/__tests__/no-wildcard-imports.test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -71,22 +71,22 @@ 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',
},
},
],
},

// 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: [
Expand All @@ -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',
},
},
],
Expand Down Expand Up @@ -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',
Expand All @@ -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',
Expand All @@ -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',
Expand All @@ -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',
Expand All @@ -277,21 +291,21 @@ 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',
},
},
],
},

// @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: [
{
Expand All @@ -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: [
{
Expand All @@ -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: [
{
Expand Down
20 changes: 11 additions & 9 deletions src/rules/no-wildcard-imports.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,11 @@ const wildcardImports = new Map([
name: 'Dialog',
from: '@primer/react/experimental',
},
{
name: 'DialogHeaderProps',
from: '@primer/react/experimental',
type: 'type',
},
],
],
[
Expand Down Expand Up @@ -132,7 +137,7 @@ const wildcardImports = new Map([

// Hooks
[
'@primer/react/lib-esm/useIsomorphicLayoutEffect',
'@primer/react/lib-esm/utils/useIsomorphicLayoutEffect',
[
{
name: 'default',
Expand All @@ -145,29 +150,26 @@ const wildcardImports = new Map([
'@primer/react/lib-esm/hooks/useResizeObserver',
[
{
name: 'default',
name: 'useResizeObserver',
from: '@primer/react',
as: 'useResizeObserver',
},
],
],
[
'@primer/react/lib-esm/hooks/useProvidedRefOrCreate',
[
{
name: 'default',
name: 'useProvidedRefOrCreate',
from: '@primer/react',
as: 'useProvidedRefOrCreate',
},
],
],
[
'@primer/react/lib-esm/hooks/useResponsiveValue',
[
{
name: 'default',
name: 'useResponsiveValue',
from: '@primer/react',
as: 'useResponsiveValue',
},
],
],
Expand Down Expand Up @@ -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])
}
Expand Down Expand Up @@ -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}'`)
}
}
},
Expand Down