Repository navigation
fix: 기업 코멘트 시간 초과와 근거 ID 표기 오류 복구 - #62
danjee2009 wants to merge 1 commit into
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1cdf4cb99b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| MAX_TOKENS: int = Field(4096, gt=0) | ||
| TIMEOUT_SECONDS: float = Field(60, gt=0, allow_inf_nan=False) |
There was a problem hiding this comment.
When operators rely on the API reference identified by README.md as the sole configuration reference, docs/api.md:410 and docs/api.md:918 still advertise a 20-second timeout and 1,500-token limit. The runtime now defaults to 60 seconds and 4,096 tokens, so deployment capacity, latency, and cost estimates based on the documented values will be incorrect; update those documented defaults with this change.
Useful? React with 👍 / 👎.
| if result is not None: | ||
| return result | ||
| if attempt == 1: | ||
| logger.warning("company_fit.retry reason=invalid_submission attempt=1") | ||
| reason = "output_truncated" if response.stop_reason == "max_tokens" else "invalid_submission" |
There was a problem hiding this comment.
For responses ending with stop_reason=max_tokens, this now falls through to a second model call, but docs/api.md:1017-1018 still says that only invalid submissions are regenerated and token truncation is never retried. Consumers using that reference to estimate call volume, cost, or timeout behavior will observe different semantics, so document the new single compact retry.
Useful? React with 👍 / 👎.
danjee2009
left a comment
danjee2009
left a comment
There was a problem hiding this comment.
기업 코멘트 생성의 timeout·token 예산, 출력 잘림 재시도, 등록된 fact_id 접두사 정규화와 관련 테스트를 검토했습니다. 확인된 실질적인 문제는 없습니다. GitHub behavior 및 checks 작업이 통과했습니다. 운영 데이터 복구 절차는 별도 작업이므로 이 리뷰에서는 실행하지 않았습니다.
기업 면접의 기본 채점은 성공해도 기업 코멘트가 비던 두 경로를 수정합니다. Sonnet 호출의 기본 전체 예산을 20초에서 60초, 출력 예산을 1,500에서 4,096토큰으로 늘리고,
[f:72]에서 복사된f:72를 등록된 근거 ID에 한해서 원본 ID로 변환합니다. 출력 잘림은 같은 전체 제한 시간 안에서 간결한 형식으로 1회 재시도하며, 기존 근거·문항·길이 검증과 점수 계산은 유지합니다.검증: 실제 누락 사례에서 20.02초 시간 초과와 근거 ID 불일치를 재현했고, 수정 후 누락 코멘트 4건을 생성했습니다. 전체 분석 테스트 1,548개 통과, 선택 의존성·브라우저 표본 조건 12개 건너뜀. 기업 관련 테스트 103개 통과, 변경 파일 Ruff 검사 통과. 운영 복구는 별도 백업·조건부 갱신 작업이며 이 PR이 DB를 자동 수정하지 않습니다.