Skip to content

Commit 0624d82

Browse files
committed
report at point of import + update tests
1 parent 8b2931c commit 0624d82

3 files changed

Lines changed: 20 additions & 1 deletion

File tree

dist/rules/no-internal.js

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -336,6 +336,20 @@ module.exports = {
336336
checkWithParent(resolvedType.symbol.valueDeclaration, node);
337337
}
338338
},
339+
340+
ImportDeclaration(node) {
341+
const tsNode = parserServices.esTreeNodeToTSNodeMap.get(node);
342+
if (!tsNode) return;
343+
344+
const resolvedModule = typeChecker.getSymbolAtLocation(tsNode.moduleSpecifier);
345+
if (resolvedModule && resolvedModule.exports) {
346+
resolvedModule.exports.forEach((exportSymbol) => {
347+
if (exportSymbol.valueDeclaration) {
348+
checkWithParent(exportSymbol.valueDeclaration, node);
349+
}
350+
});
351+
}
352+
},
339353
};
340354
}
341355
}

package.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,5 +62,6 @@
6262
},
6363
"engines": {
6464
"node": "^18.18.0 || ^20.0.0 || ^22.0.0"
65-
}
65+
},
66+
"packageManager": "[email protected]+sha512.0486e394640d3c1fb3c9d43d49cf92879ff74f8516959c235308f5a8f62e2e19528a65cdc2a3058f587cde71eba3d5b56327c8c33a97e4c4051ca48a10ca2d5f"
6667
}

tests/no-internal.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,8 @@ const getInternalInvalidTestCode = (importFrom) => dedent`
5959
`;
6060

6161
const internalInvalidErrorMessages = [
62+
{ message: commonErrorMessages.functionInternal },
63+
{ message: commonErrorMessages.classInternal },
6264
{ message: commonErrorMessages.functionInternal },
6365
{ message: commonErrorMessages.functionInternal },
6466
{ message: commonErrorMessages.functionInternal },
@@ -143,6 +145,7 @@ ruleTester.run(
143145
"checkedPackagePatterns": ["workspace-pkg-2"]
144146
}],
145147
errors: [
148+
{ message: commonErrorMessages.functionInternal },
146149
{ message: commonErrorMessages.functionInternal },
147150
{ message: commonErrorMessages.methodInternal },
148151
{ message: commonErrorMessages.methodInternal }
@@ -157,6 +160,7 @@ ruleTester.run(
157160
`,
158161
options: [{ "dontAllowWorkspaceInternal": true }],
159162
errors: [
163+
{ message: commonErrorMessages.functionInternal },
160164
{ message: commonErrorMessages.functionInternal },
161165
{ message: commonErrorMessages.methodInternal }
162166
]

0 commit comments

Comments
 (0)