엔티티 리팩터링 갈무리

[w8385]

업데이트 유형

• Hot Fix
• Release
• Develop
• Others

업데이트 개요

엔티티 구조가 완전하지 않았음
해당 구조 리팩터링에 집중

업데이트 요약

• NestJS에서 권장하는 방식으로 엔티티 구조 갈무리
  • @Schema 클래스
  • export type Document (mongoose)
  • SchemaFactory
• 개발 편의를 위한 팩토리 메서드
• constructor
• static fromDocument

수정 사항 진단

• 코드가 이 프로젝트의 스타일 지침을 따릅니다.
• 코드에 대한 자체 리뷰를 수행했습니다.
• 기능, 정책, 수정 사항, 이슈에 대한 테스트 코드를 추가했습니다.
• 변경 내용이 로컬 개발환경에서의 테스트를 통과했습니다.

[Copilot]

Copilot reviewed 12 out of 12 changed files in this pull request and generated 2 comments.

[Copilot]

Removing the instantiation (i.e., new Problem(problem)) may lead to inconsistent behavior if consumers expect an instance with methods defined on the Problem class. Consider converting the raw problem object to a Problem instance using Problem.fromDocument or the appropriate constructor.

Suggested change

	problems.push(problem);
	problems.push(new Problem(problem));

[w8385]

71ca649

[Copilot]

findByIdAndUpdate by default returns the pre-update document. Consider adding the option { new: true } to return the updated document, ensuring that Poker.fromDocument reflects the latest changes.

Suggested change

	const pokerDoc = await this.pokerModel.findByIdAndUpdate(id, poker);
	const pokerDoc = await this.pokerModel.findByIdAndUpdate(id, poker, { new: true });