Feat/#46 [💡 FEAT] AI 코드 리뷰 도입 V2 #47

LuizyHub · 2024-08-13T17:39:26Z

🚀 작업 내용

새로운 AI 도입

📸 이슈 번호

close [💡 FEAT] AI 코드 리뷰 도입 V2 #46

✍ 궁금한 점

전체코드에 대한 요약이 제공됩니다
영어로 리뷰됩니다

- PR Template을 업데이트했습니다.

* feat: 로그 레벨을 나타내는 Level enum 생성 - Level은 ERROR과, INFO를 가집니다. * feat: 로그 데이터를 담는 LogData VO 생성 - LogData VO 내에서 data에 대한 유효성 검사를 진행합니다. * test: 로그 데이터에 관한 테스트 작성 - 정상적으로 LogData를 만들 수 있다. - 빈 문자열로 LogData를 만들면 예외가 발생한다. - null로 LogData를 만들면 예외가 발생한다. * feat: Level enum에 from 메서드 추가 - level 문자열로부터 enum을 만들 수 있는 from 정적 팩토리 메서드를 추가했습니다. * test: Level enum에 from 메서드에 관한 테스트 추가 - level 문자열로부터 enum을 만들 수 있는 from 정적 팩토리 메서드에 관한 테스트를 추가했습니다. * feat: Log 도메인 객체 생성 - Log는 applicationId를 반드시 가져야합니다. - Log는 Level을 가져야하며, Level VO 내에서 유효성 검사를 진행합니다. - Log는 logData를 가지며, LogData VO 내에서 유효성 검사를 진행합니다. - Log는 로그가 생성된 timestamp를 반드시 가져야합니다. * test: Log 도메인 객체에 대한 단위 테스트 추가 - Log 도메인 객체 생성에 대한 테스트 케이스 구현 - 새로운 Log 도메인 객체 생성 테스트 - DB에 저장된 Log 도메인 객체 생성 테스트 - 정상 케이스 및 예외 케이스(null 값 처리) 테스트 포함 * refactor: domain하위 values, enums 패키지 추가 및 이동 - Level을 enums 하위로 이동 - LogData를 values 하위로 이동 - Test코드를 해당 패키지 구조에 맞게 변경 * test: LevelTest에 Dispalyname 추가 * test: LogDataTest에 Displayname 추가 - LogData VO 테스트 메세지 추가 * refactor: LogData VO를 정적 팩토리 메서드를 사용하도록 수정 - 상속에 의한 확장을 방지하고자 VO를 정적 팩토리 메서드를 이용하도록 변경했습니다. - LogDataTest의 createLogData 테스트 필드 명을 변경했습니다. * refactor: Level from메서드 수정 - from 메서드에서 values를 사용하고, 없을 경우 예외를 발생시키도록 수정함 - level의 upperCase 확인 시 NPE 발생 가능성으로 null check를 상단에 추가 * test: Level enum 테스트 추가 - Level 문자열이 앞 뒤로 공백을 포함하는 경우는 정상적으로 enum을 반환하는 것을 테스트 추가 - 잘못된 문자열이 들어오는 경우 예외를 발생시키는 테스트 추가

* feat: Project 도메인 엔티티 구현 - Project 도메인 엔티티를 구현했습니다. - Project 생성 시 전달받은 이름이 공백이거나 없을 경우 예외를 반환하는 검증로직을 추가했습니다. - 이에 따른 테스트를 진행했습니다. * chore: 테스트 메서드 명 변경 - 일부 테스트 메서드 명을 변경했습니다. * feat: 검증 로직 추가 - VARCHAR(100) 이상의 값을 저장할 수 없게 검증 로직을 추가했습니다.

* refactor: Log 패키지 위치 변경 - 도메인 코드와 test코드를 domain/log 하위로 이동했습니다. * feat: LogRepository 추가 - Log를 저장하는 save 메서드 추가 - LogId를 기준으로 Log를 찾는 findById 메서드 추가 * feat: LogRepository 테스트 추가 - Log를 저장하는 save 메서드를 테스트 - LogId를 기준으로 Log를 찾는 findById를 테스트 * refactor: rowMapper 상수화 - 매번 메서드로 생성되는 rowMapper를 상수로 변경했습니다. * refactor: key에 null이 오는 경우를 Optional chaining으로 변경 * test: Repository Test 시 Test Profile을 사용하도록 변경 * fix: findById에서 데이터가 없는 경우 에외처리 추가 - EmptyResultDataAccessException 발생 시 Optional empty 반환 * test: 없는 Log에 대한 테스트 추가 - findNonexistentLog 테스트 추가

