Replace deprecated ESLint context methods with ESLint v9 compatible alternatives

Co-authored-by: nzakas <[email protected]>

Author: Copilot

rules/detect-child-process.js

@@ -24,7 +24,7 @@ module.exports = {
     },
   },
   create(context) {
-    const sourceCode = context.sourceCode || context.getSourceCode();
+    const sourceCode = context.sourceCode;
     return {
       CallExpression: function (node) {
         if (node.callee.name === 'require') {
@@ -42,7 +42,7 @@ module.exports = {
           return;
         }
 
-        const scope = sourceCode.getScope ? sourceCode.getScope(node) : context.getScope();
+        const scope = sourceCode.getScope(node);
 
         // Reports non-literal `exec()` calls.
         if (

rules/detect-no-csrf-before-method-override.js

@@ -24,7 +24,7 @@ module.exports = {
 
     return {
       CallExpression: function (node) {
-        const token = context.getSourceCode().getTokens(node)[0];
+        const token = context.sourceCode.getTokens(node)[0];
         const nodeValue = token.value;
 
         if (nodeValue === 'express') {

rules/detect-non-literal-fs-filename.js

@@ -27,15 +27,15 @@ module.exports = {
     },
   },
   create(context) {
-    const sourceCode = context.sourceCode || context.getSourceCode();
+    const sourceCode = context.sourceCode;
     return {
       CallExpression(node) {
         // don't check require. If all arguments are Literals, it's surely safe!
         if ((node.callee.type === 'Identifier' && node.callee.name === 'require') || node.arguments.every((argument) => argument.type === 'Literal')) {
           return;
         }
 
-        const scope = sourceCode.getScope ? sourceCode.getScope(node) : context.getScope();
+        const scope = sourceCode.getScope(node);
         const pathInfo = getImportAccessPath({
           node: node.callee,
           scope,

rules/detect-non-literal-regexp.js

@@ -22,13 +22,13 @@ module.exports = {
     },
   },
   create(context) {
-    const sourceCode = context.sourceCode || context.getSourceCode();
+    const sourceCode = context.sourceCode;
 
     return {
       NewExpression(node) {
         if (node.callee.name === 'RegExp') {
           const args = node.arguments;
-          const scope = sourceCode.getScope ? sourceCode.getScope(node) : context.getScope();
+          const scope = sourceCode.getScope(node);
 
           if (
             args &&

rules/detect-non-literal-require.js

@@ -22,13 +22,13 @@ module.exports = {
     },
   },
   create(context) {
-    const sourceCode = context.sourceCode || context.getSourceCode();
+    const sourceCode = context.sourceCode;
 
     return {
       CallExpression(node) {
         if (node.callee.name === 'require') {
           const args = node.arguments;
-          const scope = sourceCode.getScope ? sourceCode.getScope(node) : context.getScope();
+          const scope = sourceCode.getScope(node);
 
           if (
             args &&

rules/detect-unsafe-regex.js

@@ -28,7 +28,7 @@ module.exports = {
   create(context) {
     return {
       Literal: function (node) {
-        const token = context.getSourceCode().getTokens(node)[0];
+        const token = context.sourceCode.getTokens(node)[0];
         const nodeType = token.type;
         const nodeValue = token.value;
 

test/utils/import-utils.js

@@ -10,14 +10,14 @@ function getGetImportAccessPathResult(code) {
   const result = [];
   const testRule = {
     create(context) {
-      const sourceCode = context.sourceCode || context.getSourceCode();
+      const sourceCode = context.sourceCode;
       return {
         'Identifier[name = target]'(node) {
           let expr = node;
           if (node.parent.type === 'MemberExpression' && node.parent.property === node) {
             expr = node.parent;
           }
-          const scope = sourceCode.getScope ? sourceCode.getScope(node) : context.getScope();
+          const scope = sourceCode.getScope(node);
 
           const info = getImportAccessPath({
             node: expr,

test/utils/is-static-expression.js

@@ -14,11 +14,11 @@ function getIsStaticExpressionResult(code) {
   const result = [];
   const testRule = {
     create(context) {
-      const sourceCode = context.sourceCode || context.getSourceCode();
+      const sourceCode = context.sourceCode;
 
       return {
         'CallExpression[callee.name = target]'(node) {
-          const scope = sourceCode.getScope ? sourceCode.getScope(node) : context.getScope();
+          const scope = sourceCode.getScope(node);
 
           result.push(
             ...node.arguments.map((expr) =>