Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Show sign in prompt when trying paid model #5497

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
chrarnoldus merged 14 commits into from
Jan 29, 2026
Merged

Show sign in prompt when trying paid model #5497

chrarnoldus merged 14 commits into from
Jan 29, 2026

Conversation

@chrarnoldus
Copy link
Collaborator

@chrarnoldus chrarnoldus commented Jan 29, 2026 •
edited
Loading

CleanShot 2026-01-29 at 17 20 05

@changeset-bot
Copy link

changeset-bot bot commented Jan 29, 2026 •
edited
Loading

🦋 Changeset detected

Latest commit: 87b3d62

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
kilo-code Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

kiloconnect[bot]
kiloconnect bot reviewed Jan 29, 2026
View reviewed changes
@kiloconnect
Copy link
Contributor

kiloconnect bot commented Jan 29, 2026 •
edited
Loading

Code Review Summary

Status: 2 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 1

Fix these issues in Kilo Cloud

Issue Details (click to expand)

WARNING

File Line Issue
src/core/task/Task.ts 4645 Unused isContextWindowExceededError variable may fail lint/typecheck

SUGGESTION

File Line Issue
webview-ui/src/components/kilocode/chat/UnauthorizedWarning.tsx 40 Consider clearing setTimeout on unmount / validating postMessage source before auto-retry
Other Observations (not in diff)

Issues found in unchanged code that cannot receive inline comments:

File Line Issue
Files Reviewed (6 files)
  • .changeset/nasty-kiwis-laugh.md - 0 issues
  • packages/types/src/message.ts - 0 issues
  • src/shared/kilocode/errorUtils.ts - 0 issues
  • src/core/task/Task.ts - 1 issue
  • webview-ui/src/components/kilocode/chat/UnauthorizedWarning.tsx - 1 issue
  • webview-ui/src/i18n/locales/en/kilocode.json - 0 issues

@chrarnoldus chrarnoldus marked this pull request as draft January 29, 2026 13:00
@markijbema
Copy link
Contributor

markijbema commented Jan 29, 2026

shouldnt it say kilo gateway

lambertjosh
lambertjosh reviewed Jan 29, 2026
View reviewed changes
@chrarnoldus chrarnoldus force-pushed the christiaan/account-cta branch from f7b54da to 79fb873 Compare January 29, 2026 14:34
@chrarnoldus chrarnoldus changed the title Christiaan/account cta Show sign in prompt when trying paid model Jan 29, 2026
@chrarnoldus chrarnoldus marked this pull request as ready for review January 29, 2026 16:54
kiloconnect[bot]
kiloconnect bot reviewed Jan 29, 2026
View reviewed changes
@chrarnoldus chrarnoldus merged commit 682a491 into main Jan 29, 2026
12 checks passed
@chrarnoldus chrarnoldus deleted the christiaan/account-cta branch January 29, 2026 18:16
This was referenced Jan 29, 2026
Merged
Open
Open
This was referenced Jan 27, 2026
Open
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lambertjosh lambertjosh lambertjosh left review comments

@kiloconnect kiloconnect[bot] kiloconnect[bot] left review comments

@markijbema markijbema markijbema approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@chrarnoldus @markijbema @lambertjosh