From 4e589f9ccf654f8a4ccd02176eeba148c1356377 Mon Sep 17 00:00:00 2001
From: Josef Pihrt <josef.pihrt@outlook.com>
Date: Fri, 10 Nov 2023 17:39:21 +0100
Subject: [PATCH 01/10] Add option 'detect_unity_features'

---
 src/Analyzers.xml                             |  4 ++++
 .../AnalyzerOptionIsObsoleteAnalyzer.cs       |  2 ++
 .../CSharp/Analysis/EnumSymbolAnalyzer.cs     |  2 ++
 .../MakeMemberReadOnlyAnalyzer.cs             |  5 ++++-
 .../UnusedMember/UnusedMemberAnalyzer.cs      |  2 +-
 .../CSharp/Extensions/CodeStyleExtensions.cs  | 13 +++++++++--
 src/Common/ConfigOptionKeys.Generated.cs      |  1 +
 src/Common/ConfigOptions.Generated.cs         |  8 +++++++
 src/ConfigOptions.xml                         |  5 +++++
 .../RCS1169MakeFieldReadOnlyTests.cs          | 22 +++++++++++++++++++
 ...S1213RemoveUnusedMemberDeclarationTests.cs | 22 +++++++++++++++++++
 .../CodeGeneration/CSharp/CodeGenerator.cs    |  5 +++--
 .../EditorConfig/EditorConfigGenerator.cs     |  4 +++-
 .../Markdown/MarkdownGenerator.cs             |  9 ++++++--
 src/Tools/Metadata/AnalyzerOptionMetadata.cs  |  4 +++-
 src/Tools/Metadata/MetadataFile.cs            |  3 ++-
 .../src/configurationFiles.generated.ts       |  9 ++++----
 17 files changed, 105 insertions(+), 15 deletions(-)

diff --git a/src/Analyzers.xml b/src/Analyzers.xml
index e8035a66a9..12fc3a0911 100644
--- a/src/Analyzers.xml
+++ b/src/Analyzers.xml
@@ -5178,6 +5178,9 @@ abstract class Foo : IFoo
     <Title>Make field read-only.</Title>
     <DefaultSeverity>Info</DefaultSeverity>
     <IsEnabledByDefault>true</IsEnabledByDefault>
+    <ConfigOptions>
+      <Option Key="detect_unity_features" />
+    </ConfigOptions>
     <Samples>
       <Sample>
         <Before><![CDATA[public class Foo
@@ -6117,6 +6120,7 @@ x = "";]]></Before>
     <IsEnabledByDefault>true</IsEnabledByDefault>
     <SupportsFadeOut>true</SupportsFadeOut>
     <ConfigOptions>
+      <Option Key="detect_unity_features" />
       <Option Key="suppress_unity_script_methods" />
     </ConfigOptions>
     <Options>
diff --git a/src/Analyzers/CSharp/Analysis/AnalyzerOptionIsObsoleteAnalyzer.cs b/src/Analyzers/CSharp/Analysis/AnalyzerOptionIsObsoleteAnalyzer.cs
index 0baeb0c4a0..daf2669f50 100644
--- a/src/Analyzers/CSharp/Analysis/AnalyzerOptionIsObsoleteAnalyzer.cs
+++ b/src/Analyzers/CSharp/Analysis/AnalyzerOptionIsObsoleteAnalyzer.cs
@@ -52,7 +52,9 @@ public override void Initialize(AnalysisContext context)
                 Validate(ref context, compilationOptions, options, Flags.RemoveEmptyLineBetweenClosingBraceAndSwitchSection, ref flags, DiagnosticRules.RemoveUnnecessaryBlankLine, ConfigOptions.BlankLineBetweenClosingBraceAndSwitchSection, LegacyConfigOptions.RemoveEmptyLineBetweenClosingBraceAndSwitchSection, "false");
                 Validate(ref context, compilationOptions, options, Flags.RemoveParenthesesFromConditionOfConditionalExpressionWhenExpressionIsSingleToken, ref flags, DiagnosticRules.AddOrRemoveParenthesesFromConditionInConditionalOperator, ConfigOptions.ConditionalOperatorConditionParenthesesStyle, LegacyConfigOptions.RemoveParenthesesFromConditionOfConditionalExpressionWhenExpressionIsSingleToken, ConfigOptionValues.ConditionalOperatorConditionParenthesesStyle_OmitWhenConditionIsSingleToken);
                 Validate(ref context, compilationOptions, options, Flags.RemoveParenthesesWhenCreatingNewObject, ref flags, DiagnosticRules.UseImplicitOrExplicitObjectCreation, ConfigOptions.ObjectCreationParenthesesStyle, LegacyConfigOptions.RemoveParenthesesWhenCreatingNewObject, ConfigOptionValues.ObjectCreationParenthesesStyle_Omit);
+#pragma warning disable CS0618 // Type or member is obsolete
                 Validate(ref context, compilationOptions, options, Flags.SuppressUnityScriptMethods, ref flags, DiagnosticRules.RemoveUnusedMemberDeclaration, ConfigOptions.SuppressUnityScriptMethods, LegacyConfigOptions.SuppressUnityScriptMethods, "true");
+#pragma warning restore CS0618 // Type or member is obsolete
                 Validate(ref context, compilationOptions, options, Flags.UseComparisonInsteadPatternMatchingToCheckForNull, ref flags, DiagnosticRules.NormalizeNullCheck, ConfigOptions.NullCheckStyle, LegacyConfigOptions.UseComparisonInsteadPatternMatchingToCheckForNull, ConfigOptionValues.NullCheckStyle_EqualityOperator);
                 Validate(ref context, compilationOptions, options, Flags.UseImplicitlyTypedArray, ref flags, DiagnosticRules.UseExplicitlyOrImplicitlyTypedArray, ConfigOptions.ArrayCreationTypeStyle, LegacyConfigOptions.UseImplicitlyTypedArray, ConfigOptionValues.ArrayCreationTypeStyle_Implicit);
                 Validate(ref context, compilationOptions, options, Flags.UseImplicitlyTypedArrayWhenTypeIsObvious, ref flags, DiagnosticRules.UseExplicitlyOrImplicitlyTypedArray, ConfigOptions.ArrayCreationTypeStyle, LegacyConfigOptions.UseImplicitlyTypedArrayWhenTypeIsObvious, ConfigOptionValues.ArrayCreationTypeStyle_ImplicitWhenTypeIsObvious);
diff --git a/src/Analyzers/CSharp/Analysis/EnumSymbolAnalyzer.cs b/src/Analyzers/CSharp/Analysis/EnumSymbolAnalyzer.cs
index 5bad422e50..b3a17d513d 100644
--- a/src/Analyzers/CSharp/Analysis/EnumSymbolAnalyzer.cs
+++ b/src/Analyzers/CSharp/Analysis/EnumSymbolAnalyzer.cs
@@ -198,7 +198,9 @@ private static void AnalyzeNamedType(SymbolAnalysisContext context)
 
                     if (CSharpUtility.IsSymbolObsolete(symbolInfo1.Symbol)
                         || CSharpUtility.IsSymbolObsolete(symbolInfo2.Symbol))
+                    {
                         continue;
+                    }
 
                     var enumMember1 = (EnumMemberDeclarationSyntax)symbolInfo1.Symbol.GetSyntax(context.CancellationToken);
 
diff --git a/src/Analyzers/CSharp/Analysis/MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs b/src/Analyzers/CSharp/Analysis/MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs
index 33a15900a2..5b147fa260 100644
--- a/src/Analyzers/CSharp/Analysis/MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs
+++ b/src/Analyzers/CSharp/Analysis/MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs
@@ -18,6 +18,7 @@ public sealed class MakeMemberReadOnlyAnalyzer : BaseDiagnosticAnalyzer
     private static readonly MetadataName Microsoft_AspNetCore_Components_CascadingParameterAttribute = MetadataName.Parse("Microsoft.AspNetCore.Components.CascadingParameterAttribute");
     private static readonly MetadataName Microsoft_AspNetCore_Components_InjectAttribute = MetadataName.Parse("Microsoft.AspNetCore.Components.InjectAttribute");
     private static readonly MetadataName Newtonsoft_Json_JsonPropertyAttribute = MetadataName.Parse("Newtonsoft.Json.JsonPropertyAttribute");
+    private static readonly MetadataName UnityEngine_SerializeFieldAttribute = MetadataName.Parse("UnityEngine.SerializeFieldAttribute");
 
     private static ImmutableArray<DiagnosticDescriptor> _supportedDiagnostics;
 
@@ -135,7 +136,9 @@ private static void AnalyzeTypeDeclaration(
                                 && fieldSymbol.DeclaredAccessibility == Accessibility.Private
                                 && !fieldSymbol.IsReadOnly
                                 && !fieldSymbol.IsVolatile
-                                && ValidateType(fieldSymbol.Type))
+                                && ValidateType(fieldSymbol.Type)
+                                && (context.DetectUnityFeatures() != true
+                                    || !fieldSymbol.HasAttribute(UnityEngine_SerializeFieldAttribute)))
                             {
                                 symbols[fieldSymbol.Name] = (declarator, fieldSymbol);
                             }
