CAMEL-23635: camel-jbang - Add shell panel to TUI

[gnodet]

CAMEL-23635

Summary

Embeds a real JLine shell inside the TUI using JLine's virtual terminal infrastructure:

• ShellPanel: new overlay panel accessible from F2 → Shell

  • Uses LineDisciplineTerminal + ScreenTerminal (VT100 emulator) + ScreenTerminalOutputStream — the same wiring pattern as JLine's own WebTerminal
  • Shell runs in a background thread via ShellBuilder with PicocliCommandRegistry — full tab completion, history, aliases, fish-style autosuggestions
  • Screen buffer dumped each frame and converted to TamboUI Span/Line widgets with proper color/attribute mapping
  • Full resize handling: size toggle + real terminal resize propagated via setSize() → SIGWINCH
  • Key events encoded as ANSI escape sequences and forwarded to the virtual terminal

• Layout: three sizes — 25%, 50%, or 100% of the content area

  • F11 grows: 25% → 50% → 100%
  • Escape shrinks: 100% → 50% → 25% → close

Test plan

• camel top → F2 → Shell → verify shell prompt appears
• Type catalog component --filter=kafka → verify output with colors
• Test tab completion on commands
• F11 → verify 50% → 100% grow
• Escape → verify 100% → 50% → 25% → close cycle
• Resize terminal window → verify shell reflows
• Reopen shell → verify previous session state

Claude Code on behalf of Guillaume Nodet

[github-actions]

🌟 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.

[davsclaus]

Oh can the toggle be 25%/50%/100% or does that not work to well ?

[gnodet]

Updated — the toggle now cycles through three sizes:

• F11 grows: 25% → 50% → 100%
• Escape shrinks: 100% → 50% → 25% → close

The split uses Constraint.percentage() so it adapts to any terminal height.

Claude Code on behalf of Guillaume Nodet

[davsclaus]

There is a merge conflict, thanks

[gnodet]

Merge conflict resolved — rebased on latest main which added the ACTION_RUN_INFRA action. Action IDs and group sizes updated accordingly.

Claude Code on behalf of Guillaume Nodet

[github-actions]

🧪 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]

Nice feature — the JLine ScreenTerminal + LineDisciplineTerminal wiring is solid, and the convertAttrToStyle() bit extraction correctly matches JLine's own generateSpanTag(). A couple of minor observations below.

This review does not replace specialized AI review tools (CodeRabbit, Sourcery) or static analysis (SonarCloud).

This review was generated by an AI agent and may contain inaccuracies. Please verify all suggestions before applying.

[davsclaus]

FQCN: org.jline.shell.Shell should be imported rather than used inline (per project convention in CLAUDE.md). The OpenRewrite build plugin will auto-fix this on mvn clean install, so just make sure to rebuild before pushing.

Suggested change

	screenTerminal = null;
	try (Shell shell = Shell.builder()

(with import org.jline.shell.Shell; added to the imports block)

[davsclaus]

Minor: close() calls stopShell() which destroys the virtual terminal and shell thread. The PR test plan mentions "Reopen shell → verify previous session state", but reopening creates a fresh shell with no prior state. Consider updating the test plan description, or alternatively only setting visible = false without destroying the shell (and deferring stopShell() to TUI exit).