Skip to content

fix(provider): 避免流式错误静默结束#62

Merged
minorcell merged 3 commits into1024XEngineer:mainfrom
pionxe:provider-stream-error-stopgap
Mar 25, 2026
Merged

fix(provider): 避免流式错误静默结束#62
minorcell merged 3 commits into1024XEngineer:mainfrom
pionxe:provider-stream-error-stopgap

Conversation

@pionxe
Copy link
Copy Markdown
Collaborator

@pionxe pionxe commented Mar 25, 2026

概要

对 provider 的流式读取链路做一次临时止血修复,避免上游流中途失败时静默结束。

本次处理

  • 在流读取失败时,向当前输出流追加一段 [STREAM_ERROR] ... 错误提示
  • 在流式内容解析失败时,向当前输出流追加一段 [STREAM_ERROR] ... 错误提示
  • 保持现有接口不变,不修改 update.go 等上层逻辑,尽量减少与并行开发中的冲突

说明

这是一个临时修复方案,目标是先避免“异常被误判为正常结束”的用户体验问题。

当前方案并未引入结构化错误传播机制,后续仍建议再推进更完整的流式事件设计。


Closes #53

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@minorcell minorcell merged commit fe78b8b into 1024XEngineer:main Mar 25, 2026
2 checks passed
@pionxe pionxe deleted the provider-stream-error-stopgap branch March 28, 2026 06:57
phantom5099 added a commit that referenced this pull request Apr 24, 2026
fix(security): keep symlink escape checks for unknown walk entries
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.

fix(provider): 流式响应错误未向上层传播,异常会被误判为正常结束

2 participants