diff --git a/src/Analyzers/CSharp/Analysis/UnusedMember/UnusedMemberAnalyzer.cs b/src/Analyzers/CSharp/Analysis/UnusedMember/UnusedMemberAnalyzer.cs
index ac7d03fc19..b3e2041b27 100644
--- a/src/Analyzers/CSharp/Analysis/UnusedMember/UnusedMemberAnalyzer.cs
+++ b/src/Analyzers/CSharp/Analysis/UnusedMember/UnusedMemberAnalyzer.cs
@@ -159,7 +159,7 @@ private static void AnalyzeTypeDeclaration(SyntaxNodeAnalysisContext context)
                             break;
 
                         if (declaration.ReturnsVoid()
-                            && context.GetSuppressUnityScriptMethods() == true)
+                            && context.DetectUnityFeatures() == true)
                         {
                             if (canContainUnityScriptMethods is null)
                             {
diff --git a/src/Common/CSharp/Extensions/CodeStyleExtensions.cs b/src/Common/CSharp/Extensions/CodeStyleExtensions.cs
index 3e7f82fbac..222937b118 100644
--- a/src/Common/CSharp/Extensions/CodeStyleExtensions.cs
+++ b/src/Common/CSharp/Extensions/CodeStyleExtensions.cs
@@ -565,12 +565,11 @@ public static BlankLineStyle GetBlankLineAfterFileScopedNamespaceDeclaration(thi
         return BlankLineStyle.None;
     }
 
+    [Obsolete]
     public static bool? GetSuppressUnityScriptMethods(this SyntaxNodeAnalysisContext context)
     {
         if (ConfigOptions.TryGetValueAsBool(context.GetConfigOptions(), ConfigOptions.SuppressUnityScriptMethods, out bool value))
-        {
             return value;
-        }
 
         if (context.TryGetOptionAsBool(LegacyConfigOptions.SuppressUnityScriptMethods, out value))
             return value;
@@ -578,6 +577,16 @@ public static BlankLineStyle GetBlankLineAfterFileScopedNamespaceDeclaration(thi
         return null;
     }
 
+    public static bool? DetectUnityFeatures(this SyntaxNodeAnalysisContext context)
+    {
+        if (ConfigOptions.TryGetValueAsBool(context.GetConfigOptions(), ConfigOptions.DetectUnityFeatures, out bool value))
+            return value;
+
+#pragma warning disable CS0612 // Type or member is obsolete
+        return GetSuppressUnityScriptMethods(context);
+#pragma warning restore CS0612 // Type or member is obsolete
+    }
+
     public static NewLinePosition GetEqualsSignNewLinePosition(this SyntaxNodeAnalysisContext context)
     {
         return context.GetConfigOptions().GetEqualsTokenNewLinePosition();
diff --git a/src/Common/ConfigOptionKeys.Generated.cs b/src/Common/ConfigOptionKeys.Generated.cs
index a35b9675c2..b5b56ea9c4 100644
--- a/src/Common/ConfigOptionKeys.Generated.cs
+++ b/src/Common/ConfigOptionKeys.Generated.cs
@@ -20,6 +20,7 @@ internal static partial class ConfigOptionKeys
         public const string ConditionalOperatorConditionParenthesesStyle      = "roslynator_conditional_operator_condition_parentheses_style";
         public const string ConditionalOperatorNewLine                        = "roslynator_conditional_operator_new_line";
         public const string ConfigureAwait                                    = "roslynator_configure_await";
+        public const string DetectUnityFeatures                               = "roslynator_detect_unity_features";
         public const string DocCommentSummaryStyle                            = "roslynator_doc_comment_summary_style";
         public const string EmptyStringStyle                                  = "roslynator_empty_string_style";
         public const string EnumFlagValueStyle                                = "roslynator_enum_flag_value_style";
diff --git a/src/Common/ConfigOptions.Generated.cs b/src/Common/ConfigOptions.Generated.cs
index 82c5cd06f0..6fc94adf3f 100644
--- a/src/Common/ConfigOptions.Generated.cs
+++ b/src/Common/ConfigOptions.Generated.cs
@@ -2,6 +2,7 @@
 
 // <auto-generated>
 
+using System;
 using System.Collections.Generic;
 
 namespace Roslynator
@@ -92,6 +93,12 @@ public static partial class ConfigOptions
             defaultValuePlaceholder: "true|false", 
             description:             "Add/remove 'ConfigureAwait(false)' call");
 
+        public static readonly ConfigOptionDescriptor DetectUnityFeatures = new(
+            key:                     ConfigOptionKeys.DetectUnityFeatures, 
+            defaultValue:            null, 
+            defaultValuePlaceholder: "true|false", 
+            description:             "Detect false positives from Unity code");
+
         public static readonly ConfigOptionDescriptor DocCommentSummaryStyle = new(
             key:                     ConfigOptionKeys.DocCommentSummaryStyle, 
             defaultValue:            null, 
@@ -176,6 +183,7 @@ public static partial class ConfigOptions
             defaultValuePlaceholder: "true|false", 
             description:             "Prefix field identifier with underscore");
 
+        [Obsolete("", error: false)]
         public static readonly ConfigOptionDescriptor SuppressUnityScriptMethods = new(
             key:                     ConfigOptionKeys.SuppressUnityScriptMethods, 
             defaultValue:            null, 
diff --git a/src/ConfigOptions.xml b/src/ConfigOptions.xml
index f8fa222c1e..459df38dea 100644
--- a/src/ConfigOptions.xml
+++ b/src/ConfigOptions.xml
@@ -173,6 +173,11 @@
   <Option Id="SuppressUnityScriptMethods">
     <ValuePlaceholder>true|false</ValuePlaceholder>
     <Description>Suppress Unity script methods</Description>
+    <ObsoleteMessage>This option is obsolete, use option 'detect_unity_features' instead.</ObsoleteMessage>
+  </Option>
+  <Option Id="DetectUnityFeatures">
+    <ValuePlaceholder>true|false</ValuePlaceholder>
+    <Description>Detect false positives from Unity code</Description>
   </Option>
   <Option Id="UseAnonymousFunctionOrMethodGroup">
     <Description>Use anonymous function or method group</Description>
diff --git a/src/Tests/Analyzers.Tests/RCS1169MakeFieldReadOnlyTests.cs b/src/Tests/Analyzers.Tests/RCS1169MakeFieldReadOnlyTests.cs
index 3703b05bf5..dcb990614d 100644
--- a/src/Tests/Analyzers.Tests/RCS1169MakeFieldReadOnlyTests.cs
+++ b/src/Tests/Analyzers.Tests/RCS1169MakeFieldReadOnlyTests.cs
@@ -377,4 +377,26 @@ class C
 }
 ");
     }
+
+    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.MakeFieldReadOnly)]
+    public async Task TestNoDiagnostic_UnitySerializeAttribute()
+    {
+        await VerifyNoDiagnosticAsync(@"
+using System;
+using UnityEngine;
+
+class C
+{
+    [SerializeField]
+    private string f;
+}
+
+namespace UnityEngine
+{
+    class SerializeFieldAttribute : Attribute
+    {
+    }
+}
+", options: Options.AddConfigOption(ConfigOptionKeys.DetectUnityFeatures, true));
+    }
 }
diff --git a/src/Tests/Analyzers.Tests/RCS1213RemoveUnusedMemberDeclarationTests.cs b/src/Tests/Analyzers.Tests/RCS1213RemoveUnusedMemberDeclarationTests.cs
index 8db3bd3c88..e20718d971 100644
--- a/src/Tests/Analyzers.Tests/RCS1213RemoveUnusedMemberDeclarationTests.cs
+++ b/src/Tests/Analyzers.Tests/RCS1213RemoveUnusedMemberDeclarationTests.cs
@@ -366,4 +366,26 @@ class MonoBehaviour
 }
 ", options: Options.AddConfigOption(ConfigOptionKeys.SuppressUnityScriptMethods, true));
     }
+
+    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.RemoveUnusedMemberDeclaration)]
+    public async Task TestNoDiagnostic_UnityScriptMethods2()
+    {
+        await VerifyNoDiagnosticAsync(@"
+using UnityEngine;
+
+class C : MonoBehaviour
+{
+    private void Awake()
+    {
+    }
+}
+
+namespace UnityEngine
+{
+    class MonoBehaviour
+    {
+    }
+}
+", options: Options.AddConfigOption(ConfigOptionKeys.DetectUnityFeatures, true));
+    }
 }
