refactor: reuse sort logic between Query and Aggregate

[singhankit001]

This PR refactors Aggregate.prototype.sort to reuse the sort argument parsing logic from Query.prototype.sort, addressing an existing TODO in lib/aggregate.js
I extracted the sort parsing logic into a new shared helper function castSort (located in lib/helpers/query/castSort.js) and updated both Query and Aggregate to use it.
This reduces code duplication and ensures consistent behavior between the two classes. As a side effect, Aggregate.prototype.sort now supports Map and Array arguments, matching the capabilities of Query.prototype.sort.

Examples

Aggregate.prototype.sort now supports Maps and Arrays of Arrays, which were previously only supported by Query.prototype.sort:

// Using a Map const map = new Map([['field', 'asc'], ['test', -1]]); await Model.aggregate().sort(map); // Now works! // Using an Array of Arrays await Model.aggregate().sort([['field', 'asc'], ['test', -1]]); // Now works

[singhankit001]

Hey @vkarpov15,
Please have a look at the PR and let me know if something needs to be changed.
Thank you

[singhankit001]

@AbdelrahmanHafez @hasezoey Can you please have a look at the PR and let me know if anything needs to be changed.
Thank you

[hasezoey]

Looks mostly good to me, though this includes a bunch of style changes which should not be part of this PR; please fix that.

[singhankit001]

@AbdelrahmanHafez @hasezoey Can you again run tests this time it would pass.

[hasezoey]

Looks good to me, though still contains some unrelated style changes (which arguably should be added to the lint, if possible).

[hasezoey]

Those are also unrelated style changes, though from what i can tell, it is not caught by the lint currently.

[vkarpov15]

Overall this looks reasonable, but please undo the unrelated style changes

[vkarpov15]

Please undo all the unrelated style changes