Skip to content

refactor: 共通VOを責務の所有先へ整理する - #573

Open
lambig wants to merge 1 commit into
mainfrom
work/v111-vo-ownership
Open

lambig wants to merge 1 commit into
mainfrom
work/v111-vo-ownership

Conversation

@lambig

@lambig lambig commented Oct 23, 2026 •

Copy link
Copy Markdown
Owner

方針レビュー待ち:DM-200 / v1。 作品・頒布・記事とイベントの関係、コア・支援・汎用の分類、依存と状態管理、今回の実装範囲をR1〜R8で先に判断します。現在の配置・規約・ArchUnit規則は未採用の比較案であり、レビュー後の修正対象です。

現在のhead c445f5289cdcd6d77eda63a88855e1ae3e1356f8 は、基準 e4ae67188c0de2d767cfc12519f38f3d62debab4 に対し、ArtistCredit/ArtistCreditName/EventReleasedAt/ExternalAudioUrlをalbumへ、EventDateAndSpaceをeventへ移した案です。commonに残した値の分類と、この配置を固定する検査も再評価します。このPRを現状のまま#200の完了根拠にはしません。

現存VOの棚卸しと、移設前後で挙動を維持した証拠は再利用できます。Corretto 25.0.4.10.1/Gradle 9.7.1、Java blocklist適用で対象732試験、統合試験コンパイル、main/test Checkstyle・PMD・Spotless、CSP関連3試験が成功しました。これは設計方針の採用を示しません。DB統合試験と全E2Eはローカル実行の対象外でした。

次の判断は、作品・頒布・記事へ属する責務、アセット・サイト運営等の支援候補、文書形式・日時・URLの汎用処理とサービス固有方針の境界です。コアでは識別・不変条件・訂正と編集・状態遷移・再構築・複数モデルの整合性を評価し、分類後に型の所属と検査を決めます。#89/#148/#562/#276と対応付け、外部所有先・公開APIの決定へ渡します。

レビュー観点:

  • 現存配置を根拠にせず、頒布を含むコアの責務から所属を説明できるか。
  • 支援・汎用をcommonの代替の置き場にせず、具体的な責務と契約で分けられるか。
  • コアの状態管理要件と機械検査の保証範囲が対応しているか。

Refs #200, #201, #248, #471, #89

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.

1 participant