Repository navigation
feat: 예산 초과 시 사용액·권장값 표시 및 --no-max-budget 추가 (#13) - #14
Merged
Merged
Conversation
- engines/ 디스패처와 격리된 read-only Codex CLI 어댑터 추가 - 캐시 키와 진단 스냅샷에 엔진 정보를 보존하고, 엔진이 다른 비교/재개는 거부 - 설정 파일(engine 필드), 리포트, 문서, OpenSpec 변경안과 구현 계획 추가 Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
- CLAUDE.md 전용 내용(개요, 카테고리 가중치, 채점 규칙, 데이터 흐름)을 AGENTS.md로 병합 - CLAUDE.md는 @AGENTS.md import만 남겨 Claude Code와 Codex가 같은 지침을 읽도록 함 - 존재하지 않는 src/checks/ 안내 제거, 모듈 목록을 실제 구조에 맞춤 Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
- 예산 초과 에러에 SDK가 보고한 사용액, 한도, 턴 수와 재시도용 --max-budget 권장값을 표시 - 일반 분석과 --diagnose의 기본 예산을 $0.50에서 $2.00으로 상향 (상수는 engines/selection.ts로 통합) - --no-max-budget 옵션 추가: SDK에 예산 상한을 넘기지 않으며, 달러 제한이 없는 Codex에서도 허용 Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
- 진단 후속 단계가 남은 예산을 초과해도 에러의 사용액·한도·권장값을 전체 실행 기준으로 표시 - --no-max-budget 실행에서 SDK가 비용을 보고하지 않아도 예산이 0으로 바뀌어 인터뷰를 건너뛰지 않도록 수정 Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Contributor
Author
|
코드 리뷰 반영 (커밋: fix: 코드 리뷰 반영):
두 결함 모두 회귀 테스트를 추가했으며 |
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.
Closes #13
변경 요약
--max-budget권장값을 표시한다. 권장값은 사용액과 한도 중 큰 값의 2배를 $0.50 단위로 올림한 값이다. 중단된 실행은 완료까지 필요한 총비용을 알 수 없기 때문에 여유를 두었다.--diagnose의 기본--max-budget을 0.50에서 2.00으로 올렸다. 상수DEFAULT_MAX_BUDGET_USD는src/engines/selection.ts에 두고 analyzer에서 다시 export한다.--no-max-budget: 예산을Infinity로 표현하고, SDK 호출 시maxBudgetUsd를 생략한다. Codex에는 달러 예산 제한이 없으므로--engine codex와 함께 써도 거부하지 않는다.docs/process-diagnosis.md의 옵션 표와 예상 비용 설명을 갱신했다.CLI 출력 예시
테스트
npx tsc --noEmit통과npm test181개 통과 (사용액 표시, 사용액 누락 시 폴백, 무제한 예산의 SDK 옵션 생략, 진단 CLI의--no-max-budget매핑 테스트 추가)🤖 Generated with Claude Code