I guess this was my naive way of keeping the return type consistent with what it was originally to ensure the resource hash was kept consistent for cases where parameters do _not_ have hashes, but something like `(into '() (vec (utils/recursive-sort-maps parameters)))` would do the same thing ... so I'm happy to change.

Truth is we could probably drop all this, but it would cause all resource existing resource hashes to change after upgrading potentially causing a spike in inserts. This would be a temporary thing only however, because on the second run we would be good again. What do you think we should do?