Skip to content

feat: add log viewer along with shortcut - #785

Draft
RobertMueller2 wants to merge 3 commits into
Satty-org:mainfrom
RobertMueller2:feat_log_viewer
Draft

RobertMueller2 wants to merge 3 commits into
Satty-org:mainfrom
RobertMueller2:feat_log_viewer

Conversation

@RobertMueller2

@RobertMueller2 RobertMueller2 commented Oct 25, 2026 •

Copy link
Copy Markdown
Member

Closes: #704

Description

Adds a ring buffer (hardcoded 4096 entries) for log messages, which can be displayed using a log viewer dialog (new custom relm component).

Based on #784, will un-draft once that is merged.

Checklist

  • I've read CONTRIBUTING.md and AI_POLICY.md
  • all commits compile
  • PR title uses (conventional commit message)[https://www.conventionalcommits.org/]
  • any merge conflicts were resolved via rebase rather than merge
  • includes LLM/Generative AI generated code - some help with the component
    • I understand the code on a technical level
    • I vouch for the code

I got some help from claude with the dropdown and a component skeleton, I'm not convinced going through string is necessary, though, or necessarily the best solution.

Todo:

  • figure out if we can implement ListModel for LogEntry, the downside would be a relm4 dependency within CLI which we did not need so far
  • actually indicate shortcut in sketchboard, just mentioning log viewer is a bit lazy 😄
  • maybe use the prefixes within the dropdown, but if that's easily doable may depend on the ListModel
  • use refresh icon for refresh button

Not to do:

  • I'm not going to add a UI element for this at this point, I'm not sure where there is a good place. this may need a followup issue.
satty-20261025-155650

@RobertMueller2 RobertMueller2 changed the title Feat log viewer feat: add log viewer along with shortcut Oct 25, 2026

This branch has not been deployed

No deployments
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.

followup for error notifications: log to ring buffer and allow to retrieve logs

1 participant