fix: add ESLint 10 compatibility by replacing deprecated context.getSourceCode()

aklkv · aklkv · commit aebd4f282ab3 · 2026-03-03T01:20:39.000-08:00
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -46,3 +46,25 @@ jobs:
 
     - run: pnpm install
     - run: pnpm test:coverage
+
+  eslint-versions:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        eslint-version: [8, 9, 10]
+
+    steps:
+    - uses: actions/checkout@v6
+    - uses: pnpm/action-setup@v4
+      with:
+        run_install: false
+    - uses: actions/setup-node@v6
+      with:
+        node-version: 22.x
+        cache: 'pnpm'
+
+    - run: pnpm install
+    - name: Install ESLint ${{ matrix.eslint-version }}
+      run: pnpm add -D eslint@${{ matrix.eslint-version }} --ignore-workspace-root-check
+    - name: Test with ESLint ${{ matrix.eslint-version }}
+      run: pnpm test
diff --git a/lib/rules/alias-model-in-controller.js b/lib/rules/alias-model-in-controller.js
@@ -28,7 +28,7 @@ module.exports = {
       context.report({ node, message });
     };
 
-    const sourceCode = context.getSourceCode();
+    const sourceCode = context.sourceCode ?? context.getSourceCode();
     const { scopeManager } = sourceCode;
 
     return {
diff --git a/lib/rules/avoid-leaking-state-in-ember-objects.js b/lib/rules/avoid-leaking-state-in-ember-objects.js
@@ -80,7 +80,7 @@ module.exports = {
       context.report({ node, message });
     };
 
-    const sourceCode = context.getSourceCode();
+    const sourceCode = context.sourceCode ?? context.getSourceCode();
     const { scopeManager } = sourceCode;
 
     return {
diff --git a/lib/rules/avoid-using-needs-in-controllers.js b/lib/rules/avoid-using-needs-in-controllers.js
@@ -27,7 +27,7 @@ module.exports = {
       context.report({ node, message });
     };
 
-    const sourceCode = context.getSourceCode();
+    const sourceCode = context.sourceCode ?? context.getSourceCode();
     const { scopeManager } = sourceCode;
 
     return {
diff --git a/lib/rules/no-actions-hash.js b/lib/rules/no-actions-hash.js
@@ -19,7 +19,7 @@ module.exports = {
   },
 
   create: (context) => {
-    const sourceCode = context.getSourceCode();
+    const sourceCode = context.sourceCode ?? context.getSourceCode();
     const { scopeManager } = sourceCode;
 
     function reportActionsProp(properties) {
diff --git a/lib/rules/no-array-prototype-extensions.js b/lib/rules/no-array-prototype-extensions.js
@@ -180,7 +180,7 @@ function applyFix(callExpressionNode, fixer, context, options = {}) {
   const propertyName = calleeProp.name;
   const calleeObj = callExpressionNode.callee.object;
   const callArgs = callExpressionNode.arguments;
-  const sourceCode = context.getSourceCode();
+  const sourceCode = context.sourceCode ?? context.getSourceCode();
 
   // Get the open parenthesis immediately after the callee name
   const openParenToken = sourceCode.getTokenAfter(calleeProp, {
@@ -644,7 +644,7 @@ module.exports = {
   },
 
   create(context) {
-    const sourceCode = context.getSourceCode();
+    const sourceCode = context.sourceCode ?? context.getSourceCode();
     const { scopeManager } = sourceCode;
     let importedGetName;
     let importedSetName;
diff --git a/lib/rules/no-assignment-of-untracked-properties-used-in-tracking-contexts.js b/lib/rules/no-assignment-of-untracked-properties-used-in-tracking-contexts.js
@@ -310,7 +310,7 @@ module.exports = {
 
         const currentClass = classStack.peek();
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode ?? context.getSourceCode();
         const nodeTextLeft = sourceCode.getText(node.left);
         const nodeTextRight = sourceCode.getText(node.right);
         const propertyName = nodeTextLeft.replace('this.', '');
@@ -348,7 +348,7 @@ module.exports = {
               );
             } else {
               // Need to add an import statement for `set`.
-              const sourceCode = context.getSourceCode();
+              const sourceCode = context.sourceCode ?? context.getSourceCode();
               return [
                 fixer.insertTextBefore(sourceCode.ast, "import { set } from '@ember/object';\n"),
                 fixer.replaceText(node, `set(this, '${propertyName}', ${nodeTextRight})`),
diff --git a/lib/rules/no-classic-classes.js b/lib/rules/no-classic-classes.js
@@ -69,7 +69,7 @@ module.exports = {
   },
 
   create(context) {
-    const sourceCode = context.getSourceCode();
+    const sourceCode = context.sourceCode ?? context.getSourceCode();
     const { scopeManager } = sourceCode;
     const options = context.options[0] || {};
     const additionalClassImports = options.additionalClassImports || [];
diff --git a/lib/rules/no-controllers.js b/lib/rules/no-controllers.js
@@ -21,7 +21,7 @@ module.exports = {
   },
 
   create: (context) => {
-    const sourceCode = context.getSourceCode();
+    const sourceCode = context.sourceCode ?? context.getSourceCode();
     const { scopeManager } = sourceCode;
 
     return {
diff --git a/lib/rules/no-deprecated-router-transition-methods.js b/lib/rules/no-deprecated-router-transition-methods.js
@@ -8,7 +8,7 @@ const decoratorUtils = require('../utils/decorators');
 
 function getBaseFixSteps(fixer, context, currentClass) {
   const fixSteps = [];
-  const sourceCode = context.getSourceCode();
+  const sourceCode = context.sourceCode ?? context.getSourceCode();
   let serviceInjectImportName = currentClass.serviceInjectImportName;
   let routerServicePropertyName = currentClass.routerServicePropertyName;
 
diff --git a/lib/rules/no-duplicate-dependent-keys.js b/lib/rules/no-duplicate-dependent-keys.js
@@ -47,7 +47,7 @@ module.exports = {
             node,
             message: ERROR_MESSAGE,
             fix(fixer) {
-              const sourceCode = context.getSourceCode();
+              const sourceCode = context.sourceCode ?? context.getSourceCode();
 
               const stringNodes = node.arguments.filter((arg) => types.isStringLiteral(arg));
               const duplicateNodes = findDuplicateStringNodes(stringNodes);
diff --git a/lib/rules/no-ember-super-in-es-classes.js b/lib/rules/no-ember-super-in-es-classes.js
@@ -57,7 +57,7 @@ module.exports = {
               return fixer.replaceText(node, `super.${method.key.name}`);
             }
 
-            const text = context.getSourceCode().getText(method.key);
+            const text = (context.sourceCode ?? context.getSourceCode()).getText(method.key);
             return fixer.replaceText(node, `super[${text}]`);
           },
         });
diff --git a/lib/rules/no-empty-attrs.js b/lib/rules/no-empty-attrs.js
@@ -28,7 +28,7 @@ module.exports = {
       context.report({ node, message });
     };
 
-    const sourceCode = context.getSourceCode();
+    const sourceCode = context.sourceCode ?? context.getSourceCode();
     const { scopeManager } = sourceCode;
 
     return {
diff --git a/lib/rules/no-get-with-default.js b/lib/rules/no-get-with-default.js
@@ -131,7 +131,7 @@ function fix({
   isImported,
   importedGetName,
 }) {
-  const sourceCode = context.getSourceCode();
+  const sourceCode = context.sourceCode ?? context.getSourceCode();
 
   const nodeObjectSourceText = sourceCode.getText(nodeObject);
   const nodePropertySourceText = sourceCode.getText(nodeProperty);
diff --git a/lib/rules/no-get.js b/lib/rules/no-get.js
@@ -232,7 +232,7 @@ module.exports = {
 
     const filename = context.getFilename();
 
-    const sourceCode = context.getSourceCode();
+    const sourceCode = context.sourceCode ?? context.getSourceCode();
     const { scopeManager } = sourceCode;
 
     // Skip mirage directory
@@ -309,7 +309,7 @@ module.exports = {
           (!node.arguments[0].value.includes('.') || !ignoreNestedPaths)
         ) {
           // Example: this.get('foo');
-          const sourceCode = context.getSourceCode();
+          const sourceCode = context.sourceCode ?? context.getSourceCode();
           reportGet({
             node,
             context,
@@ -330,7 +330,7 @@ module.exports = {
           (!node.arguments[1].value.includes('.') || !ignoreNestedPaths)
         ) {
           // Example: get(this, 'foo');
-          const sourceCode = context.getSourceCode();
+          const sourceCode = context.sourceCode ?? context.getSourceCode();
           reportGet({
             node,
             context,
@@ -352,7 +352,7 @@ module.exports = {
           typeof node.arguments[0].value === 'number'
         ) {
           // Example: this.get(5);
-          const sourceCode = context.getSourceCode();
+          const sourceCode = context.sourceCode ?? context.getSourceCode();
           reportGet({
             node,
             context,
@@ -371,7 +371,7 @@ module.exports = {
           typeof node.arguments[1].value === 'number'
         ) {
           // Example: get(this, 5);
-          const sourceCode = context.getSourceCode();
+          const sourceCode = context.sourceCode ?? context.getSourceCode();
           reportGet({
             node,
             context,
@@ -392,7 +392,7 @@ module.exports = {
           types.isLiteral(node.arguments[0].alternate)
         ) {
           // Example: this.get(foo ? 'bar' : 'baz');
-          const sourceCode = context.getSourceCode();
+          const sourceCode = context.sourceCode ?? context.getSourceCode();
           reportGet({
             node,
             context,
@@ -414,7 +414,7 @@ module.exports = {
           types.isLiteral(node.arguments[1].alternate)
         ) {
           // Example: get(foo, bar ? 'baz' : 'biz');
-          const sourceCode = context.getSourceCode();
+          const sourceCode = context.sourceCode ?? context.getSourceCode();
           reportGet({
             node,
             context,
@@ -442,7 +442,9 @@ module.exports = {
           validateGetPropertiesArguments(node.arguments, ignoreNestedPaths)
         ) {
           // Example: this.getProperties('foo', 'bar');
-          const objectText = context.getSourceCode().getText(node.callee.object);
+          const objectText = (context.sourceCode ?? context.getSourceCode()).getText(
+            node.callee.object
+          );
           const properties = node.arguments;
           reportGetProperties({ context, node, objectText, properties });
         }
@@ -454,7 +456,9 @@ module.exports = {
           validateGetPropertiesArguments(node.arguments.slice(1), ignoreNestedPaths)
         ) {
           // Example: getProperties(this, 'foo', 'bar');
-          const objectText = context.getSourceCode().getText(node.arguments[0]);
+          const objectText = (context.sourceCode ?? context.getSourceCode()).getText(
+            node.arguments[0]
+          );
           const properties = node.arguments.slice(1);
           reportGetProperties({
             context,
diff --git a/lib/rules/no-implicit-injections.js b/lib/rules/no-implicit-injections.js
@@ -222,7 +222,7 @@ module.exports = {
                 serviceName: failedConfig.service,
               },
               fix(fixer) {
-                const sourceCode = context.getSourceCode();
+                const sourceCode = context.sourceCode ?? context.getSourceCode();
 
                 // service inject is already declared
                 if (serviceInjectImportName) {
diff --git a/lib/rules/no-implicit-service-injection-argument.js b/lib/rules/no-implicit-service-injection-argument.js
@@ -53,7 +53,7 @@ module.exports = {
         node: node.decorators[0].expression,
         message: ERROR_MESSAGE,
         fix(fixer) {
-          const sourceCode = context.getSourceCode();
+          const sourceCode = context.sourceCode ?? context.getSourceCode();
 
           // Ideally, we want to match the service's filename, and kebab-case filenames are most common.
           const serviceName = emberUtils.convertServiceNameToKebabCase(
@@ -100,7 +100,7 @@ module.exports = {
             node: node.value,
             message: ERROR_MESSAGE,
             fix(fixer) {
-              const sourceCode = context.getSourceCode();
+              const sourceCode = context.sourceCode ?? context.getSourceCode();
 
               // Ideally, we want to match the service's filename, and kebab-case filenames are most common.
               const serviceName = emberUtils.convertServiceNameToKebabCase(
diff --git a/lib/rules/no-incorrect-computed-macros.js b/lib/rules/no-incorrect-computed-macros.js
@@ -55,7 +55,7 @@ module.exports = {
                 if (importNameReadOnly) {
                   return fixer.replaceText(node.callee, importNameReadOnly);
                 } else {
-                  const sourceCode = context.getSourceCode();
+                  const sourceCode = context.sourceCode ?? context.getSourceCode();
                   return [
                     fixer.insertTextBefore(
                       sourceCode.ast,
diff --git a/lib/rules/no-invalid-dependent-keys.js b/lib/rules/no-invalid-dependent-keys.js
@@ -80,7 +80,7 @@ module.exports = {
           (arg) => types.isLiteral(arg) && typeof arg.value === 'string'
         );
 
-        const sourceCode = context.getSourceCode();
+        const sourceCode = context.sourceCode ?? context.getSourceCode();
 
         for (const node of stringArgs) {
           if (hasTerminalAtEach(node.value)) {
diff --git a/lib/rules/no-noop-setup-on-error-in-before.js b/lib/rules/no-noop-setup-on-error-in-before.js
@@ -30,7 +30,7 @@ module.exports = {
     let isInBeforeEachHook = false;
     let isInBeforeHook = false;
     let hooksName;
-    const sourceCode = context.getSourceCode();
+    const sourceCode = context.sourceCode ?? context.getSourceCode();
 
     function reportErrorForNodeIfInBefore(node) {
       const isInBeforeOrBeforeEach =
diff --git a/lib/rules/no-on-calls-in-components.js b/lib/rules/no-on-calls-in-components.js
@@ -62,7 +62,7 @@ module.exports = {
       context.report({ node, message });
     };
 
-    const sourceCode = context.getSourceCode();
+    const sourceCode = context.sourceCode ?? context.getSourceCode();
     const { scopeManager } = sourceCode;
 
     return {
diff --git a/lib/rules/no-replace-test-comments.js b/lib/rules/no-replace-test-comments.js
@@ -47,7 +47,7 @@ module.exports = {
     }
     return {
       Program(/* node */) {
-        const sourceCode = context.getSourceCode() || {};
+        const sourceCode = (context.sourceCode ?? context.getSourceCode()) || {};
         const comments = sourceCode.getAllComments() || [];
         for (const comment of comments) {
           checkForRealTestsComment(comment);
diff --git a/lib/rules/no-restricted-property-modifications.js b/lib/rules/no-restricted-property-modifications.js
@@ -97,7 +97,9 @@ module.exports = {
             node,
             messageId: 'useReadOnlyMacro',
             fix(fixer) {
-              const argumentText0 = context.getSourceCode().getText(node.arguments[0]);
+              const argumentText0 = (context.sourceCode ?? context.getSourceCode()).getText(
+                node.arguments[0]
+              );
               return node.callee.type === 'MemberExpression'
                 ? fixer.replaceText(node, `${importedComputedName}.readOnly(${argumentText0})`)
                 : fixer.replaceText(node, `readOnly(${argumentText0})`);
diff --git a/lib/rules/no-shadow-route-definition.js b/lib/rules/no-shadow-route-definition.js
@@ -58,7 +58,7 @@ module.exports = {
 
     const routeMap = new Map();
 
-    const sourceCode = context.getSourceCode();
+    const sourceCode = context.sourceCode ?? context.getSourceCode();
     const { scopeManager } = sourceCode;
 
     return {
diff --git a/lib/rules/no-unnecessary-route-path-option.js b/lib/rules/no-unnecessary-route-path-option.js
@@ -28,7 +28,7 @@ module.exports = {
   ERROR_MESSAGE,
 
   create(context) {
-    const sourceCode = context.getSourceCode();
+    const sourceCode = context.sourceCode ?? context.getSourceCode();
     const { scopeManager } = sourceCode;
 
     return {
@@ -56,7 +56,7 @@ module.exports = {
             node: pathOptionNode,
             message: ERROR_MESSAGE,
             fix(fixer) {
-              const sourceCode = context.getSourceCode();
+              const sourceCode = context.sourceCode ?? context.getSourceCode();
 
               if (optionsNode.parent.type === 'VariableDeclarator') {
                 // When the options object is a separate variable.
diff --git a/lib/rules/no-unused-services.js b/lib/rules/no-unused-services.js
@@ -31,7 +31,7 @@ module.exports = {
 
   create(context) {
     const classStack = new Stack();
-    const sourceCode = context.getSourceCode();
+    const sourceCode = context.sourceCode ?? context.getSourceCode();
 
     let importedComputedName;
     let importedEmberName;
diff --git a/lib/rules/order-in-components.js b/lib/rules/order-in-components.js
@@ -91,7 +91,7 @@ module.exports = {
     let importedObserverName;
     let importedControllerName;
 
-    const sourceCode = context.getSourceCode();
+    const sourceCode = context.sourceCode ?? context.getSourceCode();
     const { scopeManager } = sourceCode;
 
     return {
diff --git a/lib/rules/order-in-controllers.js b/lib/rules/order-in-controllers.js
@@ -63,7 +63,7 @@ module.exports = {
     let importedObserverName;
     let importedControllerName;
 
-    const sourceCode = context.getSourceCode();
+    const sourceCode = context.sourceCode ?? context.getSourceCode();
     const { scopeManager } = sourceCode;
 
     return {
diff --git a/lib/rules/order-in-models.js b/lib/rules/order-in-models.js
@@ -58,7 +58,7 @@ module.exports = {
     let importedObserverName;
     let importedControllerName;
 
-    const sourceCode = context.getSourceCode();
+    const sourceCode = context.sourceCode ?? context.getSourceCode();
     const { scopeManager } = sourceCode;
 
     return {
diff --git a/lib/rules/order-in-routes.js b/lib/rules/order-in-routes.js
diff --git a/lib/rules/require-computed-macros.js b/lib/rules/require-computed-macros.js
diff --git a/lib/rules/require-computed-property-dependencies.js b/lib/rules/require-computed-property-dependencies.js
diff --git a/lib/rules/require-super-in-lifecycle-hooks.js b/lib/rules/require-super-in-lifecycle-hooks.js
diff --git a/lib/rules/require-tagless-components.js b/lib/rules/require-tagless-components.js
diff --git a/lib/rules/route-path-style.js b/lib/rules/route-path-style.js
diff --git a/lib/rules/routes-segments-snake-case.js b/lib/rules/routes-segments-snake-case.js
diff --git a/lib/rules/use-ember-get-and-set.js b/lib/rules/use-ember-get-and-set.js
diff --git a/lib/utils/property-getter.js b/lib/utils/property-getter.js
diff --git a/lib/utils/property-order.js b/lib/utils/property-order.js
