Skip to content

feat(tui-pty-e2e): Add colored debug output with indentation#74

Merged
CSRessel merged 4 commits into
devfrom
tui-pty-e2e-debug-coloring
Dec 3, 2025
Merged

feat(tui-pty-e2e): Add colored debug output with indentation#74
CSRessel merged 4 commits into
devfrom
tui-pty-e2e-debug-coloring

Conversation

@CSRessel
Copy link
Copy Markdown
Collaborator

@CSRessel CSRessel commented Dec 3, 2025

Summary

Generated with Nori

  • Add colored debug output using owo-colors library (matching exec module pattern)
  • Colors are enabled by default when stderr is a terminal and NO_COLOR is not set
  • Add 4-space indentation for screen contents and ACP tracing logs in panic output
  • Color the DEBUG_TUI_PTY debug output in poll() and wait_for() methods

Test Plan

  • Verified cargo build passes
  • Verified cargo clippy passes
  • Verified cargo fmt passes
  • Verified tests pass (1 pre-existing failure unrelated to this change)
  • Run tests with DEBUG_TUI_PTY=1 to see colored output

Share Nori with your team: https://www.npmjs.com/package/nori-ai

Add optional colored debug output using owo-colors library for the
TUI PTY E2E test harness. Colors are enabled by default when stderr
is a terminal and NO_COLOR is not set.

Changes:
- Add DebugStyles struct using owo-colors Style pattern (same as exec module)
- Color the Drop implementation output (panic screen state and tracing)
- Add 4-space indentation for screen contents and tracing logs
- Color DEBUG_TUI_PTY debug output in poll() and wait_for() methods
The Drop implementation already prints the screen contents with nice
formatting when panicking, so remove the duplication from the error
message itself.
@CSRessel CSRessel force-pushed the tui-pty-e2e-debug-coloring branch from 0536ecc to d491a17 Compare December 3, 2025 19:51
@CSRessel CSRessel force-pushed the dev branch 3 times, most recently from 2b3ed21 to e6fa5b0 Compare December 3, 2025 21:02
@CSRessel CSRessel merged commit e64d5b0 into dev Dec 3, 2025
1 of 3 checks passed
@CSRessel CSRessel deleted the tui-pty-e2e-debug-coloring branch December 3, 2025 21:29
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