Merge pull request #171 from CJackHwang/codex/fix-issue-#170-in-ds2api Enable env-backed config writeback and bootstrap missing config file#174
Conversation
Enable env-backed config writeback and bootstrap missing config file
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: efebe9ebad
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
internal/sse/content_filter_leak.go
Outdated
| idx := strings.Index(strings.ToUpper(text), "CONTENT_FILTER") | ||
| if idx < 0 { | ||
| return text | ||
| } | ||
| return strings.TrimRight(text[:idx], " \t\r\n") |
There was a problem hiding this comment.
This truncates output at the first CONTENT_FILTER substring anywhere in the chunk, so legitimate responses that mention that token (for example, explanations of moderation behavior) are silently cut or dropped. Because the filter runs on every parsed part, it can corrupt normal user-visible content; the trim should be gated to a stricter leak signature instead of an unconditional first-match.
Useful? React with 👍 / 👎.
…nfig-on-json-parse-failure-u32uz8 修复写回模式回退逻辑并从 CONTENT_FILTER 起截断流输出
1 participant
💻 变更类型 | Change Type
🔀 变更说明 | Description of Change
流截断残余数据过滤
环境变量配置模式优化等
📝 补充信息 | Additional Information