Skip to content

[Agent] 승인 안내 메시지에서 카드 중복과 원시 enum 을 뺀다 [ #457 ] - #458

Merged
dldnsgkr merged 1 commit into
developfrom
unhak/approval-chat-message
Oct 15, 2026
Merged

dldnsgkr merged 1 commit into
developfrom
unhak/approval-chat-message

Conversation

@dldnsgkr

Copy link
Copy Markdown
Collaborator

무엇을

승인 카드 위에 붙는 서버 채팅 메시지 세 종류를 줄인다.

전 후
계획 승인 - [48] CHANGE - 변경 사항 승인
결과 승인 두 문장 + - [47] RESULT: [결과 반영] … 두 문장
저장소 연결 카드 설명 반복 + 기본값 줄 + - [49] REPOSITORY_BINDING: … 작업물이 준비됐습니다. 이 프로젝트에는 GitHub 저장소가 없어 저장소 연결 승인을 요청했습니다 — <이름> + 지금 연결하지 않으면 프리뷰가 만료될 때 작업물이 사라집니다.

ApprovalTypeLabel 은 FE approvalCopy.ts 의 카드 제목과 같은 이름이다. switch 에 default 가 없어 유형이 늘면 컴파일이 깨진다.

왜

#401 운영 화면 확인(프로젝트 10, 승인 #49)에서 카드와 메시지가 같은 문장을 두 번 보여주고 원시 enum 이 채팅에 드러남(#457). 카드는 결정하면 사라지므로 메시지를 0 으로 만들지 않고, 기록 한 줄과 카드에 없는 경고만 남긴다(사용자 결정, FE 세션 합의). FE 는 이 줄을 파싱하지 않음(approvalId 매칭).

범위 밖

  • 태스크 summary(markWaitingApproval)는 카드 옆에 놓이지 않아 그대로
  • 이미 저장된 과거 메시지는 그대로(마이그레이션 없음)

검증

  • 관련 테스트 갱신 + ApprovalTypeLabelTest(모든 유형이 원시 값이 아닌 이름을 갖는지)
  • 되돌리기: 메인 코드만 되돌리면 갱신한 테스트 3건 실패
  • 전체 ./gradlew test 통과

Closes #457

🤖 Generated with Claude Code

승인 카드 바로 위 채팅 메시지가 카드 설명을 글자 그대로 반복하고 "[49] REPOSITORY_BINDING: ..."
처럼 ID·원시 enum 을 찍고 있었다(#401 운영 확인 중 발견).

- 계획 승인: "- [48] CHANGE" → "- 변경 사항 승인" (ApprovalTypeLabel, FE 카드 제목과 같은 이름)
- 결과 승인: "- [ID] RESULT: ..." 줄 제거
- 저장소 연결: 기록 한 줄(제안 이름 포함) + 프리뷰 만료 경고만 남긴다

카드는 결정하면 사라지므로 채팅에는 사람이 읽는 기록은 남긴다. 태스크 summary 는 그대로.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@dldnsgkr
dldnsgkr merged commit bf29e87 into develop Oct 15, 2026
1 check passed
@dldnsgkr
dldnsgkr deleted the unhak/approval-chat-message branch October 15, 2026 05:11
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