Skip to content

Use the deployed OpenAPI spec for API reference - #496

Merged
sjmiller609 merged 1 commit into
mainfrom
hypeship/use-deployed-openapi
Aug 10, 2026
Merged

Use the deployed OpenAPI spec for API reference#496
sjmiller609 merged 1 commit into
mainfrom
hypeship/use-deployed-openapi

Conversation

@sjmiller609

@sjmiller609 sjmiller609 commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • point the API reference at the deployed https://api.onkernel.com/spec.json
  • remove the unused checked-in generated spec

Why

The docs can read the production API's OpenAPI document directly. This removes the hosted Stainless dependency and avoids maintaining a second generated copy of the same API surface.

Companion change

kernel/kernel#3253 removes the workflow and generation target that maintained the checked-in spec.

Testing

  • parsed docs.json with jq
  • fetched the configured endpoint and validated its OpenAPI structure (98 paths, 194 schemas)
  • local Mintlify preview not run because the CLI is unavailable; CI produced a successful Mintlify deployment

Note

Low Risk
Single Mintlify config URL change for documentation only; no runtime or API behavior changes.

Overview
The API Reference tab in docs.json now pulls OpenAPI from the production endpoint https://api.onkernel.com/spec.json instead of https://app.stainless.com/api/spec/documented/kernel/openapi.documented.yml.

Docs stay aligned with the live API surface without a hosted Stainless spec or a second generated copy in this repo (maintenance removed in the companion kernel PR).

Reviewed by Cursor Bugbot for commit aa9cfb6. Bugbot is set up for automated code reviews on this repo. Configure here.

@mintlify

mintlify Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
Kernel 🟢 Ready View Preview Aug 10, 2026, 5:33 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@vercel

vercel Bot commented Aug 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Error Error Aug 10, 2026 5:33pm

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Risk assessment: Low

What changed (from the diff)

  • docs.json: API Reference OpenAPI source updated from the Stainless hosted documented YAML to https://api.onkernel.com/spec.json
  • specs/openapi.documented.yml: deleted (large generated/checked-in artifact no longer referenced)

Why this is Low risk

  • Scope is limited to Mintlify docs configuration and removal of an unused local OpenAPI file
  • No application runtime, auth, billing, permissions, or infrastructure code paths are modified
  • Blast radius is the public API Reference docs tab content/source only; correctness is easy to reason about
  • Independent checks: production spec.json is reachable and parses as OpenAPI 3.1 with a comparable path surface to the removed YAML; Mintlify Deployment already succeeded on this PR

Residual notes (non-blocking)

  • Docs rendering now depends on availability/correctness of the production OpenAPI endpoint
  • API Reference content may drift if the production spec differs from the former Stainless "documented" view

Decision: Low risk — approving. No CODEOWNERS file found; PR had no prior approvals from this automation.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

@masnwilliams masnwilliams 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.

lgtm

@sjmiller609
sjmiller609 merged commit 964946a into main Aug 10, 2026
6 of 7 checks passed
@sjmiller609
sjmiller609 deleted the hypeship/use-deployed-openapi branch August 10, 2026 17:45
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