Skip to content

fix: Get UseCase が typed Input DTO を使うよう統一 (#109)#118

Closed
hideyukiMORI wants to merge 1 commit into
mainfrom
fix/109-get-usecase-typed-input
Closed

fix: Get UseCase が typed Input DTO を使うよう統一 (#109)#118
hideyukiMORI wants to merge 1 commit into
mainfrom
fix/109-get-usecase-typed-input

Conversation

@hideyukiMORI
Copy link
Copy Markdown
Owner

Summary

  • GetNoteUseCase / GetTagUseCase / GetCommentUseCaseexecute() が生の int を受け取っていた問題を修正
  • 他の UseCase と同様に XxxInput typed DTO パターンに統一
  • GetNoteInput / GetTagInput / GetCommentInput を追加(frozen=True, slots=True
  • handler.py × 3、mcp.py、テスト × 2 の呼び出し箇所を更新

Closes #109

Test plan

  • uv run pytest が全パスすること
  • uv run mypy src/ がエラーなしであること

🤖 Generated with Claude Code

GetNoteUseCase / GetTagUseCase / GetCommentUseCase の execute() が
生の int を受け取っていた問題を修正。他の UseCase と同様に typed DTO を使うよう統一。

追加: GetNoteInput / GetTagInput / GetCommentInput (frozen=True, slots=True)
変更: handler.py × 3、mcp.py、use_case テスト × 2 の呼び出し箇所を更新

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@hideyukiMORI
Copy link
Copy Markdown
Owner Author

同等の変更が手動コミットで main に取り込み済みのため Close します(PR #143 / #144 / #150 参照)。

@hideyukiMORI hideyukiMORI deleted the fix/109-get-usecase-typed-input branch May 20, 2026 14:50
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.

fix: GetNoteUseCase / GetTagUseCase / GetCommentUseCase が typed Input DTO を使っていない

1 participant