79:T90a,## What POST /v1/systemone bodies on the dgemma door (model "dgemma" named, or images present) that also carry `audio`, `audios`, `video` or `videos` are now refused with 400 `dgemma_input` and a message naming the medium. The developer guide, the OpenAPI description and request schema, and AGENTS.md now state that images are dgemma's only media and why. ## Why The pod's structured-read interposer silently drops keys it does not read. Verified against the live pod through production: `audio: 42` and `video: [wav data URL]` both answered HTTP 200 with the text-only input token count — a confident answer about media the model never looked at, with no signal to the caller. The capability reality, checked at each layer: | Layer | image | video | audio | |---|---|---|---| | DiffusionGemma 26B-A4B in vLLM (`diffusion_gemma.py`) | yes | yes | **no — no audio input path** | | Pod interposer (`structured_server.py`, vllm-project/vllm#57250) | `images` only | dropped | dropped | | Beam hosted `jev/diffusiongemma` | 1 image max | 422 | 422 | | classifier.dev gateway (before this PR) | validated | forwarded + silently ignored | forwarded + silently ignored | Audio can never work on this model. Video can — the model declares the video modality and vllm-project/vllm#57589 fixed its multimodal path — but enabling it is a pod-side interposer change (build `video_url` parts), not a Worker one; until then refusing is the honest answer. ## Tests New unit cases: all four media keys refused on both doors with nothing called upstream; audio without images and without naming dgemma still goes to TypeSafe to validate. `npm run typecheck` clean, `bun test` 823 pass / 0 fail. 🤖 Generated with [Claude Code](https://claude.com/claude-code) ## Summary by CodeRabbit * **Behavior Changes** * DiffusionGemma requests containing audio or video are now rejected with a `400 dgemma_input` response instead of being silently ignored. Image inputs remain supported; audio-only requests routed to other models are unaffected. * **Documentation** * Updated API documentation to clarify audio and video support and the conditions for these refusals. 7a:T24fe,

Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

**Configuration used**: defaults **Review profile**: CHILL **Plan**: Advanced **Run ID**: `9089a71c-f78b-45f2-a6bd-53e8b532cb46`

📥 Commits

Reviewing files that changed from the base of the PR and between 43818f818af44b5db6ea81ec9489f427138a35f6 and 34e829b45314d305ee30659e1210ead3acf434e5.

📒 Files selected for processing (5)

* `AGENTS.md` * `src/dgemma.ts` * `src/docs.ts` * `src/openapi.ts` * `test/dgemma.test.ts`

**Included review availability:** This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.

***

📝 Walkthrough

## Walkthrough The DiffusionGemma route now refuses requests with non\-null audio or video fields. Tests cover these refusals and confirm that an audio\-only request without `dgemma` or images is routed to TypeSafe. Documentation describes the supported media and refusal responses. ### Changes **DiffusionGemma media input** | Layer / File(s) | Summary | | ---------------------------------------------------------------------------------------------------------------------------------------------- | --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | | **Route validation, tests, and media documentation**
`src/dgemma.ts`, `test/dgemma.test.ts`, `AGENTS.md`, `src/docs.ts`, `src/openapi.ts` | The route returns `dgemma_input` for non\-null `audio`, `audios`, `video`, or `videos` fields. Tests cover rejection and TypeSafe routing. Documentation states that images are the only supported DiffusionGemma media and describes audio and video refusals. | **Priority:** ⬇️ Low **Estimated code review effort:** 2 (Simple) \| \~10 minutes **Change:** Bug fix **Suggested reviewers:** `myxamediyar`

**Merge Risk:** *⚪ Minimal* · up to `34e82` The change rejects unsupported media for DiffusionGemma while preserving the described TypeSafe routing path; no merge\-blocking risk is identified. ### Security Architecture Review **Security architecture risk:** *🔵 Low* · up to `34e82` The new refusal reduces the chance of a successful\-looking answer to media the image\-capable service did not process. It changes a public API response, but the reviewed path does not show expanded access or privileges. **Retained concerns** No architecture\-level concerns identified.

Security review details

**Security Blast Radius** - *inferred* — The changed control limits unsupported\-media handling at the public dgemma request boundary; reviewed code does not show a new credential, privilege, or provider\-reachability path. **Trust Boundaries and Controls** - *observed* — For the selected dgemma route, attacker\-supplied non\-null audio or video fields trigger a refusal before model\-provider dispatch. The guard does not claim ownership of plain TypeSafe requests.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

