refactor(oocana): remove duplicate __init__ methods from schema subclasses

[leavesster]

Summary

• Remove redundant __init__ methods from PrimitiveFieldSchema, VarFieldSchema, SecretFieldSchema, ArrayFieldSchema, ObjectFieldSchema
• Subclasses now inherit the base FieldSchema.__init__() implementation

Problem

6 classes had nearly identical __init__ implementations:

def __init__(self, **kwargs):
    for key, value in kwargs.items():
        object.__setattr__(self, key, value)

Solution

Keep only one implementation in the base class FieldSchema. Subclasses that need special handling (like ArrayFieldSchema) call super().__init__() first.

Test Plan

• All schema tests pass (test_schema.py)
• Behavior unchanged

[coderabbitai]

总体说明

从 PrimitiveFieldSchema、VarFieldSchema 和 SecretFieldSchema 中移除显式 __init__ 样板代码，改为依赖数据类自动生成的构造器；同时更新 ArrayFieldSchema 和 ObjectFieldSchema 的初始化逻辑，添加属性规范化处理。

变更

变更内容	摘要
Schema类重构 oocana/oocana/schema.py	简化 PrimitiveFieldSchema、VarFieldSchema、SecretFieldSchema 的构造器实现；ArrayFieldSchema 和 ObjectFieldSchema 改为调用 super().__init__(**kwargs) 并强化属性规范化逻辑，包括将非 FieldSchema 值自动转换为 FieldSchema 对象。

代码审查工作量

🎯 3 (中等) | ⏱️ ~20 分钟

🚥 Pre-merge checks | ✅ 2

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description check	✅ Passed	描述与变更集密切相关，详细说明了问题、解决方案和测试计划，与代码改动一致。
Title check	✅ Passed	标题完全符合指定格式要求，并准确总结了主要改动：从schema子类中移除重复的__init__方法。

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull request overview

This PR refactors the schema class hierarchy by removing duplicate __init__ methods from subclasses that don't require special initialization logic, consolidating on the base class implementation.

Changes:

• Removed redundant __init__ methods from PrimitiveFieldSchema, VarFieldSchema, and SecretFieldSchema subclasses that had identical implementations to the base class
• Updated ArrayFieldSchema and ObjectFieldSchema to call super().__init__(**kwargs) instead of duplicating the base initialization logic
• All classes now properly inherit from FieldSchema.__init__() which uses object.__setattr__() to handle frozen dataclass constraints

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

oocana/oocana/schema.py (1)

66-79: ⚠️ Potential issue | 🟠 Major

移除 __init__ 后存在潜在的破坏性变更——缺少声明的字段会导致 TypeError。

当 dataclass 子类不定义自己的 __init__ 时，Python 会生成一个仅接受声明字段的 __init__，且不会调用父类的自定义 __init__。这会导致行为变化：

• 之前：这些类接受任意 kwargs（通过父类 FieldSchema 的自定义 __init__ 和 object.__setattr__）
• 现在：PrimitiveFieldSchema、VarFieldSchema、SecretFieldSchema 仅接受声明的字段（type、contentMediaType）

如果 FieldSchema.generate_schema(dict) 传入包含额外字段（如 JSON Schema 的 description、default、title 等）的字典，将抛出 TypeError: unexpected keyword argument。

此外，这与 ArrayFieldSchema 和 ObjectFieldSchema 的行为不一致——后两者仍通过自定义 __init__ 调用 super().__init__(**kwargs) 接受任意 kwargs。这个设计模式在 data.py 中也有体现（见 StoreKey 类及其注释说明需要自定义 __init__ 来兼容额外字段）。