Skip to content

docs(comment-policy): git_clone 슬라이스가 유지로 닫은 사본 넷을 ②·③ 근거로 걷는다 (4차 행 1004→997) - #145

Merged
dlddu-k3s-homelab-k3s-mcp[bot] merged 1 commit into
mainfrom
reconcile/rct_20260914-0002-opus-5-1m
Sep 21, 2026
Merged

dlddu-k3s-homelab-k3s-mcp[bot] merged 1 commit into
mainfrom
reconcile/rct_20260914-0002-opus-5-1m

Conversation

@dlddu-k3s-homelab-k3s-mcp

Copy link
Copy Markdown
Contributor

정합성 task rct_20260914-0002 (모델 tbm_session-platform-comment-redundancy) 의 실행 스테이징이다.

무엇이 어긋났나 (gap)

모델의 정합 상태는 「판정 대상 주석 중 복원 경로 넷(① 코드 ② 저장소 문서 ③ PR ④ 커밋 메시지) 어디로도 복원되는 내용을 담은 것이 없음」이다.

#140git_clone 을 표면에 세우면서 더한 96줄은 원장 4차 행에 제거 23 · 유지 73 으로 등재됐다. 그 유지 73줄 중 네 대목이 ②·③ 으로 복원되는 사본인데 「유지」로 등재돼 게이트 뒤에 잠겼다. 원장이 유지 근거로 열거한 「코드·문서 어디에도 없는 관측」 넷 중 둘이 실측에서 거짓이다.

자리 원장의 유지 근거 실측
gitBin 상수 doc (a) 「git 이 런타임 이미지에 있고 없으면 빌드가 실패한다」 — 어디에도 없다 거짓. 주석이 스스로 인용한 data-plane/Dockerfile 이 갖는다(설치와 git --version 이 같은 && 사슬) — ②. 같은 문장이 #140 본문 「착수 가능성」 전제 표에도 축자로 — ③
실패한 클론을 지우는 자리 (d) 「지우지 않으면 도구가 이름 붙이지 않은 경로가 남는다」 — 어디에도 없다 같은 PR 이 소스와 테스트 두 곳에 같은 구로 적었다. 둘 중 하나가 다른 하나의 사본 — 12차 패스가 닫은 형태와 동일
TestSessionMCPCloneNeedsASharedVolume 헤더 근거로 언급 없음 docs/session-mcp-tool-surface.md 의 「공유 볼륨이 없으면 거절한다 … 성공할 수 없는 호출로 사람을 깨우지 않는다」와 문장 대 문장 대응. 포인터조차 없다 — ②
credential helper 근거 (cloneArgs doc + 테스트 헤더) 「판단이 갈린 것 — 문서는 정책을, 주석은 왜 이 플래그 조합인가를 적어 복원 방향이 반대」 전제 불성립. 문서의 같은 문단이 플래그 층위까지 적는다(helper 목록을 비운다 · 프롬프트만 끄는 것으론 모자란다 · 배포 설정이 아니라 코드가 들고 있게 한다)

게이트는 이것을 잡지 못한다. scripts/check_comment_policy.py 는 등재 범위가 변했는지만 재측정하고 판정의 은 보지 않는다. 원장이 이 증분을 판정 완료로 등재한 순간 이 73줄은 영구 초록으로 잠겼고, 파일이 다시 바뀌지 않는 한 어떤 감지도 다시 보지 않는다. 게다가 #140 본문이 「범위 밖(후속)」에 **「새 코드 주석의 판정 자체 — tbm_session-platform-comment-redundancy 소관」**이라 적어, 이 판정은 설계상 이 축에 인계된 몫이다.

이 PR 이 한 것

주석만 바꾼다 — 운영 규칙 **「포인터는 남기고, 포인터가 가리키는 내용의 사본은 지운다」**의 형태 그대로다.

  1. session_mcp_git.gogitBin doc 2줄 → data-plane/Dockerfile 을 가리키는 포인터 1줄
  2. session_mcp_git.go — 실패한 클론 제거 자리 3줄 → spillBody 와의 대비 1줄(이유는 테스트 헤더가 정본으로 갖는다). 자매 테스트가 전부 「왜 이 단언인가」 한 줄을 들고 있다는 9~15차의 관측을 따라 테스트 쪽을 남기고 소스 쪽을 줄였다
  3. session_mcp_git_test.goTestSessionMCPCloneNeedsASharedVolume 헤더 3줄 → 문서 포인터 1줄
  4. session_mcp_git.go · session_mcp_git_test.go — credential helper 문장만 걷고 -- 둘째 가드의 근거는 유지

