Skip to content

feat(tui): expose thinking effort to status line commands - #2441

Open
ousamabenyounes wants to merge 1 commit into
MoonshotAI:mainfrom
ousamabenyounes:fix/issue-2394
Open

feat(tui): expose thinking effort to status line commands#2441
ousamabenyounes wants to merge 1 commit into
MoonshotAI:mainfrom
ousamabenyounes:fix/issue-2394

Conversation

@ousamabenyounes

Copy link
Copy Markdown

Related Issue

Resolve #2394

Problem

Custom status line commands receive the active model and mode data, but not the effective thinking effort shown by the built-in footer. This prevents scripts from presenting the complete active model configuration.

What changed

  • add the SDK ThinkingEffort value to the typed status line payload
  • populate it directly from the current TUI application state
  • document the JSON field in the English and Chinese configuration references
  • add a patch changeset

Test verification (RED → GREEN)

  • RED on the upstream implementation with the regression test: the real cat subprocess returned a JSON payload whose thinkingEffort was undefined instead of max
  • GREEN after the implementation: footer-status-line.test.ts passed all 17 tests
  • revert proof: removing the production payload mapping reproduced the same failure, with 16 passing and 1 failing test
  • the regression test waits on the component's actual refresh callback and cleans up the command runner in finally

Full local suite proof:

  • CLI package tests: 185 files passed, 2 skipped; 2,458 tests passed, 5 skipped
  • CLI typecheck and full package build passed
  • changed-file type-aware lint, workspace consistency, changeset validation, and documentation build passed
  • independent PR review passed
  • OCR/Codex review passed after replacing a fixed-delay wait with deterministic completion signaling

Checklist

  • I have read the CONTRIBUTING document.
  • I have linked a related issue, or explained the problem above.
  • I have added tests that prove my feature works.
  • Ran gen-changesets skill, or this PR needs no changeset.
  • Ran gen-docs skill, or this PR needs no doc update.

@changeset-bot

changeset-bot Bot commented Jul 31, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: f51e881

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

This PR includes changesets to release 1 package
Name Type
@moonshot-ai/kimi-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

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.

feat: expose thinkingEffort in the custom status line JSON snapshot

1 participant