feat: Shared Kernel に Affiliation(所属) を追加 by KinjiKawaguchi · Pull Request #81 · su-its/core

KinjiKawaguchi · 2026-03-12T04:49:34Z

Summary

値オブジェクトを所属する集約ディレクトリに移動し、domain/base に ValueObject を配置
Shared Kernel として Affiliation（学部・修士・博士・専門職）と StudentId 値オブジェクトを追加
静岡大学の全組織構造を型レベルの判別共用体で表現（学部ごとに異なる階層構造を正確にモデル化）
Affiliation のバリデーションは型制約で保証し、ランタイム検証はシステム境界の責務とする設計

🤖 Generated with Claude Code

## Summary - 学籍番号（StudentId）の値オブジェクトをSharedKernelとして追加 - 旧形式（8桁数字）と新形式（3桁数字+英字+4桁数字）の両方に対応 - `fromString` ファクトリメソッドで小文字→大文字正規化・空白トリムを実施 - private constructorにより直接インスタンス化を防止 - 12件のユニットテストを追加 ## Test plan - [x] 旧形式・新形式の正常系テスト - [x] 大文字正規化・空白トリムのテスト - [x] 桁数不正・英字のみ・英字位置不正・空文字等の異常系テスト - [x] equalsメソッドの等価性テスト - [x] lint, typecheck, test全てパス ## Related - PR #81 から StudentId 部分を分割したPR - マージ後、PR #81 をリベースしてAffiliation部分のみにする 🤖 Generated with [Claude Code](https://claude.com/claude-code)  --- <a href="https://app.devin.ai/review/su-its/core/pull/87" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a>  --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

静岡大学の全学部・研究科の組織構造を型レベルで表現し、 ValueObject<T> パターンでランタイムバリデーション付きの Affiliation クラス群を実装。 - UndergraduateAffiliation / MasterAffiliation / DoctoralAffiliation / ProfessionalAffiliation - 学部ごとに異なる分類子（学科・課程・専攻・コース・専修）を discriminated union で表現 - StudentId 値オブジェクト（旧形式8桁 / 新形式3桁+英字+4桁） - 関連するドメイン例外を追加 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

型レベルの判別共用体で制約が保証されるため、ランタイムバリデーションはValueObjectの責務ではなくシステム境界の責務とする。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

@see

各学部・研究科の型定義に日本語名のJSDocコメントと公式組織構成ページへの@seeリンクを追加。バリデーター削除で不要になった例外クラスも削除。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

KinjiKawaguchi

ユビキタス言語との対応といった意味でJsDocでフィールドについてそれなりに記述しておくべきだと思うがそういった取り組みが見られない

型レベルで安全性を保証する設計のため不要だったInvalidAffiliationExceptionを削除。 universityStructure.tsのフィールドにユビキタス言語との対応を示すJSDocを追加。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

教育学部のURLは EducationFacultyValue の型JSDocに既にあるため重複を解消。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

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

devin-ai-integration

Devin Review found 1 new potential issue.

devin-ai-integration · 2026-03-12T07:24:45Z

📝 Info: ValueObject.equals() uses JSON.stringify which is key-order-dependent for these complex objects

The base ValueObject.equals() at src/domain/base/ValueObject.ts:16 compares values via JSON.stringify(this.value) === JSON.stringify(vo.value). For the new Affiliation value types — which are plain objects with multiple keys — this comparison is sensitive to property insertion order. Two semantically identical affiliation objects constructed with properties in different orders would be considered unequal. This is a pre-existing design characteristic of ValueObject, not introduced by this PR, but it becomes more relevant now that ValueObject is being used with multi-field objects rather than primitives or simple types.

Was this helpful? React with 👍 or 👎 to provide feedback.

KikyoNanakusa

一旦いいと思うけど、大学の学部や学科区分は不変ではないことに今後は注意してね

KikyoNanakusa · 2026-03-12T22:45:51Z

+// ── 年次 ──
+
+export type UndergraduateYear = 1 | 2 | 3 | 4;
+export type MasterYear = 1 | 2;


M2+みたいな例外はどう扱う予定

それはM2何じゃないか、進級という概念があるんだろうか?...

KikyoNanakusa · 2026-03-12T22:50:19Z