* feat: Project 생성 로직 구현 - 서비스 - Project 생성을 진행하고 DB 저장후 결과를 반환하는 서비스 로직을 구현했습니다. - 요청에 대한 반환은 `ProjectCreateResponse` DTO를 통해 반환됩니다. - 이에 따른 테스트를 추가했습니다. * chore: 클래스 이름 변경 - ProjectCreateResponse -> ProjectCommonResponse * feat: Project 조회 로직 구현 - 서비스 - Project 조회을 진행하고 결과를 반환하는 서비스 로직을 구현했습니다. - 이름에 의한 조회가 주를 이루기 때문에 인덱싱을 추가했습니다. - 요청에 대한 반환은 `ProjectCommonResponse` DTO를 통해 반환됩니다. - 이에 따른 테스트를 추가했습니다. * feat: Project 이름 업데이트 기능 추가 - Project 이름 업데이트 기능을 추가했습니다. - 이름 검증 로직이 중복되어 `validateName` 메서드로 추출했습니다. - 이에 따른 테스트를 추가했습니다. - `@CreatedDate`, `@LastModifiedDate` 활성화를 위한 Configuration Class를 추가했습니다. * feat: Project 수정 로직 구현 - 서비스 - Project 수정을 진행하고 결과를 반환하는 서비스 로직을 구현했습니다. - 요청에 대한 반환은 `ProjectCommonResponse` DTO를 통해 반환됩니다. - 이에 따른 테스트를 추가했습니다. * feat: Project 삭제 로직 구현 - 서비스 - Project 삭제를 진행하고 삭제한 ID를 반환하는 서비스 로직을 구현했습니다. - id 기반 조회 중복 로직이 존재해 메서드를 추출했습니다. - 이에 따른 테스트를 추가했습니다. * feat: Project 서비스 트랜젝션 추가 - Project 서비스 메서드들에 대해 `@Transactional`을 적용했습니다.

* config: 프로젝트 세팅 - npm init - 배포를 위한 wepack bundler 추가 - 개발을 위한 타입스크립트 추가 * config: gitignore 파일 추가 - 빌드 디렉터리 - node modules - IDEA 설정파일 * feat: LogBat sdk 추가 - 기존 console.log, console.error를 재정의 - app_id 를 초기화 할 수 있도록 설정 - 사용 예시는 문서화 예정 * fix: cosole 바인딩 문제 해결 - 초기화 함수가 여러번 호출되어 여러번 초기화 되는것을 방지 - 제대로 console이 오버라이드 되지 않는 문제 해결 * cofing: 테스트 라이브러리 추가 - Jest 도입 * test: SDK 테스트 추가 * docs: README.md 추가 * fix: sdk 초기회 예시 변경

