Skip to content

docs(openrouter-oauth): sync with openrouter-web changes#74

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

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

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Summary

openrouter.ai/auth now accepts a key_label query param that seeds the API key label shown on the consent page (OpenRouterTeam/openrouter-web#27250).

  • Added key_label to the Step 2 redirect-parameter table (optional; user can still edit; defaults to the app title)

Source: OpenRouterTeam/openrouter-web#27250

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

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

Documents the key_label query param on the OAuth /auth redirect (Step 2 of the PKCE flow), matching the param OpenRouter's web app now accepts.

Verdict: ✅ LGTM

Details

Risk: 🟢 Low

CI: all passing ✅ (only the in-progress review check pending)

Findings: none

Research: verified against openrouter-web's auth route — key_label is accepted as an optional query param, trimmed, and used to seed the editable key-label field (falling back to the app title when omitted), matching this doc's description exactly.

Security: no concerns — docs-only change, no code path affected.

Test coverage: n/a (documentation-only change).

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