Skip to content

feat: add per-contest tables for the JAG camp and regional contests - #4224

Merged
KATO-Hiro merged 5 commits into
stagingfrom
#4026
Oct 25, 2026
Merged

KATO-Hiro merged 5 commits into
stagingfrom
#4026

Conversation

@KATO-Hiro

@KATO-Hiro KATO-Hiro commented Oct 25, 2026 •

Copy link
Copy Markdown
Collaborator

close #4026

Summary by CodeRabbit

  • 新機能
    • JAGの地域大会・合宿大会を、開催サービスに応じて分類し、コンテスト別のテーブルとして表示するようになりました。大会名や注記も表示されます。
    • JAG地区大会の表示順を整え、関連するコンテストを一覧に追加しました。
  • テスト
    • 地域大会の分類、表示内容、並び順を確認するテストを追加しました。
  • ドキュメント
    • 分類やテーブル表示に関するガイドを更新し、実装状況、検証結果、本番反映・復旧手順を整理しました。

KATO-Hiro and others added 4 commits October 25, 2026 05:15
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>
@coderabbitai

coderabbitai Bot commented Oct 25, 2026 •

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between c92661c and ee5225f.

Files selected for processing: 2
  • docs/dev-notes/2026-09-09/task-title-index-change-impact/done.md
  • docs/guides/how-to-add-contest-table-provider.md

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Walkthrough

JAG地域大会の一覧と大会別Providerを追加しました。大会をAOJまたはAtCoderに分類してグループへ登録します。関連するテスト、Providerガイド、開発記録も更新しました。

Changes: JAG地域大会テーブル

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: Normal

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でタスクを絞り込む
Loading
Merge Risk: Minimal · up to ee522

No actionable merge-blocking risk remains in this change.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed タイトルは、JAG合宿および地区大会にコンテスト単位のテーブルを追加する主要変更を明確に示しています。
Linked Issues check Passed #4026 は「AOJ JAG 地区予選」テーブルの追加を要求します。AOJ_JAG_REGIONAL_ENTRIES に地区大会・合宿の76コンテストを登録し、AojJagRegional が JagRegionalProvider を各コンテストに追加します。classifyContest により AtCoder開催分を ATCODER_JAG として扱います。プロバイダー…
Out of Scope Changes check Passed コード変更は #4026 のJAG地区大会・合宿テーブルの実装、データ登録、分類、表示検証に対応します。自動テストは実装の検証です。ガイドと開発記録はプロバイダーの登録方法、表示順、データ移行の関連情報を記録します。確認できる範囲に、課題と無関係な変更はありません。
Docstring Coverage Passed 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…
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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

@KATO-Hiro KATO-Hiro left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@KATO-Hiro
KATO-Hiro merged commit bee064d into staging Oct 25, 2026
6 checks passed
@KATO-Hiro
KATO-Hiro deleted the #4026 branch October 25, 2026 07:49

This branch was previously deployed

1 inactive deployment
preview — ee5225fe Deployed Oct 25, 2026 by KATO-Hiro via preview #9096
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feat] テーブル「AOJ JAG 地区予選」を追加しましょう

1 participant