@sun-asterisk/sunlint 1.3.41 → 1.3.42

package/skill-assets/sunlint-code-quality/rules/swift/SW029-unused-closure-parameter.md

@@ -0,0 +1,28 @@
+---
+title: Cảnh báo tham số closure không được sử dụng
+impact: MAJOR
+impactDescription: Tránh cảnh báo của compiler và làm sạch code bằng cách loại bỏ các biến không dùng tới.
+tags: swift, ios, closure, parameters, clean-code
+---
+
+## Cảnh báo tham số closure không được sử dụng
+
+Nếu một closure nhận vào các tham số nhưng bạn không sử dụng chúng trong thân hàm, hãy sử dụng dấu gạch dưới `_` thay cho tên biến. Việc này giúp giảm bớt sự lộn xộn trong code và làm rõ cho người khác biết rằng tham số đó là cố tình bị bỏ qua.
+
+**Incorrect (khai báo tên tham số nhưng không dùng):**
+
+```swift
+apiClient.fetchData { data, error in
+    print("Yêu cầu đã hoàn thành")
+}
+```
+
+**Correct (dùng dấu gạch dưới cho tham số thừa):**
+
+```swift
+apiClient.fetchData { _, _ in
+    print("Yêu cầu đã hoàn thành")
+}
+```
+
+**Tools:** SwiftLint (unused_closure_parameter)

package/skill-assets/sunlint-code-quality/rules/swift/SW030-unused-enumerated.md

@@ -0,0 +1,28 @@
+---
+title: Tránh dùng enumerated() khi không cần index
+impact: MAJOR
+impactDescription: Loại bỏ các tính toán không cần thiết và làm mã nguồn gọn gàng hơn.
+tags: swift, ios, loops, performance, enumeration
+---
+
+## Tránh dùng enumerated() khi không cần index
+
+Chỉ sử dụng phương thức `.enumerated()` khi bạn thực sự cần cả `index` và giá trị phần tử `value` trong vòng lặp. Nếu bạn chỉ cần giá trị phần tử để xử lý, hãy duyệt trực tiếp qua tập hợp. Điều này giúp tối ưu hóa hiệu năng và làm code rõ ràng hơn.
+
+**Incorrect (dùng enumerated chỉ để lấy item):**
+
+```swift
+for (index, item) in items.enumerated() {
+    print("Xử lý \(item)")
+}
+```
+
+**Correct (duyệt trực tiếp):**
+
+```swift
+for item in items {
+    print("Xử lý \(item)")
+}
+```
+
+**Tools:** SwiftLint (unused_enumerated)

package/skill-assets/sunlint-code-quality/rules/swift/SW031-unused-optional-binding.md

@@ -0,0 +1,26 @@
+---
+title: Không dùng optional binding chỉ để gọi hàm hoặc property
+impact: CRITICAL
+impactDescription: Tránh việc tạo ra các khối lồng nhau (deep nesting) không cần thiết, giúp code phẳng và dễ đọc hơn.
+tags: swift, ios, optional, chaining, binding
+---
+
+## Không dùng optional binding chỉ để gọi hàm hoặc property
+
+Tránh sử dụng `if let` hoặc `guard let` nếu mục đích duy nhất chỉ là để gọi một phương thức hoặc truy cập một thuộc tính của đối tượng optional. Thay vào đó, hãy sử dụng **Optional Chaining** (`?.`). Điều này giúp mã nguồn ngắn gọn và tránh lồng ghép nhiều cấp bậc.
+
+**Incorrect (dùng binding cho một lần gọi duy nhất):**
+
+```swift
+if let navigationController = self.navigationController {
+    navigationController.popViewController(animated: true)
+}
+```
+
+**Correct (dùng optional chaining):**
+
+```swift
+self.navigationController?.popViewController(animated: true)
+```
+
+**Tools:** SwiftLint (unused_optional_binding)

package/skill-assets/sunlint-code-quality/rules/swift/SW032-valid-ibinspectable.md

