Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(core): Module 데코레이터가 평가될 때, 모듈 그래프 탐색 방법 변경 #57

Open
biud436 opened this issue Nov 19, 2024 · 1 comment

Comments

@biud436
Copy link
Owner

biud436 commented Nov 19, 2024

모듈 옵션은 데코레이터가 평가될 때가 아니라,

루트 객체로부터 모듈 옵션을 파싱하여 트리화 한 후,

우선 위상 정렬을 통해 정렬을 해야 합니다.

그 후, onModuleInit()를 해야 합니다.

또한 실제로 모듈이 DI 객체 트리에 있는지도 확인해야 합니다.

@biud436
Copy link
Owner Author

biud436 commented Dec 17, 2024

  • 9340441 에서 모듈 정렬 기능 추가하였음.
  • 모듈 정렬 기능은 depth 비교가 아직 없어서 반쪽짜리. 그저 평탄하게 전부 펼쳐놓는 것에 불과. 위상 정렬은 dfs의 한 형태. 테스트 필요.
  • 순환 참조 부분에서 약간의 문제가 있어서 lazy하게 평가되는 모듈을 추가해야 할 것 같음.
  • onModuleInit의 호출 순서를 바꿔야 할 것으로 생각됨.
  • 변경한 후 새로운 테스트 코드 작성 필요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant