Skip to content

feat(kiloclaw): add BYOK setup link to settings page#3313

Open
evanjacobson wants to merge 3 commits into
mainfrom
improvement/kiloclaw-settings-byok-link
Open

feat(kiloclaw): add BYOK setup link to settings page#3313
evanjacobson wants to merge 3 commits into
mainfrom
improvement/kiloclaw-settings-byok-link

Conversation

@evanjacobson
Copy link
Copy Markdown
Contributor

@evanjacobson evanjacobson commented May 18, 2026

Summary

Users configuring default models in KiloClaw settings had no easy way to discover or navigate to BYOK (Bring Your Own Key) setup. This adds a link to the Kilo BYOK settings page and the BYOK setup guide documentation directly below the default model selector, guiding users to configure their own model provider keys.

Verification

  • Link to /byok navigates to Kilo BYOK settings page
  • Link to BYOK setup guide opens docs in new tab

Visual Changes

Before After
N/A Added text below default model picker with two links: one to Kilo BYOK settings, one to setup guide docs

Reviewer Notes

N/A

@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented May 18, 2026

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Executive Summary

The latest commit replaced target="_blank" with rel="noopener noreferrer" on the external docs link instead of keeping both — the link now opens in the same tab and the rel attribute is ineffective.

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
apps/web/src/app/(app)/claw/components/SettingsTab.tsx 2245 rel="noopener noreferrer" present but target="_blank" was removed — link opens in same tab; both attributes are needed together
Files Reviewed (1 file)
  • apps/web/src/app/(app)/claw/components/SettingsTab.tsx — 1 issue

Fix these issues in Kilo Cloud


Reviewed by claude-4.6-sonnet-20260217 · 391,007 tokens

Review guidance: REVIEW.md from base branch main

Comment thread apps/web/src/app/(app)/claw/components/SettingsTab.tsx Outdated
adding it back here, but I don't think that this is a risk as we own the page.

Co-authored-by: kilo-code-bot[bot] <240665456+kilo-code-bot[bot]@users.noreply.github.com>
.{' '}
<a
href="https://kilo.ai/docs/getting-started/byok"
rel="noopener noreferrer"
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.

WARNING: rel="noopener noreferrer" is present but target="_blank" was removed

The previous commit added target="_blank" (missing rel); this commit swapped them instead of adding both. The link now opens in the same tab, and the rel attribute is meaningless without target="_blank". Both attributes are needed together:

Suggested change
rel="noopener noreferrer"
target="_blank"
rel="noopener noreferrer"

(Add target="_blank" back on the line above this one, or restore both attributes on the <a> tag.)

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