Skip to content

Add API key management docs#385

Merged
IlyaasK merged 5 commits into
mainfrom
codex/api-key-management-docs
May 29, 2026
Merged

Add API key management docs#385
IlyaasK merged 5 commits into
mainfrom
codex/api-key-management-docs

Conversation

@IlyaasK
Copy link
Copy Markdown
Contributor

@IlyaasK IlyaasK commented May 28, 2026

Summary

  • Add a Guides page for creating, scoping, inspecting, renaming, deleting, and rotating API keys through the SDKs
  • Add CLI reference docs for kernel api-keys
  • Move API Keys into the Intermediate Guides sidebar and wire API Keys into CLI navigation
  • Update Projects and CLI auth docs to point to the API key docs

Validation

  • python3 -m json.tool docs.json
  • git diff --check
  • Content check: removed CLI, REST, and SDK-version guidance from info/api-keys.mdx

Note

Low Risk
Documentation-only changes with no runtime, auth, or API behavior modifications.

Overview
Adds API key management documentation across Guides and CLI reference, with navigation wired so discoverability matches Projects and auth flows.

A new Intermediate guide at info/api-keys.mdx documents SDK workflows for creating, listing, updating, and deleting keys; org vs project scope via project_id; one-time plaintext key handling; rotation steps; and a troubleshooting table. A new reference/cli/api-keys.mdx documents kernel api-keys create/list/get/update/delete, flags (including --project-id vs global --project), aliases, and JSON output.

docs.json adds info/api-keys under Intermediate Guides and reference/cli/api-keys on the CLI tab. info/projects.mdx expands how to create project-scoped keys (dashboard, CLI, SDK, guide). reference/cli.mdx adds an API Keys card; reference/cli/auth.mdx links to CLI api-keys and uses a concrete KERNEL_API_KEY example.

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

@firetiger-agent
Copy link
Copy Markdown

Firetiger deploy monitoring skipped

This PR didn't match the auto-monitor filter configured on your GitHub connection:

Any PR that changes the kernel API. Monitor changes to API endpoints (packages/api/cmd/api/) and Temporal workflows (packages/api/lib/temporal) in the kernel repo

Reason: PR only modifies documentation files (guides, CLI reference, sidebar navigation) with no changes to API endpoints or Temporal workflows.

To monitor this PR anyway, reply with @firetiger monitor this.

@mintlify
Copy link
Copy Markdown
Contributor

mintlify Bot commented May 28, 2026

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

Project Status Preview Updated (UTC)
Kernel 🟢 Ready View Preview May 28, 2026, 7:15 PM

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

Comment thread info/api-keys.mdx Outdated
Copy link
Copy Markdown
Contributor

@ehfeng ehfeng left a comment

Choose a reason for hiding this comment

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

Should this belong in "Basics"? It feels more Intermediate, as most people won't need to read these docs in order to get Kernel working for them. This feels more "Kernel is deployed, here's how you managed keys"

Comment thread info/api-keys.mdx Outdated
Comment thread info/api-keys.mdx Outdated
4. Verify the workload can call Kernel.
5. Delete the old API key.

This keeps downtime low because the old key continues working until the new key is active everywhere.
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.

Unnecessary

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

removed

@IlyaasK IlyaasK requested a review from juecd May 29, 2026 15:15
Copy link
Copy Markdown
Contributor

@juecd juecd left a comment

Choose a reason for hiding this comment

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

Thanks! Agree w/ Eric's changes, and I left a couple too.

Comment thread info/api-keys.mdx Outdated

Use the CLI when you're creating a key from your terminal. Use the SDK or REST API when your backend needs to provision keys for environments, customers, or automation jobs.

### CLI
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.

IMO we don't need a dedicated CLI section, we don't do this for any other guides. Just remove this entirely and only describe the SDK methods

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

removed

Comment thread info/api-keys.mdx Outdated
```
</CodeGroup>

### REST API
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.

Remove the REST API section, only keep SDKs

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

removed

Comment thread info/api-keys.mdx Outdated

- Use `kernel login` for CLI-based creation.
- Use `KERNEL_API_KEY` for SDK or REST-based creation.
- Use `@onkernel/sdk>=0.58.0` or `kernel>=0.58.0` for SDK API key methods.
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.

Remove this - we never mention specific versions in our guides.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

removed

Co-authored-by: Eric Feng <ehfeng@users.noreply.github.com>
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Risk assessment: Very Low

Evidence from the diff: this PR only adds/updates documentation pages and Mintlify navigation entries (docs.json, info/*.mdx, and reference/cli*.mdx). It doesn't modify executable code, auth/permission enforcement, infrastructure, schemas, or shared runtime behavior. I found no CODEOWNERS file in the repo and no existing approval on the PR, so approving under the Very Low risk rule.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 3eca23f. Configure here.

Comment thread info/api-keys.mdx Outdated
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Risk assessment: Very Low

Evidence from the current diff: changes are limited to documentation pages and Mintlify navigation entries (docs.json, info/api-keys.mdx, info/projects.mdx, reference/cli.mdx, reference/cli/api-keys.mdx, and reference/cli/auth.mdx). There are no executable code, schema, auth/permission enforcement, infrastructure, or shared runtime changes.

I also re-checked the update since the previous automation approval: it only moves the API keys page placement in navigation and trims/edits the new docs page, so the risk level has not increased. The PR already has an approval from this automation, so I’m leaving the existing approval in place and not re-approving.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

@IlyaasK
Copy link
Copy Markdown
Contributor Author

IlyaasK commented May 29, 2026

Should this belong in "Basics"? It feels more Intermediate, as most people won't need to read these docs in order to get Kernel working for them. This feels more "Kernel is deployed, here's how you managed keys"

i have put them under Guides -> Working with your browser -> Intermediate -> after the Auth subgroup, before File I/O

@IlyaasK IlyaasK requested review from ehfeng and juecd May 29, 2026 16:00
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Risk assessment: Very Low

Evidence from the current diff: the PR only updates Mintlify navigation and MDX documentation pages (docs.json, info/api-keys.mdx, info/projects.mdx, reference/cli.mdx, reference/cli/api-keys.mdx, and reference/cli/auth.mdx). It doesn't modify executable code, auth or permission enforcement, infrastructure, schemas, or shared runtime behavior.

I re-checked the synchronize update after the prior automation approval and the effective PR diff remains documentation-only, so the risk level has not increased. I found no CODEOWNERS file in the repo. Because this PR already has an approval from this automation, I'm leaving that approval in place and not re-approving.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

Copy link
Copy Markdown
Contributor

@juecd juecd left a comment

Choose a reason for hiding this comment

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

💯 Thanks Ilyaas!

@IlyaasK
Copy link
Copy Markdown
Contributor Author

IlyaasK commented May 29, 2026

💯 Thanks Ilyaas!

Thank you!

@IlyaasK IlyaasK merged commit 276d820 into main May 29, 2026
4 checks passed
@IlyaasK IlyaasK deleted the codex/api-key-management-docs branch May 29, 2026 20:05
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.

3 participants