Improve text search result output, remove use of Serilog.Sinks.Console theming to improve consistency - #481
Merged
Conversation
…ll themed output now uses Serilog.Expressions theming. Breaking change in the `seqcli print` command's template parameter; the syntax was previously undocumented but used Serilog syntax. It's now Serilog.Expressions syntax. Also improved output for spans (show elapsed timing) and switched the theme appearance to approximate the one used by Seq's `flaretl` command. Refactoring and additions largely by hand; some AI assistance for initial versions of PInvoke signatures/calls and some test cases. Assisted-by: Claude:claude-fable-5
…t, those are covered well by --native and --json, while plain text should offer an easier-on-the-eyes option
nblumhardt
marked this pull request as ready for review
July 29, 2026 23:35
liammclennan
approved these changes
Jul 30, 2026
nblumhardt
changed the title
Serilog.Sinks.Console theming to improve consistencySerilog.Sinks.Console theming to improve consistency
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
All themed output now uses Serilog.Expressions theming. Breaking change in the
seqcli printcommand's template parameter; the syntax was previously undocumented but used Serilog syntax. It's now Serilog.Expressions syntax.Also improved output for spans (show elapsed timing) and switched the theme appearance to approximate the one used by Seq's
flaretlcommand:Refactoring and additions largely by hand; some AI assistance for initial versions of PInvoke signatures/calls and some test cases.
Needs testing on Windows before it will be ready.Also now includes some additional consistency-related fixes on Windows, and drops trailing property lists from plain-text output (which previously made
seqcli searcheffectively unusable for quick log reading).