Skip to content

최종 판정(HUMAN_REVIEW)이 배지에 반영되지 않던 문제 - #16

Merged
needsbuilder merged 1 commit into
mainfrom
fix/final-verdict-badge
Aug 25, 2026
Merged

최종 판정(HUMAN_REVIEW)이 배지에 반영되지 않던 문제#16
needsbuilder merged 1 commit into
mainfrom
fix/final-verdict-badge

Conversation

@needsbuilder

Copy link
Copy Markdown
Owner

무엇이 잘못됐나

재교정 한도(max_revise)를 소진한 REVISEpipeline._final_verdict()
HUMAN_REVIEW 로 승격시킨다(spec 6.8 case d). 그런데 렌더러와 웹 배지는
승격 값인 result.fidelity.verdict 를 읽고 있었다.

결과적으로 "고치지 못해서 멈춘" 바로 그 경우가 화면에서는 성공 신호로 표시됐다:

수정 전 수정 후
result.verdict human_review human_review
HTML 배지 사실충실성: 수정 필요 사실충실성: 검수 필요
웹 UI 배지 Fidelity 재교정됨 🔁 검수 필요 ⚠️ (사람 확인 권장)

금액·기한이 여전히 누락된 출력에 "재교정됨 🔁" 이 붙는다. 또박의 핵심 약속인
"고치지 못하면 멈춘다" 와 화면이 정반대로 말하고 있었다.

재현

doc = Document(blocks=[Block(type=BlockType.PARAGRAPH,
      text="보험료 1,295,400원을 2026년 7월 17일까지 내세요.")], source_mime="text/plain")
r = simplify(doc, Level.EASY, FakeProvider(default="보험료를 기한 안에 내세요."), max_revise=2)
# r.verdict = human_review / r.fidelity.verdict = revise

고친 방법

배지가 EasyReadResult.verdict(파이프라인 최종 판정)를 따르도록 했다.
FidelityReport.verdict 는 마지막 단일 검증 결과, EasyReadResult.verdict
파이프라인 최종 판정이라는 구분을 주석으로 고정했다.

왜 안 잡혔나

기존 test_render_html_human_review_badgeresult.verdict
result.fidelity함께 HUMAN_REVIEW 로 세팅해서, 두 값이 갈라지는
분기를 한 번도 밟지 않았다. 이번에 렌더·웹 양쪽에 그 분기를 회귀 테스트로 고정했다.

검증

  • python -m pytest -q419 passed (417 + 신규 2)
  • ruff check ttobak/ tooling/ scripts/ tests/ → clean

재교정 한도를 소진한 REVISE 는 pipeline._final_verdict() 가 HUMAN_REVIEW 로
승격시킨다(spec 6.8 case d). 그런데 렌더러와 웹 배지는 승격 전 값인
result.fidelity.verdict 를 읽고 있었다 — 그래서 "고치지 못해서 멈춘" 바로 그
경우가 화면에서는 "수정 필요" / "Fidelity 재교정됨 🔁" 로 표시됐다.
금액이 여전히 누락된 출력에 성공 신호가 붙는 셈이다.

배지가 EasyReadResult.verdict(파이프라인 최종 판정)를 따르도록 고치고,
두 값이 갈라지는 경우를 렌더·웹 양쪽에서 회귀 테스트로 고정했다.
기존 테스트는 두 값을 함께 HUMAN_REVIEW 로 세팅해 이 분기를 밟지 않았다.
@needsbuilder
needsbuilder merged commit d228c10 into main Aug 25, 2026
7 checks passed
@needsbuilder
needsbuilder deleted the fix/final-verdict-badge branch August 25, 2026 03:04
needsbuilder added a commit that referenced this pull request Aug 25, 2026
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