const exports = { quantity: 1, type: 'boxes', }; exports.getQuantity = () => { foo(); }; This diff driver would mistakenly detect `exports.getQuantity = () => {`. Although, the more I think about it, the spirit of this patch seems to be "we want to show headers whenever we think we are in a function", so we don't actually need to treat 'exports' or 'module.exports' specially at all, e.g. this case should also pass our diff driver tests: const foo = {}; foo.RIGHT = () => { ChangeMe(); }; and if we do this, we will correctly handle 'exports' and 'module.exports' anyway by virtue of them being plain old JS objects.