fix(deps): update dependency json-2-csv to v5 (#4950)

* fix(deps): update dependency json-2-csv to v5

* update json2csv calls (no longer async)

---------

Co-authored-by: Matt Seddon <[email protected]>

Author: renovate[bot]

extension/package.json

@@ -1715,7 +1715,7 @@
     "appdirs": "1.1.0",
     "execa": "5.1.1",
     "fs-extra": "11.1.1",
-    "json-2-csv": "4.1.1",
+    "json-2-csv": "5.0.0",
     "json5": "2.2.3",
     "lodash.clonedeep": "4.5.0",
     "lodash.get": "4.4.2",

extension/src/fileSystem/index.test.ts

@@ -245,8 +245,8 @@ describe('writeJson', () => {
 })
 
 describe('writeCsv', () => {
-  it('should write csv into given file', async () => {
-    await writeCsv('file-name.csv', [
+  it('should write csv into given file', () => {
+    writeCsv('file-name.csv', [
       { nested: { string: 'string1' }, value: 3 },
       { nested: { string: 'string2' }, value: 4 },
       { nested: { string: 'string3' }, value: 6 }
@@ -260,8 +260,8 @@ describe('writeCsv', () => {
 })
 
 describe('writeTsv', () => {
-  it('should write tsv into given file', async () => {
-    await writeTsv('file-name.tsv', [
+  it('should write tsv into given file', () => {
+    writeTsv('file-name.tsv', [
       { nested: { string: 'string1' }, value: 3 },
       { nested: { string: 'string2' }, value: 4 },
       { nested: { string: 'string3' }, value: 6 }

extension/src/fileSystem/index.ts

@@ -384,19 +384,13 @@ export const writeJson = <
   return writeFile(path, json)
 }
 
-export const writeCsv = async (
-  path: string,
-  arr: Array<Record<string, unknown>>
-) => {
-  const csv = await json2csv(arr)
+export const writeCsv = (path: string, arr: Array<Record<string, unknown>>) => {
+  const csv = json2csv(arr)
   return writeFile(path, csv)
 }
 
-export const writeTsv = async (
-  path: string,
-  arr: Array<Record<string, unknown>>
-) => {
-  const csv = await json2csv(arr, { delimiter: { field: '\t' } })
+export const writeTsv = (path: string, arr: Array<Record<string, unknown>>) => {
+  const csv = json2csv(arr, { delimiter: { field: '\t' } })
   return writeFile(path, csv)
 }
 

extension/src/plots/model/index.ts

@@ -236,10 +236,7 @@ export class PlotsModel extends ModelWithPersistence {
   }
 
   public savePlotDataAsJson(filePath: string, plotId: string) {
-    void this.savePlotData(filePath, plotId, data => {
-      writeJson(filePath, data, true)
-      return Promise.resolve()
-    })
+    this.savePlotData(filePath, plotId, data => writeJson(filePath, data, true))
   }
 
   public savePlotDataAsCsv(filePath: string, plotId: string) {
@@ -508,10 +505,10 @@ export class PlotsModel extends ModelWithPersistence {
     return collectCustomPlotRawData(orderValue, experiments)
   }
 
-  private async savePlotData(
+  private savePlotData(
     filePath: string,
     plotId: string,
-    writeToFile: (rawData: Array<Record<string, unknown>>) => Promise<void>
+    writeToFile: (rawData: Array<Record<string, unknown>>) => void
   ) {
     const foundCustomPlot = this.customPlotsOrder.find(
       ({ metric, param }) => getCustomPlotId(metric, param) === plotId
@@ -522,7 +519,7 @@ export class PlotsModel extends ModelWithPersistence {
       : this.getSelectedTemplatePlotData(plotId)
 
     try {
-      await writeToFile(rawData)
+      writeToFile(rawData)
       void openFileInEditor(filePath)
     } catch {
       void Toast.showError('Cannot write to file')

yarn.lock

@@ -15595,10 +15595,10 @@ jsesc@~0.5.0:
   resolved "https://registry.yarnpkg.com/jsesc/-/jsesc-0.5.0.tgz#e7dee66e35d6fc16f710fe91d5cf69f70f08911d"
   integrity sha1-597mbjXW/Bb3EP6R1c9p9w8IkR0=
 
-json-2-csv@4.1.1:
-  version "4.1.1"
-  resolved "https://registry.yarnpkg.com/json-2-csv/-/json-2-csv-4.1.1.tgz#ebba9c7dd60a35675ab47435cb19e716249e5163"
-  integrity sha512-xqP/61jVw6hlW5a+JuffbfGneYxuKOIixdSjmLtBY/qeNgMIzTc+vLRkTkLwb+ZVLHMqmJz64brY9U34BvIHZA==
+json-2-csv@5.0.0:
+  version "5.0.0"
+  resolved "https://registry.yarnpkg.com/json-2-csv/-/json-2-csv-5.0.0.tgz#c8f8fc688a5c7460d48e01bce54c494f631b8bca"
+  integrity sha512-7+LmJmeO1UJqt8sqVYiSe5ufWGWPt1TwfdBSgCFLz66fq4jrX3K7+g8+J9gYc/Yp6VMfNhBSfZgaapBSt70IRQ==
   dependencies:
     deeks "3.0.2"
     doc-path "4.0.2"