Skip to content

CAMEL-23420: Add diagram modes and fullscreen scrolling to TUI monitor#23192

Merged
davsclaus merged 7 commits into
mainfrom
camel-23420-tui-text-diagram
May 13, 2026
Merged

CAMEL-23420: Add diagram modes and fullscreen scrolling to TUI monitor#23192
davsclaus merged 7 commits into
mainfrom
camel-23420-tui-text-diagram

Conversation

@davsclaus
Copy link
Copy Markdown
Contributor

@davsclaus davsclaus commented May 13, 2026

Summary

  • Add separate d (image diagram) and D (text diagram) shortcuts in the TUI monitor Routes tab
  • When terminal doesn't support images, d shows a message suggesting Shift+D instead of silently falling back to ASCII
  • Both diagram modes open in fullscreen with a live-updating route info header and TamboUI Scrollbar widgets for vertical and horizontal scrolling
  • Image diagram uses ImageData.crop() for viewport-based scrolling with crop caching to avoid unnecessary recomputation
  • Image is scaled to 50% to fit better in terminal cells
  • Reduced refresh rate (1s) and skip re-render on idle ticks when showing image diagram to prevent flicker
  • Footer shows the matching close key (d or D) depending on how the diagram was opened

Test plan

  • Run camel monitor against a running Camel integration
  • Press D on the Routes tab — verify fullscreen text diagram with route header and scrollbars
  • Press d on a terminal with image support — verify fullscreen image diagram with scrollbars, no flickering
  • Verify arrow keys scroll vertically and horizontally in both modes, PgUp/PgDn for page scrolling
  • Verify the route info header updates live (e.g., total count increases)
  • Press d on a terminal without image support — verify it shows the "use Shift+D" message
  • Press Esc or the matching key to close the diagram

Claude Code on behalf of Claus Ibsen

davsclaus and others added 5 commits May 13, 2026 13:17
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

ghost commented May 13, 2026

🌟 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
Copy link
Copy Markdown
Contributor

ghost commented May 13, 2026

🧪 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

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@davsclaus davsclaus changed the title CAMEL-23420: Add text diagram mode and fullscreen scrolling to TUI monitor CAMEL-23420: Add diagram modes and fullscreen scrolling to TUI monitor May 13, 2026
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@davsclaus davsclaus merged commit 29c99a4 into main May 13, 2026
6 checks passed
@davsclaus davsclaus deleted the camel-23420-tui-text-diagram branch May 13, 2026 13:47
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