string variable incorrectly stringified

[gregal]

Expected Behaviour

When passing a string variable to runPersistedQuery, variablesString should be formatted without underquoting it.

Example:

const variables = { slug: 'aaa' }
await aemHeadlessClient.runPersistedQuery(path, variables)

variablesString should finally be assigned:

';slug=aaa'

as it is under @adobe/[email protected].

Actual Behaviour

variablesString is assigned:

';slug="aaa"'

...with the extra double quotes. Apparently, a bug introduced in #41.

Reproduce Scenario (including but not limited to)

Steps to Reproduce

Platform and Version

@adobe/[email protected] and 3.3.1

Sample Code that illustrates the problem

Logs taken while reproducing problem

[cpilsworth]

I think this problem was introduced in #50 here
Its correct for an object

JSON.stringify({hello: "world"}) == '{"hello":"world"}'

but not for a simple string value

JSON.stringify("hello") == '"hello"'

[petergray-maker]

This is also affecting my project, it looks like the PR #53 is all good to be merged? Can it please be applied ?

Cheers