[release] FE 팝업 AB테스트 종료

[seongwon030]

#️⃣연관된 이슈

ex) #이슈번호, #이슈번호

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지/동영상 첨부 가능)

중점적으로 리뷰받고 싶은 부분(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

논의하고 싶은 부분(선택)

논의하고 싶은 부분이 있다면 작성해주세요.

🫡 참고사항

Summary by CodeRabbit

• Refactor

  • A/B 테스트 관련 로직 및 분기 제거로 팝업 표시 조건을 단순화했습니다. 이제 기기 상태, 이미지 로드 여부 및 기존 숨김 설정만으로 동작합니다.
  • 팝업 관련 이벤트 추적 페이로드에서 A/B 그룹 정보가 제거되어 전송 데이터가 간소화되었습니다.

• Tests / Stories

  • 스토리와 테스트에서 A/B 테스트 관련 설정과 검증을 삭제해 관련 시나리오를 정리했습니다.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
moadong	Ready	Preview, Comment	Feb 11, 2026 11:47am

[coderabbitai]

Caution

Review failed

The pull request is closed.

Warning

.coderabbit.yaml has a parsing error

The CodeRabbit configuration file in this repository has a parsing error and default settings were used instead. Please fix the error(s) in the configuration file. You can initialize chat with CodeRabbit to get help with the configuration file.

💥 Parsing errors (1)

Validation error: Invalid regex pattern for base branch. Received: "**" at "reviews.auto_review.base_branches[0]"

⚙️ Configuration instructions

• Please see the configuration documentation for more information.
• You can also validate your configuration using the online YAML validator.
• If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Walkthrough

AB 테스트 기능이 Popup 및 관련 파일에서 제거되었습니다. AB 관련 상수·타입·함수와 AB 그룹 기반 로직 및 모든 분석 페이로드의 abTestGroup 필드가 삭제되었습니다.

Changes

Cohort / File(s)	Summary
Popup AB 테스트 코어 로직 제거 frontend/src/pages/MainPage/components/Popup/Popup.tsx	AB_TEST_KEY, AB_TEST_SPLIT_RATIO, PopupABTestGroup 타입, getABTestGroup 함수 제거. 팝업 표시 로직에서 AB 그룹 검사 제거; imageLoaded, isMobile, 숨김 상태 기반으로 단순화. 추적 이벤트에서 abTestGroup 제거.
Banner AB 테스트 참조 제거 frontend/src/pages/MainPage/components/Banner/Banner.tsx	getABTestGroup 임포트 삭제. APP_STORE_LINK 클릭 핸들러에서 AB 그룹 조회 및 abTestGroup을 추적 페이로드에서 제거.
테스트·스토리 정리 frontend/src/pages/MainPage/components/Popup/Popup.test.tsx, frontend/src/pages/MainPage/components/Popup/Popup.stories.tsx	AB_TEST_KEY 및 getABTestGroup 관련 테스트/스토리 코드와 localStorage 설정 제거. 남은 테스트는 DAYS_TO_HIDE, isPopupHidden, POPUP_STORAGE_KEY 중심으로 단순화.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• [feature] 메인화면에 앱 출시 팝업을 추가한다 #1010: 이전 PR에서 추가된 AB 테스트 구현 및 관련 내보내기(AB_TEST_KEY, getABTestGroup)를 본 PR이 제거하므로 직접적으로 연관됨.
• [feature] 앱 다운로드 배너 트래킹에 A/B 테스트 그룹 정보 추가 #1056: Banner에서 getABTestGroup 임포트와 abTestGroup 추적을 추가한 변경과 본 PR의 제거 작업이 코드 충돌 가능성이 있어 관련됨.
• [feature] 앱 출시 배너를 추가한다 #938: Banner의 APP_STORE_LINK 클릭 처리 및 분석 페이로드 변경과 본 PR이 동일 경로를 수정하므로 관련됨.

Suggested reviewers

• oesnuj
• lepitaaar

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목이 변경 사항의 핵심을 명확하게 반영하고 있습니다. PR은 팝업 컴포넌트에서 AB테스트 기능을 완전히 제거한 내용을 담고 있으며, 제목 '[release] FE 팝업 AB테스트 종료'는 이를 정확하게 설명합니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch develop-fe

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[oesnuj]

좋은 AB 테스트였습니다
다음으로 새로운 테스트를 많이 생각해봐요

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@frontend/src/pages/MainPage/components/Popup/Popup.tsx`:
- Around line 37-50: The effect in Popup.tsx calls
trackEvent(USER_EVENT.MAIN_POPUP_NOT_SHOWN) when imageLoaded is false, causing a
premature NOT_SHOWN event; update the useEffect that references isPopupHidden(),
isMobile, imageLoaded and trackEvent so it early-returns if imageLoaded is false
(or only emits NOT_SHOWN when imageLoaded is true), ensuring only one event is
fired per user (either MAIN_POPUP_VIEWED or MAIN_POPUP_NOT_SHOWN) and use the
existing identifiers isPopupHidden, setIsOpen, trackEvent,
USER_EVENT.MAIN_POPUP_VIEWED, and USER_EVENT.MAIN_POPUP_NOT_SHOWN to locate and
adjust the logic.

🧹 Nitpick comments (1)

frontend/src/pages/MainPage/components/Popup/Popup.test.tsx (1)

12-63: sessionStorage 분기에 대한 테스트가 누락되어 있습니다.

isPopupHidden은 sessionStorage.getItem(POPUP_SESSION_KEY)를 먼저 확인하지만, 해당 분기를 검증하는 테스트가 없습니다. POPUP_SESSION_KEY를 import하고 세션 기반 숨김 동작도 테스트하면 커버리지가 더 완전해집니다.