Merge pull request #385 from dolthub/taylor/sql-parser-up

utils: Up node-sql-parser, fix and add tests

Author: tbantle22

packages/utils/package.json

@@ -59,7 +59,7 @@
     "url": "https://github.com/dolthub/react-library/issues"
   },
   "dependencies": {
-    "node-sql-parser": "^5.3.5",
+    "node-sql-parser": "^5.3.10",
     "timeago.js": "^4.0.2"
   }
 }

packages/utils/src/__tests__/helpers/mutationExamples.ts

@@ -75,4 +75,5 @@ export const mutationExamples = [
   "SET GLOBAL slow_query_log = 1",
   "SET SESSION slow_query_log = 1",
   "FLUSH PRIVILEGES",
+  `REPLACE INTO dolt_docs VALUES ("README.md", "This is a README with a \\"quote\\".")`,
 ];

packages/utils/src/__tests__/parseSqlQuery.test.ts

@@ -537,18 +537,31 @@ function getParserCol(
       column: name,
       table: table ?? null,
       type: "column_ref",
+      // TODO: Remove type cast when this issue is fixed https://github.com/taozhi8833998/node-sql-parser/issues/2522
+      collate: null as any,
     },
     as: null,
     type: includeType ? "expr" : undefined,
   };
 }
 
+function getParserStarCol(): Column {
+  return {
+    expr: {
+      column: "*",
+      table: null,
+      type: "column_ref",
+    },
+    as: null,
+  };
+}
+
 describe("test getColumns", () => {
   const tests = [
     {
       desc: "select *",
       query: "select * from `test`",
-      expected: [getParserCol("*")],
+      expected: [getParserStarCol()],
     },
     {
       desc: "select one column",

packages/utils/src/parseSqlQuery.ts

@@ -336,6 +336,7 @@ export function isMutation(q?: string): boolean {
       lower.startsWith("revoke") ||
       lower.startsWith("grant") ||
       lower.startsWith("flush") ||
+      // lower.startsWith("replace into") ||
       (lower.startsWith("with") &&
         ((lower.includes("update") && lower.includes("set")) ||
           lower.includes("delete from")))

yarn.lock

@@ -2543,7 +2543,7 @@ __metadata:
     babel-jest: "npm:^30.0.2"
     eslint: "npm:^8.57.0"
     jest: "npm:^29.7.0"
-    node-sql-parser: "npm:^5.3.5"
+    node-sql-parser: "npm:^5.3.10"
     prettier: "npm:^3.6.2"
     rollup: "npm:^4.46.2"
     rollup-plugin-dts: "npm:^6.2.1"
@@ -12697,13 +12697,13 @@ __metadata:
   languageName: node
   linkType: hard
 
-"node-sql-parser@npm:^5.3.5":
-  version: 5.3.5
-  resolution: "node-sql-parser@npm:5.3.5"
+"node-sql-parser@npm:^5.3.10":
+  version: 5.3.10
+  resolution: "node-sql-parser@npm:5.3.10"
   dependencies:
     "@types/pegjs": "npm:^0.10.0"
     big-integer: "npm:^1.6.48"
-  checksum: 221c0e5d582adf9e87a4357cc437f6f66e925eaefa889f05b8e93375274b0246edbca11a673e7d44a29888ca0ec6005b5614ac08361b0e9171d11c66ce17e91a
+  checksum: a4c417a429d4d438548f858e62be091148a52256c39b419c0e0eb259af5feb6fa555928a98e2fe3a07a974a2ce065ea12df04ac18f5a01bfa918df32cebc6a2a
   languageName: node
   linkType: hard