🐛 지원 대학 관리 검색 시 국내 대학/학기 필터 미적용 버그 수정

[whqtker]

요약

• 어드민 지원 대학 관리 탭에서 국내 대학·학기를 선택해 검색하면, 응답은 정상으로 오지만 화면에는 결과가 표시되지 않는 문제 수정
• adminApi.searchUnivApplyInfos가 검색어(value)만 서버로 전송하고 homeUniversityId/termId는 보내지 않아, 서버가 항상 현재 학기 기준으로만 응답하고 있었음
• 응답을 다시 선택한 학기 라벨과 비교해 클라이언트에서 필터링하던 로직 때문에, 현재 학기가 아닌 학기를 선택하면 응답에 데이터가 있어도 전부 걸러져 화면에는 아무것도 안 보였음
• homeUniversityId/termId를 실제로 서버에 전달하도록 수정하고, 서버가 이미 필터링하므로 불필요해진 클라이언트 측 재필터링 로직 제거

테스트 플랜

• pnpm --filter admin typecheck, lint:check 통과 (커밋 시 CI parity 훅으로 확인)
• 로컬 MySQL/Redis + 백엔드(local 프로필)를 직접 띄우고, 비현재 학기 데이터를 시드한 뒤 API 레벨에서 수정 전/후 동작 차이를 직접 재현·검증함
  • 수정 전 재현(파라미터 미전송): 응답은 정상이지만 항상 현재 학기 데이터만 반환 → 프론트 필터가 이를 전부 걸러내 빈 화면
  • 수정 후(파라미터 전송): 선택한 학기의 데이터가 정확히 반환됨
• admin 화면에서 직접 클릭 테스트는 못 함 — 로컬 admin dev 서버(vinext)가 이 변경과 무관한 기존 이슈로 dev 모드에서 500 에러를 반환해 실제 화면 확인이 막혀 있었음 (프로덕션 빌드는 정상 완료됨)

Co-Authored-By: Claude Sonnet 5 noreply@anthropic.com
https://claude.ai/code/session_01EwpgqdNTrn8dGDr7g7WnYg

[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 2:36pm
solid-connect-web-admin	Ready	Preview	Aug 1, 2026 2:36pm

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
solid-connection-web	Skipped		Aug 1, 2026 2:36pm

[coderabbitai]

Walkthrough

1. 검색 API 조건 확장
   검색 API가 검색어, 국내 대학 ID, 학기 ID를 객체로 받아 쿼리 파라미터로 전달합니다.

2. 관리 탭 검색 연동 변경
   관리 탭이 세 검색 조건을 API에 전달합니다. API 응답의 univApplyInfoPreviews를 직접 사용합니다.

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

Suggested reviewers: wibaek, mannomi

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

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.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	제목은 국내 대학 및 학기 필터 미적용 버그 수정이라는 PR의 주요 변경 사항을 정확히 설명합니다.
Description check	✅ Passed	설명은 문제 원인, 수정 내용, 테스트 결과와 미수행한 화면 테스트를 구체적으로 포함합니다.
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 fix/admin-univ-apply-search-filter

---

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

[coderabbitai]

🧹 Nitpick comments (1)

apps/admin/src/lib/api/admin.ts (1)

373-380: 🗄️ Data Integrity & Integration | 🔵 Trivial | ⚡ Quick win

1. 세 검색 조건의 전달을 회귀 테스트로 고정해 주세요.

searchUnivApplyInfos의 계약이 value 단일 값에서 세 필드로 변경되었습니다. apps/admin/src/lib/api/admin.test.ts에 value, homeUniversityId, termId가 /univ-apply-infos/search/text의 params에 모두 전달되는지 확인하는 테스트를 추가해 주세요. 향후 필터 누락 회귀를 빠르게 발견할 수 있습니다.

🤖 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 `@apps/admin/src/lib/api/admin.ts` around lines 373 - 380, In the admin API
tests, add a regression test for searchUnivApplyInfos that invokes it with
value, homeUniversityId, and termId, then verifies the GET request to
"/univ-apply-infos/search/text" includes all three fields in params.

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

Nitpick comments:
In `@apps/admin/src/lib/api/admin.ts`:
- Around line 373-380: In the admin API tests, add a regression test for
searchUnivApplyInfos that invokes it with value, homeUniversityId, and termId,
then verifies the GET request to "/univ-apply-infos/search/text" includes all
three fields in params.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 817888a8-76a9-4576-97c1-90d6cc45156d

📥 Commits

Reviewing files that changed from the base of the PR and between 3c663ae and 9308510.

📒 Files selected for processing (2)

• apps/admin/src/components/features/univ-apply-infos/tabs/UnivApplyInfoManageTab.tsx
• apps/admin/src/lib/api/admin.ts