Skip to content

fix(agent): 복사 스레드의 중복 응답 판정을 분리#33

Merged
lbox-limit merged 1 commit into
mainfrom
codex/fix-copied-thread-response-deduplication
Jul 15, 2026
Merged

fix(agent): 복사 스레드의 중복 응답 판정을 분리#33
lbox-limit merged 1 commit into
mainfrom
codex/fix-copied-thread-response-deduplication

Conversation

@lbox-limit

Copy link
Copy Markdown
Collaborator

변경 사항

  • 턴 내 중복 응답 기록을 프로세스 메모리 대신 공유 session_state DB에 저장합니다.
  • 중복 판정 키에 현재 요청 ID와 Slack 목적지·스레드 정보를 포함합니다.
  • turn-sends 구현과 테스트를 기존 session-state 모듈에 통합합니다.
  • 복사한 스레드와 후속 요청에서 같은 문구가 정상 전달되는 회귀 테스트를 추가합니다.

배경

send_message MCP 도구와 최종 응답을 처리하는 poll loop가 서로 다른 프로세스에서 실행되는데, 기존 중복 기록은 프로세스 로컬 Set에 저장되었습니다. 따라서 실제 전송 기록을 poll loop가 관찰하지 못했고, 요청이나 스레드 경계도 명시적으로 구분되지 않았습니다.

영향

같은 요청·같은 목적지·같은 스레드에서 이미 전송한 문구만 중복으로 제거됩니다. 복사한 스레드의 질문이나 이후 요청에는 동일한 답변을 다시 보낼 수 있습니다.

검증

  • bun run typecheck
  • bun test — 147 passed, 0 failed
  • Prettier 검사
  • git diff --check

@lbox-limit
lbox-limit marked this pull request as ready for review July 15, 2026 01:15
@lbox-limit
lbox-limit merged commit b1835a8 into main Jul 15, 2026
2 checks passed
@lbox-limit
lbox-limit deleted the codex/fix-copied-thread-response-deduplication branch July 15, 2026 01:15
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.

2 participants