Skip to content

fix(ui): openrouter branding in docs (SECRETS-102)#5328

Merged
IgorHorta merged 5 commits intomainfrom
SECRETS-102
Jan 30, 2026
Merged

fix(ui): openrouter branding in docs (SECRETS-102)#5328
IgorHorta merged 5 commits intomainfrom
SECRETS-102

Conversation

@IgorHorta
Copy link
Contributor

Context

Updates documentation to use "OpenRouter" branding (no hyphen) as requested by the OpenRouter team. Doc URLs, navigation, API reference paths, image paths, and snippets now use openrouter / openrouter-api-key instead of open-router / open-router-api-key. API behavior is unchanged; backend routes remain /open-router and /open-router-api-key.

Related: SECRETS-102

Screenshots

N/A (docs only)

Steps to verify the change

  1. Run docs locally and open API Reference → App Connections → OpenRouter (and Secret Rotations → OpenRouter API Key).
  2. Confirm sidebar shows List, Available, Get by ID, Get by Name, Create, Update, Delete.
  3. Confirm integration doc at /integrations/app-connections/openrouter and secret rotation doc at /documentation/platform/secret-rotation/openrouter-api-key load and images display.
  4. Confirm snippets (AppConnectionsBrowser, RotationsBrowser) reference openrouter paths.

Type

  • Fix
  • Feature
  • Improvement
  • Breaking
  • Docs
  • Chore

Checklist

  • Title follows the conventional commit format: type(scope): short description (scope is optional, e.g., fix: prevent crash on sync or fix(api): handle null response).
  • Tested locally
  • Updated docs (if needed)
  • Read the contributing guide

@maidul98
Copy link
Collaborator

maidul98 commented Jan 30, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 30, 2026

Greptile Overview

Greptile Summary

This PR updates OpenRouter branding in documentation to remove hyphens as requested by the OpenRouter team. File paths, navigation entries, and most content references have been successfully updated from open-router to openrouter and from open-router-api-key to openrouter-api-key.

Key Changes:

  • Renamed documentation files and directories for app connections and secret rotations
  • Updated docs.json navigation paths for integration and secret rotation guides
  • Moved image assets to new openrouter and openrouter-api-key directories
  • Updated browser component slugs and paths in JSX snippets
  • Added OpenRouter group to API reference navigation for app connections

Issues Found:

  • API path mismatch: Sample curl requests in API documentation use openrouter (no hyphen) but should use open-router to match backend routes
  • Missing navigation: Secret Rotations API reference section is missing the OpenRouter API Key group, so API endpoints won't appear in the sidebar

Confidence Score: 3/5

  • This PR can be merged after fixing the API path inconsistencies and adding the missing navigation section
  • Documentation-only changes with no backend modifications. Two critical issues found: (1) API sample requests use incorrect path that won't work with backend, (2) missing navigation will hide API endpoints from docs. Both are straightforward fixes.
  • Pay close attention to docs/api-reference/endpoints/app-connections/openrouter/create.mdx, docs/integrations/app-connections/openrouter.mdx, and docs/docs.json which have incorrect paths or missing configuration

Important Files Changed

Filename Overview
docs/docs.json Updated navigation paths for OpenRouter branding, but missing Secret Rotations API reference section
docs/api-reference/endpoints/app-connections/openrouter/create.mdx API endpoint documentation has path mismatch - openapi uses open-router but sample requests use openrouter
docs/api-reference/endpoints/app-connections/openrouter/update.mdx Correctly uses open-router in openapi path (backend route remains hyphenated)
docs/integrations/app-connections/openrouter.mdx Integration guide with inconsistent API path - sample request uses openrouter instead of backend's open-router

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

4 files reviewed, 3 comments

Edit Code Review Agent Settings | Greptile

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 30, 2026

Additional Comments (2)

docs/api-reference/endpoints/app-connections/openrouter/create.mdx
API path mismatch between OpenAPI definition and sample request. The openapi field uses /api/v1/app-connections/open-router (line 3) but the sample curl request on line 25 uses /api/v1/app-connections/openrouter. According to the PR description, backend routes remain /open-router, so line 25 should use open-router with hyphen.

  --url https://app.infisical.com/api/v1/app-connections/open-router \

docs/docs.json
Missing OpenRouter API Key group in Secret Rotations API reference section. The API reference navigation includes app-connections/openrouter (lines 1558-1570) but is missing the corresponding secret-rotations/openrouter-api-key section. Should add between Okta Client Secret and OracleDB Credentials:

                  },
                  {
                    "group": "OpenRouter API Key",
                    "pages": [
                      "api-reference/endpoints/secret-rotations/openrouter-api-key/create",
                      "api-reference/endpoints/secret-rotations/openrouter-api-key/delete",
                      "api-reference/endpoints/secret-rotations/openrouter-api-key/get-by-id",
                      "api-reference/endpoints/secret-rotations/openrouter-api-key/get-by-name",
                      "api-reference/endpoints/secret-rotations/openrouter-api-key/get-generated-credentials-by-id",
                      "api-reference/endpoints/secret-rotations/openrouter-api-key/list",
                      "api-reference/endpoints/secret-rotations/openrouter-api-key/rotate-secrets",
                      "api-reference/endpoints/secret-rotations/openrouter-api-key/update"
                    ]
                  },
                  {

@IgorHorta IgorHorta merged commit 3bccdf8 into main Jan 30, 2026
7 checks passed
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.

2 participants