Skip to content

CAMEL-23615: Force colored logging when TUI launches examples#23543

Merged
davsclaus merged 1 commit into
mainfrom
CAMEL-23615-tui-log-color
May 27, 2026
Merged

CAMEL-23615: Force colored logging when TUI launches examples#23543
davsclaus merged 1 commit into
mainfrom
CAMEL-23615-tui-log-color

Conversation

@davsclaus
Copy link
Copy Markdown
Contributor

Summary

  • When the TUI launches examples (via F2 → Run), stdout is redirected to a log file, so the child camel run process detects no TTY and disables ANSI colors
  • Fix: pass --logging-color=true explicitly in both launch paths (launchWithName() and launchSelectedExample())
  • The TUI's LogTab already parses ANSI escape sequences via TuiHelper.ansiToLine(), so the colored output renders correctly

Test plan

  • Compiles
  • Launch an example from TUI via F2 and verify logs are displayed with colors

🤖 Generated with Claude Code on behalf of Claus Ibsen

When the TUI launches examples via F2, stdout is redirected to a file
so the child process detects no TTY and disables ANSI colors. Pass
--logging-color=true explicitly so log output includes color codes
that the TUI's LogTab already knows how to parse and render.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟
🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run
  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot although they are normally detected and executed by CI.
  • You can label PRs using skip-tests and test-dependents to fine-tune the checks executed by this PR.
  • Build and test logs are available in the summary page. Only Apache Camel committers have access to the summary.

⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@github-actions github-actions Bot added the dsl label May 26, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🧪 CI tested the following changed modules:

  • dsl/camel-jbang/camel-jbang-plugin-tui
All tested modules (2 modules)
  • Camel :: JBang :: Plugin :: TUI
  • Camel :: Launcher :: Container

⚙️ View full build and test results

@davsclaus davsclaus merged commit 89135df into main May 27, 2026
6 checks passed
@davsclaus davsclaus deleted the CAMEL-23615-tui-log-color branch May 27, 2026 05:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant