Skip to content

Support OpenAPI server variables during onboarding#246

Merged
RhysSullivan merged 1 commit intomainfrom
openapi-server-variables
Apr 14, 2026
Merged

Support OpenAPI server variables during onboarding#246
RhysSullivan merged 1 commit intomainfrom
openapi-server-variables

Conversation

@RhysSullivan
Copy link
Copy Markdown
Owner

Summary

  • Preserve enum and description on server variables through extraction + preview (previously we flattened to just the default).
  • Rework the Add OpenAPI Source UI to render per-variable pickers (enum → select, free-form → input) and submit a resolved base URL instead of a templated one like https://{region}.sentry.io.
  • Tighten SpecPreview.servers from Schema.Array(Schema.Unknown) to Schema.Array(ServerInfo) and export ServerVariable + substituteUrlVariables from the SDK.

Why

Specs like Sentry's declare server variables with enums:

servers:
  - url: https://{region}.sentry.io
    variables:
      region:
        default: us
        enum: [us, de]

Previously the UI stored the literal https://{region}.sentry.io as the base URL, which broke requests. Users also had no way to pick de over us.

Test plan

  • bunx vitest run in packages/plugins/openapi (27 passing, incl. new extraction test for Sentry-style {region} with enum + description)
  • bunx turbo run typecheck across all 28 packages
  • Manual: onboard a Sentry-style spec, verify the region picker appears and the resolved base URL is stored

Preserve enum and description metadata on server variables through
extraction and preview so the add-source UI can render per-variable
pickers (e.g. Sentry's {region}) and submit a fully resolved base URL
instead of a templated one.
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
🔵 In progress
View logs
executor-cloud 875f341 Apr 14 2026, 09:15 PM

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Apr 14, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
executor-marketing 875f341 Commit Preview URL

Branch Preview URL
Apr 14 2026, 09:16 PM

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 14, 2026

Open in StackBlitz

@executor/sdk

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/sdk@246

@executor/plugin-file-secrets

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-file-secrets@246

@executor/plugin-google-discovery

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-google-discovery@246

@executor/plugin-graphql

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-graphql@246

@executor/plugin-keychain

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-keychain@246

@executor/plugin-mcp

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-mcp@246

@executor/plugin-onepassword

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-onepassword@246

@executor/plugin-openapi

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-openapi@246

@executor/plugin-workos-vault

npm i https://pkg.pr.new/RhysSullivan/executor/@executor/plugin-workos-vault@246

commit: 875f341

@RhysSullivan RhysSullivan merged commit 12dd581 into main Apr 14, 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.

1 participant