Repository navigation
Conversation
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
大会一覧、Provider、グループ登録 src/features/tasks/utils/contest-table/aoj_jag_regional_entries.ts, src/features/tasks/utils/contest-table/aoj_jag_providers.ts, src/features/tasks/utils/contest-table/contest_table_provider_groups.ts, src/features/tasks/utils/contest-table/*test.ts |
大会IDと注記を並べた一覧を追加しました。大会別Providerは対象タスクを絞り込み、タイトルや表示設定を提供します。グループ登録では分類に応じてATCODER_JAGまたはAOJ_JAGを設定します。テストは76件の順序、seedとの一致、Providerの表示情報などを検証します。 |
開発記録とProviderガイド docs/dev-notes/2026-09-09/task-title-index-change-impact/*, docs/guides/how-to-add-contest-table-provider.md |
PRの状態、移行・照合・検証記録を整理しました。PR2.1とPR2.5の計画書を削除しました。ガイドと調査記録に、配列の順序、分類、表示、照合情報を追記しました。 |
Priority:
Estimated code review effort: 3 (Moderate) | ~20 minutes
Change: Feature
Sequence Diagram(s)
sequenceDiagram
participant ContestProviderGroups
participant RegionalEntries
participant classifyContest
participant JagRegionalProvider
participant ContestTasks
ContestProviderGroups->>RegionalEntries: 順序付き大会エントリーを取得
ContestProviderGroups->>classifyContest: contestIdを分類
ContestProviderGroups->>JagRegionalProvider: contestTypeとエントリーを渡して生成
JagRegionalProvider->>ContestTasks: contest_idでタスクを絞り込む
Merge Risk: 
Minimal · up to ee522
-
No actionable merge-blocking risk remains in this change.
Pre-merge checks: 
5
-
Full details: Linked Issues check
-
Explanation
#4026は「AOJ JAG 地区予選」テーブルの追加を要求します。AOJ_JAG_REGIONAL_ENTRIESに地区大会・合宿の76コンテストを登録し、AojJagRegionalがJagRegionalProviderを各コンテストに追加します。classifyContestにより AtCoder開催分をATCODER_JAGとして扱います。プロバイダー、登録順、コンテストID、表示設定、タイトル、seed対象の自動テストを追加しています。
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 6 files. (2 skipped: 2 unsupported.)
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
close #4026
Summary by CodeRabbit