Skip to content

fix(mcp): use documented OpenAI file schema - #186

Merged
uvwt merged 1 commit into
mainfrom
fix/openai-documented-file-schema-20261003
Oct 3, 2026
Merged

uvwt merged 1 commit into
mainfrom
fix/openai-documented-file-schema-20261003

Conversation

@uvwt

@uvwt uvwt commented Oct 3, 2026

Copy link
Copy Markdown
Owner

Summary

  • expose file_publish.file with the documented OpenAI file object fields
  • require download_url and file_id while keeping file_name and mime_type optional
  • support secure download_url ingestion with size limits, cleanup, redirect bounds, and SSRF protection
  • keep existing connector-mounted local-path compatibility
  • add regression coverage for schema, download behavior, and blocked network ranges

Validation

  • go test ./internal/tool/media ./internal/mcp ./internal/app
  • go test ./...
  • git diff --check

@uvwt
uvwt merged commit 2de2f3d into main Oct 3, 2026
10 checks passed
@uvwt
uvwt deleted the fix/openai-documented-file-schema-20261003 branch October 3, 2026 09:43
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