[engine] git log 파싱에 사용되는 separator 개선

[yoouyeon]

Related issue

• closes [engine] git log 파싱에 사용되는 separator 개선 #759

Result

AS-IS

COMMIT_SEPARATOR와 GIT_LOG_SEPARATOR를 난수 문자열로 사용

git log format

<COMMIT_SEPARATOR><commit hash><GIT_LOG_SEPARATOR><parent hashes><GIT_LOG_SEPARATOR><refs(branches, tags><GIT_LOG_SEPARATOR><author name><GIT_LOG_SEPARATOR><author email><GIT_LOG_SEPARATOR><author date><GIT_LOG_SEPARATOR><committer name><GIT_LOG_SEPARATOR><committer email><GIT_LOG_SEPARATOR><committer date><GIT_LOG_SEPARATOR><commit message><GIT_LOG_SEPARATOR>
<diffstat>

format example

4itc2s8hH-oA64s08h19ofyms0k2vdw7l3yjed2xwranqa0bx0oyeji6ult2I9M-0XOzvHlYPegVPpzbuiugrjiony42hzos0x8rz6aopsx3euzp6sxxthncI9M-0XOzvHlYPegVPpzbHEAD -> mainI9M-0XOzvHlYPegVPpzbJohn [email protected] Sep 4 20:17:59 2022 +0900I9M-0XOzvHlYPegVPpzbJohn Park [email protected] Sep 5 20:17:59 2022 +0900I9M-0XOzvHlYPegVPpzbcommit message title
commit message bodyI9M-0XOzvHlYPegVPpzb
1	0	a.ts

TO-BE

• commit separator: \n 4개 (--numstat 옵션에서 기본적으로 적용되는 \n 2개 + 직접 지정한 \n 2개)
• git log separator: \n

git log format

<commit hash>
<parent hashes>
<refs(branches, tags>
<author name>
<author email>
<author date>
<committer name>
<committer email>
<committer date>
<commit message subject>
<commit message body>
<diffstat>

format example

ofyms0k2vdw7l3yjed2xwranqa0bx0oyeji6ult2
uiugrjiony42hzos0x8rz6aopsx3euzp6sxxthnc
HEAD -> main
John Park
[email protected]
Sun Sep 4 20:17:59 2022 +0900
John Park 2
[email protected]
Sun Sep 5 20:17:59 2022 +0900
commit message title
    commit message body
1	0	a.ts

test result

Work list

• commit을 구분하는 separator를 난수에서 "\n"의 조합으로 변경했습니다.
  • commit separator: \n 4개 (--numstat 옵션에서 기본적으로 적용되는 \n 2개 + 직접 지정한 \n 2개)
  • git log separator: \n
• commit message 데이터를 subject와 body로 분리하고, body 앞에 space(" ") 4개의 indentation을 두었습니다.
• 변경된 format에 맞게 TC를 수정했습니다.

Discussion

[ytaek]

LGGGGTM 입니다!!!

``${FRONT_NEW_LINE}${fakeCommitHash}\n${fakeCommitRef}\n${fakeAuthor}\n${fakeCommitter}`

굳이 따지자면 요런 부분들이 가독성을 조금 해쳐서, 조금 변수화 시키는것도 괜찮을 것 같습니다.
아니면 아예 string value로 그대로 표현해도 좋을 것 같아요.

[ytaek]

드디어 사라졌군요!!! 🎆🎆🎆