C#: Add all medium precision quality queries to code-quality-extended.

michaelnebel · michaelnebel · commit 8edb66fe2ac0 · 2025-08-26T15:33:15.000+02:00
diff --git a/csharp/ql/src/API Abuse/ClassDoesNotImplementEquals.ql b/csharp/ql/src/API Abuse/ClassDoesNotImplementEquals.ql
@@ -7,7 +7,8 @@
  * @problem.severity error
  * @precision medium
  * @id cs/class-missing-equals
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       maintainability
  */
 
diff --git a/csharp/ql/src/API Abuse/DisposeNotCalledOnException.ql b/csharp/ql/src/API Abuse/DisposeNotCalledOnException.ql
@@ -7,7 +7,8 @@
  * @problem.severity warning
  * @precision medium
  * @id cs/dispose-not-called-on-throw
- * @tags efficiency
+ * @tags quality
+ *       efficiency
  *       maintainability
  *       external/cwe/cwe-404
  *       external/cwe/cwe-459
diff --git a/csharp/ql/src/API Abuse/InconsistentEqualsGetHashCode.ql b/csharp/ql/src/API Abuse/InconsistentEqualsGetHashCode.ql
@@ -6,7 +6,8 @@
  * @problem.severity warning
  * @precision medium
  * @id cs/inconsistent-equals-and-gethashcode
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       maintainability
  *       external/cwe/cwe-581
  */
diff --git a/csharp/ql/src/API Abuse/IncorrectCompareToSignature.ql b/csharp/ql/src/API Abuse/IncorrectCompareToSignature.ql
@@ -5,7 +5,8 @@
  * @problem.severity warning
  * @precision medium
  * @id cs/wrong-compareto-signature
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       maintainability
  */
 
diff --git a/csharp/ql/src/API Abuse/IncorrectEqualsSignature.ql b/csharp/ql/src/API Abuse/IncorrectEqualsSignature.ql
@@ -5,7 +5,8 @@
  * @problem.severity warning
  * @precision medium
  * @id cs/wrong-equals-signature
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       maintainability
  */
 
diff --git a/csharp/ql/src/API Abuse/NonOverridingMethod.ql b/csharp/ql/src/API Abuse/NonOverridingMethod.ql
@@ -5,7 +5,8 @@
  * @problem.severity recommendation
  * @precision medium
  * @id cs/nonoverriding-method
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       readability
  *       naming
  */
diff --git a/csharp/ql/src/ASP/SplitControlStructure.ql b/csharp/ql/src/ASP/SplitControlStructure.ql
@@ -5,7 +5,8 @@
  * @problem.severity recommendation
  * @precision medium
  * @id cs/asp/split-control-structure
- * @tags maintainability
+ * @tags quality
+ *       maintainability
  *       frameworks/asp.net
  */
 
diff --git a/csharp/ql/src/Bad Practices/Comments/CommentedOutCode.ql b/csharp/ql/src/Bad Practices/Comments/CommentedOutCode.ql
@@ -5,7 +5,8 @@
  * @problem.severity recommendation
  * @precision medium
  * @id cs/commented-out-code
- * @tags maintainability
+ * @tags quality
+ *       maintainability
  *       statistical
  *       non-attributable
  */
diff --git a/csharp/ql/src/Bad Practices/Comments/TodoComments.ql b/csharp/ql/src/Bad Practices/Comments/TodoComments.ql
@@ -6,7 +6,8 @@
  * @problem.severity recommendation
  * @precision medium
  * @id cs/todo-comment
- * @tags maintainability
+ * @tags quality
+ *       maintainability
  *       external/cwe/cwe-546
  */
 
diff --git a/csharp/ql/src/Bad Practices/Declarations/NoConstantsOnly.ql b/csharp/ql/src/Bad Practices/Declarations/NoConstantsOnly.ql
@@ -6,7 +6,8 @@
  * @problem.severity recommendation
  * @precision medium
  * @id cs/constants-only-interface
- * @tags maintainability
+ * @tags quality
+ *       maintainability
  *       modularity
  */
 
