Skip to content
This repository was archived by the owner on Sep 23, 2026. It is now read-only.

feat(ui): improve shell spacing, link highlighting and notification duration - #2230

Merged
RealKai42 merged 3 commits into
MoonshotAI:mainfrom
jackfish212:better-render
May 11, 2026
Merged

RealKai42 merged 3 commits into
MoonshotAI:mainfrom
jackfish212:better-render

Conversation

@jackfish212

@jackfish212 jackfish212 commented May 11, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

This PR fixes shell UI spacing and link highlighting issues, adds task duration to background notifications, and updates the changelog with unreleased entries.


1. Shell UI spacing and link highlighting fixes

Problem: The shell UI had inconsistent spacing and broken link highlighting in certain rendering paths.

What was done:

  • Fixed console spacing in src/kimi_cli/ui/shell/console.py.
  • Added block visualization support in src/kimi_cli/ui/shell/visualize/_blocks.py.
  • Improved live view rendering in src/kimi_cli/ui/shell/visualize/_live_view.py.
  • Fixed markdown link handling in src/kimi_cli/utils/rich/markdown.py.
  • Added and updated tests in tests/ui_and_conv/test_visualize_running_prompt.py.

2. Background notification duration

Problem: Background task completion notifications did not include task duration, making it hard for users to gauge how long operations took.

What was done:

  • Added duration tracking and display in src/kimi_cli/background/manager.py.
  • Updated shell UI integration to show duration.
  • Added and updated tests in tests/background/test_manager.py and related shell prompt tests.

3. Changelog updates

What was done:

  • Added unreleased entries to CHANGELOG.md and both English and Chinese release notes.

Related Issue

Resolve #(issue_number)

Description

Checklist

  • I have read the CONTRIBUTING document.
  • I have linked the related issue, if any.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have run make gen-changelog to update the changelog.
  • I have run make gen-docs to update the user documentation.

img_v3_0211j_d64bb67d-2201-468f-8b4a-3062ff40453g img_v3_0211j_4613a8bc-0d4a-460c-84a3-dd10416fd7eg img_v3_0211j_9b5a6cef-c4fb-4c44-9c13-b199fb52614g image Open in Devin Review

- fix content block paragraph separation loss
- add blank line spacing after message echo, tool calls, and notifications
- add underline to h2 headings and adjust table border style
- restore markdown link highlight styles
- sync related tests
- include completion time and elapsed duration in background task notification body\n- add finished_at and duration_s to notification payload\n- add debug logging around task state transitions and notification publishing\n- add blank line spacing after user input echo in shell UI\n- update tests for new notification format and shell output
…ation duration

- Add changelog entries for shell UI spacing fixes, markdown link/heading/table rendering improvements\n- Add entry for background task terminal notification duration and timestamp\n- Sync English and Chinese docs changelogs via sync script

@devin-ai-integration devin-ai-integration Bot left a comment

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.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 5 additional findings.

Open in Devin Review

@RealKai42
RealKai42 merged commit 0d1de5f into MoonshotAI:main May 11, 2026
35 of 36 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants