남이 고친 서버 변경 검토 — 0.1.15 ~ 0.1.23
무엇을
decisions/302(담당 잠금 해제) 뒤로 장민석 님 검토 없이 운영에 올라간 server/ 변경을 한 번에 훑는다.
대상 — 전부 로그에 「검토 전」으로 적혀 있다
| 태그 | 누가 | 무엇 |
|---|---|---|
0.1.15 |
류준 (09-17) | C-5 이름(문맥·자기소개·호칭)·도로명 뒤 건물번호·집 전화 P4 · 어미·조사 과잉 제거 |
0.1.18 |
류준 (09-18) | closure_rule 25→29(305 EXCLUDED 중 넷 되살림) · 가족 호칭 · compliance_flag 저장 배선 신설 · compliance_rule UPSERT·severity |
0.1.19 |
정성윤 (09-19~21) | GET /health/ready · 추천 구간 분리(decisions/119) · 없는 전사 구간 404 · 콜 가드 저장 실패 200+로그 · /close 가 통화를 닫는다 · 쓰기 일곱 경로 서비스 토큰 문(120) |
0.1.22~0.1.23 |
조서희 (09-21) | agent_auth 이름으로 찾거나 만들기(decisions/406) · GET /hub/agents/me 에 display_name |
전부 이미 운영에서 돈다. 검토는 「되돌릴지」가 아니라 「구조에 맞는지·빠뜨린 경계가 있는지」다.
볼 것
- 계층 계약 4종은 KEPT 다(CI). 계약이 못 보는 것 — 포트 이름·DTO 위치·예외를 어디서 HTTP 로 바꾸는지
305가 EXCLUDED 로 뺐던 절차를 되살린 판단이 규칙표 설계와 맞는가- 저장 실패를 200+로그로 통일한 정책이 다른 경로와 같은가
완료 조건
- 고칠 것이 있으면 고치고, 없으면 「봤고 그대로 둔다」를
_logs/에 남긴다 — 침묵은 검토가 아니다 STATE.mdserver/줄의 「장민석 님 검토 전」 표기를 걷는다
2026-09-22 장민석 — 봤다.
d6f2951..HEAD(0.1.14→0.1.24)server/93파일. 계층·SEC-1·SQL↔스키마·closure_rule29 복원(공통 서류만 필수 — 305 의 제외 사유와 모순 없음)· 서비스 토큰 문·/health/ready는 그대로 둔다. 고친 것 하나:resolve_or_create동시 발급 시 PK 위반 500(ON CONFLICT+ 재조회). 미결로 넘긴 것 둘: 동명이인이 한 상담원으로 합쳐진다 · 저장 실패 정책이 경로마다 다르다(콜 가드·컴플라이언스 200+로그 / 추천·필요서류 500). 리뷰가 낸 「retrieval_ms·generation_ms가 안 쓰인다」는 오판이었다(응답 스키마로 미디에이터에 나간다). 사소:VoiceOutlierPort.judge인자 이름utterances가 실제로는 오디오 경로 — 안 고쳤다.