Before your fix, the only difference in the message (besides the backtrace, which our deprecation grouping logic filters out) was the `matcher.class.name || matcher.inspect` bit -- which would (wrongly) cause different matchers to be grouped separately.

Your `:type` value here is essentially grouping it the same way.  Different matchers will be grouped separately.  I think we want it to group all uses of the legacy matcher protocol as one `:type` (something like `"legacy_matcher_protocol"`).