useMutation should assume input is required

[paul-sachs]

The following example currently throws a typescript error:

const deleteOrg = useMutation({
  ...deleteOrganizationByName.useMutation(),
  onSuccess: (_, { name }) => {
    showToast(`Successfully deleted organization "${name ?? ""}"`);
  },
}); 

Because it's assumed params is optional. We can work around this via something like:

const deleteOrg = useMutation({
  ...deleteOrganizationByName.useMutation(),
  onSuccess: (_, input) => {
    if (input === undefined) {
      return;
    }
    const { name } = input;
    showToast(`Successfully deleted organization "${name ?? ""}"`);
  },
}); 

But given that most mutation should have some kind of input, this feels unnecessary. We already have to default name to "" because we can't assume validation but that's a larger protobuf-es thing to tackle.

[paul-sachs]

In retrospect, this is fine. There are certainly cases where a mutations might not have parameters, and currently protobuf objects default to all properties being optional.