refactor: simplify diagnostic reporting logic by using a nullable arg…

…ument instead of a list
solid-illiaaihistov · Aug 27, 2026 · Aug 21, 2026 · Aug 21, 2026 · Aug 21, 2026 · Aug 26, 2026
commit 57865ff88f2843890f671462adf7388530662c7f
diff --git a/lib/src/lints/avoid_duplicate_code/reporters/avoid_duplicate_code_reporter.dart b/lib/src/lints/avoid_duplicate_code/reporters/avoid_duplicate_code_reporter.dart
@@ -81,22 +81,19 @@ class AvoidDuplicateCodeReporter {
         .map((p) => p.exactHash)
         .contains(currentExactHash);
 
-    final (diagnosticCode, arguments) = switch (hasExactPartner) {
-      true => (_rule.exactCode, const <String>[]),
-      false => (
-        _rule.differentLiteralsCode,
-        [
-          _computeLiteralsSummary(
-            currentLiterals: target.collectLiterals(_literalsAnalyzer),
-            partnerLiterals: [
-              for (final p in internalPartners)
-                p.collectLiterals(_literalsAnalyzer),
-              ..._loadExternalLiterals(externalPartners),
-            ],
-          ),
-        ],
-      ),
-    };
+    final (diagnosticCode, argument) = hasExactPartner
+        ? (_rule.exactCode, null)
+        : (
+            _rule.differentLiteralsCode,
+            _computeLiteralsSummary(
+              currentLiterals: target.collectLiterals(_literalsAnalyzer),
+              partnerLiterals: [
+                for (final p in internalPartners)
+                  p.collectLiterals(_literalsAnalyzer),
+                ..._loadExternalLiterals(externalPartners),
+              ],
+            ),
+          );
 
     final contextMessages = _buildContextMessages(
       currentFilePath: filePath,
@@ -108,7 +105,7 @@ class AvoidDuplicateCodeReporter {
     target.report(
       _rule,
       diagnosticCode: diagnosticCode,
-      arguments: arguments,
+      arguments: [?argument],
       contextMessages: contextMessages,
     );
   }