feat(kilo cloud) Ability to add system message at the top - #5142
Conversation
![kilo-code-bot[bot]](https://avatars.githubusercontent.com/in/2193792?s=60&v=4){kind=small}
| import { Megaphone } from 'lucide-react'; | ||
|
|
||
| const SYSTEM_MESSAGE_TEXT = 'Important security notice'; | ||
| const SYSTEM_MESSAGE_URL = 'https://blog.kilo.ai/'; |
There was a problem hiding this comment.
WARNING: Link text promises a specific notice but the URL is the blog root
The visible label is "Important security notice", which sets the expectation of a specific advisory, but SYSTEM_MESSAGE_URL points at https://blog.kilo.ai/ (the blog homepage). Users clicking it land on a generic listing page with no obvious security notice. If this is a placeholder, please point it at the actual post URL before merging.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
There was a problem hiding this comment.
Yes. Text and URL pending. this was just a pre PR to push the branch in case someone needs to take over the branch.
| @@ -0,0 +1,18 @@ | |||
| import { Megaphone } from 'lucide-react'; | |||
|
|
|||
| const SYSTEM_MESSAGE_TEXT = 'Important security notice'; | |||
There was a problem hiding this comment.
SUGGESTION: Hardcoded message requires a redeploy to change or remove
Because SYSTEM_MESSAGE_TEXT and SYSTEM_MESSAGE_URL are compile-time constants, updating the message (or taking it down once the notice is no longer relevant) requires a code change and a full redeploy. For a system/announcement banner — typically used for time-sensitive comms — consider sourcing these from an env var (e.g. NEXT_PUBLIC_SYSTEM_MESSAGE_TEXT / ..._URL) and rendering nothing when unset, so the banner can be changed or removed via config. Fine to keep hardcoded if this is intentionally a one-off notice.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
There was a problem hiding this comment.
This is as per design for now.
Code Review SummaryStatus: 2 Issues Found | Recommendation: Address before merge Executive SummaryNew topbar system-message banner is safe and leak-free, but its "Important security notice" label links to the blog root and its content is hardcoded, requiring a redeploy to update or remove. Overview
Issue Details (click to expand)WARNING
SUGGESTION
Files Reviewed (2 files)
Fix these issues in Kilo Cloud Reviewed by kimi-k3 · Input: 44K · Output: 7.3K · Cached: 394.3K Review guidance: REVIEW.md from base branch |
2 participants
Summary
Adds a static banner at the top of the kilo cloud screen.
Verification
Visual Changes
Reviewer Notes