New `query` request builder from `@warp-drive/utilities/json-api` does not serialize url params as before

[BoussonKarel]

We currently are using legacy store.query(modelName, queryParams) to do requests.

This creates a URL that correctly converts our query params:

const queryParams = {
  filter: {
    1: {
      field: "field1",
      value: "1",
    },
    2: {
      field: "field2",
      value: "null",
    },
  },
};

to ?filter[1][field]=field1&filter[1][value]=1&filter[2][field]=field2&filter[2][value]=null

However, the new query from @warp-drive/utilities/json-api does not serialize this nested object the same way, it generates a URL like this: ?filter%3D%5Bobject%20Object%5D

[BoussonKarel]

The new query has urlParamsSetting, but that's just for array, and the default comma is fine for arrays.

because we do want include to be like this, next to the object:

const queryParams = {
  filter: {
    1: {
      field: "field1",
      value: "1",
    },
    2: {
      field: "field2",
      value: "null",
    },
  },
  include: ['a', 'b']
};

--> ?filter[1][field]=field1&filter[1][value]=1&filter[2][field]=field2&filter[2][value]=null&include=a,b

[BoussonKarel]

The old code at

warp-drive/warp-drive-packages/legacy/src/adapter/-private/utils/serialize-query-params.ts

Line 22 in b3372e5

function buildParams<T extends PlainObject>(

has logic for serializing plain objects as object[key]

But it looks like the new code doesn't have any logic for this at all:

warp-drive/warp-drive-packages/utilities/src/index.ts

Line 416 in b3372e5

if (Array.isArray(value)) {

Is this intended, or should a PR be created to also add this logic?