Skip to content

fix: raise inline upload limit to 10MB - #93

Merged
ChiragAgg5k merged 1 commit into
mainfrom
fix/raise-inline-upload-limit
Aug 5, 2026
Merged

fix: raise inline upload limit to 10MB#93
ChiragAgg5k merged 1 commit into
mainfrom
fix/raise-inline-upload-limit

Conversation

@eldadfux

@eldadfux eldadfux commented Aug 5, 2026

Copy link
Copy Markdown
Member

Summary

  • Raise MAX_INLINE_BYTES from 256KB to 10MB so hosted MCP can accept chat attachment bytes rewritten as inline base64 by the agent
  • Aligns with Cloud/agent chat attachment max (10MB)

Companion: appwrite/agent#1

Test plan

  • Upload a ~500KB–2MB file via hosted MCP with {filename, content, encoding: base64}
  • Confirm files under 10MB succeed and over 10MB still fail with the size error

Match Cloud/agent chat attachment max so hosted MCP can accept
attachment bytes rewritten as inline base64 by the agent.

Co-authored-by: Cursor <cursoragent@cursor.com>
@greptile-apps

greptile-apps Bot commented Aug 5, 2026

Copy link
Copy Markdown

Greptile Summary

Raises the decoded inline-upload limit from 256 KiB to 10 MiB to support hosted chat attachments.

  • Updates MAX_INLINE_BYTES to match the Cloud/agent attachment limit.
  • Documents why hosted uploads require the larger inline allowance.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete blocking or non-blocking issue identified in the changed code.

The changed constant is consumed after base64 decoding, so it directly raises the intended decoded-file limit, and the repository’s HTTP middleware replays large request bodies rather than truncating them at its sniff threshold.

Important Files Changed

Filename Overview
src/mcp_server_appwrite/constants.py The inline decoded-content cap is increased to 10 MiB with explanatory comments; no concrete defect was identified.

Reviews (1): Last reviewed commit: "fix: raise inline upload limit to 10MB" | Re-trigger Greptile

@ChiragAgg5k
ChiragAgg5k merged commit eb9d902 into main Aug 5, 2026
5 checks passed
@ChiragAgg5k
ChiragAgg5k deleted the fix/raise-inline-upload-limit branch August 5, 2026 12:04
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