-
Notifications
You must be signed in to change notification settings - Fork 0
해시트리 구축 및 OpenAPI Spec Diff 로직 구현 #19
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
해시트리 구축 및 OpenAPI Spec Diff 로직 구현 #19
Conversation
Summary of ChangesHello @toothlessdev, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 Open API Specification(OAS)의 변경사항을 효율적으로 감지하고 추적하기 위한 기반 시스템을 구축합니다. OAS 문서를 해시 기반의 트리 구조로 변환하고, 이 트리 간의 차이를 식별하여 추가, 삭제, 수정, 타입 변경 등의 상세한 변경 내역을 제공하는 diff 로직을 구현했습니다. 이는 향후 OAS 버전 관리 및 변경 로그 생성 기능의 핵심 구성 요소가 될 것입니다. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
Test Results104 tests 104 ✅ 1s ⏱️ Results for commit a02c00b. ♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
안녕하세요. 이번 PR에서 구현하신 해시 트리 기반의 OAS Spec Diff 로직을 잘 살펴보았습니다. 전체적으로 구조가 잘 잡혀있고, 변경 사항을 감지하는 로직이 체계적으로 구현된 점이 인상적입니다. 특히 added, removed, modified, type_changed 등 다양한 시나리오를 고려한 점과, 이를 뒷받침하는 테스트 코드를 충실하게 작성해주신 덕분에 코드의 신뢰성이 높아졌습니다.
몇 가지 개선점을 제안드리고자 합니다. 타입 안정성을 높이기 위한 HashNode 타입 구조 변경, diffNode 함수의 재귀 호출 효율성 개선, 그리고 diffChildNodes 함수 내 Map 초기화 코드 간소화에 대한 의견을 리뷰 코멘트로 남겼습니다. 검토해보시고 더 나은 방향으로 코드를 개선하는 데 도움이 되길 바랍니다.
훌륭한 기여에 감사드립니다!
📎 Related issues
📦 Scope
📌 Summary
🔍 Changes
✅ Checklist
✍️ Todo