fix(autoresearch): capture benchmark exit code via PIPESTATUS - #756
Merged
Conversation
EXIT_CODE was read from $? after a pipeline, which holds tee's status (~always 0), so crashed benchmark runs were logged as valid results. Also adds a Goodhart guardrail before keep decisions and checkbox state markers to the ideas backlog, plus research notes on simpler alternatives.
baleen37
enabled auto-merge (squash)
August 8, 2026 03:25
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughautoresearch 대안 조사 문서를 추가하고, benchmark 종료 상태와 metric 개선 검증 규칙을 강화했다. ideas backlog는 체크박스 상태로 시도 여부를 추적하도록 변경했다. Changesautoresearch 대안 조사
실험 실행 프로토콜
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
baleen-release-app Bot
pushed a commit
that referenced
this pull request
Aug 8, 2026
## [17.42.2](v17.42.1...v17.42.2) (2026-08-08) ### Bug Fixes * **autoresearch:** capture benchmark exit code via PIPESTATUS ([#756](#756)) ([8637cff](8637cff))
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.
Summary
autoresearch 스킬의 crash 감지 버그를 수정하고, 자율 루프의 판정 신뢰도를 높이는 최소 변경 2개를 추가합니다.
Changes
1.
EXIT_CODE캡처 버그 수정 (핵심)파이프라인 뒤의
$?는tee의 종료 상태를 담습니다.tee는 거의 항상 성공하므로 벤치마크가 죽어도EXIT_CODE=0이 되어, 모든 crash가 정상 결과로 JSONL 원장에 기록됩니다. 자율 루프에서 조용히 데이터가 오염되는 종류의 버그입니다.2. Goodhart 가드레일 —
keep판정 전, metric이 개선된 게 작업이 빨라져서인지 작업이 안 일어나게 돼서인지 확인하도록 요구. 설명되지 않는 큰 개선은 보통 벤치마크가 약해진 것(캐싱, 반복 스킵, 입력 축소, 체크 완화)입니다.3.
ideas.md체크박스 상태 —- [ ]미시도 /- [x]시도함. resume한 에이전트가 이미 시도한 아이디어를 재시도하는 것을 방지합니다. 기존에는worklog.md와 교차 확인해야 알 수 있던 정보입니다.4. 리서치 노트 추가 —
docs/superpowers/research/2026-08-08-autoresearch-alternatives.md. "현재 설계보다 더 단순한 범용 대안이 있는가"를 1차 자료로 조사한 결과. 결론은 없음 — 현재 설계가 Anthropic의 long-running agent 가이드(progress file + git history)와 이미 일치합니다. 이 레포에 리서치 노트 관례가 없어docs/superpowers/plans/의 date-prefix 규칙을 따라 형제 디렉토리research/를 신설했습니다(파일 상단에 명시).Tests
bats tests/— 70 passed, 0 failedmarkdownlint— 위반 23개 → 22개 (신규 위반 0, 긴 줄 정리로 1개 감소)shellcheck plugins/autoresearch/hooks/autoresearch-context.sh— OK0으로 잡히고,${PIPESTATUS[0]}에선17로 정확히 잡힘. 성공 케이스는0유지./code-review— 발견 사항 없음Notes
리서치에서 확인된 나머지 개선안(셸 루프 실행 모드, dashboard 재생성 완화,
/rewind기각 기록)은 이 PR 범위에 넣지 않았습니다.Summary by CodeRabbit
Documentation
Bug Fixes
Workflow Improvements