Skip to content

fix: remove WorkBuddy non-stream chat timeout - #130

Merged
caigee-cmd merged 1 commit into
mainfrom
codex/freeze-changelog-v0.3.7
Sep 10, 2026
Merged

fix: remove WorkBuddy non-stream chat timeout#130
caigee-cmd merged 1 commit into
mainfrom
codex/freeze-changelog-v0.3.7

Conversation

@caigee-cmd

Copy link
Copy Markdown
Owner

Summary

  • Remove the inherited 120-second HTTP client timeout for WorkBuddy non-streaming SSE aggregation; request context still controls cancellation.
  • Preserve body read errors instead of masking them as missing [DONE].
  • Add regression coverage for long responses, cancellation, truncated bodies, and shared client isolation.
  • Freeze existing v0.4.2 notes and add bilingual release notes.

Validation

  • go test ./internal/providers/workbuddy -run TestChatNonStreamReadLifecycle -count=5
  • go test ./...
  • python3 scripts/release-notes.py validate
  • git diff --check

@caigee-cmd
caigee-cmd merged commit a86d6ec into main Sep 10, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant