I can see what you're going for here, but most routers are designed to work with the central circular road tagged with junction=roundabout. Some routers will use that to give directions such as "Enter the roundabout, and take the 3rd exit," which is only possible if it knows which segments are part of the roundabout itself as opposed to part of the connecting roads.

However, I'd say to not simply revert, since it looks like you've made another important change that very much helps routers: separating the entry and exit ways so they don't meet at a single point. This helps them understand that drivers need to enter the roundabout and then immediately exit, as opposed to just making a slight right turn.

There's some good notes on all sorts of edge cases on the wiki at https://wiki.openstreetmap.org/wiki/Tag:junction=roundabout but I usually end up referencing other similar ones nearby like https://openstreetmap.org/way/678262785 unless I come across something particularly weird.

(Though Boise does have its share of weird roundabouts: https://openstreetmap.org/way/767448730#)