* chore: LogRepository에 잘못 사용한 생성자 변경 - AllArgsConstructor에서 RequiredArgsConstructor로 변경 * feat: CreateLogServiceRequest 레코드 추가 - Service 요청을 담당하는 CreateLogServiceRequest를 추가했습니다ㅏ. * feat: Log를 저장하는 LogService 추가 - Log를 만들고 저장하는 LogService를 추가했습니다. - AppId 검증 로직이 이후에 추가되어야 합니다. * test: Log를 저장하는 LogService에 대한 테스트 추가 - Log를 만들고 저장하는 LogService에 대한 테스트를 추가했습니다. - AppId 검증 로직에 대한 테스트가 추가되어야 합니다. * feat: Controller의 요청을 받기 위해 CreateLogRequest를 추가했습니다. - logLevel은 `@NotBlank`를 통해 검증했습니다. - logData를 `@NotBlank`를 통해 검증했습니다. - LocalDateTime을 `@NotNull`를 통해 검증했습니다. * feat: 컨트롤러 Request에서 CreateLogServiceRequest변환하는 정적 팩토리 메서드 추가 - Header에서 받아온 applicationId와 Controller Request를 파라미터로 받아서 만들도록 구현 * refactor: CreateLogRequest를 레코드를 사용하도록 변경 - CreateLogRequest를 레코드로 변경했습니다. - CreateLogServiceRequest의 of메서드를 파라미터 변경 * config: build.gradle에 validation 추가 * feat: LogController 추가 - applicationId에 대해 Positive, NotNull Validation 추가 - CreateLogRequest에 대해 Validation 추가 * test: Cntroller 테스트 환경 통합을 위해 ControllerTestSupport 추가 * test: LogController에 대한 단위 테스트 추가 - 유효한 입력 값으로 로그 생성 성공 테스트. - app_id 누락 시 400 Bad Request 반환 테스트. - 음수 app_id에 대한 400 Bad Request 반환 테스트. - logLevel 및 logData 필드에 대한 null 및 빈 문자열 입력 시 400 Bad Request 반환 테스트. - timestamp 누락 시 400 Bad Request 반환 테스트. * refactor: Transactional import를 jakarta에서 springframework로 변경 * refactor: logController에서 prefix logs는 RequestMapping을 사용하도록 수정 - `@RequestMapping("/logs")`와 `@PostMapping`을 사용하게 수정 * feat: Validation에서 message 추가하도록 변경 - LogController의 applicationId에 메세지 추가 - CreateLogRequest의 Validation에 메세지 추가 * feat: Log를 생성하는 of 정적 팩터리 메서드 추가 - LogService에서 Log를 생성 시 of 팩터리 메서드를 이용하도록 변경

* feat: AppType Enum 구현 - App Type 구분을 위한 Enum을 구현했습니다. - 테스트를 진행했습니다. * feat: App 도메인 엔티티 구현 - App 도메인 엔티티를 구현했습니다. - App 생성 시 전달받은 인자가 없을 경우 예외를 반환하는 검증로직을 추가했습니다. - 이에 따른 테스트를 진행했습니다.

* fix: 브라우저에서 동작 가능하게 개선 - 브라우저 호환 * fix: endpoint 변경

* fix: http header에 언더바 대신 하이픈 사용하도록 수정 - nginx에서 언더바는 걸러짐 * test: http header에 언더바 대신 하이픈 사용하도록 수정 - nginx에서 언더바는 걸러짐 * fix: Http header에 "_"가 포함되지 않도록 변경 - "app_id" -> "app-id" --------- Co-authored-by: luizy <[email protected]>

* feat: App token 추가 - App 생성시 UUID 기반 토큰이 생성되도록 했습니다. - 이에 따른 테스트를 수정했습니다. * feat: App 생성 로직 구현 - 서비스 - App 생성을 진행하고 DB 저장후 결과를 반환하는 서비스 로직을 구현했습니다. - 요청에 대한 반환은 `AppCommonResponse` DTO를 통해 반환됩니다. * feat: App 조회 로직 구현 - 서비스 - Token을 활용해 App을 조회하는 로직을 구현했습니다. - ID를 활용해 App을 조회하는 로직을 구현했습니다. - 이에 따른 테스트를 추가했습니다. * chore: 인덱싱 추가 - 토큰 기반 조회 쿼리 성능 향상을 위해 인덱싱을 명시했습니다. * feat: App 삭제 로직 구현 - 서비스 - App 삭제를 진행하고 성공시 삭제한 App Id를 반환하도록 했습니다. * feat: App 서비스 트랜젝션 추가 - App 서비스 메서드들에 대해 `@Transactional`을 적용했습니다.

* chore: 메서드 파라미터 변경 - ProjectService.updateProjectValues의 파라미터를 변경했습니다. * feat: 공통 응답 규격 구현 - 공통 응답 규격인 ApiCommonResponse를 구현했습니다. * feat: Project Controller 구현 - Project API Controller를 구현했습니다. (URI: `/v1/projects`) - GET `/{name}`: 프로젝트 조회 - POST `/`: 프로젝트 생성 - PUT `/{id}`: 프로젝트 수정 - DELETE `/{id}`: 프로젝트 제거 * test: Project Controller 테스트 - `@WebMvcTest` 기반 ProjectController 테스트를 진행했습니다. * chore: ControllerTestSupport 수정 - ProjectService MockBean 이동

