refactor(web): 타입 별칭과 강제 줄바꿈 정리

[pbk95120]

📝 작업 내용 요약

• 단순 재명명 타입 별칭 HomeMissionCategory를 제거하고 원본 MissionCategory를 직접 사용했습니다.
• 웹 화면의 <br /> 16곳을 제거하고 화면 폭에 맞춰 문구가 줄바꿈되도록 바꿨습니다.
• 혜택 히어로는 좁은 화면에서도 문구 2줄과 그림 비율을 유지하도록 너비를 조정했습니다.
• docs/code-conventions.md에 불필요한 타입 별칭과 강제 줄바꿈을 피하는 규칙을 추가했습니다.
• 목표 상세 재조회 버튼에 키보드 포커스 링과 44px 터치 높이를 적용하고 E2E로 확인했습니다.
• mutation 오류 규칙을 mutate의 onError와 mutateAsync의 try/catch를 모두 허용하도록 바로잡았습니다.
• Knip이 지적한 미사용 export 3개를 정리하고, 히어로 E2E가 웹폰트 로딩 후 레이아웃을 측정하도록 수정했습니다.

✅ 체크리스트

• develop 브랜치의 최신 코드를 pull 받았나요? (origin/develop 기준 새 브랜치)
• 빌드가 통과했나요? (Turbo 전체 20/20)
• vitest 테스트가 통과했나요? (웹 348개)
• Playwright e2e가 통과했나요? (24개, 320~412px 히어로 포함)
• Knip이 통과했나요? (pnpm knip, GitHub Actions 통과)

🤖 AI 리뷰

Claude 리뷰 (Codex 구현)

• Claude Opus 5 리뷰를 한 번 시도했지만 Not logged in · Please run /login으로 실행하지 못했습니다.
• 이전에 같은 로그인 오류에서 리뷰 생략 후 PR 진행을 승인받아 진행했습니다. 독립 리뷰 통과 판정은 없습니다.

💬 기타 코멘트

• 타입 별칭은 전면 금지하지 않았습니다. z.infer, 유니온, 공개 API처럼 이름이 필요한 타입은 유지합니다.

변경 사항

• HomeMissionCategory 별칭을 제거하고 MissionCategory를 직접 사용합니다.
• 혜택 안내와 빈 상태 문구, 온보딩·미션·문의 화면에서 강제 <br /> 줄바꿈을 제거합니다. text-balance, break-keep 및 너비 제약으로 화면 폭에 따라 문구를 배치합니다.
• 목표 재조회 실패 안내 버튼에 최소 높이와 키보드 포커스 스타일을 추가하고, 포커스 동작을 검증하는 E2E 테스트를 추가합니다.
• docs/code-conventions.md에 불필요한 타입 별칭과 시각적 강제 줄바꿈을 피하는 규칙을 추가합니다. mutation 오류 처리 규칙도 보완합니다.
• 혜택 문구 테스트의 텍스트 단언을 수정합니다.
• 작성자는 Turbo 빌드 20/20, 웹 Vitest 348개, Playwright E2E 23개가 통과했다고 보고했습니다.

[vercel]

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

Project	Deployment	Actions	Updated
28th-web-team-3-fe-web-dev	Ready	Preview	Sep 23, 2026 5:51am UTC

1 Skipped Deployment

Project	Deployment	Actions	Updated
28th-web-team-3-fe-admin	Ignored		Sep 23, 2026 5:51am UTC

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository: YAPP-Github/28th-Web-Team-3-FE/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: ff1a393b-6766-4c77-9143-4b5922a49303

📥 Commits

Reviewing files that changed from the base of the PR and between 5da0373 and df19771.

📒 Files selected for processing (3)

• apps/web/app/goal/_components/goal-detail.tsx
• apps/web/e2e/goal.spec.ts
• docs/code-conventions.md

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

Walkthrough

Changes

화면 텍스트 및 타입 정리

Layer / File(s)	Summary
미션 카테고리 타입 정리 apps/web/app/(tabs)/_components/weekly-mission-section.tsx, apps/web/app/(tabs)/constants/home.ts, docs/code-conventions.md	HomeMissionCategory 별칭을 제거하고 MissionCategory를 직접 사용하도록 변경했습니다. 타입 별칭 규칙과 mutation 오류 처리 규칙을 문서에 추가했습니다.
화면 텍스트 줄바꿈 처리 apps/web/app/(tabs)/..., apps/web/app/mission/..., apps/web/app/onboarding/..., docs/code-conventions.md	여러 화면의 안내 문구와 제목에서 수동 줄바꿈을 제거하고 text-balance, break-keep, 너비 제약을 적용했습니다. 관련 줄바꿈 규칙을 문서에 추가했습니다. 혜택 문구 테스트는 정확한 전체 문자열을 검증하도록 변경했습니다.

목표 재조회 버튼 포커스

Layer / File(s)	Summary
재조회 버튼 포커스 표시 apps/web/app/goal/_components/goal-detail.tsx, apps/web/e2e/goal.spec.ts	재조회 버튼에 최소 높이와 포커스 스타일을 추가했습니다. E2E 테스트에서 재조회 실패 후 키보드 포커스와 포커스 스타일을 확인합니다.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~15 minutes

Change: Refactor

Merge Risk: ⚪ Minimal · up to df197

The text, type, and focus changes have no established blocking issue in the supplied evidence and are mergeable after normal checks.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 37.50% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 16 functions across 18 files. (1 skipped:…	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목은 HomeMissionCategory 타입 별칭 제거와 강제 줄바꿈 정리라는 PR의 주요 변경 사항을 정확하고 간결하게 설명합니다.

Full details: Docstring Coverage

Explanation

Docstring coverage is 37.50% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 16 functions across 18 files. (1 skipped: 1 unsupported.)

• Fix all pre-merge checks with AI

✨ Finishing Touches 💡 2

📝 Generate docstrings 💡

• Commit to this branch
• Create a new PR

🛠️ Fix failing CI checks 💡

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

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