But after a bit more review, yup you are right that will mess with user shares, I suppouse I did not tested that :(
Then maybe we should move the logic for adding "(group)" outside of addShareWith.