Repository navigation
관측 기반 실패 code 의 일시 전환 - #126
Merged
Merged
Conversation
Closed
|
Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This was referenced Oct 18, 2026
- 팩토리 9개의 permanent 플래그를 false 로 전환, 응답 422 에서 502 로 - 원인: 받은 페이지를 보고 내린 판정은 봇 방어·앱 크래시·빈 셸로 재료가 가짜일 수 있음 - 결과: 호출자가 재시도 뒤 attempt 소진으로 닫음 - 참조: TeamPiKi/infra#145 - Gemini 예외에 retryable 축 추가, GeminiRetry·모델 프로브가 permanent 대신 그것을 봄 - 원인: LLM_INVALID_RESPONSE 가 일시가 되면 같은 요청의 즉시 재호출이 늘어남 - 결과: Gemini 호출 수 이전과 동일 - 에스컬레이션 축은 변경 없음 - 확정 유지는 INVALID_URL·BLOCKED_HOST·IMAGE_UNSUPPORTED·MODEL_* - 확정 실패를 전제한 주석 정정
- 핸들러가 cause 없는 일시 실패는 스택 없이 warn - 원인: 받은 페이지 기반 판정(상품 아님 등)이 일시로 바뀌며 전부 warn+스택이 됨 - 원인: cause 없는 예외의 스택은 던진 자리만 가리켜 정보 0 - 프로브 테스트의 permanent 단언을 retryable 로 교체 - 원인: Gemini 예외는 전부 permanent=false 라 단언이 공허함 - 카탈로그 테스트 주석의 422 전제 정정
m-a-king
force-pushed
the
fix/124-transient-observed-codes
branch
from
October 19, 2026 11:16
c9389a7 to
10f5eea
Compare
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.
한눈에
추출 실패 가운데 확정으로 응답하는 범위 축소
Situation
추가
Task
분류 플래그 전환
Action
관측 기반 팩토리 9개의 permanent 전환
Gemini 예외에 즉시 재호출 축 분리
retryable()일시 실패 로그의 스택을 cause 있을 때로 한정
확정 실패를 전제한 주석 정정
Result
검증
비용
후속
docs/api-contract.md의 확정 설명은 이 PR 밖연관 이슈