@query-doctor/core 0.2.2 → 0.2.4

package/dist/index.cjs

@@ -1222,7 +1222,31 @@ var _IndexOptimizer = class _IndexOptimizer {
   // TODO: this doesn't belong in the optimizer
   indexAlreadyExists(table, columns) {
     return this.existingIndexes.find(
-      (index) => index.index_type === "btree" && index.table_name === table && index.index_columns.length === columns.length && index.index_columns.every((c, i) => columns[i].column === c.name)
+      (index) => index.index_type === "btree" && index.table_name === table && index.index_columns.length === columns.length && index.index_columns.every((c, i) => {
+        if (columns[i].column !== c.name) {
+          return false;
+        }
+        if (columns[i].where) {
+          return false;
+        }
+        if (columns[i].sort) {
+          switch (columns[i].sort.dir) {
+            // Sorting is ASC by default in postgres
+            case "SORTBY_DEFAULT":
+            case "SORTBY_ASC":
+              if (c.order !== "ASC") {
+                return false;
+              }
+              break;
+            case "SORTBY_DESC":
+              if (c.order !== "DESC") {
+                return false;
+              }
+              break;
+          }
+        }
+        return true;
+      })
     );
   }
   /**
@@ -1422,7 +1446,7 @@ var _IndexOptimizer = class _IndexOptimizer {
   replaceUsedIndexesWithDefinition(explain, triedIndexes) {
     walkExplain(explain, (stage) => {
       const indexName = stage["Index Name"];
-      if (indexName) {
+      if (typeof indexName === "string") {
         const recommendation = triedIndexes.get(indexName);
         if (recommendation) {
           stage["Index Name"] = recommendation.definition;
@@ -1442,6 +1466,7 @@ function walkExplain(explain, f) {
       }
     }
   }
+  go(explain);
 }
 var RollbackError = class {
   constructor(value) {