Skip to content

145 fix 가이드 채팅 버그 수정#146

Merged
ff1451 merged 3 commits intodevelopfrom
145-fix-가이드-채팅-버그-수정
Mar 1, 2026

Hidden character warning

The head ref may contain hidden characters: "145-fix-\uac00\uc774\ub4dc-\ucc44\ud305-\ubc84\uadf8-\uc218\uc815"
Merged

145 fix 가이드 채팅 버그 수정#146
ff1451 merged 3 commits intodevelopfrom
145-fix-가이드-채팅-버그-수정

Conversation

@ff1451
Copy link
Collaborator

@ff1451 ff1451 commented Feb 28, 2026

Summary by CodeRabbit

릴리스 노트

  • 개선 사항

    • 가이드 페이지 자동 진행 동작 개선
  • 기타

    • 에러 추적 세션 기록 설정 업데이트
    • 채팅 메시지 전송 로직 최적화

@ff1451 ff1451 linked an issue Feb 28, 2026 that may be closed by this pull request
@ff1451 ff1451 self-assigned this Feb 28, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 28, 2026

📝 Walkthrough

Walkthrough

채팅 API에서 type 파라미터를 제거하고 SendChatMessageRequest 인터페이스를 추가하여 요청 타입을 단순화했습니다. 이에 따라 useChat 훅의 ChatType 제거, 쿼리 키 및 뮤테이션 로직 간소화가 이루어졌습니다. 추가로 useGuideSliderisLast 속성을 노출하고, GuidePage에서 타이머 기반 자동 진행 로직을 제거한 후 진행 바 콜백으로 통합했습니다. Sentry 리플레이 마스킹 설정도 변경되었습니다.

Possibly related PRs

  • [feat] 알림설정 구현 #108: Chat API의 postChatMute/postChatMessage 시그니처 변경과 useChat 훅의 toggleMute 로직이 type 파라미터 제거에 영향받습니다.
🚥 Pre-merge checks | ✅ 1 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive PR 제목이 한국어로 되어 있어 명확하지만, '버그 수정'이라는 표현이 너무 일반적입니다. 변경사항의 구체적인 내용(채팅 타입 제거, 쿼리 키 단순화 등)이 반영되지 않았습니다. 제목을 더 구체적으로 수정하세요. 예: '145 채팅 API에서 타입 파라미터 제거 및 훅 로직 단순화' 또는 '145 가이드/채팅 기능 리팩토링 및 버그 수정'
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch 145-fix-가이드-채팅-버그-수정

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/config/sentry.ts`:
- Around line 38-41: Sentry.replayIntegration 설정에서 maskAllText: false 및
blockAllMedia: false로 인해 사용자 채팅·미디어 등 민감한 정보가 전송될 위험이 있으니
Sentry.replayIntegration 호출(Sentry.replayIntegration)에서 민감 데이터가 캡처되지 않도록
maskAllText와 blockAllMedia를 true로 변경하거나, 의도된 경우 특정 요소만 마스킹하도록 mask 옵션을 사용해 민감한
셀렉터(예: 채팅 입력, 프로필 이미지 등)를 명시적으로 제외하도록 설정을 업데이트하세요.

ℹ️ Review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c284334 and d248b02.

📒 Files selected for processing (6)
  • src/apis/chat/entity.ts
  • src/apis/chat/index.ts
  • src/config/sentry.ts
  • src/pages/Chat/hooks/useChat.ts
  • src/pages/Guide/hooks/useGuideSlider.ts
  • src/pages/Guide/index.tsx

Comment on lines 38 to 41
Sentry.replayIntegration({
maskAllText: true,
blockAllMedia: true,
maskAllText: false,
blockAllMedia: false,
}),
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

개인정보 노출 위험 확인 필요

maskAllText: falseblockAllMedia: false 설정으로 Sentry 리플레이에서 사용자 텍스트와 미디어가 그대로 녹화됩니다. 채팅 내용 등 민감한 정보가 Sentry에 전송될 수 있어 GDPR/개인정보보호 관점에서 검토가 필요합니다.

의도된 변경이라면 특정 요소만 마스킹하는 mask 옵션 사용을 권장합니다.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@src/config/sentry.ts` around lines 38 - 41, Sentry.replayIntegration 설정에서
maskAllText: false 및 blockAllMedia: false로 인해 사용자 채팅·미디어 등 민감한 정보가 전송될 위험이 있으니
Sentry.replayIntegration 호출(Sentry.replayIntegration)에서 민감 데이터가 캡처되지 않도록
maskAllText와 blockAllMedia를 true로 변경하거나, 의도된 경우 특정 요소만 마스킹하도록 mask 옵션을 사용해 민감한
셀렉터(예: 채팅 입력, 프로필 이미지 등)를 명시적으로 제외하도록 설정을 업데이트하세요.

@ff1451 ff1451 merged commit d7bd59b into develop Mar 1, 2026
2 checks 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.

[fix] 가이드, 채팅 버그 수정

1 participant