Skip to content

chore: 미사용 리소스 및 데드 코드 제거#386

Merged
unam98 merged 2 commits intodevelopfrom
feature/remove-unused-resources
Apr 11, 2026
Merged

chore: 미사용 리소스 및 데드 코드 제거#386
unam98 merged 2 commits intodevelopfrom
feature/remove-unused-resources

Conversation

@unam98
Copy link
Copy Markdown
Collaborator

@unam98 unam98 commented Apr 11, 2026

작업 배경

  • Android Studio의 미사용 리소스 감지 결과를 토대로 앱 내 사용하지 않는 drawable/layout/anim/font/menu/xml 리소스와 관련 데드 코드를 일괄 정리.

변경 사항

  • res/drawable, drawable-*dpi, drawable-v24: 사용처 없는 아이콘/쉐이프/스탬프 리소스 삭제
  • res/layout: fragment_my_page, custom_dialog_*, item_info, item_discover_course_shimmer 등 미사용 레이아웃 삭제
  • res/anim, res/menu, res/font, res/xml: 미사용 애니메이션/메뉴/폰트/백업룰 XML 삭제
  • res/values/colors.xml, res/values/strings.xml: 참조 없는 color/string 항목 제거
  • util/extension/ContextExt.kt: 삭제된 custom_dialog_edit_mode 레이아웃에 의존하던 데드 함수(setEditBottomSheet, setEditBottomSheetClickListener, handleEditTextValue)와 import 제거

영향 범위

  • 리소스 / 바인딩 계층만 변경. 런타임 로직 변화 없음.
  • 삭제 대상은 모두 호출/참조처가 없음을 grep으로 확인.
  • APK 리소스 크기 소폭 감소 예상.

Test Plan

  • ./gradlew assembleDebug 빌드 성공 확인
  • 로컬 실행 후 마이페이지/코스 발견/보관함 등 주요 화면 동작 smoke test
  • CI 통과 확인

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Removed unused dialog layout files and associated edit mode functionality
    • Removed animation effects and custom icon assets no longer in use
    • Cleaned up stamp graphics, button shape definitions, and badge resources
    • Removed unused string translations, color definitions, and font configurations
    • Removed backup rules configuration

unam98 added 2 commits April 11, 2026 14:34
- drawable, layout, anim, font, menu, xml 등 사용하지 않는 리소스 파일 삭제
- colors.xml / strings.xml에서 미사용 color, string 항목 정리
- setEditBottomSheet, setEditBottomSheetClickListener, handleEditTextValue 삭제 (호출 코드 없음)
- 삭제된 custom_dialog_edit_mode 리소스 참조 정리
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 11, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: e1a010c5-995f-444d-98e8-01738ca263ae

📥 Commits

Reviewing files that changed from the base of the PR and between ca41de7 and 7a996e8.

