@@ -257,12 +257,12 @@ export default createRule<Options, MessageIds>({
       /*
        * Class field value are implicit functions.
        */
-      'PropertyDefinition > *.key:exit'(): void {
-        pushContext();
-      },
       'PropertyDefinition:exit'(): void {
         popContext();
       },
+      'PropertyDefinition > *.key:exit'(): void {
+        pushContext();
+      },
 
       /*
        * Class static blocks are implicit functions. They aren't required to use `this`,
 
@@ -22,7 +22,7 @@ export type Options = [
 
 export type MessageIds = 'duplicate' | 'unnecessary';
 
-const astIgnoreKeys = new Set(['range', 'loc', 'parent']);
+const astIgnoreKeys = new Set(['loc', 'parent', 'range']);
 
 const isSameAstNode = (actualNode: unknown, expectedNode: unknown): boolean => {
   if (actualNode === expectedNode) {
 
@@ -12,12 +12,12 @@ import {
 } from '../util';
 
 const FUNCTION_CONSTRUCTOR = 'Function';
-const GLOBAL_CANDIDATES = new Set(['global', 'window', 'globalThis']);
+const GLOBAL_CANDIDATES = new Set(['global', 'globalThis', 'window']);
 const EVAL_LIKE_METHODS = new Set([
+  'execScript',
   'setImmediate',
   'setInterval',
   'setTimeout',
-  'execScript',
 ]);
 
 export default createRule({
 
@@ -55,13 +55,13 @@ export default createRule<Options, MessageIds>({
   ],
   create(context, [options]) {
     const CLASS_DECLARATION_MERGE_NODES = new Set<AST_NODE_TYPES>([
+      AST_NODE_TYPES.ClassDeclaration,
       AST_NODE_TYPES.TSInterfaceDeclaration,
       AST_NODE_TYPES.TSModuleDeclaration,
-      AST_NODE_TYPES.ClassDeclaration,
     ]);
     const FUNCTION_DECLARATION_MERGE_NODES = new Set<AST_NODE_TYPES>([
-      AST_NODE_TYPES.TSModuleDeclaration,
       AST_NODE_TYPES.FunctionDeclaration,
+      AST_NODE_TYPES.TSModuleDeclaration,
     ]);
     const ENUM_DECLARATION_MERGE_NODES = new Set<AST_NODE_TYPES>([
       AST_NODE_TYPES.TSEnumDeclaration,
 
@@ -146,11 +146,11 @@ export default createRule<Options, MessageIds>({
     const aliasTypes = new Set([
       AST_NODE_TYPES.TSArrayType,
       AST_NODE_TYPES.TSImportType,
-      AST_NODE_TYPES.TSTypeReference,
-      AST_NODE_TYPES.TSLiteralType,
-      AST_NODE_TYPES.TSTypeQuery,
       AST_NODE_TYPES.TSIndexedAccessType,
+      AST_NODE_TYPES.TSLiteralType,
       AST_NODE_TYPES.TSTemplateLiteralType,
+      AST_NODE_TYPES.TSTypeQuery,
+      AST_NODE_TYPES.TSTypeReference,
     ]);
 
     /**
 
@@ -5,7 +5,7 @@ import { AST_NODE_TYPES, ASTUtils } from '@typescript-eslint/utils';
 
 import { createRule, getStaticStringValue, nullThrows } from '../util';
 
-const UNNECESSARY_OPERATORS = new Set(['=', '&&=', '||=', '??=']);
+const UNNECESSARY_OPERATORS = new Set(['??=', '&&=', '=', '||=']);
 
 export default createRule({
   name: 'no-unnecessary-parameter-property-assignment',
 
@@ -177,12 +177,12 @@ export default createRule({
       },
       TSEnumDeclaration: enterDeclaration,
       'TSEnumDeclaration:exit': exitDeclaration,
+      'TSModuleDeclaration:exit': exitDeclaration,
       'TSModuleDeclaration > TSModuleBlock'(
         node: TSESTree.TSModuleBlock,
       ): void {
         enterDeclaration(node.parent);
       },
-      'TSModuleDeclaration:exit': exitDeclaration,
       TSQualifiedName(node: TSESTree.TSQualifiedName): void {
         visitNamespaceAccess(node, node.left, node.right);
       },
 
@@ -52,9 +52,7 @@ export default createRule({
         const smTypeParameterVariable = nullThrows(
           (() => {
             const variable = scope.set.get(esTypeParameter.name.name);
-            return variable != null && variable.isTypeVariable
-              ? variable
-              : undefined;
+            return variable?.isTypeVariable ? variable : undefined;
           })(),
           "Type parameter should be present in scope's variables.",
         );
 
@@ -5,12 +5,12 @@ import { AST_NODE_TYPES } from '@typescript-eslint/utils';
 import { createRule, nullThrows } from '../util';
 
 type Modifier =
-  | 'private readonly'
   | 'private'
-  | 'protected readonly'
+  | 'private readonly'
   | 'protected'
-  | 'public readonly'
+  | 'protected readonly'
   | 'public'
+  | 'public readonly'
   | 'readonly';
 
 type Prefer = 'class-property' | 'parameter-property';
 
@@ -194,14 +194,6 @@ export default createRule({
         // when entering an interface reset the count of `this`s to empty.
         tsThisTypes = [];
       },
-      'TSInterfaceDeclaration TSThisType'(node: TSESTree.TSThisType): void {
-        // inside an interface keep track of all ThisType references.
-        // unless it's inside a nested type literal in which case it's invalid code anyway
-        // we don't want to incorrectly say "it refers to name" while typescript says it's completely invalid.
-        if (literalNesting === 0 && tsThisTypes != null) {
-          tsThisTypes.push(node);
-        }
-      },
       'TSInterfaceDeclaration:exit'(
         node: TSESTree.TSInterfaceDeclaration,
       ): void {
@@ -211,6 +203,14 @@ export default createRule({
         // on exit check member and reset the array to nothing.
         tsThisTypes = null;
       },
+      'TSInterfaceDeclaration TSThisType'(node: TSESTree.TSThisType): void {
+        // inside an interface keep track of all ThisType references.
+        // unless it's inside a nested type literal in which case it's invalid code anyway
+        // we don't want to incorrectly say "it refers to name" while typescript says it's completely invalid.
+        if (literalNesting === 0 && tsThisTypes != null) {
+          tsThisTypes.push(node);
+        }
+      },
       // keep track of nested literals to avoid complaining about invalid `this` uses
       'TSInterfaceDeclaration TSTypeLiteral'(): void {
         literalNesting += 1;