If you're translating vectors 10 times, or calculating cosines, or whatever, then yes, you should abstract this into separate functions. It's not like "translateVector" or "calculateCosine" are brittle abstractions or anything.
You don't know what was being done. All you know is that the manager took a peek at the PR and told the developer to revert the commit.