⛔ Files ignored due to path filters (5)
  • app/src/main/res/drawable-hdpi/report_google_form.png is excluded by !**/*.png
  • app/src/main/res/drawable-mdpi/report_google_form.png is excluded by !**/*.png
  • app/src/main/res/drawable-xhdpi/report_google_form.png is excluded by !**/*.png
  • app/src/main/res/drawable-xxhdpi/report_google_form.png is excluded by !**/*.png
  • app/src/main/res/drawable-xxxhdpi/report_google_form.png is excluded by !**/*.png
📒 Files selected for processing (55)
  • app/src/main/java/com/runnect/runnect/util/extension/ContextExt.kt
  • app/src/main/res/anim/slide_in_down.xml
  • app/src/main/res/anim/slide_in_up.xml
  • app/src/main/res/drawable-mdpi/my_page_version_info_stroke.xml
  • app/src/main/res/drawable-mdpi/upload_course_selector.xml
  • app/src/main/res/drawable-v24/radius_16dp.xml
  • app/src/main/res/drawable-v24/radius_8_kakao_yellow_button.xml
  • app/src/main/res/drawable-v24/radius_8_w1_button.xml
  • app/src/main/res/drawable/dialog_edit.xml
  • app/src/main/res/drawable/dialog_remove.xml
  • app/src/main/res/drawable/discover_upload_star.xml
  • app/src/main/res/drawable/ic_back.xml
  • app/src/main/res/drawable/ic_delete.xml
  • app/src/main/res/drawable/ic_forward.xml
  • app/src/main/res/drawable/ic_home.xml
  • app/src/main/res/drawable/marker_departure.xml
  • app/src/main/res/drawable/markerinfo.xml
  • app/src/main/res/drawable/menu_detect.xml
  • app/src/main/res/drawable/menu_draw.xml
  • app/src/main/res/drawable/menu_mypage.xml
  • app/src/main/res/drawable/menu_storage.xml
  • app/src/main/res/drawable/my_page_edit.xml
  • app/src/main/res/drawable/my_page_name_edit_dialog_radius.xml
  • app/src/main/res/drawable/my_page_nickname_edit_border.xml
  • app/src/main/res/drawable/mypage_img_stamp_basic.xml
  • app/src/main/res/drawable/mypage_img_stamp_c1.xml
  • app/src/main/res/drawable/mypage_img_stamp_c2.xml
  • app/src/main/res/drawable/mypage_img_stamp_c3.xml
  • app/src/main/res/drawable/mypage_img_stamp_r1.xml
  • app/src/main/res/drawable/mypage_img_stamp_r2.xml
  • app/src/main/res/drawable/mypage_img_stamp_r3.xml
  • app/src/main/res/drawable/mypage_img_stamp_s1.xml
  • app/src/main/res/drawable/mypage_img_stamp_s2.xml
  • app/src/main/res/drawable/mypage_img_stamp_s3.xml
  • app/src/main/res/drawable/mypage_img_stamp_u1.xml
  • app/src/main/res/drawable/mypage_img_stamp_u2.xml
  • app/src/main/res/drawable/mypage_img_stamp_u3.xml
  • app/src/main/res/drawable/radius_5dp.xml
  • app/src/main/res/drawable/searchbutton.xml
  • app/src/main/res/drawable/select.xml
  • app/src/main/res/drawable/shape_address_bar.xml
  • app/src/main/res/drawable/top_radius_white_background.xml
  • app/src/main/res/font/runnect_font.xml
  • app/src/main/res/layout/custom_dialog_edit_mode.xml
  • app/src/main/res/layout/custom_dialog_finish_run.xml
  • app/src/main/res/layout/custom_dialog_report.xml
  • app/src/main/res/layout/custom_dialog_update_require.xml
  • app/src/main/res/layout/dialog_my_page_edit_name.xml
  • app/src/main/res/layout/fragment_my_page.xml
  • app/src/main/res/layout/item_discover_course_shimmer.xml
  • app/src/main/res/layout/item_info.xml
  • app/src/main/res/menu/bottom_navigation_menu.xml
  • app/src/main/res/values/colors.xml
  • app/src/main/res/values/strings.xml
  • app/src/main/res/xml/backup_rules.xml
💤 Files with no reviewable changes (55)
  • app/src/main/res/drawable-v24/radius_8_kakao_yellow_button.xml
  • app/src/main/res/drawable/shape_address_bar.xml
  • app/src/main/res/drawable-mdpi/my_page_version_info_stroke.xml
  • app/src/main/res/drawable/mypage_img_stamp_s1.xml
  • app/src/main/res/drawable/my_page_name_edit_dialog_radius.xml
  • app/src/main/res/drawable/dialog_edit.xml
  • app/src/main/res/drawable/ic_forward.xml
  • app/src/main/res/layout/custom_dialog_finish_run.xml
  • app/src/main/res/drawable/menu_detect.xml
  • app/src/main/res/drawable/my_page_edit.xml
  • app/src/main/res/drawable/menu_draw.xml
  • app/src/main/res/drawable/mypage_img_stamp_r1.xml
  • app/src/main/res/drawable-v24/radius_16dp.xml
  • app/src/main/res/anim/slide_in_down.xml
  • app/src/main/res/drawable/markerinfo.xml
  • app/src/main/res/drawable/menu_storage.xml
  • app/src/main/res/drawable/mypage_img_stamp_s2.xml
  • app/src/main/res/drawable/top_radius_white_background.xml
  • app/src/main/res/drawable/mypage_img_stamp_u2.xml
  • app/src/main/res/drawable/radius_5dp.xml
  • app/src/main/res/drawable/mypage_img_stamp_c3.xml
  • app/src/main/res/drawable/my_page_nickname_edit_border.xml
  • app/src/main/res/drawable/discover_upload_star.xml
  • app/src/main/res/drawable-v24/radius_8_w1_button.xml
  • app/src/main/res/drawable/dialog_remove.xml
  • app/src/main/res/drawable/mypage_img_stamp_u3.xml
  • app/src/main/res/anim/slide_in_up.xml
  • app/src/main/res/drawable/menu_mypage.xml
  • app/src/main/res/drawable/mypage_img_stamp_r2.xml
  • app/src/main/res/drawable/mypage_img_stamp_c1.xml
  • app/src/main/res/xml/backup_rules.xml
  • app/src/main/res/values/colors.xml
  • app/src/main/res/drawable/mypage_img_stamp_basic.xml
  • app/src/main/res/drawable/searchbutton.xml
  • app/src/main/res/layout/custom_dialog_update_require.xml
  • app/src/main/res/drawable/ic_back.xml
  • app/src/main/res/layout/item_info.xml
  • app/src/main/res/drawable/ic_home.xml
  • app/src/main/res/drawable/ic_delete.xml
  • app/src/main/res/drawable/mypage_img_stamp_s3.xml
  • app/src/main/res/drawable/marker_departure.xml
  • app/src/main/res/drawable/mypage_img_stamp_c2.xml
  • app/src/main/res/layout/custom_dialog_edit_mode.xml
  • app/src/main/res/layout/item_discover_course_shimmer.xml
  • app/src/main/res/drawable/mypage_img_stamp_u1.xml
  • app/src/main/res/layout/custom_dialog_report.xml
  • app/src/main/res/layout/dialog_my_page_edit_name.xml
  • app/src/main/java/com/runnect/runnect/util/extension/ContextExt.kt
  • app/src/main/res/menu/bottom_navigation_menu.xml
  • app/src/main/res/layout/fragment_my_page.xml
  • app/src/main/res/font/runnect_font.xml
  • app/src/main/res/drawable/mypage_img_stamp_r3.xml
  • app/src/main/res/drawable/select.xml
  • app/src/main/res/drawable-mdpi/upload_course_selector.xml
  • app/src/main/res/values/strings.xml

📝 Walkthrough

Walkthrough

This pull request removes a substantial collection of Android resources including drawable assets, layout files, animation definitions, extension functions, and string/color resources. The changes eliminate menu navigation components, dialog layouts, UI decorations, stamp graphics, and related string references across the project.

Changes

Cohort / File(s) Summary
Code Extensions
app/src/main/java/com/runnect/runnect/util/extension/ContextExt.kt
Removed three extension functions related to edit-mode bottom sheet dialog creation and event handling (setEditBottomSheet, setEditBottomSheetClickListener, handleEditTextValue).
Animation Resources
app/src/main/res/anim/slide_in_down.xml, app/src/main/res/anim/slide_in_up.xml
Deleted vertical slide-in animation definitions for transitions (150ms duration each).
Drawable Resources — Menu Icons
app/src/main/res/drawable/menu_*.xml
Removed menu navigation icons: menu_draw, menu_storage, menu_detect, menu_mypage (vector drawables with #593EEC fills).
Drawable Resources — Dialog Icons
app/src/main/res/drawable/dialog_edit.xml, app/src/main/res/drawable/dialog_remove.xml
Removed small vector icons for edit and delete actions used in dialog overlays.
Drawable Resources — User Profile Stamps
app/src/main/res/drawable/mypage_img_stamp_*.xml
Deleted 12 badge/stamp vector drawables representing user achievement badges (basic, c1–c3, r1–r3, s1–s3, u1–u3 variants at 80dp viewport).
Drawable Resources — Navigation & UI Icons
app/src/main/res/drawable/ic_back.xml, app/src/main/res/drawable/ic_forward.xml, app/src/main/res/drawable/ic_home.xml, app/src/main/res/drawable/ic_delete.xml, app/src/main/res/drawable/discover_upload_star.xml
Removed navigation and generic action icons (24–36dp vector drawables).
Drawable Resources — Marker & Info Graphics
app/src/main/res/drawable/marker_departure.xml, app/src/main/res/drawable/markerinfo.xml, app/src/main/res/drawable/menu_detect.xml
Deleted map marker and location information vector graphics (60×93dp and 52×30dp).
Drawable Resources — Shape & Background Definitions
app/src/main/res/drawable/radius_*.xml, app/src/main/res/drawable/shape_address_bar.xml, app/src/main/res/drawable/top_radius_white_background.xml, app/src/main/res/drawable/my_page_*.xml, app/src/main/res/drawable-mdpi/*.xml, app/src/main/res/drawable-v24/*.xml
Removed rounded rectangle shapes, button backgrounds, stroke definitions, and dialog radius resources used for styling UI components.
Drawable Resources — Selection & Search
app/src/main/res/drawable/select.xml, app/src/main/res/drawable/searchbutton.xml, app/src/main/res/drawable/upload_course_selector.xml
Removed drawable selectors and utility graphics for interactive UI states.
Layout Resources — Dialog Layouts
app/src/main/res/layout/custom_dialog_edit_mode.xml, app/src/main/res/layout/custom_dialog_finish_run.xml, app/src/main/res/layout/custom_dialog_report.xml, app/src/main/res/layout/custom_dialog_update_require.xml, app/src/main/res/layout/dialog_my_page_edit_name.xml
Removed bottom sheet and modal dialog layout definitions (edit overlay, run completion, report action, deletion confirmation, name editing).
Layout Resources — Fragment & Item Layouts
app/src/main/res/layout/fragment_my_page.xml, app/src/main/res/layout/item_discover_course_shimmer.xml, app/src/main/res/layout/item_info.xml
Deleted main user profile fragment layout (539 lines) and shimmer/placeholder item layouts for course discovery and info displays.
Menu Resources
app/src/main/res/menu/bottom_navigation_menu.xml
Removed bottom navigation menu definition including four menu items and their icon/title mappings for primary app navigation.
Font Resources
app/src/main/res/font/runnect_font.xml
Deleted font-family mapping resource defining Pretendard font weights (400, 500, 600, 700).
String Resources
app/src/main/res/values/strings.xml
Removed 20 string resource entries including discovery labels, dialog prompts, button labels, validation hints, and login option text.
Color Resources
app/src/main/res/values/colors.xml
Removed three unused color definitions: purple_500, teal_700, Y1.
Backup Configuration
app/src/main/res/xml/backup_rules.xml
Deleted Android backup rules XML configuration file.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Poem

🐰 Hop, hop, hooray! The burrow's been swept clean,
Old stamps and dialogs tucked away, unseen.
Resources trimmed, menus reorganized bright,
Our cozy app now bounces with springtime delight! ✨

🚥 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 The title clearly summarizes the main change: removing unused resources and dead code from the Android app, which aligns with the extensive file deletions across layout, drawable, animation, and other resource types.
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
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/remove-unused-resources

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 and usage tips.

@unam98 unam98 self-assigned this Apr 11, 2026
@unam98 unam98 merged commit fdb0d4c into develop Apr 11, 2026
1 of 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.

1 participant