Open
Conversation
세션 키를 user_id에서 channel_id로 변경하여 같은 유저가 여러 채널에서 독립적인 세션을 가질 수 있도록 리팩토링. - ConversationSession에 channel_id 필드 추가 - get_or_create_session(channel_id, user_id) 시그니처 변경 - to_dict()에 channel_id 포함 - on_message에서 channel.id 기반 세션 조회 - 전체 테스트 채널 기반으로 수정 + 다중 채널 세션 테스트 추가 Co-Authored-By: Claude Opus 4.6 <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
ConversationSession에channel_id필드 추가, 세션 키를user_id→channel_id로 변경get_or_create_session(channel_id, user_id)시그니처로 같은 유저가 여러 채널에서 독립 세션 보유 가능on_message에서message.channel.id기반 세션 조회,to_dict()에channel_id포함Test plan
test_session_manager.py22개 테스트 통과 (채널 기반 + 다중 채널 세션 격리 테스트 추가)test_on_message.py15개 테스트 통과test_integration.py5개 테스트 통과Closes #72
🤖 Generated with Claude Code