From 762e7a35303e4a02ed21f86aba1a3cc5a8da8028 Mon Sep 17 00:00:00 2001
From: Tomas Votruba <tomas.vot@gmail.com>
Date: Tue, 20 Oct 2026 22:22:15 +0200
Subject: [PATCH] Add LineLength fixer to blink

Port Symplify LineLengthFixer: break array items, call/new/attribute
arguments and function parameters onto standalone lines when the opening
line exceeds the limit, and inline a short multiline block back onto one
line. Skips keyed arrays, heredocs, commented and promoted-property
blocks, matching the PHP fixer.

Reproduce every LineLengthFixerTest fixture as a Go test.
---
 blink/README.md                               |   2 +-
 blink/internal/fixer/fixer.go                 |   1 +
 blink/internal/fixer/rules/line_length.go     | 620 ++++++++++++++++++
 .../internal/fixer/rules/line_length_test.go  |  62 ++
 blink/internal/fixer/rules/priorities.go      |   1 +
 blink/internal/fixer/rules/registry.go        |   1 +
 6 files changed, 686 insertions(+), 1 deletion(-)
 create mode 100644 blink/internal/fixer/rules/line_length.go
 create mode 100644 blink/internal/fixer/rules/line_length_test.go

diff --git a/blink/README.md b/blink/README.md
index 88cf4797b4..c9d8d8c349 100644
--- a/blink/README.md
+++ b/blink/README.md
@@ -13,7 +13,7 @@ CI keeps it current.
 | tool | rules | of ECS |
 |---|---:|---:|
 | [ECS](https://github.com/symplify/easy-coding-standard) (baseline) | 207 | 100% |
-| blink | 210 | 101% |
+| blink | 211 | 101% |
 <!-- rule-counts:end -->
 
 ## Build
diff --git a/blink/internal/fixer/fixer.go b/blink/internal/fixer/fixer.go
index e699293592..4e843e666a 100644
--- a/blink/internal/fixer/fixer.go
+++ b/blink/internal/fixer/fixer.go
@@ -28,6 +28,7 @@ var ecsSrcOnlyFixers = map[string]bool{
 	`Symplify\CodingStandard\Fixer\Spacing\NoBlankLineBetweenImportsFixer`:                true,
 	`Symplify\CodingStandard\Fixer\Spacing\SpaceAfterCommaHereNowDocFixer`:                true,
 	`Symplify\CodingStandard\Fixer\ArrayNotation\ArrayOpenerAndCloserNewlineFixer`:        true,
+	`Symplify\CodingStandard\Fixer\LineLength\LineLengthFixer`:                            true,
 	`Symplify\CodingStandard\Fixer\Commenting\FixTagTypoFixer`:                            true,
 	`Symplify\CodingStandard\Fixer\Commenting\TypeToVarTagFixer`:                          true,
 	`Symplify\CodingStandard\Fixer\Commenting\MergeDocBlockStartFixer`:                    true,
diff --git a/blink/internal/fixer/rules/line_length.go b/blink/internal/fixer/rules/line_length.go
new file mode 100644
index 0000000000..8fd5ad0ffd
--- /dev/null
+++ b/blink/internal/fixer/rules/line_length.go
@@ -0,0 +1,620 @@
+package rules
+
+import (
+	"strings"
+
+	"blink/internal/fixer"
+	"blink/internal/token"
+	"blink/internal/tokens"
+)
+
+// ecs-src: https://github.com/ecsphp/ecs-src/blob/main/packages/coding-standard/src/Fixer/LineLength/LineLengthFixer.php
+//
+// LineLength breaks array items, method parameters, call arguments, new and
+// attribute arguments onto standalone lines when the opening line overflows the
+// limit, and inlines a short multiline block back onto one line. It ports
+// Symplify's LineLengthFixer: process every closer bottom-up so nested blocks
+// resolve outer-first, decide by the real rendered line length, and skip keyed
+// arrays, heredocs, commented and promoted-property blocks.
+type LineLength struct {
+	lineLength      int
+	breakLongLines  bool
+	inlineShortLine bool
+	configured      bool
+}
+
+const (
+	llDefaultLineLength = 120
+	llIndent            = "    "
+	llEnding            = "\n"
+)
+
+func (LineLength) Name() string {
+	return `Symplify\CodingStandard\Fixer\LineLength\LineLengthFixer`
+}
+
+func (LineLength) SourceURL() string {
+	return fixer.EcsSrcSourceBase + "LineLength/LineLengthFixer.php"
+}
+
+func (f LineLength) WithConfig(config map[string]any) fixer.Fixer {
+	f.configured = true
+	f.lineLength = configInt(config, "line_length", llDefaultLineLength)
+	f.breakLongLines = true
+	f.inlineShortLine = true
+	if v, ok := config["break_long_lines"].(bool); ok {
+		f.breakLongLines = v
+	}
+	if v, ok := config["inline_short_lines"].(bool); ok {
+		f.inlineShortLine = v
+	}
+	return f
+}
+
+// resolved fills the zero-value defaults: a bare LineLength{} means all on.
+func (f LineLength) resolved() LineLength {
+	if f.configured {
+		return f
+	}
+	return LineLength{
+		lineLength:      llDefaultLineLength,
+		breakLongLines:  true,
+		inlineShortLine: true,
+		configured:      true,
+	}
+}
+
+func (f LineLength) Fix(s *tokens.Stream) bool {
+	f = f.resolved()
+	changed := false
+	// bottom-up: an insert inside a block only shifts tokens to its right, so the
+	// not-yet-visited lower indices stay valid; the outermost closer is processed
+	// first, which is what makes nested calls resolve outer-to-inner
+	n := s.Len()
+	for pos := n - 1; pos >= 0; pos-- {
+		if pos >= s.Len() {
+			continue
+		}
+		t := s.At(pos)
+		switch {
+		case t.Kind == token.Punct && t.Value == ")":
+			if f.processMethodCall(s, pos) {
+				changed = true
+			}
+		case llIsAttribute(t):
+			if f.processAttribute(s, pos) {
+				changed = true
+			}
+		case t.Kind == token.Keyword && strings.EqualFold(t.Value, "new"):
+			if f.processOpener(s, pos) {
+				changed = true
+			}
+		case t.Kind == token.Keyword && strings.EqualFold(t.Value, "function"):
+			if f.processOpener(s, pos) {
+				changed = true
+			}
+		case t.Kind == token.Keyword && strings.EqualFold(t.Value, "use") && llIsClosureUse(s, pos):
+			if f.processOpener(s, pos) {
+				changed = true
+			}
+		case t.Kind == token.Punct && t.Value == "]" && llIsArrayClose(s, pos):
+			if f.processArrayClose(s, pos) {
+				changed = true
+			}
+		}
+	}
+	return changed
+}
+
+// processMethodCall handles a call/declaration closer ")" whose name is a plain
+// lower-cased identifier, mirroring FunctionCallNameMatcher.
+func (f LineLength) processMethodCall(s *tokens.Stream, closeIdx int) bool {
+	open := s.MatchBackward(closeIdx)
+	if open <= 0 {
+		return false
+	}
+	name := open - 1
+	if s.At(name).Kind != token.Ident {
+		return false
+	}
+	if v := s.At(name).Value; v == "" || v[0] < 'a' || v[0] > 'z' {
+		return false
+	}
+	// empty argument list
+	if open+1 < s.Len() && s.At(open+1).Kind == token.Punct && s.At(open+1).Value == ")" {
+		return false
+	}
+	// a comment inside is unsafe to reflow
+	for j := open; j <= closeIdx && j < s.Len(); j++ {
+		if k := s.At(j).Kind; k == token.Comment || k == token.DocComment {
+			return false
+		}
+	}
+	return f.transform(s, open, closeIdx)
+}
+
+// processOpener handles the new / function / closure-use keywords by locating
+// their "(" block, then applying the array/function skip rules.
+func (f LineLength) processOpener(s *tokens.Stream, pos int) bool {
+	open := -1
+	for j := pos + 1; j < s.Len(); j++ {
+		t := s.At(j)
+		if t.Kind == token.Punct && t.Value == ";" {
+			return false
+		}
+		if t.Kind == token.Punct && t.Value == "(" {
+			open = j
+			break
+		}
+	}
+	if open < 0 {
+		return false
+	}
+	closeIdx := s.MatchForward(open)
+	if closeIdx < 0 {
+		return false
+	}
+	// no arguments
+	if sigNext(s, open) == closeIdx {
+		return false
+	}
+	if f.shouldSkip(s, open, closeIdx) {
+		return false
+	}
+	return f.transform(s, open, closeIdx)
+}
+
+// processArrayClose handles a standalone "[...]" literal.
+func (f LineLength) processArrayClose(s *tokens.Stream, closeIdx int) bool {
+	open := s.MatchBackward(closeIdx)
+	if open < 0 {
+		return false
+	}
+	if f.shouldSkip(s, open, closeIdx) {
+		return false
+	}
+	return f.transform(s, open, closeIdx)
+}
+
+// shouldSkip mirrors LineLengthFixer::shouldSkip: empty block, heredoc, a keyed
+// "=>" array, or any inner comment.
+func (f LineLength) shouldSkip(s *tokens.Stream, open, closeIdx int) bool {
+	if closeIdx-open <= 1 {
+		return true
+	}
+	if nx := sigNext(s, open); nx >= 0 && nx < closeIdx && llIsHeredoc(s.At(nx)) {
+		return true
+	}
+	for j := open + 1; j < closeIdx; j++ {
+		t := s.At(j)
+		if t.Kind == token.Punct && t.Value == "=>" {
+			return true
+		}
+		if k := t.Kind; k == token.Comment || k == token.DocComment {
+			return true
+		}
+	}
+	return false
+}
+
+// transform decides break vs inline the way LineLengthTransformer does.
+func (f LineLength) transform(s *tokens.Stream, open, closeIdx int) bool {
+	if llHasPromotedProperty(s, open, closeIdx) {
+		return false
+	}
+	if f.firstLineLength(s, open) > f.lineLength {
+		if f.breakLongLines {
+			return f.breakItems(s, open, closeIdx)
+		}
+		return false
+	}
+	if f.fullLineLength(s, open, closeIdx) > f.lineLength {
+		return false
+	}
+	if !f.inlineShortLine {
+		return false
+	}
+	return f.inlineItems(s, open, closeIdx)
+}
+
+// firstLineLength renders the length of the physical line the opener sits on,
+// from line start through the first newline after the opener (ports
+// FirstLineLengthResolver + LineLengthAndPositionFactory).
+func (f LineLength) firstLineLength(s *tokens.Stream, open int) int {
+	length := 0
+	cp := open
+	for cp >= 0 {
+		t := s.At(cp)
+		if llIsNewlineOrOpenTag(t) {
+			break
+		}
+		c := t.Value
+		if nl := strings.LastIndexByte(c, '\n'); nl >= 0 {
+			length += len(c) - nl - 1
+			cp--
+			break
+		}
+		length += len(c)
+		cp--
+	}
+	if cp >= 0 && cp < s.Len() {
+		c := s.At(cp).Value
+		length += len(c) - strings.Count(c, "\n")
+	}
+	for cp2 := open + 1; cp2 < s.Len(); cp2++ {
+		t := s.At(cp2)
+		if llIsEndOfArgsLine(t) {
+			break
+		}
+		c := t.Value
+		if nl := strings.IndexByte(c, '\n'); nl >= 0 {
+			length += nl
+			break
+		}
+		length += len(c)
+	}
+	return length
+}
+
+// fullLineLength is the length the block would have inlined onto one line, with
+// each whitespace run counted as one char and the multi-line prefix/suffix
+// counted verbatim (ports LineLengthResolver::getLengthFromStartEnd).
+func (f LineLength) fullLineLength(s *tokens.Stream, open, closeIdx int) int {
+	length := 0
+	p := open
+	for p >= 0 {
+		if llIsNewlineOrOpenTag(s.At(p)) {
+			break
+		}
+		length += len(s.At(p).Value)
+		p--
+	}
+	if p >= 0 {
+		length += len(s.At(p).Value)
+	}
+	for j := open; j < closeIdx && j < s.Len(); j++ {
+		if s.At(j).Kind == token.Whitespace {
+			length++
+		} else {
+			length += len(s.At(j).Value)
+		}
+	}
+	for e := closeIdx; e < s.Len(); e++ {
+		c := s.At(e).Value
+		if len(c) > 0 && c[0] == '\n' {
+			break
+		}
+		length += len(c)
+	}
+	return length
+}
+
+// breakItems puts each item on its own line, the opener line unchanged and the
+// closer on its own line (ports TokensNewliner::breakItems with LineKind::CALLS).
+func (f LineLength) breakItems(s *tokens.Stream, open, closeIdx int) bool {
+	level := llIndentLevel(s, open)
+	itemWS := llEnding + strings.Repeat(llIndent, level+1)
+	closeWS := llEnding + strings.Repeat(llIndent, level)
+	// closer first
+	changed := f.breakCloser(s, open, closeIdx, itemWS, closeWS)
+	// commas, bottom-up
+	for i := closeIdx - 1; i >= open+1; i-- {
+		i = llSkipBlocksReversed(s, i)
+		if i < 0 {
+			break
+		}
+		if s.At(i).Kind != token.Punct || s.At(i).Value != "," {
+			continue
+		}
+		if llIsLastItem(s, i) || llIsFollowedByComment(s, i) || llIsFollowedByEmptyLine(s, i) {
+			continue
+		}
+		if llEnsureWS(s, i+1, 0, itemWS) {
+			changed = true
+		}
+	}
+	// opener last
+	if f.breakOpener(s, open, itemWS) {
+		changed = true
+	}
+	return changed
+}
+
+// breakCloser inserts the newline before the closer; for a non-method call that
+// ends with "])" it breaks before the "]" instead (shouldAddNewlineEarlier).
+func (f LineLength) breakCloser(s *tokens.Stream, open, closeIdx int, itemWS, closeWS string) bool {
+	isMethodCall := llIsMethodCall(s, open)
+	prev := closeIdx - 1
+	// TokenFinder resolves "previousPreviousToken" to endIndex-3, not endIndex-2
+	pp := closeIdx - 3
+	// method([...]) breaks before the "]"; method([]) and method([[...]]) do not
+	if !isMethodCall && prev > open && llIsArrayCloseTok(s, prev) && !llClosesEmptyArray(s, prev) {
+		if pp > open && !llIsArrayEdgeTok(s, pp) {
+			return llEnsureWS(s, prev, 0, itemWS)
+		}
+	}
+	return llEnsureWS(s, closeIdx-1, 1, closeWS)
+}
+
+// breakOpener inserts the newline after the opener.
+func (f LineLength) breakOpener(s *tokens.Stream, open int, itemWS string) bool {
+	next := open + 1
+	if next < s.Len() && s.At(next).Kind == token.Whitespace {
+		return llEnsureWS(s, next, 0, itemWS)
+	}
+	if next < s.Len() && llIsArrayOpenTok(s, next) && !llIsMethodCall(s, open) {
+		return llEnsureWS(s, open+1, 1, itemWS)
+	}
+	return llEnsureWS(s, open, 1, itemWS)
+}
+
+// inlineItems collapses a multiline block onto one line, preserving nested
+// curly/array blocks and the whitespace around a heredoc (ports TokensInliner).
+func (f LineLength) inlineItems(s *tokens.Stream, open, closeIdx int) bool {
+	changed := false
+	for i := open + 1; i < closeIdx; i++ {
+		t := s.At(i)
+		if t.Kind == token.Punct && t.Value == "{" {
+			if m := s.MatchForward(i); m > i {
+				i = m
+			}
+			continue
+		}
+		if llIsArrayOpenTok(s, i) {
+			if m := s.MatchForward(i); m > i {
+				i = m
+			}
+			continue
+		}
+		if t.Kind != token.Whitespace {
+			continue
+		}
+		prev := s.At(i - 1)
+		next := s.At(i + 1)
+		if llIsHeredoc(prev) {
+			continue
+		}
+		if (prev.Kind == token.Punct && (prev.Value == "(" || prev.Value == "[")) ||
+			(next.Kind == token.Punct && (next.Value == ")" || next.Value == "]")) {
+			if t.Value != "" {
+				s.SetValue(i, "")
+				changed = true
+			}
+		} else if t.Value != " " {
+			s.SetValue(i, " ")
+			changed = true
+		}
+	}
+	return changed
+}
+
+// processAttribute expands a too-long "#[Attr(...)]" (one comment token) onto
+// multiple lines by rewriting its text.
+func (f LineLength) processAttribute(s *tokens.Stream, pos int) bool {
+	if !f.breakLongLines {
+		return false
+	}
+	v := s.At(pos).Value
+	if !strings.Contains(v, "(") {
+		return false
+	}
+	attrFirst := v
+	if before, _, ok := strings.Cut(v, "\n"); ok {
+		attrFirst = before
+	}
+	if f.attributeColumn(s, pos)+len(attrFirst) <= f.lineLength {
+		return false
+	}
+	base := lineIndent(s, pos)
+	nv := llExpandAttribute(v, base)
+	if nv == "" || nv == v {
+		return false
+	}
+	s.SetValue(pos, nv)
+	return true
+}
+
+// attributeColumn is the rendered column at which the attribute token starts.
+func (f LineLength) attributeColumn(s *tokens.Stream, idx int) int {
+	length := 0
+	cp := idx - 1
+	for cp >= 0 {
+		t := s.At(cp)
+		if llIsNewlineOrOpenTag(t) {
+			break
+		}
+		c := t.Value
+		if nl := strings.LastIndexByte(c, '\n'); nl >= 0 {
+			length += len(c) - nl - 1
+			cp--
+			break
+		}
+		length += len(c)
+		cp--
+	}
+	if cp >= 0 && cp < s.Len() {
+		c := s.At(cp).Value
+		length += len(c) - strings.Count(c, "\n")
+	}
+	return length
+}
+
+// llExpandAttribute rewrites the first top-level "(...)" of an attribute so each
+// argument sits on its own line, indented one level past base.
+func llExpandAttribute(v, base string) string {
+	open := strings.IndexByte(v, '(')
+	if open < 0 {
+		return ""
+	}
+	closeIdx := matchParen(v, open)
+	if closeIdx < 0 {
+		return ""
+	}
+	args, _ := splitTopLevelArgs(v[open+1 : closeIdx])
+	if len(args) == 0 {
+		return ""
+	}
+	var b strings.Builder
+	b.WriteString(v[:open+1])
+	for _, arg := range args {
+		b.WriteString("\n")
+		b.WriteString(base)
+		b.WriteString(llIndent)
+		b.WriteString(arg)
+		b.WriteByte(',')
+	}
+	// drop the trailing comma of the last item (attributes keep none here)
+	out := strings.TrimSuffix(b.String(), ",")
+	out += "\n" + base + v[closeIdx:]
+	return out
+}
+
+// llEnsureWS ports PhpCsFixer Tokens::ensureWhitespaceAtIndex: overwrite an
+// existing whitespace slot in place (ignoring offset), otherwise insert a new
+// whitespace token at index+offset.
+func llEnsureWS(s *tokens.Stream, index, offset int, ws string) bool {
+	if index >= 0 && index < s.Len() && s.At(index).Kind == token.Whitespace {
+		if ws == "" {
+			s.SetValue(index, "")
+			return true
+		}
+		if s.At(index).Value != ws {
+			s.SetValue(index, ws)
+			return true
+		}
+		return false
+	}
+	at := index + offset
+	if at < 0 || at > s.Len() {
+		return false
+	}
+	s.InsertAt(at, token.Token{Kind: token.Whitespace, Value: ws})
+	return true
+}
+
+// llIndentLevel counts the indent units on the opener's physical line (ports
+// IndentDetector::detectOnPosition).
+func llIndentLevel(s *tokens.Stream, idx int) int {
+	for j := idx; j >= 1; j-- {
+		c := s.At(j).Value
+		if nl := strings.LastIndexByte(c, '\n'); nl >= 0 {
+			return strings.Count(c[nl+1:], llIndent)
+		}
+	}
+	return 0
+}
+
+func llSkipBlocksReversed(s *tokens.Stream, i int) int {
+	if i < 0 || i >= s.Len() {
+		return i
+	}
+	t := s.At(i)
+	if t.Kind == token.Punct && (t.Value == ")" || t.Value == "]") {
+		if open := s.MatchBackward(i); open >= 0 {
+			return open
+		}
+	}
+	return i
+}
+
+func llIsLastItem(s *tokens.Stream, i int) bool {
+	return i+1 < s.Len() && strings.Contains(s.At(i+1).Value, "\n")
+}
+
+func llIsFollowedByComment(s *tokens.Stream, i int) bool {
+	if i+2 >= s.Len() {
+		return false
+	}
+	if isComment(s.At(i + 2)) {
+		return true
+	}
+	return s.At(i+1).Kind == token.Whitespace && s.At(i+1).Value == " " && isComment(s.At(i+2))
+}
+
+func llIsFollowedByEmptyLine(s *tokens.Stream, i int) bool {
+	if i+1 >= s.Len() {
+		return false
+	}
+	t := s.At(i + 1)
+	return t.Kind == token.Whitespace && strings.Count(t.Value, "\n") >= 2
+}
+
+// llIsMethodCall reports whether the block opener is preceded by "->"/"?->".
+func llIsMethodCall(s *tokens.Stream, open int) bool {
+	name := sigPrev(s, open)
+	if name < 0 {
+		return false
+	}
+	arrow := sigPrev(s, name)
+	if arrow < 0 {
+		return false
+	}
+	t := s.At(arrow)
+	return t.Kind == token.Punct && (t.Value == "->" || t.Value == "?->")
+}
+
+func llHasPromotedProperty(s *tokens.Stream, open, closeIdx int) bool {
+	for j := open + 1; j < closeIdx; j++ {
+		t := s.At(j)
+		if t.Kind == token.Keyword {
+			switch strings.ToLower(t.Value) {
+			case "public", "protected", "private", "readonly":
+				return true
+			}
+		}
+	}
+	return false
+}
+
+func llIsAttribute(t token.Token) bool {
+	return t.Kind == token.Comment && strings.HasPrefix(t.Value, "#[")
+}
+
+func llIsHeredoc(t token.Token) bool {
+	return t.Kind == token.String && strings.HasPrefix(t.Value, "<<<")
+}
+
+func llIsClosureUse(s *tokens.Stream, pos int) bool {
+	nx := sigNext(s, pos)
+	return nx >= 0 && s.At(nx).Kind == token.Punct && s.At(nx).Value == "("
+}
+
+func llIsArrayClose(s *tokens.Stream, closeIdx int) bool {
+	open := s.MatchBackward(closeIdx)
+	return open >= 0 && isArrayLiteralOpen(s, open)
+}
+
+func llIsArrayOpenTok(s *tokens.Stream, i int) bool {
+	t := s.At(i)
+	return t.Kind == token.Punct && t.Value == "[" && isArrayLiteralOpen(s, i)
+}
+
+func llIsArrayCloseTok(s *tokens.Stream, i int) bool {
+	t := s.At(i)
+	if t.Kind != token.Punct || t.Value != "]" {
+		return false
+	}
+	return llIsArrayClose(s, i)
+}
+
+func llIsArrayEdgeTok(s *tokens.Stream, i int) bool {
+	t := s.At(i)
+	return t.Kind == token.Punct && (t.Value == "[" || t.Value == "]")
+}
+
+// llClosesEmptyArray reports whether the "]" at i closes an empty "[]".
+func llClosesEmptyArray(s *tokens.Stream, i int) bool {
+	open := s.MatchBackward(i)
+	return open >= 0 && sigNext(s, open) == i
+}
+
+func llIsNewlineOrOpenTag(t token.Token) bool {
+	return t.Kind == token.OpenTag || (len(t.Value) > 0 && t.Value[0] == '\n')
+}
+
+func llIsEndOfArgsLine(t token.Token) bool {
+	if len(t.Value) > 0 && t.Value[0] == '\n' {
+		return true
+	}
+	return t.Kind == token.Keyword && strings.EqualFold(t.Value, "use")
+}
diff --git a/blink/internal/fixer/rules/line_length_test.go b/blink/internal/fixer/rules/line_length_test.go
new file mode 100644
index 0000000000..ce55f29b59
--- /dev/null
+++ b/blink/internal/fixer/rules/line_length_test.go
@@ -0,0 +1,62 @@
+package rules
+
+import "testing"
+
+// TestLineLength ports Symplify LineLengthFixerTest: every Fixture/*.php.inc
+// case, before -> after (or unchanged for the correct/skip fixtures).
+func TestLineLength(t *testing.T) {
+	t.Parallel()
+	cases := []struct {
+		name string
+		in   string
+		want string
+	}{
+		{"break_attribute", "<?php\n\n#[SomeObject($someLongArgument, $someLongArgument, $anotherLongArgument, $anotherLongArgument, $passTheLimit, $passTheLimit, $passTheLimit)]\nclass SomeClass\n{\n\n}\n\n?>\n", "<?php\n\n#[SomeObject(\n    $someLongArgument,\n    $someLongArgument,\n    $anotherLongArgument,\n    $anotherLongArgument,\n    $passTheLimit,\n    $passTheLimit,\n    $passTheLimit\n)]\nclass SomeClass\n{\n\n}\n\n?>\n"},
+		{"break_method_call_with_array", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        $this->someFunction(['foo', 'bar'], ['fooooooooooooooooo', 'baaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaar']);\n    }\n}\n\n?>\n", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        $this->someFunction(\n            ['foo', 'bar'],\n            ['fooooooooooooooooo', 'baaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaar']\n        );\n    }\n}\n\n?>\n"},
+		{"break_new_some_object", "<?php\n\n$someObject = new SomeObject($someLongArgument, $someLongArgument, $anotherLongArgument, $anotherLongArgument, $passTheLimit);\n\n$someObject = new SomeObject($someLongArgument, $someLongArgument, $anotherLongArgument, $anotherLongArgument, $limits);\n\n?>\n", "<?php\n\n$someObject = new SomeObject(\n    $someLongArgument,\n    $someLongArgument,\n    $anotherLongArgument,\n    $anotherLongArgument,\n    $passTheLimit\n);\n\n$someObject = new SomeObject($someLongArgument, $someLongArgument, $anotherLongArgument, $anotherLongArgument, $limits);\n\n?>\n"},
+		{"correct", "<?php\n\ndeclare(strict_types=1);\n\nclass SomeFixer\n{\n    public function getDefinition(): FixerDefinitionInterface\n    {\n        return new FixerDefinition('Class should have suffix by parent class/interface', [\n            new CodeSample(\n                <<<CODE\n<?php\n\nclass SomeClass extends Command\n{\n}\nCODE\n            ),\n        ]);\n    }\n}\n", "<?php\n\ndeclare(strict_types=1);\n\nclass SomeFixer\n{\n    public function getDefinition(): FixerDefinitionInterface\n    {\n        return new FixerDefinition('Class should have suffix by parent class/interface', [\n            new CodeSample(\n                <<<CODE\n<?php\n\nclass SomeClass extends Command\n{\n}\nCODE\n            ),\n        ]);\n    }\n}\n"},
+		{"correct10", "<?php\n\ndeclare(strict_types=1);\n\nnamespace Pehapkari\\Website\\Tests\\Posts\\Year2016\\EventDispatcher;\n\nfinal class EventDispatchingTest\n{\n    public function test(): void\n    {\n        $this->someMethod(10, function () {\n           return 5;\n        });\n    }\n}\n", "<?php\n\ndeclare(strict_types=1);\n\nnamespace Pehapkari\\Website\\Tests\\Posts\\Year2016\\EventDispatcher;\n\nfinal class EventDispatchingTest\n{\n    public function test(): void\n    {\n        $this->someMethod(10, function () {\n           return 5;\n        });\n    }\n}\n"},
+		{"correct11", "<?php\n\ndeclare(strict_types=1);\n\nclass ListDataProvider\n{\n    public function createListDataDTOForGroup(PlayerGroup $group): void\n    {\n        $group->prefixDefaultPaginatorOptions(\"group{$group->getId()}\");\n    }\n}\n", "<?php\n\ndeclare(strict_types=1);\n\nclass ListDataProvider\n{\n    public function createListDataDTOForGroup(PlayerGroup $group): void\n    {\n        $group->prefixDefaultPaginatorOptions(\"group{$group->getId()}\");\n    }\n}\n"},
+		{"correct12", "<?php\n\ndeclare(strict_types=1);\n\nclass A\n{\n    public function __construct($orWhere)\n    {\n        $sql = 'UPDATE answer \n                SET score = score + __points__ * (\n                        SELECT COUNT(aa.id) \n                        FROM answer_to_answer_definition aa \n                        WHERE answer.id = aa.pickX_answer_id AND ('.implode(' OR ', $orWhere).')\n                ).';\n    }\n}", "<?php\n\ndeclare(strict_types=1);\n\nclass A\n{\n    public function __construct($orWhere)\n    {\n        $sql = 'UPDATE answer \n                SET score = score + __points__ * (\n                        SELECT COUNT(aa.id) \n                        FROM answer_to_answer_definition aa \n                        WHERE answer.id = aa.pickX_answer_id AND ('.implode(' OR ', $orWhere).')\n                ).';\n    }\n}"},
+		{"correct2", "<?php\n\ndeclare(strict_types=1);\n\nnamespace Symplify\\BetterPhpDocParser;\n\nuse phpDocumentor\\Reflection\\DocBlock;\nuse phpDocumentor\\Reflection\\DocBlock\\Serializer;\nuse Symplify\\BetterPhpDocParser\\Renderer\\OriginalSpacingCompleter;\nuse Symplify\\PackageBuilder\\Reflection\\PrivatesCaller;\n\nfinal class FixedSerializer extends Serializer\n{\n    public function getDocComment(DocBlock $docBlock): string\n    {\n        // tags\n        $comment = (new OriginalSpacingCompleter())->completeTagSpaces($comment, $this->originalContent);\n    }\n}\n", "<?php\n\ndeclare(strict_types=1);\n\nnamespace Symplify\\BetterPhpDocParser;\n\nuse phpDocumentor\\Reflection\\DocBlock;\nuse phpDocumentor\\Reflection\\DocBlock\\Serializer;\nuse Symplify\\BetterPhpDocParser\\Renderer\\OriginalSpacingCompleter;\nuse Symplify\\PackageBuilder\\Reflection\\PrivatesCaller;\n\nfinal class FixedSerializer extends Serializer\n{\n    public function getDocComment(DocBlock $docBlock): string\n    {\n        // tags\n        $comment = (new OriginalSpacingCompleter())->completeTagSpaces($comment, $this->originalContent);\n    }\n}\n"},
+		{"correct3", "<?php\n\ndeclare(strict_types=1);\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        $this->assertSame([\n            'one' => 1,\n            'two' => 2,\n        ], $parameterProvider->provide());\n    }\n}\n", "<?php\n\ndeclare(strict_types=1);\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        $this->assertSame([\n            'one' => 1,\n            'two' => 2,\n        ], $parameterProvider->provide());\n    }\n}\n"},
+		{"correct4", "<?php\n\ndeclare(strict_types=1);\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        $this->assertSame([\n            'one' => 1,\n            'two' => 2,\n        ], $parameterProvider->provide());\n\n        $this->easyCodingStandardStyle->error(sprintf(\n            'Found %d error%s.',\n            $errorCount,\n            $errorCount === 1 ? '' : 's'\n        ));\n    }\n}\n", "<?php\n\ndeclare(strict_types=1);\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        $this->assertSame([\n            'one' => 1,\n            'two' => 2,\n        ], $parameterProvider->provide());\n\n        $this->easyCodingStandardStyle->error(sprintf(\n            'Found %d error%s.',\n            $errorCount,\n            $errorCount === 1 ? '' : 's'\n        ));\n    }\n}\n"},
+		{"correct5", "<?php\n\ndeclare(strict_types=1);\n\nfinal class BasicallyAnyClass\n{\n    private function shouldSkip()\n    {\n    }\n\n    private function removeTagForMissingParameters(MethodWrapper $methodWrapper, DocBlockWrapper $docBlockWrapper): void\n    {\n    }\n}\n", "<?php\n\ndeclare(strict_types=1);\n\nfinal class BasicallyAnyClass\n{\n    private function shouldSkip()\n    {\n    }\n\n    private function removeTagForMissingParameters(MethodWrapper $methodWrapper, DocBlockWrapper $docBlockWrapper): void\n    {\n    }\n}\n"},
+		{"correct6", "<?php\n\ndeclare(strict_types=1);\n\n$array = [1 => 'looooooooooooooooooooooooooongArraaaaaaaaaaay', 2 => 'looooooooooooooooooooooooooooongArraaaaaaaaaaay'];\n", "<?php\n\ndeclare(strict_types=1);\n\n$array = [1 => 'looooooooooooooooooooooooooongArraaaaaaaaaaay', 2 => 'looooooooooooooooooooooooooooongArraaaaaaaaaaay'];\n"},
+		{"correct7", "<?php\n\ndeclare(strict_types=1);\n\n$array = [\n    'looooooooooooooooooooooooooooooooongArraaaaaaaaaaay',\n    'looooooooooooooooooooooooooooooooongArraaaaaaaaaaaaaaay'\n];\n", "<?php\n\ndeclare(strict_types=1);\n\n$array = [\n    'looooooooooooooooooooooooooooooooongArraaaaaaaaaaay',\n    'looooooooooooooooooooooooooooooooongArraaaaaaaaaaaaaaay'\n];\n"},
+		{"correct8", "<?php\n\ndeclare(strict_types=1);\n\nnamespace Pehapkari\\Website\\Tests\\Posts\\Year2016\\EventDispatcher;\n\nfinal class EventDispatchingTest\n{\n    public function test(): void\n    {\n        $notifyMeOnVideoPublishedEventSubscriber = new NotifyMeOnVideoPublishedEventSubscriber;\n        $eventDispatcher->addSubscriber($notifyMeOnVideoPublishedEventSubscriber);\n    }\n}\n", "<?php\n\ndeclare(strict_types=1);\n\nnamespace Pehapkari\\Website\\Tests\\Posts\\Year2016\\EventDispatcher;\n\nfinal class EventDispatchingTest\n{\n    public function test(): void\n    {\n        $notifyMeOnVideoPublishedEventSubscriber = new NotifyMeOnVideoPublishedEventSubscriber;\n        $eventDispatcher->addSubscriber($notifyMeOnVideoPublishedEventSubscriber);\n    }\n}\n"},
+		{"correct9", "<?php\n\ndeclare(strict_types=1);\n\nnamespace Pehapkari\\Website\\Tests\\Posts\\Year2016\\EventDispatcher;\n\nfinal class EventDispatchingTest\n{\n    public function test(): void\n    {\n        $this->someMethod(function () {\n           return 5;\n        });\n    }\n}\n", "<?php\n\ndeclare(strict_types=1);\n\nnamespace Pehapkari\\Website\\Tests\\Posts\\Year2016\\EventDispatcher;\n\nfinal class EventDispatchingTest\n{\n    public function test(): void\n    {\n        $this->someMethod(function () {\n           return 5;\n        });\n    }\n}\n"},
+		{"correct_heredoc", "<?php\n\ndeclare(strict_types=1);\n\nnamespace Rector\\Php\\Rector\\FuncCall;\n\nfinal class SensitiveDefineRector\n{\n    public function getNowDoc()\n    {\n        return new RectorDefinition(\n            [new CodeSample(\n                <<<'PHP'\ndefine('FOO', 42, true); \nPHP\n            )]\n        );\n    }\n\n    public function getHereDoc()\n    {\n        return new RectorDefinition(\n            [new CodeSample(\n                <<<PHP\ndefine('FOO', 42, true); \nPHP\n            )]\n        );\n    }\n}\n\n?>\n", "<?php\n\ndeclare(strict_types=1);\n\nnamespace Rector\\Php\\Rector\\FuncCall;\n\nfinal class SensitiveDefineRector\n{\n    public function getNowDoc()\n    {\n        return new RectorDefinition([new CodeSample(<<<'PHP'\ndefine('FOO', 42, true); \nPHP\n            )]);\n    }\n\n    public function getHereDoc()\n    {\n        return new RectorDefinition([new CodeSample(<<<PHP\ndefine('FOO', 42, true); \nPHP\n            )]);\n    }\n}\n\n?>\n"},
+		{"inline_new_arguments", "<?php\n\n$someObject = new SomeClass(\n    $someLongArgument,\n    $anotherLongArgument\n);\n\n?>\n", "<?php\n\n$someObject = new SomeClass($someLongArgument, $anotherLongArgument);\n\n?>\n"},
+		{"method_call_arguments", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        $this->someFunctionCall($someLoooooooooooooongArgument, $anotherLoooooooooooooooooooooooooooooooooooongArgumentTo);\n\n        $this->someOtherFunctionCall(\n            $superShort,\n            $anotherShort\n        );\n    }\n}\n\n?>\n", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        $this->someFunctionCall(\n            $someLoooooooooooooongArgument,\n            $anotherLoooooooooooooooooooooooooooooooooooongArgumentTo\n        );\n\n        $this->someOtherFunctionCall($superShort, $anotherShort);\n    }\n}\n\n?>\n"},
+		{"skip_correct_file", "<?php\n\nclass Foo\n{\n    public function render(AbstractFile $file, array $parameters): string\n    {\n        return $this->codeBlocksProtector->protectContentFromCallback($file->getContent(), function (string $content) use (\n            $file,\n            $parameters\n        ) {\n            return '...';\n        });\n    }\n}\n", "<?php\n\nclass Foo\n{\n    public function render(AbstractFile $file, array $parameters): string\n    {\n        return $this->codeBlocksProtector->protectContentFromCallback($file->getContent(), function (string $content) use (\n            $file,\n            $parameters\n        ) {\n            return '...';\n        });\n    }\n}\n"},
+		{"skip_property_promotion", "<?php\n\ndeclare(strict_types=1);\n\nnamespace Symplify\\CodingStandard\\Tests\\Fixer\\LineLength\\LineLengthFixer\\Fixture;\n\nfinal class SkipPropertyPromotion\n{\n    public function __construct(\n        public int $value,\n        public int $name\n    ) {\n    }\n}\n", "<?php\n\ndeclare(strict_types=1);\n\nnamespace Symplify\\CodingStandard\\Tests\\Fixer\\LineLength\\LineLengthFixer\\Fixture;\n\nfinal class SkipPropertyPromotion\n{\n    public function __construct(\n        public int $value,\n        public int $name\n    ) {\n    }\n}\n"},
+		{"skip_valid_attribute", "<?php\n\ndeclare(strict_types=1);\n\nnamespace Symplify\\CodingStandard\\Tests\\Fixer\\LineLength\\LineLengthFixer\\Fixture;\n\nuse Doctrine\\ORM\\Mapping\\Entity;\n\n#[Entity]\nclass SkipValidAttribute\n{\n}\n", "<?php\n\ndeclare(strict_types=1);\n\nnamespace Symplify\\CodingStandard\\Tests\\Fixer\\LineLength\\LineLengthFixer\\Fixture;\n\nuse Doctrine\\ORM\\Mapping\\Entity;\n\n#[Entity]\nclass SkipValidAttribute\n{\n}\n"},
+		{"use_arguments", "<?php\n\nclass SomeClass\n{\n    public function someFunction() {\n        $someLongVariableName = $anotherLongVariableName = $yetAnotherLongVariableName = $andOneMoreLongVariableName = null;\n\n        return function () use ($someLongVariableName, $anotherLongVariableName, $yetAnotherLongVariableName, $andOneMoreLongVariableName) {};\n\n        return function ($argument) use ($someLongVariableName, $anotherLongVariableName, $yetAnotherLongVariableName, $andOneMoreLongVariableName) {};\n    }\n}\n\n?>\n", "<?php\n\nclass SomeClass\n{\n    public function someFunction() {\n        $someLongVariableName = $anotherLongVariableName = $yetAnotherLongVariableName = $andOneMoreLongVariableName = null;\n\n        return function () use (\n            $someLongVariableName,\n            $anotherLongVariableName,\n            $yetAnotherLongVariableName,\n            $andOneMoreLongVariableName\n        ) {};\n\n        return function ($argument) use (\n            $someLongVariableName,\n            $anotherLongVariableName,\n            $yetAnotherLongVariableName,\n            $andOneMoreLongVariableName\n        ) {};\n    }\n}\n\n?>\n"},
+		{"wrong", "<?php\n\nclass SomeClass\n{\n    public function someFunction(SomeLongArgument $someLongArgument, AnotherLongArgument $anotherLongArgument, $passTheLimit)\n    {\n    }\n\n    public function someFineFunction(SomeLongArgument $someLongArgumentFine, AnotherLongArgument $anotherLongArgumentTo)\n    {\n    }\n\n    public function someLongFunction(SomeLongArgument $someLongArgumentNotFine, AnotherLongArgument $anotherLongArgument)\n    {\n    }\n}\n\n?>\n", "<?php\n\nclass SomeClass\n{\n    public function someFunction(\n        SomeLongArgument $someLongArgument,\n        AnotherLongArgument $anotherLongArgument,\n        $passTheLimit\n    )\n    {\n    }\n\n    public function someFineFunction(SomeLongArgument $someLongArgumentFine, AnotherLongArgument $anotherLongArgumentTo)\n    {\n    }\n\n    public function someLongFunction(\n        SomeLongArgument $someLongArgumentNotFine,\n        AnotherLongArgument $anotherLongArgument\n    )\n    {\n    }\n}\n\n?>\n"},
+		{"wrong10", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        if (something) {\n            throw new SomeException(sprintf('SuperLongMethod', 'SuperLongMethod', 'SuperLongMethod', 'SuperLongMethod'));\n        }\n    }\n}\n\n?>\n", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        if (something) {\n            throw new SomeException(sprintf(\n                'SuperLongMethod',\n                'SuperLongMethod',\n                'SuperLongMethod',\n                'SuperLongMethod'\n            ));\n        }\n    }\n}\n\n?>\n"},
+		{"wrong11", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        return $this->assertSame(\n            $parameterProvider->provide(),\n            [\n                'one' => 1,\n                'two' => 2,\n            ]\n        );\n    }\n}\n\n?>\n", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        return $this->assertSame($parameterProvider->provide(), [\n                'one' => 1,\n                'two' => 2,\n            ]);\n    }\n}\n\n?>\n"},
+		{"wrong12", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        yield \\Namespace1\\Namespace2\\Namespace3\\Namespace4\\Namespace5\\Namespace6\\MyClass::someLongStaticMethod(self::getArgument($parameter), self::getArgument($parameter));\n    }\n}\n\n?>\n", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        yield \\Namespace1\\Namespace2\\Namespace3\\Namespace4\\Namespace5\\Namespace6\\MyClass::someLongStaticMethod(\n            self::getArgument($parameter),\n            self::getArgument($parameter)\n        );\n    }\n}\n\n?>\n"},
+		{"wrong13", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        yield CreateRoleCommand::fromValueObjects(RoleId::fromString(self::REALTY_ECONOMY), 'Předpisář', ScopeId::fromString(ScopeList::REALTY), PrivilegeList::REALTY_PRIVILEGES);\n    }\n}\n\n?>\n", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        yield CreateRoleCommand::fromValueObjects(\n            RoleId::fromString(self::REALTY_ECONOMY),\n            'Předpisář',\n            ScopeId::fromString(ScopeList::REALTY),\n            PrivilegeList::REALTY_PRIVILEGES\n        );\n    }\n}\n\n?>\n"},
+		{"wrong14", "<?php\n\n$array = ['looooooooooooooooooooooooooooooooongArraaaaaaaaaaay', 'looooooooooooooooooooooooooooooooongArraaaaaaaaaaaaaaay'];\n\n?>\n", "<?php\n\n$array = [\n    'looooooooooooooooooooooooooooooooongArraaaaaaaaaaay',\n    'looooooooooooooooooooooooooooooooongArraaaaaaaaaaaaaaay'\n];\n\n?>\n"},
+		{"wrong15", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        $result = ['loooooooooooooooooooooooooooooooongArraaaaaaaaaaay', 'loooooooooooooooooooooongArraaaaaaaaaaaaaaay'];\n    }\n}\n\n?>\n", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        $result = [\n            'loooooooooooooooooooooooooooooooongArraaaaaaaaaaay',\n            'loooooooooooooooooooooongArraaaaaaaaaaaaaaay'\n        ];\n    }\n}\n\n?>\n"},
+		{"wrong16", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        return [\n            # form chain method calls\n            [__DIR__ . '/NestedMethodCallSource/FormChainMethodCalls.php.inc', 0, 'addRule', ['Stub_Nette\\Forms\\Rules']],\n        ];\n    }\n}\n\n?>\n", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        return [\n            # form chain method calls\n            [\n                __DIR__ . '/NestedMethodCallSource/FormChainMethodCalls.php.inc',\n                0,\n                'addRule',\n                ['Stub_Nette\\Forms\\Rules']\n            ],\n        ];\n    }\n}\n\n?>\n"},
+		{"wrong18", "<?php\n\nclass A\n{\n    public function __construct($orWhere)\n    {\n        $sql = 'UPDATE answer \n                SET score = score + __points__ * (\n                        SELECT COUNT(aa.id) \n                        FROM answer_to_answer_definition aa \n                        WHERE answer.id = aa.pickX_answer_id AND ('.implode(' OR ', $orWhere).') AND answer.stuff IN (1, 2, 3, 4)\n                ).';\n    }\n}\n?>\n", "<?php\n\nclass A\n{\n    public function __construct($orWhere)\n    {\n        $sql = 'UPDATE answer \n                SET score = score + __points__ * (\n                        SELECT COUNT(aa.id) \n                        FROM answer_to_answer_definition aa \n                        WHERE answer.id = aa.pickX_answer_id AND ('.implode(\n                            ' OR ',\n                            $orWhere\n                        ).') AND answer.stuff IN (1, 2, 3, 4)\n                ).';\n    }\n}\n?>\n"},
+		{"wrong2", "<?php\n\nclass SomeClass\n{\n    public function someFunction(\n        SomeLongArgument $someLongArgument,\n        AnotherLongArgument $anotherLongArgument\n    ) {\n    }\n\n    public function someOtherFunction(\n        SomeLongArgument $someLongArgument,\n        AnotherLongArgument $anotherLongArgument,\n        LongOneArgument $longOneArgument\n    ) {\n    }\n}\n\n?>\n", "<?php\n\nclass SomeClass\n{\n    public function someFunction(SomeLongArgument $someLongArgument, AnotherLongArgument $anotherLongArgument) {\n    }\n\n    public function someOtherFunction(\n        SomeLongArgument $someLongArgument,\n        AnotherLongArgument $anotherLongArgument,\n        LongOneArgument $longOneArgument\n    ) {\n    }\n}\n\n?>\n"},
+		{"wrong8", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        $this->assertSame(['one' => 1, 'two' => 2, 'fooooooooooooooooooo' => 'baaaaaar'], $parameterProvider->provide());\n    }\n}\n\n?>\n", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        $this->assertSame(\n            ['one' => 1, 'two' => 2, 'fooooooooooooooooooo' => 'baaaaaar'],\n            $parameterProvider->provide()\n        );\n    }\n}\n\n?>\n"},
+		{"wrong9", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        foreach ($finder as $fileInfo) {\n            $processInput[] = sprintf('%s=%s', $fileInfo->getRelativePathname(), $newPackageDirectory . '/' . $fileInfo->getRelativePathname());\n        }\n    }\n}\n\n?>\n", "<?php\n\nclass SomeClass\n{\n    public function someFunction()\n    {\n        foreach ($finder as $fileInfo) {\n            $processInput[] = sprintf(\n                '%s=%s',\n                $fileInfo->getRelativePathname(),\n                $newPackageDirectory . '/' . $fileInfo->getRelativePathname()\n            );\n        }\n    }\n}\n\n?>\n"},
+		{"wrong_array_bracket_position_in_arg_list", "<?php\n\nclass Bar\n{\n    public function foo(string $arg1, string $arg2, string $arg3, string $arg4, string $arg5, string $arg6, array $arg7 = []): void\n    {\n        // Do stuff\n    }\n}\n?>\n", "<?php\n\nclass Bar\n{\n    public function foo(\n        string $arg1,\n        string $arg2,\n        string $arg3,\n        string $arg4,\n        string $arg5,\n        string $arg6,\n        array $arg7 = []\n    ): void\n    {\n        // Do stuff\n    }\n}\n?>\n"},
+	}
+
+	for _, tc := range cases {
+		t.Run(tc.name, func(t *testing.T) {
+			got, changed := apply(t, LineLength{}, tc.in)
+			if got != tc.want {
+				t.Fatalf("got:\n%q\nwant:\n%q", got, tc.want)
+			}
+			if wantChanged := tc.in != tc.want; changed != wantChanged {
+				t.Fatalf("changed=%v want=%v", changed, wantChanged)
+			}
+		})
+	}
+}
diff --git a/blink/internal/fixer/rules/priorities.go b/blink/internal/fixer/rules/priorities.go
index ecf165eece..bc2f23efc4 100644
--- a/blink/internal/fixer/rules/priorities.go
+++ b/blink/internal/fixer/rules/priorities.go
@@ -22,6 +22,7 @@ var fixerPriority = map[string]int{
 	"ArrayIndentationFixer":                    29,
 	"ArrayListItemNewlineFixer":                40,
 	"StandaloneLineInMultilineArrayFixer":      5,
+	"LineLengthFixer":                          5,
 	"ArraySyntaxFixer":                         37,
 	"AssignNullCoalescingToCoalesceEqualFixer": -1,
 	"AttributeBlockNoSpacesFixer":              0,
diff --git a/blink/internal/fixer/rules/registry.go b/blink/internal/fixer/rules/registry.go
index c35749fbb1..11b2207dd4 100644
--- a/blink/internal/fixer/rules/registry.go
+++ b/blink/internal/fixer/rules/registry.go
@@ -241,6 +241,7 @@ func StructuralFixers() []fixer.Fixer {
 		MethodChainingIndentation{},
 		ArrayListItemNewline{},
 		StandaloneLineInMultilineArray{},
+		LineLength{},
 		ArrayIndentation{},
 		NoExtraBlankLines{}, // after import removal, which can leave extra blanks
 	}