feat: add SwiftLint workflow and configure rules for snapshot tests

Author: coenttb

.github/workflows/swiftlint.yml

@@ -0,0 +1,48 @@
+name: SwiftLint
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+    branches:
+      - main
+
+concurrency:
+  group: swiftlint-${{ github.ref }}
+  cancel-in-progress: true
+
+jobs:
+  lint:
+    name: SwiftLint
+    runs-on: ubuntu-latest
+    container: swift:6.2
+    steps:
+      - uses: actions/checkout@v5
+
+      - name: Restore swiftlint cache
+        id: cache-swiftlint-restore
+        uses: actions/cache/restore@v4
+        with:
+          path: /usr/local/bin/swiftlint
+          key: ${{ runner.os }}-swiftlint-v1
+
+      - name: Install swiftlint
+        if: steps.cache-swiftlint-restore.outputs.cache-hit != 'true'
+        run: |
+          git clone --depth 1 --branch main https://github.com/realm/SwiftLint.git
+          cd SwiftLint
+          swift build -c release
+          cp .build/release/swiftlint /usr/local/bin/
+          cd ..
+          rm -rf SwiftLint
+
+      - name: Lint
+        run: swiftlint lint --strict --reporter github-actions-logging
+
+      - name: Save swiftlint cache
+        uses: actions/cache/save@v4
+        if: steps.cache-swiftlint-restore.outputs.cache-hit != 'true'
+        with:
+          path: /usr/local/bin/swiftlint
+          key: ${{ runner.os }}-swiftlint-v1

.swiftlint.yml

@@ -3,41 +3,42 @@ disabled_rules:
   - identifier_name
   - nesting  # Allow deeply nested enums for namespacing
   - redundant_discardable_let  # Preserve 'let _ =' in result builder contexts (required for Swift 6.0+)
+  - opening_brace  # Handled by swift-format
+  - trailing_comma  # Handled by swift-format
+  - trailing_whitespace  # Preserve whitespace in snapshot tests
+  - orphaned_doc_comment  # Allow module-level documentation
+  - large_tuple  # CSS/HTML properties may require tuples with multiple components
+  - mark  # Allow flexible MARK comment formatting
+  - redundant_string_enum_value  # Explicit enum values improve clarity for CSS/HTML keywords
 
 opt_in_rules:
   - empty_count
   - explicit_init
   - sorted_imports
   - force_cast
   - force_try
-  - force_unwrapping
   - closure_spacing
   - operator_usage_whitespace
   - private_outlet
   - redundant_nil_coalescing
-  - unused_capture_list
-  - empty_string
-  - explicit_enum_raw_value
   - literal_expression_end_indentation
   - single_test_class
   - sorted_first_last
   - vertical_whitespace
-  - strict_fileprivate
   - legacy_random
   - no_extension_access_modifier
   - colon
-  - trailing_comma
 
 included:
   - Package.swift
   - Sources
+  - Tests
 
 excluded:
   - Carthage
   - Pods
   - fastlane
   - build
-  - Tests
 
 analyzer_rules:
   - unused_import
@@ -47,3 +48,35 @@ colon:
 
 vertical_whitespace:
   max_empty_lines: 1
+
+# Increased length limits for test files with extensive snapshots
+type_body_length:
+  warning: 800
+  error: 900
+
+file_length:
+  warning: 900
+  error: 1000
+
+function_body_length:
+  warning: 270
+  error: 300
+
+# Allow short type names for HTML/CSS elements (H1, BR, B, RGB, HSL, etc.)
+type_name:
+  min_length:
+    warning: 1
+    error: 0
+  max_length:
+    warning: 40
+    error: 50
+
+# Allow more function parameters for HTML/CSS functions with many attributes
+function_parameter_count:
+  warning: 8
+  error: 10
+
+# Increased cyclomatic complexity for CSS/HTML property parsing functions
+cyclomatic_complexity:
+  warning: 16
+  error: 20