refactor(container): /workspace/app 을 한 곳에서만 적는다 — 흩어진 사본 26개 정리

[dldnsgkr]

왜

같은 경로 문자열이 일곱 파일에 흩어져 있었다. 세 파일이 각자 APP_DIR 상수를 따로 선언하고, 나머지는 명령 문자열에 그대로 박아 넣었다 — "cd /workspace/app && ..." 를 손으로 잇는 자리가 열 곳.

파일	출현
PreviewBranchPushService	14
CodeAgentService (프롬프트)	5
BackendSourceClone · PreviewWorkspaceService	각 2
ChangeService · DeployAgentService · PreviewRuntimeDetector	각 1

값이 흩어져 있으면 "이 경로를 쓰는 곳이 어디인가" 에 답이 안 나오고, 그 약속이 깨졌을 때 무엇이 함께 깨지는지도 안 보인다. 실제로 두 번 연달아 터졌다:

• fix(agent): 프레임워크 없는 프로젝트도 /workspace/app 에 만든다 — 저장소 push 가 막히던 문제 #303 — 프레임워크 없는 프로젝트는 스캐폴더가 안 돌아 코드가 /workspace 루트에 쌓였다. push 가 can't cd to /workspace/app 으로 죽었는데, 프리뷰는 index.html 폴백이 있어 화면은 멀쩡히 동작해 승인 단계에 와서야 드러났다.
• fix(deploy): 정적 사이트 배포 실패 수정 + 승인 카드에 사용자용 문구 #307 — 같은 계열. 배포 워크플로가 Node 프로젝트를 전제해 정적 사이트에서 죽었다.

무엇을

• ContainerPaths.APP_DIR 하나로 모으고, "cd APP_DIR && X" 손 조립은 inApp(X) 로 바꿨다. 그 손 조립이 사본이 늘어나던 주된 경로였다.
• 둘 곳은 agent.infrastructure.docker — 여섯 파일 모두 이미 그 패키지를 import 하고 있어(DockerContainerService) 새 결합이 생기지 않는다.
• 코드 리터럴은 이제 ContainerPaths 한 줄뿐이다(나머지는 주석과 LLM 프롬프트).

프롬프트는 손대지 않았다

모델이 읽는 텍스트 블록이라 상수를 끼워 넣으면 읽기 나빠진다. 대신 CodeAgentPromptPathContractTest 가 프롬프트와 상수가 갈라지는 것을 잡는다.

assertThat(prompt).contains("MUST live in " + ContainerPaths.APP_DIR);
assertThat(prompt).contains(ContainerPaths.inApp("npm run build"));

사람이 두 곳을 같이 고치는 것에 기대지 않는다 — 이번 정리의 요지가 "약속을 한 곳에 적는다" 인데, 프롬프트만 예외로 두면 그 자리가 다시 사각지대가 된다.

검증

전체 1330개 통과. 동작 변경 없음 — 값과 생성되는 명령 문자열이 모두 이전과 동일하다.

🤖 Generated with Claude Code

https://claude.ai/code/session_013y8USoCXTsRTATAhy88M93