Groovy: fix parsing of property expressions in parens (#7506)

Author: greg-at-moderne

rewrite-groovy/src/main/java/org/openrewrite/groovy/GroovyParserVisitor.java

@@ -3326,6 +3326,12 @@ private String readConstantSegmentBeforeNextInterpolation(Delimiter delimiter) {
                 // Only for groovy 3+, because lower versions do always return `-1` for objectExpression.lineNumber / objectExpression.columnNumber
                 MethodCallExpression expr = (MethodCallExpression) node;
                 return determineParenthesisLevel(expr, expr.getObjectExpression().getLineNumber(), expr.getLineNumber(), expr.getObjectExpression().getColumnNumber(), expr.getColumnNumber());
+            } else if (node instanceof PropertyExpression && source.charAt(indexOfNextNonWhitespace(cursor, source)) == '(') {
+                // Groovy doesn't set _INSIDE_PARENTHESES_LEVEL on parenthesized PropertyExpressions like `(a.b)` but does
+                // extend the column range to cover the wrapping parens. Only apply when the cursor is positioned at `(` to
+                // avoid double-counting in cases where a caller already consumed the wrapping parens (e.g. map entry keys).
+                PropertyExpression expr = (PropertyExpression) node;
+                return determineParenthesisLevel(expr, expr.getObjectExpression().getLineNumber(), expr.getLineNumber(), expr.getObjectExpression().getColumnNumber(), expr.getColumnNumber());
             }
         }
 

rewrite-groovy/src/test/java/org/openrewrite/groovy/tree/FieldAccessTest.java

@@ -33,6 +33,17 @@ void starAccess() {
         );
     }
 
+    @Test
+    void starAccessInsideParenthesesFollowedByMethodCall() {
+        rewriteRun(
+          groovy(
+            """
+              def updates = (push*.remoteUpdates).flatten()
+              """
+          )
+        );
+    }
+
     @Test
     void fieldAccess() {
         rewriteRun(