fix: api/user/me PATCH 중복 프로필 이미지 업로드 로직 제거

[evenif99]

@

Summary

• api/user/me/route.ts PATCH 핸들러에서 프로필 이미지 업로드 로직 제거
• 이미 api/user/me/profile-image/route.ts로 분리되어 있어 중복 코드였음
• supabaseAdmin import, 파일 검증 상수, 이미지 시그니처 체커, 업로드 로직 삭제 (78줄 제거)

Test plan

• GET /api/user/me 정상 동작 확인
• PATCH /api/user/me 닉네임/이메일/MBTI 수정 정상 동작 확인
• POST /api/user/me/profile-image 프로필 이미지 업로드 정상 동작 확인

🤖 Generated with Claude Code
@

Summary by CodeRabbit

릴리스 노트

• Bug Fixes
  • 사용자 프로필 업데이트 API에서 프로필 이미지 업로드 기능이 제거되었습니다. 이제 이메일, 닉네임, MBTI 정보만 업데이트할 수 있습니다.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
talky-owl	Ready	Preview, Comment	Jun 22, 2026 9:17am

[coderabbitai]

📝 Walkthrough

Walkthrough

PATCH /api/user/me 라우트에서 프로필 이미지 업로드 관련 코드가 전부 제거되었습니다. Supabase 스토리지 import, 파일 크기/타입 검증 상수, 시그니처 검증 헬퍼, 업로드 블록이 삭제되었고, PATCH 흐름의 변경 감지 조건도 이미지 필드 없이 단순화되었습니다.

Changes

PATCH /api/user/me 이미지 업로드 로직 제거

Layer / File(s)	Summary
Supabase import 및 이미지 상수/헬퍼 제거 src/app/api/user/me/route.ts	supabaseAdmin, PROFILE_IMAGES_BUCKET import와 MAX_FILE_SIZE, ALLOWED_TYPES, hasAllowedImageSignature 선언을 삭제하여 외부 스토리지 의존을 완전히 제거했다.
PATCH 유효성 검사 흐름 단순화 및 업로드 블록 제거 src/app/api/user/me/route.ts	fieldErrors 반환 및 NO_CHANGES 분기에서 profileImage 조건을 제거하고, 바이트 변환 → 시그니처 검증 → 스토리지 업로드 → 공개 URL 생성 블록을 삭제하여 prisma.user.update로 바로 진행하도록 변경했다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐇 토끼가 코드를 살펴보니,
이미지 업로드가 스르륵 사라졌네.
Supabase도, 시그니처도 안녕~
PATCH 흐름이 날아갈 듯 가벼워졌어!
🌸 간결함이 최고의 아름다움이라네.

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

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.
Description check	❓ Inconclusive	PR 설명이 필수 템플릿의 대부분 섹션을 채우지 않았으나, Summary와 Test plan을 제공하여 변경사항의 목적과 검증 방법을 명확히 설명하고 있습니다.	작업 내용, 담당 영역, 관련 Issue, 변경 사항, 작업 범위 확인 등 필수 섹션을 템플릿에 맞춰 작성하면 더 명확한 PR 문서화가 가능합니다.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	제목이 PR의 주요 변경사항을 명확하게 요약하고 있습니다. PATCH 핸들러에서 중복된 프로필 이미지 업로드 로직을 제거했다는 내용을 정확하게 전달합니다.
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.

_{✏️ 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 fix/remove-duplicate-profile-upload

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

src/app/api/user/me/route.ts (1)

130-134: ⚠️ Potential issue | 🟠 Major | ⚡ Quick win

PATCH 입력 계약이 깨져 profileImage가 누락/무시됩니다.

Line 130의 NO_CHANGES 판단이 data(email/nickname/mbti)만 기준이라, 현재 클라이언트가 보내는 profileImage 입력과 계약이 어긋납니다.
src/domains/user/api.ts와 src/app/(page)/mypage/edit/page.tsx는 아직 PATCH /api/user/me에 profileImage를 보내므로, 이미지 단독 수정은 400(NO_CHANGES), 다른 필드와 동시 수정은 200이지만 이미지 반영이 누락되는 침묵 실패가 발생합니다.

PATCH에서 profileImage를 명시적으로 거절(전용 엔드포인트 안내)하거나, 클라이언트를 POST /api/user/me/profile-image로 동시에 전환해 API 계약을 맞춰야 합니다.

제안 패치 (침묵 실패 방지 가드)

 export async function PATCH(request: NextRequest) {
@@
     const data: Record<string, string | null> = {}
     const fieldErrors: { field: string; code: string; message: string }[] = []
+    const profileImageValue = formData.get('profileImage')
+
+    if (profileImageValue !== null) {
+      return NextResponse.json<ApiResponse>(
+        {
+          success: false,
+          error: {
+            code: 'PROFILE_IMAGE_ENDPOINT_CHANGED',
+            message: '프로필 이미지는 /api/user/me/profile-image 엔드포인트를 사용해주세요.',
+          },
+        },
+        { status: 400 },
+      )
+    }
@@
     if (Object.keys(data).length === 0) {

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@src/app/api/user/me/route.ts` around lines 130 - 134, The NO_CHANGES
validation at line 130 only checks if data (email/nickname/mbti) is empty but
doesn't account for the profileImage field that clients are still sending,
causing silent failures when profileImage changes are ignored or returning
NO_CHANGES when only profileImage is provided. You need to either explicitly
reject the profileImage field in the PATCH handler with guidance to use a
dedicated endpoint, or include profileImage in the data validation logic to
ensure proper handling aligned with the client's current expectations.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Outside diff comments:
In `@src/app/api/user/me/route.ts`:
- Around line 130-134: The NO_CHANGES validation at line 130 only checks if data
(email/nickname/mbti) is empty but doesn't account for the profileImage field
that clients are still sending, causing silent failures when profileImage
changes are ignored or returning NO_CHANGES when only profileImage is provided.
You need to either explicitly reject the profileImage field in the PATCH handler
with guidance to use a dedicated endpoint, or include profileImage in the data
validation logic to ensure proper handling aligned with the client's current
expectations.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: f15b6b66-a038-47ab-984c-ab997d97207f

📥 Commits

Reviewing files that changed from the base of the PR and between 19916c2 and 9670696.

📒 Files selected for processing (1)

• src/app/api/user/me/route.ts