eslint-plugin-sonarjs 4.0.3 → 4.1.0

package/cjs/S4324/generated-meta.js

@@ -31,7 +31,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
+exports.skipOnGeneratedSource = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
 __exportStar(require("./meta.js"), exports);
 exports.meta = {
     type: 'suggestion',
@@ -49,3 +49,4 @@ exports.sonarKey = 'S4324';
 exports.scope = 'Main';
 exports.languages = ['ts'];
 exports.requiredDependency = [];
+exports.skipOnGeneratedSource = false;

package/cjs/S4328/generated-meta.js

@@ -31,7 +31,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
+exports.skipOnGeneratedSource = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
 __exportStar(require("./meta.js"), exports);
 exports.meta = {
     type: 'suggestion',
@@ -53,3 +53,4 @@ exports.sonarKey = 'S4328';
 exports.scope = 'Main';
 exports.languages = ['ts'];
 exports.requiredDependency = [];
+exports.skipOnGeneratedSource = false;

package/cjs/S4328/rule.js

@@ -58,7 +58,8 @@ const builtin_modules_1 = __importDefault(require("builtin-modules"));
 const typescript_1 = __importDefault(require("typescript"));
 const generate_meta_js_1 = require("../helpers/generate-meta.js");
 const meta = __importStar(require("./generated-meta.js"));
-const dependencies_js_1 = require("../helpers/package-jsons/dependencies.js");
+const dependencies_js_1 = require("../helpers/dependency-manifests/dependencies.js");
+const deno_js_1 = require("../helpers/dependency-manifests/resolvers/deno.js");
 const messages = {
     removeOrAddDependency: 'Either remove this import or add it as a dependency.',
 };
@@ -109,6 +110,10 @@ function raiseOnImplicitImport(module, loc, dependencies, filename, host, option
     if (['node:', 'data:', 'file:'].some(prefix => moduleName.startsWith(prefix))) {
         return;
     }
+    // Deno inline npm imports (e.g. 'npm:zod@4.3.6') are self-declaring — no manifest entry needed.
+    if ((0, deno_js_1.parseInlineNPMImport)(moduleName) !== undefined) {
+        return;
+    }
     const packageName = getPackageName(moduleName);
     if (whitelist.includes(packageName)) {
         return;
@@ -116,7 +121,7 @@ function raiseOnImplicitImport(module, loc, dependencies, filename, host, option
     if (builtin_modules_1.default.includes(packageName)) {
         return;
     }
-    for (const dependency of dependencies) {
+    for (const dependency of dependencies.keys()) {
         if (typeof dependency === 'string') {
             if (dependency === packageName) {
                 return;

package/cjs/S4335/generated-meta.js

@@ -31,7 +31,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
+exports.skipOnGeneratedSource = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
 __exportStar(require("./meta.js"), exports);
 exports.meta = {
     type: 'problem',
@@ -49,3 +49,4 @@ exports.sonarKey = 'S4335';
 exports.scope = 'Main';
 exports.languages = ['ts'];
 exports.requiredDependency = [];
+exports.skipOnGeneratedSource = false;

package/cjs/S4423/generated-meta.js

@@ -31,7 +31,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
+exports.skipOnGeneratedSource = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
 __exportStar(require("./meta.js"), exports);
 exports.meta = {
     type: 'problem',
@@ -49,3 +49,4 @@ exports.sonarKey = 'S4423';
 exports.scope = 'Main';
 exports.languages = ['js', 'ts'];
 exports.requiredDependency = ['aws-cdk-lib'];
+exports.skipOnGeneratedSource = false;

package/cjs/S4426/generated-meta.js

@@ -31,7 +31,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
+exports.skipOnGeneratedSource = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
 __exportStar(require("./meta.js"), exports);
 exports.meta = {
     type: 'problem',
@@ -49,3 +49,4 @@ exports.sonarKey = 'S4426';
 exports.scope = 'Main';
 exports.languages = ['js', 'ts'];
 exports.requiredDependency = [];
+exports.skipOnGeneratedSource = false;

package/cjs/S4502/generated-meta.js

@@ -31,12 +31,12 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
+exports.skipOnGeneratedSource = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
 __exportStar(require("./meta.js"), exports);
 exports.meta = {
     type: 'problem',
     docs: {
-        description: 'Disabling CSRF protections is security-sensitive',
+        description: 'CSRF protections should not be disabled',
         recommended: true,
         url: 'https://sonarsource.github.io/rspec/#/rspec/S4502/javascript',
         requiresTypeChecking: false,
@@ -49,3 +49,4 @@ exports.sonarKey = 'S4502';
 exports.scope = 'Main';
 exports.languages = ['js', 'ts'];
 exports.requiredDependency = [];
+exports.skipOnGeneratedSource = false;

package/cjs/S4507/generated-meta.js

@@ -31,12 +31,12 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
+exports.skipOnGeneratedSource = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
 __exportStar(require("./meta.js"), exports);
 exports.meta = {
     type: 'problem',
     docs: {
-        description: 'Delivering code in production with debug features activated is security-sensitive',
+        description: 'Debugging features should not be enabled in production',
         recommended: true,
         url: 'https://sonarsource.github.io/rspec/#/rspec/S4507/javascript',
         requiresTypeChecking: false,
@@ -49,3 +49,4 @@ exports.sonarKey = 'S4507';
 exports.scope = 'Main';
 exports.languages = ['js', 'ts'];
 exports.requiredDependency = [];
+exports.skipOnGeneratedSource = false;

package/cjs/S4524/generated-meta.js

@@ -31,7 +31,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
+exports.skipOnGeneratedSource = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
 __exportStar(require("./meta.js"), exports);
 exports.meta = {
     type: 'suggestion',
@@ -49,3 +49,4 @@ exports.sonarKey = 'S4524';
 exports.scope = 'Main';
 exports.languages = ['js', 'ts'];
 exports.requiredDependency = [];
+exports.skipOnGeneratedSource = true;

package/cjs/S4619/generated-meta.js

@@ -31,7 +31,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
+exports.skipOnGeneratedSource = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
 __exportStar(require("./meta.js"), exports);
 exports.meta = {
     type: 'suggestion',
@@ -49,3 +49,4 @@ exports.sonarKey = 'S4619';
 exports.scope = 'Main';
 exports.languages = ['js', 'ts'];
 exports.requiredDependency = [];
+exports.skipOnGeneratedSource = false;

package/cjs/S4621/generated-meta.js

@@ -31,7 +31,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
+exports.skipOnGeneratedSource = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
 __exportStar(require("./meta.js"), exports);
 exports.meta = {
     type: 'suggestion',
@@ -49,3 +49,4 @@ exports.sonarKey = 'S4621';
 exports.scope = 'Main';
 exports.languages = ['ts'];
 exports.requiredDependency = [];
+exports.skipOnGeneratedSource = false;

package/cjs/S4622/generated-meta.js

@@ -31,7 +31,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
+exports.skipOnGeneratedSource = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
 __exportStar(require("./meta.js"), exports);
 exports.meta = {
     type: 'suggestion',
@@ -53,3 +53,4 @@ exports.sonarKey = 'S4622';
 exports.scope = 'Main';
 exports.languages = ['ts'];
 exports.requiredDependency = [];
+exports.skipOnGeneratedSource = false;

package/cjs/S4623/generated-meta.js

@@ -31,7 +31,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
+exports.skipOnGeneratedSource = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
 __exportStar(require("./meta.js"), exports);
 exports.meta = {
     type: 'suggestion',
@@ -49,3 +49,4 @@ exports.sonarKey = 'S4623';
 exports.scope = 'Main';
 exports.languages = ['ts'];
 exports.requiredDependency = [];
+exports.skipOnGeneratedSource = false;

package/cjs/S4624/generated-meta.js

@@ -31,7 +31,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.requiredEcmaVersion = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
+exports.requiredEcmaVersion = exports.skipOnGeneratedSource = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
 __exportStar(require("./meta.js"), exports);
 exports.meta = {
     type: 'suggestion',
@@ -49,4 +49,5 @@ exports.sonarKey = 'S4624';
 exports.scope = 'Main';
 exports.languages = ['js', 'ts'];
 exports.requiredDependency = [];
+exports.skipOnGeneratedSource = true;
 exports.requiredEcmaVersion = 2015;

package/cjs/S4624/rule.js

@@ -51,9 +51,8 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.rule = void 0;
-const ancestor_js_1 = require("../helpers/ancestor.js");
+const ast_js_1 = require("../helpers/ast.js");
 const generate_meta_js_1 = require("../helpers/generate-meta.js");
-const collection_js_1 = require("../helpers/collection.js");
 const meta = __importStar(require("./generated-meta.js"));
 exports.rule = {
     meta: (0, generate_meta_js_1.generateMeta)(meta, {
@@ -64,11 +63,7 @@ exports.rule = {
     create(context) {
         return {
             'TemplateLiteral TemplateLiteral': (node) => {
-                const ancestors = (0, ancestor_js_1.ancestorsChain)(node, new Set(['TemplateLiteral']));
-                const nestingTemplate = (0, collection_js_1.last)(ancestors);
-                const { start: nestingStart, end: nestingEnd } = nestingTemplate.loc;
-                const { start: nestedStart, end: nestedEnd } = node.loc;
-                if (nestedStart.line === nestingStart.line || nestedEnd.line === nestingEnd.line) {
+                if ((0, ast_js_1.isOnEnclosingTemplateDelimiterLine)(node)) {
                     context.report({
                         messageId: 'nestedTemplateLiterals',
                         node,

package/cjs/S4634/generated-meta.js

@@ -31,7 +31,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.requiredEcmaVersion = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
+exports.requiredEcmaVersion = exports.skipOnGeneratedSource = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
 __exportStar(require("./meta.js"), exports);
 exports.meta = {
     type: 'suggestion',
@@ -49,4 +49,5 @@ exports.sonarKey = 'S4634';
 exports.scope = 'Main';
 exports.languages = ['js', 'ts'];
 exports.requiredDependency = [];
+exports.skipOnGeneratedSource = true;
 exports.requiredEcmaVersion = 2015;

package/cjs/S4721/generated-meta.js

@@ -31,21 +31,22 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
+exports.skipOnGeneratedSource = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
 __exportStar(require("./meta.js"), exports);
 exports.meta = {
     type: 'problem',
     docs: {
-        description: 'Using shell interpreter when executing OS commands is security-sensitive',
-        recommended: true,
+        description: 'OS commands should not be executed using a shell interpreter',
+        recommended: false,
         url: 'https://sonarsource.github.io/rspec/#/rspec/S4721/javascript',
         requiresTypeChecking: false,
     },
     fixable: undefined,
-    deprecated: false,
+    deprecated: true,
     defaultOptions: [],
 };
 exports.sonarKey = 'S4721';
 exports.scope = 'Main';
 exports.languages = ['js', 'ts'];
 exports.requiredDependency = [];
+exports.skipOnGeneratedSource = false;

package/cjs/S4782/generated-meta.js

@@ -31,7 +31,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
+exports.skipOnGeneratedSource = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
 __exportStar(require("./meta.js"), exports);
 exports.meta = {
     type: 'suggestion',
@@ -49,3 +49,4 @@ exports.sonarKey = 'S4782';
 exports.scope = 'Main';
 exports.languages = ['ts'];
 exports.requiredDependency = [];
+exports.skipOnGeneratedSource = false;

package/cjs/S4782/rule.js

@@ -49,29 +49,45 @@ var __importStar = (this && this.__importStar) || (function () {
         return result;
     };
 })();
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.rule = void 0;
+const typescript_1 = __importDefault(require("typescript"));
 const generate_meta_js_1 = require("../helpers/generate-meta.js");
 const parser_services_js_1 = require("../helpers/parser-services.js");
 const location_js_1 = require("../helpers/location.js");
+const type_origin_js_1 = require("../helpers/type-origin.js");
 const meta = __importStar(require("./generated-meta.js"));
 exports.rule = {
     meta: (0, generate_meta_js_1.generateMeta)(meta, { hasSuggestions: true }),
     create(context) {
-        if (!(0, parser_services_js_1.isRequiredParserServices)(context.sourceCode.parserServices)) {
-            return {};
-        }
-        const compilerOptions = context.sourceCode.parserServices.program.getCompilerOptions();
-        if (compilerOptions.exactOptionalPropertyTypes) {
+        const parserServices = context.sourceCode.parserServices;
+        const hasTypeChecking = (0, parser_services_js_1.isRequiredParserServices)(parserServices);
+        const compilerOptions = hasTypeChecking
+            ? parserServices.program.getCompilerOptions()
+            : undefined;
+        if (compilerOptions?.exactOptionalPropertyTypes) {
             return {};
         }
+        const canResolveTypes = hasTypeChecking
+            ? (compilerOptions?.strictNullChecks ?? compilerOptions?.strict ?? false)
+            : false;
         function checkProperty(node) {
             const tsNode = node;
             const optionalToken = context.sourceCode.getFirstToken(node, token => token.value === '?');
-            if (!tsNode.optional || !optionalToken) {
+            const rootType = tsNode.typeAnnotation?.typeAnnotation;
+            if (!tsNode.optional || !optionalToken || !rootType) {
                 return;
             }
-            const typeNode = getUndefinedTypeAnnotation(tsNode.typeAnnotation);
+            let typeNode;
+            if (rootType.type === 'TSUnionType') {
+                typeNode = findSyntacticUndefinedTypeNode(rootType);
+            }
+            if (!typeNode && canResolveTypes) {
+                typeNode = findSemanticUndefinedTypeNode(rootType, parserServices);
+            }
             if (typeNode) {
                 const suggest = getQuickFixSuggestions(context, optionalToken, typeNode);
                 (0, location_js_1.report)(context, {
@@ -86,18 +102,84 @@ exports.rule = {
         };
     },
 };
-function getUndefinedTypeAnnotation(tsTypeAnnotation) {
-    if (tsTypeAnnotation) {
-        return getUndefinedTypeNode(tsTypeAnnotation.typeAnnotation);
+function findSemanticUndefinedTypeNode(rootType, services) {
+    const tsTypeNode = services.esTreeNodeToTSNodeMap.get(rootType);
+    const checker = services.program.getTypeChecker();
+    const type = checker.getTypeAtLocation(tsTypeNode);
+    if (!type.isUnion()) {
+        return undefined;
     }
-    return undefined;
+    const hasUndefined = type.types.some(isUndefinedType);
+    const hasAllUndefined = type.types.every(isUndefinedType);
+    if (!hasUndefined || hasAllUndefined) {
+        return undefined;
+    }
+    // If the user wrote `... | undefined` anywhere in the annotation along a
+    // path that propagates into the top-level resolved union (e.g. inside the
+    // type argument of a distributive external generic like Vue's `MaybeRef`),
+    // the inline keyword is editable — flag without consulting the classifier.
+    if (hasInlineUnionPositionUndefined(rootType, services)) {
+        return rootType;
+    }
+    // Otherwise `undefined` must come from a declaration: only flag when it's
+    // reachable from a user-editable (internal) top-level member. Pure external
+    // references like `React.ReactNode` stay suppressed.
+    const { internal } = (0, type_origin_js_1.classifyTypesByOrigin)(rootType, services);
+    const undefinedReachableFromInternal = internal.some(member => containsUndefined(member, services));
+    return undefinedReachableFromInternal ? rootType : undefined;
+}
+/**
+ * Looks for a `TSUndefinedKeyword` written in union position along a path
+ * that propagates into the top-level resolved union. We only descend through:
+ *
+ *  - `TSUnionType.types` — unions are naturally distributive at every level.
+ *  - `TSTypeReference.typeArguments.params`, but only when the reference's
+ *    own resolved type is a union containing `undefined`. This filters out
+ *    non-distributive wrappers like `Map<K, V | undefined>`, where the inner
+ *    `| undefined` is buried inside the value type and cannot reach the
+ *    property's top-level union.
+ */
+function hasInlineUnionPositionUndefined(root, services) {
+    const stack = [root];
+    while (stack.length > 0) {
+        const node = stack.pop();
+        if (node.type === 'TSUndefinedKeyword' && node.parent?.type === 'TSUnionType') {
+            return true;
+        }
+        pushPropagatingChildren(node, stack, services);
+    }
+    return false;
+}
+function pushPropagatingChildren(node, stack, services) {
+    if (node.type === 'TSUnionType') {
+        stack.push(...node.types);
+        return;
+    }
+    if (node.type === 'TSTypeReference' && propagatesUndefined(node, services)) {
+        const params = node.typeArguments?.params;
+        params && stack.push(...params);
+    }
+}
+function isUndefinedType(t) {
+    return (t.flags & typescript_1.default.TypeFlags.Undefined) !== 0;
+}
+function propagatesUndefined(node, services) {
+    const tsNode = services.esTreeNodeToTSNodeMap.get(node);
+    const type = services.program.getTypeChecker().getTypeAtLocation(tsNode);
+    return type.isUnion() && type.types.some(isUndefinedType);
+}
+function containsUndefined(node, services) {
+    const tsNode = services.esTreeNodeToTSNodeMap.get(node);
+    const type = services.program.getTypeChecker().getTypeAtLocation(tsNode);
+    const members = type.isUnion() ? type.types : [type];
+    return members.some(isUndefinedType);
 }
-function getUndefinedTypeNode(typeNode) {
+function findSyntacticUndefinedTypeNode(typeNode) {
     if (typeNode.type === 'TSUndefinedKeyword') {
         return typeNode;
     }
-    else if (typeNode.type === 'TSUnionType') {
-        return typeNode.types.map(getUndefinedTypeNode).find(tpe => tpe !== undefined);
+    if (typeNode.type === 'TSUnionType') {
+        return typeNode.types.map(findSyntacticUndefinedTypeNode).find(type => type !== undefined);
     }
     return undefined;
 }

package/cjs/S4790/generated-meta.js

@@ -31,12 +31,12 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
+exports.skipOnGeneratedSource = exports.requiredDependency = exports.languages = exports.scope = exports.sonarKey = exports.meta = void 0;
 __exportStar(require("./meta.js"), exports);
 exports.meta = {
     type: 'problem',
     docs: {
-        description: 'Using weak hashing algorithms is security-sensitive',
+        description: 'Weak hashing algorithms should not be used',
         recommended: true,
         url: 'https://sonarsource.github.io/rspec/#/rspec/S4790/javascript',
         requiresTypeChecking: false,
@@ -49,3 +49,4 @@ exports.sonarKey = 'S4790';
 exports.scope = 'Main';
 exports.languages = ['js', 'ts'];
 exports.requiredDependency = [];
+exports.skipOnGeneratedSource = false;