Skip to content

Conversation

@inochidainini
Copy link
Contributor

@inochidainini inochidainini commented Oct 31, 2025

User description

#1162
src/repository/gorm2/migrate/やsrc/repository/gorm2/schema/あたりで使われているLauncherVersionで、Editionに置き換えると既にある名前と被るものはそのままにしています。


PR Type

Enhancement


Description

  • LauncherVersionをEditionへ全面リネーム

  • ドメイン/値オブジェクト/サービス更新

  • ハンドラとリポジトリのAPI整合

  • テスト群を新名称に追随


Diagram Walkthrough

flowchart LR
  Values["values: LauncherVersion* -> Edition*"] -- "型/関数名" --> Domain["domain: LauncherVersion -> Edition"]
  Domain -- "コンストラクタ/メソッド" --> Service["service: 引数/戻り値をEdition系へ"]
  Service -- "呼出し" --> Handler["handler: 変換/バリデーション更新"]
  Service -- "依存" --> Repository["repository: インタフェース/実装更新"]
  Repository -- "DB I/O" --> Migrate["gorm migrate: カラム名と関連更新"]
  Tests["tests: 参照名置換"] --> Handler
  Tests --> Service
  Tests --> Repository
Loading

File Walkthrough

Relevant files
Tests
9 files
edition_test.go
テストをEdition型と新値型へ更新                                                                           
+82/-82 
edition_test.go
サービスEdition周りのテスト置換                                                                           
+43/-43 
edition_test.go
Editionリポジトリのテスト更新                                                                             
+25/-25 
launcher_user_test.go
ランチャーユーザー関連でEditionID採用                                                                   
+30/-30 
v2_game_play_log_test.go
再生ログ周りでEdition型に変更                                                                             
+18/-18 
game_play_log_test.go
再生ログサービスのEdition型置換                                                                           
+18/-18 
game_play_log_test.go
ハンドラの再生ログテストEdition化                                                                         
+7/-7     
edition_auth_test.go
Edition認証ハンドラのテスト更新                                                                           
+9/-9     
launcher_test.go
EditionNameバリデーションテスト                                                                       
+5/-5     
Enhancement
17 files
edition.go
Editionリポジトリ実装の型置換                                                                             
+28/-28 
edition.go
Editionサービスのシグネチャ全面更新                                                                       
+16/-16 
edition_auth.go
認証サービスをEdition戻り値へ                                                                             
+7/-7     
edition_auth.go
Edition認証インターフェース置換                                                                           
+7/-7     
edition.go
Editionハンドラの値型/変換更新                                                                           
+11/-11 
edition.go
EditionリポジトリIFの型名統一                                                                           
+11/-11 
edition.go
型エイリアス含むIFをEditionへ                                                                           
+11/-11 
launcher_version.go
ドメイン名をEditionへリネーム                                                                             
+23/-23 
v1.go
マイグレーション構造体の命名整合                                                                                 
+10/-10 
v2_game_play_log.go
V2再生ログIFをEditionIDへ                                                                           
+3/-3     
game_play_log.go
再生ログ実装のEdition型対応                                                                               
+3/-3     
launcher.go
値オブジェクトをEdition系列へ改名                                                                         
+21/-21 
game_play_log.go
ハンドラのEditionID受け渡し変更                                                                         
+3/-3     
v2_game_play_log.go
版別統計取得でEdition名利用                                                                               
+3/-3     
game_play_stats.go
統計モデルのEdition型置換                                                                                 
+5/-5     
checker.go
EditionID認証と戻り値型更新                                                                             
+2/-2     
edition_auth.go
EditionIDでのキー操作へ統一                                                                             
+2/-2     
Additional files
12 files
v1.yaml +8/-8     
game_play_log.go +3/-3     
context.go +3/-3     
access_token.go +1/-1     
access_token.go +8/-8     
launcher_session_test.go +4/-4     
launcher_user.go +2/-2     
product_key.go +2/-2     
launcher_user.go +1/-1     
product_key.go +2/-2     
v2_game_play_log.go +1/-1     
errors.go +1/-1     

@inochidainini inochidainini requested a review from a team as a code owner October 31, 2025 14:27
@github-actions
Copy link

Migrate lint ✘

Lint output
Abort: Starting with v0.38, 'atlas migrate lint' is available only to Atlas Pro users.

Get started for free by running the following command:

	atlas login

Note, the command and existing code remain open source and available in the Community Edition.
Learn more: https://atlasgo.io/blog-v038#change-in-v038-atlas-migrate-lint

@github-actions
Copy link

github-actions bot commented Oct 31, 2025

Migrate lint ✅

Lint output

@codecov
Copy link

codecov bot commented Oct 31, 2025

Codecov Report

❌ Patch coverage is 37.83784% with 69 lines in your changes missing coverage. Please review.
✅ Project coverage is 52.89%. Comparing base (b3a3c40) to head (6494440).
⚠️ Report is 7 commits behind head on main.

Files with missing lines Patch % Lines
src/repository/gorm2/edition.go 11.11% 24 Missing ⚠️
src/domain/launcher_version.go 0.00% 11 Missing ⚠️
src/domain/values/launcher.go 27.27% 8 Missing ⚠️
src/repository/gorm2/access_token.go 0.00% 8 Missing ⚠️
src/service/v2/edition_auth.go 0.00% 7 Missing ⚠️
src/domain/game_play_stats.go 0.00% 3 Missing ⚠️
src/handler/v2/checker.go 0.00% 2 Missing ⚠️
src/repository/gorm2/product_key.go 0.00% 2 Missing ⚠️
src/service/v2/edition.go 80.00% 2 Missing ⚠️
src/domain/game_play_log.go 0.00% 1 Missing ⚠️
... and 1 more
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1417   +/-   ##
=======================================
  Coverage   52.89%   52.89%           
=======================================
  Files         136      136           
  Lines       10003    10003           
=======================================
  Hits         5291     5291           
  Misses       4398     4398           
  Partials      314      314           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Member

@ikura-hamu ikura-hamu left a comment

Choose a reason for hiding this comment

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

ごめん、説明が不足してたんですが、OpenAPIに関連する部分はAPIの互換性を保つためにそのまま残しておいてほしいです。OpenAPIのyamlファイルと生成されたコードとかです

@inochidainini
Copy link
Contributor Author

コンフリクトの解消、migrate/v1.goの変更の取り消し、謎のresult.goを削除しました。

Copy link
Member

@ikura-hamu ikura-hamu left a comment

Choose a reason for hiding this comment

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

ありがとう。okです。マージしちゃってください

@inochidainini inochidainini merged commit 49f7e51 into main Nov 15, 2025
10 of 11 checks passed
@inochidainini inochidainini deleted the LauncherVersionToEditon branch November 15, 2025 11:25
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.

3 participants