Skip to content

[CHA-173] PostHog 핵심 전환 이벤트를 계측한다 - #276

Merged
DongjaJ merged 2 commits into
mainfrom
posthog-ga-vs-clarity
Aug 22, 2026
Merged

[CHA-173] PostHog 핵심 전환 이벤트를 계측한다#276
DongjaJ merged 2 commits into
mainfrom
posthog-ga-vs-clarity

Conversation

@DongjaJ

@DongjaJ DongjaJ commented Aug 22, 2026

Copy link
Copy Markdown
Member

무엇을 변경했나요?

  • 추천·시뮬레이션·비교 시작 및 결과 저장 성공 이벤트 6개를 PostHog와 GA에 함께 전송합니다.
  • 비식별 속성만 수집하고, 분석 SDK 오류가 제품 흐름을 막지 않도록 각 호출을 독립적으로 격리했습니다.
  • 핵심 이벤트와 분석 실패 격리 동작에 대한 대표 테스트를 추가했습니다.
  • 이벤트 발생 조건·속성·콘솔 운영 기준을 docs/analytics.md에 정리했습니다.

왜 변경했나요?

핵심 제품 전환과 저장 행동을 측정하면서도 분석 도구 장애가 사용자 저장·이동·토스트 흐름에 영향을 주지 않도록 하기 위함입니다.

Linear: https://linear.app/chaesozip/issue/CHA-173/posthog-핵심-전환-이벤트-계측

Closes CHA-173

확인 사항

필수 확인

  • node --run build 통과
  • node --run lint 통과
  • node --run test 통과 또는 관련 테스트 확인
  • 관련 이슈를 연결했습니다. 예: Closes #123

기능 검증

  • 변경된 기능을 로컬에서 직접 확인했습니다
  • 사용자 플로우가 의도대로 동작하는지 확인했습니다
  • API 연동 또는 상태 관리 변경 사항을 확인했습니다
  • 에러/로딩/빈 상태 등 예외 케이스를 확인했습니다
  • 변경된 동작에 대한 테스트를 추가하거나 업데이트했습니다

설정 및 문서

  • 사용자에게 노출되는 문서를 업데이트했습니다. 예: README.md, 사용 가이드 등
  • 새로운 의존성이나 설정 변경 사항을 문서화했습니다 (추가 사항 없음)
  • 환경 변수 추가/변경 사항을 명확히 설명했습니다 (추가 사항 없음)
  • 민감한 값, 토큰, 인증 정보가 포함되지 않았습니다

해당되는 경우

  • 보안 영향을 검토했습니다
  • 성능 영향을 검토했습니다
  • 기존 동작을 변경하는 내용은 PR 설명에 명확히 작성했습니다
  • 스크린샷 또는 화면 녹화를 첨부했습니다

@linear-code

linear-code Bot commented Aug 22, 2026

Copy link
Copy Markdown

CHA-173

@coderabbitai

coderabbitai Bot commented Aug 22, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 350ce4a4-51f3-40a9-8ecb-ef372c543354


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@github-actions github-actions Bot added size/L and removed size/M labels Aug 22, 2026
@DongjaJ
DongjaJ marked this pull request as ready for review August 22, 2026 04:27
@github-actions
github-actions Bot requested a review from yereong August 22, 2026 04:27
@codecov

codecov Bot commented Aug 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 92.85714% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...ommend-onboarding/ui/recommend-onboarding-page.tsx 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@DongjaJ
DongjaJ merged commit ce415e1 into main Aug 22, 2026
8 checks passed
@DongjaJ
DongjaJ deleted the posthog-ga-vs-clarity branch August 22, 2026 04:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants