Skip to content

Commit b57d4e1

Browse files
tombeckenhamclaude
andcommitted
docs(ai-openrouter): correct stale SDK version in getVideoContent bypass comment
Live-verified against @openrouter/sdk 0.12.79: getVideoContent still rejects the real 'video/mp4' response ('Unexpected Status or Content-Type') even though the body is a valid MP4, so the manual authenticated download (and its injectable fetch seam) stays. Only the comment's stale 0.12.35 reference was wrong. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent b2c1f07 commit b57d4e1

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

  • packages/ai-openrouter/src/adapters

packages/ai-openrouter/src/adapters/video.ts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -367,9 +367,12 @@ export class OpenRouterVideoAdapter<
367367
// header (verified live: plain GET returns 401), so they can't be
368368
// handed to a browser <video> tag. Download the content server-side and
369369
// return a data URL instead — same policy as the OpenAI video adapter.
370-
// Fetched directly rather than via the SDK's `getVideoContent`, whose
371-
// response matcher (as of @openrouter/sdk 0.12.35) only accepts
372-
// `application/octet-stream` while the live endpoint serves `video/mp4`.
370+
// Fetched directly rather than via the SDK's `getVideoContent`: through
371+
// @openrouter/sdk 0.12.79 it still rejects the real response with
372+
// "Unexpected Status or Content-Type: Status 200 Content-Type video/mp4"
373+
// even though the body is a valid MP4 (verified live) — its response
374+
// matcher doesn't accept `video/mp4`. Revisit if a future SDK build fixes
375+
// this; the manual download keeps working in the meantime.
373376
const doFetch = this.clientConfig.fetch ?? globalThis.fetch
374377
const contentResponse = await doFetch(contentUrl, {
375378
headers: { Authorization: `Bearer ${this.clientConfig.apiKey}` },

0 commit comments

Comments
 (0)