AnnotationSpecsAccessorに定型指摘に関するメソッドを追加しました

[yuji38kwmt]

Summary

• Add InspectionPhrase TypedDict and keep inspection_phrases on AnnotationSpecsAccessor
• Add ID-based get_inspection_phrase helpers
• Add tests for ID lookup and not-found behavior

Tests

• make format
• make lint
• uv run pytest tests/util/test_annotation_specs.py

[kci-pr-agent]

Title

Add inspection phrase accessor

---

Description

• InspectionPhrase TypedDict を追加

• get_inspection_phrase 関数を実装

• AnnotationSpecsAccessor に定型指摘取得メソッド追加

• テストで取得とエラーケースを検証

---

Changes walkthrough 📝

	Relevant files
Enhancement	annotation_specs.py 定型指摘アクセサ機能の追加                                                                                        annofabapi/util/annotation_specs.py InspectionPhrase TypedDict 定義追加 get_inspection_phrase 関数実装 AnnotationSpecsAccessor に inspection_phrases プロパティとメソッド追加 +51/-0
Tests	test_annotation_specs.py 定型指摘テストの追加                                                                                              tests/util/test_annotation_specs.py テストセットアップに inspection_phrases データ追加 get_inspection_phrase の正常系・異常系テスト追加 +47/-0

---

Need help?

Type /help how to ... in the comments thread for any questions about PR-Agent usage.

Check out the documentation for more information.

[kci-pr-agent]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 PR contains tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Missing Key Validation annotation_specsにinspection_phrasesキーが存在しない場合、__init__内でKeyErrorが発生します。デフォルト値の設定や存在チェックの追加を検討してください。 def __init__(self, annotation_specs: dict[str, Any]) -> None: self.annotation_specs = annotation_specs self.labels: list[LabelDefinition] = annotation_specs["labels"] self.additionals: list[AttributeDefinition] = annotation_specs["additionals"] self.inspection_phrases: list[InspectionPhrase] = annotation_specs["inspection_phrases"] テスト不足 同一のinspection_phrase_idを持つ要素が複数ある場合の例外処理がテストされていません。重複ID時のエラーケースを追加で検証すると良いでしょう。 inspection_phrase = get_inspection_phrase(self.inspection_phrases, inspection_phrase_id="1") assert inspection_phrase["id"] == "1" def test_get_inspection_phrase_not_found(self): with pytest.raises(ValueError): get_inspection_phrase(self.inspection_phrases, inspection_phrase_id="3")

[kci-pr-agent]

PR Code Suggestions ✨

No code suggestions found for the PR.

[Copilot]

Pull request overview

AnnotationSpecsAccessor に「定型指摘（inspection_phrases）」を扱う型・取得ユーティリティを追加し、IDベースで定型指摘を取り出せるようにする変更です。アノテーション仕様（annotation specs）周りの参照APIを拡張し、呼び出し側の実装を簡潔にする目的に沿っています。

Changes:

• InspectionPhrase（TypedDict）を追加し、annotation specs 内の inspection_phrases を型付け
• get_inspection_phrase() と AnnotationSpecsAccessor.get_inspection_phrase() により、IDで定型指摘を取得可能に
• ID取得と not-found のテストを追加

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
annofabapi/util/annotation_specs.py	定型指摘のTypedDictとID取得ヘルパー、Accessorメソッドを追加
tests/util/test_annotation_specs.py	定型指摘のID取得・not-found 挙動のテストを追加