Skip to content
This repository was archived by the owner on May 13, 2026. It is now read-only.

Merge pull request #368 from CJackHwang/codex/fix-review-issues-for-pr-#364 Restore thinking fallback for tool-call detection and drop history.txt wrapper tags#369

Merged
CJackHwang merged 33 commits into
mainfrom
dev
Apr 29, 2026

Conversation

@CJackHwang

Copy link
Copy Markdown
Owner

💻 变更类型 | Change Type

  • ✨ feat
  • 🐛 fix
  • ♻️ refactor
  • 💄 style
  • 👷 build
  • ⚡️ perf
  • 📝 docs
  • 🔨 chore

🔀 变更说明 | Description of Change

优化工具调用
识图模式上线

📝 补充信息 | Additional Information

RinZ27 and others added 30 commits April 28, 2026 23:49
build: improve Docker robustness and fix potential security issues
feat: parse split context files in list view
Both READMEs are 400+ lines with 14 top-level sections and multiple
subsections but have no navigation aid. Add a Table of Contents at the
top of each file to help readers quickly find relevant sections.

Changes:
- README.MD: add 目录 section with links to all h2/h3 headings
- README.en.md: add Table of Contents with matching structure
docs: add Table of Contents to README.MD and README.en.md
…-automation-scripts

Verify GHCR latest tag matches release and show version source/latest in dashboard
…r-config.json-permissions

Return config persistence warning when config path is read-only; default container config to /data/config.json and update docs
…n-update-in-automation-scripts

Revert "Verify GHCR latest tag matches release and show version source/latest in dashboard"
fix(openai): keep citation indexes one-based with zero-based references
Fix stream compatibility and vision model exposure
…back

fix(sse): batch tiny stream chunks before emitting
…r-#364

Restore thinking fallback for tool-call detection and drop history.txt wrapper tags
@vercel

vercel Bot commented Apr 29, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ds2api Ready Ready Preview, Comment Apr 29, 2026 3:40pm

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0438ce9a12

ℹ️ 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".

Comment thread internal/deepseek/client/client_continue.go Outdated
Comment thread internal/config/paths.go Outdated
@CJackHwang CJackHwang merged commit 94c1aca into main Apr 29, 2026
14 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request] 适配视觉模式 [Bug] Bug Report: SSE 流式输出 token 分块过细,导致客户端超时

6 participants