nit: add a comment explaining how this will be called and which structure we have in data