fix: 달력 검증 도구가 장중 실행에서 오늘을 휴장으로 오판하던 문제 - #460
Merged
Merged
Conversation
머지 직후 오늘 날짜까지 넣고 돌려 보니 2026-08-27을 휴장 오류로 잡았다. 장중(10:16)이라 지수 봉이 아직 안 올라왔을 뿐인데 '데이터 없음'을 '휴장'으로 읽었다. 데이터가 아직 없는 최근 날은 휴장이 아니라 모름이다. 확인 범위를 지수의 마지막 봉 날짜까지로 자르고, 어디까지 확인했는지 출력에 적는다. 없는 사실을 오류로 보고하지 않는 게 이 도구의 존재 이유다. 매일 오탐이 하나씩 뜨면 진짜 달력 오류가 묻힌다 — 이 저장소가 경보 위생으로 다섯 번 데인 그 패턴이다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
문제
#459 머지 직후 오늘 날짜까지 넣고 돌려 보니 오탐이 났다.
장중(10:16)이라 KOSPI 봉이 아직 안 올라왔을 뿐인데, 도구가 '데이터 없음'을 '휴장'으로 읽었다.
왜 그냥 두면 안 되나
장중에 돌릴 때마다 오늘이 오류로 뜬다. 매일 오탐이 하나씩 쌓이면 진짜 달력 오류가 그 안에 묻힌다. 이 저장소가 경보 위생으로 다섯 번 데인 패턴이고,
docs/OPERATING_PRINCIPLES.md원칙 5에 적어 둔 것이기도 하다.무엇보다 이 도구는 없는 사실을 오류로 보고하지 않는 것이 존재 이유다. 그 자신이 그걸 어기면 쓸 수 없다.
조치
데이터가 아직 없는 최근 날은 휴장이 아니라 모름이다. 확인 범위를 지수의 마지막 봉 날짜까지로 자르고, 어디까지 확인했는지 출력에 적는다.
회귀 테스트를 붙였다. 기존 로직 테스트도 새 상한과 겹치지 않는 날짜로 옮겼다(6/3이 실제 휴장일로 등록돼 있어 순수 로직 검증에 섞였다).
전체 스위트 1983개 통과.