diff --git a/src/Tools/CodeGeneration/CSharp/CodeGenerator.cs b/src/Tools/CodeGeneration/CSharp/CodeGenerator.cs
index ccc6ceb7f5..c0ba0723ff 100644
--- a/src/Tools/CodeGeneration/CSharp/CodeGenerator.cs
+++ b/src/Tools/CodeGeneration/CSharp/CodeGenerator.cs
@@ -18,7 +18,7 @@ public static class CodeGenerator
     public static CompilationUnitSyntax GenerateConfigOptions(IEnumerable<AnalyzerOptionMetadata> options, IEnumerable<AnalyzerMetadata> analyzers)
     {
         CompilationUnitSyntax compilationUnit = CompilationUnit(
-            UsingDirectives("System.Collections.Generic"),
+            UsingDirectives("System", "System.Collections.Generic"),
             NamespaceDeclaration(
                 "Roslynator",
                 ClassDeclaration(
@@ -38,7 +38,8 @@ public static CompilationUnitSyntax GenerateConfigOptions(IEnumerable<AnalyzerOp
                                         Argument(NameColon("defaultValue"), (f.DefaultValue is not null) ? StringLiteralExpression(f.DefaultValue) : NullLiteralExpression()),
                                         Argument(NameColon("defaultValuePlaceholder"), StringLiteralExpression(f.DefaultValuePlaceholder)),
                                         Argument(NameColon("description"), StringLiteralExpression(f.Description))),
-                                    default(InitializerExpressionSyntax)));
+                                    default(InitializerExpressionSyntax)))
+                                .AddObsoleteAttributeIf(f.IsObsolete);
                         })
                         .Concat(new MemberDeclarationSyntax[]
                             {
diff --git a/src/Tools/CodeGeneration/EditorConfig/EditorConfigGenerator.cs b/src/Tools/CodeGeneration/EditorConfig/EditorConfigGenerator.cs
index 958052bff4..bbb72c79e2 100644
--- a/src/Tools/CodeGeneration/EditorConfig/EditorConfigGenerator.cs
+++ b/src/Tools/CodeGeneration/EditorConfig/EditorConfigGenerator.cs
@@ -36,7 +36,9 @@ public static string GenerateEditorConfig(RoslynatorMetadata metadata, bool comm
 
             var isSeparatedWithNewLine = true;
 
-            foreach (AnalyzerOptionMetadata option in metadata.ConfigOptions.OrderBy(f => f.Key))
+            foreach (AnalyzerOptionMetadata option in metadata.ConfigOptions
+                .Where(f => !f.IsObsolete)
+                .OrderBy(f => f.Key))
             {
                 if (optionMap.TryGetValue(option.Key, out HashSet<AnalyzerMetadata> analyzers)
                     && !isSeparatedWithNewLine)
diff --git a/src/Tools/CodeGeneration/Markdown/MarkdownGenerator.cs b/src/Tools/CodeGeneration/Markdown/MarkdownGenerator.cs
index 315d2f5a8f..9f5c92753b 100644
--- a/src/Tools/CodeGeneration/Markdown/MarkdownGenerator.cs
+++ b/src/Tools/CodeGeneration/Markdown/MarkdownGenerator.cs
@@ -210,8 +210,13 @@ static IEnumerable<object> CreateConfiguration(AnalyzerMetadata analyzer, Immuta
 
                         sb.Append("# ");
 
-                        if (!string.IsNullOrEmpty(en.Current.Description))
-                            sb.AppendLine(en.Current.Description);
+                        string description = en.Current.Description;
+
+                        if (en.Current.IsObsolete)
+                            description = "[deprecated] " + en.Current.ObsoleteMessage;
+
+                        if (!string.IsNullOrEmpty(description))
+                            sb.AppendLine(description);
 
                         string defaultValue = en.Current.DefaultValue;
 
diff --git a/src/Tools/Metadata/AnalyzerOptionMetadata.cs b/src/Tools/Metadata/AnalyzerOptionMetadata.cs
index bb5d583191..fba31aeec4 100644
--- a/src/Tools/Metadata/AnalyzerOptionMetadata.cs
+++ b/src/Tools/Metadata/AnalyzerOptionMetadata.cs
@@ -4,7 +4,9 @@
 
 namespace Roslynator.Metadata;
 
-public record AnalyzerOptionMetadata(string Id, string Key, string DefaultValue, string DefaultValuePlaceholder, string Description)
+public record AnalyzerOptionMetadata(string Id, string Key, string DefaultValue, string DefaultValuePlaceholder, string Description, string ObsoleteMessage)
 {
     public List<AnalyzerOptionValueMetadata> Values { get; } = new();
+
+    public bool IsObsolete => !string.IsNullOrEmpty(ObsoleteMessage);
 }
diff --git a/src/Tools/Metadata/MetadataFile.cs b/src/Tools/Metadata/MetadataFile.cs
index 1f5b3349fc..d6b1f9ab04 100644
--- a/src/Tools/Metadata/MetadataFile.cs
+++ b/src/Tools/Metadata/MetadataFile.cs
@@ -290,7 +290,8 @@ public static IEnumerable<AnalyzerOptionMetadata> ReadOptions(string filePath)
                 Key: "roslynator_" + key,
                 DefaultValue: defaultValue,
                 DefaultValuePlaceholder: defaultValuePlaceholder,
-                Description: element.Element("Description").Value
+                Description: element.Element("Description").Value,
+                ObsoleteMessage: element.Element("ObsoleteMessage")?.Value
             );
 
             analyzerOption.Values.AddRange(values ?? Enumerable.Empty<AnalyzerOptionValueMetadata>());
diff --git a/src/VisualStudioCode/package/src/configurationFiles.generated.ts b/src/VisualStudioCode/package/src/configurationFiles.generated.ts
index d0158985f7..662b5bebda 100644
--- a/src/VisualStudioCode/package/src/configurationFiles.generated.ts
+++ b/src/VisualStudioCode/package/src/configurationFiles.generated.ts
@@ -75,6 +75,9 @@ roslynator_analyzers.enabled_by_default = true|false
 #roslynator_configure_await = true|false
 # Applicable to: rcs1090
 
+#roslynator_detect_unity_features = true|false
+# Applicable to: rcs1169, rcs1213
+
 #roslynator_doc_comment_summary_style = multi_line|single_line
 # Applicable to: rcs1253
 
@@ -117,9 +120,6 @@ roslynator_analyzers.enabled_by_default = true|false
 
 #roslynator_prefix_field_identifier_with_underscore = true|false
 
-#roslynator_suppress_unity_script_methods = true|false
-# Applicable to: rcs1213
-
 #roslynator_tab_length = <NUM>
 # Default: 4
 # Applicable to: rcs0056
@@ -641,6 +641,7 @@ roslynator_analyzers.enabled_by_default = true|false
 
 # Make field read-only
 #dotnet_diagnostic.rcs1169.severity = suggestion
+# Options: roslynator_detect_unity_features
 
 # Use read-only auto-implemented property
 #dotnet_diagnostic.rcs1170.severity = suggestion
@@ -762,7 +763,7 @@ roslynator_analyzers.enabled_by_default = true|false
 
 # Remove unused member declaration
 #dotnet_diagnostic.rcs1213.severity = suggestion
-# Options: roslynator_suppress_unity_script_methods
+# Options: roslynator_detect_unity_features, roslynator_suppress_unity_script_methods
 
 # Unnecessary interpolated string
 #dotnet_diagnostic.rcs1214.severity = suggestion

From 616f63fc99f59ab0a5a63f62166e2744c0830437 Mon Sep 17 00:00:00 2001
From: Josef Pihrt <josef.pihrt@outlook.com>
Date: Fri, 10 Nov 2023 17:47:13 +0100
Subject: [PATCH 02/10] update

---
 ChangeLog.md | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/ChangeLog.md b/ChangeLog.md
index f86fe801ac..d02a57be26 100644
--- a/ChangeLog.md
+++ b/ChangeLog.md
@@ -17,6 +17,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Fix refactoring [Inline method](https://josefpihrt.github.io/docs/roslynator/refactorings/RR0062) ([PR](https://github.com/dotnet/roslynator/pull/1234))
 - [CLI] Fix globbing ([PR](https://github.com/dotnet/roslynator/pull/1238))
 - [CLI] Remove assembly resolving ([PR](https://github.com/dotnet/roslynator/pull/1237))
+- Detect false positive from Unity code ([RCS1169](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1169)) ([PR](https://github.com/dotnet/roslynator/pull/1245))
+  - Introduce config option `detect_unity_features`
+  - Make option `suppress_unity_script_methods` obsolete
 
 ## [4.6.1] - 2023-10-23
 

From b5f23cb4de0a670ee67379ab56d164b7c63922d5 Mon Sep 17 00:00:00 2001
From: Josef Pihrt <josef.pihrt@outlook.com>
Date: Fri, 10 Nov 2023 17:50:37 +0100
Subject: [PATCH 03/10] update

---
 .../CSharp/Extensions/CodeStyleExtensions.cs  | 20 +++++++------------
 1 file changed, 7 insertions(+), 13 deletions(-)

diff --git a/src/Common/CSharp/Extensions/CodeStyleExtensions.cs b/src/Common/CSharp/Extensions/CodeStyleExtensions.cs
index 222937b118..7e1c582879 100644
--- a/src/Common/CSharp/Extensions/CodeStyleExtensions.cs
+++ b/src/Common/CSharp/Extensions/CodeStyleExtensions.cs
@@ -565,26 +565,20 @@ public static BlankLineStyle GetBlankLineAfterFileScopedNamespaceDeclaration(thi
         return BlankLineStyle.None;
     }
 
-    [Obsolete]
-    public static bool? GetSuppressUnityScriptMethods(this SyntaxNodeAnalysisContext context)
+    public static bool? DetectUnityFeatures(this SyntaxNodeAnalysisContext context)
     {
-        if (ConfigOptions.TryGetValueAsBool(context.GetConfigOptions(), ConfigOptions.SuppressUnityScriptMethods, out bool value))
+        if (ConfigOptions.TryGetValueAsBool(context.GetConfigOptions(), ConfigOptions.DetectUnityFeatures, out bool value))
             return value;
 
-        if (context.TryGetOptionAsBool(LegacyConfigOptions.SuppressUnityScriptMethods, out value))
+#pragma warning disable CS0618 // Type or member is obsolete
+        if (ConfigOptions.TryGetValueAsBool(context.GetConfigOptions(), ConfigOptions.SuppressUnityScriptMethods, out value))
             return value;
+#pragma warning restore CS0618 // Type or member is obsolete
 
-        return null;
-    }
-
-    public static bool? DetectUnityFeatures(this SyntaxNodeAnalysisContext context)
-    {
-        if (ConfigOptions.TryGetValueAsBool(context.GetConfigOptions(), ConfigOptions.DetectUnityFeatures, out bool value))
+        if (context.TryGetOptionAsBool(LegacyConfigOptions.SuppressUnityScriptMethods, out value))
             return value;
 
-#pragma warning disable CS0612 // Type or member is obsolete
-        return GetSuppressUnityScriptMethods(context);
-#pragma warning restore CS0612 // Type or member is obsolete
+        return null;
     }
 
     public static NewLinePosition GetEqualsSignNewLinePosition(this SyntaxNodeAnalysisContext context)

From 643270d0c15e963fb98df3d77337dd32c3191122 Mon Sep 17 00:00:00 2001
From: Josef Pihrt <josef.pihrt@outlook.com>
Date: Fri, 10 Nov 2023 20:26:09 +0100
Subject: [PATCH 04/10] update

---
 src/Analyzers.xml                                  |  4 ++--
 .../MakeMemberReadOnlyAnalyzer.cs                  |  2 +-
 .../Analysis/UnusedMember/UnusedMemberAnalyzer.cs  |  2 +-
 .../CSharp/Extensions/CodeStyleExtensions.cs       |  4 ++--
 src/Common/ConfigOptionKeys.Generated.cs           |  2 +-
 src/Common/ConfigOptions.Generated.cs              | 14 +++++++-------
 src/ConfigOptions.xml                              |  8 ++++----
 .../RCS1169MakeFieldReadOnlyTests.cs               |  2 +-
 .../RCS1213RemoveUnusedMemberDeclarationTests.cs   |  2 +-
 .../package/src/configurationFiles.generated.ts    | 10 +++++-----
 10 files changed, 25 insertions(+), 25 deletions(-)

diff --git a/src/Analyzers.xml b/src/Analyzers.xml
index 12fc3a0911..df0a70d9f1 100644
--- a/src/Analyzers.xml
+++ b/src/Analyzers.xml
@@ -5179,7 +5179,7 @@ abstract class Foo : IFoo
     <DefaultSeverity>Info</DefaultSeverity>
     <IsEnabledByDefault>true</IsEnabledByDefault>
     <ConfigOptions>
-      <Option Key="detect_unity_features" />
+      <Option Key="suppress_unity" />
     </ConfigOptions>
     <Samples>
       <Sample>
@@ -6120,7 +6120,7 @@ x = "";]]></Before>
     <IsEnabledByDefault>true</IsEnabledByDefault>
     <SupportsFadeOut>true</SupportsFadeOut>
     <ConfigOptions>
-      <Option Key="detect_unity_features" />
+      <Option Key="suppress_unity" />
       <Option Key="suppress_unity_script_methods" />
     </ConfigOptions>
     <Options>
diff --git a/src/Analyzers/CSharp/Analysis/MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs b/src/Analyzers/CSharp/Analysis/MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs
index 5b147fa260..aad46abf6a 100644
--- a/src/Analyzers/CSharp/Analysis/MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs
+++ b/src/Analyzers/CSharp/Analysis/MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs
@@ -137,7 +137,7 @@ private static void AnalyzeTypeDeclaration(
                                 && !fieldSymbol.IsReadOnly
                                 && !fieldSymbol.IsVolatile
                                 && ValidateType(fieldSymbol.Type)
-                                && (context.DetectUnityFeatures() != true
+                                && (context.SuppressUnity() != true
                                     || !fieldSymbol.HasAttribute(UnityEngine_SerializeFieldAttribute)))
                             {
                                 symbols[fieldSymbol.Name] = (declarator, fieldSymbol);
diff --git a/src/Analyzers/CSharp/Analysis/UnusedMember/UnusedMemberAnalyzer.cs b/src/Analyzers/CSharp/Analysis/UnusedMember/UnusedMemberAnalyzer.cs
index b3e2041b27..58f571da17 100644
--- a/src/Analyzers/CSharp/Analysis/UnusedMember/UnusedMemberAnalyzer.cs
+++ b/src/Analyzers/CSharp/Analysis/UnusedMember/UnusedMemberAnalyzer.cs
@@ -159,7 +159,7 @@ private static void AnalyzeTypeDeclaration(SyntaxNodeAnalysisContext context)
                             break;
 
                         if (declaration.ReturnsVoid()
-                            && context.DetectUnityFeatures() == true)
+                            && context.SuppressUnity() == true)
                         {
                             if (canContainUnityScriptMethods is null)
                             {
diff --git a/src/Common/CSharp/Extensions/CodeStyleExtensions.cs b/src/Common/CSharp/Extensions/CodeStyleExtensions.cs
index 7e1c582879..bd70b10d13 100644
--- a/src/Common/CSharp/Extensions/CodeStyleExtensions.cs
+++ b/src/Common/CSharp/Extensions/CodeStyleExtensions.cs
@@ -565,9 +565,9 @@ public static BlankLineStyle GetBlankLineAfterFileScopedNamespaceDeclaration(thi
         return BlankLineStyle.None;
     }
 
-    public static bool? DetectUnityFeatures(this SyntaxNodeAnalysisContext context)
+    public static bool? SuppressUnity(this SyntaxNodeAnalysisContext context)
     {
-        if (ConfigOptions.TryGetValueAsBool(context.GetConfigOptions(), ConfigOptions.DetectUnityFeatures, out bool value))
+        if (ConfigOptions.TryGetValueAsBool(context.GetConfigOptions(), ConfigOptions.SuppressUnity, out bool value))
             return value;
 
 #pragma warning disable CS0618 // Type or member is obsolete
diff --git a/src/Common/ConfigOptionKeys.Generated.cs b/src/Common/ConfigOptionKeys.Generated.cs
index b5b56ea9c4..7cd419e091 100644
--- a/src/Common/ConfigOptionKeys.Generated.cs
+++ b/src/Common/ConfigOptionKeys.Generated.cs
@@ -20,7 +20,6 @@ internal static partial class ConfigOptionKeys
         public const string ConditionalOperatorConditionParenthesesStyle      = "roslynator_conditional_operator_condition_parentheses_style";
         public const string ConditionalOperatorNewLine                        = "roslynator_conditional_operator_new_line";
         public const string ConfigureAwait                                    = "roslynator_configure_await";
-        public const string DetectUnityFeatures                               = "roslynator_detect_unity_features";
         public const string DocCommentSummaryStyle                            = "roslynator_doc_comment_summary_style";
         public const string EmptyStringStyle                                  = "roslynator_empty_string_style";
         public const string EnumFlagValueStyle                                = "roslynator_enum_flag_value_style";
@@ -35,6 +34,7 @@ internal static partial class ConfigOptionKeys
         public const string ObjectCreationParenthesesStyle                    = "roslynator_object_creation_parentheses_style";
         public const string ObjectCreationTypeStyle                           = "roslynator_object_creation_type_style";
         public const string PrefixFieldIdentifierWithUnderscore               = "roslynator_prefix_field_identifier_with_underscore";
+        public const string SuppressUnity                                     = "roslynator_suppress_unity";
         public const string SuppressUnityScriptMethods                        = "roslynator_suppress_unity_script_methods";
         public const string TabLength                                         = "roslynator_tab_length";
         public const string TrailingCommaStyle                                = "roslynator_trailing_comma_style";
diff --git a/src/Common/ConfigOptions.Generated.cs b/src/Common/ConfigOptions.Generated.cs
index 6fc94adf3f..594a84af71 100644
--- a/src/Common/ConfigOptions.Generated.cs
+++ b/src/Common/ConfigOptions.Generated.cs
@@ -93,12 +93,6 @@ public static partial class ConfigOptions
             defaultValuePlaceholder: "true|false", 
             description:             "Add/remove 'ConfigureAwait(false)' call");
 
-        public static readonly ConfigOptionDescriptor DetectUnityFeatures = new(
-            key:                     ConfigOptionKeys.DetectUnityFeatures, 
-            defaultValue:            null, 
-            defaultValuePlaceholder: "true|false", 
-            description:             "Detect false positives from Unity code");
-
         public static readonly ConfigOptionDescriptor DocCommentSummaryStyle = new(
             key:                     ConfigOptionKeys.DocCommentSummaryStyle, 
             defaultValue:            null, 
@@ -183,12 +177,18 @@ public static partial class ConfigOptions
             defaultValuePlaceholder: "true|false", 
             description:             "Prefix field identifier with underscore");
 
+        public static readonly ConfigOptionDescriptor SuppressUnity = new(
+            key:                     ConfigOptionKeys.SuppressUnity, 
+            defaultValue:            null, 
+            defaultValuePlaceholder: "true|false", 
+            description:             "Suppress false positives from Unity code");
+
         [Obsolete("", error: false)]
         public static readonly ConfigOptionDescriptor SuppressUnityScriptMethods = new(
             key:                     ConfigOptionKeys.SuppressUnityScriptMethods, 
             defaultValue:            null, 
             defaultValuePlaceholder: "true|false", 
-            description:             "Suppress Unity script methods");
+            description:             "Suppress false positives from Unity script methods");
 
         public static readonly ConfigOptionDescriptor TabLength = new(
             key:                     ConfigOptionKeys.TabLength, 
diff --git a/src/ConfigOptions.xml b/src/ConfigOptions.xml
index 459df38dea..fef633c1e0 100644
--- a/src/ConfigOptions.xml
+++ b/src/ConfigOptions.xml
@@ -172,12 +172,12 @@
   </Option>
   <Option Id="SuppressUnityScriptMethods">
     <ValuePlaceholder>true|false</ValuePlaceholder>
-    <Description>Suppress Unity script methods</Description>
-    <ObsoleteMessage>This option is obsolete, use option 'detect_unity_features' instead.</ObsoleteMessage>
+    <Description>Suppress false positives from Unity script methods</Description>
+    <ObsoleteMessage>This option is obsolete, use option 'roslynator_suppress_unity' instead.</ObsoleteMessage>
   </Option>
-  <Option Id="DetectUnityFeatures">
+  <Option Id="SuppressUnity">
     <ValuePlaceholder>true|false</ValuePlaceholder>
-    <Description>Detect false positives from Unity code</Description>
+    <Description>Suppress false positives from Unity code</Description>
   </Option>
   <Option Id="UseAnonymousFunctionOrMethodGroup">
     <Description>Use anonymous function or method group</Description>
diff --git a/src/Tests/Analyzers.Tests/RCS1169MakeFieldReadOnlyTests.cs b/src/Tests/Analyzers.Tests/RCS1169MakeFieldReadOnlyTests.cs
index dcb990614d..7fbb66d52e 100644
--- a/src/Tests/Analyzers.Tests/RCS1169MakeFieldReadOnlyTests.cs
+++ b/src/Tests/Analyzers.Tests/RCS1169MakeFieldReadOnlyTests.cs
@@ -397,6 +397,6 @@ class SerializeFieldAttribute : Attribute
     {
     }
 }
-", options: Options.AddConfigOption(ConfigOptionKeys.DetectUnityFeatures, true));
+", options: Options.AddConfigOption(ConfigOptionKeys.SuppressUnity, true));
     }
 }
diff --git a/src/Tests/Analyzers.Tests/RCS1213RemoveUnusedMemberDeclarationTests.cs b/src/Tests/Analyzers.Tests/RCS1213RemoveUnusedMemberDeclarationTests.cs
index e20718d971..50de829cf4 100644
--- a/src/Tests/Analyzers.Tests/RCS1213RemoveUnusedMemberDeclarationTests.cs
+++ b/src/Tests/Analyzers.Tests/RCS1213RemoveUnusedMemberDeclarationTests.cs
@@ -386,6 +386,6 @@ class MonoBehaviour
     {
     }
 }
-", options: Options.AddConfigOption(ConfigOptionKeys.DetectUnityFeatures, true));
+", options: Options.AddConfigOption(ConfigOptionKeys.SuppressUnity, true));
     }
 }
diff --git a/src/VisualStudioCode/package/src/configurationFiles.generated.ts b/src/VisualStudioCode/package/src/configurationFiles.generated.ts
index 662b5bebda..b359bede94 100644
--- a/src/VisualStudioCode/package/src/configurationFiles.generated.ts
+++ b/src/VisualStudioCode/package/src/configurationFiles.generated.ts
@@ -75,9 +75,6 @@ roslynator_analyzers.enabled_by_default = true|false
 #roslynator_configure_await = true|false
 # Applicable to: rcs1090
 
-#roslynator_detect_unity_features = true|false
-# Applicable to: rcs1169, rcs1213
-
 #roslynator_doc_comment_summary_style = multi_line|single_line
 # Applicable to: rcs1253
 
@@ -120,6 +117,9 @@ roslynator_analyzers.enabled_by_default = true|false
 
 #roslynator_prefix_field_identifier_with_underscore = true|false
 
+#roslynator_suppress_unity = true|false
+# Applicable to: rcs1169, rcs1213
+
 #roslynator_tab_length = <NUM>
 # Default: 4
 # Applicable to: rcs0056
@@ -641,7 +641,7 @@ roslynator_analyzers.enabled_by_default = true|false
 
 # Make field read-only
 #dotnet_diagnostic.rcs1169.severity = suggestion
-# Options: roslynator_detect_unity_features
+# Options: roslynator_suppress_unity
 
 # Use read-only auto-implemented property
 #dotnet_diagnostic.rcs1170.severity = suggestion
@@ -763,7 +763,7 @@ roslynator_analyzers.enabled_by_default = true|false
 
 # Remove unused member declaration
 #dotnet_diagnostic.rcs1213.severity = suggestion
-# Options: roslynator_detect_unity_features, roslynator_suppress_unity_script_methods
+# Options: roslynator_suppress_unity, roslynator_suppress_unity_script_methods
 
 # Unnecessary interpolated string
 #dotnet_diagnostic.rcs1214.severity = suggestion

From 1996582372fc2f5ca853695d1ccf37e738134fce Mon Sep 17 00:00:00 2001
From: Josef Pihrt <josef.pihrt@outlook.com>
Date: Fri, 10 Nov 2023 21:45:07 +0100
Subject: [PATCH 05/10] update

---
 ChangeLog.md                                         |  4 ++--
 src/Analyzers.xml                                    |  4 ++--
 .../MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs |  2 +-
 .../Analysis/UnusedMember/UnusedMemberAnalyzer.cs    |  2 +-
 src/Common/CSharp/Extensions/CodeStyleExtensions.cs  |  4 ++--
 src/Common/ConfigOptionKeys.Generated.cs             |  2 +-
 src/Common/ConfigOptions.Generated.cs                | 12 ++++++------
 src/ConfigOptions.xml                                |  7 ++++---
 .../Analyzers.Tests/RCS1169MakeFieldReadOnlyTests.cs |  2 +-
 .../RCS1213RemoveUnusedMemberDeclarationTests.cs     |  2 +-
 .../package/src/configurationFiles.generated.ts      | 10 +++++-----
 11 files changed, 26 insertions(+), 25 deletions(-)

diff --git a/ChangeLog.md b/ChangeLog.md
index d02a57be26..68d8eb93e7 100644
--- a/ChangeLog.md
+++ b/ChangeLog.md
@@ -18,8 +18,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - [CLI] Fix globbing ([PR](https://github.com/dotnet/roslynator/pull/1238))
 - [CLI] Remove assembly resolving ([PR](https://github.com/dotnet/roslynator/pull/1237))
 - Detect false positive from Unity code ([RCS1169](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1169)) ([PR](https://github.com/dotnet/roslynator/pull/1245))
-  - Introduce config option `detect_unity_features`
-  - Make option `suppress_unity_script_methods` obsolete
+  - Introduce config option `roslynator_unity_code_analysis.enabled`
+  - Make option `roslynator_suppress_unity_script_methods` obsolete
 
 ## [4.6.1] - 2023-10-23
 
diff --git a/src/Analyzers.xml b/src/Analyzers.xml
index df0a70d9f1..8e14a83a93 100644
--- a/src/Analyzers.xml
+++ b/src/Analyzers.xml
@@ -5179,7 +5179,7 @@ abstract class Foo : IFoo
     <DefaultSeverity>Info</DefaultSeverity>
     <IsEnabledByDefault>true</IsEnabledByDefault>
     <ConfigOptions>
-      <Option Key="suppress_unity" />
+      <Option Key="unity_code_analysis.enabled" />
     </ConfigOptions>
     <Samples>
       <Sample>
@@ -6120,7 +6120,7 @@ x = "";]]></Before>
     <IsEnabledByDefault>true</IsEnabledByDefault>
     <SupportsFadeOut>true</SupportsFadeOut>
     <ConfigOptions>
-      <Option Key="suppress_unity" />
+      <Option Key="unity_code_analysis.enabled" />
       <Option Key="suppress_unity_script_methods" />
     </ConfigOptions>
     <Options>
diff --git a/src/Analyzers/CSharp/Analysis/MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs b/src/Analyzers/CSharp/Analysis/MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs
index aad46abf6a..c15bb467b0 100644
--- a/src/Analyzers/CSharp/Analysis/MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs
+++ b/src/Analyzers/CSharp/Analysis/MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs
@@ -137,7 +137,7 @@ private static void AnalyzeTypeDeclaration(
                                 && !fieldSymbol.IsReadOnly
                                 && !fieldSymbol.IsVolatile
                                 && ValidateType(fieldSymbol.Type)
-                                && (context.SuppressUnity() != true
+                                && (context.GetUnityCodeAnalysisEnabled() != true
                                     || !fieldSymbol.HasAttribute(UnityEngine_SerializeFieldAttribute)))
                             {
                                 symbols[fieldSymbol.Name] = (declarator, fieldSymbol);
diff --git a/src/Analyzers/CSharp/Analysis/UnusedMember/UnusedMemberAnalyzer.cs b/src/Analyzers/CSharp/Analysis/UnusedMember/UnusedMemberAnalyzer.cs
index 58f571da17..f588452f7b 100644
--- a/src/Analyzers/CSharp/Analysis/UnusedMember/UnusedMemberAnalyzer.cs
+++ b/src/Analyzers/CSharp/Analysis/UnusedMember/UnusedMemberAnalyzer.cs
@@ -159,7 +159,7 @@ private static void AnalyzeTypeDeclaration(SyntaxNodeAnalysisContext context)
                             break;
 
                         if (declaration.ReturnsVoid()
-                            && context.SuppressUnity() == true)
+                            && context.GetUnityCodeAnalysisEnabled() == true)
                         {
                             if (canContainUnityScriptMethods is null)
                             {
diff --git a/src/Common/CSharp/Extensions/CodeStyleExtensions.cs b/src/Common/CSharp/Extensions/CodeStyleExtensions.cs
index bd70b10d13..f6b032d627 100644
--- a/src/Common/CSharp/Extensions/CodeStyleExtensions.cs
+++ b/src/Common/CSharp/Extensions/CodeStyleExtensions.cs
@@ -565,9 +565,9 @@ public static BlankLineStyle GetBlankLineAfterFileScopedNamespaceDeclaration(thi
         return BlankLineStyle.None;
     }
 
-    public static bool? SuppressUnity(this SyntaxNodeAnalysisContext context)
+    public static bool? GetUnityCodeAnalysisEnabled(this SyntaxNodeAnalysisContext context)
     {
-        if (ConfigOptions.TryGetValueAsBool(context.GetConfigOptions(), ConfigOptions.SuppressUnity, out bool value))
+        if (ConfigOptions.TryGetValueAsBool(context.GetConfigOptions(), ConfigOptions.UnityCodeAnalysisEnabled, out bool value))
             return value;
 
 #pragma warning disable CS0618 // Type or member is obsolete
diff --git a/src/Common/ConfigOptionKeys.Generated.cs b/src/Common/ConfigOptionKeys.Generated.cs
index 7cd419e091..1907f5092e 100644
--- a/src/Common/ConfigOptionKeys.Generated.cs
+++ b/src/Common/ConfigOptionKeys.Generated.cs
@@ -34,10 +34,10 @@ internal static partial class ConfigOptionKeys
         public const string ObjectCreationParenthesesStyle                    = "roslynator_object_creation_parentheses_style";
         public const string ObjectCreationTypeStyle                           = "roslynator_object_creation_type_style";
         public const string PrefixFieldIdentifierWithUnderscore               = "roslynator_prefix_field_identifier_with_underscore";
-        public const string SuppressUnity                                     = "roslynator_suppress_unity";
         public const string SuppressUnityScriptMethods                        = "roslynator_suppress_unity_script_methods";
         public const string TabLength                                         = "roslynator_tab_length";
         public const string TrailingCommaStyle                                = "roslynator_trailing_comma_style";
+        public const string UnityCodeAnalysisEnabled                          = "roslynator_unity_code_analysis.enabled";
         public const string UseAnonymousFunctionOrMethodGroup                 = "roslynator_use_anonymous_function_or_method_group";
         public const string UseBlockBodyWhenDeclarationSpansOverMultipleLines = "roslynator_use_block_body_when_declaration_spans_over_multiple_lines";
         public const string UseBlockBodyWhenExpressionSpansOverMultipleLines  = "roslynator_use_block_body_when_expression_spans_over_multiple_lines";
diff --git a/src/Common/ConfigOptions.Generated.cs b/src/Common/ConfigOptions.Generated.cs
index 594a84af71..0a6f922d1e 100644
--- a/src/Common/ConfigOptions.Generated.cs
+++ b/src/Common/ConfigOptions.Generated.cs
@@ -177,12 +177,6 @@ public static partial class ConfigOptions
             defaultValuePlaceholder: "true|false", 
             description:             "Prefix field identifier with underscore");
 
-        public static readonly ConfigOptionDescriptor SuppressUnity = new(
-            key:                     ConfigOptionKeys.SuppressUnity, 
-            defaultValue:            null, 
-            defaultValuePlaceholder: "true|false", 
-            description:             "Suppress false positives from Unity code");
-
         [Obsolete("", error: false)]
         public static readonly ConfigOptionDescriptor SuppressUnityScriptMethods = new(
             key:                     ConfigOptionKeys.SuppressUnityScriptMethods, 
@@ -202,6 +196,12 @@ public static partial class ConfigOptions
             defaultValuePlaceholder: "include|omit|omit_when_single_line", 
             description:             "Include/omit trailing comma in initializer or enum");
 
+        public static readonly ConfigOptionDescriptor UnityCodeAnalysisEnabled = new(
+            key:                     ConfigOptionKeys.UnityCodeAnalysisEnabled, 
+            defaultValue:            null, 
+            defaultValuePlaceholder: "true|false", 
+            description:             "Enabled additional code analysis to detect Unity-specific types");
+
         public static readonly ConfigOptionDescriptor UseAnonymousFunctionOrMethodGroup = new(
             key:                     ConfigOptionKeys.UseAnonymousFunctionOrMethodGroup, 
             defaultValue:            null, 
diff --git a/src/ConfigOptions.xml b/src/ConfigOptions.xml
index fef633c1e0..04bfd0273a 100644
--- a/src/ConfigOptions.xml
+++ b/src/ConfigOptions.xml
@@ -173,11 +173,12 @@
   <Option Id="SuppressUnityScriptMethods">
     <ValuePlaceholder>true|false</ValuePlaceholder>
     <Description>Suppress false positives from Unity script methods</Description>
-    <ObsoleteMessage>This option is obsolete, use option 'roslynator_suppress_unity' instead.</ObsoleteMessage>
+    <ObsoleteMessage>This option is obsolete, use option 'roslynator_unity_code_analysis.enabled' instead.</ObsoleteMessage>
   </Option>
-  <Option Id="SuppressUnity">
+  <Option Id="UnityCodeAnalysisEnabled">
+    <Key>unity_code_analysis.enabled</Key>
     <ValuePlaceholder>true|false</ValuePlaceholder>
-    <Description>Suppress false positives from Unity code</Description>
+    <Description>Enabled additional code analysis to detect Unity-specific types</Description>
   </Option>
   <Option Id="UseAnonymousFunctionOrMethodGroup">
     <Description>Use anonymous function or method group</Description>
diff --git a/src/Tests/Analyzers.Tests/RCS1169MakeFieldReadOnlyTests.cs b/src/Tests/Analyzers.Tests/RCS1169MakeFieldReadOnlyTests.cs
index 7fbb66d52e..b0da020306 100644
--- a/src/Tests/Analyzers.Tests/RCS1169MakeFieldReadOnlyTests.cs
+++ b/src/Tests/Analyzers.Tests/RCS1169MakeFieldReadOnlyTests.cs
@@ -397,6 +397,6 @@ class SerializeFieldAttribute : Attribute
     {
     }
 }
-", options: Options.AddConfigOption(ConfigOptionKeys.SuppressUnity, true));
+", options: Options.AddConfigOption(ConfigOptionKeys.UnityCodeAnalysisEnabled, true));
     }
 }
diff --git a/src/Tests/Analyzers.Tests/RCS1213RemoveUnusedMemberDeclarationTests.cs b/src/Tests/Analyzers.Tests/RCS1213RemoveUnusedMemberDeclarationTests.cs
index 50de829cf4..e87eb4eb6e 100644
--- a/src/Tests/Analyzers.Tests/RCS1213RemoveUnusedMemberDeclarationTests.cs
+++ b/src/Tests/Analyzers.Tests/RCS1213RemoveUnusedMemberDeclarationTests.cs
@@ -386,6 +386,6 @@ class MonoBehaviour
     {
     }
 }
-", options: Options.AddConfigOption(ConfigOptionKeys.SuppressUnity, true));
+", options: Options.AddConfigOption(ConfigOptionKeys.UnityCodeAnalysisEnabled, true));
     }
 }
diff --git a/src/VisualStudioCode/package/src/configurationFiles.generated.ts b/src/VisualStudioCode/package/src/configurationFiles.generated.ts
index b359bede94..28f88c7783 100644
--- a/src/VisualStudioCode/package/src/configurationFiles.generated.ts
+++ b/src/VisualStudioCode/package/src/configurationFiles.generated.ts
@@ -117,9 +117,6 @@ roslynator_analyzers.enabled_by_default = true|false
 
 #roslynator_prefix_field_identifier_with_underscore = true|false
 
-#roslynator_suppress_unity = true|false
-# Applicable to: rcs1169, rcs1213
-
 #roslynator_tab_length = <NUM>
 # Default: 4
 # Applicable to: rcs0056
@@ -127,6 +124,9 @@ roslynator_analyzers.enabled_by_default = true|false
 #roslynator_trailing_comma_style = include|omit|omit_when_single_line
 # Applicable to: rcs1260
 
+#roslynator_unity_code_analysis.enabled = true|false
+# Applicable to: rcs1169, rcs1213
+
 #roslynator_use_anonymous_function_or_method_group = anonymous_function|method_group
 # Applicable to: rcs1207
 
@@ -641,7 +641,7 @@ roslynator_analyzers.enabled_by_default = true|false
 
 # Make field read-only
 #dotnet_diagnostic.rcs1169.severity = suggestion
-# Options: roslynator_suppress_unity
+# Options: roslynator_unity_code_analysis.enabled
 
 # Use read-only auto-implemented property
 #dotnet_diagnostic.rcs1170.severity = suggestion
@@ -763,7 +763,7 @@ roslynator_analyzers.enabled_by_default = true|false
 
 # Remove unused member declaration
 #dotnet_diagnostic.rcs1213.severity = suggestion
-# Options: roslynator_suppress_unity, roslynator_suppress_unity_script_methods
+# Options: roslynator_suppress_unity_script_methods, roslynator_unity_code_analysis.enabled
 
 # Unnecessary interpolated string
 #dotnet_diagnostic.rcs1214.severity = suggestion

From c84dc71e135abbb16ac3f247ce642dcf81e83c4b Mon Sep 17 00:00:00 2001
From: Josef Pihrt <josef.pihrt@outlook.com>
Date: Fri, 10 Nov 2023 21:47:42 +0100
Subject: [PATCH 06/10] update

---
 src/ConfigOptions.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/ConfigOptions.xml b/src/ConfigOptions.xml
index 04bfd0273a..0d05331dc4 100644
--- a/src/ConfigOptions.xml
+++ b/src/ConfigOptions.xml
@@ -178,7 +178,7 @@
   <Option Id="UnityCodeAnalysisEnabled">
     <Key>unity_code_analysis.enabled</Key>
     <ValuePlaceholder>true|false</ValuePlaceholder>
-    <Description>Enabled additional code analysis to detect Unity-specific types</Description>
+    <Description>Enable additional analysis to detect Unity-specific code</Description>
   </Option>
   <Option Id="UseAnonymousFunctionOrMethodGroup">
     <Description>Use anonymous function or method group</Description>

From 0db08d94106918514e3666345d8f7a496ae05c1c Mon Sep 17 00:00:00 2001
From: Josef Pihrt <josef.pihrt@outlook.com>
Date: Fri, 10 Nov 2023 21:49:38 +0100
Subject: [PATCH 07/10] update

---
 src/Common/ConfigOptions.Generated.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Common/ConfigOptions.Generated.cs b/src/Common/ConfigOptions.Generated.cs
index 0a6f922d1e..f7674e8184 100644
--- a/src/Common/ConfigOptions.Generated.cs
+++ b/src/Common/ConfigOptions.Generated.cs
@@ -200,7 +200,7 @@ public static partial class ConfigOptions
             key:                     ConfigOptionKeys.UnityCodeAnalysisEnabled, 
             defaultValue:            null, 
             defaultValuePlaceholder: "true|false", 
-            description:             "Enabled additional code analysis to detect Unity-specific types");
+            description:             "Enable additional analysis to detect Unity-specific code");
 
         public static readonly ConfigOptionDescriptor UseAnonymousFunctionOrMethodGroup = new(
             key:                     ConfigOptionKeys.UseAnonymousFunctionOrMethodGroup, 

From aeb05a7feab58d57200295c11827eb45baff8be6 Mon Sep 17 00:00:00 2001
From: Josef Pihrt <josef.pihrt@outlook.com>
Date: Fri, 10 Nov 2023 22:12:43 +0100
Subject: [PATCH 08/10] update

---
 .../Analysis/MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs   | 2 +-
 .../CSharp/Analysis/UnusedMember/UnusedMemberAnalyzer.cs        | 2 +-
 src/Common/CSharp/Extensions/CodeStyleExtensions.cs             | 2 +-
 src/Tools/CodeGeneration/EditorConfig/EditorConfigGenerator.cs  | 1 +
 .../ConfigurationFileGenerator/Properties/launchSettings.json   | 2 +-
 .../package/src/configurationFiles.generated.ts                 | 2 +-
 6 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/src/Analyzers/CSharp/Analysis/MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs b/src/Analyzers/CSharp/Analysis/MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs
index c15bb467b0..683e8ec563 100644
--- a/src/Analyzers/CSharp/Analysis/MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs
+++ b/src/Analyzers/CSharp/Analysis/MakeMemberReadOnly/MakeMemberReadOnlyAnalyzer.cs
@@ -137,7 +137,7 @@ private static void AnalyzeTypeDeclaration(
                                 && !fieldSymbol.IsReadOnly
                                 && !fieldSymbol.IsVolatile
                                 && ValidateType(fieldSymbol.Type)
-                                && (context.GetUnityCodeAnalysisEnabled() != true
+                                && (context.IsUnityCodeAnalysisEnabled() != true
                                     || !fieldSymbol.HasAttribute(UnityEngine_SerializeFieldAttribute)))
                             {
                                 symbols[fieldSymbol.Name] = (declarator, fieldSymbol);
diff --git a/src/Analyzers/CSharp/Analysis/UnusedMember/UnusedMemberAnalyzer.cs b/src/Analyzers/CSharp/Analysis/UnusedMember/UnusedMemberAnalyzer.cs
index f588452f7b..2f3a8e80b2 100644
--- a/src/Analyzers/CSharp/Analysis/UnusedMember/UnusedMemberAnalyzer.cs
+++ b/src/Analyzers/CSharp/Analysis/UnusedMember/UnusedMemberAnalyzer.cs
@@ -159,7 +159,7 @@ private static void AnalyzeTypeDeclaration(SyntaxNodeAnalysisContext context)
                             break;
 
                         if (declaration.ReturnsVoid()
-                            && context.GetUnityCodeAnalysisEnabled() == true)
+                            && context.IsUnityCodeAnalysisEnabled() == true)
                         {
                             if (canContainUnityScriptMethods is null)
                             {
diff --git a/src/Common/CSharp/Extensions/CodeStyleExtensions.cs b/src/Common/CSharp/Extensions/CodeStyleExtensions.cs
index f6b032d627..d16661b50b 100644
--- a/src/Common/CSharp/Extensions/CodeStyleExtensions.cs
+++ b/src/Common/CSharp/Extensions/CodeStyleExtensions.cs
@@ -565,7 +565,7 @@ public static BlankLineStyle GetBlankLineAfterFileScopedNamespaceDeclaration(thi
         return BlankLineStyle.None;
     }
 
-    public static bool? GetUnityCodeAnalysisEnabled(this SyntaxNodeAnalysisContext context)
+    public static bool? IsUnityCodeAnalysisEnabled(this SyntaxNodeAnalysisContext context)
     {
         if (ConfigOptions.TryGetValueAsBool(context.GetConfigOptions(), ConfigOptions.UnityCodeAnalysisEnabled, out bool value))
             return value;
diff --git a/src/Tools/CodeGeneration/EditorConfig/EditorConfigGenerator.cs b/src/Tools/CodeGeneration/EditorConfig/EditorConfigGenerator.cs
index bbb72c79e2..1a4bfba0d2 100644
--- a/src/Tools/CodeGeneration/EditorConfig/EditorConfigGenerator.cs
+++ b/src/Tools/CodeGeneration/EditorConfig/EditorConfigGenerator.cs
@@ -88,6 +88,7 @@ public static string GenerateEditorConfig(RoslynatorMetadata metadata, bool comm
                         + string.Join(
                             ", ",
                             analyzer.ConfigOptions
+                                .Where(f => metadata.ConfigOptions.FirstOrDefault(ff => ff.Key == f.Key)?.IsObsolete != true)
                                 .OrderBy(f => f.Key)
                                 .Select(f2 => metadata.ConfigOptions.First(f => f.Key == f2.Key).Key)));
                 }
diff --git a/src/Tools/ConfigurationFileGenerator/Properties/launchSettings.json b/src/Tools/ConfigurationFileGenerator/Properties/launchSettings.json
index fb7f3bf2ab..c539c89264 100644
--- a/src/Tools/ConfigurationFileGenerator/Properties/launchSettings.json
+++ b/src/Tools/ConfigurationFileGenerator/Properties/launchSettings.json
@@ -2,7 +2,7 @@
   "profiles": {
     "CodeGenerator": {
       "commandName": "Project",
-      "commandLineArgs": "\"../../../../..\""
+      "commandLineArgs": "\"../../../../..\" \"../../../../../Tools/ConfigurationFileGenerator/configuration.md\" \"../../../../../../tools/build/configuration.md\""
     }
   }
 }
\ No newline at end of file
diff --git a/src/VisualStudioCode/package/src/configurationFiles.generated.ts b/src/VisualStudioCode/package/src/configurationFiles.generated.ts
index 28f88c7783..25d6d21ff5 100644
--- a/src/VisualStudioCode/package/src/configurationFiles.generated.ts
+++ b/src/VisualStudioCode/package/src/configurationFiles.generated.ts
@@ -763,7 +763,7 @@ roslynator_analyzers.enabled_by_default = true|false
 
 # Remove unused member declaration
 #dotnet_diagnostic.rcs1213.severity = suggestion
-# Options: roslynator_suppress_unity_script_methods, roslynator_unity_code_analysis.enabled
+# Options: roslynator_unity_code_analysis.enabled
 
 # Unnecessary interpolated string
 #dotnet_diagnostic.rcs1214.severity = suggestion

From cbd5103b0e3044a23c01c0a66eb3f9fa19f11d31 Mon Sep 17 00:00:00 2001
From: Josef Pihrt <josef.pihrt@outlook.com>
Date: Fri, 10 Nov 2023 23:09:34 +0100
Subject: [PATCH 09/10] update

---
 ChangeLog.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ChangeLog.md b/ChangeLog.md
index 68d8eb93e7..adcc1bfb15 100644
--- a/ChangeLog.md
+++ b/ChangeLog.md
@@ -18,7 +18,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - [CLI] Fix globbing ([PR](https://github.com/dotnet/roslynator/pull/1238))
 - [CLI] Remove assembly resolving ([PR](https://github.com/dotnet/roslynator/pull/1237))
 - Detect false positive from Unity code ([RCS1169](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1169)) ([PR](https://github.com/dotnet/roslynator/pull/1245))
-  - Introduce config option `roslynator_unity_code_analysis.enabled`
+  - Introduce config option `roslynator_unity_code_analysis.enabled = true|false`
   - Make option `roslynator_suppress_unity_script_methods` obsolete
 
 ## [4.6.1] - 2023-10-23

From 99135d423c6559726014bff285dfc64f6a85bc3b Mon Sep 17 00:00:00 2001
From: Josef Pihrt <josef.pihrt@outlook.com>
Date: Fri, 10 Nov 2023 23:19:34 +0100
Subject: [PATCH 10/10] update

---
 src/Common/ConfigOptions.Generated.cs                  | 4 ++--
 src/ConfigOptions.xml                                  | 9 ++++-----
 src/Tools/CodeGeneration/Markdown/MarkdownGenerator.cs | 9 ++-------
 src/Tools/Metadata/AnalyzerOptionMetadata.cs           | 4 +---
 src/Tools/Metadata/MetadataFile.cs                     | 2 +-
 5 files changed, 10 insertions(+), 18 deletions(-)

diff --git a/src/Common/ConfigOptions.Generated.cs b/src/Common/ConfigOptions.Generated.cs
index f7674e8184..8fe858adf9 100644
--- a/src/Common/ConfigOptions.Generated.cs
+++ b/src/Common/ConfigOptions.Generated.cs
@@ -182,7 +182,7 @@ public static partial class ConfigOptions
             key:                     ConfigOptionKeys.SuppressUnityScriptMethods, 
             defaultValue:            null, 
             defaultValuePlaceholder: "true|false", 
-            description:             "Suppress false positives from Unity script methods");
+            description:             "[deprecated] This option is obsolete, use option 'roslynator_unity_code_analysis.enabled' instead.");
 
         public static readonly ConfigOptionDescriptor TabLength = new(
             key:                     ConfigOptionKeys.TabLength, 
@@ -200,7 +200,7 @@ public static partial class ConfigOptions
             key:                     ConfigOptionKeys.UnityCodeAnalysisEnabled, 
             defaultValue:            null, 
             defaultValuePlaceholder: "true|false", 
-            description:             "Enable additional analysis to detect Unity-specific code");
+            description:             "Enable code analysis to detect Unity-specific code");
 
         public static readonly ConfigOptionDescriptor UseAnonymousFunctionOrMethodGroup = new(
             key:                     ConfigOptionKeys.UseAnonymousFunctionOrMethodGroup, 
diff --git a/src/ConfigOptions.xml b/src/ConfigOptions.xml
index 0d05331dc4..c492b771a8 100644
--- a/src/ConfigOptions.xml
+++ b/src/ConfigOptions.xml
@@ -170,15 +170,14 @@
     <ValuePlaceholder>true|false</ValuePlaceholder>
     <Description>Prefix field identifier with underscore</Description>
   </Option>
-  <Option Id="SuppressUnityScriptMethods">
+  <Option Id="SuppressUnityScriptMethods" IsObsolete="true">
     <ValuePlaceholder>true|false</ValuePlaceholder>
-    <Description>Suppress false positives from Unity script methods</Description>
-    <ObsoleteMessage>This option is obsolete, use option 'roslynator_unity_code_analysis.enabled' instead.</ObsoleteMessage>
+    <Description>[deprecated] This option is obsolete, use option 'roslynator_unity_code_analysis.enabled' instead.</Description>
   </Option>
   <Option Id="UnityCodeAnalysisEnabled">
     <Key>unity_code_analysis.enabled</Key>
-    <ValuePlaceholder>true|false</ValuePlaceholder>
-    <Description>Enable additional analysis to detect Unity-specific code</Description>
+    <ValuePlaceholder>true|false</ValuePlaceholder> 
+    <Description>Enable code analysis to detect Unity-specific code</Description>
   </Option>
   <Option Id="UseAnonymousFunctionOrMethodGroup">
     <Description>Use anonymous function or method group</Description>
diff --git a/src/Tools/CodeGeneration/Markdown/MarkdownGenerator.cs b/src/Tools/CodeGeneration/Markdown/MarkdownGenerator.cs
index 9f5c92753b..315d2f5a8f 100644
--- a/src/Tools/CodeGeneration/Markdown/MarkdownGenerator.cs
+++ b/src/Tools/CodeGeneration/Markdown/MarkdownGenerator.cs
@@ -210,13 +210,8 @@ static IEnumerable<object> CreateConfiguration(AnalyzerMetadata analyzer, Immuta
 
                         sb.Append("# ");
 
-                        string description = en.Current.Description;
-
-                        if (en.Current.IsObsolete)
-                            description = "[deprecated] " + en.Current.ObsoleteMessage;
-
-                        if (!string.IsNullOrEmpty(description))
-                            sb.AppendLine(description);
+                        if (!string.IsNullOrEmpty(en.Current.Description))
+                            sb.AppendLine(en.Current.Description);
 
                         string defaultValue = en.Current.DefaultValue;
 
diff --git a/src/Tools/Metadata/AnalyzerOptionMetadata.cs b/src/Tools/Metadata/AnalyzerOptionMetadata.cs
index fba31aeec4..3400609f43 100644
--- a/src/Tools/Metadata/AnalyzerOptionMetadata.cs
+++ b/src/Tools/Metadata/AnalyzerOptionMetadata.cs
@@ -4,9 +4,7 @@
 
 namespace Roslynator.Metadata;
 
-public record AnalyzerOptionMetadata(string Id, string Key, string DefaultValue, string DefaultValuePlaceholder, string Description, string ObsoleteMessage)
+public record AnalyzerOptionMetadata(string Id, string Key, string DefaultValue, string DefaultValuePlaceholder, string Description, bool IsObsolete)
 {
     public List<AnalyzerOptionValueMetadata> Values { get; } = new();
-
-    public bool IsObsolete => !string.IsNullOrEmpty(ObsoleteMessage);
 }
diff --git a/src/Tools/Metadata/MetadataFile.cs b/src/Tools/Metadata/MetadataFile.cs
index d6b1f9ab04..0214fe8a84 100644
--- a/src/Tools/Metadata/MetadataFile.cs
+++ b/src/Tools/Metadata/MetadataFile.cs
@@ -291,7 +291,7 @@ public static IEnumerable<AnalyzerOptionMetadata> ReadOptions(string filePath)
                 DefaultValue: defaultValue,
                 DefaultValuePlaceholder: defaultValuePlaceholder,
                 Description: element.Element("Description").Value,
-                ObsoleteMessage: element.Element("ObsoleteMessage")?.Value
+                IsObsolete: element.AttributeValueAsBooleanOrDefault("IsObsolete")
             );
 
             analyzerOption.Values.AddRange(values ?? Enumerable.Empty<AnalyzerOptionValueMetadata>());