Skip to content

Comments

feat(ui): add toggleable in-client console#1569

Merged
chsami merged 2 commits intodevelopmentfrom
codex/add-toggleable-console-window
Oct 16, 2025
Merged

feat(ui): add toggleable in-client console#1569
chsami merged 2 commits intodevelopmentfrom
codex/add-toggleable-console-window

Conversation

@chsami
Copy link
Owner

@chsami chsami commented Oct 16, 2025

Summary

  • add an embedded log console panel beneath the client canvas with a toggle button in the toolbar
  • stream System.out and SLF4J output to the in-client console via a custom logback appender

Testing

  • mvn -pl runelite-client -DskipTests package (fails: dependency resolution for com.google.inject:guice-bom 4.1.0 is blocked by HTTP 403 in this environment)

https://chatgpt.com/codex/tasks/task_e_68f006ee88fc832884d71d4eafe0b4cc

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 16, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

This change adds an in-app log console to the client UI. It introduces a console container in ClientPanel with APIs to set the console component, query visibility, and adjust minimum/preferred sizes when visible. ClientUI wires a LogConsolePanel into the layout, adds a toolbar toggle with open/closed icons, and manages visibility updates. Logging is routed to the console via a new ConsoleLogAppender (logback) and by teeing System.out to both the original stream and the console’s OutputStream. LogConsolePanel provides a read-only, auto-scrolling text area with trimming and an OutputStream that buffers and forwards lines to the panel on the EDT.

✨ Finishing touches
  • 📝 Generate docstrings

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7cb9069 and f1b1f55.

📒 Files selected for processing (4)
  • runelite-client/src/main/java/net/runelite/client/ui/ClientPanel.java (2 hunks)
  • runelite-client/src/main/java/net/runelite/client/ui/ClientUI.java (8 hunks)
  • runelite-client/src/main/java/net/runelite/client/ui/ConsoleLogAppender.java (1 hunks)
  • runelite-client/src/main/java/net/runelite/client/ui/LogConsolePanel.java (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@chsami chsami marked this pull request as ready for review October 16, 2025 03:33
@chsami chsami merged commit f87d050 into development Oct 16, 2025
3 checks passed
@coderabbitai coderabbitai bot mentioned this pull request Oct 16, 2025
@chsami chsami deleted the codex/add-toggleable-console-window branch November 16, 2025 11:52
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