Skip to content

Commit 8ef9075

Browse files
refactor: add warning
1 parent 868546e commit 8ef9075

2 files changed

Lines changed: 16 additions & 0 deletions

File tree

lib/NodeStuffPlugin.js

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -190,6 +190,16 @@ class NodeStuffPlugin {
190190
) {
191191
hooks.propertyInDestructuring.tap(PLUGIN_NAME, (usingProperty) => {
192192
if (property === usingProperty.id) {
193+
if (warning) {
194+
parser.state.module.addWarning(
195+
new NodeStuffInWebError(
196+
usingProperty.loc,
197+
expressionName,
198+
warning
199+
)
200+
);
201+
}
202+
193203
return `${property}: ${JSON.stringify(fn(parser.state.module))},`;
194204
}
195205
});

test/configCases/node/filename-and-dirname/warnings.js

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,18 @@ module.exports = [
55
/"__filename" has been used/,
66
/"import.meta.dirname" has been used/,
77
/"import.meta.filename" has been used/,
8+
/"import.meta.dirname" has been used/,
9+
/"import.meta.filename" has been used/,
810
/"__dirname" has been used/,
911
/"__filename" has been used/,
1012
/"import.meta.dirname" has been used/,
1113
/"import.meta.filename" has been used/,
14+
/"import.meta.dirname" has been used/,
15+
/"import.meta.filename" has been used/,
1216
/"__dirname" has been used/,
1317
/"__filename" has been used/,
1418
/"import.meta.dirname" has been used/,
19+
/"import.meta.filename" has been used/,
20+
/"import.meta.dirname" has been used/,
1521
/"import.meta.filename" has been used/
1622
];

0 commit comments

Comments
 (0)