Skip to content

feat(cli): compact tool status with subtle trace logs#89

Merged
xinhuagu merged 1 commit into
mainfrom
feat/cli-tool-trace-compact
Feb 25, 2026
Merged

feat(cli): compact tool status with subtle trace logs#89
xinhuagu merged 1 commit into
mainfrom
feat/cli-tool-trace-compact

Conversation

@xinhuagu

Copy link
Copy Markdown
Owner

Summary\n- reuse a single status entry for repeated same-name tools instead of growing many lines\n- add subtle gray trace lines for tool lifecycle visibility:\n - [tool:start] ...\n - [tool:done] ...\n - [tool:error] ...\n- keep status panel behavior and tests updated\n\n## Validation\n- ./gradlew :aceclaw-cli:test --no-daemon

@greptile-apps

greptile-apps Bot commented Feb 25, 2026

Copy link
Copy Markdown

Greptile Summary

Implemented compact tool status display with trace logs and status entry reuse:

  • Added subtle gray trace lines ([tool:start], [tool:done], [tool:error]) for tool lifecycle visibility
  • Status panel now reuses a single entry for repeated same-name tools instead of creating new entries
  • Test coverage validates the reuse behavior by checking internal status map size

The implementation follows defensive coding practices with proper null checks and bounds validation. The trace log formatting is consistent with existing patterns in the codebase.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • The changes are well-contained to UI/UX improvements in the terminal output layer. The code follows defensive coding patterns from CLAUDE.md (null checks, bounds validation). The new test verifies the core behavior. No logical errors or security issues detected.
  • No files require special attention

Important Files Changed

Filename Overview
aceclaw-cli/src/main/java/dev/aceclaw/cli/ForegroundOutputSink.java Added trace logs for tool lifecycle and status entry reuse for repeated tool calls
aceclaw-cli/src/test/java/dev/aceclaw/cli/ForegroundOutputSinkTest.java Added test to verify status entry reuse for repeated same-name tools

Last reviewed commit: d1464fc

@xinhuagu xinhuagu merged commit 0d75252 into main Feb 25, 2026
3 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.

1 participant