event.params in log query event is not always a valid JSON

[unlight]

Bug description

event.params in log query event is not always a valid JSON

How to reproduce

1. git clone https://github.com/prisma/prisma-examples.git
2. cd prisma-examples\typescript\script
3. npm install
4. Change script.ts

const prisma = new PrismaClient({
  log: [
    {
      emit: 'event',
      level: 'query',
    },
  ],
})

prisma.$on('query', (event) => {
  console.log(event)
})

await prisma.post.create({
  data: {
    title: `"title"`,
  },
})

5. Run npx ts-node ./script.ts
6. Check string in params

{
  timestamp: 2022-03-20T13:45:23.604Z,
  query: 'INSERT INTO `main`.`Post` (`title`, `published`) VALUES (?,?) RETURNING id',
  params: '[""title"",false]', // <--- invalid json
}

Invalid escaping happens with:

1. String with double quotes
2. Dates

Expected behavior

event.params should be valid JSON

Prisma information

Valid JSON and therefore param substitution will be helpful in custom loggers for prisma

unlight/prisma-query-log#4

Environment & setup

node v16.13.2

Prisma Version

prisma                  : 3.11.0
@prisma/client          : 3.11.0
Current platform        : windows
Query Engine (Node-API) : libquery-engine b371888aaf8f51357c7457d836b86d12da91658b (at node_modules\@prisma\engines\query_engine-windows.dll.node)
Migration Engine        : migration-engine-cli b371888aaf8f51357c7457d836b86d12da91658b (at node_modules\@prisma\engines\migration-engine-windows.exe)
Introspection Engine    : introspection-core b371888aaf8f51357c7457d836b86d12da91658b (at node_modules\@prisma\engines\introspection-engine-windows.exe)
Format Binary           : prisma-fmt b371888aaf8f51357c7457d836b86d12da91658b (at node_modules\@prisma\engines\prisma-fmt-windows.exe)
Default Engines Hash    : b371888aaf8f51357c7457d836b86d12da91658b
Studio                  : 0.458.0

[terrierscript]

Workaround: I can parse invalid params as CSV ( this sample use papaparse )

import * as Papa from "papaparse"

const parseParam = (param: string) => {
  const removeBlacket = param
    .replace(/^\[/, "")
    .replace(/\]$/, "")
  const parsed = Papa.parse(removeBlacket, {
    dynamicTyping: true
  })
  return parsed.data[0]
}

[revero-doug]

you'll want to pass this option to Papa.parse as well: transform: a => a == "null" ? null : a or else you'll get instances of null as strings

[revero-doug]

update: in the absence of escaping, csv parsing does not generally work, either. whether papaparse or any other known workaround works for you will be highly dependent on the contents of your params