Skip to content

[codex] 修复 file 命令注册、SSE 解析和 region 校验#129

Closed
NianJiuZst wants to merge 4 commits into
MiniMax-AI:mainfrom
NianJiuZst:codex/file-cli-fixes
Closed

[codex] 修复 file 命令注册、SSE 解析和 region 校验#129
NianJiuZst wants to merge 4 commits into
MiniMax-AI:mainfrom
NianJiuZst:codex/file-cli-fixes

Conversation

@NianJiuZst
Copy link
Copy Markdown
Contributor

改动内容

  • 注册了已有的 file uploadfile listfile delete 命令,并把 file 资源补进帮助页
  • 补全了 file listfile delete 的专门测试文件
  • 修复了 SSE 流式解析的分片边界问题,避免 eventid、多行 data 在网络 chunk 中丢失
  • 增加了 region 配置校验,对无效的 --region / MINIMAX_REGION 提前报 usage error
  • 清理了 music generate 中未使用的 import,修复 lint 问题

为什么要改

  • file 相关命令之前只是实现了文件,没有接入 registry,导致用户实际上无法通过 CLI 调用
  • SSE 解析器之前依赖单个网络 chunk 内的数据完整性,真实流式响应分片后会丢字段或丢内容
  • region 配置如果传了非法值,之前会被直接当成 Region 使用,错误不会在入口处及时暴露

用户影响

  • 现在可以正常使用 mmx file upload / list / delete
  • 流式输出在真实网络环境下更稳定
  • 配置错误会更早、更明确地报出,而不是进入后续请求流程后才表现异常

验证

  • bun run typecheck
  • bun run lint
  • bun test(212 pass)
  • bun run build

@RyanLee-Dev
Copy link
Copy Markdown
Collaborator

#131

@RyanLee-Dev
Copy link
Copy Markdown
Collaborator

Merged

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.

2 participants