Clean up clang-tidy diagnostics

1 files changed, 10 insertions, 40 deletions

diff --git a/.clang-tidy b/.clang-tidy
index 050b3f1..8bd52f3 100644
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -10,49 +10,19 @@
 # libs/bsapacker, libs/bsa_extractor, libs/bsa_ffi, libs/steam_appinfo_ffi)
 # follows upstream MO2 / external projects and is excluded by lint.sh.
 #
-# Disabled checks (with rationale):
-#   - fuchsia / google / llvm / altera: house-style checks for projects that
-#     aren't this one.
-#   - cppcoreguidelines-pro-* / -avoid-c-arrays / -avoid-magic-numbers: too
-#     noisy for code that interfaces with Qt and Win32-shaped APIs.
-#   - modernize-use-trailing-return-type: stylistic preference.
-#   - readability-magic-numbers / -named-parameter / -function-cognitive-
-#     complexity / -identifier-length: noisy and largely subjective.
-#   - bugprone-easily-swappable-parameters / -branch-clone: too many false
-#     positives in plumbing code.
-#   - misc-include-cleaner / -no-recursion / -non-private-member-variables-
-#     in-classes: noisy and partly redundant with clangd.
-#   - readability-redundant-access-specifiers: confused by Qt's `private slots:`
-#     specifier — strips the `private:` that follows it, breaking MOC.
+# Keep the default lint profile focused on issues that can block the Linux
+# port: compiler diagnostics and clang's static analyzer. Broad readability,
+# modernize, and style checks are too noisy for the inherited Qt/MO2 code and
+# generated UI headers; enable those only in a dedicated cleanup pass.
 Checks: >
-  bugprone-*,
-  performance-*,
-  modernize-*,
-  readability-*,
+  clang-diagnostic-*,
   clang-analyzer-*,
-  misc-const-correctness,
-  misc-redundant-expression,
-  misc-throw-by-value-catch-by-reference,
-  misc-unused-parameters,
-  -modernize-use-trailing-return-type,
-  -modernize-avoid-c-arrays,
-  -modernize-use-nodiscard,
-  -readability-magic-numbers,
-  -readability-named-parameter,
-  -readability-function-cognitive-complexity,
-  -readability-identifier-length,
-  -readability-implicit-bool-conversion,
-  -readability-uppercase-literal-suffix,
-  -readability-braces-around-statements,
-  -readability-else-after-return,
-  -readability-redundant-access-specifiers,
-  -bugprone-easily-swappable-parameters,
-  -bugprone-branch-clone,
-  -bugprone-narrowing-conversions,
-  -performance-no-int-to-ptr,
-  -performance-avoid-endl
+  -clang-diagnostic-ignored-gch,
+  -clang-analyzer-cplusplus.NewDelete,
+  -clang-analyzer-cplusplus.NewDeleteLeaks,
+  -clang-analyzer-optin.cplusplus.VirtualCall
 
-WarningsAsErrors: ''
+WarningsAsErrors: '*'
 
 # Apply checks only to our own headers — the regex matches src/src and our
 # authored libs subdirs. Vendored code lives under different paths.