Skip to content

feat(contests): add GAME_24 contest to xDPC contest table group - #4120

Merged
KATO-Hiro merged 5 commits into
stagingfrom
#3966
Oct 6, 2026
Merged

KATO-Hiro merged 5 commits into
stagingfrom
#3966

Conversation

@KATO-Hiro

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

Copy link
Copy Markdown
Collaborator

close #3966

Summary by CodeRabbit

  • 新機能
    • 「組合せゲーム 24 題」に対応しました。問題一覧で、既存の「xDPC」とあわせて問題を閲覧できます。
  • ドキュメント
    • コンテストの追加手順やデータ確認時の注意点を更新しました。
    • データベースの変更・保存データ・テストデータに関するガイドを更新しました。
  • テスト
    • 問題一覧での表示順と、コンテストの分類・表示を確認するテストを追加しました。

KATO-Hiro and others added 5 commits October 6, 2026 12:59
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Rename the group label to xDPC・x 24 題 and place Game 24 after FPS 24.

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

coderabbitai Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: AtCoder-NoviSteps/AtCoderNoviSteps/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 6e376f78-71bf-4c35-88eb-28967b45230f
📥 Commits

Reviewing files that changed from the base of the PR and between 2bdf923 and f5ca37e.

📒 Files selected for processing (23)
  • .agents/skills/add-contest-table-provider/SKILL.md
  • .agents/skills/add-contest-table-provider/instructions.md
  • docs/guides/agent-rules/prisma-db.md
  • docs/guides/agent-rules/svelte-components.md
  • docs/guides/agent-rules/testing.md
  • docs/guides/how-to-add-contest-table-provider.md
  • e2e/problems_contest_table.spec.ts
  • prisma/migrations/20261006125938_add_game_24_to_contest_type/migration.sql
  • prisma/schema.prisma
  • prisma/tasks.ts
  • src/features/tasks/utils/contest-table/contest_table_provider_groups.test.ts
  • src/features/tasks/utils/contest-table/contest_table_provider_groups.ts
  • src/features/tasks/utils/contest-table/game24_provider.test.ts
  • src/features/tasks/utils/contest-table/game24_provider.ts
  • src/lib/contests/fixtures/contest_name_labels.ts
  • src/lib/contests/fixtures/contest_type.ts
  • src/lib/contests/types/contest.ts
  • src/lib/contests/utils/classification.test.ts
  • src/lib/contests/utils/classification.ts
  • src/lib/contests/utils/labels/index.test.ts
  • src/lib/contests/utils/labels/index.ts
  • src/lib/contests/utils/priority.test.ts
  • src/lib/contests/utils/priority.ts

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

Walkthrough

組合せゲーム24題のGAME_24種別、game-24の問題データ、コンテストテーブルプロバイダーを追加しました。コンテスト分類と表示順のテストを追加し、プロバイダー追加手順と開発ガイドラインを更新しました。

Changes

組合せゲーム24題

Layer / File(s) Summary
コンテスト種別と分類
prisma/schema.prisma, prisma/migrations/*, src/lib/contests/*
GAME_24をenumとコンテスト分類に追加しました。ラベルと優先度を設定し、関連テストを追加しました。
問題データの登録
prisma/tasks.ts
game-24の問題XからAまでをtasksに登録しました。
コンテストテーブル表示
src/features/tasks/utils/contest-table/*, e2e/problems_contest_table.spec.ts
Game24Providerを追加し、dpsプリセットに登録しました。プロバイダーのフィルター、テーブル、見出しの表示順をテストします。
プロバイダー追加手順
.agents/skills/add-contest-table-provider/*, docs/guides/how-to-add-contest-table-provider.md
GAME_24のtask_id形式を追記し、Seed checkとContestType追加手順を更新しました。

エージェント向けルール

Layer / File(s) Summary
開発ガイドラインの更新
docs/guides/agent-rules/prisma-db.md, docs/guides/agent-rules/svelte-components.md, docs/guides/agent-rules/testing.md
本番マイグレーション、保存値、反復キー、fixtureやhelperの扱いに関する記述を更新しました。

Priority: ➖ Normal

Estimated code review effort: 3 (Moderate) | ~25 minutes

Change: Feature

Sequence Diagram(s)

sequenceDiagram
  participant dpsPreset
  participant Game24Provider
  participant TaskResults
  participant ContestTable
  dpsPreset->>Game24Provider: GAME_24 providerを登録
  Game24Provider->>TaskResults: contestTypeとcontest_idで絞り込み
  Game24Provider->>ContestTable: 問題インデックスを使ってテーブルを生成
Loading

Merge Risk: ⚪ Minimal · up to f5ca3

No actionable merge-blocking issue was identified; the change is ready for normal checks.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning 機能実装、provider 手順書、contest provider 向けの手順変更は [#3966] に関連します。一方、docs/guides/agent-rules/prisma-db.md のルートハンドラー規則、svelte-components.md の localStorage・{#each} 規則、testing.md の fixture 規則の変更は、GAME_2… 上記3つの一般ルール変更をこの PR から除き、GAME_24 の取り込みと、その実装に必要な手順書・テストの変更に限定してください。
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed タイトルは、GAME_24をxDPCコンテスト表グループに追加する主な変更を明確に示しています。
Linked Issues check ✅ Passed [#3966] の要件は「組合せゲーム 24 題」の取り込み対応です。GAME_24 の型・マイグレーション・分類・ラベルを追加し、24 問のデータを登録しています。Game24Provider を実装して xDPC の contest table グループに追加し、関連する単体テストと E2E テストも追加しています。
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 1…
Full details: Out of Scope Changes check

Explanation

機能実装、provider 手順書、contest provider 向けの手順変更は [#3966] に関連します。一方、docs/guides/agent-rules/prisma-db.md のルートハンドラー規則、svelte-components.md の localStorage・{#each} 規則、testing.md の fixture 規則の変更は、GAME_24 の取り込みや contest table の追加に結び付かない一般ルールの変更です。

  • Fix all pre-merge checks with AI
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 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.

@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 8025ace into staging Oct 6, 2026
3 checks passed
@KATO-Hiro
KATO-Hiro deleted the #3966 branch October 6, 2026 13:24

This branch was successfully deployed

1 active deployment
preview — f5ca37ef Deployed Oct 6, 2026 by KATO-Hiro via preview #8847
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] 「組合せゲーム 24 題」をインポートできるようにしましょう

1 participant