[sunjoolee]

Related Issue

• Closes [Etc] 개발/운영 앱 구분을 위한 ui 추가 #272

작업 상세 내용

• 개발/운영 빌드 구분을 위한 디버그 태그 추가

Summary by CodeRabbit

릴리스 노트

• Chores
  • 애플리케이션 버전이 업데이트되었습니다.

[coderabbitai]

Walkthrough

앱 버전 코드를 4에서 6으로 업데이트하고, DEBUG 빌드에서만 표시되는 개발/운영 앱 구분용 DebugTag UI를 MainActivity에 추가했습니다. AppNavHost를 Box로 감싸서 조건부 렌더링을 처리합니다.

Changes

Cohort / File(s)	Summary
Build Configuration app/build.gradle.kts	versionCode를 4에서 6으로 증가.
Debug UI Enhancement app/src/main/java/com/emotionstorage/emotionstorage/MainActivity.kt	AppNavHost를 Box로 감싸고 DEBUG 빌드에서 빨간 배경의 DebugTag 배너를 상단에 조건부 렌더링. modifier 추가 및 getGoogleIdToken 전달 로직 유지.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Suggested labels

enhancement

Suggested reviewers

• harry7408

---

안녕하세요! 이 PR을 살펴보니 개발/운영 환경을 구분하기 위한 UI 개선이네요. 아주 깔끔하게 처리했습니다! 👍

DEBUG 빌드에서만 표시되는 방식이라 운영 버전에는 영향을 미치지 않을 것 같고, Box로 감싸서 레이아웃을 관리하는 것도 좋은 접근입니다. 다만 리뷰할 때 한 가지 확인해보면 좋을 것 같아요:

• DebugTag의 padding이나 높이가 적절한지 (화면 구성에 방해가 되지 않는지)
• getGoogleIdToken 호출이 여전히 정상적으로 작동하는지

버전 코드 업데이트도 함께 진행되었으니, 릴리스 준비가 거의 다 되어가는 것 같네요! 좋은 작업입니다. 😊

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목이 주요 변경사항인 디버그 태그 추가를 명확하게 반영하고 있으며, 개발/운영 앱 구분 목적도 잘 드러나 있습니다.
Description check	✅ Passed	PR 설명이 필수 템플릿의 모든 섹션(Related Issue, 작업 상세 내용)을 포함하고 있으며, 관련 이슈와 작업 내용이 명확하게 기술되어 있습니다.
Linked Issues check	✅ Passed	코드 변경사항이 #272에서 요구한 개발/운영 앱 구분을 위한 UI 요소 추가라는 목표를 충실하게 구현하고 있습니다.
Out of Scope Changes check	✅ Passed	모든 변경사항(디버그 태그 추가, 버전 코드 증가, 코드 포맷팅)이 #272의 요구사항 범위 내에 있으며 관련성이 있습니다.

_{✏️ 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 feat/add-debug-tag

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.

---

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.}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@app/build.gradle.kts`:
- Line 19: versionCode가 4에서 6으로 건너뛴 것 같으니 먼저 프로젝트의 다른 브랜치, 릴리스 태그, Play Console
또는 배포 기록에서 versionCode 5가 이미 사용되었는지 확인하고, 사용되지 않았다면 app/build.gradle.kts의
versionCode 할당(심볼: versionCode)을 5로 되돌리거나 의도적 건너뛰기라면 그 의도를 명시하는 주석 또는 릴리스 노트를
추가하여 버전 추적 혼동을 방지하세요.

🧹 Nitpick comments (3)

app/src/main/java/com/emotionstorage/emotionstorage/MainActivity.kt (3)

34-47: Box 래핑과 DEBUG 조건부 렌더링, 깔끔하게 잘 구현하셨어요! 👍

BuildConfig.DEBUG 체크로 디버그 빌드에서만 태그를 보여주는 방식 좋습니다. 다만 한 가지 확인해 주시면 좋을 점이 있어요:

Line 31에서 enableEdgeToEdge를 사용하고 계신데, DebugTag가 Alignment.TopCenter로 배치되면 상태바(status bar) 영역 뒤에 가려질 수 있어요. statusBarsPadding() 또는 WindowInsets를 활용해서 상태바 아래에 표시되도록 하면 더 확실하게 보일 거예요~

💡 상태바 인셋 적용 제안

+import androidx.compose.foundation.layout.statusBarsPadding

 Box(modifier = Modifier.fillMaxSize()) {
     if (BuildConfig.DEBUG) {
-        DebugTag()
+        DebugTag(modifier = Modifier.statusBarsPadding())
     }

---

53-75: Color.White를 fully qualified name 대신 import로 사용하면 가독성이 훨씬 좋아져요.

Lines 67-72에서 androidx.compose.ui.graphics.Color.White를 전체 경로로 참조하고 계신데요, 이렇게 하면 코드가 많이 길어지고 읽기 어려워져요. android.graphics.Color와의 이름 충돌 때문에 이렇게 하신 거라면, import alias를 사용하는 방법도 있어요!

♻️ import alias 활용 제안

+import androidx.compose.ui.graphics.Color as ComposeColor

 `@Composable`
 private fun BoxScope.DebugTag() {
     Box(
         modifier =
             Modifier
                 .align(Alignment.TopCenter)
                 .zIndex(100f)
                 .background(MooiTheme.colorScheme.error)
                 .padding(2.dp),
         contentAlignment = Alignment.Center,
     ) {
         Text(
             text = "Debug",
-            color =
-                androidx
-                    .compose
-                    .ui
-                    .graphics
-                    .Color
-                    .White,
+            color = ComposeColor.White,
         )
     }
 }

---

59-59: zIndex(100f) — 매직 넘버 대신 의미 있는 상수를 사용하면 더 좋아요.

현재는 문제없이 동작하겠지만, 나중에 다른 오버레이 요소가 추가될 때 100f라는 값의 의미를 바로 파악하기 어려울 수 있어요. 지금 당장은 큰 문제는 아니지만, 참고해 주세요~

[harry7408]

💯