Skip to content

Commit b8c6a51

Browse files
committed
[New] no-rename-default: Forbid importing a default export by a different name
1 parent e1bd0ba commit b8c6a51

34 files changed

+1434
-0
lines changed

CHANGELOG.md

+1
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ This change log adheres to standards from [Keep a CHANGELOG](https://keepachange
88

99
### Added
1010
- [`dynamic-import-chunkname`]: add `allowEmpty` option to allow empty leading comments ([#2942], thanks [@JiangWeixian])
11+
- [`no-rename-default`]: Forbid importing a default export by a different name ([#3006], thanks [@whitneyit])
1112

1213
### Changed
1314
- [Docs] `no-extraneous-dependencies`: Make glob pattern description more explicit ([#2944], thanks [@mulztob])

README.md

+1
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@ This plugin intends to support linting of ES2015+ (ES6+) import/export syntax, a
3737
| [no-mutable-exports](docs/rules/no-mutable-exports.md) | Forbid the use of mutable exports with `var` or `let`. | | | | | | |
3838
| [no-named-as-default](docs/rules/no-named-as-default.md) | Forbid use of exported name as identifier of default export. | | ☑️ 🚸 | | | | |
3939
| [no-named-as-default-member](docs/rules/no-named-as-default-member.md) | Forbid use of exported name as property of default export. | | ☑️ 🚸 | | | | |
40+
| [no-rename-default](docs/rules/no-rename-default.md) | Forbid importing a default export by a different name. | | 🚸 | | | | |
4041
| [no-unused-modules](docs/rules/no-unused-modules.md) | Forbid modules without exports, or exports without matching import in another module. | | | | | | |
4142

4243
### Module systems

config/warnings.js

+1
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ module.exports = {
77
rules: {
88
'import/no-named-as-default': 1,
99
'import/no-named-as-default-member': 1,
10+
'import/no-rename-default': 1,
1011
'import/no-duplicates': 1,
1112
},
1213
};

docs/rules/no-rename-default.md

+31
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
# import/no-rename-default
2+
3+
⚠️ This rule _warns_ in the 🚸 `warnings` config.
4+
5+
<!-- end auto-generated rule header -->
6+
7+
Prohibit importing a default export by another name.
8+
9+
## Rule Details
10+
11+
Given:
12+
13+
```js
14+
// api/get-users.js
15+
export default async function getUsers() {}
16+
```
17+
18+
...this would be valid:
19+
20+
```js
21+
import getUsers from './api/get-users.js';
22+
```
23+
24+
...and the following would be reported:
25+
26+
```js
27+
// Caution: `get-users.js` has a default export `getUsers`.
28+
// This imports `getUsers` as `findUsers`.
29+
// Check if you meant to write `import getUsers from './api/get-users'` instead.
30+
import findUsers from './get-users';
31+
```

src/index.js

+1
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ export const rules = {
2020
'no-named-as-default': require('./rules/no-named-as-default'),
2121
'no-named-as-default-member': require('./rules/no-named-as-default-member'),
2222
'no-anonymous-default-export': require('./rules/no-anonymous-default-export'),
23+
'no-rename-default': require('./rules/no-rename-default'),
2324
'no-unused-modules': require('./rules/no-unused-modules'),
2425

2526
'no-commonjs': require('./rules/no-commonjs'),

src/rules/no-rename-default.js

+269
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,269 @@
1+
/**
2+
* @fileOverview Rule to warn about importing a default export by different name
3+
* @author James Whitney
4+
*/
5+
6+
import docsUrl from '../docsUrl';
7+
import ExportMapBuilder from '../exportMap/builder';
8+
import path from 'path';
9+
10+
//------------------------------------------------------------------------------
11+
// Rule Definition
12+
//------------------------------------------------------------------------------
13+
14+
/** @type {import('@typescript-eslint/utils').TSESLint.RuleModule} */
15+
const rule = {
16+
meta: {
17+
type: 'suggestion',
18+
docs: {
19+
category: 'Helpful warnings',
20+
description: 'Forbid importing a default export by a different name.',
21+
recommended: false,
22+
url: docsUrl('no-named-as-default'),
23+
},
24+
schema: [
25+
{
26+
type: 'object',
27+
properties: {
28+
commonjs: {
29+
default: false,
30+
type: 'boolean',
31+
},
32+
preventRenamingBindings: {
33+
default: true,
34+
type: 'boolean',
35+
},
36+
},
37+
additionalProperties: false,
38+
},
39+
],
40+
},
41+
42+
create(context) {
43+
const {
44+
commonjs = false,
45+
preventRenamingBindings = true,
46+
} = context.options[0] || {};
47+
48+
function findDefaultDestructure(properties) {
49+
const found = properties.find((property) => {
50+
if (property.key.name === 'default') {
51+
return property;
52+
}
53+
});
54+
return found;
55+
}
56+
57+
function getDefaultExportName(targetNode) {
58+
if (targetNode == null) {
59+
return;
60+
}
61+
switch (targetNode.type) {
62+
case 'AssignmentExpression': {
63+
if (!preventRenamingBindings) {
64+
// Allow assignments to be renamed when the `preventRenamingBindings`
65+
// option is set to `false`.
66+
//
67+
// export default Foo = 1;
68+
return;
69+
}
70+
return targetNode.left.name;
71+
}
72+
case 'CallExpression': {
73+
const [argumentNode] = targetNode.arguments;
74+
return getDefaultExportName(argumentNode);
75+
}
76+
case 'ClassDeclaration': {
77+
if (targetNode.id && typeof targetNode.id.name === 'string') {
78+
return targetNode.id.name;
79+
}
80+
// Here we have an anonymous class. We can skip here.
81+
return;
82+
}
83+
case 'ExportSpecifier': {
84+
return targetNode.local.name;
85+
}
86+
case 'FunctionDeclaration': {
87+
return targetNode.id.name;
88+
}
89+
case 'Identifier': {
90+
if (!preventRenamingBindings) {
91+
// Allow identifier to be renamed when the `preventRenamingBindings`
92+
// option is set to `false`.
93+
//
94+
// const foo = 'foo';
95+
// export default foo;
96+
return;
97+
}
98+
return targetNode.name;
99+
}
100+
default:
101+
// This type of node is not handled.
102+
// Returning `undefined` here signifies this and causes the check to
103+
// exit early.
104+
}
105+
}
106+
107+
function getDefaultExportNode(exportMap) {
108+
const defaultExportNode = exportMap.exports.get('default');
109+
if (defaultExportNode == null) {
110+
return;
111+
}
112+
switch (defaultExportNode.type) {
113+
case 'ExportDefaultDeclaration': {
114+
return defaultExportNode.declaration;
115+
}
116+
case 'ExportNamedDeclaration': {
117+
return defaultExportNode.specifiers.find((specifier) => specifier.exported.name === 'default');
118+
}
119+
default:
120+
return;
121+
}
122+
}
123+
124+
function getExportMap(source, context) {
125+
const exportMap = ExportMapBuilder.get(source.value, context);
126+
if (exportMap == null) {
127+
return;
128+
}
129+
if (exportMap.errors.length > 0) {
130+
exportMap.reportErrors(context, source.value);
131+
return;
132+
}
133+
return exportMap;
134+
}
135+
136+
function handleImport(node) {
137+
const exportMap = getExportMap(node.parent.source, context);
138+
if (exportMap == null) {
139+
return;
140+
}
141+
142+
const defaultExportNode = getDefaultExportNode(exportMap);
143+
if (defaultExportNode == null) {
144+
return;
145+
}
146+
147+
const defaultExportName = getDefaultExportName(defaultExportNode);
148+
if (defaultExportName === undefined) {
149+
return;
150+
}
151+
152+
const importTarget = node.parent.source.value;
153+
const importBasename = path.basename(exportMap.path);
154+
155+
if (node.type === 'ImportDefaultSpecifier') {
156+
const importName = node.local.name;
157+
158+
if (importName === defaultExportName) {
159+
return;
160+
}
161+
162+
context.report({
163+
node,
164+
message: `Caution: \`${importBasename}\` has a default export \`${defaultExportName}\`. This imports \`${defaultExportName}\` as \`${importName}\`. Check if you meant to write \`import ${defaultExportName} from '${importTarget}'\` instead.`,
165+
});
166+
167+
return;
168+
}
169+
170+
if (node.type !== 'ImportSpecifier') {
171+
return;
172+
}
173+
174+
if (node.imported.name !== 'default') {
175+
return;
176+
}
177+
178+
const actualImportedName = node.local.name;
179+
180+
if (actualImportedName === defaultExportName) {
181+
return;
182+
}
183+
184+
context.report({
185+
node,
186+
message: `Caution: \`${importBasename}\` has a default export \`${defaultExportName}\`. This imports \`${defaultExportName}\` as \`${actualImportedName}\`. Check if you meant to write \`import { default as ${defaultExportName} } from '${importTarget}'\` instead.`,
187+
});
188+
}
189+
190+
function handleRequire(node) {
191+
if (
192+
!commonjs
193+
|| node.type !== 'VariableDeclarator'
194+
|| !node.id || !(node.id.type === 'Identifier' || node.id.type === 'ObjectPattern')
195+
|| !node.init || node.init.type !== 'CallExpression'
196+
) {
197+
return;
198+
}
199+
200+
let defaultDestructure;
201+
if (node.id.type === 'ObjectPattern') {
202+
defaultDestructure = findDefaultDestructure(node.id.properties);
203+
if (defaultDestructure === undefined) {
204+
return;
205+
}
206+
}
207+
208+
const call = node.init;
209+
const [source] = call.arguments;
210+
211+
if (
212+
call.callee.type !== 'Identifier' || call.callee.name !== 'require' || call.arguments.length !== 1
213+
|| source.type !== 'Literal'
214+
) {
215+
return;
216+
}
217+
218+
const exportMap = getExportMap(source, context);
219+
if (exportMap == null) {
220+
return;
221+
}
222+
223+
const defaultExportNode = getDefaultExportNode(exportMap);
224+
if (defaultExportNode == null) {
225+
return;
226+
}
227+
228+
const defaultExportName = getDefaultExportName(defaultExportNode);
229+
const requireTarget = source.value;
230+
const requireBasename = path.basename(exportMap.path);
231+
const requireName = node.id.type === 'Identifier' ? node.id.name : defaultDestructure.value.name;
232+
233+
if (defaultExportName === undefined) {
234+
return;
235+
}
236+
237+
if (requireName === defaultExportName) {
238+
return;
239+
}
240+
241+
if (node.id.type === 'Identifier') {
242+
context.report({
243+
node,
244+
message: `Caution: \`${requireBasename}\` has a default export \`${defaultExportName}\`. This requires \`${defaultExportName}\` as \`${requireName}\`. Check if you meant to write \`const ${defaultExportName} = require('${requireTarget}')\` instead.`,
245+
});
246+
return;
247+
}
248+
249+
context.report({
250+
node,
251+
message: `Caution: \`${requireBasename}\` has a default export \`${defaultExportName}\`. This requires \`${defaultExportName}\` as \`${requireName}\`. Check if you meant to write \`const { default: ${defaultExportName} } = require('${requireTarget}')\` instead.`,
252+
});
253+
}
254+
255+
return {
256+
ImportDefaultSpecifier(node) {
257+
handleImport(node);
258+
},
259+
ImportSpecifier(node) {
260+
handleImport(node);
261+
},
262+
VariableDeclarator(node) {
263+
handleRequire(node);
264+
},
265+
};
266+
},
267+
};
268+
269+
module.exports = rule;
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export default async () => {};
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export default () => {};
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export default class {};
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export default {};
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export default 123;
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export default arrowAsync = async () => {};
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export default arrow = () => {};
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export default User = class MyUser {}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export default User = class {}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export default fn = function myFn() {};
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export default fn = function () {};
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export default generator = function* myGenerator() {};
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export default generator = function* () {};
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export default class User {};
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
export const barNamed1 = 'bar-named-1';
2+
export const barNamed2 = 'bar-named-2';
3+
4+
const bar = 'bar';
5+
6+
export default bar;
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
export const fooNamed1 = 'foo-named-1';
2+
export const fooNamed2 = 'foo-named-2';
3+
4+
const foo = 'foo';
5+
6+
export default foo;
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export default function getUsersSync() {}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export default async function getUsers() {}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
export default function* reader() {}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
const foo = function bar() {};
2+
3+
export default foo;
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
function bar() {}
2+
3+
export default bar;
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
import foo from '../default-const-foo';
2+
import withLogger from './hoc-with-logger';
3+
4+
export default withLogger(foo);
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
import getUsers from '../default-fn-get-users';
2+
import withLogger from './hoc-with-logger';
3+
4+
export default withLogger(getUsers);

0 commit comments

Comments
 (0)