@@ -0,0 +1,26 @@
+---
+title: Không dùng @IBInspectable với kiểu không hỗ trợ và const
+impact: MAJOR
+impactDescription: Tránh lỗi biên dịch hoặc các hành vi không xác định trong Interface Builder.
+tags: swift, ios, uikit, interface-builder, ibinspectable
+---
+
+## Không dùng @IBInspectable với kiểu không hỗ trợ và const
+
+`@IBInspectable` chỉ nên được sử dụng với các kiểu dữ liệu được Interface Builder hỗ trợ (Int, CGFloat, String, Bool, UIColor, v.v.). Ngoài ra, thuộc tính được đánh dấu `@IBInspectable` phải là biến (`var`) để có thể thay đổi giá trị từ Storyboard/XIB. Không sử dụng với hằng số `let`.
+
+**Incorrect (dùng kiểu không hỗ trợ hoặc let):**
+
+```swift
+@IBInspectable let cornerRadius: CGFloat = 5.0 // Sai: Phải dùng var
+@IBInspectable var customObject: MyObject?    // Sai: Kiểu không được hỗ trợ
+```
+
+**Correct (dùng var với kiểu hợp lệ):**
+
+```swift
+@IBInspectable var cornerRadius: CGFloat = 5.0
+@IBInspectable var isShadowEnabled: Bool = true
+```
+
+**Tools:** SwiftLint (valid_ibinspectable)

package/skill-assets/sunlint-code-quality/rules/swift/SW033-vertical-parameter-alignment.md

@@ -0,0 +1,36 @@
+---
+title: Các parameter phải thẳng hàng khi gọi hàm
+impact: LOW
+impactDescription: Cải thiện khả năng đọc mã nguồn và duy trì phong cách viết code nhất quán trong dự án.
+tags: swift, ios, formatting, style-guide, parameters
+---
+
+## Các parameter phải thẳng hàng khi gọi hàm
+
+Khi gọi một hàm có nhiều tham số và bạn quyết định xuống dòng, mỗi tham số (parameter) nên nằm trên một dòng riêng biệt và phải được căn lề thẳng hàng với nhau hoặc theo chuẩn thụt lề của dự án. Điều này giúp người đọc dễ dàng quét qua danh sách đối số đang được truyền vào.
+
+**Incorrect (căn lề không đồng nhất):**
+
+```swift
+animate(withDuration: 0.3, animations: {
+    // ...
+    }, completion: { _ in
+        // ...
+})
+```
+
+**Correct (căn lề thẳng hàng hoặc xuống dòng rõ ràng):**
+
+```swift
+animate(
+    withDuration: 0.3,
+    animations: {
+        // ...
+    },
+    completion: { _ in
+        // ...
+    }
+)
+```
+
+**Tools:** SwiftLint (vertical_parameter_alignment_on_call, vertical_parameter_alignment)

package/skill-assets/sunlint-code-quality/rules/swift/SW034-void-return.md

@@ -0,0 +1,28 @@
+---
+title: Dùng -> Void thay vì -> () cho function type
+impact: LOW
+impactDescription: Tăng tính nhất quán và dễ đọc cho các khai báo closure hoặc function type.
+tags: swift, ios, convention, void, function-type
+---
+
+## Dùng -> Void thay vì -> () cho function type
+
+Swift Coding Convention ưu tiên sử dụng từ khóa `Void` thay cho cặp ngoặc đơn rỗng `()` khi định nghĩa kiểu trả về của một hàm (function type) hoặc closure. `Void` mang tính tường minh và dễ đọc hơn khi xuất hiện trong các khai báo phức tạp.
+
+**Incorrect (dùng () cho kiểu trả về):**
+
+```swift
+let completion: () -> () = {
+    print("Xong")
+}
+```
+
+**Correct (dùng Void):**
+
+```swift
+let completion: () -> Void = {
+    print("Xong")
+}
+```
+
+**Tools:** SwiftLint (void_return)

package/skill-assets/sunlint-code-quality/rules/swift/SW035-weak-delegate.md

@@ -0,0 +1,28 @@
+---
+title: Delegate phải được đánh dấu là weak
+impact: HIGH
+impactDescription: Tránh giữ chặt đối tượng (strong reference) dẫn đến retain cycle và rò rỉ bộ nhớ (memory leak).
+tags: swift, ios, delegate, weak, memory-leak
+---
+
+## Delegate phải được đánh dấu là weak
+
+Trong mô hình Delegation, đối tượng nhận ủy quyền (delegate) thường sở hữu đối tượng thực hiện ủy quyền. Để tránh vòng lặp tham chiếu mạnh (retain cycle), thuộc tính `delegate` phải luôn được khai báo với từ khóa `weak`. Điều này đảm bảo rằng khi View Controller hoặc đối tượng chủ quản bị giải phóng, delegate cũng có thể được giải phóng theo.
+
+**Incorrect (giữ tham chiếu mạnh):**
+
+```swift
+class MyView: UIView {
+    var delegate: MyViewDelegate? // Nguy cơ memory leak
+}
+```
+
+**Correct (dùng tham chiếu yếu):**
+
+```swift
+class MyView: UIView {
+    weak var delegate: MyViewDelegate? // An toàn
+}
+```
+
+**Tools:** SwiftLint (weak_delegate)