执行摘要

• 一句话：添加防御性 getattr 禁令并优化 dataclasses 规则
• 推荐动作：值得合并，作为代码风格和质量控制的补充。建议开发者在 AI 辅助中注意新规则，避免过度使用 getattr。可考虑后续添加 CI 检查（如 ruff 规则）来自动化此类风格问题。

功能与动机

AI 代理和防御性编程习惯倾向于在字段始终存在的对象上使用 getattr/hasattr，这会隐藏真正的错误并破坏严格的类型检查。PR body 以 server_args.revision 为例说明这种用法会吞掉字段重命名时的 AttributeError，给读者造成困惑。

实现拆解

1. 新增 no-getattr-defensive.md 规则文件：在 .claude/rules/ 目录下创建，路径限定 **/*.py，明确禁止过度防御的 getattr/hasattr。推荐两种替代方案：isinstance 类型收窄后直接访问字段，或始终将字段设为 None 后做 None/非 None 检查。规则引用 mm_utils.py 中的好例子和 template_detection.py 中的坏例子作为佐证。

2. 润色 no-dataclasses.md：将“不映射到 Rust 结构体以用于计划中的 Rust 迁移”改为“不能干净地支持多语言（例如计划中的 Rust 迁移）”，更强调通用性。同时微调 LoadSnapshot 注释，推荐使用 frozen= 和 omit_defaults=。删除关于第三方 API 强制使用 @dataclass 的边界隔离条款。

3. 无运行时或测试变更：两个文件均为 .claude/rules/ 下的 AI 辅助规则配置，不涉及构建、测试或部署流程。

关键文件：

• .claude/rules/no-getattr-defensive.md（模块 AI规则；类别 docs；类型 documentation）: 核心新增文件，明确禁止防御性 getattr/hasattr，并提供具体替代方案和实例。
• .claude/rules/no-dataclasses.md（模块 AI规则；类别 docs；类型 documentation；符号 LoadSnapshot）: 润色现有规则，使表述更通用，微调注释与删除边界隔离条款。

关键符号：LoadSnapshot

评论区精华

PR 无 review 讨论或评论，仅有一条来自 gemini-code-assist[bot] 的自动配额警告，无实际技术讨论。

• 暂无高价值评论线程

风险与影响

• 风险：低风险。变更仅影响 .claude/rules/ 下的文档文件，不触及任何运行时代码、测试、配置或 CI 流程。新规则如果被 AI 工具严格应用，可能导致对少量合法使用 getattr（如动态属性访问）的误报，但规则本身明确禁止的是“防御性”使用，且允许通过例子澄清。润色 no-dataclasses.md 删除了关于第三方 API 边界隔离的说明，可能降低对该情况的明确指导。
• 影响：直接影响 AI 辅助工具（如 Claude）在编写代码时的行为引导，长期可减少防御性 getattr 导致的隐藏 bug。对用户、系统运行和部署无即时影响。团队需知晓新规则并确保 AI 工具正确加载。
• 风险标记：仅文档变更，无风险

关联脉络

• 暂无明显关联 PR