Skip to content

feat(display): add tool progress message toggle#361

Merged
chenhg5 merged 1 commit intochenhg5:mainfrom
xukp20:feat/tool-message-toggle
Apr 2, 2026
Merged

feat(display): add tool progress message toggle#361
chenhg5 merged 1 commit intochenhg5:mainfrom
xukp20:feat/tool-message-toggle

Conversation

@xukp20
Copy link
Copy Markdown
Contributor

@xukp20 xukp20 commented Mar 29, 2026

Summary

Add a dedicated toggle for tool progress messages so users can hide verbose tool execution updates without enabling full quiet mode.

This keeps normal replies and thinking output visible while suppressing only tool-use and tool-result progress noise.

Fixes #173

What changed

  • add display.tool_messages config with default value true
  • suppress EventToolUse and EventToolResult rendering when the toggle is disabled
  • add /config tool_messages true|false for runtime control
  • keep existing quiet behavior unchanged, with quiet still taking precedence
  • update config examples and regression coverage

Why

Today users who want less noisy output often need to rely on broader quiet settings, which also hide useful non-tool feedback.

This change introduces a narrower control that affects only tool progress messages while preserving the rest of the conversation flow.

Validation

  • go build ./...
  • go test ./... -v -race
  • go test ./... -coverprofile=coverage.out -covermode=atomic
  • go test -v -tags=smoke ./tests/e2e/...
  • go test -v -tags=regression ./tests/e2e/...
  • go test -bench=. -benchmem -tags=performance ./tests/performance/...
  • $(go env GOPATH)/bin/staticcheck ./core/... ./config/... ./cmd/cc-connect/...

Copy link
Copy Markdown
Owner

@chenhg5 chenhg5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Useful feature with clean implementation.

Review notes:

  • ✅ CI passes
  • Adds display.tool_messages config (default true)
  • Allows hiding verbose tool messages without full quiet mode
  • Good test coverage
  • Fixes issue #173

Approved for merge.

@chenhg5 chenhg5 merged commit b081eb1 into chenhg5:main Apr 2, 2026
5 checks passed
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.

[Feature] Disable message for tools

2 participants