Skip to content

fix(shell): hide context usage until warning threshold#2300

Open
binichallein wants to merge 1 commit into
MoonshotAI:mainfrom
binichallein:fix/context-usage-indicator-2291
Open

fix(shell): hide context usage until warning threshold#2300
binichallein wants to merge 1 commit into
MoonshotAI:mainfrom
binichallein:fix/context-usage-indicator-2291

Conversation

@binichallein
Copy link
Copy Markdown

@binichallein binichallein commented May 15, 2026

Related Issue

Resolve #2291

Description

  • Stop rendering the shell bottom toolbar context usage meter while usage is below the warning threshold.
  • Keep the context meter visible once usage reaches 80%, where the user may need to act.
  • Preserve right-side toast behavior so transient status messages still appear in the same toolbar area.

Verification

  • RED: uv run pytest tests/ui_and_conv/test_prompt_tips.py -k "context or running_prompt_uses_shared_toolbar" -vv failed before the implementation because low context usage was still rendered.
  • GREEN: uv run pytest tests/ui_and_conv/test_prompt_tips.py -k "context or running_prompt_uses_shared_toolbar" -vv
  • uv run pytest tests/ui_and_conv/test_prompt_tips.py tests/core/test_status_formatting.py -vv
  • uv run ruff check src/kimi_cli/ui/shell/prompt.py tests/ui_and_conv/test_prompt_tips.py
  • uv run ruff format --check src/kimi_cli/ui/shell/prompt.py tests/ui_and_conv/test_prompt_tips.py
  • uv run pyright src/kimi_cli/ui/shell/prompt.py tests/ui_and_conv/test_prompt_tips.py
  • git diff --check

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. (N/A: small UI behavior fix; no user docs/changelog generated.)
  • I have run make gen-docs to update the user documentation. (N/A: no docs generation needed.)

Open in Devin Review

Copy link
Copy Markdown
Contributor

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

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 2 additional findings.

Open in Devin Review

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.

Remove constantly-updating context usage indicator from UI

1 participant