Fix porting bug in isArgumentAndStartLineOverlapsExpressionBeingCalled (#1948)

jakebailey · web-flow · commit 04872922847d · 2025-10-24T17:35:19.000Z
diff --git a/internal/format/indent.go b/internal/format/indent.go
@@ -127,7 +127,7 @@ func getActualIndentationForNode(current *ast.Node, parent *ast.Node, cuurentLin
 }
 
 func isArgumentAndStartLineOverlapsExpressionBeingCalled(parent *ast.Node, child *ast.Node, childStartLine int, sourceFile *ast.SourceFile) bool {
-	if !(ast.IsCallExpression(child) && slices.Contains(parent.AsCallExpression().Arguments.Nodes, child)) {
+	if !(ast.IsCallExpression(parent) && slices.Contains(parent.AsCallExpression().Arguments.Nodes, child)) {
 		return false
 	}
 	expressionOfCallExpressionEnd := parent.Expression().End()

Original file line number	Diff line number	Diff line change
`@@ -127,7 +127,7 @@ func getActualIndentationForNode(current ast.Node, parent ast.Node, cuurentLin`
`127`	`127`	`}`
`128`	`128`
`129`	`129`	`func isArgumentAndStartLineOverlapsExpressionBeingCalled(parent ast.Node, child ast.Node, childStartLine int, sourceFile *ast.SourceFile) bool {`
`130`		`- if !(ast.IsCallExpression(child) && slices.Contains(parent.AsCallExpression().Arguments.Nodes, child)) {`
	`130`	`+ if !(ast.IsCallExpression(parent) && slices.Contains(parent.AsCallExpression().Arguments.Nodes, child)) {`
`131`	`131`	`return false`
`132`	`132`	`}`
`133`	`133`	`expressionOfCallExpressionEnd := parent.Expression().End()`