Skip to content

docs(openrouter-images): sync with openrouter-web changes#76

Open
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1783530209-update-openrouter-images
Open

docs(openrouter-images): sync with openrouter-web changes#76
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1783530209-update-openrouter-images

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Summary

openrouter-web #27306 flipped the media_type contract on POST /api/v1/images output: it's now present whenever the format is identifiable (including image/png), and omitted only when the format couldn't be determined. Previously it was omitted for PNG and only set for vector outputs like SVG.

  • Updated the "API Response Shapes" section in SKILL.md to the new contract and added media_type to the sample response
  • Updated the stale saveImage doc comment in scripts/lib.ts (behavior unchanged — extension already followed media_type when present)

Link to Devin session: https://openrouter.devinenterprise.com/sessions/a07b68fd118d4ef09bc088001c9bbde3

…identifiable

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author
Original prompt from Automation

You are the public skills sync pipeline. Your job is to scan PRs merged to main on OpenRouterTeam/openrouter-web in the last 24 hours, determine if any changes warrant updating the public agent skills in OpenRouterTeam/skills, and open PRs to the skills repo when updates are needed.

#``# Step 1: Identify relevant merged PRs

Clone both repos:

cd /home/ubuntu/repos/openrouter-web
git fetch origin main
git log --since='24 hours ago' --merges --first-parent origin/main --pretty=format:'%H|%s' --name-only

For each merged PR, collect the list of changed files. Only PRs that touch the paths in the mapping below are relevant. Skip bot commits (devin-ai-integration[bot], Devin AI, GitHub Actions Bot, openrouter[bot], opencode-agent[bot], OpenRouter SDK Bot, Claude, detail-app[bot], forge-the-devex-intern[bot], dependabot[bot]).

If no relevant PRs were merged, post a short message to the #devrel Slack channel (C08LQFVSC7Q) saying no skill-relevant changes were detected today, and exit.

#``# Step 2: Map changed files to affected skills

Use this mapping to determine which public skills are potentially affected:

openrouter-web paths Affected skill(s) in OpenRouterTeam/skills
sdks/typescript/, packages/llm-interfaces/ openrouter-typescript-sdk, openrouter-agent-migration
services/cfw-*/src/routes/*analytics*, analytics query/response schemas openrouter-analytics, openrouter-analytics-query, openrouter-analytics-schema
Generation routes, /api/v1/generation openrouter-generations
Image generation adapters, packages/image-generation/ openrouter-images
supabase/seeds/, model schemas, /api/v1/models route openrouter-models
OAuth/auth routes, PKCE flow code openrouter-oauth
STT adapters, services/cfw-stt-api/, /api/v1/audio/transcriptions openrouter-stt
TTS routes, /api/v1/audio/speech openrouter-tts
Video adapters, /api/v1/videos openrouter-video
``@`openr... (3383 chars truncated...)

@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@perry-the-pr-maintainer perry-the-pr-maintainer Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perry's Review

Docs-only sync: updates the media_type contract description in SKILL.md and a doc comment in lib.ts to match openrouter-web#27306's contract flip (media_type now set whenever the format is identifiable, not just for non-PNG).

Verdict: ✅ LGTM

Details

Risk: 🟢 Low

CI: all checks pending/none required; no failures

Findings: none — verified against openrouter-web commit 518f0b0457 (#27306): the response schema, route handler, and media_type contract all match the new docs. saveImage() in scripts/lib.ts is functionally unchanged (doc comment only), confirmed by diffing against the pre-PR version.

Research: cross-checked openrouter-web's image-generation response schema and image-generation route handler at the referenced PR's merge commit — the new docs accurately describe the shipped contract.

Security: no concerns — docs/comments only, no code paths changed

Test coverage: n/a — no behavior change

Unresolved threads: none

Review: tier=trivial · model=claude-sonnet-latest · score=0.1

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.

0 participants