Revert " feat: add /web command and improve Web UI experience"#858
Revert " feat: add /web command and improve Web UI experience"#858
Conversation
This reverts commit 483127c.
There was a problem hiding this comment.
Pull request overview
This pull request reverts PR #855, which added a /web slash command to switch from the CLI to the Web UI and made various UI improvements. The revert removes the /web command functionality and restores the previous state of several UI components.
Changes:
- Removed
/webslash command andSwitchToWebexception class from the CLI - Reverted Web UI component changes including session-based conditional rendering
- Reverted markdown rendering plugin configuration (disabled katex math rendering)
- Reverted code block overflow detection optimization
- Updated welcome message to remove reference to
/webcommand
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
src/kimi_cli/ui/shell/slash.py |
Removed /web slash command and its import of SwitchToWeb |
src/kimi_cli/ui/shell/__init__.py |
Removed SwitchToWeb import and exception handling |
src/kimi_cli/cli/__init__.py |
Removed SwitchToWeb exception class and web switching logic from reload loop |
src/kimi_cli/app.py |
Updated welcome message to remove /web command instructions |
web/src/App.tsx |
Removed useTheme hook initialization |
web/src/features/chat/components/chat-workspace-header.tsx |
Reverted conditional rendering logic for UI elements based on session selection |
web/src/components/ai-elements/streamdown.tsx |
Reverted to empty rehype plugins array (disabling math formula rendering) |
web/src/components/ai-elements/code-block.tsx |
Removed overflow detection logic and unused React imports |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| isOverflowing && "overscroll-contain", | ||
| )} | ||
| > | ||
| {/* Scrolling container */} |
There was a problem hiding this comment.
There is trailing whitespace after the comment on line 343. This should be removed to maintain code cleanliness.
| {/* Scrolling container */} | |
| {/* Scrolling container */} |
| <ContextContent align="end" sideOffset={16} > | ||
| <ContextContentBody className="space-y-4"> | ||
| <ContextRawUsage /> | ||
|
|
There was a problem hiding this comment.
There is an empty line with trailing whitespace on line 94. This should be removed to maintain code cleanliness.
| </Tooltip> | ||
| </> | ||
| )} | ||
|
|
There was a problem hiding this comment.
There is an extra blank line (line 144) that appears to be unintentional. This should be removed to maintain consistent spacing.
Reverts #855