fix: summary.ts의 git 호출도 $ 대신 Bun.spawn 헬퍼로 읽는다

[say8425]

왜

#74는 showBytes만 Bun.spawn으로 옮겼다. 남은 $ 중 **가장 위험한 곳은 summary.ts**다 — 빈 상태 카드와 자동 base 전환(워크트리에서 미커밋 0건인 흔한 상태)이 부르는 요약인데, flight 밖이라 타임아웃도 503도 재시도도 닿지 않는다. 걸리면 요약이 영영 안 온다.

실제로 걸리는지 먼저 쟀다(Bun 1.3.12, 실제 getRepoSummary 반복 호출):

baseFiles의 diff --name-only 출력	순차 호출 결과
130KB (변경 3,000 파일)	800번 전부 깨끗
437KB (변경 10,000 파일)	1회차 18번째에 행업, 2회차 150번 깨끗

→ 실재하지만 드물다. 그리고 결정론적으로 재현되지 않는다(8개 동시 호출로도 5라운드 중 간헐적).

무엇을

• server/gitOutput.ts (신규) — gitBytes(args) / gitText(args). showBytes에 있던 Bun.spawn 패턴과 그 근거·동작 계약을 한 곳으로 옮긴다: 종료 코드 무시(.nothrow()와 같음), 스폰 실패는 throw, stdout을 먼저 비운 뒤 exited(지금 Bun은 반대 순서도 교착하지 않지만 — 1.3.12에서 50MB까지 실측 — 구현 세부에 기대지 않는 관례). 호출처가 둘이 된 시점이라 헬퍼로 뺐다. 출력이 64KB를 넘을 수 있는 새 git 호출은 여기를 타야 한다 — 아직 $로 남은 호출(diff.ts의 name-status·ls-files, 지문의 status, refs.ts)은 이 PR 범위 밖이다.
• showBytes → gitBytes. 동작 동일.
• summary.ts → 여섯 호출 전부 gitText. 파일에서 $가 사라진다. 작은 출력(branch --show-current, rev-parse --short, rev-list --count)까지 옮긴 건 "이 파일엔 $가 없다"를 눈으로 확인되는 불변식으로 만들려는 것이다. 인자는 예전에도 보간 하나가 인자 하나였고 지금은 argv라, 전달되는 값은 같다.
• CLAUDE.md — 남은 $ 목록에서 summary.ts를 빼고 "덮지 않는 곳"에 반영, 새 git 호출 규칙이 헬퍼를 가리키게 했다. 그리고 #76에서 제가 test-bun13 문단을 끼워 넣으면서 그 뒤로 밀려 설치가 없는 잡을 설명하는 것처럼 읽히던 "bun install --frozen-lockfile." 한 마디를 "나머지 잡은 …로 설치한다"로 바로잡았다.
• test-bun13 — 새 회귀망도 Bun 1.3.14로 돈다(의존성 없음).

TDD — RED를 어디에 걸었나

summary.ts의 행업은 결정론적으로 재현되지 않으므로 그 파일에 대한 RED 테스트는 만들 수 없다. 그래서 RED는 새 헬퍼에 걸었다: git-output.test.ts는 확실히 걸리는 모양(200KB git show 12개를 8-way로, 3라운드)으로 gitText를 찌르고, 내용을 바이트 단위로 대조한다.

	Bun 1.3.12
헬퍼 없음	❌ Cannot find module '../server/gitOutput.ts'
Bun.spawn 구현	✅ 3/3
뮤테이션: gitBytes를 $로	❌ gitText 버스트가 10000ms 안에 settle하지 않았다 (2/2)

UTF-8 디코딩(한글 파일)과 종료 코드 무시(없는 rev:path → "")도 단언한다.

리뷰: Critical 0 · Important 1 · Minor 5. 리뷰어가 RED($ 뮤테이션 3/3 실패)와 잘림 뮤테이션(Expected 200009 / Received 147456)을 따로 재현했다. 반영한 것 — Important 1(헬퍼 주석이 "64KB 넘는 호출은 전부 여기를 탄다"고 거짓 주장), Minor 2("exited를 먼저 기다리면 교착"은 측정 안 된 주장 — 재 보니 거짓이라 관례로 고쳐 씀), Minor 3·4(CLAUDE.md 서술 정확도). 반영 안 한 것 — beforeAll·TextDecoder 재사용(리포 관례와 무시할 비용).

⚠️ 알려진 한계: summary.ts를 $로 되돌리는 것은 어떤 테스트도 못 잡는다(소스를 grep하는 테스트는 쓰지 않는다). CLAUDE.md 규칙과 리뷰로 지킨다.

테스트 플랜

bun test              # 791 pass (788 + 새 3)
bun run test:coverage # 100% 게이트 통과 (gitOutput.ts 100/100)
bun run typecheck     # 통과
bun run lint          # 에러 0, 경고 21 = main과 동일 (신규 0)
bun run format:check  # 통과
# test-bun13 모의: node_modules 없는 클론에서 Bun 1.3.14로 두 파일 → 4 pass

기존 summary.test.ts 11종·empty-state.test.ts가 리팩터의 동작 보존을 지킨다(관련 파일 묶음 115 pass).

🤖 Generated with Claude Code

https://claude.ai/code/session_01XAe9zh6siLHSR4nPpqm8QK