| Check name | Status | Explanation | |:--------------------------:|:-------- |:--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | | Description Check | ✅ Passed | Check skipped \- CodeRabbit’s high\-level summary is enabled. | | Title check | ✅ Passed | The title clearly and concisely summarizes the main change: dgemma now refuses audio and video instead of silently dropping them. | | Docstring Coverage | ✅ Passed | Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 4 files. (1 skipped: 1 … | | Linked Issues check | ✅ Passed | Check skipped because no linked issues were found for this pull request. | | Out of Scope Changes check | ✅ Passed | Check skipped because no linked issues were found for this pull request. |

✨ Finishing Touches

📝 Generate docstrings

- [ ] Commit to this branch - [ ] Create a new PR

🧪 Generate unit tests (beta)

- [ ] Commit to this branch - [ ] Create a new PR

*** Thanks for using [CodeRabbit](https://coderabbit.ai?utm_source=oss&utm_medium=github&utm_campaign=mrmps/classifier-dev&utm_content=138)! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout\-out.

❤️ Share

- [x] () - [Mastodon](https://mastodon.social/share?text=I%20just%20used%20%40coderabbitai%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20the%20proprietary%20code.%20Check%20it%20out%3A%20https%3A%2F%2Fcoderabbit.ai) - [Reddit](https://www.reddit.com/submit?title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&text=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code.%20Check%20it%20out%3A%20https%3A//coderabbit.ai) - [LinkedIn](https://www.linkedin.com/sharing/share-offsite/?url=https%3A%2F%2Fcoderabbit.ai&mini=true&title=Great%20tool%20for%20code%20review%20-%20CodeRabbit&summary=I%20just%20used%20CodeRabbit%20for%20my%20code%20review%2C%20and%20it%27s%20fantastic%21%20It%27s%20free%20for%20OSS%20and%20offers%20a%20free%20trial%20for%20proprietary%20code)

_{Comment `@coderabbitai help` to get the list of available commands.} 63:["$","$L76",null,{"entries":[{"key":["platform",{"userId":"anon"},"pulls","page",{"owner":"mrmps","repo":"classifier-dev","pullNumber":138}],"data":{"repositoryId":"4395","detail":{"author":{"avatarUrl":"https://cdn.trylle.com/media/images/ed7982acbfeb80f61cea463bd42d4f1f?key=AdIPzhx%2Bz7JEI3YrvAKRBYhoVRsCPK%2Bq4fYv6QcjJAndwHy8bYmoDB89zhgboDuds7RkIeXfWwqLQvoHbbasZiBpL4Bo8y1fd2bsRFiAd6PTa6qCkLCPuTPiBL2lTrABBG37CCosYXE5IH6suxuUtU1tkDcIhPsiWFqhgwBgozepr%2Fbs3xVwFEDesw%3D%3D","externalProvider":"github","login":"mrmps","type":"User","url":"https://github.com/mrmps"},"authorId":"41216139","baseRefName":"main","closedAt":null,"comments":1,"createdAt":"2026-09-27T06:35:41Z","headRefName":"claude/dgemma-multimodal","id":"4903034","isDraft":false,"labels":[],"number":138,"repository":{"name":"classifier-dev","owner":"mrmps","provider":"forgejo"},"state":"open","title":"Refuse audio and video at the dgemma door instead of silently dropping them","updatedAt":"2026-09-27T06:39:05Z","url":"https://trylle.com/mrmps/classifier-dev/pulls/138","body":"$79","additions":54,"deletions":3,"changedFiles":5,"commits":1,"reviewComments":0,"headSha":"34e829b45314d305ee30659e1210ead3acf434e5","baseSha":"b0e3b449a9c438a1fc31de2d50a8f1546b7d7c49","headRepoOwner":"mrmps","headRepoName":"classifier-dev","isMerged":false,"mergeCommitSha":null,"mergedBy":null,"mergeable":false,"mergeableState":"dirty","conflictingFiles":["AGENTS.md","src/dgemma.ts","src/docs.ts","src/openapi.ts","test/dgemma.test.ts"],"requestedTeams":[],"requestedReviewers":[],"graphqlId":"issue:138"},"comments":[{"id":26671801,"graphqlId":"26671801","body":"$7a","createdAt":"2026-09-27T06:36:03Z","author":{"avatarUrl":"https://cdn.trylle.com/media/images/d39e0f17743c08d7bf625ba6f80c545a?key=Ac3dCIMfeEeTRToXUBeRdRfe7IGnH7FaknGcutnXETLNIDauVDMEeYfL%2BNJ1UW6PltLPYQ8AAbXVYTqKXZ2sbxmdR%2FJeAINpLPz%2Fgbhi3if42haJXKHU1rlVIjj4bCQzj6%2FWYhkNT360oU7wzsBjYZsV7FyaaYuuIlrb3O4TqxAm%2FsfmH46nFvfFacg%3D","externalProvider":"github","login":"coderabbitai[bot]","type":"Bot","url":"https://github.com/apps/coderabbitai"}}],"commits":[{"sha":"34e829b45314d305ee30659e1210ead3acf434e5","message":"Refuse audio and video at the dgemma door instead of silently dropping them\n\nThe pod's structured-read interposer ignores body keys it does not read,\nso a System One body that named model \"dgemma\" (or carried images) and\nalso carried audio or video came back as a confident answer about media\nthe model never saw — verified against the live pod, where audio: 42 and\nvideo: 42 both answered 200 with the text-only token count. DiffusionGemma\nhas no audio input at all, and its video path (vllm-project/vllm#57589) is\nnot served by the interposer yet, so the honest answer at the door is a\n400 dgemma_input naming the unsupported medium.\n\nThe developer guide, OpenAPI description and request schema now say that\nimages are dgemma's only media, and AGENTS.md records why: enabling video\nis a pod-side interposer change, not a Worker one.\n\nCo-Authored-By: Claude Fable 5 \n","createdAt":"2026-09-26T23:35:19-07:00","author":{"avatarUrl":"","login":"","name":"Michael Ryaboy","type":"User","url":""},"verification":{"verified":false,"reason":"gpg.error.not_signed_commit"}}],"events":[],"reviews":[],"stackEvents":[],"commentPagination":{"totalCount":1,"perPage":30,"loadedPages":[1]},"eventPagination":{"loadedPages":[1],"hasMore":false},"headRefDeleted":false},"fetchedAt":1794408667684}],"children":"$L7b"}] 65:null 6c:[["$","title","0",{"children":"Commits · Refuse audio and video at the dgemma door instead of silently dropping them · mrmps/classifier-dev | Trylle"}],["$","meta","1",{"name":"description","content":"Commits on Refuse audio and video at the dgemma door instead of silently dropping them (#138) in mrmps/classifier-dev."}],["$","meta","2",{"name":"application-name","content":"Trylle"}],["$","link","3",{"rel":"manifest","href":"/manifest.json","crossOrigin":"$undefined"}],["$","meta","4",{"name":"keywords","content":"Commits · Refuse audio and video at the dgemma door instead of silently dropping them · mrmps/classifier-dev, mrmps/classifier-dev, mrmps/classifier-dev repository, classifier-dev, TypeScript repository, source code, trylle, agent-native git platform, git platform for agents and humans, git platform for ai agents, coding agent pull requests, ai code review, repository automation, stacked pull requests, two-way github sync, github-compatible actions, try cli"}],["$","meta","5",{"name":"robots","content":"index, follow, max-image-preview:large, max-snippet:-1, max-video-preview:-1"}],["$","meta","6",{"name":"mobile-web-app-capable","content":"yes"}],["$","meta","7",{"name":"googlebot","content":"index, follow, max-image-preview:large, max-snippet:-1, max-video-preview:-1"}],["$","meta","8",{"name":"twitter:domain","content":"trylle.com"}],["$","meta","9",{"name":"twitter:url","content":"https://trylle.com/mrmps/classifier-dev/pulls/138/commits"}],["$","link","10",{"rel":"canonical","href":"https://trylle.com/mrmps/classifier-dev/pulls/138/commits"}],["$","meta","11",{"name":"format-detection","content":"telephone=no"}],["$","meta","12",{"name":"mobile-web-app-capable","content":"yes"}],["$","meta","13",{"name":"apple-mobile-web-app-title","content":"Trylle"}],["$","meta","14",{"name":"apple-mobile-web-app-status-bar-style","content":"black-translucent"}],["$","meta","15",{"property":"og:title","content":"Commits · Refuse audio and video at the dgemma door instead of silently dropping them · mrmps/classifier-dev | Trylle"}],["$","meta","16",{"property":"og:description","content":"Commits on Refuse audio and video at the dgemma door instead of silently dropping them (#138) in mrmps/classifier-dev."}],["$","meta","17",{"property":"og:url","content":"https://trylle.com/mrmps/classifier-dev/pulls/138/commits"}],["$","meta","18",{"property":"og:site_name","content":"Trylle"}],["$","meta","19",{"property":"og:locale","content":"en_US"}],["$","meta","20",{"property":"og:image","content":"https://trylle.com/api/og.png?kind=repo&owner=mrmps&repo=classifier-dev"}],["$","meta","21",{"property":"og:image:secure_url","content":"https://trylle.com/api/og.png?kind=repo&owner=mrmps&repo=classifier-dev"}],["$","meta","22",{"property":"og:image:type","content":"image/png"}],["$","meta","23",{"property":"og:image:width","content":"1200"}],["$","meta","24",{"property":"og:image:height","content":"630"}],["$","meta","25",{"property":"og:image:alt","content":"mrmps/classifier-dev repository preview on Trylle"}],["$","meta","26",{"property":"og:type","content":"website"}],["$","meta","27",{"name":"twitter:card","content":"summary_large_image"}],["$","meta","28",{"name":"twitter:site","content":"@trylle"}],["$","meta","29",{"name":"twitter:title","content":"Commits · Refuse audio and video at the dgemma door instead of silently dropping them · mrmps/classifier-dev | Trylle"}],["$","meta","30",{"name":"twitter:description","content":"Commits on Refuse audio and video at the dgemma door instead of silently dropping them (#138) in mrmps/classifier-dev."}],["$","meta","31",{"name":"twitter:image","content":"https://trylle.com/api/og.png?kind=repo&owner=mrmps&repo=classifier-dev"}],["$","meta","32",{"name":"twitter:image:alt","content":"mrmps/classifier-dev repository preview on Trylle"}],["$","link","33",{"rel":"icon","href":"/favicon.png","type":"image/png"}],"$L7c","$L7d","$L7e"]

Commits on Sun, Sep 27, 2026