Conversation
- バックエンドセキュリティテスト追加 (validate_test.ts) - フロントエンドテスト環境整備 (Vitest + happy-dom) - store, fetchApi, msgFromClient のユニットテスト追加 - deno.jsonにテストタスク追加 (test, test:frontend, test:all) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- XSS対策: handleJoinRoomにJoinRoomRequestSchema適用 - CSRF対策: Origin検証ミドルウェア追加（API/WebSocket両対応） - エラーハンドリング: サイレントcatch排除、エラーログ追加 - テスト追加: sanitize()エッジケース、XSSサニタイズ、CSRF検証 - E2E環境整備: helpers.ts、install_browsers.ts追加、テスト修正 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- webSocket.tsのユニットテスト追加（13テスト） - WebSocket URL生成（HTTP/HTTPS） - 接続設定（immediate, autoReconnect, heartbeat） - 認証メッセージ送信 - roomメッセージ受信処理 - 不正JSON/空データのエラーハンドリング フロントエンドテスト: 27 → 40件 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- DisconnectTimerManager: 切断タイマー管理をクラス化 - parseJsonBody: JSON解析ロジックを共通化 - updateParticipant: 参加者更新パターンを抽出 - extractRoomId: URLルーティングパターンを統一 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- roomCleanup: 空ルームの自動クリーンアップ機能 - WatcherManager: KV Watcherの停止管理（AbortController利用） - トークンTTL: 24時間の有効期限をDeno KV expireInで設定 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Phase 1: セキュリティ・安定性
Phase 2: テストカバレッジ向上
Phase 3: リファクタリング
Phase 4: 最適化
Test plan
🤖 Generated with Claude Code