It would be useful if we had a global concept of storage state, for all modules and systems to use consistently. For example, a view can be custom, default, or overridden. So can a layout. So can anything else that's exportable, like an image style!
Right now views uses constants for these things (VIEWS_STORAGE_OVERRIDE, VIEWS_STORAGE_NORMAL) but it would be good if we didn't need to use these constants for other unrelated things, like layouts or image styles. It would also be good if they were not these silly constants (integers) but were instead strings, like "custom", "default", or "overridden".
Recent comments
Worth mentioning as it was in zulip chat by @yorkshirepudding I just tested and the two work together without issue as long as you don't use the "Exclude comments from Content...
Search does not include comments
There is also Node Comments Search - this searches the node and all comments (default search is node and first page of comments) as part of the content search. Izzy's module above is a separate...
Search does not include comments
I have created a new module comment_search
Search does not include comments