[FEAT] 알람 생성 시 미션 종류 기본값을 없음으로 설정합니다

[DongChyeon]

Related issue 🛠

closed #258

어떤 변경사항이 있었나요?

• 🐞 BugFix Something isn't working
• 🎨 Design Markup & styling
• 📃 Docs Documentation writing and editing (README.md, etc.)
• ✨ Feature Feature
• 🔨 Refactor Code refactoring
• ⚙️ Setting Development environment setup
• ✅ Test Test related (Junit, etc.)

CheckPoint ✅

PR이 다음 요구 사항을 충족하는지 확인하세요.

• PR 컨벤션에 맞게 작성했습니다. (필수)
• merge할 브랜치의 위치를 확인해 주세요(main❌/develop⭕) (필수)
• Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다. (필수)
• BugFix의 경우, 버그의 원인을 파악하였습니다. (선택)

Work Description ✏️

• 알람 생성 시 미션 종류 기본값을 없음 으로 설정

Uncompleted Tasks 😅

N/A

To Reviewers 📢

Summary by CodeRabbit

• Bug Fixes
  • 알람 생성/편집 시 기본 미션이 ‘탭’에서 ‘없음’으로 변경되었습니다. 이제 새 알람을 만들 때 미션이 자동으로 적용되지 않아, 원치 않는 미션 실행을 방지하고 필요한 경우에만 직접 선택할 수 있습니다. 기존에 저장된 알람의 설정에는 영향이 없습니다.

[coderabbitai]

Walkthrough

AlarmAddEditContract.kt에서 AlarmMissionState의 missionType 기본값을 MissionType.TAP에서 MissionType.NONE으로 변경했습니다. 다른 구조적 또는 동작 변화는 없습니다.

Changes

Cohort / File(s)	Change Summary
알람 미션 기본값 변경 feature/home/src/main/java/com/yapp/home/alarm/addedit/AlarmAddEditContract.kt	AlarmMissionState의 missionType 기본값을 MissionType.TAP → MissionType.NONE으로 변경

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	제목은 알람 생성 시 미션 종류 기본값을 ‘없음’으로 설정한다는 핵심 변경 사항을 명확하고 간결하게 요약하고 있어 변경 내용을 스캔하는 동료가 쉽게 이해할 수 있습니다.
Linked Issues Check	✅ Passed	PR에서 AlarmMissionState의 기본 missionType을 MissionType.NONE으로 변경한 구현이 이슈 #258의 요구사항인 ‘알람 생성 시 미션 종류 기본값을 없음으로 설정’과 정확히 일치합니다.
Out of Scope Changes Check	✅ Passed	요약된 변경 사항은 오직 missionType 기본값 수정 한 가지뿐이며 이슈에서 정의되지 않은 다른 코드나 파일 수정이 포함되지 않아 범위를 벗어난 변경이 없습니다.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/#258-alarm-default-mission-none

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f3cf8e5 and 16c46f6.

📒 Files selected for processing (1)

• feature/home/src/main/java/com/yapp/home/alarm/addedit/AlarmAddEditContract.kt (1 hunks)

🧰 Additional context used

🧠 Learnings (1)

📓 Common learnings

Learnt from: DongChyeon
PR: YAPP-Github/Orbit-Android#234
File: feature/home/src/main/java/com/yapp/home/alarm/component/bottomsheet/AlarmMissionBottomSheet.kt:73-76
Timestamp: 2025-07-23T10:29:14.146Z
Learning: AlarmMissionBottomSheet에서 missionType/missionCount 파라미터는 현재 저장된 값을 UI에 표시하기 위해 사용되고, selectedMissionType/selectedMissionCount는 사용자가 변경 중인 내부 작업 상태를 관리하기 위해 사용된다. onDone이나 onSave 콜백을 통해 명시적으로 저장할 때만 변경사항이 반영되는 UX 패턴이다.

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: build

🔇 Additional comments (2)

feature/home/src/main/java/com/yapp/home/alarm/addedit/AlarmAddEditContract.kt (2)

47-50: 기본값을 NONE으로 변경 — PR 목적 정확히 달성

신규 알람 생성 시 미션 기본값이 “없음”으로 표시되도록 정확히 반영되었습니다. 도메인 MissionType.NONE 사용도 일관됩니다. LGTM.

---

100-103: 검증 요청: ‘미리보기’ 진입·저장 흐름에서 MissionType.NONE 처리 확인 필요

• 미리보기(NavigateToMissionPreview) 호출부: MissionType.NONE일 때 호출이 차단(가드)되어 있는지 확인.
• AlarmMissionBottomSheet 초기 표기: NONE이 UI에 '없음'으로 정상 노출되는지 확인(리소스에서 'alarm_add_edit_selected_mission_none'='없음' 확인됨).
• 저장/매핑: MissionType.NONE일 때 missionCount는 UI에서 숨김 및 서버/DB 전송 금지인지, 아니면 서버 스키마가 NONE+count 조합을 허용하는지 확인.

자동 스크립트 실행에서 .kt 타입 인식 오류로 호출부/가드/매퍼 검증이 불가했습니다 — 다음 파일들을 직접 점검하세요: feature/home/src/main/java/com/yapp/home/alarm/addedit/AlarmAddEditContract.kt (~100–103행), feature/home/src/main/java/com/yapp/home/alarm/component/bottomsheet/AlarmMissionBottomSheet.kt (~73–76행), 및 toAlarm/mapper 구현.

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

• Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
• Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.

---

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

[codecov]

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 5.85%. Comparing base (3108e6f) to head (16c46f6).
⚠️ Report is 3 commits behind head on develop.

Files with missing lines	Patch %	Lines
...om/yapp/home/alarm/addedit/AlarmAddEditContract.kt	0.00%	1 Missing ⚠️

❌ Your project status has failed because the head coverage (5.85%) is below the target coverage (60.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files

@@            Coverage Diff            @@
##             develop    #259   +/-   ##
=========================================
  Coverage       5.85%   5.85%           
  Complexity        67      67           
=========================================
  Files             53      53           
  Lines           4847    4847           
  Branches         706     706           
=========================================
  Hits             284     284           
  Misses          4537    4537           
  Partials          26      26           

Files with missing lines	Coverage Δ
...om/yapp/home/alarm/addedit/AlarmAddEditContract.kt	0.00% <0.00%> (ø)

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.