Improve bike searching concern

[sethherr]

The BikeSearchable concern is confusing and not performant.

• It should improve performance (look at our Skylight, consider the results from rack-mini-profiler)
• It shouldn't reduce passing executed sql queries (currently done in non_serial_matches for the color search), and should instead use correct sql.
• It should be restructured to being a service instead of a concern
• It should (probably) use private methods - even if this means we have to use send in tests (I feel better about doing this since watching episodes of on writing software well)

Note: it was written as a concern rather than a service originally was so that the methods could be written in a more functionalish style, without having to pass the activerecord collection around - I would like to maintain the funcationalish-ness, but now think passing the collection around is would be better than what we are doing