[Feat]: Remove extensions field from Message and Artifact
- 主要言語
- Shell
- スター
- 25.8k
- フォーク
- 2.6k
- 平均マージ
- 3日 6時間
- マージ済み PR(30日)
- 16
説明
### Is your feature request related to a problem? Please describe.
Artifact and Message types include an extensions array even though the extension data is anyway expected to be partitioned by URI within the metadata field.
This introduces redundancy and makes it easier to form an incorrect response (eg. metadata modified, extensions not updated).
### Describe the solution you'd like
Remove the field.
### Describe alternatives you've considered
Clarify semantics of the field.
### Additional context
_No response_
### Code of Conduct
- [x] I agree to follow this project's Code of Conduct
コントリビューションガイド
調査の方向性
コードベース内で Message 型と Artifact 型の定義を探します。おそらくスキーマまたはプロトコル定義ファイルにあります。両方の型から extensions フィールドを削除し、それへの参照がすべて更新されていることを確認します。これらの型のシリアライゼーションまたはバリデーションに関連する既存のテストがあればすべて実行して、変更を検証します。
索引モデルが issue の本文から書いたものです。
評価
- 領域
- backend-api-design
- issue の種類
- リファクタリング
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 活発
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 55/100