Skip to content

fix: limit tool progress card to 16 lines max - #147

Merged
lishuceo merged 1 commit into
mainfrom
feat/claude-session-cf656c
Mar 12, 2026
Merged

fix: limit tool progress card to 16 lines max#147
lishuceo merged 1 commit into
mainfrom
feat/claude-session-cf656c

Conversation

@lishuceo

Copy link
Copy Markdown
Owner

Summary

  • 工具调用进度卡片默认最多显示 16 行(之前是 40 行),超出部分省略,减少卡片过长干扰阅读

Test plan

  • 52 个 message-builder 测试全部通过
  • 截断逻辑不受影响(测试显式传 maxDisplayed=3)

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@claude

claude Bot commented Mar 12, 2026

Copy link
Copy Markdown

Claude finished @lishuceo's task in 28s —— View job


Review: PR #147

  • Check previous review comments — no previous comments
  • Read changed file and understand context
  • Post review summary

Approved — Clean, minimal change.

The PR changes the default maxDisplayed parameter from 40 to 16 in buildToolProgressCard(). The truncation logic (slice(-maxDisplayed) + omission notice) remains unchanged and correctly handles the new limit. Existing tests that explicitly pass maxDisplayed=3 are unaffected.

No bugs, no security issues, no architectural concerns.

@lishuceo
lishuceo merged commit 20304d2 into main Mar 12, 2026
4 checks passed
@lishuceo
lishuceo deleted the feat/claude-session-cf656c branch March 12, 2026 05:52
lishuceo added a commit that referenced this pull request Apr 8, 2026
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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.

1 participant