diff --git a/csharp/ql/src/Bad Practices/ErroneousClassCompare.ql b/csharp/ql/src/Bad Practices/ErroneousClassCompare.ql
@@ -5,7 +5,8 @@
  * @problem.severity warning
  * @precision medium
  * @id cs/class-name-comparison
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       correctness
  *       external/cwe/cwe-486
  */
diff --git a/csharp/ql/src/Bad Practices/Implementation Hiding/AbstractToConcreteCollection.ql b/csharp/ql/src/Bad Practices/Implementation Hiding/AbstractToConcreteCollection.ql
@@ -7,7 +7,8 @@
  * @problem.severity warning
  * @precision medium
  * @id cs/cast-from-abstract-to-concrete-collection
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       maintainability
  *       modularity
  *       external/cwe/cwe-485
diff --git a/csharp/ql/src/Bad Practices/Implementation Hiding/StaticArray.ql b/csharp/ql/src/Bad Practices/Implementation Hiding/StaticArray.ql
@@ -5,7 +5,8 @@
  * @problem.severity recommendation
  * @precision medium
  * @id cs/static-array
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       maintainability
  *       modularity
  *       external/cwe/cwe-582
diff --git a/csharp/ql/src/Bad Practices/Naming Conventions/ConfusingMethodNames.ql b/csharp/ql/src/Bad Practices/Naming Conventions/ConfusingMethodNames.ql
@@ -5,7 +5,8 @@
  * @problem.severity recommendation
  * @precision medium
  * @id cs/confusing-method-name
- * @tags maintainability
+ * @tags quality
+ *       maintainability
  *       readability
  *       naming
  */
diff --git a/csharp/ql/src/Bad Practices/Naming Conventions/ConfusingOverridesNames.ql b/csharp/ql/src/Bad Practices/Naming Conventions/ConfusingOverridesNames.ql
@@ -6,7 +6,8 @@
  * @problem.severity recommendation
  * @precision medium
  * @id cs/confusing-override-name
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       readability
  *       naming
  */
diff --git a/csharp/ql/src/Bad Practices/Naming Conventions/ControlNamePrefixes.ql b/csharp/ql/src/Bad Practices/Naming Conventions/ControlNamePrefixes.ql
@@ -6,7 +6,8 @@
  * @problem.severity recommendation
  * @precision medium
  * @id cs/web/unprefixed-control-name
- * @tags maintainability
+ * @tags quality
+ *       maintainability
  */
 
 import csharp
diff --git a/csharp/ql/src/Bad Practices/Naming Conventions/DefaultControlNames.ql b/csharp/ql/src/Bad Practices/Naming Conventions/DefaultControlNames.ql
@@ -6,7 +6,8 @@
  * @problem.severity recommendation
  * @precision medium
  * @id cs/forms/default-control-name
- * @tags readability
+ * @tags quality
+ *       readability
  *       naming
  */
 
diff --git a/csharp/ql/src/Bad Practices/UseOfSystemOutputStream.ql b/csharp/ql/src/Bad Practices/UseOfSystemOutputStream.ql
@@ -5,7 +5,8 @@
  * @problem.severity recommendation
  * @precision medium
  * @id cs/console-output
- * @tags maintainability
+ * @tags quality
+ *       maintainability
  */
 
 import csharp
diff --git a/csharp/ql/src/Bad Practices/VirtualCallInConstructorOrDestructor.ql b/csharp/ql/src/Bad Practices/VirtualCallInConstructorOrDestructor.ql
@@ -6,7 +6,8 @@
  * @precision medium
  * @id cs/virtual-call-in-constructor
  * @alternate-ids cs/virtual-call-in-constructor-or-destructor
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       maintainability
  *       modularity
  */
diff --git a/csharp/ql/src/Concurrency/SynchSetUnsynchGet.ql b/csharp/ql/src/Concurrency/SynchSetUnsynchGet.ql
@@ -6,7 +6,8 @@
  * @problem.severity error
  * @precision medium
  * @id cs/unsynchronized-getter
- * @tags correctness
+ * @tags quality
+ *       correctness
  *       concurrency
  *       external/cwe/cwe-662
  */
