Skip to content

fix(coralbricks): drop PDF input from GLM 5.3 Flash - #7086

Merged
rekram1-node merged 1 commit into
anomalyco:devfrom
divyvasal:fix-coralbricks-flash-modalities
Sep 14, 2026
Merged

rekram1-node merged 1 commit into
anomalyco:devfrom
divyvasal:fix-coralbricks-flash-modalities

Conversation

@divyvasal

Copy link
Copy Markdown
Contributor

#6795 added coralbricks/glm-5.3-flash-fp4 on top of the zhipuai/glm-5.3-flash baseline, which lists input = ["text", "image", "video", "pdf"]. The CoralBricks endpoint serves text, image and video for this model but not PDF, so this overrides [modalities] to match what it accepts.

Checked against the production endpoint on 2026-09-14:

input request shape result
image Chat Completions image_url 200, correct answer
video Chat Completions video_url 200, correct answer
PDF Chat Completions file part 400
PDF Messages document block 400
PDF Responses input_file 200, but the file is not passed to the model

bun run validate passes. bun test shows the same 7 failures as a clean dev checkout, none touching this file.

cc @rekram1-node

🤖 Generated with Claude Code

The zhipuai/glm-5.3-flash baseline lists PDF input, which the
CoralBricks endpoint does not accept for this model. Override
[modalities] to the text, image and video input it serves.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

No actionable findings.

@github-actions github-actions Bot added the reviewer: ready Automated review found no actionable items label Sep 14, 2026
@divyvasal

Copy link
Copy Markdown
Contributor Author

Update on the table above: the Responses input_file row now returns 400 as well. The gateway was silently dropping the part; that's fixed on the production endpoint as of today, so PDF input is rejected on every wire. No change to this PR.

@rekram1-node
rekram1-node merged commit 5c21a5d into anomalyco:dev Sep 14, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

reviewer: ready Automated review found no actionable items

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants