Yes, returning `data = data || [];` is good -- I think handling the empty results in the callback is better than returning the string