Add ExplicitThis recipe to make 'this.' prefix explicit.

Author: motlin

src/main/java/org/openrewrite/staticanalysis/ExplicitThis.java

@@ -0,0 +1,339 @@
+/*
+ * Copyright 2025 the original author or authors.
+ * <p>
+ * Licensed under the Moderne Source Available License (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ * <p>
+ * https://docs.moderne.io/licensing/moderne-source-available-license
+ * <p>
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.openrewrite.staticanalysis;
+
+import lombok.EqualsAndHashCode;
+import lombok.Value;
+import org.openrewrite.*;
+import org.openrewrite.java.JavaVisitor;
+import org.openrewrite.java.tree.Expression;
+import org.openrewrite.java.tree.J;
+import org.openrewrite.java.tree.J.FieldAccess;
+import org.openrewrite.java.tree.J.Identifier;
+import org.openrewrite.java.tree.JLeftPadded;
+import org.openrewrite.java.tree.JavaType;
+import org.openrewrite.java.tree.JavaType.Method;
+import org.openrewrite.java.tree.Space;
+import org.openrewrite.marker.Markers;
+
+import java.time.Duration;
+import java.util.Collections;
+
+@Value
+@EqualsAndHashCode(callSuper = false)
+public class ExplicitThis extends Recipe {
+
+    @Override
+    public String getDisplayName() {
+        return "`field` → `this.field`";
+    }
+
+    @Override
+    public String getDescription() {
+        return "Add explicit 'this.' prefix to field and method access.";
+    }
+
+    @Override
+    public Duration getEstimatedEffortPerOccurrence() {
+        return Duration.ofSeconds(5);
+    }
+
+    @Override
+    public TreeVisitor<?, ExecutionContext> getVisitor() {
+        return new ExplicitThisVisitor();
+    }
+
+    private static final class ExplicitThisVisitor extends JavaVisitor<ExecutionContext> {
+
+        private boolean isStatic;
+        private boolean isInsideFieldAccess;
+
+        private static class ClassContext {
+            final JavaType.FullyQualified type;
+            final boolean isAnonymous;
+
+            ClassContext(JavaType.FullyQualified type, boolean isAnonymous) {
+                this.type = type;
+                this.isAnonymous = isAnonymous;
+            }
+        }
+
+        @Override
+        public J visitFieldAccess(FieldAccess fieldAccess, ExecutionContext executionContext) {
+            boolean previousIsInsideFieldAccess = this.isInsideFieldAccess;
+            this.isInsideFieldAccess = true;
+
+            J result = super.visitFieldAccess(fieldAccess, executionContext);
+
+            this.isInsideFieldAccess = previousIsInsideFieldAccess;
+            return result;
+        }
+
+        @Override
+        public J visitIdentifier(J.Identifier identifier, ExecutionContext executionContext) {
+            J.Identifier id = (J.Identifier) super.visitIdentifier(identifier, executionContext);
+
+            if (this.isStatic) {
+                return id;
+            }
+
+            if (this.isInsideFieldAccess) {
+                return id;
+            }
+
+            JavaType.Variable fieldType = id.getFieldType();
+            if (fieldType == null) {
+                return id;
+            }
+
+            if (fieldType.getOwner() == null || !(fieldType.getOwner() instanceof JavaType.Class)) {
+                return id;
+            }
+
+            // Skip static fields - check the Modifier.STATIC flag (0x0008)
+            if ((fieldType.getFlagsBitMap() & 0x0008L) != 0) {
+                return id;
+            }
+
+            String name = id.getSimpleName();
+            if ("this".equals(name) || "super".equals(name)) {
+                return id;
+            }
+
+            if (this.isPartOfDeclaration()) {
+                return id;
+            }
+
+            J.FieldAccess fieldAccess = this.createFieldAccess(id);
+            return fieldAccess != null ? fieldAccess : id;
+        }
+
+        @Override
+        public J visitBlock(J.Block block, ExecutionContext executionContext) {
+            if (!block.isStatic()) {
+                return super.visitBlock(block, executionContext);
+            }
+
+            boolean previousStatic = this.isStatic;
+            this.isStatic = true;
+
+            J.Block result = (J.Block) super.visitBlock(block, executionContext);
+
+            this.isStatic = previousStatic;
+            return result;
+        }
+
+        @Override
+        public J visitMethodDeclaration(J.MethodDeclaration method, ExecutionContext executionContext) {
+            boolean previousStatic = this.isStatic;
+
+            JavaType.Method methodType = method.getMethodType();
+            if (methodType != null) {
+                // Check if the method is static - set isStatic flag using Modifier.STATIC (0x0008)
+                this.isStatic = (methodType.getFlagsBitMap() & 0x0008L) != 0;
+            }
+
+            J.MethodDeclaration result = (J.MethodDeclaration) super.visitMethodDeclaration(method, executionContext);
+
+            this.isStatic = previousStatic;
+
+            return result;
+        }
+
+        @Override
+        public J visitMethodInvocation(J.MethodInvocation method, ExecutionContext executionContext) {
+            J.MethodInvocation m = (J.MethodInvocation) super.visitMethodInvocation(method, executionContext);
+
+            if (this.isStatic) {
+                return m;
+            }
+
+            if (m.getName().getSimpleName().equals("super") || m.getName().getSimpleName().equals("this")) {
+                return m;
+            }
+
+            Method methodType = m.getMethodType();
+            // Skip if already qualified, type info is missing, or the method is static (Modifier.STATIC = 0x0008)
+            if (
+                    m.getSelect() != null ||
+                            methodType == null ||
+                            (methodType.getFlagsBitMap() & 0x0008L) != 0
+            ) {
+                return m;
+            }
+
+            JavaType.FullyQualified methodOwnerType = methodType.getDeclaringType();
+            if (methodOwnerType == null) {
+                return m;
+            }
+
+            ClassContext currentContext = this.getCurrentClassContext();
+            if (currentContext == null) {
+                return m;
+            }
+
+            Expression thisExpression = this.createQualifiedThisExpression(currentContext, methodOwnerType);
+            if (thisExpression == null) {
+                return m;
+            }
+
+            return m.withSelect(thisExpression);
+        }
+
+        private boolean isPartOfDeclaration() {
+            Cursor parent = this.getCursor().getParent();
+            if (parent == null || !(parent.getValue() instanceof J.VariableDeclarations.NamedVariable)) {
+                return false;
+            }
+            J.VariableDeclarations.NamedVariable namedVar = (J.VariableDeclarations.NamedVariable) parent.getValue();
+            return namedVar.getName() == this.getCursor().getValue();
+        }
+
+        private ClassContext getCurrentClassContext() {
+            Cursor currentCursor = this.getCursor().dropParentUntil(p ->
+                p instanceof J.ClassDeclaration ||
+                (p instanceof J.NewClass && ((J.NewClass) p).getBody() != null) ||
+                p == Cursor.ROOT_VALUE
+            );
+
+            if (currentCursor.getValue() instanceof J.ClassDeclaration) {
+                J.ClassDeclaration currentClass = currentCursor.getValue();
+                JavaType.FullyQualified currentClassType = currentClass.getType();
+                if (currentClassType == null) {
+                    return null;
+                }
+                String currentClassName = this.getSimpleClassName(currentClassType.getFullyQualifiedName());
+                boolean currentIsAnonymous = this.isAnonymousClassName(currentClassName);
+                return new ClassContext(currentClassType, currentIsAnonymous);
+            } else if (currentCursor.getValue() instanceof J.NewClass) {
+                J.NewClass newClass = currentCursor.getValue();
+                JavaType type = newClass.getType();
+                if (!(type instanceof JavaType.FullyQualified)) {
+                    return null;
+                }
+                return new ClassContext((JavaType.FullyQualified) type, true);
+            }
+            return null;
+        }
+
+        private Expression createQualifiedThisExpression(ClassContext currentContext, JavaType.FullyQualified targetType) {
+            if (currentContext.type.getFullyQualifiedName().equals(targetType.getFullyQualifiedName())) {
+                return new Identifier(
+                        Tree.randomId(),
+                        Space.EMPTY,
+                        Markers.EMPTY,
+                        Collections.emptyList(),
+                        "this",
+                        currentContext.type,
+                        null
+                );
+            }
+
+            if (currentContext.isAnonymous) {
+                String ownerClassName = this.getSimpleClassName(targetType.getFullyQualifiedName());
+                if (this.isAnonymousClassName(ownerClassName)) {
+                    return null;
+                }
+                return this.createOuterThisReference(targetType, ownerClassName);
+            }
+
+            String simpleClassName = this.getSimpleClassName(targetType.getFullyQualifiedName());
+            return this.createOuterThisReference(targetType, simpleClassName);
+        }
+
+        private J.FieldAccess createOuterThisReference(JavaType.FullyQualified ownerType, String simpleClassName) {
+            J.Identifier outerClassIdentifier = new J.Identifier(
+                    Tree.randomId(),
+                    Space.EMPTY,
+                    Markers.EMPTY,
+                    Collections.emptyList(),
+                    simpleClassName,
+                    ownerType,
+                    null
+            );
+
+            J.Identifier thisIdentifier = new J.Identifier(
+                    Tree.randomId(),
+                    Space.EMPTY,
+                    Markers.EMPTY,
+                    Collections.emptyList(),
+                    "this",
+                    ownerType,
+                    null
+            );
+
+            return new J.FieldAccess(
+                    Tree.randomId(),
+                    Space.EMPTY,
+                    Markers.EMPTY,
+                    outerClassIdentifier,
+                    JLeftPadded.build(thisIdentifier),
+                    null
+            );
+        }
+
+        private J.FieldAccess createFieldAccess(J.Identifier identifier) {
+            JavaType.Variable fieldType = identifier.getFieldType();
+            if (fieldType == null || fieldType.getOwner() == null) {
+                return null;
+            }
+
+            JavaType.FullyQualified fieldOwnerType = (JavaType.FullyQualified) fieldType.getOwner();
+
+            ClassContext currentContext = this.getCurrentClassContext();
+            if (currentContext == null) {
+                return null;
+            }
+
+            Expression thisExpression = this.createQualifiedThisExpression(currentContext, fieldOwnerType);
+            if (thisExpression == null) {
+                return null;
+            }
+
+            return new J.FieldAccess(
+                    Tree.randomId(),
+                    identifier.getPrefix(),
+                    Markers.EMPTY,
+                    thisExpression,
+                    JLeftPadded.build(identifier.withPrefix(Space.EMPTY)),
+                    identifier.getFieldType()
+            );
+        }
+
+        /**
+         * Extracts the simple class name from a fully qualified class name.
+         * Handles both package-separated names (dots) and inner class separators (dollar signs).
+         * Examples: "com.example.Outer$Inner" -> "Inner", "com.example.Outer" -> "Outer"
+         */
+        private String getSimpleClassName(String fullyQualifiedName) {
+            int lastDot = fullyQualifiedName.lastIndexOf('.');
+            int lastDollar = fullyQualifiedName.lastIndexOf('$');
+            int lastSeparator = Math.max(lastDot, lastDollar);
+            return lastSeparator >= 0 ? fullyQualifiedName.substring(lastSeparator + 1) : fullyQualifiedName;
+        }
+
+        /**
+         * Detects if a class name represents an anonymous class.
+         * Anonymous classes are identified by numeric names (generated by the compiler as 1, 2, 3, etc.).
+         */
+        private boolean isAnonymousClassName(String simpleName) {
+            if (simpleName == null || simpleName.isEmpty()) {
+                return false;
+            }
+            return Character.isDigit(simpleName.charAt(0));
+        }
+    }
+}