Skip to content

docs(openrouter-typescript-sdk): sync with openrouter-web changes#68

Open
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1782666182-update-openrouter-typescript-sdk
Open

docs(openrouter-typescript-sdk): sync with openrouter-web changes#68
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1782666182-update-openrouter-typescript-sdk

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Summary

openrouter-web #26432 added a fourth image detail level, original, as an OpenRouter extension to the image_url.detail field (it requests true original-resolution media and downgrades to high for providers without an original-resolution tier). The change touched packages/llm-interfaces/ (new image-detail-levels.ts enum) and the public OpenAPI spec (projects/docs/fern/openapi/openapi.yaml), so the documented enum surface is now stale in the SDK skill.

The openrouter-typescript-sdk SKILL.md documented the multimodal input_image type with only the old three-value enum:

- | { type: 'input_image'; imageUrl: string; detail?: 'auto' | 'low' | 'high' }
+ | { type: 'input_image'; imageUrl: string; detail?: 'auto' | 'low' | 'high' | 'original' }

Also added a one-line note explaining what detail controls and that original is an OpenRouter extension with a high fallback.

No other public-facing skills documented the detail enum, so this is the only skill affected.

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

… level

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

Adds the original value to the detail enum in the MultimodalMessage type, syncing the SDK skill with the openrouter-web enum expansion, and adds a one-sentence note explaining the value's semantics and high-fallback behavior.

Verdict: ✅ LGTM

Details

Risk: 🟢 Low — documentation-only change to SKILL.md

CI: perry/review in progress (only check) ✅

Findings: none

Codex (HEAVY_SECONDARY_MODEL): skipped (trivial tier)

Research: skipped (trivial tier)

Security: no concerns

Test coverage: N/A — documentation-only PR

Unresolved threads: none

Scope: first review (full)

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

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