Skip to content

fix(strings): 사람에게 보이는 코드 문자열의 용어 정리 - #116

Merged
LivingLikeKrillin merged 1 commit into
mainfrom
fix/code-string-terms-2
Oct 18, 2026
Merged

LivingLikeKrillin merged 1 commit into
mainfrom
fix/code-string-terms-2

Conversation

@LivingLikeKrillin

Copy link
Copy Markdown
Owner

개요

문서 용어집을 정비한 PR #112 및 미들웨어 코드 문자열을 다룬 #113의 후속 작업으로, 프로덕션 코드 전반에 남아 있던 사용자 대상 문자열 리터럴을 용어집 8장의 새 표준 용어로 일괄 정비합니다. 식별자나 통신 규격은 변경하지 않고 안내 메시지만 수정하여 외부 연동 시스템에 미치는 영향을 차단합니다.

주요 변경 사항

1. 용어 변경 내역 및 발생 횟수

  • 38개 프로덕션 파일의 80개 라인에서 총 81개 문자열을 새 용어로 대체
  • 변경 대상 모듈: 벤더 어댑터(digit, orbit, spot, g1), adapter-core, adapter-host, client, contracts, uplink, gate checks, harness, mimic, picasso 미들웨어(AdmissionGate, Approval, Middleware, RemedyDesk, DefinedCapability, MissionValidator), registry 서비스 및 컨트롤러
  • 개정판 -> 리비전: 24건
  • 거절 -> 거부: 22건
  • 신원 -> 식별 정보: 11건
  • 능력 -> 케이퍼빌리티: 10건
  • 투영 -> 프로젝션: 3건
  • 출하 -> 프로덕션: 2건
  • 실물 -> 실제 하드웨어: 2건
  • 남쪽 -> 사우스바운드: 2건
  • 거동 -> 동작: 2건
  • 벤더 표면 -> 벤더 인터페이스: 1건
  • 걸음 -> 단계: 1건
  • 대장 -> 레지스터: 1건
  • 서 있는 -> 활성: 1건
  • OrbitAdapter의 사건 -> 실행 이벤트: Orbit 실행 이벤트 목록을 의미하므로 인시던트 대신 실행 이벤트로 반영(1건)
  • 새 용어의 받침 유무에 맞춰 한국어 조사 수정

2. 유지된 항목 및 외부 연동처 영향

  • 식별자, 열거형(enum) 이름, JSON 키, reason 코드, 시험 이름, 주석은 변경 없이 유지
  • 변경된 문자열은 사람에게 보여주는 메시지 전용이며 코드 내부에서 텍스트 직접 비교를 수행하지 않음 확인
  • 외부 이용처인 picasso-ops 코드 읽기 전용 검토 완료: HTTP 상태 코드와 reason 코드로만 분류하므로 기능 영향 없음 확인
  • picasso-ops의 가상 픽스처 1개에 이전 문장이 남아 있으나 동작에 문제 없음 확인
  • handoff/ 디렉터리 및 문서 전반에서 변경된 문장을 인용하지 않음 확인
  • SOURCE_OBSERVATION_UNUSED의 검증기 다음 작업 문구(식별 정보) 변경: picasso-ops의 S3b 계약 문서 인용문은 다음 서브모듈 갱신 시 반영 예정

3. 시험 단언문 수정

  • 프로덕션 문자열을 그대로 비교하는 시험 파일 2개 수정
  • RemedyApprovalTest의 기대값을 걸음 대신 단계 수와 파라미터 수가 다르다 문자열로 갱신
  • IngestTest의 contains 비교를 개정판 대신 모르는 리비전으로 갱신

검증 결과

  • ./gradlew build -x :harness:test -x :registry:test 빌드 수행: 시험 1,567건 통과, 실패 0(JUnit XML)
  • 결함 주입 검증: RemedyApprovalTest 단언문을 이전 문구(걸음)로 되돌렸을 때 정상적으로 실패(1건 실패, JUnit XML)하여 검증 유효성 확인
  • :registry:test 실행 중 이전 문구를 비교하던 IngestTest 단언문(개정판 비교, 396건 중 1건 실패)을 모르는 리비전으로 수정한 뒤 재실행 통과(21건 통과, 실패 0)
  • :harness:test 210건 및 orbit 어댑터 31건 전체 통과(실패 0)
  • 전체 시험 소스 대상 이전 용어 조각 검색 결과, 시험 실패 메시지 외에 추가적인 문자열 비교 로직 부재 확인

🤖 Generated with Claude Code

- 용어집 8장에 맞춰 38개 프로덕션 파일 내 `개정판`, `거절`, `신원` 등 81개 문자열을 리비전, 거부, 식별 정보 등 표준 용어로 변경
- 식별자, 열거형, JSON 키, `reason` 코드, 주석은 기존대로 유지하여 외부 연동 시스템과의 호환성 확보
- 변경된 메시지와 직접 일치 여부를 대조하는 `RemedyApprovalTest`, `IngestTest` 단언문 반영 및 전체 빌드 검증 완료

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@LivingLikeKrillin
LivingLikeKrillin merged commit e6f77b7 into main Oct 18, 2026
1 check passed
@LivingLikeKrillin
LivingLikeKrillin deleted the fix/code-string-terms-2 branch October 18, 2026 09:58
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.

1 participant