Repository navigation
Conversation
관리자 페이지의 관리자·판매자 계정 생성에서 username 규칙에 영문 대문자를 더함. - USERNAME_PATTERN: [a-z0-9._-] → [A-Za-z0-9._-] - 입력한 대소문자 그대로 저장·표시, 로그인은 대소문자 무관 - 대소문자만 다른 username은 USERNAME_TAKEN — account_credential.username 정렬이 utf8mb4_unicode_ci라 사전 조회·unique 인덱스 모두 같은 값으로 봄. 마이그레이션 없음 - SDL 설명 2곳(AdminCreateAdminInput·AdminCreateSellerInput) 갱신 테스트 - input spec 2개: 대문자 거절 케이스 → 허용 케이스 - 관리자·판매자 생성: 대소문자만 다른 username 충돌 2건, 대문자 그대로 저장 1건 - AccountCredentialRepository.findCredentialByUsername: 대소문자 무관 조회 3건
feat: 계정 아이디에 영문 대문자 허용
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 32 minutes. View limit detailsLimit details: You’ve used the included review currently available. Review configuration: ⚙️ Run configuration
📒 Files selected for processing (2)
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (8)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthrough관리자와 판매자 username에서 영문 대문자를 허용하도록 정규식과 입력 설명을 변경했습니다. 대소문자만 다른 username의 조회 및 중복 처리 사례를 검증하는 테스트를 추가했습니다. Changes관리자 username 대소문자 처리
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to 대문자 username을 허용해도 대소문자만 다른 계정은 중복으로 처리되고 대소문자 비구분 로그인이 유지됩니다. 확인된 병합 차단 위험은 없습니다. 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
🧹 knip — dead-code 리포트전체 리포트
|
🩺 NestJS Doctor — 90/100 (Excellent)진단 484건 (error 12).
architecture / security 상위 항목
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0e366005b8
ℹ️ 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".
| /** 정책: 소문자·숫자·`.`·`_`·`-`만. 대소문자 혼용 username 충돌을 원천 차단한다. */ | ||
| export const USERNAME_PATTERN = /^[a-z0-9._-]+$/; | ||
| /** 정책: 영문 대소문자·숫자·`.`·`_`·`-`만. 대소문자만 다른 username은 컬럼 정렬(ci)이 같은 값으로 봐 중복으로 막는다. */ | ||
| export const USERNAME_PATTERN = /^[A-Za-z0-9._-]+$/; |
There was a problem hiding this comment.
When ADMIN_SEED_USERNAME contains a newly valid mixed-case value such as Ops.Admin, seedAdmins still passes it to prisma/seed/credential-policy.ts, whose lowercase-only regex throws before the account is inserted. This leaves the documented optional admin-seeding path unable to use the username policy introduced here; update that validator and its error message, preferably by reusing the shared policy.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
반영: 시드 username 검증이 생성 정책(USERNAME_PATTERN·길이 상수)을 직접 쓰도록 바꿈 — fix/release-review-seed-username → develop PR로 반영 후 이 릴리즈에 포함.
Coverage report
Test suite run success4262 tests passing in 384 suites. Report generated by 🧪jest coverage report action from 8e20ef8 |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
릴리즈 #527 Codex 리뷰 반영. #526에서 생성 정책에 대문자를 허용했는데 prisma/seed/credential-policy.ts는 소문자 전용 정규식을 따로 들고 있어 ADMIN_SEED_USERNAME=Ops.Admin 같은 값이 시드에서 거절됨. - 시드 검증이 USERNAME_PATTERN·MIN/MAX_USERNAME_LENGTH를 직접 쓰도록 변경(정책 사본 제거) - 위반 메시지: 소문자 → 영문 대소문자, 값은 계속 싣지 않음 테스트 - src/test/seed-credential-policy.spec.ts: username 10건을 시드 검증과 AdminCreateAdminInput이 같은 판정을 내리는지 표로 확인, 메시지에 값 미포함·password 위반 메시지 각 1건 - 반증: 수정 전 시드 정책으로 되돌리면 대문자 케이스 2건 실패
fix: 시드 username 검증을 생성 정책과 맞춤
|
@coderabbitai review |
|
1 participant
관리자 페이지에서 관리자·판매자 계정을 만들 때 아이디에 영문 대문자를 허용하는 변경과 릴리즈 리뷰 수정을 운영에 반영합니다.
.·_·-USERNAME_TAKEN(컬럼 정렬utf8mb4_unicode_ci)USERNAME_PATTERN·길이 상수를 직접 사용운영 반영 사항입니다.