From fc8cd7101742f2f299de72252eac78f9a106a85c Mon Sep 17 00:00:00 2001
From: Tomas Votruba <tomas.vot@gmail.com>
Date: Tue, 6 Oct 2026 00:12:10 +0200
Subject: [PATCH] Process single-line docblocks in NoSuperfluousPhpdocTags

The fixer skipped single-line "/** ... */" docblocks, so a standalone
"/** @inheritDoc */" (or a single-line superfluous tag) was kept and later
expanded to multi-line instead of removed. php-cs-fixer processes both forms;
blink now does too, removing the tag and leaving "/** */" for NoEmptyPhpdoc.
---
 .../fixer/rules/no_superfluous_phpdoc_tags.go      |  2 +-
 .../no_superfluous_phpdoc_tags_config_test.go      | 14 ++++++++++++++
 2 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/blink/internal/fixer/rules/no_superfluous_phpdoc_tags.go b/blink/internal/fixer/rules/no_superfluous_phpdoc_tags.go
index fa99afb015..ddf69ef71a 100644
--- a/blink/internal/fixer/rules/no_superfluous_phpdoc_tags.go
+++ b/blink/internal/fixer/rules/no_superfluous_phpdoc_tags.go
@@ -91,7 +91,7 @@ func (f NoSuperfluousPhpdocTags) Fix(s *tokens.Stream) bool {
 			continue
 		}
 		d, ok := parseDoc(t.Value)
-		if !ok || d.single {
+		if !ok {
 			continue
 		}
 		docChanged := false
diff --git a/blink/internal/fixer/rules/no_superfluous_phpdoc_tags_config_test.go b/blink/internal/fixer/rules/no_superfluous_phpdoc_tags_config_test.go
index 3541651408..59bfc82531 100644
--- a/blink/internal/fixer/rules/no_superfluous_phpdoc_tags_config_test.go
+++ b/blink/internal/fixer/rules/no_superfluous_phpdoc_tags_config_test.go
@@ -26,6 +26,20 @@ func TestNoSuperfluousPhpdocTagsConfig(t *testing.T) {
 	if want := "<?php\nclass A {\n    /**\n     */\n    public function f() {}\n}\n"; !changed || got != want {
 		t.Fatalf("remove_inheritdoc: changed=%v got=%q want=%q", changed, got, want)
 	}
+
+	// a single-line docblock is processed too: remove_inheritdoc drops the tag,
+	// leaving "/** */" for NoEmptyPhpdoc to clear (php-cs-fixer behavior)
+	single := "<?php\nclass A {\n    /** @inheritDoc */\n    public function f() {}\n}\n"
+	got, changed = apply(t, rm.(fixerRule), single)
+	if want := "<?php\nclass A {\n    /** */\n    public function f() {}\n}\n"; !changed || got != want {
+		t.Fatalf("single-line remove_inheritdoc: changed=%v got=%q want=%q", changed, got, want)
+	}
+	// a single-line superfluous @param is removed with allow_mixed=false
+	singleParam := "<?php\nclass A {\n    /** @param int $x */\n    public function f(int $x) {}\n}\n"
+	got, changed = apply(t, noMixed.(fixerRule), singleParam)
+	if want := "<?php\nclass A {\n    /** */\n    public function f(int $x) {}\n}\n"; !changed || got != want {
+		t.Fatalf("single-line superfluous @param: changed=%v got=%q want=%q", changed, got, want)
+	}
 }
 
 func TestNoSuperfluousPhpdocTagsInheritDocWithDescription(t *testing.T) {