Could maybe even add a checkmark I icon to clarify the selected state.

fortunately this segmented button pattern is a convention/UI used in many other places throughout the app

Does that make it clear to your users or new users?

ideally both, the Share dialog isn't the first UI that you'll interact with

but there's more work to be done here