순 제거 7줄. 원장 4차 행 1004 → 997 · 지문 72b829cf0933 → 3ba913a73f22, 합계 마커 3127 → 3120, 판정 상세 docs/comment-policy/passes/2026-09-14-session-mcp-git-copies.md 신설.

유지가 옳다고 같은 밀도로 재확인한 것

일괄 제거로 기울지 않도록 패스 문서에 표로 남겼다 — 5분·256자·4 KiB 상한의 출처(문서가 「경계값은 전부 코드가 갖는다 … 여기 적으면 다음 변경에 조용히 거짓이 된다」로 그 자리를 명시적으로 비워 두어 ② 복원 경로가 원리적으로 없다), -- 가 선행 대시 검사와 다르게 실패하는 둘째 가드라는 것, 프롬프트를 끄면 타임아웃까지 막히는 대신 즉시 실패한다는 것, boundedBuffer 의 메모리 근거, 하네스 카운터가 「git 이 돌지 않았다」를 단언하는 자리라는 설명.

검증

로컬에서 전부 rc=0 으로 확인했다.

검사 결과
python3 scripts/check_comment_policy.py OK — 판정 대상 3168줄/134파일 · 판정 완료 3120줄(98.5%) · 등재 범위 15 · R5 스캔 19파일 1581줄
gofmt -l data-plane 빈 출력
go vet ./... (data-plane) rc=0
go test ./... (data-plane) ok github.com/dlddu/session-platform/data-plane/cmd/agent
scripts/check-render-fidelity.py · scripts/check-fidelity-allowlist.py · scripts/e2e/check-scenario-mapping.sh rc=0
코드 동작 불변 git diff -U0 의 변경 줄 중 // 로 시작하지 않는 것 0건 — 바뀐 것이 전부 주석임을 기계로 확인

머지 순서 / 충돌

이 PR 은 새 충돌면을 만들지 않는다. 차등 시험 병합(한 쌍씩, git merge-tree)으로 실측:

열린 PR vs main vs 이 브랜치
#143 ledger.md 충돌 같은 ledger.md 하나
#142 ledger.md 충돌 같은 ledger.md 하나
#127 ledger.md · doc-tracker.md · e2e.md 충돌 같은 셋
#144 · #130 무충돌 무충돌

즉 세 PR 모두 이미 main 과 충돌(#140 착지의 소행)하고, 이 PR 이 더하는 파일은 없다. ledger.md#140 자신이 「설계상 직렬화 지점」으로 적은 파일이다.

나중에 머지되는 쪽이 리베이스해서 산술을 다시 맞춘다. 합계 마커는 각 PR 의 제거 줄 수를 누적한 값이어야 하고(R4), 해당 행의 줄 수·지문은 게이트 FAIL 출력에서 재도출한다 — 손으로 짓지 말 것. ⚠️ #143 의 계획값(3054 → 3043)은 #140 착지 전 기준이라 이미 낡았다; 그 PR 도 리베이스가 필요하다.

리뷰 관점: 이 PR 은 네 자리를 「사본」으로 판정한 것이 맞는지가 전부다. 원본 네 곳(data-plane/Dockerfile 런타임 스테이지 · docs/session-mcp-tool-surface.md 「지금 있는 것」의 git_clone 문단 · #140 본문 · 같은 PR 이 쓴 테스트 헤더)을 열어 대조하면 판정이 재현된다.

🤖 Generated with Claude Code

…행 1004→997)

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
@dlddu-k3s-homelab-k3s-mcp
dlddu-k3s-homelab-k3s-mcp Bot force-pushed the reconcile/rct_20260914-0002-opus-5-1m branch from dc281db to 9949447 Compare September 21, 2026 07:45
@dlddu-k3s-homelab-k3s-mcp
dlddu-k3s-homelab-k3s-mcp Bot merged commit b7ff93f into main Sep 21, 2026
15 checks passed
@dlddu-k3s-homelab-k3s-mcp
dlddu-k3s-homelab-k3s-mcp Bot deleted the reconcile/rct_20260914-0002-opus-5-1m branch September 21, 2026 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants