Currently, the only place we are using this is in the dispatch controller in task list api and returning the result directly, so it doesn't matter too much whether we convert it here or in the controller. For all other purposes, I expect we would be using find_by_criteria to get particular task statuses which is returning the cursor. Having said that, I like the idea to keep it consistent with the return type of find_by_criteria call. I will change it and convert it to list in the dispatch controller.