diff --git a/csharp/ql/src/Concurrency/UnsafeLazyInitialization.ql b/csharp/ql/src/Concurrency/UnsafeLazyInitialization.ql
@@ -6,7 +6,8 @@
  * @problem.severity error
  * @precision medium
  * @id cs/unsafe-double-checked-lock
- * @tags correctness
+ * @tags quality
+ *       correctness
  *       concurrency
  *       external/cwe/cwe-609
  */
diff --git a/csharp/ql/src/Concurrency/UnsynchronizedStaticAccess.ql b/csharp/ql/src/Concurrency/UnsynchronizedStaticAccess.ql
@@ -6,7 +6,8 @@
  * @problem.severity error
  * @precision medium
  * @id cs/unsynchronized-static-access
- * @tags concurrency
+ * @tags quality
+ *       concurrency
  *       external/cwe/cwe-362
  *       external/cwe/cwe-567
  */
diff --git a/csharp/ql/src/Documentation/XmldocExtraParam.ql b/csharp/ql/src/Documentation/XmldocExtraParam.ql
@@ -6,7 +6,8 @@
  * @problem.severity recommendation
  * @precision medium
  * @id cs/xmldoc/unknown-parameter
- * @tags maintainability
+ * @tags quality
+ *       maintainability
  */
 
 import Documentation
diff --git a/csharp/ql/src/Documentation/XmldocExtraTypeParam.ql b/csharp/ql/src/Documentation/XmldocExtraTypeParam.ql
@@ -6,7 +6,8 @@
  * @problem.severity recommendation
  * @precision medium
  * @id cs/xmldoc/unknown-type-parameter
- * @tags maintainability
+ * @tags quality
+ *       maintainability
  */
 
 import Documentation
diff --git a/csharp/ql/src/Documentation/XmldocMissing.ql b/csharp/ql/src/Documentation/XmldocMissing.ql
@@ -6,7 +6,8 @@
  * @problem.severity recommendation
  * @precision medium
  * @id cs/xmldoc/missing-xmldoc
- * @tags maintainability
+ * @tags quality
+ *       maintainability
  */
 
 import Documentation
diff --git a/csharp/ql/src/Language Abuse/ForeachCapture.ql b/csharp/ql/src/Language Abuse/ForeachCapture.ql
@@ -5,7 +5,8 @@
  * @problem.severity recommendation
  * @precision medium
  * @id cs/captured-foreach-variable
- * @tags portability
+ * @tags quality
+ *       portability
  *       maintainability
  *       language-features
  *       external/cwe/cwe-758
diff --git a/csharp/ql/src/Language Abuse/UselessCastToSelf.ql b/csharp/ql/src/Language Abuse/UselessCastToSelf.ql
@@ -5,7 +5,8 @@
  * @problem.severity warning
  * @precision medium
  * @id cs/useless-cast-to-self
- * @tags maintainability
+ * @tags quality
+ *       maintainability
  *       language-features
  *       external/cwe/cwe-561
  */
diff --git a/csharp/ql/src/Language Abuse/UselessIsBeforeAs.ql b/csharp/ql/src/Language Abuse/UselessIsBeforeAs.ql
@@ -5,7 +5,8 @@
  * @problem.severity recommendation
  * @precision medium
  * @id cs/useless-is-before-as
- * @tags maintainability
+ * @tags quality
+ *       maintainability
  *       language-features
  *       external/cwe/cwe-561
  */
diff --git a/csharp/ql/src/Language Abuse/UselessNullCoalescingExpression.ql b/csharp/ql/src/Language Abuse/UselessNullCoalescingExpression.ql
@@ -6,7 +6,8 @@
  * @problem.severity error
  * @precision medium
  * @id cs/coalesce-of-identical-expressions
- * @tags maintainability
+ * @tags quality
+ *       maintainability
  *       language-features
  *       external/cwe/cwe-561
  */
diff --git a/csharp/ql/src/Language Abuse/UselessTypeTest.ql b/csharp/ql/src/Language Abuse/UselessTypeTest.ql
@@ -5,7 +5,8 @@
  * @problem.severity warning
  * @precision medium
  * @id cs/useless-type-test
