Conversation
|
Caution Review failedThe pull request is closed. Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. Walkthrough[Review] 新しい Flutter ベースの AI チャットフロントエンドおよび関連ドキュメント、モノレポ設計文書、Flutter プロジェクト設定を追加し、バックエンド依存のセキュリティ関連バージョンを更新します（WebSocket と REST を用いるリアルタイム会話フローを含む）。 Changes
Sequence Diagram(s)sequenceDiagram
actor User
participant Flutter_UI as "Flutter UI"
participant ApiService as "ApiService"
participant Backend as "Backend API"
participant WSService as "WebSocketService"
User->>Flutter_UI: 起動 / メッセージ送信
Flutter_UI->>ApiService: createSession()
ApiService->>Backend: POST /api/chat/sessions
Backend-->>ApiService: { sessionId, ... }
ApiService-->>Flutter_UI: ChatSession
Flutter_UI->>WSService: connect(sessionId)
WSService->>Backend: WS /api/chat/ws/{sessionId} 接続
Backend-->>WSService: 接続確立
WSService-->>Flutter_UI: WebSocketMessage.connected()
User->>Flutter_UI: 送信テキスト
Flutter_UI->>WSService: sendMessage(text)
WSService->>Backend: { type: "message", content: text }
Backend-->>WSService: { type: "stream", content: chunk }
WSService-->>Flutter_UI: WebSocketMessage.stream(chunk)
Flutter_UI->>Flutter_UI: ストリーミングバッファに追加
Backend-->>WSService: { type: "complete", content: full }
WSService-->>Flutter_UI: WebSocketMessage.complete(full)
Flutter_UI->>Flutter_UI: ボットメッセージを確定して表示
Flutter_UI-->>User: ボット応答を表示
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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. Comment |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
This PR is being reviewed by Cursor Bugbot
Details
You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 8
🤖 Fix all issues with AI agents
In `@flutter/.gitignore`:
- Around line 1-48: Update .gitignore to match the official Flutter/Dart
template by adding the missing patterns: add .packages and .package-config.json,
ignore .flutter_build/ and flutter_export_environment.sh, add *.dill, exclude
android/.gradle/, ignore ios/Flutter/Flutter.framework and
ios/Flutter/Flutter.podspec, add patterns for **/GeneratedPluginRegistrant.* and
IDE folders/files .vscode/ and *.code-workspace; place these alongside the
existing Flutter/Dart entries (near .dart_tool/ and /build/) so generated build
artifacts and workspace files are consistently excluded.
In `@flutter/lib/main.dart`:
- Around line 7-18: main() currently awaits dotenv.load(fileName: '.env')
without handling failures; wrap the dotenv.load call in a try-catch inside
main() to catch exceptions from missing or unreadable .env files (referencing
the main() function and dotenv.load), log the error via your logging mechanism
(or print), and apply sensible defaults or continue startup without crashing
(e.g., fall back to default env values or skip loading); ensure
WidgetsFlutterBinding.ensureInitialized() remains before loading and that runApp
still executes even when dotenv.load fails.
In `@flutter/lib/screens/chat_screen.dart`:
- Line 46: The widget subscribes to _wsService.messageStream via
_wsService.messageStream.listen(...) but never stores the StreamSubscription,
causing a potential memory leak; modify the state (e.g., _ChatScreenState) to
import dart:async, store the returned StreamSubscription from
_wsService.messageStream.listen(...) in a field (e.g., _messageSub), and cancel
that subscription in the widget's dispose() method by calling
_messageSub.cancel(); ensure the subscription field is nullable/checked before
cancelling and that you remove any direct listen(...) calls that don't assign
the result.
In `@flutter/lib/services/websocket_service.dart`:
- Around line 32-55: The connection state is set too early: move setting
_isConnected = true until after the WebSocket handshake completes by awaiting
the channel readiness (use await _channel!.ready) before adding the connected
message so that WebSocketChannel.connect() does not falsely mark the socket
connected; also make dispose() await disconnect() (either by marking dispose()
async and awaiting disconnect() or ensuring disconnect() completes before
calling _messageController.close()) to guarantee proper shutdown order and
resource cleanup.
In `@flutter/lib/widgets/message_bubble.dart`:
- Line 89: Replace the deprecated withOpacity call on the color value: find the
expression "color: colorScheme.onSurface.withOpacity(0.6)" in MessageBubble (or
the widget in message_bubble.dart) and change it to use the new withValues API
by calling withValues(alpha: 0.6) on colorScheme.onSurface so the color respects
Flutter 3.27+ recommendations.
In `@flutter/pubspec.yaml`:
- Around line 45-47: Remove the .env entry from the assets list in pubspec.yaml
so the environment file is not bundled into the app; instead, stop loading
API_BASE_URL and WS_BASE_URL from a packaged .env at runtime and inject them at
build or runtime (use --dart-define to pass API_BASE_URL/WS_BASE_URL into the
Dart build, or fetch secrets from your backend/API, or inject via CI/CD during
release builds). If you need local dev convenience, gate any .env-based loading
behind a development-only flag so releases never include .env in assets.
In `@flutter/README.md`:
- Around line 129-131: The README references a non-existent test path
test/models/message_test.dart while the actual test is at test/widget_test.dart;
update the README to run the correct command (flutter test
test/widget_test.dart) or split/move the existing test into
test/models/message_test.dart and adjust any imports/fixtures accordingly so the
documented path matches the repository layout (ensure test names in README and
the test runner command reference the same file).
🧹 Nitpick comments (19)
backend/pyproject.toml (2)
35-40: [LGTM] セキュリティ依存関係の更新が適切です。
urllib32.6.3 は「リダイレクト時に解凍爆弾のセーフガードがバイパスされる」高深刻度のセキュリティ問題を修正しています。filelock3.20.3 は、SoftFileLock 実装の TOCTOU レースコンディション脆弱性を修正しています。コメントを「Security Fix」に統一したのは良いですが、将来のセキュリティ監査のために CVE 番号をコメントに残すことを検討してください（例:
urllib3は CVE-2026-21441、filelockは CVE-2026-22701）。
37-38: [Nitpick] コメントスタイルの一貫性について。
marshmallowは CVE 番号付きのコメントが残っていますが、他のセキュリティ修正は汎用的な「Security Fix」コメントに変更されています。一貫性のため、統一することを検討してください。💡 コメントスタイルの統一案
選択肢 A: すべて CVE 番号を残す（推奨：監査に有用）
- # Security Fix - "urllib3>=2.6.3", + # Security: CVE-2026-21441 対応 + "urllib3>=2.6.3", # Security: CVE-2025-68480 対応 "marshmallow>=3.26.2", - # Security Fix - "filelock>=3.20.3", - # Security Fix - "aiohttp>=3.13.3", + # Security: CVE-2026-22701 対応 + "filelock>=3.20.3", + # Security: CVE-2025-69223 対応 + "aiohttp>=3.13.3",選択肢 B: すべて汎用コメントに統一
- # Security: CVE-2025-68480 対応 + # Security Fix "marshmallow>=3.26.2",docs/design/monorepo.md (1)
109-141: ✅ CI/CD とフロー環境の分離が実用的GitHub Actions のパスフィルタリングと選択的な開発環境セットアップにより、効率的なモノレポ運用が可能になっています。
💡 横断的な変更への対応ガイダンスの追加を検討
現在の CI/CD 例は個別スタックの変更を扱っていますが、以下のような横断的な変更への対応についても言及すると、より包括的なガイドになります：
docs/ディレクトリの変更時の挙動（全スタックのビルドをトリガーすべきか？）- ルートレベルの設定ファイル（
.github/workflows/内の共通ワークフロー等）の変更時の対応- 複数スタックに影響する変更（例: API スキーマ変更）の調整方法
追加例：
# 複数パスが変更された場合の対応例 on: push: paths: - 'backend/**' - 'docs/**' # ドキュメント変更も検証対象に含める場合flutter/lib/widgets/chat_input.dart (1)
63-64: [nit]onSubmittedでのenabledチェックの追加を検討
TextFieldがenabled: widget.enabledで制御されているため、通常は問題ありませんが、防御的コーディングとして_handleSend内でもenabled状態をチェックすることを検討してください。♻️ 提案
void _handleSend() { + if (!widget.enabled) return; final text = _controller.text.trim(); if (text.isEmpty) return; widget.onSend(text); _controller.clear(); _focusNode.requestFocus(); }flutter/lib/models/message.dart (2)
44-51: [確認]toJsonにisStreamingが含まれていない点について
isStreamingフィールドがtoJsonに含まれていませんが、これはUI状態であり永続化不要という意図でしょうか？意図的であれば問題ありませんが、コメントで明示すると理解しやすくなります。📝 ドキュメントの追加案
+ /// JSONにシリアライズ（isStreamingはUI状態のため除外） Map<String, dynamic> toJson() { return { 'id': id, 'content': content, 'role': role.name, 'timestamp': timestamp.toIso8601String(), }; }
1-58: [推奨]==とhashCodeのオーバーライドを検討
Messageクラスをリスト内での比較やセット操作で使用する場合、==演算子とhashCodeのオーバーライドが有用です。特にListViewでの効率的な再描画や状態管理で役立ちます。♻️ 提案
`@override` bool operator ==(Object other) => identical(this, other) || other is Message && runtimeType == other.runtimeType && id == other.id; `@override` int get hashCode => id.hashCode;flutter/test/widget_test.dart (4)
1-5: [should] テストファイル名がテスト内容と一致していませんこのファイルは
MessageとChatSessionモデルのユニットテストを含んでいますが、ファイル名はwidget_test.dartになっています。models_test.dartやmessage_test.dart、session_test.dartのように分割することで、テストの管理がしやすくなります。
23-36: [should]toJsonテストでtimestampフィールドのアサーションが欠けています
MessageモデルのtoJsonテストでtimestampフィールドの変換結果を検証していません。ISO8601形式への変換が正しく行われることを確認するアサーションを追加してください。♻️ 提案する修正
expect(json['id'], '123'); expect(json['content'], 'Hello'); expect(json['role'], 'user'); + expect(json['timestamp'], isNotNull); });
55-69: [nit] エッジケースのテストを追加することを検討してください
ChatSession.fromJsonテストでは正常系のみをカバーしています。以下のケースもテストすると堅牢性が向上します：
user_idがnullの場合（デフォルト値''への fallback）statusがnullの場合（デフォルト値'active'への fallback）created_atがnullの場合
87-90: [nit] 末尾の空行を削除してくださいファイル末尾に不要な空行があります。
flutter/lib/models/session.dart (3)
17-27: [should]session_idのnullチェックとDateTime.parseのエラーハンドリングを追加してください
session_idがnullの場合、as Stringのキャストで例外がスローされます。また、DateTime.parseは無効な日付形式でFormatExceptionをスローする可能性があります。♻️ 提案する修正
factory ChatSession.fromJson(Map<String, dynamic> json) { + final sessionId = json['session_id']; + if (sessionId == null) { + throw ArgumentError('session_id is required'); + } + + DateTime? createdAt; + if (json['created_at'] != null) { + try { + createdAt = DateTime.parse(json['created_at'] as String); + } catch (_) { + createdAt = null; + } + } + return ChatSession( - sessionId: json['session_id'] as String, + sessionId: sessionId as String, userId: json['user_id'] as String? ?? '', status: json['status'] as String? ?? 'active', - createdAt: json['created_at'] != null - ? DateTime.parse(json['created_at'] as String) - : null, + createdAt: createdAt, metadata: json['metadata'] as Map<String, dynamic>?, ); }
2-40: [nit]Messageモデルとの一貫性のためcopyWithメソッドの追加を検討してください
MessageモデルにはcopyWithメソッドがありますが、ChatSessionにはありません。セッションのステータス更新などで便利な場合があります。
41-44: [nit] 末尾の空行を削除してくださいファイル末尾に不要な空行があります。
flutter/lib/screens/chat_screen.dart (2)
11-22: [imo] Riverpodを使用していますが、プロバイダーが活用されていません。
ConsumerStatefulWidgetを継承していますが、ref.watch()やref.read()が使用されておらず、サービスは直接インスタンス化されています。テスト容易性と依存性注入の観点から、サービスをプロバイダー経由で提供することを検討してください。現状の実装でも動作しますが、将来的にプロバイダーを導入する場合は以下のようなパターンが考えられます：
// providers.dart final apiServiceProvider = Provider((ref) => ApiService()); final webSocketServiceProvider = Provider((ref) => WebSocketService()); // chat_screen.dart final apiService = ref.read(apiServiceProvider);
229-235: [imo]withOpacityは非推奨のためwithValuesへの移行を検討してください。Flutter では
withOpacityは非推奨となり、withValuesの使用が推奨されています。withOpacityは alpha 値を 8 ビット整数に変換して精度が低下しますが、withValuesは alpha を浮動小数点値として保持し、ワイドガマット色に対応しています。♻️ 修正案
- color: Theme.of(context).colorScheme.primary.withOpacity(0.5), + color: Theme.of(context).colorScheme.primary.withValues(alpha: 0.5),- color: Theme.of(context).colorScheme.onSurface.withOpacity(0.6), + color: Theme.of(context).colorScheme.onSurface.withValues(alpha: 0.6),flutter/lib/services/api_service.dart (2)
20-38: [want] HTTP リクエストにタイムアウトを設定することを推奨します。現在の実装ではネットワーク障害時にリクエストが無期限にハングする可能性があります。
httpパッケージでは直接タイムアウトを設定できないため、Future.timeout()でラップするか、タイムアウト付きのIOClientを使用してください。♻️ 修正案 (Future.timeout を使用)
Future<ChatSession> createSession({Map<String, dynamic>? metadata}) async { - final response = await _client.post( + final response = await _client.post( Uri.parse('$baseUrl/api/chat/sessions'), headers: {'Content-Type': 'application/json'}, body: jsonEncode({ 'metadata': metadata ?? {'language': 'ja'}, }), - ); + ).timeout( + const Duration(seconds: 30), + onTimeout: () => throw ApiException( + statusCode: 408, + message: 'リクエストがタイムアウトしました', + ), + ); // ... }
42-45: [nits] GET リクエストにはContent-Typeヘッダーは不要です。
Content-Typeはリクエストボディの形式を示すヘッダーですが、GET リクエストにはボディがないため不要です。動作には影響しませんが、削除するとクリーンになります。final response = await _client.get( Uri.parse('$baseUrl/api/chat/sessions/$sessionId'), - headers: {'Content-Type': 'application/json'}, );flutter/lib/services/websocket_service.dart (2)
105-109: [want]dispose()内でdisconnect()を await していません。
disconnect()は非同期メソッドですが、dispose()内で await されていないため、_messageController.close()が先に実行される可能性があります。♻️ 修正案
- void dispose() { - disconnect(); - _messageController.close(); - } + Future<void> dispose() async { + await disconnect(); + await _messageController.close(); + }呼び出し元（
ChatScreen.dispose()）でも対応が必要です：`@override` void dispose() { _messageSubscription?.cancel(); _scrollController.dispose(); _wsService.dispose(); // void -> Future<void> になるため、非同期処理が必要な場合は別途対応 _apiService.dispose(); super.dispose(); }
40-47: [imo] StreamController が閉じられた後にイベントが追加される可能性があります。
onErrorやonDoneコールバックはdispose()後に呼ばれる可能性があります。閉じられたStreamControllerにadd()すると例外が発生します。♻️ 修正案
+ bool _isDisposed = false; + + void _safeAdd(WebSocketMessage message) { + if (!_isDisposed && !_messageController.isClosed) { + _messageController.add(message); + } + } // 使用箇所を _safeAdd に置き換え onError: (dynamic error) { - _messageController.add(WebSocketMessage.error(error.toString())); + _safeAdd(WebSocketMessage.error(error.toString())); _isConnected = false; },
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (2)
backend/uv.lockis excluded by!**/*.lockflutter/pubspec.lockis excluded by!**/*.lock
📒 Files selected for processing (19)
backend/pyproject.tomldocs/design/0-index.mddocs/design/directory-structure.mddocs/design/monorepo.mddocs/design/tech-stack.mdflutter/.gitignoreflutter/README.mdflutter/analysis_options.yamlflutter/env.exampleflutter/lib/main.dartflutter/lib/models/message.dartflutter/lib/models/session.dartflutter/lib/screens/chat_screen.dartflutter/lib/services/api_service.dartflutter/lib/services/websocket_service.dartflutter/lib/widgets/chat_input.dartflutter/lib/widgets/message_bubble.dartflutter/pubspec.yamlflutter/test/widget_test.dart
🧰 Additional context used
🧠 Learnings (13)
📚 Learning: 2025-12-10T23:48:35.607Z
Learnt from: CR
Repo: zucky2021/ai-chatbot-api PR: 0
File: docs/design/lambda/dir-structure.md:0-0
Timestamp: 2025-12-10T23:48:35.607Z
Learning: lambda-monorepo構造に準拠してLambda関数のディレクトリ構造を設計する
Applied to files:
docs/design/monorepo.mddocs/design/directory-structure.mddocs/design/0-index.md
📚 Learning: 2025-12-10T23:47:56.547Z
Learnt from: CR
Repo: zucky2021/ai-chatbot-api PR: 0
File: docs/design/0-index.md:0-0
Timestamp: 2025-12-10T23:47:56.547Z
Learning: Reference and maintain the directory structure documentation in directory-structure.md
Applied to files:
docs/design/directory-structure.md
📚 Learning: 2025-12-10T23:48:25.109Z
Learnt from: CR
Repo: zucky2021/ai-chatbot-api PR: 0
File: docs/design/lambda/0-index.md:0-0
Timestamp: 2025-12-10T23:48:25.109Z
Learning: Organize Lambda function project using the directory structure documented in ./dir-structure.md
Applied to files:
docs/design/directory-structure.mddocs/design/0-index.md
📚 Learning: 2025-12-10T23:48:35.607Z
Learnt from: CR
Repo: zucky2021/ai-chatbot-api PR: 0
File: docs/design/lambda/dir-structure.md:0-0
Timestamp: 2025-12-10T23:48:35.607Z
Learning: Applies to docs/design/lambda/lambda/app.py : CDKアプリケーションのエントリーポイントは `lambda/app.py` として配置する
Applied to files:
docs/design/directory-structure.md
📚 Learning: 2025-12-10T23:47:56.547Z
Learnt from: CR
Repo: zucky2021/ai-chatbot-api PR: 0
File: docs/design/0-index.md:0-0
Timestamp: 2025-12-10T23:47:56.547Z
Learning: Reference and maintain the Lambda documentation in lambda/0-index.md
Applied to files:
docs/design/0-index.md
📚 Learning: 2025-12-10T23:48:35.607Z
Learnt from: CR
Repo: zucky2021/ai-chatbot-api PR: 0
File: docs/design/lambda/dir-structure.md:0-0
Timestamp: 2025-12-10T23:48:35.607Z
Learning: Applies to docs/design/lambda/lambda/functions/*/ : Lambda関数は `lambda/functions/` ディレクトリ配下に独立したディレクトリとして配置する
Applied to files:
docs/design/0-index.md
📚 Learning: 2025-12-10T23:48:35.607Z
Learnt from: CR
Repo: zucky2021/ai-chatbot-api PR: 0
File: docs/design/lambda/dir-structure.md:0-0
Timestamp: 2025-12-10T23:48:35.607Z
Learning: Applies to docs/design/lambda/lambda/infra/ : CDKスタック定義、Lambda関数のデプロイ設定、EventBridgeルール、IAMロール定義は `lambda/infra/` ディレクトリに配置する
Applied to files:
docs/design/0-index.md
📚 Learning: 2025-12-10T23:48:35.607Z
Learnt from: CR
Repo: zucky2021/ai-chatbot-api PR: 0
File: docs/design/lambda/dir-structure.md:0-0
Timestamp: 2025-12-10T23:48:35.607Z
Learning: Applies to docs/design/lambda/lambda/{shared,functions/shared}/ : Lambda関数間で共有するコード（設定管理、ロギング、データベース接続など）は `lambda/shared/` または `lambda/functions/shared/` に配置する
Applied to files:
docs/design/0-index.md
📚 Learning: 2025-12-10T23:48:35.607Z
Learnt from: CR
Repo: zucky2021/ai-chatbot-api PR: 0
File: docs/design/lambda/dir-structure.md:0-0
Timestamp: 2025-12-10T23:48:35.607Z
Learning: Applies to docs/design/lambda/lambda/cdk.json : CDK設定は `lambda/cdk.json` ファイルで定義する
Applied to files:
docs/design/0-index.md
📚 Learning: 2025-12-10T23:48:35.607Z
Learnt from: CR
Repo: zucky2021/ai-chatbot-api PR: 0
File: docs/design/lambda/dir-structure.md:0-0
Timestamp: 2025-12-10T23:48:35.607Z
Learning: Applies to docs/design/lambda/lambda/uv.lock : 依存関係ロックファイルは `lambda/uv.lock` として管理する
Applied to files:
docs/design/0-index.md
📚 Learning: 2025-12-10T23:48:35.607Z
Learnt from: CR
Repo: zucky2021/ai-chatbot-api PR: 0
File: docs/design/lambda/dir-structure.md:0-0
Timestamp: 2025-12-10T23:48:35.607Z
Learning: Applies to docs/design/lambda/lambda/functions/*/lambda_function.py : Lambda関数のエントリーポイントは `lambda_function.py` として配置する
Applied to files:
docs/design/0-index.md
📚 Learning: 2025-12-10T23:48:35.607Z
Learnt from: CR
Repo: zucky2021/ai-chatbot-api PR: 0
File: docs/design/lambda/dir-structure.md:0-0
Timestamp: 2025-12-10T23:48:35.607Z
Learning: Applies to docs/design/lambda/lambda/pyproject.toml : Lambda関数の実行時依存関係（boto3、psycopg2-binary、asyncpgなど）は `lambda/pyproject.toml` の `dependencies` セクションに定義する
Applied to files:
backend/pyproject.toml
📚 Learning: 2025-12-10T23:48:08.157Z
Learnt from: CR
Repo: zucky2021/ai-chatbot-api PR: 0
File: docs/design/dataflow.md:0-0
Timestamp: 2025-12-10T23:48:08.157Z
Learning: Use WebSocket for real-time bidirectional communication between frontend and backend for chat message transmission
Applied to files:
flutter/lib/services/websocket_service.dart
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Cursor Bugbot
🔇 Additional comments (25)
backend/pyproject.toml (1)
41-42: info: aiohttp は間接依存として適切にセキュリティバージョンに固定されていますコードベースで aiohttp の直接インポートはありませんが、backend/pyproject.toml に直接依存として列挙されていることから、LangChain などの依存パッケージが使用している間接依存を意図的にセキュリティアップデートバージョンに固定しているものと考えられます。aiohttp 3.13.3 は複数の脆弱性を修正しており、この固定は適切です。
docs/design/0-index.md (1)
9-10: 👍 LGTM: インデックスの論理的な再構成モノレポ設計を Lambda の前に配置することで、より広範なアーキテクチャ上の概念を先に説明する構成になっており、ドキュメントの流れが改善されています。
docs/design/directory-structure.md (2)
7-7: 👍 インラインコメントでディレクトリの役割が明確化各ディレクトリに使用技術スタックのコメントを追加することで、プロジェクト構成の理解が容易になっています。
Also applies to: 10-10, 25-27
16-24: ✅ Flutter プロジェクト構造が標準的Flutter プロジェクトの標準的なディレクトリ構造に従っており、models/、screens/、services/、widgets/ といった一般的なアーキテクチャパターンが採用されています。
docs/design/monorepo.md (4)
1-17: ✅ プロジェクト概要とリポジトリ構成が明確モノレポ構成の概要と各ディレクトリの役割が簡潔に説明されており、プロジェクト全体の構造が理解しやすくなっています。
19-95: 👍 選定理由と代替案の比較が充実モノレポ、Git Submodule、Git Subtree、独立リポジトリの4つの選択肢について、それぞれのメリット・デメリットを明確に比較しており、意思決定の根拠が十分に説明されています。個人開発・学習目的という文脈に即した判断基準も適切です。
96-108: ✅ ディレクトリごとの独立性が適切に設計各スタックが独自の依存関係管理を持つ設計により、技術スタック間の分離が明確になっています。uv、pnpm、pub といった各エコシステムに最適なツールが選択されている点も適切です。
142-155: ✅ モノレポの限界と参考資料が確認されましたモノレポが適さなくなるケースが明確に定義されており、将来的な判断基準として有効です。参考資料のすべてのリンク（monorepo.tools、Git Submodules、Git Subtree）は有効で、HTTP 200ステータスで確認されました。
docs/design/tech-stack.md (3)
85-93: info: Dart の選定理由が適切に記載されています。型安全性、async/await による非同期処理、null safety など、現代的なプログラミング言語として重要な特徴が網羅されており、公式サイトのリンクも有効です。
94-102: info: Riverpodの選定は適切コンパイル時の安全性やテスタビリティの向上など、Riverpodの利点が正確に説明されています。Providerの進化版として位置づけられている点も適切です。公式サイト（https://riverpod.dev/）も確認しており、リンクが有効であることを確認しました。
73-84: info: Flutter の選定理由が明確で、クロスプラットフォーム開発、宣言的UI、ホットリロード、Material Design 3対応など主要な利点が適切に記載されています。公式サイトのリンクも有効です。flutter/analysis_options.yaml (2)
5-38: [LGTM] 適切な linter ルールの設定エラー防止、スタイル、ドキュメンテーションのカテゴリに分類された包括的な linter ルールが設定されています。生成されたファイル（
*.g.dart、*.freezed.dart）の除外設定も適切です。
3-3: nits: ファイルの末尾に余分な空行があります。3行目の
include: package:flutter_lints/flutter.yamlは正しい指定です。flutter_lintsパッケージは Flutter チームから公式に推奨されている現在のスタンダードであり、pubspec.yamlに正しく依存関係として追加されています。ただし、40-42行目の末尾に余分な空行を削除してください。⛔ Skipped due to learnings
Learnt from: CR Repo: zucky2021/ai-chatbot-api PR: 0 File: docs/development/coding-rules/0-index.md:0-0 Timestamp: 2025-12-10T23:49:14.522Z Learning: Refer to the レビュー (Review) documentation for code review standardsflutter/lib/widgets/chat_input.dart (2)
18-36: [LGTM] 適切なリソース管理
TextEditingControllerとFocusNodeの両方がdispose()で正しく破棄されています。これによりメモリリークを防止しています。
38-93: [LGTM] UIの実装が適切
SafeArea、テーマカラーの使用、マルチライン対応のテキストフィールド、無効状態のヒントテキスト切り替えなど、適切なUI実装です。flutter/lib/widgets/message_bubble.dart (1)
16-112: [LGTM] メッセージバブルの実装が適切ユーザー/アシスタントによる整列の切り替え、Markdownサポート、ストリーミングインジケーター、タイムスタンプ表示など、チャットUIに必要な機能が適切に実装されています。
MediaQueryを使用した最大幅の制約（80%）も良いアプローチです。flutter/env.example (1)
1-3: [info] 環境変数のテンプレートファイルローカル開発用のデフォルト設定が適切に定義されています。
.envファイルはflutter/.gitignoreで正しく除外されています。flutter/lib/models/message.dart (1)
33-42: [ask]fromJsonでの空のIDフォールバックについて質問
id: json['id'] as String? ?? ''で、json['id']がnullの場合に空文字列にフォールバックしていますが、これは意図した動作ですか？テスト（widget_test.dart:18）ではmessage.idに'123'が期待されており、空文字列は比較やデータベースのユニークキーとして問題を起こす可能性があります。対比として、
ChatSession.fromJsonではjson['session_id'] as Stringとして値が必須になっています。Message IDも同様に必須とするか、それとも特定の理由で空文字列フォールバックが必要な場合は、その背景を教えていただけますか？[ask]
flutter/lib/main.dart (1)
20-54: [approve] テーマ設定が適切に構成されていますMaterial 3の使用、ライト/ダークテーマの両対応、システムテーマモードへの追従など、良い実装です。
flutter/README.md (1)
1-149: [approve] ドキュメントが充実していますセットアップ手順、プロジェクト構成、技術スタック、トラブルシューティングなど、開発者が必要とする情報が網羅されています。
flutter/pubspec.yaml (1)
18-20: nits: 互換性の懸念は不要です
flutter_riverpod: ^3.1.0とriverpod_annotation: ^4.0.0は互換性があります。異なるメジャーバージョンでも、Riverpodの公式ドキュメントでは両パッケージの組み合わせがサポートされており、pubspec.lockでも正常に解決されています。Likely an incorrect or invalid review comment.
flutter/lib/screens/chat_screen.dart (1)
144-150: [ask] 会話クリア時の意図を確認してください。現在の実装ではローカルのメッセージリストのみをクリアし、セッションは維持されます。バックエンド側に会話履歴が保持されている場合、AI の応答コンテキストはリセットされない可能性があります。
新しいセッションを作成して完全にリセットする動作が必要な場合は、以下のような修正が考えられます：
onPressed: () async { setState(() { _messages.clear(); _error = null; _isLoading = true; }); await _wsService.disconnect(); await _initialize(); // 新しいセッションを作成 },flutter/lib/services/api_service.dart (1)
9-17: [good] コンストラクタでの依存性注入のサポートが良い設計です。
http.ClientとbaseUrlをオプションパラメータとして受け取ることで、テスト時にモックを注入しやすくなっています。flutter/lib/services/websocket_service.dart (2)
112-151: [good]WebSocketMessageクラスの設計が良いです。イミュータブルなデータクラスとして適切に設計されており、名前付きコンストラクタでタイプセーフなメッセージ生成が可能です。
constコンストラクタの使用も効率的です。Dart 3 以降では
sealed classを使用すると、switch 文での網羅性チェックが可能になります（オプション）。
7-16: [good] リアルタイム通信に WebSocket を使用する設計方針に沿った実装です。Based on learnings, WebSocket を使用したリアルタイム双方向通信の実装が適切に行われています。ストリーミングメッセージの処理とイベント駆動型のアーキテクチャが良く設計されています。
✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Fix all issues with AI agents
In `@flutter/lib/screens/chat_screen.dart`:
- Around line 59-93: The switch in _handleWebSocketMessage completes normally
because each case (WebSocketMessageType.stream, .complete, .error,
.disconnected) is missing a terminating break/return; update each case block to
end with a break (or return if you intend to exit early) after calling
setState/_scrollToBottom so Dart's switch_case_completes_normally error is
resolved and control doesn't fall through to the next case.
In `@flutter/lib/widgets/message_bubble.dart`:
- Around line 85-107: The alpha parameter passed to ColorScheme.withValues is
currently an integer (from (0.6 * 255).round()) which is invalid; update both
calls (colorScheme.onSurface.withValues(...) and
colorScheme.onPrimaryContainer.withValues(...)) to pass a floating-point value
in the [0.0, 1.0] range (e.g., 0.6) for the alpha argument so the timestamp Text
and the "入力中..." Text use correct opacity.
♻️ Duplicate comments (4)
flutter/lib/main.dart (1)
7-24: [must] LGTM - dotenvのエラーハンドリングが適切に実装されています前回のレビューで指摘された
.envファイル読み込み失敗時のエラーハンドリングが正しく実装されています。try-catchでエラーをキャッチし、アプリ自体は起動を継続する設計になっています。flutter/lib/screens/chat_screen.dart (3)
45-46: CR: StreamSubscription を保持して dispose で cancel してください。Also applies to: 127-131
96-112: CR: 切断後送信時の例外ハンドリングが必要です。
172-202: CR: _error があっても _sessionId != null だと表示されません。
🧹 Nitpick comments (4)
flutter/lib/main.dart (1)
26-60: [imo] MaterialAppの設定は適切ですMaterial 3を使用し、システムテーマモードに対応した実装です。
ColorScheme.fromSeedを使ったシードカラーベースのテーマ設定は良いプラクティスです。♻️ 任意: AppBarThemeの重複を解消
// 共通のAppBarThemeを抽出可能 const _appBarTheme = AppBarTheme( centerTitle: true, elevation: 0, );flutter/test/session_test.dart (1)
7-21: [imo] fromJsonテストでcreatedAtの検証も追加を検討基本的なJSONパースのテストは良好です。
created_atフィールドも正しくパースされているか確認すると、より堅牢なテストになります。expect(session.createdAt, isNotNull); expect(session.createdAt!.year, 2024);flutter/lib/services/api_service.dart (1)
74-93: [imo] getConversationHistoryの404ハンドリングを検討してくださいセッションが存在しない場合（404）に空のリストを返すか、nullを返すかの明示的なハンドリングがあると、呼び出し側での処理が一貫します。
また、
data.cast<Map<String, dynamic>>()は配列内に予期しない型がある場合に実行時エラーになる可能性があります。♻️ 提案
+ if (response.statusCode == 404) { + return []; + } + if (response.statusCode != 200) { throw ApiException( statusCode: response.statusCode, message: '会話履歴の取得に失敗しました', ); } final data = jsonDecode(response.body) as List<dynamic>; - return data.cast<Map<String, dynamic>>(); + return data.whereType<Map<String, dynamic>>().toList();flutter/lib/screens/chat_screen.dart (1)
36-55: CR: await 後の setState に mounted チェックを入れてください。
非同期後に dispose 済みだと例外化するため、ガードを入れるのが安全です。♻️ 修正案
Future<void> _initialize() async { try { // セッション作成 final session = await _apiService.createSession(); + if (!mounted) return; _sessionId = session.sessionId; // WebSocket接続 await _wsService.connect(_sessionId!); + if (!mounted) return; // メッセージストリームを購読 _wsService.messageStream.listen(_handleWebSocketMessage); setState(() { _isLoading = false; }); } catch (e) { + if (!mounted) return; setState(() { _isLoading = false; _error = 'セッション作成に失敗しました: $e'; }); } }
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (9)
.gitignoreflutter/.gitignoreflutter/lib/main.dartflutter/lib/models/session.dartflutter/lib/screens/chat_screen.dartflutter/lib/services/api_service.dartflutter/lib/widgets/message_bubble.dartflutter/test/message_test.dartflutter/test/session_test.dart
✅ Files skipped from review due to trivial changes (1)
- .gitignore
🚧 Files skipped from review as they are similar to previous changes (2)
- flutter/.gitignore
- flutter/lib/models/session.dart
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: CR
Repo: zucky2021/ai-chatbot-api PR: 0
File: docs/design/tech-stack.md:0-0
Timestamp: 2025-12-19T14:16:50.221Z
Learning: Applies to docs/design/backend/**/*.py : Implement WebSocket for real-time bidirectional communication and AI streaming responses
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Cursor Bugbot
🔇 Additional comments (16)
flutter/test/session_test.dart (3)
23-36: [must] LGTM - toJsonテストは適切です
ChatSessionからJSONへの変換が正しく行われることを確認しています。
38-49: [must] LGTM - nullハンドリングのテストは適切です
session_idがnullの場合に空文字列にフォールバックすることを確認しています。堅牢なエラーハンドリングのテストです。
51-62: [must] LGTM - 無効な日付のハンドリングテストは適切です無効な日付文字列が渡された場合に
createdAtがnullになることを確認しています。flutter/lib/widgets/message_bubble.dart (2)
21-42: [must] LGTM - メッセージバブルのレイアウトは適切ですユーザーとアシスタントで異なるアライメントと色を使用し、角丸の形状でチャットらしいUIを実現しています。
maxWidth制約による幅制限も適切です。
52-69: [must] LGTM - Markdownレンダリングの設定は適切ですアシスタントメッセージに対してMarkdownをレンダリングし、コードブロックのスタイリングとテキスト選択機能を提供しています。
flutter/test/message_test.dart (3)
7-20: [must] LGTM - fromJsonテストは基本的なケースをカバーしていますJSONからMessageオブジェクトへの変換が正しく動作することを確認しています。
22-36: [must] LGTM - toJsonテストは適切にUTCを使用しています
DateTime.utcを使用することでタイムゾーンに依存しない確定的なテストになっています。ISO 8601形式の出力確認も適切です。
38-51: [must] LGTM - copyWithテストは適切です元のオブジェクトのフィールドが保持され、指定したフィールドのみが更新されることを確認しています。イミュータブルパターンの正しい実装テストです。
flutter/lib/services/api_service.dart (5)
9-24: [must] LGTM - ApiServiceの設計は適切です依存性注入パターンを採用し、
http.Clientのモック化によるテストが容易になっています。タイムアウト例外をconstで定義することでメモリ効率も良好です。
26-47: [must] LGTM - createSessionは適切に実装されていますPOSTリクエストに必要なヘッダーとボディが正しく設定され、成功時のステータスコード(200/201)の確認も適切です。
50-71: [must] LGTM - getSessionは適切です404の場合にnullを返すパターンは、セッションが存在しない場合の正当なユースケースを適切に処理しています。
96-105: [must] LGTM - healthCheckは適切ですヘルスチェックの失敗時に例外を投げずにfalseを返すのは、UIでの利用に適したパターンです。
107-124: [must] LGTM - リソース管理と例外クラスは適切です
dispose()でHTTPクライアントを適切にクローズし、ApiExceptionはデバッグしやすいtoString()を提供しています。flutter/lib/screens/chat_screen.dart (3)
115-125: CR: 自動スクロールの実装はシンプルで良いです。
221-247: CR: 空状態のUIは分かりやすくて良いです。
249-269: CR: ストリーミング中の一時メッセージ表示は自然なUXです。
✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@flutter/lib/screens/chat_screen.dart`:
- Around line 36-56: The _initialize method calls setState after awaiting async
work and can trigger "setState called after dispose"; wrap all post-await
setState calls in a mounted check (e.g., if (!mounted) return; or if (mounted)
setState(...)) and do the same inside _handleWebSocketMessage if it calls
setState; also ensure the WebSocket subscription via
_wsService.messageStream.listen is cancelled in dispose (store the
StreamSubscription) to avoid callbacks after dispose; update references:
_initialize, _apiService.createSession, _wsService.connect,
_wsService.messageStream, _handleWebSocketMessage, and setState accordingly.
♻️ Duplicate comments (3)
flutter/lib/services/websocket_service.dart (1)
36-71: 指摘: 接続確立前に_isConnectedを true にしないでください。ハンドシェイク完了前に接続済み扱いになると、失敗時に UI/送信側が誤動作します。
await _channel!.ready後に状態更新してください。🔧 修正案
- _channel = WebSocketChannel.connect(uri); - _isConnected = true; + _channel = WebSocketChannel.connect(uri); + await _channel!.ready; + _isConnected = true;web_socket_channel WebSocketChannel.ready Future<void> handshake await Dart 3.0.3flutter/lib/screens/chat_screen.dart (2)
18-47: 指摘: メッセージ購読の解除がなくメモリリークになります。
listen()の戻り値を保持し、dispose()で必ずcancel()してください。🔧 修正案
+import 'dart:async'; import 'package:flutter/material.dart'; import 'package:flutter_riverpod/flutter_riverpod.dart'; class _ChatScreenState extends ConsumerState<ChatScreen> { final List<Message> _messages = []; final ScrollController _scrollController = ScrollController(); final ApiService _apiService = ApiService(); final WebSocketService _wsService = WebSocketService(); + StreamSubscription<WebSocketMessage>? _messageSubscription; Future<void> _initialize() async { // ... - _wsService.messageStream.listen(_handleWebSocketMessage); + _messageSubscription = + _wsService.messageStream.listen(_handleWebSocketMessage); } `@override` void dispose() { + _messageSubscription?.cancel(); _scrollController.dispose(); _wsService.dispose(); _apiService.dispose(); super.dispose(); }Also applies to: 131-136
100-116: 指摘: WebSocket 送信失敗時の例外を握り潰さないでください。切断後に
StateErrorが投げられるため UI がクラッシュします。🔧 修正案
void _sendMessage(String content) { if (content.trim().isEmpty) return; setState(() { _messages.add(Message( id: DateTime.now().millisecondsSinceEpoch.toString(), content: content, role: MessageRole.user, timestamp: DateTime.now(), )); _streamingContent = ''; }); - _wsService.sendMessage(content); + try { + _wsService.sendMessage(content); + } catch (e) { + setState(() { + _error = '送信に失敗しました: $e'; + }); + return; + } _scrollToBottom(); }
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
flutter/lib/screens/chat_screen.dartflutter/lib/services/websocket_service.dartflutter/lib/widgets/message_bubble.dart
🚧 Files skipped from review as they are similar to previous changes (1)
- flutter/lib/widgets/message_bubble.dart
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-12-10T23:48:08.157Z
Learnt from: CR
Repo: zucky2021/ai-chatbot-api PR: 0
File: docs/design/dataflow.md:0-0
Timestamp: 2025-12-10T23:48:08.157Z
Learning: Use WebSocket for real-time bidirectional communication between frontend and backend for chat message transmission
Applied to files:
flutter/lib/services/websocket_service.dart
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Cursor Bugbot
🔇 Additional comments (11)
flutter/lib/services/websocket_service.dart (6)
26-34: 指摘: 安全なストリーム追加のガードが明確です。
_isDisposedとisClosedを両方見る実装は堅実です。
74-102: 指摘: 受信メッセージの型分岐が明快で堅牢です。例外時も
WebSocketMessage.errorに落とす流れは妥当です。
104-116: 指摘: 送信ペイロードの構造が明確です。JSON 形式も一貫していて読みやすいです。
118-126: 指摘: 切断処理の後始末が丁寧です。サブスク解除 → sink close → null 化の順序は適切です。
128-143: 指摘: dispose の順序制御が良いです。
whenCompleteで controller close を遅らせる設計は実用的です。
146-185: 指摘: WebSocketMessage と enum の設計が分かりやすいです。型安全かつ用途別のコンストラクタで扱いやすいです。
flutter/lib/screens/chat_screen.dart (5)
59-97: 指摘: ストリーミング/完了/エラーの分岐が明確です。UI 反映とスクロール処理の流れが読みやすいです。
119-129: 指摘: 自動スクロールの実装は適切です。
addPostFrameCallbackとhasClientsの組み合わせは安全です。
139-223: 指摘: UI の状態分岐が分かりやすいです。読み込み/エラー/通常表示の切替が明確です。
225-251: 指摘: 空状態のガイド表示が良いです。色やメッセージのトーンが UI に馴染んでいます。
253-274: 指摘: ストリーミング中のメッセージ表現が自然です。
isStreamingを Message に持たせる設計は UI 拡張に有利です。
✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.
| Future<void> _initialize() async { | ||
| try { | ||
| // セッション作成 | ||
| final session = await _apiService.createSession(); | ||
| _sessionId = session.sessionId; | ||
|
|
||
| // WebSocket接続 | ||
| await _wsService.connect(_sessionId!); | ||
|
|
||
| // メッセージストリームを購読 | ||
| _wsService.messageStream.listen(_handleWebSocketMessage); | ||
|
|
||
| setState(() { | ||
| _isLoading = false; | ||
| }); | ||
| } catch (e) { | ||
| setState(() { | ||
| _isLoading = false; | ||
| _error = 'セッション作成に失敗しました: $e'; | ||
| }); | ||
| } |
There was a problem hiding this comment.
指摘: 非同期後の setState は mounted を確認してください。
画面遷移直後に完了すると setState() called after dispose になります。
🔧 修正案
Future<void> _initialize() async {
try {
final session = await _apiService.createSession();
+ if (!mounted) return;
_sessionId = session.sessionId;
await _wsService.connect(_sessionId!);
+ if (!mounted) return;
_wsService.messageStream.listen(_handleWebSocketMessage);
- setState(() {
+ if (!mounted) return;
+ setState(() {
_isLoading = false;
});
} catch (e) {
- setState(() {
+ if (!mounted) return;
+ setState(() {
_isLoading = false;
_error = 'セッション作成に失敗しました: $e';
});
}
}🤖 Prompt for AI Agents
In `@flutter/lib/screens/chat_screen.dart` around lines 36 - 56, The _initialize
method calls setState after awaiting async work and can trigger "setState called
after dispose"; wrap all post-await setState calls in a mounted check (e.g., if
(!mounted) return; or if (mounted) setState(...)) and do the same inside
_handleWebSocketMessage if it calls setState; also ensure the WebSocket
subscription via _wsService.messageStream.listen is cancelled in dispose (store
the StreamSubscription) to avoid callbacks after dispose; update references:
_initialize, _apiService.createSession, _wsService.connect,
_wsService.messageStream, _handleWebSocketMessage, and setState accordingly.
| _error = 'セッション作成に失敗しました: $e'; | ||
| }); | ||
| } | ||
| } |
There was a problem hiding this comment.
setState called after dispose in async initialization
Medium Severity
The async _initialize() method is called from initState() and awaits network operations (createSession, connect) before calling setState(). If the user navigates away while these operations are pending, the widget is disposed but _initialize continues. The subsequent setState() calls at lines 48 and 52 throw "setState() called after dispose()" because there's no mounted check before calling setState.
flutter/pubspec.yaml
Outdated
|
|
||
| assets: | ||
| - assets/ | ||
| - .env |
There was a problem hiding this comment.
Missing assets directory causes Flutter build failure
High Severity
The pubspec.yaml declares assets/ as a required asset directory, but no assets/ directory exists in the Flutter project. This causes flutter pub get and flutter build to fail with an error like "unable to find directory entry in pubspec.yaml: assets/". The app cannot be built in its current state.
| role: json['role'] == 'assistant' ? MessageRole.assistant : MessageRole.user, | ||
| timestamp: json['timestamp'] != null | ||
| ? DateTime.parse(json['timestamp'] as String) | ||
| : DateTime.now(), |
There was a problem hiding this comment.
DateTime.parse throws on invalid timestamp strings
Medium Severity
Message.fromJson uses DateTime.parse() which throws a FormatException if the timestamp string is malformed. In contrast, ChatSession.fromJson correctly uses DateTime.tryParse() which returns null safely. If the backend API returns an invalid timestamp, the app will crash when parsing messages.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
| default: | ||
| break; | ||
| } | ||
| } |
There was a problem hiding this comment.
Missing mounted check before setState in async callbacks
Medium Severity
The _handleWebSocketMessage method calls setState() without checking mounted, while the onError callback and _showSnackBar correctly include this check. Similarly, _initialize calls setState() after async operations without verifying the widget is still mounted. If the user navigates away while receiving WebSocket messages or during initialization, this causes "setState() called after dispose()" Flutter errors.
Additional Locations (1)
1 participant
Note
Introduces a cross-platform Flutter client and hardens backend dependencies.
flutter/module with Riverpod-based UI,WebSocketstreaming,Markdownrendering, dark mode, and unit tests formodels(message/session)docs/design/monorepo.md; updatetech-stack.md(Flutter section) anddirectory-structure.md; update indexaiohttpto3.13.3,urllib3to>=2.6.3,filelockto>=3.20.3; addpyasn1>=0.6.2; lockfile updated andbackendmetadata reflects changes.gitignore(addtodo.md)Written by Cursor Bugbot for commit 4ba2c02. This will update automatically on new commits. Configure here.
Summary by CodeRabbit
新機能
ドキュメント
Chores
Tests
✏️ Tip: You can customize this high-level summary in your review settings.