+import type {
+	DoctoralAffiliationValue,
+	MasterAffiliationValue,
+	ProfessionalAffiliationValue,


これは何を想定されている

https://www.shizuoka.ac.jp/subject/

を見ると博士課程以外に専門職学位課程というのがあるらしいのでそれ。ただ英訳については疑義あり

## Summary - `Event.removeExhibitMemberId`のボディが空で、展示からのメンバー削除がno-opだった問題を修正 - 該当Exhibitからメンバーを削除し、他のExhibitにも所属していなければEvent.memberIdsからも削除する - PR #81 のレビューで指摘された pre-existing issue への対応 🤖 Generated with [Claude Code](https://claude.com/claude-code)  --- <a href="https://app.devin.ai/review/su-its/core/pull/85" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a>  --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

## Summary - 学籍番号（StudentId）の値オブジェクトをSharedKernelとして追加 - 旧形式（8桁数字）と新形式（3桁数字+英字+4桁数字）の両方に対応 - `fromString` ファクトリメソッドで小文字→大文字正規化・空白トリムを実施 - private constructorにより直接インスタンス化を防止 - 12件のユニットテストを追加 ## Test plan - [x] 旧形式・新形式の正常系テスト - [x] 大文字正規化・空白トリムのテスト - [x] 桁数不正・英字のみ・英字位置不正・空文字等の異常系テスト - [x] equalsメソッドの等価性テスト - [x] lint, typecheck, test全てパス ## Related - PR #81 から StudentId 部分を分割したPR - マージ後、PR #81 をリベースしてAffiliation部分のみにする 🤖 Generated with [Claude Code](https://claude.com/claude-code)  --- <a href="https://app.devin.ai/review/su-its/core/pull/87" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a>  --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

## Summary - `Event.removeExhibitMemberId`のボディが空で、展示からのメンバー削除がno-opだった問題を修正 - 該当Exhibitからメンバーを削除し、他のExhibitにも所属していなければEvent.memberIdsからも削除する - PR #81 のレビューで指摘された pre-existing issue への対応 🤖 Generated with [Claude Code](https://claude.com/claude-code)  --- <a href="https://app.devin.ai/review/su-its/core/pull/85" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a>  --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

## Summary - 値オブジェクトを所属する集約ディレクトリに移動し、`domain/base` に `ValueObject` を配置 - Shared Kernel として `Affiliation`（学部・修士・博士・専門職）と `StudentId` 値オブジェクトを追加 - 静岡大学の全組織構造を型レベルの判別共用体で表現（学部ごとに異なる階層構造を正確にモデル化） - Affiliation のバリデーションは型制約で保証し、ランタイム検証はシステム境界の責務とする設計 🤖 Generated with [Claude Code](https://claude.com/claude-code)  --- <a href="https://app.devin.ai/review/su-its/core/pull/81" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a>  --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Copilot AI review requested due to automatic review settings March 12, 2026 04:49

Copilot started reviewing on behalf of KinjiKawaguchi March 12, 2026 04:49 View session

This comment was marked as resolved.

Sign in to view

KinjiKawaguchi force-pushed the feat/shared-kernel-affiliation branch from f84d664 to 7d0867c Compare March 12, 2026 04:53

This comment was marked as resolved.

Sign in to view

This was referenced Mar 12, 2026

fix: Event.removeExhibitMemberIdの未実装を修正 #85

Merged

電子カルテ情報のcore移植 #54

Closed

feat!: SharedKernel StudentIdの実装 #87

Merged

KinjiKawaguchi and others added 3 commits March 12, 2026 16:06

KinjiKawaguchi changed the title ~~feat: Shared Kernel に Affiliation・StudentId を追加~~ feat: Shared Kernel に Affiliation を追加 Mar 12, 2026

KinjiKawaguchi force-pushed the feat/shared-kernel-affiliation branch from 05fb324 to 0bb3aba Compare March 12, 2026 07:06

This comment was marked as resolved.

Sign in to view

KinjiKawaguchi commented Mar 12, 2026

View reviewed changes

Comment thread src/domain/exceptions/DomainExceptions.ts Outdated

KinjiKawaguchi and others added 2 commits March 12, 2026 16:16

refactor: ファイルレベルJSDocから教育学部固有のリンクを除去

7a57050

教育学部のURLは EducationFacultyValue の型JSDocに既にあるため重複を解消。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

This comment was marked as resolved.

Sign in to view

KinjiKawaguchi and others added 2 commits March 12, 2026 16:21

style: DomainExceptions.tsの空行をdevelopと一致させる

17939eb

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

docs: Affiliation型にユビキタス言語「所属」との対応をJSDocで明示

752e2ed

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

KinjiKawaguchi changed the title ~~feat: Shared Kernel に Affiliation を追加~~ feat: Shared Kernel に Affiliation(所属) を追加 Mar 12, 2026

KinjiKawaguchi requested review from KikyoNanakusa and Mel-906 March 12, 2026 07:24

devin-ai-integration Bot reviewed Mar 12, 2026

View reviewed changes

KikyoNanakusa reviewed Mar 12, 2026

View reviewed changes

KikyoNanakusa approved these changes Mar 13, 2026

View reviewed changes

KinjiKawaguchi merged commit 7a7c4bb into develop Mar 14, 2026
4 checks passed

KinjiKawaguchi deleted the feat/shared-kernel-affiliation branch March 14, 2026 13:55

Conversation

KinjiKawaguchi commented Mar 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Uh oh!

This comment was marked as resolved.

Uh oh!

This comment was marked as resolved.

Uh oh!

This comment was marked as resolved.

Uh oh!

KinjiKawaguchi left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

This comment was marked as resolved.

Uh oh!

devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

Uh oh!

devin-ai-integration Bot Mar 12, 2026

Choose a reason for hiding this comment

Uh oh!

KikyoNanakusa left a comment

Choose a reason for hiding this comment

Uh oh!

KikyoNanakusa Mar 12, 2026

Choose a reason for hiding this comment

Uh oh!

KinjiKawaguchi Mar 12, 2026

Choose a reason for hiding this comment

Uh oh!

KikyoNanakusa Mar 12, 2026

Choose a reason for hiding this comment

Uh oh!

KinjiKawaguchi Mar 12, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

KinjiKawaguchi commented Mar 12, 2026 •

edited

Loading