Repository navigation
Conversation
…ngs) - Delete ICache<TKey,TValue> interface, LRUCache<TKey,TValue> implementation, and SqliteLogCache - Remove CacheSize and EnableIndexing settings from AppSettings - Remove cache management UI (Clear Cache button, cache size input) from SettingsWindow - Remove ClearCacheAsync command, CacheSize and EnableIndexing properties from SettingsViewModel - Remove LRUCacheTests and update AppSettingsTests/SettingsServiceTests accordingly
- Add UseMultilineRegex setting to AppSettings (persisted to disk) - Add UseMultiline property to SearchResultHighlighter; when enabled, uses Singleline|Multiline regex flags so '.' spans newlines and matches are highlighted across line boundaries via ColorizeLineFromResults() - Add UseMultilineRegex observable property to MainViewModel with OnUseMultilineRegexChanged handler that invalidates cached regex, fires OnMultilineRegexModeChanged event, and saves the setting - Wire OnMultilineRegexModeChanged in MainWindow.xaml.cs to update the highlighter and re-execute search / filter when active - Move the regex (.*) checkbox next to the search box and add a new multiline toggle button (≡) that is only enabled when regex mode is on - Update ApplySearchFilter to handle multiline matches spanning multiple lines and show 'multiline regex' in status text
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
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.
This pull request removes the in-memory LRU cache implementation and its interfaces, along with related configuration and tests, in favor of persistent caching (such as the SQLite cache). It also introduces support for multiline regex search and updates search highlighting logic to support this feature. Additionally, default filters are added for log filtering.
Key changes:
Removal of in-memory caching
LRUCacheclass and its interfaceICachefrom the codebase, along with all related tests and configuration settings (CacheSize,EnableIndexing). This shifts caching responsibility to persistent solutions like the SQLite cache. [1] [2] [3] [4] [5] [6] [7]Search and filter improvements
UseMultilineRegexproperty toAppSettingsand updated theSearchResultHighlighterto support multiline regex search, including a newUseMultilineproperty and logic to handle regex options accordingly. [1] [2] [3]Filters
filters.json, including filters for errors, warnings, info, and several regex-based searches (GUIDs, emails, IPs, and custom patterns).