Skip to content

fix file upload endpoint#183

Merged
NianJiuZst merged 1 commit into
MiniMax-AI:mainfrom
minmax:fix/file-upload-endpoint
Jul 14, 2026
Merged

fix file upload endpoint#183
NianJiuZst merged 1 commit into
MiniMax-AI:mainfrom
minmax:fix/file-upload-endpoint

Conversation

@minmax

@minmax minmax commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Fix the MiniMax file upload endpoint used by mmx file upload and the File SDK.

The current endpoint is /v1/files, which returns 404 against the public API. The documented and working upload endpoint is /v1/files/upload.

Changes

  • Update fileUploadEndpoint() to use /v1/files/upload
  • Add endpoint coverage for the file upload URL

Verification

  • bun test test/client/endpoints.test.ts test/commands/file/upload.test.ts test/sdk/file.test.ts
  • bun run typecheck
  • bun run build

Notes

I verified the failure manually with mmx file upload, which POSTed to /v1/files and received a 404. Posting the same multipart form to /v1/files/upload succeeds and returns a file.file_id.

@NianJiuZst NianJiuZst self-assigned this Jul 14, 2026
@NianJiuZst NianJiuZst self-requested a review July 14, 2026 07:44

@NianJiuZst NianJiuZst left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@minmax Thanks for your contribution! Approve

@NianJiuZst NianJiuZst merged commit bd52360 into MiniMax-AI:main Jul 14, 2026
2 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.

2 participants