Skip to content

feat(mission): 목업 생성 로딩 시간 적용 - #254

Merged
jongse7 merged 1 commit into
developfrom
feat(mission)/mock-generation-loading
Aug 22, 2026
Merged

feat(mission): 목업 생성 로딩 시간 적용#254
jongse7 merged 1 commit into
developfrom
feat(mission)/mock-generation-loading

Conversation

@jongse7

@jongse7 jongse7 commented Aug 22, 2026

Copy link
Copy Markdown
Member

📝 작업 내용 요약

  • 데모용 목업 미션 생성도 AI 생성처럼 8~12초의 로딩 시간을 유지합니다.
  • 미션 생성 job이 완료돼도 최소 로딩 시간과 초안 준비 상태가 모두 충족된 뒤 결과 화면으로 이동합니다.
  • 즉시 완료 응답에서도 8초 대기 후 이동하는 동작을 테스트했습니다.

✅ 체크리스트

  • develop 브랜치의 최신 코드를 pull 받았나요?
  • 빌드가 통과했나요?
  • vitest 테스트가 통과했나요?
  • Playwright e2e가 통과했나요?

🤖 AI 리뷰

Claude 리뷰 (Codex 구현)

🟢 승인 가능

  • useState(getLoadingDuration)으로 마운트마다 대기 시간을 한 번만 정하고, useEffect cleanup으로 타이머를 정리합니다.
  • SUCCEEDED, draftsAvailable, 최소 로딩 시간 조건이 모두 충족될 때만 router.replace합니다.
  • 참고: 현재 테스트는 최소값인 8초 경계만 검증합니다.

💬 기타 코멘트

변경 사항

  • 목업 미션 생성에도 MissionLoading의 8~12초 최소 로딩 시간을 적용했습니다.
  • 로딩 시간이 지나고 작업이 성공하며 초안이 준비된 경우에만 결과 화면으로 이동합니다.
  • 성공 상태 테스트에서 즉시 완료 응답 후 8초가 지나야 이동하는 동작을 검증했습니다.
  • 테스트 전후에 Math.random 스파이와 mock을 정리하도록 보완했습니다.

@jongse7 jongse7 self-assigned this Aug 22, 2026
@jongse7
jongse7 requested a review from pbk95120 August 22, 2026 02:55
@vercel

vercel Bot commented Aug 22, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
28th-web-team-3-fe-web-dev Ready Ready Preview Aug 22, 2026 2:55am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
28th-web-team-3-fe-admin Ignored Ignored Aug 22, 2026 2:55am

@coderabbitai

coderabbitai Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: eca86ca7-ff6e-4824-aa0c-c230d81a4415

📥 Commits

Reviewing files that changed from the base of the PR and between a052858 and b4b9dac.

📒 Files selected for processing (2)
  • apps/web/app/mission/_components/mission-loading.tsx
  • apps/web/app/mission/new/loading/page.test.tsx

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


Walkthrough

미션 로딩에 8~12초의 최소 로딩 시간을 추가했습니다. 로딩 시간이 지나고 작업이 성공하며 초안이 있을 때 결과 페이지로 이동합니다. 테스트는 타이머와 무작위 값을 제어해 지연 이동을 검증합니다.

Changes

미션 로딩 지연

Layer / File(s) Summary
최소 로딩 시간 및 이동 조건
apps/web/app/mission/_components/mission-loading.tsx
8~12초의 최소 로딩 시간을 한 번 정하고 경과 여부를 추적합니다. 작업 성공과 초안 존재 조건에 시간 경과 조건을 추가합니다.
지연 이동 테스트 검증
apps/web/app/mission/new/loading/page.test.tsx
Math.random과 타이머를 제어합니다. 초기 조회 후 즉시 이동하지 않음을 확인하고, 8초 후 결과 페이지 이동을 검증합니다.

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

Merge Risk: ⚪ Minimal · up to b4b9d

This localized change adds the intended minimum loading delay before navigation, and no actionable merge-blocking risk remains after normal checks and review.

Suggested reviewers: pbk95120

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 2 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 목업 미션 생성에 8~12초의 로딩 시간을 적용하는 주요 변경 사항을 명확하게 설명합니다.
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.
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat(mission)/mock-generation-loading

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.

@jongse7
jongse7 merged commit 3ca212f into develop Aug 22, 2026
6 checks passed
@jongse7
jongse7 deleted the feat(mission)/mock-generation-loading branch August 22, 2026 03:00
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