ga event를 심는다.

[YOOJS1205]

Motivation

• ga event를 심는다.

Reference	Links
Issue

---

Modification

• ㄱㄱㄱ

---

Result

X

---

[github-actions]

🤖 AI Code Review (OpenAI)

• index.html/gtag 설정 부분 + debug_mode: true 설정 +
  실제 운영 환경에서 debug_mode가 활성화되면 불필요한 로그가 많이 발생할 수 있어 성능 저하 및 개인정보 노출 위험이 있음.
  → 운영 배포 시에는 debug_mode를 환경변수로 분기 처리하여 false로 설정하는 것이 좋음.

• src/api/useKakaoShareUrl.ts/useEffect 내부 + ReactGA.event 호출 위치 +
  공유 이벤트를 try 블록 안에서 바로 호출하는데, 만약 window.Kakao.Share.sendDefault 호출이 실패하면 이벤트가 기록되지 않을 수 있음.
  → 공유 성공 여부에 따라 이벤트를 기록하도록 콜백이나 Promise 처리 후 호출하는 것이 정확한 이벤트 추적에 유리함.

• src/components/common/LinkShareBottomSheet/hooks.ts/클립보드 복사 성공 후 이벤트 호출 +
  클립보드 복사 성공 시마다 'poll_shared' 이벤트를 보내는데, useKakaoShareUrl 훅에서도 동일 이벤트를 보내고 있음.
  → 동일 이벤트가 중복 기록될 가능성이 있으므로 이벤트 명이나 호출 위치를 명확히 구분하거나 중복 호출을 방지하는 로직 필요.

• src/pages/Home/Home.tsx/handleCloseCoachMark 함수 내부 +
  사용자 온보딩 종료 시점에 'user_signup' 이벤트를 보내는데, 함수명이 닫기 처리임에도 이벤트명이 'signup'으로 되어 있어 의미가 혼동될 수 있음.
  → 이벤트명과 함수 동작이 일치하도록 명확한 네이밍 또는 주석 추가가 필요.

• src/components/poll/regist/PollRegistButton/PollRegistButton.tsx/이벤트 전송 시점 +
  투표 생성 후 navigate 호출 전에 이벤트를 보내고 있는데, navigate로 인해 컴포넌트가 언마운트되면 이벤트 전송이 중단될 수 있음.
  → 이벤트 전송 완료 후 navigate 호출하거나, 비동기 이벤트 전송을 보장하는 로직 적용 권장.

• nit

  1. src/api/useKakaoShareUrl.ts, src/components/common/LinkShareBottomSheet/hooks.ts, src/pages/Home/Home.tsx 등 여러 파일에서 ReactGA import가 중복되는데, 프로젝트 규모에 따라 공통 유틸로 분리 고려 가능.
  2. index.html 내 스크립트 태그 닫는 위치가 diff상 명확하지 않으나, 스크립트 태그 내 코드 정렬 및 주석 추가로 가독성 개선 가능.
  3. 이벤트명 문자열은 상수로 관리하면 오타 방지 및 유지보수에 도움됨.

자동 생성된 리뷰입니다. 참고용으로만 사용해주세요.