Conversation
There was a problem hiding this comment.
Code Review
이번 풀 리퀘스트는 개별 브랜치별로 임시 worktree를 생성하여 수행하던 가상 머지 방식을 git merge-tree --stdin을 활용한 일괄 라운드 기반 병렬 수집 방식으로 개선하여 성능을 최적화합니다. 리뷰에서는 git merge-tree 실행 시 -z 옵션이 누락되어 NUL 구분자 파싱이 정상적으로 동작하지 않는 치명적인 오류를 지적하였으며, 일관성을 위해 stdin 지원 여부 확인 함수에도 -z 옵션을 추가할 것을 제안했습니다.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
This file contains hidden or 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
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.
🔗 연관된 이슈
git merge-tree --stdin으로 branch 조합의 병합 결과를 수집한다 #43🎯 의도
git merge-tree --stdin기반 수집으로 교체하여 최대 465개 branch 조합을 worktree와 index 변경 없이 검사하기 위한 구조 전환모든 조합을 한 Git process에서 처리조건 대신 현재 확정된원형 라운드별 Git process 1개실행과 최대min(4, availableParallelism())개 라운드 병렬 처리 적용📝 작업 내용
📌 요약
git merge-tree --stdin --name-only --messages출력 해석기 추가git merge-tree수집 경로로 교체🔍 상세
git merge-tree --stdin지원 여부 단일 확인spawn으로 Git process 1개를 생성하고 commit OID pair를 표준 입력으로 전달clean,confirmed_conflict, merged tree OID, conflict file, conflict type과 path 관계 수집stderr기록min(4, availableParallelism())개 라운드 병렬 처리merge_check_failed격리baseBranch를 포함한 branch 비교 조합을 구성한다 #42 pair 입력 순서 복원baseBranch포함 결과만 기존GitMergeSignal로 변환collectGitMergeSignal호출 형태와worktreeRoot타입 호환성 유지npm run watch, 외부 서비스 호출, workflow dispatch, release 미실행📸 영상 / 이미지 (Optional)
해당 없음