fix(session-recovery): handle empty content from interrupted reasoning #6
Conversation
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
- Check for staged changes before commit - Check if tag exists before creating - Check if release exists before creating
- Add THINKING_TYPES set including 'reasoning' type (OpenCode's thinking) - Add hasNonEmptyOutput() to detect messages with only thinking/meta parts - Add findEmptyContentMessage() to scan all messages for empty content - Handle step-start/step-finish meta parts in empty content detection - Patch interrupted messages with '(interrupted)' text before falling back to revert
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
reasoning타입과step-start/step-finish메타 파트를 인식하도록 개선Problem
API 에러:
messages.53: all messages must have non-empty content except for the optional final assistant message원인: assistant가 thinking을 뱉다가 중단되면
reasoning파트만 남고 실제text파트가 없어서 API가 거부Changes
THINKING_TYPESset에reasoning타입 추가hasNonEmptyOutput():step-start/step-finish메타 파트 무시findEmptyContentMessage(): 전체 메시지 스캔하여 빈 content 찾기"(interrupted)"텍스트로 패치 시도 후 revert로 폴백