this feels a little misleading if future code ever tries to do something with these ids like get a doc out of the DB. not knowing full context I'd be ok just adding a code comment here to make it obvious to the user that you're returning modified ids