Swift 代码审查

审查工作流

请严格按以下顺序执行。在满足下方所有 通过条件（Pass） 之前，不得输出任何问题发现。

1. Swift / 工具链基线 — 明确语言与工具上下文：检查 Package.swift 中的 // swift-tools-version，以及清单文件中各 target 的 Swift 语言版本或 swiftSettings；若为 Xcode 项目，则检查 project 或 target 的构建设置中是否存在 SWIFT_VERSION（或等效配置）；如本次审查仅限单个文件，请明确标注。
   通过条件： 在提出依赖严格并发模型、迁移专用语法或特定 SDK 可用性的建议前，须先明确声明具体的 Swift 语言版本或模式（例如：Swift 6 语言模式、tools 5.10）。

2. 通读上下文代码 — 对每个被修改的 .swift 文件，需完整阅读其所在类型、函数、方法或属性的全部定义，而不仅限于 diff 片段。
   通过条件： 每个被修改的文件，均至少完整阅读了一个包含变更的顶层符号（类型或成员）。

3. 划定检查清单范围 — 借助《快速参考》确定适用的《审查检查清单》条目及对应参考文档；打开相关参考文件；跳过明显与 diff 无关的条目。
   通过条件： 审查报告（或工作笔记）中需列出实际应用的检查清单类别，或对不适用项标注 N/A 并附上一句紧扣 diff 内容的理由（例如：“变更中未使用 SwiftUI / @Observable”）。

4. 报告前验证 — 加载并执行 review-verification-protocol。
   通过条件： 对你将报告的每一项问题，该技能所要求的 硬性门禁（按序执行） 均已满足（完整符号阅读、在标记“未使用”前完成使用搜索、在标记“缺失处理”前确认调用方、严重性已校准、提供 [FILE:LINE] 定位依据）。

硬性门禁（相同顺序，精简版）

输出格式

请按如下格式报告问题：

[FILE:LINE] 问题标题
Severity: Critical | Major | Minor | Informational
问题描述及影响说明。

快速参考

审查检查清单

• 不对运行时数据（网络响应、用户输入、文件内容等）使用强制解包（!）
• 作为属性存储的闭包应使用 [weak self]
• delegate 属性应声明为 weak
• 独立的异步操作应使用 async let 或 TaskGroup
• 长时间运行的 Task 应检查 Task.isCancelled
• actor 应具备需保护的可变状态（避免无状态 actor）
• Sendable 类型必须真正线程安全（警惕 @unchecked 的误用）
• 错误必须显式处理（禁止空的 catch 块）
• 自定义错误类型应遵循 LocalizedError 协议，并提供描述性消息
• 嵌套的 @Observable 对象也必须标记为 @Observable
• @Bindable 仅用于 Observable 对象的双向绑定

何时加载参考文档

• 审查 async/await、actors 或 TaskGroup → concurrency.md
• 审查 @Observable 或 SwiftUI 状态管理 → observable.md
• 审查错误处理或 throws 机制 → error-handling.md
• 常规 Swift 代码审查 → common-mistakes.md

审查关键问题

1. 可能存在并发执行的异步操作是否使用了 async let？
2. actor 状态是否可能在挂起点之间发生变化（引发重入 bug）？
3. @unchecked Sendable 是否有实际的同步机制作为保障？
4. 错误是否被记录并以提供有效上下文的方式向用户呈现？
5. 是否存在因闭包或 delegate 导致循环引用的风险？