github-actions · 2024-08-13T17:54:21Z

Risk Level 4 - /home/runner/work/Team5-Guys/Team5-Guys/index_test.ts

API Key Exposure: The appId is being set as a static property and can be exposed if not handled properly. Consider using environment variables or a secure vault to manage sensitive information.
Error Handling: The error handling in the sendLog method could be improved. Instead of just logging the error, consider implementing a retry mechanism or a fallback strategy to ensure logs are not lost.
Console Method Override: The overridden console methods could lead to unexpected behavior if other parts of the application rely on the original console methods. Ensure that this is well-documented and consider providing a way to restore the original methods if needed.

🔒📈⚠️

Powered by Code Review GPT

miiiinju1

good

tidavid1

👍🏻

tidavid1 and others added 18 commits August 12, 2024 17:00

style: PR Template Update (#26)

b7a7789

- PR Template을 업데이트했습니다.

Fix/#38 JS SDK가 브라우저에서 동작하지 않는 이슈 (#39)

a4d16f0

* fix: 브라우저에서 동작 가능하게 개선 - 브라우저 호환 * fix: endpoint 변경

config: code-review-gpt로 리뷰어 변경

a0e024a

config: gpt-4o-mini로 모델 설정

1927c05

Update code-review.yml

4f4dd61

TEST

600200e

Update code-review.yml

7f53961

add permissions

e065ffc

LuizyHub added 4 commits August 14, 2024 02:55

Update code-review.yml

7546b00

Create index.ts

22b6d92

Delete index.ts

f84a0a0

Delete index_test.ts

bb0ce11

tidavid1 deleted the branch dev August 14, 2024 03:55

tidavid1 closed this Aug 14, 2024

LuizyHub changed the title ~~Test PR~~ [💡 FEAT] AI 코드 리뷰 도입 V2 #46 Aug 14, 2024

LuizyHub changed the title ~~[💡 FEAT] AI 코드 리뷰 도입 V2 #46~~ Feat#46 [💡 FEAT] AI 코드 리뷰 도입 V2 Aug 14, 2024

LuizyHub changed the title ~~Feat#46 [💡 FEAT] AI 코드 리뷰 도입 V2~~ Feat/#46 [💡 FEAT] AI 코드 리뷰 도입 V2 Aug 14, 2024

LuizyHub requested a review from a team August 14, 2024 07:18

LuizyHub self-assigned this Aug 14, 2024

LuizyHub added 🚀 Config 의존성 관리, 빌드, 배포 등 개발 환경에 관련된 작업 ⭐️ Feat 새로운 기능이나 요청 labels Aug 14, 2024

LuizyHub reopened this Aug 14, 2024

miiiinju1 approved these changes Aug 14, 2024

View reviewed changes

tidavid1 approved these changes Aug 14, 2024

View reviewed changes

LuizyHub closed this Aug 14, 2024

LuizyHub deleted the Feat/#46 branch August 14, 2024 08:24

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feat/#46 [💡 FEAT] AI 코드 리뷰 도입 V2 #47

Feat/#46 [💡 FEAT] AI 코드 리뷰 도입 V2 #47

LuizyHub commented Aug 13, 2024 •

edited

Loading

github-actions bot commented Aug 13, 2024 •

edited

Loading

miiiinju1 left a comment

tidavid1 left a comment

Feat/#46 [💡 FEAT] AI 코드 리뷰 도입 V2 #47

Feat/#46 [💡 FEAT] AI 코드 리뷰 도입 V2 #47

Conversation

LuizyHub commented Aug 13, 2024 • edited Loading

🚀 작업 내용

📸 이슈 번호

✍ 궁금한 점

github-actions bot commented Aug 13, 2024 • edited Loading

Powered by Code Review GPT

miiiinju1 left a comment

Choose a reason for hiding this comment

tidavid1 left a comment

Choose a reason for hiding this comment

LuizyHub commented Aug 13, 2024 •

edited

Loading

github-actions bot commented Aug 13, 2024 •

edited

Loading