Ignore unowned(unsafe) captures by default

CHANGELOG.md

@@ -22,6 +22,12 @@
 
 ### Bug Fixes
 
+* Make `unowned_variable_capture` ignore `unowned(unsafe)` captures by
+  default and add an `include_unsafe` option for projects that still want them
+  reported.  
+  [Yurii Bakurov](https://github.com/Yurii201811)
+  [#6817](https://github.com/realm/SwiftLint/issues/6817)
+
 * Fix baseline writing to store file locations as paths relative to the current working directory,
   restoring baseline portability and avoiding absolute `file://` paths in generated baseline files.  
   [SimplyDanny](https://github.com/SimplyDanny)

Source/SwiftLintBuiltInRules/Rules/Lint/UnownedVariableCaptureRule.swift

@@ -3,7 +3,7 @@ import SwiftSyntax
 
 @SwiftSyntaxRule(optIn: true)
 struct UnownedVariableCaptureRule: Rule {
-    var configuration = SeverityConfiguration<Self>(.warning)
+    var configuration = UnownedVariableCaptureConfiguration()
 
     static let description = RuleDescription(
         identifier: "unowned_variable_capture",
@@ -15,6 +15,7 @@ struct UnownedVariableCaptureRule: Rule {
             "foo { [weak self] param in _ }",
             "foo { [weak bar] in _ }",
             "foo { [weak bar] param in _ }",
+            "foo { [unowned(unsafe) self] in _ }",
             "foo { bar in _ }",
             "foo { $0 }",
             """
@@ -30,15 +31,24 @@ struct UnownedVariableCaptureRule: Rule {
         triggeringExamples: #examples([
             "foo { [↓unowned self] in _ }",
             "foo { [↓unowned bar] in _ }",
+            "foo { [↓unowned(safe) self] in _ }",
             "foo { [bar, ↓unowned self] in _ }",
+            "foo { [↓unowned(unsafe) self] in _ }"
+                .asExample(configuration: ["include_unsafe": true]),
         ])
     )
 }
 
 private extension UnownedVariableCaptureRule {
     final class Visitor: ViolationsSyntaxVisitor<ConfigurationType> {
         override func visitPost(_ node: TokenSyntax) {
-            if case .keyword(.unowned) = node.tokenKind, node.parent?.is(ClosureCaptureSpecifierSyntax.self) == true {
+            guard case .keyword(.unowned) = node.tokenKind,
+                  let specifier = node.parent?.as(ClosureCaptureSpecifierSyntax.self) else {
+                return
+            }
+
+            let isUnsafe = specifier.tokens(viewMode: .sourceAccurate).contains { $0.text == "unsafe" }
+            if !isUnsafe || configuration.includeUnsafe {
                 violations.append(node.positionAfterSkippingLeadingTrivia)
             }
         }

Source/SwiftLintBuiltInRules/Rules/RuleConfigurations/UnownedVariableCaptureConfiguration.swift

@@ -0,0 +1,9 @@
+import SwiftLintCore
+
+@AutoConfigParser
+struct UnownedVariableCaptureConfiguration: SeverityBasedRuleConfiguration {
+    @ConfigurationElement(key: "severity")
+    private(set) var severityConfiguration = SeverityConfiguration<Parent>(.warning)
+    @ConfigurationElement(key: "include_unsafe")
+    private(set) var includeUnsafe = false
+}

Tests/IntegrationTests/Resources/default_rule_configurations.yml

@@ -1335,6 +1335,7 @@ unneeded_throws_rethrows:
     correctable: true
 unowned_variable_capture:
   severity: warning
+  include_unsafe: false
   meta:
     opt-in: true
     correctable: false