Skip to content

pref:修复自定义端口未生效问题#452

Merged
phantom5099 merged 2 commits into1024XEngineer:mainfrom
phantom5099:main
Apr 24, 2026
Merged

pref:修复自定义端口未生效问题#452
phantom5099 merged 2 commits into1024XEngineer:mainfrom
phantom5099:main

Conversation

@phantom5099
Copy link
Copy Markdown
Collaborator

发现自定义chat端口被回退成只使用默认端口,于是修复

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

Copy link
Copy Markdown

@fennoai fennoai Bot left a comment

Choose a reason for hiding this comment

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

发现 1 处需要修正的问题(其余审查项未发现值得提交的问题)。

docHeight := max(0, a.height-a.styles.doc.GetVerticalFrameSize())
if docWidth < 73 || docHeight < 36 {
if docWidth < 60 || docHeight < 30 {
return strings.TrimRight(a.styles.doc.Render(lipgloss.Place(docWidth, docHeight, lipgloss.Left, lipgloss.Top, "Window too small.\nPlease resize to at least 73x36.")), "\n")
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

docWidth/docHeight 的最小阈值已改为 60x30,但提示文案仍写 73x36。这会误导用户,建议将文案同步为 60x30,或恢复原阈值以保持一致。

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 24, 2026

Codecov Report

❌ Patch coverage is 64.70588% with 6 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/provider/openaicompat/generate_sdk.go 53.84% 3 Missing and 3 partials ⚠️

📢 Thoughts on this report? Let us know!

@phantom5099 phantom5099 merged commit baae5fc into 1024XEngineer:main Apr 24, 2026
2 of 3 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