TypeScript: Resolvers(query.task/stats/exportErrors/exportAnnotations): Return toObject() in order to convert _id to string

[alukach]

For any method where a Task is returned, we see the following error:

Type 'Document<unknown, {}, { type: "GetStats" | "ExportAnnotations" | "ExportImageErrors" | "CreateDeployment" | "UpdateDeployment" | "DeleteDeployment"; user: string; projectId: string; status: "SUBMITTED" | ... 2 more ... | "COMPLETE"; created: Date; updated: Date; output?: any; }> & { ...; } & { ...; }' is not assignable to type 'Task'.
  Types of property '_id' are incompatible.
    Type 'ObjectId' is not assignable to type 'string'.ts(2322)

This seems to be an issue where GraphQL expected the _id to be a string:

animl-api/src/@types/graphql.ts

Line 1038 in 27b3e3d

_id: Scalars['ID']['output'];

animl-api/src/@types/graphql.ts

Line 10 in 27b3e3d

ID: { input: string; output: string; }

However, Mongoose shows it as an mongoose.Types.ObjectId. Calling .toObject() on the response object rectifies this. This may become a common pattern for our resolvers... I would like to imagine that there's a better way to handle this, but I'm not sure what that would be, perhaps something with the Scalars for the output?

[alukach]

I actually think the reason that this conversion works is because of Automattic/mongoose#12883, ie it's masking an actual issue.