AI 요약 기능 도입을 위한 사이드바뷰 도입 및 엔진 호출방식 변경 #754
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
choisohyun
approved these changes
Oct 5, 2024
| { | ||
| "id": "githruSidebarView", | ||
| "title": "Githru Sidebar", | ||
| "icon": "images/logo.png" |
There was a problem hiding this comment.
(질문) 요 로고 이미지는 vscode 패키지 안에 올려두지 않아도 동작이 되는 장치가 있는 걸까요?? diff에 이미지는 없는 것 같아서 여쭤봅니다!
There was a problem hiding this comment.
기존에 vscode 패키지안에 있던 로고입니다!
| private async callApi(apiNumber: number): Promise<string> { | ||
| const engine = AnalysisEngine.getInstance(); | ||
| try { | ||
| const summary = await engine.geminiCommitSummary(); |
There was a problem hiding this comment.
epic에 머지되면 여기에 각각 함수 부르게 하면 되겠군요! 👍
ytaek
approved these changes
Oct 6, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related issue
#635 #733
Result
2024-10-05.194358.mp4
githru의 AI 커맨드 창을 호출할 수 있습니다.analysis-engine의 인스턴스가extension의 초기화 이후에도 외부에서 호출할 수 있고 단일한 인스턴스를 유지합니다.Work list
engine.ts를 새로 작성하여 기존 엔진의 내용을 해당 파일로 이동하였습니다.Discussion
사이드바 도입에 대해서
githru유저의 옵션 사양입니다.analysis-engine이 초기화될 때 가지고 있어야하는 내용이 아닙니다. 만약 이 내용들을analysis-engine이 가지고 있다면, 초기화 될 때 모든 AI에 대해서 API 요청을 보내야합니다.엔진의 싱글톤 변경에 대해서
analysis-engine이extension의 최초 실행 함수 내부에 들어가있어서 최초로 한번 실행되고 있었고, 이후에 외부에서 엔진이 호출되는 일은 없었습니다.analysis-engine이 최초로 초기화 된 이후, 해당 엔진 (동일한 인스턴스)에 요청을 보내야합니다. 따라서, 초기화 이후extension외부에서 호출하여 사용할 수 있어야합니다.