- * @tags maintainability
+ * @tags quality
+ *       maintainability
  *       language-features
  *       external/cwe/cwe-561
  */
diff --git a/csharp/ql/src/Language Abuse/UselessUpcast.ql b/csharp/ql/src/Language Abuse/UselessUpcast.ql
@@ -5,7 +5,8 @@
  * @problem.severity warning
  * @precision medium
  * @id cs/useless-upcast
- * @tags maintainability
+ * @tags quality
+ *       maintainability
  *       language-features
  *       external/cwe/cwe-561
  */
diff --git a/csharp/ql/src/Likely Bugs/Dynamic/BadDynamicCall.ql b/csharp/ql/src/Likely Bugs/Dynamic/BadDynamicCall.ql
@@ -5,7 +5,8 @@
  * @problem.severity error
  * @precision medium
  * @id cs/invalid-dynamic-call
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       correctness
  *       logic
  *       external/cwe/cwe-628
diff --git a/csharp/ql/src/Likely Bugs/EqualsUsesAs.ql b/csharp/ql/src/Likely Bugs/EqualsUsesAs.ql
@@ -5,7 +5,8 @@
  * @problem.severity warning
  * @precision medium
  * @id cs/equals-uses-as
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       maintainability
  */
 
diff --git a/csharp/ql/src/Likely Bugs/EqualsUsesIs.ql b/csharp/ql/src/Likely Bugs/EqualsUsesIs.ql
@@ -5,7 +5,8 @@
  * @problem.severity warning
  * @precision medium
  * @id cs/equals-uses-is
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       maintainability
  */
 
diff --git a/csharp/ql/src/Likely Bugs/InconsistentCompareTo.ql b/csharp/ql/src/Likely Bugs/InconsistentCompareTo.ql
@@ -5,7 +5,8 @@
  * @problem.severity warning
  * @precision medium
  * @id cs/inconsistent-compareto-and-equals
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       maintainability
  */
 
diff --git a/csharp/ql/src/Likely Bugs/LeapYear/UnsafeYearConstruction.ql b/csharp/ql/src/Likely Bugs/LeapYear/UnsafeYearConstruction.ql
@@ -5,7 +5,8 @@
  * @problem.severity warning
  * @precision medium
  * @id cs/unsafe-year-construction
- * @tags date-time
+ * @tags quality
+ *       date-time
  *       reliability
  */
 
diff --git a/csharp/ql/src/Likely Bugs/MishandlingJapaneseEra.ql b/csharp/ql/src/Likely Bugs/MishandlingJapaneseEra.ql
@@ -5,7 +5,8 @@
  * @kind problem
  * @problem.severity warning
  * @precision medium
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       date-time
  */
 
diff --git a/csharp/ql/src/Likely Bugs/ObjectComparison.ql b/csharp/ql/src/Likely Bugs/ObjectComparison.ql
@@ -6,7 +6,8 @@
  * @problem.severity warning
  * @precision medium
  * @id cs/reference-equality-with-object
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       correctness
  *       external/cwe/cwe-595
  */
diff --git a/csharp/ql/src/Likely Bugs/RecursiveOperatorEquals.ql b/csharp/ql/src/Likely Bugs/RecursiveOperatorEquals.ql
@@ -5,7 +5,8 @@
  * @problem.severity error
  * @precision medium
  * @id cs/recursive-operator-equals-call
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       maintainability
  */
 
diff --git a/csharp/ql/src/Likely Bugs/Statements/UseBraces.ql b/csharp/ql/src/Likely Bugs/Statements/UseBraces.ql
@@ -6,7 +6,8 @@
  * @problem.severity warning
  * @precision medium
  * @id cs/misleading-indentation
- * @tags changeability
+ * @tags quality
+ *       changeability
  *       correctness
  *       logic
  */
diff --git a/csharp/ql/src/Linq/BadMultipleIteration.ql b/csharp/ql/src/Linq/BadMultipleIteration.ql
diff --git a/csharp/ql/src/Useless code/PointlessForwardingMethod.ql b/csharp/ql/src/Useless code/PointlessForwardingMethod.ql
