Skip to content

fix(session): always post session info as persistent message#6

Merged
Chanki-Min merged 3 commits intomainfrom
worktree-fix-always-new-session
Mar 10, 2026
Merged

fix(session): always post session info as persistent message#6
Chanki-Min merged 3 commits intomainfrom
worktree-fix-always-new-session

Conversation

@Chanki-Min
Copy link
Copy Markdown
Owner

Summary

  • 세션 정보가 ephemeral 메시지로 전송되어 conversations.replies에서 조회 불가 → 항상 새 세션이 생성되는 버그 수정
  • 3곳(mention 취소, mention noreply, assistant 취소)의 ephemeral 전송을 persistent(postMessage/say)로 변경
  • 세션 ID 추출 로직을 extractSessionIdFromMessage로 분리하고 blocks/rich_text fallback 추가

Test plan

  • bun test 전체 110 테스트 통과
  • 실제 Slack에서 같은 스레드 두 번째 요청 시 [session] Resuming session 로그 확인
  • 취소 후 같은 스레드에서 재요청 시 세션 resume 확인
  • noreply 모드에서 세션 resume 확인

🤖 Generated with Claude Code

Chanki-Min and others added 3 commits March 9, 2026 15:37
…inuity

Session info was sent as ephemeral messages in 3 paths (mention cancelled,
mention noreply, assistant cancelled), making them invisible to
conversations.replies and preventing session resumption.

- Mention handler cancelled: postEphemeral → postMessage for session info
- Mention handler noreply: postReplyOrEphemeral → postMessage for session info
- Assistant handler cancelled: remove cancelled/success branch, always use say()
- Extract session ID parsing into extractSessionIdFromMessage with block fallback
- Add rich_text block traversal for Slack API format conversion robustness

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
After making session info always persistent, the cancelled flag is no
longer needed for conditional posting. Simplify the error handling to
just skip appending cancelled errors to the stream.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Chanki-Min Chanki-Min merged commit df81bdb into main Mar 10, 2026
1 check passed
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.

1 participant