🐛 이메일 로그인 실패 시 토스트가 표시되지 않는 문제 수정

[manNomi]

요약

• 이메일 로그인 실패(401) 시 사용자에게 아무 피드백도 표시되지 않던 문제를 수정했습니다.
• showIconToast에 중복 억제(dedupe) opt-out 옵션을 추가해, 연속 재시도 시에도 토스트가 매번 노출되도록 했습니다.

원인

로그인 실패 시 토스트를 띄우는 코드 경로가 세 겹으로 막혀 있었습니다.

1. 전역 MutationCache.onError가 status 401이면 "인증 오류는 인터셉터 리다이렉트 토스트에서만 처리"라는 이유로 early return 합니다. (apps/web/src/lib/react-query/queryClient.ts)
2. 그러나 로그인 API(/auth/email/sign-in)는 publicAxiosInstance를 사용합니다. 토스트를 띄우는 응답 인터셉터(redirectToLogin)는 axiosInstance에만 등록되어 있고 publicAxiosInstance에는 인터셉터가 없어, 1번 주석의 전제가 로그인에 대해서만 성립하지 않았습니다.
3. usePostEmailAuth에는 로컬 onError 핸들러가 없었습니다.

그 결과 비밀번호를 틀리면 버튼이 로그인 중... → 로그인으로 되돌아올 뿐, 어떤 안내도 표시되지 않았습니다.

변경 내용

apps/web/src/apis/Auth/postEmailLogin.ts

• onError를 추가해 실패 시 토스트를 노출합니다. 메시지는 서버 응답(error.response?.data?.message)을 우선 사용하고, 없으면 "이메일 또는 비밀번호를 확인해주세요."로 폴백합니다.
• 401 외 status에서 전역 토스트와 중복되지 않도록 meta: SKIP_GLOBAL_ERROR_TOAST_META를 지정했습니다. (기존 postSchoolEmail.ts 패턴 재사용)
• 리다이렉트는 추가하지 않았습니다. 로그인 페이지에 그대로 머무릅니다.

apps/web/src/lib/toast/showIconToast.tsx

• 세 번째 인자로 { dedupe?: boolean }(기본값 true)를 추가했습니다.
• 기존 activeToastKeys는 icon:message 키로 3.5초간 동일 토스트를 억제하는데, 로그인 실패처럼 같은 에러로 즉시 재시도하는 케이스에서는 두 번째 시도부터 토스트가 뜨지 않는 문제가 있었습니다.
• 기존 호출부는 인자를 넘기지 않으므로 동작이 그대로 유지되며, 로그인 실패에서만 opt-out 합니다.

범위 밖 (의도적으로 제외)

• queryClient.ts의 전역 401 skip 로직: axiosInstance 경유 요청에 대해서는 여전히 올바르므로 그대로 두었습니다.
• apps/university-web(리라이트 진행 중), apps/admin, 카카오/애플 로그인 훅.

검증

• pnpm --filter @solid-connect/web run typecheck 통과
• pnpm --filter @solid-connect/web run lint 통과 (biome 473 files)
• 커밋 시 pre-commit 훅의 ci:check(lint:check + typecheck:ci) 두 커밋 모두 통과

🤖 Generated with Claude Code

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
solid-connect-university-web	Ready	Preview	Aug 1, 2026 6:40am
solid-connection-web	Ready	Preview	Aug 1, 2026 6:40am

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
solid-connect-web-admin	Skipped		Aug 1, 2026 6:40am

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: b1e23679-6dcb-41f2-85e6-220e1092435c

📥 Commits

Reviewing files that changed from the base of the PR and between e96a187 and 51a4426.

📒 Files selected for processing (2)

• apps/web/src/apis/Auth/postEmailLogin.ts
• apps/web/src/app/login/LoginContent.tsx

💤 Files with no reviewable changes (1)

• apps/web/src/app/login/LoginContent.tsx

🚧 Files skipped from review as they are similar to previous changes (1)

• apps/web/src/apis/Auth/postEmailLogin.ts

---

Walkthrough

1. 토스트 중복 제어 확장
   showIconToast에 dedupe 옵션을 추가했습니다. false이면 중복 차단과 cooldown 키 삭제 예약을 건너뜁니다.

2. 이메일 로그인 오류 표시 변경
   서버 오류 메시지를 우선 표시합니다. 메시지가 없으면 상태 코드별 또는 기본 문구를 표시합니다. 각 로그인 시도에서 토스트가 표시되도록 dedupe: false를 전달합니다.

3. 로그인 입력 제출 처리 정리
   이메일과 비밀번호 입력 필드의 수동 Enter 키 처리를 제거했습니다. 기존 폼 제출 흐름을 사용합니다.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Suggested reviewers: wibaek

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	이메일 로그인 실패 시 토스트가 표시되지 않는 문제를 수정한다는 핵심 변경을 명확하게 설명합니다.
Description check	✅ Passed	변경 내용, 원인, 범위, 검증 결과를 구체적으로 설명하며 작업 내용과 특이 사항을 대부분 충족합니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/login-error-toast

---

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

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e96a187960

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Use a non-credential fallback for transport failures

When the request fails without a response or response message—for example while offline, on timeout, or because of a CORS failure—this always tells the user to check their credentials. The new mutation metadata also suppresses the global handler, which previously surfaced the Axios error message for these non-401 failures, so users now receive misleading recovery guidance; reserve the credential fallback for authentication responses and retain a connectivity/general fallback for other failures.

Useful? React with 👍 / 👎.

[chatgpt-codex-connector]

Prevent duplicate toasts for Enter submissions

When a user submits the login form with Enter, LoginContent.tsx invokes handleSubmit(onSubmit)() in its input onKeyDown handler and then allows the same key event to perform the form's native submit, producing two mutations before isPending can re-render the form. With deduplication explicitly disabled here, both error callbacks now render identical stacked toasts for a single keypress; remove/coalesce the duplicate submission path or otherwise suppress errors from the same user action while still allowing later retries.

Useful? React with 👍 / 👎.